/srv/irclogs.ubuntu.com/2011/04/15/#launchpad-dev.txt

wallyworld_sinzui: me and thumper are sick of each other's company in mumble. you around?00:09
sinzui2 minutes please00:09
wallyworld_ack00:09
thumpersinzui: time's up00:12
abentleylifeless: I can't QA this.  I need henninge.00:14
lifelessabentley: thanks for trying00:14
abentleylifeless: np00:15
lifelessabentley: is it possible to check it doesn't break anything (vs fixing the issue) ?00:15
abentleylifeless: I don't believe so.00:16
lifelessah well, ok.00:16
lifelessI will mail henninge a ping00:16
abentleylifeless: 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:17
lifelessabentley: is it a permissions thing, or knowledge?00:29
abentleylifeless: knowledge00:29
lifelessk00:29
lifelessI might see if I can figure it out later today, and if so qa it00:30
wgrantlifeless, abentley: https://translations.qastaging.launchpad.net/ubuntu/hardy/+source/gnome-applets/+pots/gnome-applets-2.0/bs/+translate, down the bottom.00:33
wgrantDoesn't seem to be fixed.00:34
lifelesswgrant: have you left rosetta experts etc?00:37
wgrantlifeless: I logged out.00:38
wgrantActually, I guess this might be a different bug.00:38
wgrantSince the bad string still shows up under Shared when I'm logged in.00:38
abentleywgrant: the english when logged out is "To prevent privacy issues, this translation is not available to anonymous users,00:39
abentleyif you want to see it, please, log in first.", which is correct.00:39
abentleywgrant: the fact that it says the bit about dummy translations under "shared" is another issue.00:40
abentleywgrant: 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
abentleywgrant: it was logged-in users without write access who were getting the issue.00:41
wgrantabentley: As an unprivileged user I see the contributions.00:43
wgrantBut same on production.00:45
abentleywgrant: I don't see how that could happen.  Do you see entry fields where you could put translations?00:47
wgrantabentley: No.00:47
wgrantIt's read-only.00:48
wgrantOh.00:48
wgrantHmm.00:48
wgrantDo suggestions count? I guess they probably do.00:48
abentleywgrant: they do.00:48
wgrantAh.00:49
wgrantI guess I can make it read-only if I change my licensing settings.00:49
wgrantApparently not.00:50
abentleywgrant: I believe that will prevent you from seeing it at all.00:50
wgrantI no longer have suggestion widgets, but the contributions are still there.00:50
abentleywgrant: my steps to reproduce involve setting the pillar to "closed" translation permissions.00:51
sinzuilifeless: skype again?00:55
wgrantabentley: I might revert the fix from DF, close Ubuntu there, and see what happens.00:56
lifelesssinzui: please00:59
lifelesssinzui: 5 minutes first, gotta ring a supplier00:59
sinzuiokay, 5 minutes00:59
wgrantlifeless: Still can't reproduce :(01:05
lifelesssinzui: https://bugs.launchpad.net/launchpad/+bug/74551201:36
_mup_Bug #745512: product-release-finder broken <qa-untestable> <regression> <Launchpad itself:Fix Committed by lifeless> < https://launchpad.net/bugs/745512 >01:36
poolielifeless: the privacy banner thing makes me think it would be nice to have a scope that matches user-agent02:22
wallyworld_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:28
wallyworld_i added something to lp/app/widgets/configure.zcml and took me a bit to figure out why it wasn't working02:29
lifelesspoolie: it would03:02
lifelesswallyworld_: well03:03
lifelessI'd expect lp/app/configure.zcml to list it03:03
lifelesswallyworld_: I don't know why it isn't listed03:03
wgrantOK, now why is buildout crashing with an assertionerror...03:04
wallyworld_lifeless: np. i'll add it. just wanted to check if there was a known reason for omitting it03:04
lifelesswallyworld_: not that *I* know of, but I'm not as cluey as say gary or curtis03:04
lifelesswhee03:05
lifeless771 Time Outs - yesterday03:05
lifeless6926 Exceptions03:05
wgrantHeh, yeah.03:05
wgrantgrar03:06
lifeless?03:06
lifelessalso03:07
lifelesssob03:07
lifeless SourcePackage.setpackaging() now deletes an existing Packaging record and creates a new one, if one exists,03:07
lifeless-> index bloat.03:07
wgrantYay03:07
lifelesslooks like we can deploy to 1282103:10
wgrantlifeless: Can you make compile on natty?03:13
wgrantI could until this morning :/03:13
lifelesswgrant: I don't know03:13
lifelesswgrant: (because I develop on lucid, our deployment environment)03:23
wgrantlifeless: Ah, of course.03:24
LPCIBotProject windmill build #177: STILL FAILING in 1 hr 3 min: https://lpci.wedontsleep.org/job/windmill/177/03:30
james_wpoolie, was bug 761327 supposed to be against launchpad?03:34
_mup_Bug #761327: would like a feature flag scope on browser user agent <feature-flags> <launchpadlib :Triaged> < https://launchpad.net/bugs/761327 >03:34
poolieoops yes03:41
poolie'too many matches' fail03:41
LPCIBotYippie, build fixed!04:20
LPCIBotProject devel build #640: FIXED in 5 hr 7 min: https://lpci.wedontsleep.org/job/devel/640/04:20
StevenKwgrant: https://code.launchpad.net/~stevenk/launchpad/drop-psc/+merge/5780706:10
StevenKlifeless: 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:11
wgrantStevenK: Let's not drop it yet.06:12
wgrantStevenK: Leave it running for a week to make sure we don't miss anything.06:12
StevenKMeh, it was five minutes work, it can wait.06:12
wgrantIt's possible (although rather unlikely) that there is still bad stuff being created.06:12
=== almaisan-away is now known as al-maisan
lifelessStevenK: I had a similar migration I turned off eagerly.06:15
lifelessStevenK: then it broke staging06:15
lifelessStevenK: *trust* me on this.06:15
StevenKAnd DF06:15
StevenK:-)06:15
lifelessonce all systems have either caught up on their own or been restored to it safe to nuke06:15
LPCIBotProject windmill build #178: STILL FAILING in 1 hr 7 min: https://lpci.wedontsleep.org/job/windmill/178/08:04
StevenKRargh, now it's rvba!08:33
rvbaStevenK: ?08:33
StevenKrvba: There's a 'hello-rvba' in dogfood's NEW queue!08:34
StevenKI think it's your doing!08:34
rvbaStevenK: right ... nice it went through.08:34
StevenKWell, it's in NEW, what do you want me to do with it?08:34
StevenKwgrant: Can haz a hand?08:34
rvbaStevenK: would be great yes ... and you could update df while you've at it?08:35
wgrantStevenK: 'sup?08:35
StevenKDF has been updated for 15 minutes.08:35
rvbaStevenK: ok then, I'll be able to qa yesterday's work08:35
StevenKwgrant: I'd like to force a maverick build record into NEEDSBUILD without it building.08:35
StevenKrvba: What I mean is that hello-rvba is blocking my QA -- shall I accept it, or reject it?08:36
wgrant/srv/launchpad.net/codelines/buildd-manager stop08:36
rvbaStevenK: accept it please08:36
StevenKwgrant: No such file or directory?08:40
adeuringgood morning08:41
StevenKwgrant: Right, buildd-manager killed08:41
StevenKrvba: Accepted, er, about 4 minutes ago08:42
StevenKrvba: I've only process-accepted, so it's out of the ACCEPTED queue08:42
wgrantStevenK: Hm, maybe it's just in plain /srv/launchpad.net.08:43
wgrantI can never remember.08:43
rvbaStevenK: anyway, I'm stuck because the page https://dogfood.launchpad.net/ubuntu/natty/+localpackagediffs is borked08:43
StevenKwgrant: Tis08:43
StevenKwgrant: Anyway, what's the next step?08:43
wgrantStevenK: Set one to NEEDSBUILD.08:43
wgrantSQL is your friend.08:43
StevenKwgrant: In BPB, right?08:43
wgrantStevenK: BFJ08:44
StevenKBleh, BFJ has no distroseries08:44
wgrantPB08:45
wgrantEr.08:45
wgrantdistroarchseries is on BPB, yeah.08:45
wgrantYou'll need to join BPB->PB->BFJ08:45
lifelessor merge them08:48
dpmhi 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:51
wgrantdpm: Is import_into set?08:55
dpmwgrant, what's import_into? I cannot find it in the api docs, and I did not set it08:56
wgrantdpm: 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:57
dpmwgrant, ah so perhaps the setStatus call is missing an import_into parameter, you think?08:58
dpm(I mean in the api)08:59
dpmor the import queue entry is missing the attribute08:59
wgrantdpm: Well, you'd need to set the target POFile or POTemplate.08:59
wgrantWhich means we'd need to export those two onto the API, which isn't done yet.09:00
dpmok, thanks wgrant, I think I'll file a bug, then09:01
henningewgrant, dpm: that is correct09:01
henninge(missing API export for approval)09:01
henningeAnd "Good morning" ;)09:01
henningeHi lifeless09:01
henningelifeless: looking at that QA09:02
dpmthanks henninge, and good morning ;) I'll file a bug then - any additional info I should mention there?09:02
lifelesshenninge: thanks!09:02
henningedpm: "Enable queue approval through API" should be enough. You can mention what wgrant just said about being able to set potemplate and pofile09:03
dpmok, thanks henninge09:04
henningedpm: but there may be more to it because POTempalte and POFile entries might need to geet created during approval.09:04
dpmok, I'll add all the info09:05
=== adeuring changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: adeuring | https://code.launchpad.net/launchpad-project/+activereviews
dpmhenninge, 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 <api> <lp-translations> <Launchpad itself:Triaged> <Ubuntu Translations:Triaged> < https://launchpad.net/bugs/671329 >09:20
dpm*noticed09:20
LPCIBotProject devel build #641: FAILURE in 5 hr 14 min: https://lpci.wedontsleep.org/job/devel/641/09:34
=== Ursinha-afk is now known as Ursinha
allenapadeuring: 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/5777810:11
adeuringallenap: sure, I'll look10:12
allenapadeuring: Thank you :)10:12
lifelessmrevell: you have mail10:37
mrevellThanks lifeless!10:38
=== al-maisan is now known as almaisan-away
lifelessmrevell: re the downtime10:50
lifelessmrevell: do you mean to say 'rob, you should tweet on identi.ca on monday morning about this'10:51
lifelessmrevell: or 'cannot do monday as there isn't enough warning' ?10:51
mrevelllifeless, 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.10:54
danilosadeuring, hi, got time to review https://code.launchpad.net/~danilo/launchpad/remove-primary-duplicate-reason/+merge/57839?11:21
adeuringdanilos: I'll look at your mp when I've finished a review for Gavin11:22
danilosadeuring, cool, thanks11:25
danilosadeuring, fwiw, this one is 40 lines, so shouldn't be too hard11:25
adeuringallenap: 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:36
allenapadeuring: Yeah, I left those there for completeness, but if they're puzzling then I shall remove them. Thank you for the review!11:37
jmlhttps://answers.launchpad.net/testresources/+addquestion is 34 queries for me. wtf.11:47
adeuringdanilos: r=me11:47
danilosadeuring, thanks11:48
jmllifeless: 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:49
jmlhttp://code.mumak.net/2008/10/testresources-some-examples.html fwiw11:54
LPCIBotProject windmill build #179: STILL FAILING in 1 hr 1 min: https://lpci.wedontsleep.org/job/windmill/179/12:13
bigjoolsjml: got a minute to talk about testtools?12:39
jmlbigjools: in a few minutes, need to grab some sandwiches from the kitchen first.12:40
bigjoolsok - just trying to work out how to use MatchesStructure and failing hard12:40
jmlbigjools: ah ok.12:45
jmlbigjools: I haven't used it before. Taking a look now.12:46
jmlbigjools: What are you trying to do?12:46
bigjoolsjml: ok I think I got it now.  The docs are wrong I think.12:46
jmlbigjools: what's the prob?12:46
bigjoolshttp://readthedocs.org/docs/testtools/en/latest/for-test-authors.html#matchers12:46
bigjoolsit says:12:46
bigjoolsmatcher = MatchesStructure({'a', Equals(1), 'b', Equals(2)})12:46
bigjoolsI had to use = MatchesStructure(a=Equals(1), b=Equals(2))12:47
bigjoolsseems like I am the first to use this matcher in LP :)12:47
jmlwouldn't surprise me :)12:47
bigjoolsit's very handy though12:48
jmlmwhudson wrote it for some Linaro stuff he was doing12:48
bigjoolsI can use it to test new model objects12:48
jmla thing that I often do with matchers is make new ones using lambda12:48
jmlIsAThing = lambda x, y: MatchesStructure(a=Equals(x), b=Equals(y))12:49
jmlbigjools: just landed a doc fix. thanks.13:07
bigjoolsjml: cool, thanks :)13:07
bigjoolsit was invalid syntax anyway :)13:07
jmlhah, yes.13:12
jmljust goes to show that the best documentation review is from someone who's trying to figure something out.13:13
lifelessmrevell: thanks please do13:16
lifelessjml: anywhere I chat on13:16
adeuringgary_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:23
gary_posteradeuring...I don't know. :-/  benji, any ideas?13:24
* benji looks.13:25
=== bac changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: adeuring, bac | https://code.launchpad.net/launchpad-project/+activereviews
bachi adeuring13:27
adeuringhi bac!13:28
benjiadeuring: I'm not certain, but I think using IField will give you a parameter that can take any value13:38
adeuringbenji: let me check again, but I think I tried that too,. just a second...13:39
adeuringbenji: InvalidInterface: Concrete attribute, IField13:40
benjihmm, let me look at it a little more13:40
adeuringbenji, 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 class13:41
adeuring...and no hassles with a "generic object" parameter13:42
benjiadeuring: oh, it wants a schema field, not an interface; if you still want to try something try using an instance of zope.schema.Field13:47
adeuringbenji: Ah, thanks!13:47
=== almaisan-away is now known as al-maisan
adeuringbenji: "@operation_parameters(obj=Field())"  results in "zope.interface.exceptions.InvalidInterface: Concrete attribute, Field". Or did I misunderstand your suggestion?13:54
benjiThat was my suggestion, unfortunately.13:55
benjiadeuring: if the parameter is an object, I'm pretty sure this will work: @operation_parameters(foo=Reference(schema=Interface, title=_('Foo')))13:58
adeuringbenji: zope.interface.exceptions.InvalidInterface: Concrete attribute, Field14:00
benjithat's really strange because LP uses code like that all over the place14:01
benjiI 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:02
benjis/your/you're/14:03
adeuringbenji: 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 think14:17
benjiadeuring: you may be right; I had forgotten about the "change the interface later" thing we do14:18
abentleyhenninge: https://code.launchpad.net/~abentley/launchpad/sharing-spinners14:20
abentleyhenninge: https://bugs.launchpad.net/launchpad-project/+bugs?field.tag=exploratory-testing+upstream-translations-sharing&field.tags_combinator=ALL14:31
jcsackettbenji: are you the right person to bother if i'm hacking on launchpadlib?15:27
benjijcsackett: most likely, what's up?15:27
jcsackettbenji: 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/test15:28
benjijcsackett: unfortunately the docs are not tested; I would love for that to change though.15:29
jcsackettbenji: 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
jcsackettthanks!15:29
benjinp15:30
=== al-maisan is now known as almaisan-away
abentleyadeuring or bac: could you please review https://code.launchpad.net/~abentley/launchpad/sharing-spinners/+merge/57888 ?16:11
adeuringabentley: I'll look16:11
abentleyadeuring: thanks.16:11
abentleyadeuring: I noticed a conflict.  Updating...16:12
adeuringok16:12
abentleyadeuring: updated.16:18
* adeuring is looking16:18
jcsackettadeuring, bac: can one of you take a look at https://code.launchpad.net/~jcsackett/launchpadlib/add-questionset/+merge/5789116:23
bacyes16:23
bacjcsackett: done.  easy peasy.16:27
jcsackettthanks, bac.16:28
* jcsackett wonders what the landing procedure is for launchpadlib...16:31
bigjoolscommit, pray16:31
jmljcsackett: yeah, that's an interesting question, isn't it.16:32
jcsackettjml: it is. i worry that bigjools is right. it looks like you just commit to a checkout version and push...16:32
bacadeuring: so we don't step on each other's toes would you be sure to hit the 'claim review' button?16:38
adeuringbac: whoops... ok16:38
bacthanks!16:38
bacjcsackett: are you going to update the version of launchpadlib LP uses?  i see we're way back on 1.9.216:39
bacer, 1.9.316:39
jcsackettbac: er, i haven't thought about the version that launchpad uses.16:40
benjijcsackett: https://dev.launchpad.net/HackingLazrLibraries#Landing%20your%20branches16:40
jcsackettthanks, benji.16:42
bacjcsackett: sorry, lost my internet16:43
bacjcsackett: it isn't required but we should bump it at some point16:44
jcsackettbac: dig.16:44
adeuringabentley: r=me16:54
abentleyadeuring: thanks.16:54
jcsackettsinzui: have some time to mumble?16:59
=== beuno is now known as beuno-lunch
sinzuijcsackett: sorry, screen locked up17:05
jcsackettsinzui: all good.17:05
sinzuijcsackett: I think my session is corrupted. I need a few more minutes17:08
jcsackettsinzui: ok.17:08
jcsackettsinzui: i could hear you. i think you cannot hear me?17:11
* jcsackett sees sinzui disconnected.17:11
=== beuno-lunch is now known as beuno
beunoI can't assign bugs to people18:08
beunois that a known problem?18:08
jmlbeuno: no, it's not a known problem.18:51
jmlbeuno: it works for me.18:51
beunojml, can I debug somehow?18:52
beunoI click on the person and nothing happens18:52
jmlbeuno: what browser?18:52
beunojml, FF 3.6, stock 10.1018:52
jmlhm.18:53
jmlbeuno: that's just weird.18:53
jmlbeuno: I'm in a bad position to help you right now.18:53
jmlbeuno: sorry.18:53
beunojml, np18:53
jmlg'night all.19:24
jmlWill see you around.19:25
abentleybac: could you please review https://code.launchpad.net/~abentley/launchpad/sharing-overlay-buttons/+merge/57935 ?19:55
bacabentley: sure, you'll be next19:55
abentleybac: thanks.19:55
=== Ursinha is now known as Ursinha-afk
abentleybac: resubmitted due to missing prerequisite branch: https://code.launchpad.net/~abentley/launchpad/sharing-overlay-buttons/+merge/5793819:59
bacabentley: looking now20:48
abentleybac: ack20:48
baclooks fine abentley20:52
abentleybac: thanks.20:52
abentleybac: Can you do one more?  It's real short: https://code.launchpad.net/~abentley/launchpad/fix-branch-precondition2/+merge/5794520:53
bacsure20:53
=== bac changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews
lifelessmoin21:22

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!