[12:41] <dilys> Merge to rocketfuel@canonical.com/launchpad--devel--0: [r=BjornT]  implement a BugTaskAssigneeWidget, which makes it easier to 'take' a bug (patch-2275: brad.bollenbach@canonical.com)
[02:56] <ddaa> new original TDD dilemma
[02:56] <ddaa> I get a compulsion to write a test
[02:56] <ddaa> and it fails
[02:57] <ddaa> yet, I'm unable to say what exactly the test is doing...
[02:57] <ddaa> probably means I need to call it a day...
[03:28] <dilys> Merge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  fix branches presentation on product page to make ddaa happy (patch-2276: mpt@canonical.com)
[09:07] <stub> BjornT: Are you crawling the linkchecker report, or finding these bugs independantly?
[09:13] <BjornT> stub: i looked through the linkchecker report and filed bugs on some occurring errors.
[09:14] <stub> BjornT: Ta - saved me from doing something similar ;)
[09:14] <BjornT> stub: np :)
[09:15] <stub> Ahh... just got your email
[09:54] <sabdfl> stub: thanks for the approval
[09:55] <sabdfl> can i ask for another quickie? 
[09:55] <sabdfl> ALTER TABLE Bounty DROP COLUMN duration;
[09:55] <sabdfl> ALTER TABLE Bounty DROP COLUMN difficulty;
[09:57] <sabdfl> actually, i'll keep the difficulty one
[10:04] <sabdfl> ping: anybody?
[10:09] <stub> sabdfl: Fine by me. Stick it in the existing patch if you like.
[10:34] <carlos> morning
[10:38] <carlos> sabdfl, the new launchpad front page rocks
[10:43] <carlos> morgs, hi, around?
[12:18] <morgs> carlos: I'm back...
[12:20] <carlos> morgs, I think I found a bug on the Register
[12:20] <carlos> morgs, https://staging.ubuntu.com/products/wordpress/1.3-rc07012005PTBR
[12:20] <carlos> https://staging.ubuntu.com/errors/showEntry.html?id=1124446832.290.621101847901
[12:21] <morgs> carlos: thanks, checking...
[12:21] <carlos> morgs, is it know / fixed?
[12:21] <morgs> carlos: no, I have not seen this before
[12:21] <carlos> ok
[12:22] <morgs> thanks!
[12:22] <carlos> morgs, where is "the register" product?
[12:22] <carlos> morgs, shoul I use launchpad directly?
[12:23] <carlos>  s/shoul/should/
[12:23] <morgs> carlos: yes, file it against launchpad.
[12:23] <carlos> ok
[12:24] <carlos> morgs, it works on production
[12:24] <carlos> so seems like it's a problem with the development branch and a missing pagetest...
[12:24] <morgs> yes I see - OK, so it's a recent change
[12:26] <carlos> morgs, https://launchpad.net/malone/bugs/1810
[12:27] <morgs> carlos: thanks, I'll check it out right now
[01:06] <Kinnison> So, is there a way to get the launchpad test suite to stop at its first error?
[01:08] <carlos> Kinnison, no idea
[01:08] <Kinnison> argh
[01:10] <dilys> Merge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  Make request_country handle nested proxies and add tests (patch-2277: stuart.bishop@canonical.com)
[03:30] <ddaa> Anybody here, _please_ _please_ review some of the cscvs branches on the review page
[03:31] <ddaa> There are like two or three months worth of changes and coding style cleanups pending.
[03:35] <BjornT> ddaa: i think i'll have time for some reviews today, just have to finish what i'm doing now
[03:39] <ddaa> Mh... actually that was a bit of an overstatement
[03:40] <ddaa> BjornT: thanks, ATM I cannot even make useful submission branches for my new work
[03:43] <BjornT> ddaa: np. i was planning to do that anyway, quite a lot of branches are in need of reviewing...
[03:50] <Kinnison> Hmm, I can't seem to login to launchpad
[03:50] <ddaa> It worked about one hour ago for me.
[03:51] <Kinnison> Hmm, third time lucky
[03:51] <Kinnison> bizarre
[03:53] <BjornT> ddaa: well, it's more like "should take time to do reviews" :P
[04:20] <ddaa> anybody knows a CVS with a file that has its description set?
[04:21] <bradb> morning
[04:22] <bradb> ddaa: Tip for PendingReviews: Don't use the General Queue.
[04:23] <ddaa> bradb: I have two other possible algorithms
[04:23] <ddaa> 1. dogpile on spiv, since he's the one most familiar with this code
[04:23] <ddaa> 2. use a pseudorandom generator
[04:23] <ddaa> any better suggestion?
[04:24] <bradb> No. I used algorithm #1 for fmt:approximateduration.
[04:25] <ddaa> bah, I'll see how BjornT does... I'm trying to have spiv do some actual coding on importd, so flooding him with reviews does not make much sense.
[04:28] <bradb> BjornT does good, thorough code review, IME.
[04:47] <mpt> carlos: ping
[04:47] <carlos> mpt, pong
[04:49] <mpt> carlos: If you're here when stub next comes online, could you ask him to cherry-pick the bar-charts fix?
[04:49] <carlos> It's better to send it by email
[04:49] <carlos> I will do it now
[04:49] <mpt> Eh, I suppose this is what e-mail is for, right
[04:49] <mpt> thanks
[04:57] <dilys> Merge to rocketfuel@canonical.com/launchpad--devel--0: Implement TeamsInAuthserver. [r=salgado]  (patch-2278: andrew.bennetts@canonical.com)
[05:34] <bradb> mpt: Do we yet have a way to display a priority-dependant icon for a bug task?
[05:37] <mpt> bradb: No, the bug to allow that is assigned to you :-)
[05:38] <mpt> bradb: https://launchpad.net/malone/bugs/1736
[05:40] <bradb> Cool, I'll do that now.
[05:41] <ddaa> duh
[05:41] <bradb> Maybe I should call it fmt:icon
[05:41] <ddaa> pointless python snippet of the day
[05:41] <bradb> task/priority/fmt:icon
[05:41] <bradb> mpt: What do you think?
[05:41] <ddaa>         for branch in self.branch_hash.values():
[05:41] <ddaa>             yield branch
[05:42] <ddaa> either use iterators, or don't, but this is just pointless
[05:54] <carlos> mpt, btw, stub is already on weekend, not sure if he will be able to do the cherry pick...
[05:57] <mpt> carlos: So it'll be in the Monday rollout, then
[05:58] <mpt> bradb: seems reasonable, though I'm not the world's expert on fmt:
[06:02] <carlos> mpt, I think so
[06:02] <carlos> mpt, I suppose I could ask stub for it, but I don't think is so urgent to ask him to work outside normal hours
[06:18] <kiko> bradb-lunch, I wonder if SteveA would think that fmt:icon is not as good as idea as the "magic rendering box"
[06:19] <sivang> hi all
[06:21] <kiko> hey sivan
[06:22] <mpt> kiko: That depends on how flexible the magic is
[06:23] <sivang> hey kiko , 'sup? I wonder what are those "tales" you are talking about ? :-) 
[06:23] <kiko> sivang, they are like bad dreams, except worse 
[06:24] <sivang> OMG :) Now really, what role do they serve in launchpad's system?
[06:24] <mpt> kiko: The icon could be next to a bug number + description (in three different table cells!), or to a context (on the bug page)
[06:25] <kiko> I see
[06:25] <kiko> sivang, they are used in the page templates, roughly
[06:26] <mpt> Tales of woe
[06:26] <mpt> Tales of joy
[06:26] <mpt> Tales of mystery and imagination
[06:26] <sivang> mpt:  :-) 
[06:27] <sivang> k guys, ncie to know YATAL (yet another thing about launchpad) , I'm off again now , laterz all
[06:27] <kiko> lol
[06:28] <sivang> pretty nice watching it now, that the "revolution" is almost done :)
[06:29] <mpt> "almost done"?
[06:30] <mpt> Did I sleep in last night, and wake up in 2011? :-)
[06:30] <sivang> mpt: ah, that was more of a wishful thinking, but it's certainly on the right track :)
[06:30] <mpt> yes
[06:30] <sivang> mpt: hehehe
[06:31] <sivang> The fact that I am chatting with you people in this channel, is actually big part of it.
[06:31] <mpt> Yes, in the plan for world domination, winning over sivang is half the battle
[06:32] <sivang> hehehe :)
[06:32] <mpt> :-)
[06:34] <kiko> carlos?
[06:35] <carlos> kiko, ?
[06:35] <kiko> carlos, tell me about the migration script on production :)
[06:36] <carlos> kiko, stub told me that it worked on staging without major problems
[06:36] <carlos> and I suppose it's being executed atm on production
[06:36] <kiko> carlos, are we good for a release on wednesday?
[06:36] <carlos> kiko, If the script has been executed on production, we can do it on Monday
[06:37] <kiko> I'd like us to do some serious QA monday and tuesday
[06:37] <carlos> kiko, ok
[06:37] <kiko> however
[06:37] <kiko> is production "current"?
[06:38] <carlos> current == all code needed for 1.0?
[06:38] <carlos> yes
[06:38] <carlos> more than two weeks ago
[06:38] <kiko> that's not what I meant
[06:38] <kiko> I am concerned that wednesday is going to be post-rollout
[06:39] <kiko> (since we roll out tuesday)
[06:39] <kiko> does that make sense, carlos?
[06:40] <carlos> kiko, if we do all testing on staging
[06:40] <carlos> kiko, that should be ok
[06:40] <carlos> you wanted to test it on monday and tuesday
[06:41] <kiko> I'm thinking now tuesday and wednesday
[06:42] <kiko> I would much prefer testing production as much as possible
[06:42] <carlos> kiko, production update will be on Tuesday
[06:42] <kiko> we can of course test staging monday, production tuesday (after stub's rollout) and then release wednesday
[06:42] <carlos> not Monday
[06:42] <kiko> I know
[06:42] <carlos> kiko, it sounds ok for me
[06:43] <kiko> I hadn't thought of the update angle when I suggested it at first
[06:43] <kiko> okay cool
[06:43] <kiko> we can call for a Rosetta QA day on monday on staging if you like
[06:43] <kiko> what do you think?
[06:45] <carlos> kiko, only launchpaders have access to staging
[06:46] <carlos> I think it's better do it on Tuesday on production (and do some extra test on Monday using staging)
[06:46] <carlos> de todas formas, no han habido cambios significativos esta semana
[06:46] <carlos> hmmm
[06:46] <carlos> kiko, sorry, my brain....
[06:47] <sivang> mpt: are you the one responsible for LaunchpadIntegrationHelpPage ?
[06:47] <carlos> kiko, we didn't change anything big this week
[06:49] <kiko> yeah
[06:49] <mpt> sivang: Yes
[06:50] <sivang> mpt: would you like to discuss some of it next week? I'd like to help with it
[06:52] <mpt> sivang: Sure. Apparently it's too complicated, and should just be a wiki-like page for now
[06:53] <sivang> mpt: k, cool, I'd like also to experiment with having the apps fetch the info and possibly store locally in the app etc.. but we'll discuss next week, I presume if you have time 
[06:55] <ddaa> kiko: what do you think of the list of open issues I made per your request?
[06:56] <mpt> sivang: That'd be great
[07:02] <sivang> mpt: cool, laterz.
[07:20] <bradb> kiko: will the magic rendering box be working within the next week?
[07:21] <bradb> sounds like something that'll happen at some yet undetermined future date
[07:40] <ddaa> cscvs contains the weirdest bits of python ever...
[07:40] <ddaa> fparser.parseBranches("1.1", string.rfind("1,1", "."))
[07:42] <bradb> mpt: btw, i think I'm going to make it bugtask/fmt:icon, to be more consistent with fmt:url (I imagine, afterall, that it only makes sense for the little bug icon to be displayed in a consistent fashion everywhere, i.e. that we wouldn't expect to be able to do crazy things like bugtask/priority/fmt:icon, bugtask/status/fmt:icon, etc.)
[07:42] <bradb> Even if Launchpad sprouts another magic black box, this little pieces will form good building blocks, IMHO.
[07:44] <bradb> mpt: What's the expected behaviour if priority is Wontfix?
[07:44] <bradb> (for the time between now and when we drop that priority)
[07:59] <kiko> bradb, no, it won't -- I think you're doing the right thing, but I wanted to point out the magic rendering box angle
[08:00] <bradb> fair enough, good point
[08:02] <dilys> Merge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  bounty polish (patch-2279: mark.shuttleworth@canonical.com)
[08:07] <Kinnison> ciao dudes
[08:07] <kiko> ahoy Kinnison 
[08:07] <kiko> we'll touch base mondy
[08:07] <Kinnison> coolio
[08:07] <Kinnison> have a good w/e
[08:07] <kiko> it ended up that I had a ton of things to look at
[08:08] <kiko> (inbox out of control too :-()
[08:08] <bradb> I've got fmt:icon's tests passing here, and working as expected in the MaloneSearchResults-style sp bug listing: http://69.70.209.33:8086/distros/debian/woody/+sources/mozilla-firefox/+bugs (the listing itself is not done yet, of course; I have to add IBugTask.statusdisplay, and possibly the summary or a snippet of the description, if no summary, to the listing.)
[08:08] <bradb> mpt: ^^
[08:08] <kiko> the summary, definitely
[08:08] <kiko> mpt, will have a format for you, surely
[08:09] <bradb> interesting, mpt didn't show a summary in MSR
[08:09] <bradb> er, interesting*ly*, i meant
[08:09] <kiko> we'll just have to prod him into it :)
[08:09] <bradb> yeah :)
[08:09] <kiko> he'll be down in a minute
[08:09] <bradb> ok
[08:15] <kiko> bradb, I was also going to say you should try and keep the interface for the search listings template as similar as possible to the table layout template
[08:15] <kiko> so that we can offer both views if we want
[08:15] <kiko> by interface think template API 
[08:16] <mpt> bradb: Use ++resource++bug for wontfix, I guess
[08:16] <mpt> bradb: Summary is also in SimplifyingMalone
[08:21] <sabdfl> so dudes, how's it going?
[08:21] <sabdfl> my little fingers are buring from that bounty polish
[08:21] <sabdfl> burning, even
[08:21] <sabdfl> kiko: this w/e i'm going to work on a little present for you in LP
[08:21] <sabdfl> i'll be travelling sun-wed, so it may not land next week
[08:21] <sabdfl> but you're going to like it, i think
[08:23] <kiko> I'm looking at the UBZ plan
[08:23] <kiko> everytime baz builds a pristine tree I stop work for 30 minutes
[08:28] <bradb> kiko: similar interface, right. as per an earlier discussion with mpt, I'm doing my best to prepare for the possibility of riots when the table view is gone (i.e. not "removing" any of the table view code, and hopefully keeping it easy to go back to that layout as an option when people start screaming.)
[08:30] <kiko> bradb, I'd like to have the option, to be honest. for one, the table layout is the only one which currently support client-side sorting
[08:30] <kiko> in my dream world it would be a matter of swapping a single line in the callsite
[08:31] <kiko> can that be made possible or is it crack?
[08:31] <bradb> I'd like to have the option too. How do we reconcile the difference in vertical real estate usage between the list and table view? Show portlets on both sides for list view and only on the right side for table view?
[08:31] <bradb> kiko: with a little bit of ninjaneering, to be sure
[08:34] <kiko> bradb, you mean horizontal real estate.
[08:34] <kiko> hmmm
[08:35] <kiko> well, that's a secondary issue to me
[08:35] <kiko> if the API is kept exactly the same I can work on that later
[08:35] <kiko> if the API diverges however, it's going to be hell to start and that makes the work less interesting
[08:35] <kiko> @#@!#@
[08:37] <bradb> yeah
[08:39] <bradb> before i submit for review, i'll make sure it's only a one-line change to switch between table or list
[08:40] <kiko> thanks
[08:44] <bradb> BjornT: Would you mind taking on DistroReleaseCVEReport?
[08:45] <BjornT> bradb: sure, i can do that
[08:45] <bradb> the untriaged bugs and (a new version of the) distro release bugs listing has to be doled out too, but i'd rather wait till MaloneSearchResults is merged to think about who should do what there
[08:45] <bradb> BjornT: great, thanks
[09:39] <carlos> andandare, hi?
[10:05] <kiko> does anyone know ian bicking?
[10:05] <bradb> it's been quite a while since i corresponded with him
[10:09] <andandare> hi carlos
[10:09] <carlos> andandare, hi
[10:09] <kiko> bradb, what's he like
[10:11] <bradb> Seemed like a nice enough guy. He's got a blog and he's not afraid to use it.
[10:11] <andandare> carlos: well, i had tried before to have a private chat with you, may I now?
[10:11] <kiko> bradb, yeah, I've read it. he's always idle on IRC, though
[10:12] <carlos> andandare, seems like my router does not accepts dcc chats, lets try just a query...
[10:13] <bradb> kiko: At a guess, I'd say he's easier to reach by email.
[10:15] <andandare> carlos:ok.
[11:00] <bradb> mpt: http://69.70.209.33:8086/distros/debian/woody/+sources/mozilla-firefox/+bugs -- I've got basically all the data in place now; would you be interested in tuning up the CSS so that the display doesn't suck?
[11:05] <bradb> i.e. you can branch and merge from me, if you want
[11:08] <cyphase> hey everyone
[11:10] <mpt> bradb: sure, e-mail me the branch details and I'll look at it on Sunday/Monday
[11:11] <bradb> sounds good, thanks
[11:11] <mpt> bradb: The table view has never included the Summary field, so one step to "reconcil[ing]  the difference in vertical real estate usage between the list and table view" would be to not include it in the list view either. :-)
[11:11] <mpt> Most lists of results will be much longer than that example page.
[11:11] <bradb> s/vertical/horizontal/
[11:12] <bradb> i don't get it, how does that reconcile the difference?
[11:13] <bradb> the list view is much narrower than the table view, summary or not
[11:13] <bradb> or maybe you actually meant vertical, where i meant horizontal, even though i said vertical
[11:13] <bradb> :P
[11:15] <kiko> is PQM happy or is it wedged?
[11:15] <bradb> mpt: after i'm done status/commit, I'll email you the details. so in about 30 mins.
[11:16] <dilys> Merge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  fix a few trivialities in the Rosetta POParser that hamper imports, order the translation barchart in descending mode when clicking on the barchart column, and fix bug 1678: Sorting bugs by source package name produces a system error. O PQM please accept this young virgin and process my modification request? (patch-2280: christian.reis@canonical.com)
[11:16] <bradb> you got lucky
[11:22] <kiko> woot
[11:22] <kiko> bradb, question for you
[11:22] <kiko> AttributeError: 'SimpleViewClass from /home/kiko/devel/rocketfuel2/' object has no attribute 'schema'
[11:22] <kiko> has anyone seen something like this?
[11:23] <bradb> for what url?
[11:24] <bradb> (and what does the ZCML registration for that page look like?)
[11:26] <bradb> (is it possible that you overrode the __init__ without calling the base class __init__?)
[11:26] <bradb> (the __init__ of the view class, that is)
[11:27] <kiko> http://localhost:8086/products/firefox/1.0.0
[11:27] <kiko> in rocketfuel tip
[11:27] <kiko> see if it blows up for you
[11:28] <bradb> yes, it does, so let's see...
[11:29] <bradb> i think i know why
[11:30] <bradb> it inherits from SQLObjectEditView, because it's used as an edit view
[11:30] <bradb> but, on this page, it's not actually used as an edit view, it's just a browser:page
[11:31] <bradb> so, when the view is constructed to render this page, it's not initialized with a .schema attribute, like it would be if it were an addform or editform
[11:31] <bradb> when it's constructed, it's the base class's (i.e. SQLObjectEditView) __init__ that tries to access that .schema, and dies
[11:31] <kiko> that's a regression; it used to work
[11:34] <bradb> the solution is to create a separate FooEditView to use for the edit views, and make this view's view class not inherit from SQLObjectEditView
[11:35] <kiko> why did it regress, I wonder -- any clue?
[11:36] <bradb> Because there's no test for it, I'd imagine.
[11:36] <kiko> yeah, I guess
[11:36] <kiko> hum hum
[11:37] <bradb> I just stumbled on the same problem with distro milestones. Accessing distro milestone bugs raises an exception because, clearly, there's no test for that page.
[12:01] <kiko> darn