[13:17] total for landing (including benji's removal of feature flags): 11718 line diff ;) easy, peasy, right [13:18] heh [13:19] still need one review, so I'll have to bother team yellow for that since I am the other OCR today, and jtv has not responded to review earlier today (and he's probably gone by now) [13:27] benji, gmb, danilos: skype at :30 [13:28] ack [13:37] bac, btw, you don't have to review the branch if you don't feel like it, I'll just wait for the next OCR to show up or get it reviewed tomorrow morning [13:37] danilos: no, it'll be good as i haven't started a new task yet. which MP? [13:38] bac, https://code.launchpad.net/~danilo/launchpad/bug-772754-other-subscribers-actions/+merge/64187 [13:38] gmb: you still going to have time for a lp2kanban day this week? [13:38] bac, it's mostly JS, slightly oversized, and mostly ugly JS unit tests for ajax calls :/ [13:39] bac: I think so, but it's likely to be tomorrow or Friday. [13:39] Probably tomorrow. [13:39] ] === Ursula is now known as Ursinha [14:42] hi danilos -- your branch looks good [14:42] but when i was playing with it i saw this for the first time [14:42] http://people.canonical.com/~bac/youaresubscribed.png [14:42] that edit icon mid-sentence sure looks funny [14:42] bac, right, that's part of Gary's branch which is way to the bottom of the stack :) [14:43] danilos: ok [14:43] bac, I'll note that down (I have some more reservations about the approach he took for different subscription levels, but we've discussed those already :)) [14:44] danilos: ok, i just wanted to ensure that wasn't a done deal [14:44] bac, this is a pretty big change altogether, and basically a rewrite of the entire subscribing functionality for the bug page [14:44] danilos: no, i understand, i just didn't know where that part fit in the big picture [14:44] bac, well, I assume it's going to stay that way unless these issues get raised as critical in the user testing or something like that [14:44] and since the reviews have been spread around no one but you and gary have a clear view, so i asked. [14:45] bac, yeah, I reviewed that branch for Gary, but didn't find that much of an issue (I miss radio boxes much more instead) [14:45] danilos: r=bac [14:45] bac, cool, thanks [14:45] danilos: sorry, i couldn't find anything to complain about. :) [14:46] bac, heh, that's ok, I'll live by with it :) [14:46] bac, thanks again, now you are all equal in having reviewed two branches each :)) [14:46] * bac is relieved [14:47] oh, in the combined gigantic branch, I did review two as well, yay :) [16:23] gmb, benji: fwiw the lp2kanban seems to be working really well now. we're only making ~5 calls to kanban no faster than 1/sec. I'm going to write the dude and tell him we'd like to run it hourly. [16:24] Cool. [16:24] very nice [16:24] I'll try not to break it tomorrow, then :) [16:25] so a new card just has to have "sync:" as the title and the bug number. [16:32] gmb: you might want to look at the tests tomorrow, a test run of a fresh build fails for me [16:33] benji: Right. Since I don't know the code at all, the tests seemed like a good place to start anyway. [16:35] gmb: the test failure was shallow, I just fixed it so you'll have passing tests to look at, even better [16:35] Excellent, thanks. [16:50] benji, gmb: somehow i overlooked those tests and thus they rotted. there are a couple of new functions that could use some unit testing. [16:52] gmb: if you get around to adding the cronjob would you reply to stephen? [16:52] bac: Sure. === Ursinha is now known as Ursula === Ursula is now known as Ursinha === Ursinha is now known as Ursinha-nom === Ursinha-nom is now known as Ursinha [21:33] hi benji, you about? [21:33] bac: yep [21:34] benji: hey i'm stuck trying to do some API exporting [21:34] you're the new leonard right? [21:36] bac: I consider it an honor to carry his mantle. [21:36] that's the spirit. when did you last watch "Big Lebowski"? [21:37] so, i'm trying to export this thing for soyuz called enabled_restricted_families, which is a collection of processor families, arm, etc [21:37] OEM needs it so they can change their archives to get build support for ARM [21:37] once, fairly recently -- one too many times, I'm afraid [21:38] that field is a collection of IProcessorFamily, which i must also export [21:38] it's all here: http://pastebin.ubuntu.com/627617/ [21:38] but when i do a 'make build' i get [21:38] ZopeXMLConfigurationError: File "/home/bac/launchpad/lp-branches/bug-776437/lib/lp/services/messages/configure.zcml", line 38.2-41.6 [21:38] ConfigurationError: ('Invalid value for', 'factory', "('The Processor Family Name', (, )) in canonical.launchpad.browser.message.message_to_canonical_url_data") [21:38] looking [21:39] can't figure out why that is complaining. perhaps some zcml goodness i've neglected? have you seen that error before? [21:40] I haven't seen that one, but I think it's only incidentally a ZCML thing (after all, you did't make any ZCMl changes) let me look at the code that generates that error to see what it might be unhappy with [21:42] i'm confused why the IMessage subsystem is even involved [21:43] hmm, I wonder if it's because of the single/plural bit that collections do [21:47] what do you mean? [21:47] i know of singular_name and plural_name but those are optional [21:48] ah, indeed, you're not using those [21:53] wow, I have no clue why it's saying that [21:53] still looking [22:03] bac: you have a real stumper there; I think I'd need to build your branch to help any more, just looking at it isn't helping [22:03] let me push it [22:05] benji: on a related note, i wonder if i'm going about this wrong [22:05] how so? [22:05] if IProcessorFamily name attribute is unique, perhaps the call should just return and take lists of name strings [22:05] i think there is precedence for that [22:06] the call to the requested export of enabled_restricted_families, i mean [22:08] makes sense [22:12] the branch, as is, lives at lp:~bac/launchpad/bug-776437 if you're interested in puzzle solving [22:14] bac: well, at the moment I'm interested in dinner ;) maybe we can get back together tomorrow (ooh, well, if I don't have jury duty) [22:14] good luck. when is your phone call to find out? [22:14] or do you definitely have to go? [22:14] 45 minutes [22:15] actually i tried really hard last time to be seated [22:15] i figured that jury pool needed all the help it could get [22:16] it's not definate yet but I think there's a good chance because I got a call yesterday reminding me that unless canceled I would have to serve [22:18] well, I normally wouldn't mind much but it's far away (about an hour and 15 minute drive one way) and my wife's pregnancy isn't doing so hot (had to go to the ER last night) [22:18] ok, off to make dinner [22:18] bye [22:18] just read the part about your wife. hope she's ok.