[00:05] wgrant: mumble? [00:08] * sinzui wanders off to feed children [00:17] sinzui: Sorry, here now. [00:20] Hello wgrant. Do you have time to mumble? [00:21] wgrant: I also want a review of this script to remove projects without any license: http://pastebin.ubuntu.com/606665/ [00:55] Has anyone else seen EC2 fail notfound-traversals.txt spuriously? http://paste.ubuntu.com/606695/ [00:56] jtv: It was not spurious. I broke trunk for a few hours last night. [01:10] jtv: How did the moldavian deletion go? [01:10] Ah, the question is gone. That is a good sign. [01:10] Thanks. [01:14] Project db-devel build #544: STILL FAILING in 5 hr 30 min: https://lpci.wedontsleep.org/job/db-devel/544/ [01:18] -> airport to pick up family; bbiab [01:44] wgrant: broke trunk, eh? well done! I was getting nervous because my branch failed twice with different test failures. [01:50] lifeless: rabbit failed lucid_db_lp [01:54] Project windmill-db-devel build #273: STILL FAILING in 1 hr 10 min: https://lpci.wedontsleep.org/job/windmill-db-devel/273/ [01:56] Project devel build #713: STILL FAILING in 5 hr 34 min: https://lpci.wedontsleep.org/job/devel/713/ [02:23] Project windmill-devel build #75: STILL FAILING in 1 hr 9 min: https://lpci.wedontsleep.org/job/windmill-devel/75/ [02:49] lifeless, back? [02:59] Hmm, only 541 timeouts yesterday? [03:00] I guess UDS might be reducing it a bit... but 50%? [03:08] wgrant: how does it compare to previous UDSes? [03:09] Project windmill-devel build #76: STILL FAILING in 45 min: https://lpci.wedontsleep.org/job/windmill-devel/76/ [03:16] wgrant: I see you added a qa-untestable tag to my bug just around the time the tagger was supposed to add a qa-needstesting to it. [03:17] It's actually qa-ok, but I wanted to avoid concurrency issues. [03:17] jtv: Well, you removed the needstesting tag which upsets qa-tagger. [03:17] You can't remove QA tags until the revision is deployed. [03:19] Also, why was the tagger meant to add qa-needstesting? [03:19] It had already, but you removed it. [03:20] And the next buildbot run won't finish for half an hour, and then an hour later it will tag the two involved bugs, neither of which are yours. [03:20] So, I am confuse! [03:20] But I am also hungry, so I shall lunch to resolve my confusion. [03:20] Or at least defer it. [03:21] wgrant: the first branch was deployed, and the second was going to be deployed. [03:21] Both on the same bug. [03:25] jtv: yes [03:25] lifeless: didn't know how far a drive it was etc. :) Trust all is well. [03:26] I wanted to ask for feed back [03:26] yup [03:26] *feedback [03:26] about the UI work I'm sketching out for async package syncs. [03:26] sure, I'll be passing by the keybard for a bit, and once folk are settled, fully here. [03:27] OK, I'll start typing. :) [03:27] We have a Job for sync'ing packages into a derived distro. [03:27] jtv: I still don't understand why the qa-needstesting tag was removed. But it seems all OK now. [03:28] There's a UI that shows differences between a derived distroseries and its parent(s). [03:28] That's where you can request that some of these differences be synchronized, in various ways. [03:28] Some or all of these requests will be asynchronous. [03:29] At first I tried to sketch out a way where the UI remembers "you requested async sync of packages [x, y, z]; these went into jobs {a: [x], b: [y, z]} (one job per source archive, but potentially multiple packages per job) [03:29] " [03:30] But it's costly and awkward and ignores other pending requests. [03:30] If you try to repeat a sync, you get an error. [03:30] (I'm not too enthusiastic about that notion, but there may be good reasons just outside my current field of view). [03:31] I want to break this down into stages that I/we can implement in realistic time: basics first, shiny later. [03:32] My current plan: have the static page always (request or no request) show which of the visible distroseries differences have sync jobs pending. [03:32] (Polling and error reporting come later) [03:33] Since it makes no sense to repeat a pending sync request, I was thinking to _replace_ the checkbox next to a difference that's waiting to be synced away with a little watch as used elsewhere. [03:33] Is that feasible in the current model? [03:34] I don't know. [03:34] (it's the right thing to do, but I don't think it's efficiently doable without a separate jo) [03:34] +b [03:34] I think you're talking about finding out which jobs are pending. I'm talking about the UI. [03:35] We unfortunately already have a model to work within, and that needs to influence the UI decisions :( [03:35] Yes yes yes I know. But there is _some_ specificity to the search and I'm assured the queue won't be very large. [03:35] It will fairly often have thousands of packages. [03:36] But not terribly many jobs, perhaps. [03:36] Packages aren't where the cost is. [03:36] Other ways to find this out quickly lead to extra client/server interactions, which I expect to be much more costly. [03:38] We know which jobs and which destination archive we're interested in when we render the page, and that helps; no need to send an API request from the client after rendering, listing the packages you're interested in. [03:39] It may also be slightly jarring to have your view changed too long after the page has rendered, possibly with consequences for the placement of widgets you're about to click on. [03:40] so [03:40] with queues, in general, we shouldn't assume we can tell whats in the queue [03:40] this is a rabbit limitation AIUI, and though we don't use rabbit today, I'm leery of depending on things we know it doesn't know [03:41] Sensible, but I have a feeling that a lot will go overboard when we do that anyway. [03:42] Most of the wait, for one. [03:42] well [03:42] the latency is 60 seconds for a job runner [03:42] nevertheless [03:42] lets talk concurrency for a second [03:42] two users may both submit a request to sync overlapping packages at the same time [03:43] unless we are -very- careful in our model, we won't trigger unique constraints [03:43] not sanely [03:43] I think if someone wants package version X synced from A to B [03:43] and its queued twice [03:43] shrug [03:43] let the second sync say 'noop, DONE' [03:43] IMNSHO [03:43] Sure, but... [03:44] in other words, make syncing an idempotent operation [03:44] shrug [03:44] Been over that for about as much as I feel we need to right now. [03:44] It's not quite that simple. [03:44] jtv: ok [03:44] Because binaries are fun. [03:44] I'm saying it doesn't make much sense to request a repeated sync; not that I want to avoid it at all costs. [03:45] I was just thinking it might be a minimally invasive, yet effective way of expressing the situation: don't bother requesting a sync here, it's already pending. [03:47] so, internet fail [03:48] Thank you, Natty, for leaving a loophole that gives me access to the "reconnect" button on my chat client that you think I don't want to access ever. [03:49] :> [03:50] To repeat then: [03:50] In the current pre-rabbit situation, this way of doing it has the advantage of making the first step in providing async visibility very easy to implement. [03:52] The main questions for that are, I think: (1) does this display help the user at all? (Actually I'd also want some kind of "sync pending" notice in there, obviously) and (2) does it form any kind of usable basis for the later steps (tracking progress, reporting errors) if we should decide to drop the jobs and use rabbit instead? [03:53] If we later find that it becomes too difficult to mark pending syncs in the static page, we can always drop that bit. But I would hope that unless and until that happens, the work will have helped us move closer to a "nice" UI. [03:55] Trying to do a "nice" UI right away would probably involve making the whole sync request Ajaxy to track the jobs it created (if any), which goes at the cost of getting the first bit of visibility into pending requests usable later than with this approach. [03:56] I'm agnostic here [03:56] I can see merits in both approachs [03:56] I agree that they are to some degree at cross purposes [03:56] I think from an adminny perspective we want to know about things stuck in the queue [03:56] and to be able to fix (e.g. in the current code do a cancel) them [03:58] Well if a job fails, it fails. [03:59] If it's stuck, the whole queue is stuck. [03:59] (Which should show up as oopses, I guess, though my UI would also show it as an increasing amount of pending differences in the display) [04:00] We'd lose that advantage with rabbit I guess, but maybe at that point we'd just accept having to replace that visibility with something as a cost of moving ahead, and my current idea will have served its purpose as a stepping stone. [04:02] I hate being under the weather on a holiday. [04:02] its a holiday? [04:02] Well my girlfriend says it is. [04:03] Can't find it online, but here you're never sure about those things. [04:03] We do have something different next week, but my web searches put it at up to 3 different days this year, at a length of either 1 or 2 days. [04:03] Nondeterministic calendars are a great job-security scheme for religious establishments. [04:03] I think Thailand has more holidays than any other place I know [04:04] I try not even to register the slightly iffy ones ("government holiday" etc.) but when you get right down to it, there's a vast sea of iffy holidays. [04:05] I believe in rare cases the official word comes down literally the day before. [04:05] Not to mention curfews and such. [04:11] bah [04:11] where is that bug about the 404 for pages depending on vhost [04:14] Damn, can't search for "404"! [04:14] bug 422960? [04:14] <_mup_> Bug #422960: appear to be failing to record oops for all +translate HTTP 503 errors < https://launchpad.net/bugs/422960 > [04:15] bug 626878? [04:15] <_mup_> Bug #626878: pages only on subdomains cause user confusion (e.g. launchpad.net/project/+activereviews is a 404) < https://launchpad.net/bugs/626878 > [04:15] bug 574697? [04:15] <_mup_> Bug #574697: Launchpad strips incoming TE header < https://launchpad.net/bugs/574697 > [04:20] #626878: [04:20] thanks [04:20] <_mup_> Bug #626878: pages only on subdomains cause user confusion (e.g. launchpad.net/project/+activereviews is a 404) < https://launchpad.net/bugs/626878 > [04:21] Used google with site: [04:22] 'None' is a subscriber to this bug apparently [04:23] lifeless: I think your Chromium is bad :( [04:23] wgrant: probably; how so? [04:24] Well, BjornT is not None. [04:24] wgrant: different bugs [04:24] And the recipe owner picker works for everybody else. [04:24] bug 735972 [04:24] <_mup_> Bug #735972: Person:+related-software timeouts < https://launchpad.net/bugs/735972 > [04:25] (can I close that picker bug now?) [04:25] wgrant: give me a chance to dup [04:25] it takes a while to reopen 70 or so tabs [04:25] so I've been being productive in the meantime [04:26] Heh [04:26] openid soooooo slow [04:26] (well, our sso is) [04:30] oh man +related-software has lots of room to improve [04:42] ughhhh [04:42] I can't find this darn bug [04:42] the one about bzr push not being able to create a spn [04:43] Bug #386596 [04:43] <_mup_> Bug #386596: pushing to a packaging branch can't create a new package < https://launchpad.net/bugs/386596 > [04:43] hth [04:43] thanks [04:43] Found using Launchpad's search, even. [04:44] * wgrant stabs asuka. [04:44] Update faster. [04:44] thanks! [04:44] * lifeless escalates [05:17] whats a sensible fs for an archive mirror? [06:25] stub: so, yo [06:26] lifeless: yo [06:26] shall we try skynet? [06:27] I'm going to boot my laptop to try and narrow down issues - there is some other weirdness happening with that network card and natty. [06:27] ok === stub1 is now known as stub [07:08] Project devel build #714: STILL FAILING in 5 hr 11 min: https://lpci.wedontsleep.org/job/devel/714/ [07:14] Is that more rabbit failure? [07:14] Why, yes, it is. [07:18] StevenK: That happened on lucid_db_lp earlier. [07:20] I've come to the conclusion that rabbit sucks, and I can't get it to start on the Jenkins build slaves. Not sure why that afflicts buildbot. [07:22] lifeless: "Ubuntu currently has a problem with things that haven't built yet being unable to have bugs filed on them" [07:22] Huh? [07:22] It's got nothing to do with builds. [07:24] StevenK: https://bugs.launchpad.net/launchpad/+bug/655385/comments/14 [07:24] <_mup_> Bug #655385: Allow bug status change from Triaged only for bug supervisor < https://launchpad.net/bugs/655385 > [07:24] bah [07:24] stub: https://bugs.launchpad.net/launchpad/+bug/655385/comments/14 [07:24] <_mup_> Bug #655385: Allow bug status change from Triaged only for bug supervisor < https://launchpad.net/bugs/655385 > [07:24] wgrant: orly? [07:24] wgrant: whats it got to do with? [07:24] lifeless: It's all source-based. [07:24] Doesn't go near binaries. [07:24] wgrant: source publication then ? [07:24] Yes. [07:24] fine :> [07:29] wgrant: so i didn't say builds in that bug [07:29] Ubuntu currently has a problem with things that haven't built yet being unable to have bugs filed on them [07:29] wgrant: I said 'published packages', what was wrong with that ? [07:30] "things that haven't built yet" [07:30] ah, fixorating [07:30] thanks [07:38] Project db-devel build #545: STILL FAILING in 5 hr 44 min: https://lpci.wedontsleep.org/job/db-devel/545/ [08:14] Curse you, lifeless! Escalating bugs :( [08:14] Still, 215 criticals. [08:15] Well, plus loggerhead, but we don't talk about him any more. [08:15] 241 [08:15] launchpad-project [08:15] Hm. I see 237. [08:15] Are there private bugs I cannot see? :( [08:15] lpstatus also sees 10-15 more than I can. [08:16] I see 234 now [08:16] yeah, lpstats counts by category [08:16] Still 237 for me. [08:17] Ah, so if something is in more than one category it will show it twice? [08:17] http://paste.ubuntu.com/606803/ [08:17] wgrant: exactly [08:17] :( [08:17] (space bar broken, slow typing) [08:17] jml: Is that anonymous? [08:17] I know of two private bugs, maybe there is a third. [08:18] wgrant: it's not anonymous [08:19] Project windmill-db-devel build #274: STILL FAILING in 1 hr 10 min: https://lpci.wedontsleep.org/job/windmill-db-devel/274/ [08:19] Then why do I see 237 at https://bugs.launchpad.net/launchpad-project/+bugs?field.importance=Critical :( [08:19] hmm [08:19] Possibly multi-task bugs. [08:19] I see that too [08:19] and 234 on the portlet [08:19] Indeed. [08:20] Possibly 237 tasks, 234 bugs. [08:20] yeah, makes sense [08:21] http://pastebin.ubuntu.com/606805/ [08:21] Anyhow, minor progress. [08:21] that's my script [08:21] We may finally be getting somewhere. [08:21] Thanks. [08:21] wgrant: -44 is awesome [08:21] I also have four or five other fixes in review. [08:21] So we nearly made it to 50 for the week. [08:21] also, for pie purposes, I'm not counting newly escalated bugs [08:21] Ah, excellent. [08:22] They always seemed a bit disenheartening. [08:22] wgrant: sorry [08:48] good morning [08:54] Hmm, isn't hardy desktop EOL'd now? [08:54] Yes, yesterday. [08:54] ubuntu-mozilla-daily still builds for it :( [08:54] And keeps lpia alive. [08:54] Damn you lpia, damn you! [08:55] It was announced? [08:55] A month ago. [08:55] I think I have a little more hate for lpia than you :-P [08:56] At least I didn't mention Hildon. [08:56] Because that would just be mean. [08:56] HULK SMASH [09:00] Hello [09:00] allenap: Morning! Can you look at https://code.launchpad.net/~stevenk/launchpad/generic-overrides/+merge/60730 again? [09:01] allenap: I think I've addressed your concerns, and have even written you a note. [09:07] Morning StevenK, sure I'll look at that now. [09:08] Bah. [09:08] wgrant? [09:08] allenap, jtv: Your packagecopyjob changes have conflicted in stable->db-devel. [09:08] Could be a bit of a mess. [09:08] Thanks for the notice. [09:08] StevenK: I'm reasonably happy with your branch now. [09:09] What does tradition dictate? Gloves or no gloves? [09:09] wgrant: Woohoo. Okay, I'll look in a minute, thanks for letting me know. [09:10] allenap: I'm checking it out right now [09:10] jtv: Awesome. [09:10] Which gives allenap more time to read my horrible diffs. [09:11] Bah, there's no functools.compose. [09:11] How stupid. [09:11] I haven't had to deal with this in the current devel/db-devel configuration... can I still branch off db-devel, merge in devel, resolve conflicts, and land into db-devel? [09:11] jtv: Merge in *stable*. [09:11] jtv: But yes. [09:11] ahhh [09:11] Otherwise you'll preempt buildbot. [09:11] And make us all sad. [09:11] Hmmm conflicts don't look very bad from here. [09:11] There is an apparent argument reordering. [09:12] Oh that [09:12] The visible conflict is easily resolvable. [09:12] But there may be deeper things. [09:12] The change suggests that arguments have changed. [09:12] Or that there may be additional IPackageCopyJobSource callsites. [09:12] When it's now IPlainPackageCopyJobSource, and in a different module. [09:12] I switched two params around in the interface to make them match the implementation order. [09:13] Should have fairly minimal impact but oh, the trouble I had once with this kind of inconsistent parameter ordering between interface and model... [09:15] StevenK: Hm, why do you eager load BPN and SPN? [09:16] StevenK: Given that you get the IDs from the objects, you presumably have them already. [09:16] "lol" [09:17] I can just return the source object directly [09:17] Oh blast I may lose connectivity. [09:17] I also question the way you are given archtags and return DASes, but that may possibly be reasonable. [09:18] wgrant: Because FromExistingOverridePolicy does it too [09:18] But you wrote that as well, so you can't reason against it :P [09:18] Oh, clearly [09:20] Sigh. Now calculateSourceOverrides() for UnknownOverridePolicy is 2 lines. [09:21] Why so many? [09:21] Oh, for default_component? [09:21] Yes [09:22] cronscripts/foaf-update-karma-cache.py upsets me. [09:23] +4, -16. I am an idiot === adeuring changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: adeuring | https://code.launchpad.net/launchpad-project/+activereviews [09:23] adeuring: Hi! [09:23] hi wgrant! [09:23] Oh look, an OCR. *pounce* [09:23] adeuring: https://code.launchpad.net/~wgrant/launchpad/bug-307269/+merge/60721 and https://code.launchpad.net/~wgrant/launchpad/bug-449561/+merge/60729 both need a review, if you could. [09:24] wgrant: sure [09:24] allenap: Are you blind yet? I have yet more changes, but I can hold off if you wish [09:24] StevenK: I've just approved it. === almaisan-away is now known as al-maisan [09:25] allenap: Just glance at http://pastebin.ubuntu.com/606829/ then? [09:27] StevenK: Haha, oh yes :) +1 [09:27] allenap: few questions about the jobs! I guess the version in source_packages would correspond to the parent_source_version in the DSD? [09:28] * StevenK attempts to not swear about himself or wgrant in the commit message [09:28] jtv: Yes, I think so, but I'll check. [09:28] StevenK: I didn't write those! [09:28] Much of the rest, sure... [09:28] allenap: it's also not always clear to me which lists of source-packages tuples are (name, version) and which have the third entry. [09:29] wgrant: But you pointed it out, so pffft [09:29] Ah, I see. [09:29] jtv: Yes, I should probably land a branch to remove the ambiguity. [09:30] Heh... you may want to give my existing conflict branch time to settle. :) [09:30] Any idea what time jools's demo is? [09:30] jtv: Yes, parent_source_version. [09:30] Thanks. [09:31] No, no idea. [09:31] May be useful to be at the ready just prior & during. ;-) [09:31] Argh! Natty, stop switching my desktops around! [09:32] jtv: I was planning on denying everything. [09:32] Good start. [09:32] But you don't know how crafty he is. He may lead with "Steve implemented this great feature here..." [09:32] [09:33] wgrant: Btw, what would functools.compose do? [09:34] allenap: functools.compose(f, g) would basically do lambda *args, **kwargs: f(g(*args, **kwargs)) [09:34] But it was vetoed. [09:34] Because you can do it with a lambda. [09:35] Despite the fact that you can do functools.partial even more cleanly with a lambda. [09:36] wgrant: Yes, odd decision. [09:38] wgrant: one MPP approved. [09:58] wgrant: your other MP looks good too. just one question: Bug 449561 says not only that anonymous users get the edit icon for the assignee, but also that anon users have the option to use "mark as duplicate". What about this problem? And did you check for other possible edit options presented to anon users, as mentioned in the bug report? [09:58] <_mup_> Bug #449561: Bug task assignee picker shown to unauthenticated users < https://launchpad.net/bugs/449561 > [09:59] adeuring: That was fixed at some indeterminable time in the past. [09:59] adeuring: I can't see any other unguarded JS picker activations on the page. [09:59] wgrant: ah. cool. so, r=me [09:59] adeuring: Thanks! [10:44] francis just said he thought question comments were added to the api and curtis blogged about this [10:44] but that doesn't seem to be true... [10:46] which bit? [10:46] did sinzui not blog? [10:46] or are they not in the API? [10:48] both [10:49] neither seems to have happened [10:49] but i might just be missing it [10:51] jtv: How goes the conflict? My inbox is upset with it. [10:51] I can have a look if you're busy/gone. [10:55] I'm just looking at it [10:55] wgrant: Or have you beat me? [10:55] I've had my fix reviewed. [10:56] jml, well, https://bugs.launchpad.net/launchpad/+bug/782093 it can be duped if appropriate [10:56] <_mup_> Bug #782093: question/answer comments not available in api < https://launchpad.net/bugs/782093 > [10:57] poolie: https://launchpad.net/+apidoc/devel.html#question [10:57] jtv: The merge conflict or something else? [10:57] stub: the merge conflict. [10:57] * stub uncommits and leaves it to jtv [10:58] poolie: ah, the questions collection isn't exposed. [10:58] its controllable, but not exposed. [10:58] questions are exposed [10:58] The collection is exposed but not iterable. [10:58] and the spam control knob is exposed [10:59] wgrant: https://launchpad.net/+apidoc/devel.html#question has no comments_link [10:59] wgrant: is that what you mean ? [10:59] Oh, you said questions collection. [10:59] The comments collection isn't, no. [10:59] questions comment collection, I meant [10:59] Right. [11:11] lifeless: right, the comments [11:11] i thought you closed a bug about this the other day [11:11] but perhaps we were confused baout the comments vs the questions [11:11] anyhow, it would be nice for ISD but is not urgent [11:11] What does ISD want them for? [11:11] I thought they had other solutions for RnR. [11:12] apparently they get questions across a lot of projects and they want to summarize them [11:12] Ahh, so something more sensible this time :) [11:12] They shouldn't be too difficult to export. [11:12] Just no point if nobody was going to use them. [11:15] lifeless: in https://bugs.launchpad.net/launchpad/+bug/373078/comments/6 aaron says 'Needs fixing is meant to mean "This is approved but I'd like you to fix X" [11:15] <_mup_> Bug #373078: code review merge status and docs need clarity, particularly for 'tweak' cases < https://launchpad.net/bugs/373078 > [11:15] it's a shame launchpad's own use of them isn't consistent with that [11:15] [11:15] I thought Approve with comments meant that :/ [11:15] but perhaps that will get fixed [11:15] it seems quite ambiguous and inconsistent across projects [11:16] anyhow i shall try to remember that in future [11:16] Do I land conflict fixes with release-critical then? [11:17] jtv: No. [11:17] Does it want it? [11:17] I '[rs=wgrant][no-qa] Whatever' [11:17] Seems to, yes. [11:17] I think you are probably misreading it. [11:17] Could well be. [11:17] I hate those messages. [11:17] It will probably accept release-critical=, but does not require it. [11:18] What was your commit message? [11:18] And I also hate $#%@ unity hiding my windows from me. [11:18] Hang on [11:19] Manual submission notes: your submission message needs at least one of [r=...] [rs=...] or [release-critical=...] AND at least one of [11:19] Ah, it was the other one that's missing. [11:19] no-qa. [11:19] Right. [11:19] The next attempt failed for yet other reasons. [11:19] Even more confusingly, testfix is exempt from no-qa [11:19] poolie: the labels are a bit arbitrary [11:19] poolie: aside from that may /mean/, I think a label with 'needs' in it does not communicate 'should' or 'might' [11:20] poolie: thanks for trying to remember in future [11:21] Well, that seems to be in. [11:22] well, i'm not an authoritative reviewer so semantically i can't say "must" anyhow [11:22] i guess it means "please consider this before merging" [11:23] mmm, you're in the review team, so the ui shows you as authoritative [11:23] even if it showed you as community, it seems a courtesy not to send conflicting signals ;) [11:26] poolie: how has uds been? [11:29] sorry, i just set the status the way i would in bzr [11:29] just wanted to save him needing to hmake a second submission [11:29] good [11:29] sneezy [11:29] ubuflu is in fine form [11:31] heh, I'm very glad I'm not there then ;) [11:33] interesting session right now about python versions [11:33] mm [11:33] I've been doing this work for too long. My first reaction to the word "sneezy" was "oh they're planning Ubuntu releases up to S." [11:33] it seems like a lot of things are internal teem meetings [11:34] poolie: you got me curious, doc/developers/code-review.txt doesn't say how bzr uses votes [11:34] and points at https://help.launchpad.net/Code/Review [11:34] anyhow, enough of that [11:34] 10:30 on friday :) [11:35] the informational or seminar type sessions are not clearly distinguished which means some of them turn out to be less interesting thatn you might think [11:35] up until wednesday the schedule churned a great deal [11:35] it was crazy [11:35] apparently the scheduler in a couple of cases thought it would be clever to schedule things into sessions that had already passed [11:35] look at all that wasted space last monday! [11:36] have a good weekend [12:53] Project windmill-devel build #77: STILL FAILING in 1 hr 9 min: https://lpci.wedontsleep.org/job/windmill-devel/77/ [12:56] Project db-devel build #546: STILL FAILING in 5 hr 17 min: https://lpci.wedontsleep.org/job/db-devel/546/ [13:26] adeuring: Could you take a look at https://code.launchpad.net/~gmb/launchpad/bug-777783/+merge/60910 for me? [13:32] gmb: sure [13:32] Thanks [13:33] gmb: Hi. [13:33] wgrant: Wotcher. [13:34] gmb: I'm gardening our various queues, and noticed that you have two approved, unlanded branches from about 6 months ago. [13:34] Orly? [13:34] wgrant: I'll take a look, thanks for the heads-up [13:34] First section of https://code.launchpad.net/launchpad/+activereviews [13:35] wgrant: I've marked them both merged (I think they were subsumed by other branches, IIRC). Thanks again. [13:35] Thanks! [13:38] jelmer: https://code.launchpad.net/~jelmer/launchpad/publisher-use-debian-2/+merge/45011 appears to have been approved just days prior to your defection to bzr -- should I try to land it, or do you recall why it didn't? [13:43] wgrant: feel free to land it [13:43] wgrant, I didn't because I was worried I wasn't going to be able to deal with any fallout [13:48] gmb: r=me [13:49] jelmer: Ah, sure. I'll throw it at ec2. [13:49] Thanks. [13:50] poolie: jml: I have not made questions usable over the API yet. That is probably 2 branches away from being complete. jcsackett landed feature to hide question comments. [13:52] sinzui: Hi. I was wondering if you could have a look at the tests in https://code.launchpad.net/~wgrant/launchpad/bug-449561/+merge/60729 [13:53] okay [14:00] wgrant: I am a little apprehensive about the "client.waits.sleep(milliseconds=1000)" line. Is there an element that we could use "client.waits.forElement" instead? [14:01] sinzui: That was my question. [14:01] sinzui: I am checking for the absence of an element that is created asynchronously after page load. [14:01] I don't know if there is some post-load event that I can hook into. [14:02] Project windmill-db-devel build #275: STILL FAILING in 1 hr 8 min: https://lpci.wedontsleep.org/job/windmill-db-devel/275/ [14:02] sinzui: This is why I waited for you/deryck. [14:02] 'cause I don't really know how to avoid it. [14:04] wgrant: can't you use forElement(HIDDEN_ASSIGN_BUTTON)? forElement is an assert [14:04] sinzui: That [14:04] That's how the button is in the HTMl. [14:04] The JS hook changes it. [14:05] I am asserting that it does not execute. [14:12] wgrant: We should at least use lp.testing.windmill.constants.SLEEP so that the test is consistent with other tests [14:14] I had hoped to catch a deryck of some variety today, but that seems unlikely at this point :( [14:14] adeuring: Thanks for the review. [14:15] derycks now come in variety packs? I would like a green one please [14:15] https://lpbuildbot.canonical.com/builders/lucid_lp/builds/942/steps/shell_6/logs/summary is interesting. [14:16] Is something pruning /tmp a bit aggressively? [14:16] Both rabbit and lp-serve complain about similar things. [14:17] wgrant: deryck is listed as on leave in canonicaladmin [14:18] This is unfortunate. [14:18] I guess it will have to wait until next week. [14:18] How can bac be off twice on the same day. If he is time traveling, I would advice not meeting himself. That always causes embarrassment. [14:22] wgrant: I don't think you need to wait given the large number of SLEEPs I see in windmill tests. Use the constant. That would be my only requirement to land. [14:23] sinzui: Thanks. I know deryck hates it when people use sleep, but I don't see a clear way around it here. === bigjools is now known as edbot === edbot is now known as bigjools [14:48] Project windmill-devel build #78: STILL FAILING in 46 min: https://lpci.wedontsleep.org/job/windmill-devel/78/ [14:55] man, LP is fast. [14:59] Project devel build #715: STILL FAILING in 5 hr 30 min: https://lpci.wedontsleep.org/job/devel/715/ [14:59] bigjools: You're not being sarcastic [14:59] ? [14:59] StevenK: a first for me, I know [15:15] jcsackett: the new comment hide/unhide functionality is nice. I wonder if we should make the hidden comments collapsed or something so they aren't quite so visible to those of us blessed with the ability to see them. [15:17] I also think the "Mark as spam" button should probably turn into an icon somewhere near the comment number. But those are just tweaks -- I'm really glad I can remove my scripts now :) [15:20] benji: that's a good idea. [15:21] wgrant: basically you mean move it from the footer to the header? [15:21] jcsackett: And make it smaller. [15:21] It currently stands out a lot. [15:22] Possibly it should be an edit icon next to the number that brings up the AJAX boolean picker. [15:22] Like the one used for affects-me-too. [15:23] wgrant: not sure about the latter, i'm not a huge fan of multiclick steps for simple actions. but that's a personal preference. [15:23] you may be right that it fits lp's flow better that way. [15:23] Yeah, fair point. [15:23] It may be OK to just move it to the header. [15:23] But it's a bit obtrusive at the momnet. [15:23] wgrant: yeah, it's a little on the big side now. [15:24] * jcsackett makes note to file bugs on both those points. [15:24] Thanks for all your work on this. [15:24] Is the yellow background on "hidden" comments like the one on https://bugs.launchpad.net/launchpad/+bug/1234 common? It kind of makes the spam stand out, which is ironic. :) [15:24] <_mup_> Bug #1234: Gina is an unmaintainable mess of command line options, environment variables and shell scripts < https://launchpad.net/bugs/1234 > [15:24] I'm not sure why a highlight was chosen :/ [15:27] benji, wgrant: i'm not sure about that either, but the hidden comment style was already established, and i didn't want to mess with css much at the time. [15:27] sensible [15:27] i think it should be within a branch of work to turn the hidden comment style into a collapsible tho. [15:28] bac referred to the style as "eye-gouging" which isn't too far off the mark.:-P [15:28] lol [15:28] Heh [15:28] Maybe it's there to convince us to stop the spam from arriving in the first place :) [15:28] good motivation indeed [15:28] i like that theory. it's not ugly, it's "motivational" [15:28] :-P [15:29] * benji is good at creating "motivational" UI. [15:29] * jcsackett laughs. [15:30] I guess that gives whole new meaning to "motivational speaker". [15:30] by this standard, that would be someone throwing things at you from the stage until you made something of yourself, right? :-P [15:31] I think it would be a very unattractive person speaking to you. I might now qualify to be a motivational speaker. [15:35] Project windmill-devel build #79: STILL FAILING in 47 min: https://lpci.wedontsleep.org/job/windmill-devel/79/ [15:41] hi sinzui [15:42] i wasn't trying to rush you [15:42] it was just something that selene raised in a session here at UDS [15:47] poolie. Okay. I have not announced answers of the API because it is clearly not usable. When I can use it for my projects, and I will blog about it [15:49] sinzui: are you, or do you know, a good person to grab for help on a blueprints question in #launchpad? [15:51] No is a good person since no-one who wrote it or uses it works on Lp [15:51] jcsackett: I will see if I can help [15:54] sinzui, i was misinformed and thought they were public [15:54] anyhow, she thanks you for adding them [15:55] poolie: I think many people misunderstood what was exported to allow admins to hide comments. It was a single method on question, not the actual objects [16:01] yes i misunderstood that [16:01] well, [16:03] i think it's reasonable to want it regardless of your work [16:03] you don't have to do it right now [16:29] wgrant, benji, poolie: just wanted to let you know the various feedback has been captured as filed bugs. [16:30] great! thanks [16:30] poolie: come to think of it, i *think* you provided feedback, i may have mixed up emails. :-P === al-maisan is now known as almaisan-away [16:38] poolie: Thanks for getting that bug in the queue... === Ursinha is now known as Ursinha-ohnomnom [16:40] sinzui: Not trying to rush you; I just mentioned it in a session today and poolie wanted to help. :) [16:41] (I'm very happy that questions are being added and I don't have to do it myself!) [16:42] I am waiting the QA the change that allows me to add and remove my teams as answer contacts. I am starting the branch that lets you search for question in a couple hours [16:43] My goal is to generate a list of open questions for a given team's projects, without having to check each one manually. [16:44] ... just making it easier to do support for dozens of projects. [16:44] ToyKeeper: then we are in agreement. That is the actual test I am writing. When the test is complete, then I can blog that I have something useful [16:44] Cool. :) === adeuring changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews === Ursinha-ohnomnom is now known as Ursinha === matsubara is now known as matsubara-lunch [18:18] Project db-devel build #547: STILL FAILING in 5 hr 21 min: https://lpci.wedontsleep.org/job/db-devel/547/ [18:24] sinzui: could you, if you get the time, look at https://code.launchpad.net/~jcsackett/launchpad/apocalyptic-messages-0/+merge/60953? [18:24] it's somewhat terrifying, but all mechanical, i believe, and you have a lot of experience with the apocalypse. [18:42] jcsackett: I am reviewing it now [18:51] jcsackett: 1. line 1008 shows the addition of the services.message package. I think that list is alphabetical. [18:53] jcsackett: 2. You create a doc/ dir and moved tests there, but I do not see the test_doc.py that loads the the doctests. I was expecting something like lp.services.mail.tests.test_doc.py [18:54] jcsackett: did you forget to commit test_doc? [18:54] sinzui: i think i may have. [18:54] one sec. [18:55] sinzui: yup, forgot the all important 'bzr add'. [18:55] it's pushed up now. [18:55] * sinzui waits [18:56] sorry about that, sinzui. [19:01] jcsackett: message.txt does not look special. It does not have a special setup, teardown or layer. Did you try the generic test_doc.py that is used in mail. I think it will work [19:01] sinzui: i don't think so, i just cribbed it from one of the services modules. i'll try out mail's version. [19:02] also, that test may play DatabaseFunctionalLayer. The layer it is using starts the librarian, which may not be needed [19:02] * sinzui reads the test [19:02] sinzui: the layer is what it originally had in canonical; that said it may very well work with something else and i'm happy to change it. [19:03] Almost everything in canonical.launchpad runs on the wrong layer. Always question why a test runs on the LaunchpadFunctionalLayer [19:06] thanks jcsackett, that's a good feature to have [19:06] i'm sure it will save time === matsubara-lunch is now known as matsubara [19:25] thanks, poolie. [19:25] sinzui: got it. [19:25] faboo [19:25] Sickn3ss [19:26] freaking unity lock up.... [19:27] Project windmill-devel build #80: STILL FAILING in 1 hr 9 min: https://lpci.wedontsleep.org/job/windmill-devel/80/ [19:33] sinzui: the test needs the librarian; it's doing uploads on msgs. i changed the layer and get all sort of upload errors. [19:34] okay [19:34] but it does not need a special setup/teardown does it? [19:34] nope. [19:35] cribbing the test_doc in mail works fine but for the layer. [19:37] great. r=me the land with that test [19:39] sinzui: cool, thanks. [19:41] * jcsackett ssh's to reboot his dev machine. again. [20:17] Project windmill-db-devel build #276: STILL FAILING in 49 min: https://lpci.wedontsleep.org/job/windmill-db-devel/276/ [20:26] Project devel build #716: STILL FAILING in 5 hr 27 min: https://lpci.wedontsleep.org/job/devel/716/ [21:03] Project windmill-devel build #81: STILL FAILING in 46 min: https://lpci.wedontsleep.org/job/windmill-devel/81/ [22:07] sinzui: i have enabled thumb-click drag on ubuntu on my macbook. [22:07] \o/ [22:08] sinzui: follow the instructions here, even tho it is not for the 5-1 macbook. [22:08] https://help.ubuntu.com/community/MacBookPro7-1/Maverick#Touchpad [22:08] * sinzui reads [22:09] you will then need to muck about with sensitivity/acceleration some in the mouse settings. [22:09] b/c it becomes VERY speedy moving the cursor around. === Ursinha is now known as Ursinha-afk