[12:12] <jabudia> hola
[12:13] <daf> hola
[12:15] <jabudia> speak Spanish, daf ?
[12:15] <daf> poquito
[12:16] <jabudia> I'm new to irc, sorry if I put my leg on it.
[12:17] <daf> I think you mean "put my foot in it" :)
[12:17] <jabudia> is this the same as "messanger", just with out sily icons?
[12:17] <jabudia> yeah, foot
[12:17] <jabudia> :-)
[12:17] <daf> yes, I suppose you could say that
[12:19] <jabudia> hum... right. I don't want to disturb you. Just found irc freenode as the place for extending the services of launchpad.net
[12:20] <daf> no worries
[12:20] <jabudia> I'm exited - this is fantastic - I'm siting at home -Barcelona, Spain, 00:30. Where are you, if can ask?
[12:22] <daf> I'm in Cambridge. 23:30
[12:22] <daf> I'm about to go to bed
[12:22] <daf> Barcelona is a great city
[12:22] <jabudia> ou, good night, then.
[12:24] <ddaa> good night folks
[12:24] <jabudia> night, ddaa
[12:28] <carlos> night
[12:51] <daf> spiv: ping
[12:55] <spiv> daf: pong
[12:55] <daf> what should Branch.url be for push branches?
[12:55] <daf> NULL, or the bazaar.launchpad.net URL?
[12:56] <spiv> NULL.
[12:57] <daf> ok, then this sample data is wrong
[12:57] <spiv> It's computable from the name, product.name and owner.name.
[12:57] <daf> I thought it was odd for it to be set
[12:57] <daf> next question:
[12:57] <daf> BranchPullListing
[12:58] <daf> actually, more specifically:
[12:58] <daf> BranchSet.get_supermirror_pull_list
[12:58] <daf> shouldn't it have a url IS NOT NONE clause?@
[12:59] <spiv> Yes and no ;)
[12:59] <spiv> The ILIKE clause is buggy without one.
[12:59] <daf> oh dear
[12:59] <spiv> But the ILIKE clause is unnecessary.
[12:59] <daf> I killed the ILIKE clause
[01:00] <spiv> Branches where url IS NULL still need to be pulled.
[01:00] <spiv> Just they are pulled from a different source.
[01:00] <daf> ok
[01:00] <daf> then I will:
[01:00] <mpt> Gooooooooooooooooooooooooooood afternoon Launchpadders!
[01:00] <spiv> My supermirrorsftp-integration branch has small changes here, to add a "pull_url" property.
[01:01] <daf> UPDATE Branch SET url = NULL WHRE url ILIKE 'http://bazaar.launchapd.net/%';
[01:01] <spiv> daf: Only in the sampledata, or as a db patch?
[01:01] <daf> and update the test for BranchPullListing to expect branches with NULL URLs
[01:01] <spiv> daf: Skim this diff: https://chinstrap.ubuntu.com/~jamesh/pending-reviews/spiv/launchpad/supermirrorsftp-integration/full-diff
[01:02] <spiv> daf: It may save you a little work to borrow those changes from that diff.
[01:02] <daf> spiv: my primary concern is to make this test pass so I can land optional-branch-title and unblock ddaa
[01:02] <spiv> daf: I want you to land it too, so I can remove the overlapping work from that branch :)
[01:03] <jblack> I don't speak portugues and babelfish isn't helping. What does "A realizar-se 'as dezenove horas do dia vinte e cinco de marco de dois mil e seis" mean?
[01:03] <daf> spiv: ah, you've already made changes to that test
[01:04] <daf> jblack: something about 19 hours of the 25th of Marth 2006?
[01:04] <jblack> Thats what I thought
[01:04] <mpt> sabdfl, pong
[01:04] <daf> mpt: that table looks ugly centered IMHO
[01:05] <sabdfl> mpt: hi
[01:05] <sabdfl> good morning
[01:05] <mpt> sabdfl, I've just checked StaffCalendar and it is up-to-date
[01:05] <sabdfl> thanks
[01:05] <mpt> I'm working afternoons and evenings so I get at least an hour or two with the Europeans present :-)
[01:05] <sabdfl> we appreciate it :-)
[01:06] <sabdfl> what's the best way to get a two-column page up?
[01:06] <sabdfl> i still need the menu, though
[01:06] <sabdfl> so would need navigation above the menu, on the left, i guess
[01:06] <mpt> hmmm
[01:06] <daf> spiv: 
[01:06] <daf>      def test_branch_pull_render(self):
[01:06] <daf> +        from canonical.launchpad.database import Branch
[01:06] <daf> what's that in aid of?
[01:07] <mpt> sabdfl, bradb did two-column for his bug listing pages
[01:07] <spiv> daf: ...
[01:07] <mpt> following my suggestions
[01:07] <sabdfl> but differently from the way i'd like it :-)
[01:07] <spiv> daf: laziness, I think.
[01:07] <daf> :)
[01:07] <spiv> (I thought I'd already removed that import)
[01:07] <daf> hmm, looks like there's quite a bit of overlap
[01:09] <spiv> daf: Ah, I copied it from an existing test.
[01:09] <sabdfl> mpt: any suggestions? can i ask you to put together a standard page template that does this, or is that a major effort?
[01:09] <sabdfl> sitemap and menu on the left, no portlets on the right
[01:09] <mpt> sabdfl, that wouldn't be difficult, it's adding another conditional block to main-template.pt
[01:10] <mpt> which already has a conditional block for "no columns at all"
[01:10] <sabdfl> ok
[01:10] <ajmitch_> what date order does launchpad use for polls, etc? yyyy-mm-dd or yyyy-dd-mm?
[01:11] <daf> I don't think anybody uses y-d-m
[01:11] <daf> m-d-y, d-m-y, or y-m-d
[01:11] <ajmitch_> daf: apparantly germany does?
[01:11] <mpt> sabdfl, look at buglisting-default.pt in rocketfuel, particularly the <tal:layout> in the first line, the <div style=...>, and the <div id="portal-column-two">
[01:11] <ajmitch_> daf: I'll fix this poll up then, thanks
[01:12] <daf> no problem
[01:12] <daf> I think YYYY-MM-DD is an ISO standard
[01:12] <ajmitch_> that's what I thought
[01:12] <ajmitch_> sistpoty: I think a new poll is needed now
[01:13] <daf> date -I
[01:13] <sistpoty> ajmitch_: yes
[01:13] <ajmitch_> unless someone wants to adjust dates manually
[01:13] <daf> I is for ISO, but it's not documented
[01:13] <mpt> daf, perhaps the narrowness of the column makes the table look oddly indented instead of centered?
[01:13] <daf> mpt: my thoughts exactly
[01:14] <sistpoty> I just broke the poll... I set the date to 2006-13-02 which became 2006-02-15 for some reason... should I report a bug about that somewhere? (or is it not worth fixing ;)
[01:14] <daf> that is a bug
[01:14] <daf> the date was not properly validated
[01:14] <daf> please file a bug
[01:14] <sistpoty> daf: ok, will do... thx
[01:15] <daf> thanks to you for reporting it
[01:15] <sistpoty> np ;)
[01:16] <mpt> http://www.cl.cam.ac.uk/~mgk25/iso-time.html
[01:16] <daf> that's the badger
[01:16] <sabdfl> mpt: is that all i need to do a two-column page with menu on the left?
[01:18] <daf> gosh, March
[01:19] <mpt> sabdfl, that page has the menu on the right, so you'd need to change the style= attributes if you wanted it on the left
[01:19] <sabdfl> ok, thanks. is it obvious how to preserve the sitemap?
[01:20] <mpt> sabdfl, you'd need to insert it yourself by copying the line main-template.pt uses <div tal:replace="structure context/@@+sitemap" />
[01:20] <sabdfl> ok, thanks
[01:20] <sabdfl> night all
[01:20] <mpt> 'night
[01:20] <mpt> Somebody stolen a March on you, daf?
[01:22] <mpt> (BROOKE-TAYLOR: "Whatever does this mean, sir?" CLEESE: "We shall have to have two Februaries!")
[01:23] <daf> spiv: I can't see how to make this test sane without implementing Branch.pull_url
[01:24] <spiv> daf: Yeah, I think you need it.
[01:24] <spiv> daf: I'd just steal it from my branch.
[01:24] <daf> spiv: right, I'll do that
[01:24] <spiv> daf: Don't forget the interface bit for it, too :)
[01:25] <spiv> daf: I'd be very happy to reduce the size of my branch a bit, this stuff is a bit tangential to the sftp bits.
[01:25] <daf> cool
[01:26] <spiv> daf: having pull_url merged sooner rather than later will make working on bug 32106 a bit easier, too.
[01:26] <Ubugtu> malone bug 32106 in launchpad "Extend supermirror-pull-list.txt for vcs-imports" [Normal,In progress]  http://launchpad.net/bugs/32106
[01:30] <daf> spiv: as long as you're happy to sort out the conflicts :)
[01:32] <sistpoty> daf: bug reported as 33203 ;)
[01:32] <daf> bug 33203
[01:32] <Ubugtu> malone bug 33203 in launchpad "poll: dates are not properly validated" [Normal,Unconfirmed]  http://launchpad.net/bugs/33203
[01:32] <daf> sistpoty: great, thanks
[01:32] <daf> spiv: awesome, that works
[01:33] <daf> spiv: now either I submit a merge with these stolen changes, or I push it back to you for another review pass
[01:44] <daf> spiv: I'm happy that the changes and the test coverage are good
[01:44] <daf> but the changes since the last review are non-trivial
[01:45] <daf> even though most of them were stolen from you ;)
[01:45] <daf> so, up to you
[01:46] <spiv> Yeah, I'm hapyp to sort out the conflicts.
[01:46] <spiv> happy, rather :)
[01:46] <spiv> This is the optional-branch-title branch?
[01:47] <daf> yes
[01:47] <daf> now the optional-branch-title-and-push-url branch
[01:47] <spiv> :)
[01:47] <spiv> If the non-trivial bits are just the pull_url bits, then I'm happy with those.
[01:48] <spiv> Those bits have already been reviewed in my branch by jamesh.
[01:48] <daf> ah, good
[01:48] <daf> I'll paste a diff against RF
[01:48] <daf> just in case
[01:48] <spiv> Sure.
[01:48] <daf> https://chinstrap.ubuntu.com/~dsilvers/paste/filekiPzci.html
[01:49] <daf> hmm, how come I didn't notice branch-pages.txt before?
[01:50] <daf> most of the tests I added should be moved to there
[01:50] <spiv> daf: In get_supermirror_pull_queue, supermirror_root now seems totally unused.
[01:52] <daf> spiv: good point
[01:53] <spiv> And yeah, moving those view tests into branch-pages.txt is a good idea.
[01:53] <daf> in fact, canonical.config is not used at all
[01:53] <spiv> Well, leave it there, we'll have use for that later.
[01:53] <spiv> As mentioned in an email a while ago.
[01:53] <daf> er, really?
[01:54] <spiv> Let me dig it up...
[01:54] <daf> it's trivial to add the import back later
[01:54] <daf> and WAGNI
[01:54] <spiv> daf: ddaa's mail of Feb 15th, "Re: REVIEW: daf/launchpad/optional-branch-title"
[01:54] <daf> the future is an illusion
[01:54] <daf> etc.
[01:55] <daf> ok, I'll leave it in
[01:55] <spiv> Well, at least keep that snippet somewhere handy, ready to reapply sometime soon ;)
[01:55] <daf> shelved
[01:56] <spiv> Ah, yep, the supermirror_url method uses that config value.
[01:56] <spiv> Oh, you were talking about the import.
[01:56] <spiv> I misread :)
[01:56] <daf> yes, the import :)
[01:56] <spiv> I thought you were referring to the added config value.  Yeah, imports are trivial to add and remove :)
[01:57] <spiv> I must have seemed mad ;)
[01:57] <daf> yes!
[01:57] <daf> you're making much more sense now
[01:57] <daf> I was thinking maybe it was later than I thught
[02:03] <daf> spiv: ok, anything else?
[02:05] <spiv> Nope, looks good to me.
[02:05] <daf> groovy
[02:05] <daf> merge submitted
[02:05] <daf> I'm going to bed
[02:05] <spiv> G'night :)
[04:11] <sladen> mpt: am I allowed to add questions to  https://wiki.launchpad.canonical.com/BugWorkflow  ?
[04:11] <sladen> mpt: if not, where's the best place to raise them
[04:11] <sladen> mpt: btw, it looks *much* nicer to use
[04:25] <mpt> sladen, sure, add to the "Unresolved issues" section
[04:29] <sladen> mpt: does that include the stuff I /query'ed to you
[04:30] <mpt> sladen, yes please
[04:30] <mpt> I'm cooking now, I'll look at the questions later
[04:31] <ajmitch_> early dinner?
[05:07] <spiv> lifeless: the food pyramid of testing: http://photos1.blogger.com/blogger/7546/664/1600/testing-pyramid.jpg
[05:42] <lifeless> spiv: ahhhahha. YES
[05:45] <spiv> lifeless: from http://agiletesting.blogspot.com/2006/02/thoughts-on-giving-successful-talk.html, but it doesn't really need much context :)
[06:05] <sladen> spiv: I seem to get away with using no slides at all :)
[07:09] <dilys> Merge to devel/launchpad/: [r=spiv]  make branch titles and summaries optional (r3209: Dafydd Harries, David Allouche)
[07:59] <mpt> ajmitch_, no, I was cooking a roast :-)
[08:00] <ajmitch_> mpt: a shame I'm not in town otherwise I'd have to come & visit :)
[08:01] <mpt> Roast chicken, kumara, carrots and cauliflower
[08:01] <mpt> and broken glass
[08:02] <mpt> (I dropped the cauliflower)
[08:02] <ajmitch_> ah, a shame
[08:31] <sabdfl> mpt_: i looked at the 2col solution you proposed
[08:31] <sabdfl> can i instead ask you to whip up a new template?
[08:31] <sabdfl> so instead of:
[08:31] <sabdfl>       metal:use-macro="context/@@main_template/master"
[08:31] <sabdfl> we can do:
[08:31] <sabdfl>       metal:use-macro="context/@@main_2col_template/master"
[08:31] <sabdfl> this one should still support 2 sets of portlets, just put the second one below the first
[08:32] <mpt_> ok
[08:32] <sabdfl> this would give a very quick-and-dirty way of converting a page from 3col to 2col
[08:32] <sabdfl> to be cleaned up later
[08:32] <sabdfl> portlets on the left, sitemap, then menu, then portlet
[08:32] <sabdfl> thanks
[08:35] <mpt_> grrrrr
[08:52] <Mez> morning again sabdfl 
[08:53] <sabdfl> hey Mez
[08:53] <Mez> weird thing is I havent been to sleep yet!
[08:53] <Mez> lol
[08:53] <Mez> cant at the moment... damn pain
[08:57] <carlos> morning
[09:15] <sivang> morning
[09:15] <sivang> Mez: dude, you've been up all night?
[09:17] <Mez> of course
[09:18] <sivang> Mez: oh man, what sort of pain is that you are experiencing?
[09:18] <Mez> sivang, lots of it
[09:18] <sivang> hangover?
[09:19] <Mez> no - slipping on wet floor - leg going at an angle it shouldnt and cracking head odd wall kinda pain
[09:21] <sivang> so, you're basically insomniac or something..
[09:21] <sivang> or just been a specific bad night? :)
[09:22] <sivang> (I had 3 of those in a row till now lol)
[09:35] <Mez> sivang: had massive fall at work ... in a lot of pain
[09:37] <sivang> ah, bad. I hope you get better soon
[11:48] <carlos> stub: hi, around?
[11:48] <stub> carlos: yes
[11:48] <carlos> stub: I need help with the migration script we talked about yesterday
[11:49] <carlos> stub: do you have some time to help me?
[11:49] <stub> Sure.
[11:49] <carlos> first, let me show you the db changes I'm going to do
[11:50] <carlos> stub: https://chinstrap.ubuntu.com/~dsilvers/paste/fileMJXz0O.html
[11:50] <stub> Email might be better for this than IRC - can you send me a message explaining what you have got and what you need to do?
[11:50] <carlos> stub: sure
[11:51] <carlos> thank you 
[12:32] <Kinnison> morning cprov
[12:43] <carlos> stub: mail sent
[12:44] <carlos> cprov: morning
[12:44] <cprov> carlos: morning dude, how is rosetta "auto-import" stuff going in production ?
[12:45] <carlos> cprov: the amount of files broke the page were we review the imports giving timeouts. I'm working on fixing that
[12:45] <carlos> but the soyuz bridge is working perfectly
[12:46] <cprov> carlos: yep, it was kind of expected ... but still sad :(
[12:46] <cprov> carlos: will it take much longer to add batching support on that page ? or are you thinking in something else ?
[12:47] <carlos> cprov: the way that pages works makes batching impossible as it is
[12:47] <carlos> cprov: I'm finishing already some changes to allow batching there
[12:47] <carlos> I mean, it's not just a matter of adding batching support to that page ;-)
[12:49] <cprov> carlos: I see, the solution must involve a kinda of extra "touch", but it's okay, i think you know what is required. Let me know if you need any help
[12:50] <carlos> cprov: thanks
[12:51] <cprov> carlos: np, anytime, btw have you seem steve last days ?
[12:52] <carlos> cprov: yes, on Monday he was online and sent me a review. Not sure if yesterday he was around...
[12:53] <cprov> carlos: right, tks, I think I can call him if he doesn't show up, have some reviews to sort out.
[01:00] <matsubara> good morning!
[01:01] <carlos> Mar  1 12:43:31 gandalf postfix/smtp[14934] : 1102E174148: to=<stuart.bishop@canonical.com>, relay=fiordland.ubuntu.com[82.211.81.145] , delay=1, status=sent (250 Ok: queued as DE40EB68024)
[01:01] <carlos> stub: you should have it
[01:02] <carlos> matsubara: morning!
[01:02] <stub> carlos: Can you please resend to stuart@stuartbishop.net ? 
[01:02] <carlos> sure
[01:04] <stub> carlos: got it
[01:07] <stub> and the original just arrived too
[01:09] <Toma-> im trying to add a Spec, but it wont recognise my name? Im logged in as "Toma" and i put "Toma" in the name and says its invalid.
[01:10] <carlos> Toma-: put there your email
[01:10] <Toma-> rightio. thx
[01:10] <Toma-> still error.
[01:10] <Toma-> :/
[01:11] <carlos> which kind of error?
[01:11] <Toma-> ahhh done
[01:11] <Toma-> had to drop the @blah.com part
[01:16] <daf> carlos: apart from the data migration, how is #33020 doing?
[01:16] <carlos> daf: I need to port the admin interface to the new procedure and fix any test broke and I'm done
[01:17] <daf> ok...
[01:17] <daf> that doesn't sound too bad
[01:17] <daf> think you can get it done today?
[01:17] <carlos> yes
[01:17] <daf> I want to translate Dapper
[01:17] <carlos> :-P
[01:18] <daf> when do you think I can do it? :)
[01:20] <carlos> daf: Well, I don't think today it will be reviewed and cherry picked. Perhaps tomorrow and we still need to review some imports
[01:21] <daf> ok, so #33020 is the last blocker?
[01:21] <carlos> daf: that means that some parts of dapper will be available to translate (in fact some of them are already available)
[01:22] <carlos> daf: I need to prepare a script to import translations from packages built before the soyuz integration into Rosetta and ask stub to run it on production after a code review
[01:22] <carlos> to have all resources imported
[01:22] <daf> ok, where is GNOME at?
[01:23] <carlos> well, GNOME will be imported without that script
[01:23] <carlos> as I think we had a recent release
[01:23] <carlos> hmm or will have one soon
[01:23] <carlos> and soyuz will handle those packages
[01:24] <carlos> the 'problem' is only with packages that are not updated too often
[01:24] <carlos> so I don't think it's a big issue
[01:24] <carlos> anyway I will do it this week
[01:25] <daf> hmm, I wonder why epiphany isn't at https://launchpad.net/distros/ubuntu/dapper/+lang/cy
[01:26] <daf> and why this page doesn't seem to be sorted
[01:29] <carlos> daf: dapper imports are not completely done... so I guess epiphany is not imported
[01:30] <daf> there's no epiphany for hoary either
[01:31] <carlos> daf: I think the package is epiphany-browser
[01:31] <carlos> epiphany is a game
[01:32] <daf> eah, I know
[01:33] <daf> https://launchpad.net/distros/ubuntu/+source/epiphany-browser -- no translations menu item sucks
[01:33] <carlos> daf: oh, you said hoary not breezy... yes, seems like Hoary translations were not imported
[01:33] <daf> https://launchpad.net/distros/ubuntu/dapper/+source/epiphany-browser/+translations
[01:34] <daf> it didn't get imported for some reason
[01:34] <daf> how can I find out why?
[01:34] <carlos> daf: Once this bug is fixed I will review breezy and hoary and import any missing package. The new queue system will allow us to handle those missing imports (usually broken due the lack of .pot files)
[01:34] <carlos> daf: take a look to the po attach logs
[01:34] <daf> well, I don't care about hoary
[01:34] <daf> where are the logs?
[01:34] <carlos> daf: if you don't care about hoary....
[01:35] <carlos> daf: why do you want to take a look to the logs?
[01:35] <carlos> daf: look at the errors mailing list all logs are sent there
[01:35] <daf> oh, I'm confused
[01:35] <daf> it did get imported for breezy
[01:35] <carlos> daf: right ;-)
[01:35] <daf> today is clearly not a good day for my brain
[01:36] <daf> cprov: can we expect to see kiko today or is he still recovering from carnaval?
[01:36] <AlinuxOS> carlos, when can I translate Warty ?
[01:36] <AlinuxOS> :)
[01:37] <carlos> AlinuxOS: no language packs fro Warty.... no imports into rosetta for it ;-)
[01:37] <AlinuxOS> carlos, :((((
[01:37] <AlinuxOS> tragedy!
[01:37] <AlinuxOS> :P
[01:37] <carlos> AlinuxOS: in fact in two months warty is unsupported, right?
[01:38] <carlos> you can always fork it.... :-P
[01:43] <AlinuxOS> :)
[01:47] <cprov> daf: sorry the delay, I really don't know about kiko, but I'd expect him today, the holliday is pretty much gone already
[01:47] <daf> cprov: he doesn't drink, so he can't be hung over :)
[01:48] <cprov> daf: so his holliday was pointless, if you don't drink you can just work through it, like I did ;)
[01:49] <daf> ha
[01:53] <daf> carlos: did you see the comments on https://launchpad.net/products/rosetta/+bug/44?
[01:53] <Ubugtu> malone bug 44 in rosetta "Messages should be searchable." [Wishlist,Confirmed]  
[01:53] <carlos> daf: yes, I did
[01:54] <carlos> daf: haven't time to answer, anyway... I tried it, but I was not able to get more than 24 hours in a day...
[01:54] <daf> :)
[01:56] <carlos> https://launchpad.net/products/rosetta/+spec/translate-specific-string <- The spec needs some extra work
[01:59] <carlos> stub: Do you need anything from me for the data migration ?
[02:01] <stub> carlos: It seemed fairly straight forward. I'll try assemblinb a patch and pass it back to you to verify. Hopefully it will still seem straightforward when I'm done :-)
[02:02] <carlos> daf: answered
[02:03] <daf> thanks dude
[02:03] <carlos> stub: ok, thanks! :-)
[02:17] <cprov> matsubara: heads up on bug # 33141, okay ?
[02:17] <cprov> uhm bug 33141
[02:17] <Ubugtu> malone bug 33141 in soyuz "duplicate links on distribution release source package page" [Major,Confirmed]  http://launchpad.net/bugs/33141
[02:17] <cprov> ohhh, lazy bot
[02:19] <matsubara> cprov: ok
[02:44] <ddaa> hey
[02:44] <ddaa> daf: how is it goin'?
[02:48] <daf> dd	yo
[02:48] <daf> ahem
[02:49] <daf> ddaa: yo
[02:49] <daf> ddaa: o-0b-t has landed
[02:49] <ddaa> yay!
[02:49] <ddaa> how did you solve the pull-list test conundrum?
[02:49] <daf> I ened up pulling in spiv's .pull_url changes
[02:50] <ddaa> mh? spiv's patches landed too?
[02:50] <daf> no
[02:50] <daf> I copy+pasted his .pull_url cahanges
[02:51] <ddaa> I sorta guess that would have required another review... but I do not complain
[02:51] <daf> spiv looked at my changes before I merged
[02:51] <ddaa> well since he is your reviewer :)
[02:52] <ddaa> nepotism, kinda cool
[02:52] <daf> and his pullb_url stuff had been reviewed by jamesh
[02:52] <daf> so it's all good
[02:53] <ddaa> So, I guess you are basically off the hook.
[02:54] <ddaa> daf: thanks a lot for your help
[02:54] <ddaa> now, I need to get in touch with spiv and BjornT
[02:54] <ddaa> BjornT: ping
[02:54] <daf> ddaa: de rien
[02:59] <ddaa> daf: about yesterday's bounce
[03:00] <ddaa> my ISP's DC had a power outage during a routine test of the emergency power
[03:00] <ddaa> that broke some massive havoc
[03:02] <daf> ok -- it wasn't anything urgent
[03:02] <daf> I was just forwarding you the PQM failure in case you wanted to look at it while I was out
[03:02] <daf> at least, it's not urgent any more
[03:02] <ddaa> it's FIXED now, until the next time
[03:04] <ddaa> mpt: "At a certain point Launchpad will be so easy to understand that making it even easier to understand will make it noticably less efficient to use" I appreciate your optimism :)
[03:05] <ddaa> I rather expect that Launchpad is going to stay a moving target so the UI will never be able to completely catch up.
[03:10] <bradb> ddaa: He did say it would take several years. mpt is usually sharply realistic. :)
[03:10] <ddaa> Well, I think that the target will keep on moving for years :)
[03:10] <sabdfl> stub: ping
[03:10] <ddaa> but I'm usually rather pessimistic :)
[03:11] <stub> sabdfl: pong
[03:11] <sabdfl> got an sql fragment for review
[03:12] <stub> argh! spam!
[03:13] <sabdfl> :-)
[03:13] <stub> sabdfl: https://chinstrap.ubuntu.com/~dsilvers/paste/ 
[03:13] <sabdfl> couldn't find the handy paste thingie
[03:13] <sabdfl> ah
[03:13] <stub> Its in the other channels topic
[03:14] <sabdfl> https://chinstrap.ubuntu.com/~dsilvers/paste/filexXnBSb.html
[03:14] <sabdfl> there's a comment for goalstatus, yes
[03:16] <stub> sabdfl: approved then. patch-40-25-0
[03:18] <sabdfl> danke
[03:50] <BjornT> ddaa: pong
[03:51] <ddaa> BjornT: can you review spiv's rename-buttsource branch please. I have a branch which needs to land next week which depends on it.
[03:51] <sabdfl> anybody else ever seen an overview menu that refuses to display?
[03:51] <ddaa> (I would also like if you could review my branch as well, but it's a bit moot before rename-buttsource has landed)
[03:52] <ddaa> Hu actually, I rephrase that, my dependent branch needs to be _in production_ next week.
[03:53] <BjornT> ddaa: sure, i was planning to do it today. i don't think i have time to review your branch today, but i can do it tomorrow. (actually, i might have time today, we'll see)
[03:54] <ddaa> please, the schedule is starting to be very tight to get all we need for bzr imports in production by monday
[03:56] <BjornT> ddaa: ok, i'll try to do it today. be aware though, i'm in a bad mood from spending almost two hours at the migration department ;)
[03:56] <ddaa> that's okay, it's nothing compared to the mood I'm in after spending 3.5 hours _not_ to get a ticket for a concert...
[03:57] <ddaa> (I'll try again tonight, though)
[03:58] <sabdfl> anybody, anybody.... menu help?
[03:59] <BjornT> sabdfl: what do you need help with?
[04:00] <sabdfl> my overview menu refuses to show itself
[04:01] <BjornT> sabdfl: maybe you forgot to declare a facet for the page, on which the menu should be displayed?
[04:01] <sabdfl> hmm...will check, good idea
[04:02] <sabdfl> this is for the sprint stuff, which i think predated more recent menu work
[04:09] <sabdfl> thanks dude
[04:10] <BjornT> np, happy to help
[04:15] <kiko> hey subarus
[04:15] <sabdfl> hey kiko
[04:15] <kiko> sabdfl! how's london feeling?
[04:15] <sabdfl> great to be home
[04:15] <kiko> I can imagine after such a long haul
[04:15] <sabdfl> have taken two days from ceoing to work on this thing of yours called "Launchpad"
[04:16] <kiko> tell me all about it
[04:16] <ddaa> somebody mentions it has plenty of a real cool UI stuff called "portlets"
[04:19] <bradb> ddaa: Did you just get a "success" message for the job that just finished in pqm?
[04:20] <bradb> oh, good, my changes really did land. they're in rf, despite no commit mail.
[04:22] <sabdfl> dudes
[04:22] <sabdfl> its snowing
[04:22] <ddaa> bradb: yes
[04:23] <ddaa> it's been ages since I got a PQM message like that
[04:23] <ddaa> in london? fwiw, it's snowing in paris too
[04:25] <kiko> snowing in london? in march? that's surprising
[04:25] <ddaa> yay launchpad conflicts galore!
[04:32] <bradb> bradb@oxygen:~/canonical/malone-bug-32709 $ bzr push --no-tree
[04:32] <bradb> bzr: ERROR: unknown long option '--no-tree' for command push
[04:33] <bradb> !
[04:33] <bradb> bzr seems to change behaviour from one invocation to the next
[04:33] <kiko> if you apt-get upgrade between them, then..
[04:34] <bradb> sure, but even not doing that :)
[04:36] <bradb> It seems to be due to not having a .bzr/x-push-data
[04:38] <jamesh> --no-tree is a bzrtools push option
[04:38] <jamesh> maybe the bzr there doesn't have bzrtools install
[04:38] <jamesh> ed
[04:39] <bradb> jamesh: Nope, it worked fine on other branches, and is now working fine on this branch again.
[04:39] <bradb> I blame it on clever, self-mutating code.
[04:39] <jamesh> bradb: the bzrtools push command falls back to the builtin push command if you haven't asked to do an rsync push
[04:39] <bradb> it's a bug either way, in any case
[04:40] <jamesh> so that would explain the connection to .bzr/x-push-data
[04:40] <kiko> it's very odd behaviour, I agree
[04:40] <jamesh> it'd be a lot simpler if bzrtools just used a different command name
[04:40] <jamesh> or if the rsync push stuff got integrated into the normal command
[04:41] <kiko> indeed.
[04:52] <BjornT> ddaa: which branch did you want me to review? you have two branches in the general queue.
[04:54] <ddaa> productseries-branch-oops, but please hold on, I'm resolving conflicts with the optional-branch-title which has landed
[04:55] <BjornT> ok, ping me when i can start reviewing it.
[05:00] <kiko> bradb, do you know what causes oops OOPS-59B295 ?
[05:00] <Ubugtu> https://chinstrap.ubuntu.com/~jamesh/oops.cgi/59B295
[05:02] <bradb> kiko: Yeah, I've got a fix for it, but I had a small traversal issue to work out (adding /wobbly to the end of the URL breaks), but I hope to land it this morning.
[05:03] <bradb> I'm fixing that issue right now though.
[05:03] <bradb> And still wondering where the heck this morning's commit mail is disappearing to.
[05:08] <seb128> hi
[05:08] <seb128> bradb: well done for the upstreamtask change :)
[05:08] <bradb> :)
[05:09] <seb128> that's reduce pages to open from 3 to 1 :)
[05:09] <seb128> I mean for that action
[05:09] <seb128> which is nice ;)
[05:10] <irvin> window /5
[05:14] <daf> matsubara-lunch: ProgrammingError: ERROR: duplicate key violates unique constraint "bugtask_distinct_sourcepackage_assignment" -- I thought we had fixed that
[05:15] <kiko> we did
[05:15] <kiko> that oops is early
[05:16] <kiko> wait for my report analysis, daf ;)
[05:16] <daf> slacker
[05:16] <daf> wtf is "smtb" anyway?
[05:17] <kiko> suicide
[05:17] <kiko> mountain
[05:17] <kiko> biking
[05:17] <kiko> the carnival sport of preference in this fair city
[05:17] <kiko> however
[05:17] <kiko> I ended up biking to the beach and half-back
[05:17] <kiko> on the road bike
[05:18] <daf> half back?
[05:18] <kiko> so it wasn't entirely true
[05:18] <daf> https://chinstrap.ubuntu.com/~dsilvers/paste/filedsrZbU.html
[05:18] <kiko> well, it would have been 900km back the whole way, and I needed to work today, so we caught the bus in So Jos yesterday and saved 300km
[05:18] <daf> ^^^ fux for big #31589
[05:19] <daf> does it look sane to you?
[05:19] <daf> bug 31589
[05:19] <Ubugtu> malone bug 31589 in launchpad "Attempting to set redirection_url to a tuple instead of a string in login machinery" [Normal,In progress]  http://launchpad.net/bugs/31589
[05:20] <kiko> daf, does that work? it replaces some pretty kooky code, but I don't know if they are equivalent. can you give me an analysis?
[05:20] <daf> sure thing
[05:21] <daf> just as soon as I remember how it works
[05:22] <daf> I think it's the difference between "not request.form.get('redirection_url')" and "'redirection_url' not in request.form"
[05:22] <daf> i.e. the former also returns true when bool(form['redirection_url'] ) is False
[05:23] <daf> so we get the form element twice when form['redirection_url']  == ""
[05:23] <daf> am I making sense?
[05:26] <daf> to put it another way, request/form/redirection_url|nothing doesn't distinguish between "element not in form" and "element in form but has a false value"
[05:26] <kiko> yes
[05:27] <kiko> and you want to check only the former, not the latter?
[05:27] <daf> yes
[05:27] <daf> exactly
[05:27] <kiko> rs=kiko
[05:27] <daf> woots
[05:27] <kiko> can't r= because I can't really understand it
[05:27] <kiko> but your code is definitely clearer
[05:27] <kiko> so if it is also broken
[05:28] <kiko> it is at least less confusingly broken
[05:28] <kiko> now
[05:28] <kiko> daf, where's the test?
[05:28] <daf> yeah, the test
[05:29] <daf> I'm not sure how to test this
[05:29] <kiko> a pagetest?
[05:29] <daf> because I'm making a negative assertion
[05:29] <daf> "this <input> doesn't appear in the form when I pass this parameter"
[05:31] <daf> this page is pretty hairy
[05:32] <daf> it deals with 7 different cases
[05:32] <kiko> it is
[05:32] <kiko> perhaps salgado has an idea on how to test it?
[05:32] <bradb> daf: "foo not in bar"?
[05:33] <daf> bradb: in a page test?
[05:33] <bradb> daf: yeah
[05:33] <bradb> It's just Python code.
[05:33] <daf> yo mean "<inmput....>" not in GET(....)?
[05:34] <bradb> daf: If you really mean to test that, yeah.
[05:34] <bradb> Although I'd probably first store the response in a var.
[05:36] <daf> hmm, maybe I can work out a way of cleaning up the logic
[05:36] <daf> and then testing the logic more directly
[05:36] <daf> since it won't be in the page template any more
[05:37] <BjornT> daf: what page is sending the redirection_url parameter? it seems wrong that you can override which page to redirect to by sending an extra parameter to the login page.
[05:39] <daf> BjornT: I don't think we know that
[05:40] <daf> BjornT: it's just that James managed to reproduce the symptoms in the report by passing in an empty redirection_url
[05:42] <kiko> bradb, ping?
[05:42] <bradb> kiko: pong
[05:42] <kiko> bradb, do you know what causes oops OOPS-59B295 ?
[05:42] <Ubugtu> https://chinstrap.ubuntu.com/~jamesh/oops.cgi/59B295
[05:43] <bradb> kiko: Same answer as when you asked earlier :)
[05:43] <kiko> I didn't see i
[05:43] <kiko> t
[05:43] <kiko> oh, sorry
[05:44] <kiko> I was busy writing up the report
[05:44] <kiko> bradb, is there a bug number for it?
[05:44] <bradb> bug 32709
[05:44] <Ubugtu> malone bug 32709 in launchpad "Cannot view status of bug in some cases" [Normal,In progress]  http://launchpad.net/bugs/32709
[05:44] <BjornT> hmm, i wonder why redirection_url is needed at all...
[05:47] <bradb> BjornT:
[05:47] <bradb> ForbiddenAttribute: ('publishTraverse', <zope.app.pagetemplate.simpleviewclass.SimpleViewClass from /home/bradb/canonical/malone-bug-32709/lib/canonical/launchpad/templates/bugtask-view.pt object at 0x31854c90>)
[05:47] <bradb> If $url returns a view, then how do I make $url/wobbly 404 and not raise that error?
[05:48] <kiko> interesting
[05:48] <kiko> I just saw this error in the oops reports
[05:50] <BjornT> bradb: i think you have to define a publishTraverse() method, and either return None or raise NotFound, can't remember which.
[05:51] <bradb> ok, thanks, I'll give it a shot
[05:52] <BjornT> although i think there is a bug somewhere, i would assume +someview/wobbly to be a 404 by default.
[05:53] <bradb> me too
[05:57] <bradb> BjornT: Oh, scarier still, it seems that it only applies to browser:schemadisplay.
[05:57] <bradb> Because +editstatus seems not to have this problem.
[06:02] <kiko> BjornT, bradb: this causes the same problem:
[06:02] <kiko> https://launchpad.net/distros/ubuntu/breezy/+source/gnome-terminal/+pots/gnome-terminal/fr/+translate/@@/launchpad.js
[06:02] <BjornT> bradb: ok, don't spend too much time on it then, +viewstatus isn't a schemadisplay anymore in my branch.
[06:02] <kiko> I think it's a fundamental issue
[06:02] <kiko> not something callsites are doing
[06:03] <bradb> BjornT: Does http://localhost:8086/distros/debian/+source/mozilla-firefox/+bug/1/+viewstatus/wobbly not error for you then?
[06:05] <BjornT> bradb: it's a 404 in my branch
[06:06] <bradb> lucky you. ok, i'll leave it to your branch to have that fixed and just land the fix for bug 32709 in this branch
[06:06] <Ubugtu> malone bug 32709 in malone "Cannot view status of bug in some cases because of a KeyError: "milestone_widget" being raised" [Major,In progress]  http://launchpad.net/bugs/32709
[06:41] <salgado> can anybody run a query on production for me?
[06:43] <G0SUB> any rosetta admins here?
[06:43] <G0SUB> jordi ?
[06:50] <carlos> G0SUB: hi
[06:51] <G0SUB> carlos hello!
[06:51] <G0SUB> carlos we want to get our language team registered for Ubuntu Translations
[06:51] <carlos> did you follow the procedure at wiki.ubuntu.com/RosettaFAQ ?
[06:51] <G0SUB> carlos yep
[06:52] <jordi> G0SUB: what team is this?
[06:52] <G0SUB> carlos I have mailed rosetta@ubuntu.com as well as rosetta-users
[06:52] <G0SUB> jordi Bengali
[06:52] <carlos> G0SUB: jordi will handle it
[06:52] <carlos> jordi: hi 
[06:52] <G0SUB> jordi Steve told me to CC you the mail
[06:52] <jordi> G0SUB: yeah, should have done last night, but run out of time
[06:52] <jordi> G0SUB: let's go for it, let me open your email
[06:52] <G0SUB> jordi oh, okay
[06:53] <G0SUB> jordi awesome :)
[06:54] <jordi> G0SUB: ok, first thing:
[06:54] <sabdfl> once i have a branch on chinstrap that i'm updating, should i use --overwrite?
[06:54] <jordi> https://launchpad.net/projects/ubuntu-l10n-bn/
[06:54] <jordi> this is wrong, you should have created a team, not a project. Go here:
[06:55] <G0SUB> jordi yeah, I was confused
[06:55] <jordi> https://launchpad.net/people/+newteam
[06:55] <jordi> oh
[06:55] <jordi> it's created already
[06:55] <G0SUB> what is already created?
[06:55] <jordi> carlos: I guess you can't add people now
[06:55] <jordi> G0SUB: https://launchpad.net/people/ubuntu-l10n-bn
[06:56] <jordi> hey sabdfl 
[06:56] <sabdfl> hiya jordi
[06:56] <G0SUB> jordi so what next?
[06:56] <carlos> jordi: no, you need to ask it to any launchpad admin
[06:57] <jordi> G0SUB: ^^ -- A launchpad admin will do it for us, hopefully now
[06:58] <jordi> hmm, no steve, no lifeless
[06:58] <kiko> carlos, why don't we fix those permissions while we are add it?
[06:58] <kiko> sabdfl, no, you shouldn't ever need to --overwrite
[06:59] <carlos> kiko: I promised Jordi to fix it this week, don't worry
[06:59] <kiko> okay
[06:59] <kiko> s/add/at 
[06:59] <sabdfl> kiko: what's the fastest push once i have already done a slow push?
[06:59] <G0SUB> jordi aargh!
[07:00] <G0SUB> jordi any idea when that project was created?
[07:00] <kiko> sabdfl, I don't quite understand what you mean. you shouldn't ever need to do a "slow push" if you cp -ax your branches in chinstrap, --overwrite once, and then bzr push away for its lifetime.
[07:00] <sabdfl> bzr push sftp://... etc?
[07:00] <kiko> oh
[07:00] <kiko> don't use sftp://
[07:00] <kiko> use chinstrap:/home
[07:01] <carlos> sabdfl: rsync is always the fastest way to do a push
[07:01] <jordi> G0SUB: I thought you created the project
[07:01] <G0SUB> jordi nope, we have been hijacked
[07:01] <jordi> woa
[07:01] <kiko> sabdfl, and what carlos is saying is that sftp:// implies sftp, not rsync
[07:01] <G0SUB> jordi that's some other party
[07:02] <G0SUB> jordi there was no bn team since yesterday
[07:02] <G0SUB> till yesterday
[07:02] <carlos> right, and the chinstrap:/home... url uses rsync. I'm not suggesting use rsync directly
[07:02] <jordi> G0SUB: We got two emails in the last days from Baishampayan Ghose <b.ghose@gnu.org.in>
[07:02] <jordi> requesting it
[07:02] <G0SUB> jordi that's me
[07:02] <jordi> ok
[07:03] <jordi> carlos: can we know when https://launchpad.net/people/ubuntu-l10n-bn/ was created?
[07:03] <jordi> ugh
[07:03] <jordi> today
[07:03] <jordi> this sounds like a joke.
[07:04] <G0SUB> jordi see?
[07:04] <soumyadip> G0SUB, 
[07:04] <carlos> jordi: why?
[07:04] <G0SUB> soumyadip the BD guys have hijacked it ...
[07:04] <soumyadip> duh ?
[07:04] <jordi> carlos: G0SUB asked on the 27th for a Bengali team. Today, one appeared in launchpad, but has no relation to him
[07:05] <jordi> G0SUB: who are the "BD guys"? Do you know them?
[07:05] <G0SUB> jordi BD is Bangladesh
[07:05] <G0SUB> jordi we are from India
[07:05] <soumyadip> jordi, the BD guys are people from Bangladesh
[07:05] <G0SUB> jordi we have the same language but two locales
[07:06] <soumyadip> jordi, ideally we'd like to have a single project, under the aegis of bengalinux.org
[07:06] <carlos> G0SUB: does the language change a lot?
[07:06] <G0SUB> carlos yes
[07:06] <G0SUB> carlos quite a lot
[07:07] <soumyadip> carlos, not a lot, but significant enough for Mozilla to allot two different projects :)
[07:07] <jordi> G0SUB: what variant is the normal "bn" translations, without a country code?=
[07:07] <carlos> then ubuntu-l10n-bn should not be used
[07:07] <soumyadip> jordi, usually bn_IN
[07:07] <G0SUB> jordi well, it's bn_IN
[07:08] <jordi> ie, is there a bn and bn_BD translations, or is there bn_IN and bn_BD, and nobody uses plain "bn"?
[07:08] <soumyadip> jordi, the second one
[07:08] <soumyadip> bn_IN and bn_BD
[07:08] <jordi> ok, as the Chinese then.
[07:08] <soumyadip> yup
[07:09] <jordi> We'll rename this team to bn-BD, yours will be bn-in then
[07:09] <carlos> jordi: then ignore ubuntu-l10n-bn for now and try to contact them to rename it to the right name
[07:09] <jordi> yep
[07:11] <jblack> Anybody on hte launchpad team with experience building packages?
[07:12] <G0SUB> jordi can you assign ubuntu-l10n-in to us then?
[07:12] <jordi> G0SUB: you mean ubuntu-l10n-bn-in?
[07:12] <G0SUB> yeah, that
[07:13] <jordi> yes.
[07:13] <jordi> Let me finish writing this email to the BD dudes
[07:13] <G0SUB> jordi okay
[07:13] <jordi> When you said "BD guys", I thought you meant "BAD guys", and I thought we'd have a big flamewar between the two groups. :)
[07:13] <G0SUB> jordi lol
[07:16] <jordi> G0SUB: ok, can you create your bn-in team and add your people to it?
[07:16] <jordi> G0SUB: I'll send an admin request so it gets added to Ubuntu translators.
[07:17] <G0SUB> jordi okay, sure
[07:19] <jordi> G0SUB: you tell me when you're ready.
[07:20] <G0SUB> jordi ok
[07:21] <jordi> G0SUB: you also need to provide us with the Plural Forms information for bengali. Do you know it?
[07:22] <G0SUB> jordi there must be some for bn_BD ?
[07:22] <jordi> G0SUB: for bn, I guess
[07:22] <jordi> hopefully that works equally for both?
[07:22] <G0SUB> jordi yes, that's the same
[07:23] <jordi> ok, can you tell us?
[07:23] <jordi> G0SUB: heh, the day before you asked for it we got a request for a Tamil team as well.
[07:23] <jordi> G0SUB: things are moving in India. :)
[07:23] <G0SUB> jordi heh
[07:24] <G0SUB> jordi btw, what do I put in the ``contact email address''?
[07:24] <G0SUB> I put in my email ID and it says taken?
[07:24] <jordi> leave it blank
[07:24] <G0SUB> ok
[07:25] <jordi> if you have a mailing list and so on, you could direct your email there. Without a value, you'll get the email instead
[07:25] <kiko> carlos, where can I see the dapper imports happening?
[07:25] <jordi> G0SUB: I love Indian scripts.
[07:26] <G0SUB> jordi heh, and we all love Pizza, and Pastas
[07:26] <carlos> kiko: launchpad.net/rosetta/imports
[07:27] <carlos> kiko: but it's broken atm as I notify on Monday.
[07:27] <carlos> kiko: finishing the fix today
[07:27] <kiko> the performance issue?
[07:27] <carlos> kiko: yes
[07:27] <carlos> kiko: timeouts break it
[07:29] <kiko> salgado, ping about bug 33299?
[07:29] <Ubugtu> malone bug 33299 in launchpad "foaf.nickname.generate_nick() might generate names that don't meet the valid_name constraint" [Normal,Confirmed]  http://launchpad.net/bugs/33299
[07:29] <G0SUB> jordi done ... https://launchpad.net/people/bn-in-team
[07:30] <jordi> G0SUB: eek
[07:30] <G0SUB> jordi what?
[07:30] <jordi> G0SUB: can you rename it to "ubuntu-l10n-bn-in"? :)
[07:30] <G0SUB> okay
[07:30] <jordi> G0SUB: it's the standard name ofr all the ubuntu teams.
[07:30] <G0SUB> fine
[07:30] <kiko> salgado, unping, I guess -- you seem to have a good idea of what to do.
[07:31] <G0SUB> jordi https://launchpad.net/people/ubuntu-l10n-bn-in
[07:32] <jordi> G0SUB: great
[07:32] <jordi> G0SUB: request sent
[07:33] <G0SUB> fine, thanks
[07:33] <jordi> G0SUB: hopefully lifeless can have a look when he wakes up
[07:33] <salgado> kiko, it's just the name_sanity_pattern that is broken
[07:33] <kiko> cool
[07:33] <jordi> carlos: lifeless didn't do the Akan change on IRC when I asked because IIRC he wanted to check it out with you before. Can you confirm https://launchpad.net/products/rosetta/+bug/31835 is correct?
[07:34] <Ubugtu> malone bug 31835 in rosetta "Akkadian assigned to "ak" code in ubuntu-translators group" [Major,Unconfirmed]  
[07:34] <jordi> carlos: ie, confirm in the bug log
[07:34] <jordi> G0SUB: I'm leaving office now. We'll see what the Bangladesh guys reply.
[07:34] <carlos> sure
[07:35] <G0SUB> jordi fine, thanks a lot .. I will be back again tomorrow
[07:35] <jordi> G0SUB: cool
[07:35] <jordi> laters
[07:38] <ddaa> daf: I have a variant of your failure now... except the assertEqual fails and give me two identical lists...
[07:38] <ddaa> I am utterly confused...
[07:38] <carlos> jordi: This is what we have in our database....
[07:38] <carlos> launchpad_dev=# SELECT code, englishname from language where code like 'ak%';
[07:38] <carlos>  code | englishname
[07:38] <carlos> ------+-------------
[07:38] <carlos>  ak   | Akkadian
[07:38] <carlos>  aka  | Akan
[07:38] <carlos> (2 rows)
[07:40] <jordi> carlos: ak -> Akan ; akk -> Akkadian
[07:40] <jordi> needs to be fixed
[07:40] <carlos> jordi: that needs stub to fix the DB
[07:40] <carlos> jordi: and that means that iso-codes package is broken
[07:40] <jordi> hm
[07:40] <jordi> Will file a bug on that then
[07:41] <jordi> aka     aka     ak      Akan
[07:41] <jordi> akk     akk     XX      Akkadian
[07:41] <jordi> might have been fixed later onp
[07:41] <carlos> would be
[07:42] <G0SUB> where can I find martin pitt?
[07:42] <G0SUB> what's his nick?
[07:43] <carlos> G0SUB: pitti
[07:43] <G0SUB> oh, him?
[07:43] <G0SUB> okay
[07:54] <kiko> ol tambaqui 
[08:13] <kiko> BjornT, did the work on fixing the bugwatch script fall off the radar?
[08:17] <BjornT> kiko: hmm, seems so :-/. i did create a branch for the bug, but never got around fixing it. i'll fix it now then.
[08:17] <kiko> thanks
[08:19] <Mez> er, hey
[08:19] <Mez> who do i speak to to get the default assignee for a source package changed
[08:20] <kiko> there is no default assignee
[08:20] <kiko> do you mean bug contact?
[08:20] <Mez> yeah
[08:20] <Mez> as in when bugs are opened on a package they get assigned to a team instead of just me
[08:21] <kiko> Mez, they don't get assigned to a team
[08:21] <kiko> a team may get subscribed
[08:21] <kiko> you can assign yourself however
[08:22] <Mez> ok
[08:22] <Mez> well basically
[08:22] <kiko> matsubara, can you please dupe bug 33302
[08:22] <Ubugtu> malone bug 33302 in rosetta "Error OOPS-60C360" [Normal,Unconfirmed]  http://launchpad.net/bugs/33302
[08:22] <Mez> at the moment - a certain package - I get sent emails automatically about it ... but i want the katapult team to get the emails not just me
[08:23] <kiko> Mez, so you want to change the bug contact for that certain package in ubuntu.
[08:23] <kiko> that's easy!
[08:23] <kiko> visit the source package's page
[08:23] <Mez> yeah 
[08:23] <kiko> there's a menu option for bugmail settings (I believe that's the wording)
[08:23] <kiko> unmark katapult
[08:23] <kiko> mark yourself
[08:23] <kiko> easy!
[08:35] <kiko> matsubara, please dupe bug 33203 as well -- or perhaps it's not a dupe?
[08:35] <Ubugtu> malone bug 33203 in launchpad "poll: dates are not properly validated" [Normal,Unconfirmed]  http://launchpad.net/bugs/33203
[08:36] <matsubara> kiko: duped the 33302 and will check the other now.
[08:36] <kiko> thanks
[08:48] <matsubara> kiko: I can't reproduce 33203. It seems to me that the validator is doing the right job.
[08:49] <kiko> matsubara, look at the form data?
[08:55] <matsubara> kiko: what do you mean? I'm not getting any error and when I use a invalid date, the validator gives me a nice Invalid date format. When I tried with the dates supplied on the bug report, the form changes the format to YYYY-DD-MM to the ISO format YYYY-MM-DD. I think it's a feature, isn't it?
[09:11] <matid> Hi, how many karma points do I get for a translation if I'm in the translation group responsible for that package, so I don't create suggestions only?
[09:13] <matid> From what I have found on wiki it seems that I get 2 points - one for adding a suggestion and another one for getting this suggestion approved (even if it's done automatically)
[09:14] <matid> Is it so?
[09:17] <ddaa> BjornT: pushing fixed productseries-branch-oops
[09:17] <ddaa> we are almost there guys, just three outstanding branches
[09:20] <sabdfl> lifeless: can i run a test story now?
[09:20] <kiko> sabdfl, yes.
[09:20] <sabdfl> how?
[09:20] <kiko> just python test.py -f . story-name
[09:21] <ddaa> sabdfl: https://wiki.launchpad.canonical.com/LaunchpadHackingFAQ#head-b454516710233a7eee2eb4b44d7bc6e4a97e04a3
[09:21] <kiko> so if it is something like ticket-tracker
[09:21] <sabdfl> thanks guys
[09:21] <kiko> python test.py -f . ticket-tracker
[09:23] <BjornT> ddaa: ok, tell me when it's fully pushed.
[09:25] <ddaa> I'm going to look at spivs work on the vcs-imports-pull-list, maybe I can finish the important bits tonight
[09:27] <sabdfl> ddaa: question for you
[09:27] <matid> Do you guys know why is this bug happening: https://launchpad.net/products/rosetta/+bug/31146
[09:27] <Ubugtu> malone bug 31146 in rosetta "Too many fields in Polish translation of Ubuntu Documentation (quicktour)" [Normal,Unconfirmed]  
[09:28] <matid> I've found it in other packages too
[09:30] <ddaa> sabdfl: ?
[09:30] <matid> Or maybe should I just ignore it and fill every field with the same translation?
[09:31] <sabdfl> ddaa: sorry, distracted :-)
[09:31] <sabdfl> how much do you know about svn and cvs commit messages?
[09:31] <ddaa> BjornT: NOW
[09:31] <sabdfl> is there a place to put arbitrary, interested metadata in there?
[09:31] <BjornT> ok, i'll start reviewing it soon.
[09:32] <matid> I see I'm interruping, nvm
[09:32] <ddaa> sabdfl: SVN has revision properties, or so I heard
[09:32] <ddaa> They have some sort of very limited use routinely. I do not remember what.
[09:34] <ddaa> sabdfl: I do not know of CVS having any such metadata, if that's important, I think I can find a better answer by looking up the protocol docs.
[09:35] <ddaa> Though, thinking of it, some people manage to piggyback metadata on cvs commits. Some recent cvs servers actually store a commit id to identify file commits that were created as part of the same commit operation.
[09:35] <ddaa> (i.e. actually storing full-tree commit information)
actually CVS does not really have revision model at all, so you can say many different things depending on your perspective. Some people says it Conceivably Versions Something</rant>
[09:37] <sabdfl> ok, what about SVN?
[09:37] <sabdfl> here's the real question
[09:38] <sabdfl> fast forward to when we have SVN imports smooth
[09:38] <sabdfl> we will be publishing a bzr branch that has a revision for each revision in the SVN branch, right?
[09:38] <ddaa> I'll look up the specifics. I'm not sure I'm not confused.
[09:38] <ddaa> righty (insofar SVN has branches at all)
[09:39] <sabdfl> trunk
[09:39] <sabdfl> or stable branch
[09:39] <sabdfl> now, people bzr branch off those branches
[09:39] <sabdfl> and work offline
[09:39] <sabdfl> in teams
[09:39] <sabdfl> bound branches
[09:39] <sabdfl> whoooot!
[09:39] <sabdfl> they do something interesting
[09:39] <sabdfl> and they want to land it on the SVN trunk
[09:40] <sabdfl> i'd like you to think about finding a way that they could encode the bzr information into that landing metadata, so that, when our svn importer comes along, it can sanely reflect the full cycle that has happened
[09:40] <sabdfl> see, the bzr code branched from a revision, and it remembers that
[09:40] <sabdfl> when it lands in svn, it can leave everything that we need to know so that we can make that svn branch look just like a bzr branch
[09:41] <sabdfl> if someone else continues working on that branch, and merges from the bzr trunk, they should see that many of their revisions have actually landed on trunk
[09:41] <sabdfl> make sense?
[09:41] <sabdfl> specifically for svn
[09:41] <sabdfl> i'm not concerned about cvs (at this stage ;-)
[09:42] <sabdfl> the reason i am asking about this is as follows
[09:42] <sabdfl> python has switched upstream TRUNK to SVN
[09:42] <sabdfl> we will import and publish that
[09:42] <ddaa> I think that would be worth investigating. I can already think of one clear corner case, and I'm certain there are more.
[09:42] <ddaa> I do like the plan.
[09:42] <sabdfl> it would be nice if the folks who use our bzr branches can do so completely - landing code
[09:42] <sabdfl> ok, we're on roughly the same page then
[09:42] <sabdfl> no need for action now, but bear it in mind
[09:43] <sabdfl> we can talk more at the sprint
[09:43] <sabdfl> ddaa: will you file a braindump spec, called SvnLaundromat, and subscribe me?
[09:43] <sabdfl> copy this conversation into it
[09:43] <sabdfl> no need for further work
[09:43] <ddaa> I very much like anything that allow bzr to "subvert" projects by leveraging the vcs-imports.
[09:44] <ddaa> I think I'll give it a more descriptive name, or mpt is going to hurt me for creating MORE obscure names, but ok.
[09:45] <ddaa> Something like BzrRoundtripSvn
[09:46] <sabdfl> sounds good
[09:48] <ddaa> I think we are already basically in sync on the big picture. The details would need to be hammered with several days of deep speccing in a quiet place.
[09:48] <ddaa> (that is _not_ on a sprint)
[09:48] <ddaa> also, the svn import support itself needs to go a long way before we can think of that. For example, we do not support renames yet.
[09:50] <elmo> can sqlobject's select() be passed a column to distinct on?
[09:53] <salgado> elmo, no, the distinct argument takes only True or False
[09:53] <elmo> meh
[09:54] <elmo> salgado: thanks
[09:55] <salgado> np
[10:00] <ddaa> sabdfl: you are subscribed https://launchpad.net/products/launchpad/+spec/bzr-roundtrip-svn/
[10:01] <sabdfl> danke
[10:12] <elmo> sigh
[10:15] <elmo> does anyone know what the API change in the librarian was recently, or can remember something that would help me find the thread in the ml archives?
[10:19] <ddaa> elmo: maybe it is spiv/launchpad/librarian-database-agreement
[10:20] <lifeless> ddaa: we dont? I thought we did support svn renames
[10:21] <ddaa> AFAIK, only as delete-create
[10:21] <elmo> ddaa: hum?
[10:21] <ddaa> elmo: hip shooting, that's the most recent ML traffic (in launchpad-reviews) that suggest that something might have changed in the librarian
[10:21] <ddaa> Not much substance though.
[10:22] <ddaa> So, short have having a librarian guru around, I'd look at the changes introduced by that branch.
[10:24] <ddaa> lifeless: because a rename in svn is really copy+delete, so some whole-changeset logic is required to actually do a copy or a rename.
[10:24] <ddaa> and that's just with single file renames
[10:24] <ddaa> with directories, there's a whole other can of worms
[10:26] <lifeless> so.. 
[10:26] <lifeless> svn loses information here
[10:26] <lifeless> we support copied file
[10:26] <lifeless> s
[10:26] <ddaa> as adds
[10:27] <lifeless> how does svn support reames ?
[10:27] <ddaa> I'm not 100% sure (say, 80%), but I believe it's a add with copy_from (a copy) and a delete.
[10:28] <ddaa> you know, svn has his wacky notion of tracking file copying
[10:28] <lifeless> yes
[10:28] <ddaa> from which they derive these thingy they call "branches".
[10:28] <lifeless> well
[10:28] <lifeless> they have a big hammer
[10:29] <lifeless> and they have built up a workable system from it
[10:29] <lifeless> yes they are A+D pairs to do a rename 
[10:30] <lifeless> but they dont have a 'rename' in the db (at least for the api versions I learnt)
[10:30] <ddaa> lifeless: I checked the ChangesIterator.py, my recollection is correct. We do not synthesize renames.
[10:30] <lifeless> right
[10:30] <lifeless> when you say 'support renames for svn' what do you mean ?
[10:31] <ddaa> many many levels there
[10:32] <carlos> I need an expert that knows about GeneralFormView
[10:32] <carlos> BjornT: around?
[10:32] <BjornT> carlos: yes, i'm around
[10:32] <ddaa> simplest level is "traverse all changes in a revision, group copy from _file_ A with delete A to synthesize a file rename, bail out if something nasty occurs (e.g. multiple copies)"
[10:33] <carlos> BjornT: do you have sometime to help me with a GeneralFormView?
[10:33] <elmo> whee... ltraceing a launchpad script != clever
[10:34] <lifeless> elmo: !!!
[10:34] <lifeless> elmo: whats wrong ?
[10:34] <elmo> 4 minutes of wall time and 90M of output later, it's still doing python importing AFAICT
[10:34] <BjornT> carlos: sure
[10:34] <ddaa> the most complex level would look like something "process directory copies and deletes, combine them with one another (for simultaneous renames of nested dirs), combine them with file renames.
[10:34] <ddaa> When multiple copies of the same file happen on the same revision, make an educated guess of which one should be a rename".
[10:35] <lifeless> elmo: I meant 'why do you need to do that'
[10:36] <carlos> BjornT: I'm having problems fixing https://launchpad.net/products/rosetta/+bug/29944
[10:36] <Ubugtu> malone bug 29944 in rosetta "Import queue: ForbiddenAttribute: pofile oopses when importing gnome-panel for Georgian" [Normal,In progress]  
[10:36] <carlos> BjornT: you should have the code on your tree
[10:36] <ddaa> And then there's this thing about copies from other branches, where we ideally should be able to use consistent file ids.
[10:37] <elmo> lifeless: I thought it'd be an easy way to see the http conversation between me and the librarian
[10:38] <lifeless> elmo: doh!
[10:38] <carlos> BjornT: your code lacks the pofile attribute on the Interface
[10:38] <lifeless> tethereal is my favourite toy for that
[10:39] <carlos> BjornT: and I added it to my branch 
[10:39] <elmo> yeah, unfortunately this is on ftp-master, so it does a lot of talking to the librarian
[10:39] <lifeless> whats going wrong ?
[10:40] <carlos> BjornT: the problem is that if the interface has it as (pofile = Attribute...) the error is still there. I need to use Int or Choice to be able to use self.context.pofile
[10:40] <elmo> I'm getting 503'd trying to download a file, in a script which worked last week
[10:40] <carlos> BjornT: and I need to understand it...
[10:40] <lifeless> thats a server error
[10:41] <ddaa> lifeless: I need to focus on fixing branch-pull-list.txt for vcs-imports, spiv appears to have stalled on that :(
[10:41] <lifeless> are you talking to the librarian public interface or to the control interface ?
[10:41] <lifeless> spiv: ping
[10:41] <lifeless> ddaa: he should be around immenintley.
[10:41] <lifeless> ddaa: I will ask him. How are vcs-imports different to push branches in this context ?
[10:42] <ddaa> different path to synthesize
[10:42] <elmo> lifeless: I'm just using LibrarianClient
[10:42] <ddaa> lifeless: push branches are on local filesystem, vcs-imports are on some remote host
[10:42] <lifeless> elmo: is your lp source tree up to date? we put a version signature-enforcing check into librarian
[10:42] <lifeless> ddaa: and theres some flag to detect this ?
[10:42] <elmo> lifeless: for download?
[10:42] <BjornT> carlos: hmm, that sounds strange. what interface is this?
[10:43] <elmo> lifeless: and no, my source tree is the soyuz branch, it's probably unamusingly out of date
[10:43] <lifeless> elmo: anything that talks to the control interface. I am guessing that your script talks to the control interface
[10:43] <elmo> a branch of the soyuz branch
[10:43] <elmo> lifeless: LibrarianClient() talks to the control interface?
[10:43] <ddaa> lifeless: we agree to use branch.owner=vcs_imports as in interim measure, then change the code to use Branch.origin_type (or something like that) enum of PULL, PUSH and IMPORT.
[10:43] <carlos> BjornT: translationimportqueue.py
[10:44] <lifeless> ddaa: ok. I have paged this in. leave it with me and expect an email overnight
[10:44] <carlos> BjornT: you have there an IEditTranslationImportQueue interface that is used to render the form
[10:44] <carlos> BjornT: but the actual interface is ITranslationImportQueue
[10:44] <lifeless> elmo: what methods on LibrarianClient do you call ?
[10:45] <ddaa> lifeless: not a very complicated patch, most of it would be testing. You'll need the sampledata in my productseries-branch-oops branch and the rename in spiv's rename-buttsource (both in BjornT's review queue).
[10:45] <BjornT> carlos: and you added pofile to ITranslationImportQueue, right?
[10:45] <elmo> lifeless: just getFileByAlias()
[10:45] <carlos> BjornT: right
[10:45] <kiko> bug 6026?
[10:45] <Ubugtu> malone bug 6026 in malone "Oops from changing bug's product when milestone is set" [Normal,In progress]  http://launchpad.net/bugs/6026
[10:46] <carlos> BjornT: atm I merged both interfaces but I don't think that's needed
[10:46] <carlos> I tried a lot of things until detected where the problem is
[10:46] <ddaa> lifeless: but the fact is that I'm still far from my time quota today, and I have nothing else to work on right now :)
[10:46] <lifeless> elmo: for debugging, print out the result of librarianclient.getUrlForAlias(aliasid) 
[10:46] <ddaa> (though, I can work in advance for next week)
[10:47] <lifeless> ddaa: if spiv has agreed to do it, and you have items that are not blocked yet, I would work on those
[10:48] <lifeless> elmo: you should be able to get the url that spits out
[10:48] <lifeless> elmo: s/get/wget/
[10:49] <lifeless> elmo: and no, AFAICT that code path should not be touching the control path
[10:49] <elmo> it's getting worse, now I'm  getting -ECONN, WTF is going on
[10:50] <lifeless> want me to phone-ping spiv ? is 8:50 so not unreasonable
[10:50] <elmo> nah, it's almost certainly me doing something stupid
[10:50] <BjornT> carlos: i don't know why it doesn't work, it whould work with an Attribute. if you mail me your diff and explanation how to reproduce the error, i can take a look at it tomorrow.
[10:50] <elmo> I'll keep banging my head against the desk
[10:50] <lifeless> ok
[10:50] <lifeless> use a towel, bloodstains on wood are a biatch
[10:50] <carlos> BjornT: I suppose it's ok if I use 'Int' in the mean time, right?
[10:51] <carlos> we are not rendering that field anyway
[10:52] <ddaa> lifeless: ack
[10:52] <BjornT> carlos: is it possible to use a Choice field? (that is, is there a suitable vocabulary for it). if so, use a Choice field, otherwise an Int field with an XXX comment.
[10:52] <ddaa> anyway it looks like I might go offline with -EPARTNER pretty soon tonight
[10:52] <carlos> BjornT: no, we don't have a vocabulary for it, that's why I'm suggesting to use an Int
[10:53] <carlos> BjornT: will do it that way.
[10:53] <carlos> BjornT: thanks
[10:55] <BjornT> kiko: could you do a quick review of the fix for bug 32600? https://chinstrap.ubuntu.com/~dsilvers/paste/filew8J9Ac.html
[10:55] <Ubugtu> malone bug 32600 in malone "checkwatches.py blows up if an Bugzilla instance with an unsupported version is encountered." [Normal,In progress]  http://launchpad.net/bugs/32600
[10:56] <kiko> BjornT, maybe "UnsupportedBugTracker", period?
[10:56] <kiko> I mean, you could potentially reuse this for other cases
[10:57] <kiko> BjornT, other than that, r=kiko
[10:57] <BjornT> kiko: well, you mean combining this new exception with UnknownBugTrackerTypeError?
[10:57] <kiko> BjornT, yes.
[10:59] <BjornT> kiko: hmm, i'd rather have two different exceptions. in one case we know nothing about the bug tracker, and will be hard to fix the problem. in the other, where we support the bug tracker, but not this particular version, it's easier to fix.
[10:59] <elmo> what's a good sys.exit(0) type thing to forcibly ensure I'm running/importing the code I think I'm importing?
[11:00] <lifeless> os._exit
[11:00] <lifeless> thats a hard DO IT NOW.. REALLY call
[11:01] <kiko> BjornT, reasonable. r=kiko
[11:01] <lifeless> ok, 0900, time for more knitting, see you all on the flip side
[11:01] <BjornT> thanks
[11:02] <elmo> lifeless: thanks
[11:03] <elmo> Q#%$YG!#%GHQ#W
[11:22] <ddaa> BjornT: review? please?
[11:23] <BjornT> ddaa: already sent you the review
[11:24] <ddaa> one minute ago :)
[11:25] <ddaa> BjornT: looks like all the things you commented one are bad conflict resolutions
[11:25] <ddaa> I'll double check them. Is that approved otherwise?
[11:26] <BjornT> yeah, i thought so. yes, it's approved.
[11:26] <ddaa> okay, then it's blocked on spiv's rename branch.
[11:26] <ddaa> I'll try and address the review for it right now.
[11:28] <ddaa> BjornT: can the production rename thing be done through the UI?
[11:29] <ddaa> If no, I'll make a sql patch instead of patch on current.sql
[11:31] <BjornT> ddaa: i'm not sure. since there's code that depends on the name, if you change it before the rollout, things might break. and if you rollout with having change the name, maybe things will break so that you can't change the name through the UI.
[11:32] <BjornT> ddaa: a patch is probably the way to go, you could ask stub what he thinks. you still need to patch current.sql, though.
[11:32] <BjornT> the database patches are run before current.sql is loaded
[11:32] <ddaa> ha, right, so the actual data changes become no-ops
[11:33] <ddaa> I'll do it right now. It's blocking me.
[11:36] <ddaa> mh... stub not here... :(
[11:43] <sabdfl> who did the cleanup to prevent add and edit forms rendering?
[11:43] <sabdfl> very good work
[11:43] <sabdfl> cleans up the page tests nicely
[11:43] <BjornT> sabdfl: thanks :)
[11:43] <sabdfl> night, then :-)
[11:45] <spiv> lifeless: pong
[11:46] <spiv> lifeless: I was waiting for daf's branch to merge, which it now has, so it won't take me long now.
[11:46] <spiv> (otherwise there were three branches with overlapping changes to this code)
[11:47] <ddaa> spiv!
[11:47] <spiv> ddaa: Good morning
[11:48] <ddaa> spiv: as I said to lifeless, you'll need the sampledata in my productseries-branch-oops branch, which depends on rename-buttsource
[11:49] <sabdfl> hey spiv
[11:49] <sabdfl> can i ask for a quick update on supermirror push / pull?
[11:50] <spiv> sabdfl: It's exactly one review comment away from merging, which will get solved today.
[11:50] <sabdfl> damn, you're good :-)
[11:50] <ddaa> spiv: here would be a plan: you do the db patch for rename-buttsource, I do the cleanups in productseries-branch-oops, you merge both branches, then do the pull-list update.
[11:51] <ddaa> There's also a test case which was modified by productseries-branch-oops that will need updating with pull-list, so you'll have to include productseries-branch-oops anyway.
[11:51] <ddaa> spiv: do you think you can handle that today?
[11:52] <spiv> ddaa: Yep, definitely.
[11:52] <ddaa> Then we have the launchpad meeting and say "everything is in place, bzr imports go live monday"
[11:52] <spiv> ddaa: I like that plan :)
[11:53] <ddaa> (unless I forgot something along the way, which is not all that unlikely considering the number of moving parts we've been juggling with)
[11:54] <spiv> Hmm, weird merge conflict.
[11:55] <ddaa> spiv: it does not seem to occur when merging the other way around
[11:56] <spiv> ddaa: When merging rename-buttsource into rocketfue.
[11:56] <ddaa> yup, I had a weird bunch of conflicts as well...
[11:56] <spiv> Just one conflict:
[11:56] <spiv> bzr: WARNING: Conflict adding file lib/canonical/launchpad/doc/validation.txt.  Moved existing file to lib/canonical/launchpad/doc/validation.txt.moved.
[11:56] <ddaa> ha, yes, weird
[11:56] <spiv> But the only revision merged didn't touch or add that file.
[11:57] <ddaa> I noticed that, that indeed looks like a bug
[11:57] <ddaa> it just waaaaay below by troubling threshold ATM :)
[11:57] <spiv> Oddly enough, the pending-reviews page doesn't have this issue.
[11:57] <spiv> So I guess it might be a bug in newer bzrs?
[11:57] <ddaa> Probably a bug in the new fancy merge stuff from abentley.
[11:58] <ddaa> or a bug in our data that's uncovered by the merge stuff
[11:58] <spiv> Right.
[11:58] <spiv> I'll ask #bzr.