[12:15] <sabdf1> kiko: ping
[12:15] <kiko> sabdf1, yeees?
[12:15] <kiko> sabdf1, for how long are you going to be in wiesbaden?
[12:16] <kiko> sabdf1, yeees?
[12:16] <sabdf1> bugrit
[12:16] <kiko> hmmm!
[12:17] <kiko-zzz> man am I sleepy today
[12:17] <sabdfl> kiko-zzz: did you see my privmsg?
[12:18] <sabdfl> resent
[12:18] <LarstiQ> kiko-zzz: aren't you in brazil?
[12:18] <sabdfl> kiko-zzz: 2 days, by the looks of things
[12:18] <kiko-zzz> ah saw it now
[12:18] <kiko-zzz> sabdfl, okay. I arrive sunday
[12:18] <sabdfl> you arrive where sunday?
[12:19] <kiko-zzz> in the city with the complicated airport
[12:19] <kiko-zzz> LONDON
[12:27] <sabdfl> ah. will be great to have you here!
[12:27] <sabdfl> soyuz sprint?
[12:31] <kiko-zzz> sabdfl, yes! have you forgotten? next time I'll fly malcc down here if you are not excited too!
[12:41] <MilanZR> hey ppl
[12:42] <MilanZR> what's up?
[12:42] <kiko-zzz> going to bed
[12:42] <kiko-zzz> that's the latest craze! :)
[12:42] <kiko-zzz> LarstiQ, and yes, I am, but I'm tired!
[12:42] <LarstiQ> kiko-zzz: sure, but you always surprise me by going zzz before I do.
[09:00] <SteveA> morning
[09:09] <jamesh> lifeless: looks like the guile-gnome developers are looking at switching to bzr
[09:10] <jamesh> (they were previously using arch)
[09:10] <lifeless> cool
[09:24] <carlos> stub: morning
[09:36] <stub> carlos: Morning
[09:36] <carlos> stub: https://sodium.ubuntu.com/~andrew/paste/filekQo3Cb.html
[09:37] <carlos> stub: I reverted the temporary table solution and seems like it's taking only 20 minutes more, but it works
[09:38] <stub> So 3.5 hours if we don't rebuild the distrorelease stats until after (I've already commented that bit out in rocketfuel)
[09:38] <stub> The results look good as far as you can tell?
[09:38] <carlos> stub: the temporary table trick added a lot of complexity to the queries and I was not able to find the problem so I timed the initial solution and got that output
[09:39] <jamesh> stub: I noticed one more project that had been renamed to "*-project" because of an inactive product of the same name: mercury-project
[09:39] <carlos> stub: well, I went to sleep before the process finished and I'm going to run now a test as kiko asked me that will get a .po file output from dapper and from edgy and get a diff
[09:40] <carlos> stub: to validate that the migration didn't miss anything
[09:40] <stub> yup.
[09:40] <carlos> I'm going to execute it right now, so I don't think we can do the scheduled migration until two or three hours
[09:40] <carlos> but with that, we will be completely sure that the migration worked as it should
[09:42] <stub> We won't be able to run the migration today as we need to notify the community of the downtime and coordinate with the admins since they need to do hardware stuff at the same time.
[09:45] <carlos> I see
[09:46] <carlos> well, at least I will leave everything in place...
[09:46] <carlos> stub: any chance to connect as the launchpad user to jubany's database?
[09:46] <stub> Sure
[09:46] <carlos> I need more permissions to do the check
[09:46] <carlos> I'm using the launchpad account
[09:46] <carlos> thanks
[09:48] <stub> carlos: done
[09:50] <carlos> thanks
[09:53] <carlos> stub: ok, so the check process started
[09:54] <carlos> if everything is ok, we can be completely sure that the migration works perfectly
[09:54] <carlos> stub: btw, I guess the Breezy -> Dapper migration could be done without shutting down launchpad, right?
[09:55] <carlos> stub: I think it's fast enough
[09:55] <stub> Not really - it will still lock tables for 10 or 15 minutes. Not worth the hassle of trying to do it live.
[09:55] <carlos> ok
[10:00] <BjornT> stub: do you have time to help me with an sql query?
[10:00] <stub> BjornT: Sure
[10:01] <BjornT> stub: see https://sodium.ubuntu.com/~andrew/paste/filef51SCf.html
[10:02] <BjornT> stub: basically what i want is a query which is more efficient than issuing the two separate queries, combining them in python
[10:02] <BjornT> stub: what i'm looking for is to get all the tags used by a product/distribution, together with the number of open bugs for each tag
[10:05] <siretart> hi. some time ago, there has been a bug import from sourceforge regarding the python project.
[10:05] <siretart> does this offer apply to other sourceforge projects as well?
[10:06] <danilos> siretart: it sure does
[10:06] <jamesh> siretart: I suppose so.  The import code I've written isn't specific to Python
[10:09] <siretart> ok. it wasn't really mentioned on the MaloneHighlights wiki page. wanted to be sure
[10:11] <jamesh> BjornT: perhaps COUNT(BugTask.status in (...))?
[10:13] <danilos> siretart: it's probably not yet announced, but I guess it will be at some point
[10:14] <jamesh> siretart: at the moment we've been focusing on making it work for Python, but should be able to migrate other projects afterwards.
[10:15] <BjornT> jamesh: doesn't seem to work. when i try that, i get a count of 1 for 'doc', when it should be 0.
[10:18] <siretart> jamesh: ah, ok. so I'll wait for the announcment before I propose the project I have in mind to switch to lp
[10:18] <siretart> btw, has python decided yet if they switch?
[10:19] <jamesh> BjornT: okay.  Probably SUM(CASE WHEN BugTask.status in (...) THEN 1 ELSE 0 END) would do the trick
[10:19] <jamesh> BjornT: not sure how well it'd perform
[10:19] <jamesh> siretart: I think they've definitely decided to switch away from SF.  They haven't yet decided what to switch to
[10:20] <jamesh> siretart: there are four proposals to compare
[10:21] <jamesh> BjornT: alternatively, COUNT(NULLIF(BugTask.status in (...), FALSE))
[10:21] <carlos> danilos: dude, I had to revert the use of a temporary table for translations migration
[10:22] <carlos> danilos: the whole process takes only 30 minutes more without it and it's much more easy to understand and fix...
[10:24] <BjornT> jamesh: yeah, both seems to work. stub, would any of the alternatives proposed by jamesh perform better than the current two queries?
[10:25] <danilos> carlos: well, if that's what you want to do, sure, go ahead
[10:25] <danilos> carlos: btw, can't we do it in two steps as well? first breezy -> dapper migrations
[10:25] <danilos> carlos: and then when that lands on production/staging, try dapper->edgy
[10:25] <carlos> danilos: I did it yesterday night and timed it because I was not able to detect the error
[10:26] <carlos> it's not yet merged
[10:26] <danilos> carlos: so, no errors without temp table?
[10:26] <carlos> right
[10:26] <carlos> no errors
[10:26] <danilos> carlos: then sure, lets get it over with even if it takes longer
[10:27] <danilos> carlos: btw, what time are we talking about?
[10:27] <carlos> 3 hours and a half for both process
[10:27] <danilos> on production?
[10:27] <carlos> danilos: I talked with stuart about running it in two steps
[10:27] <carlos> danilos: yeah, on production
[10:27] <danilos> wow, that's longer than expected
[10:28] <danilos> oh, you're talking about 3.5h for both of them together?
[10:28] <carlos> danilos: but he told me that breezy -> dapper requires also to shutdown launchpad
[10:28] <carlos> danilos: yeah
[10:28] <carlos> danilos: https://sodium.ubuntu.com/~andrew/paste/filekQo3Cb.html
[10:28] <carlos> danilos: the final run will not execute the distrorelease statistics update
[10:29] <danilos> carlos: yeah, so it will be shorter?
[10:29] <carlos> danilos: we think it would take around 3 hours and a half once without that part
[10:29] <carlos> those statistics are updated every day anyway
[10:30] <danilos> carlos: ah, ok
[10:30] <danilos> carlos: sounds fine; any idea about when it will happen?
[10:30] <carlos> in fact...
[10:31] <carlos> stub, danilos: Dapper -> Edgy doesn't need the pofile statistics updates, that's 40 minutes less 
[10:31] <danilos> carlos: pofile? that takes like a second on each run
[10:31] <carlos> danilos: not today, seems like we need to coordinate with admins to do some admin stuff at the same time we do the copy
[10:31] <danilos> carlos: or distrorelease
[10:31] <danilos> ?
[10:31] <carlos> 02:01:11 INFO    Filling POSelection table...
[10:31] <carlos> 02:42:48 INFO    Updating POFile's statistics
[10:32] <danilos> yeah, that's poselection filling, no?
[10:32] <carlos> oh right
[10:32] <carlos> I misread the log
[10:32] <carlos> ;-)
[10:32] <danilos> :)
[10:32] <carlos> please, ignore me ;-)
[10:32] <stub> BjornT: https://sodium.ubuntu.com/~andrew/paste/fileFusYcu.html
[10:32] <danilos> nah, you're just starting to get overly optimistic with this :P
[10:33] <carlos> danilos: well, I guess the fact that I didn't sleep enough today is affecting me...
[10:33] <danilos> carlos: ah well :)
[10:35] <BjornT> stub: thanks
[10:36] <stub> BjornT: https://sodium.ubuntu.com/~andrew/paste/fileI8mq4T.html is better
[10:37] <jamesh> stub: will that double count bugs with multiple tasks?
[10:38] <stub> Can't have a bug with two tasks on the same product, and we are filtering by product
[10:38] <jamesh> good point
[10:38] <jamesh> til we have tasks on product series, I guess :)
[11:48] <poningru> is staging.ubuntu.com working right now?
[11:57] <carlos> poningru: it is, but the server is a bit overloaded atm
[11:57] <carlos> so it's being slow
[11:58] <poningru> ah ok thanks
[11:59] <poningru> yeah the proxy gave me a 502 proxy error
[12:30] <lucasvo> what's staging.ubuntu.com?
[12:31] <jamesh> lucasvo: staging.launchpad.net
[12:31] <lucasvo> jamesh: and what is it?
[12:31] <lucasvo> it looks normal to mee. 
[12:32] <jamesh> lucasvo: a server running with a copy of the production database and the current version of the LP code
[12:32] <lucasvo> oh, ok
[12:32] <jamesh> (as opposed to the production server which gets weekly code rollouts)
[12:37] <carlos> danilos: please, could you review and add anything you think I missed from https://launchpad.canonical.com/EdgyTranslations ?
[12:50] <Ubugtu> New bug reported: Malone bug 56792 in rosetta "request to translate variables" [Untriaged,Unconfirmed]  https://launchpad.net/bugs/56792
[12:56] <danilos> carlos: sure
[12:57] <carlos> danilos: thanks
[01:18] <heno> jamesh: ping?
[01:18] <jamesh> heno: pong
[01:19] <heno> jamesh: good blog entry on shared bzr hosting in lp, thanks :)
[01:19] <heno> wish I had read it before doing this https://launchpad.net/people/henrik/+branch/onboard/main
[01:20] <heno> I think I left of --create-prefix so it's still a private branch
[01:20] <jamesh> heno: --create-prefix isn't what makes it a private vs. team branch
[01:20] <jamesh> heno: it is the directory you upload it to on sftp://bazaar.launchpad.net
[01:21] <heno> I now have a total of two branches under ~henrik at https://launchpad.net/products/onboard
[01:21] <jamesh> if you create branches under ~henrik/ then they are personal.  If you create them under ~teamname they are team branches
[01:21] <LarstiQ> heno: you can use that as your personal branch though.
[01:21] <heno> can you tell if the 'main' one work for vthe team?
[01:21] <jamesh> heno: it is possible to change a branch'
[01:21] <jamesh> s ownership, but I don't think it is currently exposed to normal users
[01:21] <jamesh> one of the LP admins could do it for you though
[01:22] <jamesh> (we will expose it to branch owners in future)
[01:22] <heno> jamesh: cool, who should I ask?
[01:22] <jamesh> I am not sure who is best to annoy at this time of day.  Stub could do it if he's around
[01:23] <heno> does LP have an RT queue?
[01:23] <jamesh> kiko would be able to do it later on
[01:23] <LarstiQ> afaik, heno already has a team shared branch though?
[01:23] <LarstiQ> heno: you could add me to the team, and I could tell you if I see it.
[01:23] <jamesh> we've got a support tracker
[01:23] <heno> LarstiQ: yes, I'll do that. see if you can upload
[01:24] <jamesh> LarstiQ: doesn't look like it.  Both branches are registered to him
[01:24] <heno> LarstiQ: what is your LP username?
[01:24] <LarstiQ> heno: larstiq
[01:24] <lifeless> jamesh: whats up ?
[01:25] <jamesh> lifeless: heno wants the registrant for https://launchpad.net/people/henrik/+branch/onboard/main changed to the team "onboard"
[01:25] <jamesh> heno: ^^ is that correct?
[01:25] <heno> jamesh: yes please
[01:25] <heno> and the other one can be deleted
[01:25] <LarstiQ> jamesh: I see
[01:25] <heno> if that is possible
[01:25] <lifeless> we dont do deletions
[01:26] <heno> ok
[01:26] <lifeless> there you go
[01:26] <lifeless> https://launchpad.net/people/onboard/+branch/onboard/main
[01:27] <jamesh> heno: so that's changed the SFTP URL you need to push to
[01:27] <heno> lifeless: thanks!
[01:27] <jamesh> heno: but other people on the team will be able to commit too
[01:27] <heno> jamesh: to bla-bla/~onboard/bla right?
[01:27] <jamesh> heno: ftp://bazaar.launchpad.net/~onboard/onboard/main
[01:27] <jamesh> sftp, even
[01:27] <heno> jamesh: yes that's what we want
[01:28] <heno> cool, looks good
[01:28] <heno> LarstiQ: want to test it (I've added you to the team)?
[01:28] <jamesh> heno: if you were following the instructions in my blog article, you can issue "bzr bind sftp://bazaar.launchpad.net/~onboard/onboard/main" to convert your local branch to a checkout of the team branch
[01:29] <heno> jamesh: if I were following the instructions in your blog I would probably not have messed it up ;p
[01:30] <heno> jamesh: I'm happy to just keep my own branch local
[01:30] <Nafallo> jamesh: do you have an URL to that? I'd like to read it :-)
[01:30] <heno> we have a small team so we don't need several public branches
[01:30] <heno> Nafallo: http://blogs.gnome.org/view/jamesh/2006/08/17/1
[01:30] <Nafallo> thanks heno :-)
[01:32] <lifeless> heno you might like to set a branch summary etc
[01:32] <lifeless> just click on branch details in launchpad
[01:32] <jamesh> heno: if you have other people committing to the "main" branch, it would be a good idea to use a checkout for merging changes from your local branch
[01:32] <LarstiQ> heno: yes
[01:32] <heno> lifeless: ok, will do
[01:33] <heno> jamesh: right, I should read up a bit more on the bzr site about checkouts
[01:33] <LarstiQ> I'm not seeing the branch yet
[01:34] <jamesh> LarstiQ: are you trying to check it out via sftp or http?
[01:34] <LarstiQ> jamesh: sftp
[01:34] <jamesh> LarstiQ: it should be available as soon as you were added to the team
[01:35] <LarstiQ> jamesh: I can reach ~onboard, but that is emptyu
[01:36] <jamesh> that's weird
[01:36] <heno> I have the same problem
[01:37] <heno> bzr checkout sftp://bazaar.launchpad.net/~onboard/onboard/main
[01:37] <heno> should work right?
[01:37] <LarstiQ> jamesh: any idea how much of a difference between mirrored and hosted branches there is?
[01:38] <jamesh> oh.  this wasn't a push branch to start with?
[01:39] <heno> no it was mirrored in from my webspace
[01:39] <jamesh> LarstiQ: mirrored branches do not show up on the SFTP server
[01:40] <jamesh> heno: okay.  You'll need to push a branch to sftp://bazaar.launchpad.net/~onboard/onboard/$SOMETHING
[01:40] <jamesh> if you want the branch to be called main, then you'll need to rename the existing "main" branch
[01:41] <heno> jamesh: where something is _different_ from 'main'?
[01:41] <heno> which I cannot do myself right?
[01:41] <LarstiQ> jamesh: what happens if you push to the sftp server in their location?
[01:41] <jamesh> doesn't look like you can change the name as a normal user at the moment either.
[01:42] <LarstiQ> ehm, I thought you could
[01:42] <heno> so can we change the names of the current branches to void1 and void2 ?
[01:42] <LarstiQ> hmmno, no access 
[01:42] <jamesh> LarstiQ: the sftp server won't let you push to a name taken by an existing mirrored branch
[01:42] <heno> they aren't of any use I'm affraid
[01:42] <LarstiQ> jamesh: so how does one convert from the one to the other?
[01:43] <jamesh> lifeless: would you be able to rename https://launchpad.net/people/onboard/+branch/onboard/main to something other than "main"? (maybe old-main)
[01:43] <jamesh> LarstiQ: at the moment you don't.
[01:43] <LarstiQ> jamesh: hmm.
[01:43] <LarstiQ> jamesh: people frequently run into issues in this area
[01:43] <jamesh> LarstiQ: I'm not saying it isn't a problem.
[01:43] <jamesh> we need to improve things in this area
[01:46] <lifeless> jamesh: ok
[01:46] <lifeless> ywhy ?
[01:47] <lifeless> why not push --overwrite ?
[01:49] <stub> How log does a bzr landing take roughly?
[01:49] <lifeless> landing ?
[01:49] <jamesh> lifeless: the branch was a pull branch.  You can't sftp push to a pull branch
[01:50] <jamesh> and they want to use that name
[01:50] <lifeless> ah
[01:50] <lifeless> true. we need a knob for that
[01:50] <heno> well, I'm not too worried about the name
[01:50] <heno> just a working repo, and ideally tidy up the loose ends a bit
[01:51] <lifeless> renamed
[01:52] <heno> lifeless: thanks
[01:53] <LarstiQ> heno: ok, time to push your branch under ~onboard
[02:00] <heno> LarstiQ: done. I got '0 revisions uploaded' though
[02:00] <heno> but it appears on LP now
[02:00] <jamesh> heno: I think that's a bug in the version of bzr.  It should have worked still
[02:01] <LarstiQ> heno: that is a known bug in bzr 0.8
[02:01] <LarstiQ> heno: but that should really be fixed in 0.9?
[02:01] <heno> jamesh: ok. The web page doesn't show any revision either, but if you say it probably works then cool
[02:01] <heno> LarstiQ: can you just add a comment to a file, commit and push?
[02:02] <jamesh> heno: there is a delay between pushing to sftp and the branch being published via HTTP and the revision info being scanned
[02:02] <LarstiQ> sure
[02:02] <heno> so we can test it
[02:02] <jamesh> heno: but the branch is available via SFTP to other team members immediately
[02:04] <LarstiQ> heno: done, go ahead and pull
[02:05] <heno> LarstiQ: \o/ works :)
[02:05] <heno> Thanks everyone!
[02:06] <LarstiQ> heno: great, now you can get rid of that revision :)
[02:06] <jose__> hello, anyone here?
[02:06] <heno> right, so I need to find the command for rolling back ...
[02:06] <LarstiQ> heno: bzr uncommit
[02:06] <LarstiQ> heno: and then bzr push --overwrite
[02:06] <heno> thx, ok
[02:07] <LarstiQ> heno: after that, you can throw me out of the team again
[02:07] <heno> ok :)
[02:07] <jose__> please i am new to launchpad. i would like someone to explain relationship between an official translation from l10n.kde.org team and the team registered at launchpad
[02:08] <LarstiQ> jose__: I don't know what the kde situation is, but in general, a product on launchpad can specify wether they use rosetta (translation part of lp) at all or not
[02:09] <LarstiQ> jose__: if they do, then supposedly they are the same team or have something set up for merging
[02:10] <heno> LarstiQ: done. you are still listed as an inactive member though. I think you have to 'leave' the team yourself
[02:11] <jose__> well, i am trying to set up a slovak translation team at l10n.kde.org but now the problem appeared that a certain apps (e. g. akregator) is both translated at KDE svn and launchpad
[02:11] <LarstiQ> heno: will do
[02:12] <jose__> so the translators are asking - where should we post our translations..
[02:12] <LarstiQ> heno: hmm, it said I'm not an active member so I don't have to leave it.
[02:13] <LarstiQ> jose__: I'm just a user, you'll probably want to talk to carlos or jordi
[02:13] <heno> LarstiQ: heh, would you prefer I reactivate you so you can leave properly?
[02:13] <LarstiQ> carlos, jordi: can you help jose?
[02:13] <jose__> and if someone do a translation on launchpad. will it make it back to kde svn? if so. whenabout
[02:13] <jose__> LarstiQ: are they around?
[02:13] <LarstiQ> heno: we can try that, see what happens then :)
[02:13] <carlos> jose__: no, we don't have a way to send translations back automatically
[02:14] <carlos> is a duty of the translators
[02:14] <LarstiQ> jose__: apparently, yes :)
[02:14] <heno> LarstiQ: done
[02:14] <LarstiQ> heno: done.
[02:14] <jose__> carlos: then for kde programs the right way to commit translations is using the kde svn system, right?
[02:15] <heno> LarstiQ: same result
[02:15] <heno> So I guess 'LP never deletes anything' is correct :)
[02:15] <LarstiQ> heno: fwiw, I'm not seeing myself anymore on the people/onboard page
[02:15] <carlos> jose__: if you just one to translate for KDE, yes
[02:16] <LarstiQ> heno: I also can't access ~onboard on the sftp server anymore
[02:16] <carlos> jose__: if you want to translate the specific translation we have in Ubuntu, would be really good if you also upload your changes in Rosetta
[02:17] <heno> LarstiQ: I still see you at: https://launchpad.net/people/onboard/+members
[02:17] <heno> in two different browsers, logged in or not
[02:17] <jose__> means that translation in ubuntu and in the rest of distributions (using official KDE translation) may live a completely separate lives, doesnt it?
[02:17] <heno> and after Ctrl+Shift refresh
[02:18] <jordi> jose__: yes, but we try to encourage people to collaborate with upstream KDE/GNOME, etc
[02:18] <LarstiQ> heno: ah, I see.
[02:18] <carlos> jose__: well, we have some problems on it, yes, but we are working on ways to prevent that and to diverge as less as possible
[02:19] <LarstiQ> heno: cool, thanks for pointing that out :)
[02:19] <LarstiQ> jamesh: know anything about the people part of launchpad?
[02:20] <LarstiQ> heno: I think you're right that lp will never forget this.
[02:20] <jose__> carlos: could you please send me the whole discussion (from the point i started to ask
[02:20] <carlos> ?
[02:20] <jose__> on my email. i found out my irssi does not save configuration and a few lines are lost
[02:20] <jose__> damn
[02:20] <carlos> sure
[02:21] <jose__> thank you
[02:21] <jose__> jose1711 gmail com please
[02:21] <jose__> copy-paste is just fine
[02:22] <carlos> jose__: sent
[02:23] <jose__> thank you very much
[02:24] <jose__> please what is your relationship to launchpad? administrator?
[02:27] <carlos> jose__: I'm one of the Rosetta developers
[02:28] <jose__> carlos: thx
[02:28] <carlos> you are welcome
[02:49] <malcc> I'd like to nominate the following line of code from distribution.py for an award of some appropriate kind:
[02:49] <malcc>         suffixes = [suffix for suffix, ignored in suffixpocket.items()] 
[02:52] <LarstiQ> instead of just keys() eh?
[03:13] <carlos> danilos: hi, did you have time to read https://launchpad.canonical.com/EdgyTranslations ?
[03:13] <carlos> I would like to send it to kiko
[03:13] <carlos> before I leave today
[03:21] <danilos> carlos: yeah, that's basically what we discussed earlier
[03:21] <danilos> carlos: so, I have no objections, and don't have any suggestions off the top of my head
[03:21] <carlos> Oh, really?
[03:21] <carlos> O:-)
[03:22] <matsubara> carlos, danilos: could you take a look at this: https://launchpad.net/bugs/56792 and confirm or reject appropriately? I don't think it's a bug but better to confirm with you.
[03:22] <Ubugtu> Malone bug 56792 in rosetta "request to translate variables" [Untriaged,Unconfirmed]  
[03:23] <danilos> matsubara: I am not sure what this is about 
[03:24] <matsubara> the translation string contains only variables, so there's nothing to translate.
[03:25] <carlos> rejected
[03:25] <carlos> If that's a bug, it's in the application. But usually, as there are more than one variable, I guess they are set to translate because depending on the language the order could change.
[03:25] <danilos> matsubara: well, it's not a bug, it's sometimes required, and it's up to template author (application author) to decide if it is to be included for translation or not
[03:26] <matsubara> danilos, carlos: ok, thank you guys.
[03:26] <danilos> one should probably ask for translator comment from application author
[03:26] <carlos> danilos: yeah, I guess that's actually the bug
[03:26] <danilos> carlos: well, I noted that in bugreport
[03:27] <carlos> danilos: ok, thanks
[03:56] <heno> Another question about https://launchpad.net/people/onboard/+branch/onboard/main
[03:57] <heno> Does someone have to be a member of the 'onboard' team to branch or checkout?
[03:58] <LarstiQ> not via http no
[03:58] <LarstiQ> heno: via http, anyone can do that
[03:58] <LarstiQ> it is just the sftp location that requires team membership
[03:58] <heno> ah, good point
[04:00] <salgado> BjornT, ping?
[04:01] <BjornT> salgado: pong
[04:01] <salgado> hi BjornT.  about the email you just sent to me...
[04:02] <salgado> that docstring is wrong. the main use case I have is checking if the request's status is APPROVED
[04:02] <salgado> what do you think of having an isApproved() and another isApprovedOrShipped()?
[04:02] <BjornT> salgado: why don't you check the status directly instead?
[04:03] <BjornT> salgado: i think isApproved is a confusing name for something that does a simple status comparison
[04:04] <salgado> mainly because I do that in /a lot/ of different places, and using a method makes things easier in this case
[04:06] <salgado> do you really think we'd make things clearer by checking the status directly in all these places?
[04:07] <BjornT> salgado: well, the name doesn't make things clearer, though :-/. statusIsApproved would be clearer in that case.
[04:08] <BjornT> salgado: i think checking the status would be clearer, since shipped orders are also approved. but i haven't looked at all the call sites (for the one you added, using isApproved() or isShipped() was confusing for me)
[04:09] <BjornT> there it would have been much clearer checking the status directly.
[04:11] <kiko> hello old men and women
[04:11] <LarstiQ> hello young grasshopper.
[04:11] <kiko> LarstiQ, how are you doing today? did you fix that bug yet? :)
[04:12] <LarstiQ> kiko: no, fighting with paramiko .deb first
[04:12] <kiko> LarstiQ, I know some guys that know something about debs
[04:13] <salgado> BjornT, I really don't like the idea of checking the status directly... how about the statusIsApproved() idea you gave?
[04:13] <LarstiQ> kiko: heh :)
[04:17] <BjornT> salgado: well, would something break if you made isApproved() return True if status is SHIPPED? if it does, then i guess statusIsApproved would be acceptable, at least it makes it clear what the method is checking.
[04:17] <salgado> BjornT, yes, things will break if I do that
[04:19] <salgado> BjornT, but if I rename isApproved() I'd have to rename the other methods that check for other statuses --it wouldn't make sense to have some of them called statusIsFoo and others called isFoo
[04:24] <BjornT> salgado: right. and since this is existing code, maybe we shouldn't change it too much. so, an alternative would be to correct the docstrings, to say what the methods really do. then also improve the comment before the 'isApproved() or isShipped()' part.
[04:28] <salgado> BjornT, yeah, I wasn't planning to leave that docstring as it is, definitely.  anyway, I'll change the docstrings and the comment.  will send the diff to you quickly
[04:34] <salgado> BjornT, hmm, apparently I forgot to commit one small comment Steve asked me to add on webapp/publisher.py.  this is the diff: https://sodium.ubuntu.com/~andrew/paste/fileGUoApq.html
[04:37] <BjornT> salgado: ok, r=me
[04:37] <salgado> BjornT, cool, thank you
[05:40] <salgado> BjornT, still around?
[05:40] <BjornT> salgado: yeah
[05:42] <salgado> BjornT, so, there's one issue with these changes you've reviewed that I haven't noticed before.  I'm using canonical_url(shipit_request) in a script, which means I don't have a browser request and thus I can't generate the URL
[05:43] <salgado> I guess I'll have to hardcode the root url in this case?
[05:43] <salgado> well, I can use the shipit root_url from the config file
[05:44] <salgado> yeah, I don't think that's too evil... what do you think?
[05:46] <Ubugtu> New bug reported: Malone bug 56820 in rosetta "Po export script is not robust enough" [Medium,Confirmed]  https://launchpad.net/bugs/56820
[05:46] <carlos> enjoy your weekend!!!
[05:46] <carlos> danilos: btw, edgy opening testing worked perfectly
[05:47] <kiko-fud> danilos, good news finally did you notice!
[05:48] <kiko-fud> salgado, look at this message in bug 56820:
[05:48] <Ubugtu> Malone bug 56820 in rosetta "Po export script is not robust enough" [Medium,Confirmed]  https://launchpad.net/bugs/56820
[05:48] <kiko-fud> RuntimeError: Non-merged person u'enver-altin-frontsite' has no email addresses!
[05:49] <salgado> that's not good. :/
[05:50] <BjornT> salgado: can't you fall back on some rooturl if request is None? (instead of asserting that a request has to be used for shipit)
[05:52] <salgado> BjornT, yes, that would be an option.  it sounds good to me, I think
[05:53] <salgado> SteveA, around?
[06:00] <salgado> BjornT, I think it'd be reasonable to fail with an AssertionError if the request is not None and neither provide one of the shipit layers, because that's not supposed to happen. what do you think?
[06:07] <BjornT> salgado: yeah, it seems reasonable to me.
[06:09] <salgado> BjornT, does https://sodium.ubuntu.com/~andrew/paste/filePe3GXF.html look okay?
[06:11] <BjornT> salgado: looks good to me
[07:27] <danilo_> kiko-fud: if you have the time, please do another review of bug-2237; thanks and enjoy the weekend :)
[07:30] <Ubugtu> New bug reported: Malone bug 56830 in soyuz "cron.germinate breaking cron.daily" [High,Confirmed]  https://launchpad.net/bugs/56830
[07:35] <Ubugtu> New bug reported: Malone bug 56831 in launchpad "Import key at +editpgpkeys page needs better fingerprint validation" [Low,Confirmed]  https://launchpad.net/bugs/56831
[07:49] <sabdfl> kiko-fud: back?
[07:50] <sabdfl> sivang: https://staging.launchpad.net/distros/ubuntu/+specs (yes it's there :-))
[09:53] <salgado> kiko, I found what's causing that issue you showed me yesterday
[09:54] <kiko> salgado, ah?
[09:54] <salgado> kiko, the shipit one
[09:54] <kiko> yes yes
[10:00] <Ubugtu> New bug reported: Malone bug 56839 in malone ""Affects" table on bug page should have "lesser" font size" [Low,Unconfirmed]  https://launchpad.net/bugs/56839
[10:00] <Ubugtu> New bug reported: Malone bug 56840 in malone ""Also Affects" should be in actions menu" [Low,Unconfirmed]  https://launchpad.net/bugs/56840
[10:05] <Ubugtu> New bug reported: Malone bug 56841 in malone ""See all" bug trackers link shouldn't use "+" icon" [Untriaged,Unconfirmed]  https://launchpad.net/bugs/56841
[10:07] <salgado> kiko, it's quite simple, but I wrote some tests to make sure we don't have any other regressions on this area.  can you review it for me?
[10:07] <LarstiQ> the 'new bug reported' bit is a bit long
[10:08] <bradb> kiko: ping?
[10:08] <LarstiQ> Seveas: could you reduct that? Perhaps to 'New Malone bug ###:'
[10:08] <LarstiQ> though that looks like it is a bug on the malone product
[10:08] <LarstiQ> ah, I see
[10:08] <kiko> oi bradb 
[10:08] <kiko> salgado, uhh sure
[10:09] <salgado> kiko, https://sodium.ubuntu.com/~andrew/paste/file3lcvJl.html
[10:09] <bradb> kiko: we have launchpad.Driver security checkers on IDistroRelease and IProductSeries, but not IDistribution or IProduct.
[10:09] <bradb> i'm thinking of maybe defining and IHasDrivers interface, and implementing the security checker on that?
[10:10] <bradb> (then all four interfaces could inherit from it, and we could write just one security checker)
[10:10] <bradb> right now two different .Driver checkers have the same code
[10:10] <kiko> yeeeah.
[10:10] <kiko> sure.
[10:10] <Ubugtu> New bug reported: Malone bug 56842 in malone "Bug listings should begin with Importance words, not icon" [Low,Unconfirmed]  https://launchpad.net/bugs/56842
[10:11] <bradb> kiko: ok. mind if i land this as a small separate patch?
[10:12] <kiko> bradb, before the main patch? not at all
[10:12] <bradb> cool, changing now
[10:15] <Seveas> LarstiQ, yeah, I could do that
[10:16] <kiko> salgado, ah. we were checking too early. :)
[10:16] <LarstiQ> Seveas: I realized one of the malones was a product though
[10:16] <kiko> salgado, r=kiko, and ask stub to cherry-pick please.
[10:16] <salgado> kiko, exactly
[10:16] <salgado> kiko, thanks!
[10:16] <Seveas> LarstiQ, 'reported' is a bit unneccessary and the first Malone is a constant (it's not like it reports bugd from other trackers)
[10:17] <LarstiQ> Seveas: but it might do that in the future?
[10:17] <LarstiQ> or well, handle that whenever it happens
[10:18] <Seveas> it might, but I don't see it happening
[10:18] <Seveas> it's not like you'd want new gnome bugzilla bugs in here
[10:18] <Seveas> (and it currently supports only malone anyway)
[10:19] <LarstiQ> is it purely meant for #launchpad then?
[10:19] <Seveas> and #ubuntu-bugs
[10:19] <Seveas> bzr also uses malone as tracker
[10:19] <LarstiQ> Seveas: yes, as native one. But there are more trackers where bzr bugs get filed.
[10:19] <Seveas> good point
[10:25] <Ubugtu> New bug: #56845 in malone "Bug listing sort order should allow sorting by In Progress first" [Untriaged,Unconfirmed]  http://launchpad.net/bugs/56845
[10:25] <Seveas> LarstiQ, --^
[10:25] <LarstiQ> Seveas: looks good
[10:26] <LarstiQ> I'd probably move the colon from bug: to malone:, but no big deal
[10:27] <LarstiQ> Seveas: you already saw the request to also have the submitter in there, right?
[10:27] <Seveas> yes
[10:27] <LarstiQ> k
[10:27] <Seveas> dunno if it's filed already
[10:27] <Seveas> anyway, have to finish working on other parts of the bot now so that has to wait
[10:33] <LarstiQ> kiko: I don't suppose you have one of those failed-in-transmission branches handy somewhere?
[10:33] <kiko> LarstiQ, I don't right now, no. it's easy to create one though..
[10:34] <LarstiQ> kiko: I'll do some more testing first
[10:39] <kiko> salgado, https://sodium.ubuntu.com/~andrew/paste/filemEuvA4.html
[10:39] <kiko> salgado, do you know how those inline doctests in browser/bugtask.py work? how do you run them?
[10:40] <salgado> kiko, you have to create a test_foo.py file somwhere that runs them
[10:40] <kiko> salgado, oh. that sucks. :-(
[10:41] <salgado> indeed
[10:42] <kiko> and then just run it as usual, test_person?
[10:42] <salgado> kiko, don't we need to match lower case a-f on "re.match(r"^[\dA-F] +$"" too?
[10:42] <kiko> salgado, I expect the code sending it in has already done that. 
[10:43] <salgado> I don't know if that works, what I usually do is canonical.launchpad.ftests.test_foo
[10:43] <salgado> I mean ./test.py canonical.launchpad.ftests.test_foo
[10:43] <salgado> (or whatever the path to the test is)
[10:43] <salgado> I need to eat
[10:44] <kiko> sure
[10:46] <WebMaven> Howdy.
[10:47] <WebMaven> sabdfl: is Canonical going to have any presence at the Plone Conference?
[10:47] <sabdfl> WebMaven: i don't believe so
[10:47] <WebMaven> sabdfl: OK.
[10:48] <WebMaven> sabdfl: I was asking because of the Zope3 sprints there.
[10:49] <sabdfl> didn't know they were happening! stevea might have an opinion on that
[10:49] <WebMaven> Hmm.
[10:50] <WebMaven> I guess you're not aware that Plone is moving to greater dependency on Zope3 stuff (through FIve, mostly)?
[10:53] <WebMaven> Here is the list of Sprints: http://www.openplans.org/projects/seattle-sprint-2006/proposed-topics
[10:53] <kiko> sabdfl, WebMaven's been trying to get in touch with SteveA this week -- he's interested in doing some Z3-related upstream work
[10:54] <WebMaven> kiko: I've since talked to him.
[10:54] <kiko> ah, you have!
[10:54] <kiko> excellent
[10:54] <WebMaven> He put me in touch with niemeyer
[10:55] <kiko> cool
[10:55] <WebMaven> sabdfl: Some context: I am starting a new venture, and was looking for anything that might make Zope3 more suitable for large-scale deployments.
[10:56] <sabdfl> WebMaven: are you interested in relational backends?
[10:56] <WebMaven> sabdfl: yes, I've been having that conv. with niemeyer.
[10:56] <WebMaven> and with SteveA.
[10:57] <WebMaven> When you release stuff I'll be very interested.
[10:57] <WebMaven> Meanwhile I am prototyping woth SQLObject and SQLOS.
[10:57] <niemeyer> sabdfl: WebMaven is interested in having access to things we've been working on, and helping to improve them if they're considered adequate to what he's doing.
[10:58] <WebMaven> Heh.
[10:58] <WebMaven> I would have said 'appropriate' rather than 'adequate'.
[10:58] <sabdfl> well, all in due course. collaboration will be more than welcome.
[10:58] <WebMaven> Yep.
[10:58] <sabdfl> till then, we have operational goals to meet
[10:58] <WebMaven> Yep. I understand.
[10:59] <WebMaven> Sorry, I actually wasn't going to bring any of this up with you.
[10:59] <niemeyer> WebMaven: Sorry, I'm not a native english speaker, so I don't even see the difference of these words in the given context.
[11:00] <WebMaven> niemeyer: appropriate == suitable, adequate == good enough
[11:03] <niemeyer> WebMaven: You mentioned to me you could contribute with documentation being a good writer and so on, and even NDAs and whatnot. If you're interested, sabdfl is the one who may decide on it.
[11:03] <niemeyer> WebMaven: Ah, I see.. I still don't see the difference (and neither does my dictionary), but whatever.
[11:04] <WebMaven> Well, I'm perfectly willing to make that pitch, although this channel is probabaly not the place to do it (being public and all).
[11:04] <dsas> niemeyer: adequate usually has a "but could be better" sort of connotation to it.
[11:05] <WebMaven> niemeyer: 'adequate' is making a subtle implication regarding quality.
[11:05] <bradb> sabdfl: I have a patch which simplifies the launchpad.Driver permissions checking by refactoring it into an IHasDrivers interface, with just one security checker for all driver-related things. Interested in reviewing it?
[11:05] <niemeyer> dsas: Understood. Thanks
[11:05] <WebMaven> It's the difference between asking 'is it right for me' and 'is it good enough'
[11:06] <bradb> sabdfl: it's a very small patch, removes more code than it adds, and removes duplicated checker code in security.py
[11:06] <niemeyer> WebMaven: If something is good enough, it could be right for you..
[11:06] <WebMaven> niemeyer: sure, there is overlap between the terms.
[11:07] <mpt> jamesh / lifeless / spiv, ping
[11:07] <WebMaven> sabdfl: if you have the personal bandwidth, We can PM about this.
[11:13] <WebMaven> sabdfl: OK, I guess no answer means 'not enough personal bandwidth'. ;-)
[11:15] <lifeless> mpt: ?
[11:16] <sabdfl> bradb: +1
[11:16] <sabdfl> no need for me to review, as long as it does not change functionality and passes all prior tests
[11:16] <bradb> sabdfl: right ok, thanks
[11:16] <sabdfl> without changing the tests ;-)
[11:16] <bradb> heh
[11:17] <sabdfl> i think the distro team may need an additional permission, the way mdz was talking last night, but i'll only get to the bottom of that in wiesbaden next year
[11:17] <sabdfl> erk
[11:17] <sabdfl> week
[11:18] <bradb> ok