[00:09] sinzui: me and thumper are sick of each other's company in mumble. you around? [00:09] 2 minutes please [00:09] ack [00:12] sinzui: time's up [00:14] lifeless: I can't QA this. I need henninge. [00:14] abentley: thanks for trying [00:15] lifeless: np [00:15] abentley: is it possible to check it doesn't break anything (vs fixing the issue) ? [00:16] lifeless: I don't believe so. [00:16] ah well, ok. [00:16] I will mail henninge a ping [00:17] lifeless: To check it doesn't break anything, I would need to get it to display the thing I changed, and that's what I need hennninge for. [00:29] abentley: is it a permissions thing, or knowledge? [00:29] lifeless: knowledge [00:29] k [00:30] I might see if I can figure it out later today, and if so qa it [00:33] lifeless, abentley: https://translations.qastaging.launchpad.net/ubuntu/hardy/+source/gnome-applets/+pots/gnome-applets-2.0/bs/+translate, down the bottom. [00:34] Doesn't seem to be fixed. [00:37] wgrant: have you left rosetta experts etc? [00:38] lifeless: I logged out. [00:38] Actually, I guess this might be a different bug. [00:38] Since the bad string still shows up under Shared when I'm logged in. [00:39] wgrant: the english when logged out is "To prevent privacy issues, this translation is not available to anonymous users, [00:39] if you want to see it, please, log in first.", which is correct. [00:40] wgrant: the fact that it says the bit about dummy translations under "shared" is another issue. [00:41] wgrant: the bug was not exhibited when users were logged out. AFAIK, they were always getting ""To prevent privacy issues, this translation is not available to anonymous..." [00:41] wgrant: it was logged-in users without write access who were getting the issue. [00:43] abentley: As an unprivileged user I see the contributions. [00:45] But same on production. [00:47] wgrant: I don't see how that could happen. Do you see entry fields where you could put translations? [00:47] abentley: No. [00:48] It's read-only. [00:48] Oh. [00:48] Hmm. [00:48] Do suggestions count? I guess they probably do. [00:48] wgrant: they do. [00:49] Ah. [00:49] I guess I can make it read-only if I change my licensing settings. [00:50] Apparently not. [00:50] wgrant: I believe that will prevent you from seeing it at all. [00:50] I no longer have suggestion widgets, but the contributions are still there. [00:51] wgrant: my steps to reproduce involve setting the pillar to "closed" translation permissions. [00:55] lifeless: skype again? [00:56] abentley: I might revert the fix from DF, close Ubuntu there, and see what happens. [00:59] sinzui: please [00:59] sinzui: 5 minutes first, gotta ring a supplier [00:59] okay, 5 minutes [01:05] lifeless: Still can't reproduce :( [01:36] sinzui: https://bugs.launchpad.net/launchpad/+bug/745512 [01:36] <_mup_> Bug #745512: product-release-finder broken < https://launchpad.net/bugs/745512 > [02:22] lifeless: the privacy banner thing makes me think it would be nice to have a scope that matches user-agent [02:28] lifeless: do you know why lp/app/widgets/configure.zcml is not listed in canonical/launchpad/configure.zcml? hence it is not being loaded :-( [02:29] i added something to lp/app/widgets/configure.zcml and took me a bit to figure out why it wasn't working [03:02] poolie: it would [03:03] wallyworld_: well [03:03] I'd expect lp/app/configure.zcml to list it [03:03] wallyworld_: I don't know why it isn't listed [03:04] OK, now why is buildout crashing with an assertionerror... [03:04] lifeless: np. i'll add it. just wanted to check if there was a known reason for omitting it [03:04] wallyworld_: not that *I* know of, but I'm not as cluey as say gary or curtis [03:05] whee [03:05] 771 Time Outs - yesterday [03:05] 6926 Exceptions [03:05] Heh, yeah. [03:06] grar [03:06] ? [03:07] also [03:07] sob [03:07] SourcePackage.setpackaging() now deletes an existing Packaging record and creates a new one, if one exists, [03:07] -> index bloat. [03:07] Yay [03:10] looks like we can deploy to 12821 [03:13] lifeless: Can you make compile on natty? [03:13] I could until this morning :/ [03:13] wgrant: I don't know [03:23] wgrant: (because I develop on lucid, our deployment environment) [03:24] lifeless: Ah, of course. [03:30] Project windmill build #177: STILL FAILING in 1 hr 3 min: https://lpci.wedontsleep.org/job/windmill/177/ [03:34] poolie, was bug 761327 supposed to be against launchpad? [03:34] <_mup_> Bug #761327: would like a feature flag scope on browser user agent < https://launchpad.net/bugs/761327 > [03:41] oops yes [03:41] 'too many matches' fail [04:20] Yippie, build fixed! [04:20] Project devel build #640: FIXED in 5 hr 7 min: https://lpci.wedontsleep.org/job/devel/640/ [06:10] wgrant: https://code.launchpad.net/~stevenk/launchpad/drop-psc/+merge/57807 [06:11] lifeless: It doesn't matter. staging should get everything due to the DB restore, and if qastaging if missing some changelogs, we don't care. [06:12] StevenK: Let's not drop it yet. [06:12] StevenK: Leave it running for a week to make sure we don't miss anything. [06:12] Meh, it was five minutes work, it can wait. [06:12] It's possible (although rather unlikely) that there is still bad stuff being created. === almaisan-away is now known as al-maisan [06:15] StevenK: I had a similar migration I turned off eagerly. [06:15] StevenK: then it broke staging [06:15] StevenK: *trust* me on this. [06:15] And DF [06:15] :-) [06:15] once all systems have either caught up on their own or been restored to it safe to nuke [08:04] Project windmill build #178: STILL FAILING in 1 hr 7 min: https://lpci.wedontsleep.org/job/windmill/178/ [08:33] Rargh, now it's rvba! [08:33] StevenK: ? [08:34] rvba: There's a 'hello-rvba' in dogfood's NEW queue! [08:34] I think it's your doing! [08:34] StevenK: right ... nice it went through. [08:34] Well, it's in NEW, what do you want me to do with it? [08:34] wgrant: Can haz a hand? [08:35] StevenK: would be great yes ... and you could update df while you've at it? [08:35] StevenK: 'sup? [08:35] DF has been updated for 15 minutes. [08:35] StevenK: ok then, I'll be able to qa yesterday's work [08:35] wgrant: I'd like to force a maverick build record into NEEDSBUILD without it building. [08:36] rvba: What I mean is that hello-rvba is blocking my QA -- shall I accept it, or reject it? [08:36] /srv/launchpad.net/codelines/buildd-manager stop [08:36] StevenK: accept it please [08:40] wgrant: No such file or directory? [08:41] good morning [08:41] wgrant: Right, buildd-manager killed [08:42] rvba: Accepted, er, about 4 minutes ago [08:42] rvba: I've only process-accepted, so it's out of the ACCEPTED queue [08:43] StevenK: Hm, maybe it's just in plain /srv/launchpad.net. [08:43] I can never remember. [08:43] StevenK: anyway, I'm stuck because the page https://dogfood.launchpad.net/ubuntu/natty/+localpackagediffs is borked [08:43] wgrant: Tis [08:43] wgrant: Anyway, what's the next step? [08:43] StevenK: Set one to NEEDSBUILD. [08:43] SQL is your friend. [08:43] wgrant: In BPB, right? [08:44] StevenK: BFJ [08:44] Bleh, BFJ has no distroseries [08:45] PB [08:45] Er. [08:45] distroarchseries is on BPB, yeah. [08:45] You'll need to join BPB->PB->BFJ [08:48] or merge them [08:51] hi all, could someone give me a hand? I'm trying to use the API to manage the import queue entries in Ubuntu. While on the web UI I can change any entry's status, I don't seem to have the permissions when trying to do it with launchpadlib -> http://pastebin.ubuntu.com/594384/ Perhaps I'm not changing the status the way it's supposed to be done. Any ideas? [08:55] dpm: Is import_into set? [08:56] wgrant, what's import_into? I cannot find it in the api docs, and I did not set it [08:57] dpm: An import queue entry can't be approved until it has somewhere to be imported into. I'm not sure you can set that through the API. [08:58] wgrant, ah so perhaps the setStatus call is missing an import_into parameter, you think? [08:59] (I mean in the api) [08:59] or the import queue entry is missing the attribute [08:59] dpm: Well, you'd need to set the target POFile or POTemplate. [09:00] Which means we'd need to export those two onto the API, which isn't done yet. [09:01] ok, thanks wgrant, I think I'll file a bug, then [09:01] wgrant, dpm: that is correct [09:01] (missing API export for approval) [09:01] And "Good morning" ;) [09:01] Hi lifeless [09:02] lifeless: looking at that QA [09:02] thanks henninge, and good morning ;) I'll file a bug then - any additional info I should mention there? [09:02] henninge: thanks! [09:03] dpm: "Enable queue approval through API" should be enough. You can mention what wgrant just said about being able to set potemplate and pofile [09:04] ok, thanks henninge [09:04] dpm: but there may be more to it because POTempalte and POFile entries might need to geet created during approval. [09:05] ok, I'll add all the info === adeuring changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: adeuring | https://code.launchpad.net/launchpad-project/+activereviews [09:20] henninge, ah, I've just noted I had filed a very similar bug a while ago, so I've just updated the info (bug 671329) [09:20] <_mup_> Bug #671329: Enable translation import queue approval through API

< https://launchpad.net/bugs/671329 > [09:20] *noticed [09:34] Project devel build #641: FAILURE in 5 hr 14 min: https://lpci.wedontsleep.org/job/devel/641/ === Ursinha-afk is now known as Ursinha [10:11] adeuring: Are you in the mood for performance branch? It's a bit big, but reasonably straightforward... I hope :) https://code.launchpad.net/~allenap/launchpad/localpackagediffs-performance-bug-751321-queries/+merge/57778 [10:12] allenap: sure, I'll look [10:12] adeuring: Thank you :) [10:37] mrevell: you have mail [10:38] Thanks lifeless! === al-maisan is now known as almaisan-away [10:50] mrevell: re the downtime [10:51] mrevell: do you mean to say 'rob, you should tweet on identi.ca on monday morning about this' [10:51] mrevell: or 'cannot do monday as there isn't enough warning' ? [10:54] lifeless, Sorry that I wasn't clear. I meant that I am happy to announce this for Monday, if that's when you want to do it, because I think that a full weekend's notice is plenty for such a short and localised disruption. [11:21] adeuring, hi, got time to review https://code.launchpad.net/~danilo/launchpad/remove-primary-duplicate-reason/+merge/57839? [11:22] danilos: I'll look at your mp when I've finished a review for Gavin [11:25] adeuring, cool, thanks [11:25] adeuring, fwiw, this one is 40 lines, so shouldn't be too hard [11:36] allenap: very nice work, r=me. Just one details puzzled me: There are two XXX comments in a template "this cell needs 0 QUERIES per row". Zero queries doesn't sound like a bug ;) [11:37] adeuring: Yeah, I left those there for completeness, but if they're puzzling then I shall remove them. Thank you for the review! [11:47] https://answers.launchpad.net/testresources/+addquestion is 34 queries for me. wtf. [11:47] danilos: r=me [11:48] adeuring, thanks [11:49] lifeless: I've got a guy asking questions about testresources on an old blog post. He has asked a question that's a bit tougher than I'm willing to help with right now. Which forum should I direct him to for help? [11:54] http://code.mumak.net/2008/10/testresources-some-examples.html fwiw [12:13] Project windmill build #179: STILL FAILING in 1 hr 1 min: https://lpci.wedontsleep.org/job/windmill/179/ [12:39] jml: got a minute to talk about testtools? [12:40] bigjools: in a few minutes, need to grab some sandwiches from the kitchen first. [12:40] ok - just trying to work out how to use MatchesStructure and failing hard [12:45] bigjools: ah ok. [12:46] bigjools: I haven't used it before. Taking a look now. [12:46] bigjools: What are you trying to do? [12:46] jml: ok I think I got it now. The docs are wrong I think. [12:46] bigjools: what's the prob? [12:46] http://readthedocs.org/docs/testtools/en/latest/for-test-authors.html#matchers [12:46] it says: [12:46] matcher = MatchesStructure({'a', Equals(1), 'b', Equals(2)}) [12:47] I had to use = MatchesStructure(a=Equals(1), b=Equals(2)) [12:47] seems like I am the first to use this matcher in LP :) [12:47] wouldn't surprise me :) [12:48] it's very handy though [12:48] mwhudson wrote it for some Linaro stuff he was doing [12:48] I can use it to test new model objects [12:48] a thing that I often do with matchers is make new ones using lambda [12:49] IsAThing = lambda x, y: MatchesStructure(a=Equals(x), b=Equals(y)) [13:07] bigjools: just landed a doc fix. thanks. [13:07] jml: cool, thanks :) [13:07] it was invalid syntax anyway :) [13:12] hah, yes. [13:13] just goes to show that the best documentation review is from someone who's trying to figure something out. [13:16] mrevell: thanks please do [13:16] jml: anywhere I chat on [13:23] gary_poster: again a question about canAcess(): How can we say in the @operation_parameters decorator that the object can have more or less any schema? (I tried "obj=Reference(schema=Interface)", but that's a bit too naive...) [13:24] adeuring...I don't know. :-/ benji, any ideas? [13:25] * benji looks. === bac changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: adeuring, bac | https://code.launchpad.net/launchpad-project/+activereviews [13:27] hi adeuring [13:28] hi bac! [13:38] adeuring: I'm not certain, but I think using IField will give you a parameter that can take any value [13:39] benji: let me check again, but I think I tried that too,. just a second... [13:40] benji: InvalidInterface: Concrete attribute, IField [13:40] hmm, let me look at it a little more [13:41] benji, gary_poster: but I think meanwhile that we could as well use obj.canAccess(user, attrs), instead of user.canAccess(obj, attrs). All we need then is a simple mixin class [13:42] ...and no hassles with a "generic object" parameter [13:47] adeuring: oh, it wants a schema field, not an interface; if you still want to try something try using an instance of zope.schema.Field [13:47] benji: Ah, thanks! === almaisan-away is now known as al-maisan [13:54] benji: "@operation_parameters(obj=Field())" results in "zope.interface.exceptions.InvalidInterface: Concrete attribute, Field". Or did I misunderstand your suggestion? [13:55] That was my suggestion, unfortunately. [13:58] adeuring: if the parameter is an object, I'm pretty sure this will work: @operation_parameters(foo=Reference(schema=Interface, title=_('Foo'))) [14:00] benji: zope.interface.exceptions.InvalidInterface: Concrete attribute, Field [14:01] that's really strange because LP uses code like that all over the place [14:02] I wonder if your getting that error from something else; try commenting out the method that you're working on and see if the error persists. [14:03] s/your/you're/ [14:17] benji: can you give me an example where Reference(schema=Interface, title=_('Foo') is _really_ used? That's quite often used to avoid circular imports; the schema is later changed, I think [14:18] adeuring: you may be right; I had forgotten about the "change the interface later" thing we do [14:20] henninge: https://code.launchpad.net/~abentley/launchpad/sharing-spinners [14:31] henninge: https://bugs.launchpad.net/launchpad-project/+bugs?field.tag=exploratory-testing+upstream-translations-sharing&field.tags_combinator=ALL [15:27] benji: are you the right person to bother if i'm hacking on launchpadlib? [15:27] jcsackett: most likely, what's up? [15:28] benji: do you know if the docs in launchpadlib are just intended as examples of usage, or if they actually run as doctests? b/c they don't appear to be pulled in when you run bin/test [15:29] jcsackett: unfortunately the docs are not tested; I would love for that to change though. [15:29] benji: ah. dig. that is a shame, as the docs looked like the right place for me to add a change for what i'm doing. i'll just add a unit test then. [15:29] thanks! [15:30] np === al-maisan is now known as almaisan-away [16:11] adeuring or bac: could you please review https://code.launchpad.net/~abentley/launchpad/sharing-spinners/+merge/57888 ? [16:11] abentley: I'll look [16:11] adeuring: thanks. [16:12] adeuring: I noticed a conflict. Updating... [16:12] ok [16:18] adeuring: updated. [16:18] * adeuring is looking [16:23] adeuring, bac: can one of you take a look at https://code.launchpad.net/~jcsackett/launchpadlib/add-questionset/+merge/57891 [16:23] yes [16:27] jcsackett: done. easy peasy. [16:28] thanks, bac. [16:31] * jcsackett wonders what the landing procedure is for launchpadlib... [16:31] commit, pray [16:32] jcsackett: yeah, that's an interesting question, isn't it. [16:32] jml: it is. i worry that bigjools is right. it looks like you just commit to a checkout version and push... [16:38] adeuring: so we don't step on each other's toes would you be sure to hit the 'claim review' button? [16:38] bac: whoops... ok [16:38] thanks! [16:39] jcsackett: are you going to update the version of launchpadlib LP uses? i see we're way back on 1.9.2 [16:39] er, 1.9.3 [16:40] bac: er, i haven't thought about the version that launchpad uses. [16:40] jcsackett: https://dev.launchpad.net/HackingLazrLibraries#Landing%20your%20branches [16:42] thanks, benji. [16:43] jcsackett: sorry, lost my internet [16:44] jcsackett: it isn't required but we should bump it at some point [16:44] bac: dig. [16:54] abentley: r=me [16:54] adeuring: thanks. [16:59] sinzui: have some time to mumble? === beuno is now known as beuno-lunch [17:05] jcsackett: sorry, screen locked up [17:05] sinzui: all good. [17:08] jcsackett: I think my session is corrupted. I need a few more minutes [17:08] sinzui: ok. [17:11] sinzui: i could hear you. i think you cannot hear me? [17:11] * jcsackett sees sinzui disconnected. === beuno-lunch is now known as beuno [18:08] I can't assign bugs to people [18:08] is that a known problem? [18:51] beuno: no, it's not a known problem. [18:51] beuno: it works for me. [18:52] jml, can I debug somehow? [18:52] I click on the person and nothing happens [18:52] beuno: what browser? [18:52] jml, FF 3.6, stock 10.10 [18:53] hm. [18:53] beuno: that's just weird. [18:53] beuno: I'm in a bad position to help you right now. [18:53] beuno: sorry. [18:53] jml, np [19:24] g'night all. [19:25] Will see you around. [19:55] bac: could you please review https://code.launchpad.net/~abentley/launchpad/sharing-overlay-buttons/+merge/57935 ? [19:55] abentley: sure, you'll be next [19:55] bac: thanks. === Ursinha is now known as Ursinha-afk [19:59] bac: resubmitted due to missing prerequisite branch: https://code.launchpad.net/~abentley/launchpad/sharing-overlay-buttons/+merge/57938 [20:48] abentley: looking now [20:48] bac: ack [20:52] looks fine abentley [20:52] bac: thanks. [20:53] bac: Can you do one more? It's real short: https://code.launchpad.net/~abentley/launchpad/fix-branch-precondition2/+merge/57945 [20:53] sure === bac changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews [21:22] moin