/srv/irclogs.ubuntu.com/2010/12/06/#launchpad-dev.txt

lifeless=== Top 10 Time Out Counts by Page ID ===02:17
lifeless    Hard / Soft  Page ID02:17
lifeless     296 / 6927  Archive:+index02:17
lifeless      83 /  377  BugTask:+index02:17
lifeless      43 /  169  ProjectGroupSet:CollectionResource:#project_groups02:17
lifeless      22 /  261  Distribution:+bugs02:17
lifeless      12 /   29  Archive:+copy-packages02:17
lifeless      10 /  344  POFile:+translate02:17
lifeless      10 /   27  MailingListApplication:MailingListAPIView02:17
lifeless       9 /   34  DistroSeries:+queue02:17
lifeless       8 /   29  Milestone:+index02:17
lifeless       8 /   11  DistroSeriesLanguage:+index02:17
wgrantlifeless: Do you have a +copy-packages OOPS? Is it repeated DistroArchSeries queries again?02:40
wgrantRepeated DistroArchSeries and Archive queries, inf act.02:40
wgrant(wow, that is a lot of Archive:+index soft timeouts. I wouldn't have guessed that it had that many hits)02:41
lifelesswgrant: this is why I worry :)02:54
wgrantlifeless: If you'd let PQM be taken out of RC earlier, it would be fixed by now :P02:57
wgrantlifeless: Could you check a +copy-packages OOPS? I think it's the same issue as +queue -- SPR.getBuildByArch being awful.03:02
lifelesswgrant: I'm on leave, nutting to do with me ;)03:05
wgrantPfft.03:06
lifelessArchive:+index 2075303:06
lifeless99% under 15.8403:06
wgrantOw.03:07
lifelesswgrant: I do03:09
lifelesswgrant: https://bugs.launchpad.net/soyuz/+bug/57545003:14
_mup_Bug #575450: Archive:+copy-packages nearly unusable due to timeouts <ppa> <timeout> <Soyuz:Triaged> <https://launchpad.net/bugs/575450>03:14
wgrantlifeless: Wow, those are really awful queries.03:15
lifelesstada03:15
wgrantNot what I thought it would be.03:15
wgrantThanks.03:15
lifelessand 150 noddy repeats03:15
wgrantThose are hopefully fixed by my Archive:+index branch.03:16
wgrantBut they will probably reveal more.03:16
lifelesssure03:16
lifelessa query scaling test for copy-packages would be nice03:16
wgrantBut by that point I probably won't have to pester you.03:17
wgrantYeah.03:17
wgrantBut it's so awful at the moment that it's pointless to test.03:17
lifelesshttps://launchpad.net/ubuntu/+archive/primary/+copy-packages03:17
lifelesslooks similar to me03:17
lifelesswgrant: its never pointless to stop the erosion03:17
wgrantlifeless: Pointless? No. Depressing and embarrassing? Definitely.03:18
wgrantlifeless: I think we need to segregate OOPSes by HTTP method.04:17
wgrantLooking at that +copy-packages OOPS again, it's a GET, not a POST.04:18
wgrantSo it's just batchnavigator's COUNTing stupidity.04:18
lifelesswgrant: file a bug on oops-tools, I think that that is a good, valid, heuristic.04:42
wgrantlifeless: k, thanks.04:43
wgrantlifeless: How are we going to fix batchnavigator? Can we get postgres to give us an estimate?04:56
lifelessbroadly04:57
lifelessa) there is an estimator in the code base already04:58
lifelessb) we don't want to pass storm result sets outside the persistence layer anyway04:58
lifelessc) while counting can be slow, we should be able to do better than 3 seconds04:58
lifeless\o/ parallel testing for testr07:21
lifelessabout 0.3 seconds (25%) off of the time to test testtools.07:22
wgrantI love well-designed tests.07:38
lifelesswgrant: did you file that bug about presentation of the 'Ubuntu also tracks bugs for packages derived from this project: bzr in ubuntu.' links ?07:50
wgrantlifeless: I didn't.07:50
wgrantI feel like I should talk to someone first.07:50
lifelessthats a mispattern07:50
lifelessI'm sure you can point out that the current approach doesn't scale to even just the distros hosted in LP>07:51
lifelessthe mispattern is needing discussion *before* creating a place to talk about the issues07:52
wgrantWell, It is tempting to at least raise it with Curtis first.07:52
lifelessits been 1? 2? weeks.07:53
lifelessI call timeout07:53
lifeless(the timeout should be ~ 5 minutes)07:53
wgrantWellll, I forgot.07:53
lifelesswgrant: exactly07:53
wgrantYou are on leave; you cannot call timeout :P07:53
lifelesscan too; nyarh nyarh nyarh07:53
jelmermørning lifeless, wgrant07:53
lifelessø?07:54
wgrantEvening jelmer.07:54
jelmerI'm feeling creative this morning.07:54
wgrantMy compose key is sadly broken :(07:54
lifelessö07:54
wgrantThére wë go.07:54
jelmerwgrant: do you need one for Strine?07:54
wgrantjelmer: Apparently.07:55
=== almaisan-away is now known as al-maisan
adeuringood morning08:26
al-maisanmoin adeuring :)08:27
adeuringhey al-maisan!08:28
lifelesswgrant: btw, how many cpus do you have?08:28
wgrantlifeless: 6 cores.08:29
wgrantMorning adeuring, al-maisan.08:29
lifelesswgrant: you might want to try glueing latest testr to running lp tests in vm's.08:29
lifeless(one vm per test command invocation)08:29
al-maisanhello wgrant, how are things?08:29
lifelesswgrant: (if you want 6 way concurrency in an lp test run)08:30
wgrantal-maisan: Pretty good.08:30
wgrantEnjoying my final week of freedom.08:30
al-maisan:)08:30
al-maisanthere are some pretty interesting discussions going on re soyuz :)08:30
wgrantYou could say that.08:31
StevenKwgrant: Yeah, right. You're spending it working on Launchpad08:32
wgrantStevenK: Says he who is on leave but present...08:32
lifelessStevenK: if you love your job, you never need to work :)08:32
StevenKI'm too attached to IRC to /quit08:32
al-maisantrue .. but also slightly dangerous ;)08:32
* StevenK is going to spend some time this work working on Hudson08:33
* jelmer waves to StevenK, al-maisan, adeuring08:34
StevenKjelmer: O hai!08:34
* al-maisan waves back :)08:34
wgrantlifeless: Is there any documentation on the testr parallelisation functionality?08:35
wgrantTest runs in half an hour would be pretty cool.08:35
lifelesswgrant: MANUAL.txt08:35
lifelesswgrant: testr run --help08:35
lifelesswgrant: NEWS08:35
wgrantlifeless: I suppose I'll need a very recent (!)release?08:36
lifelesstip08:36
wgrantk08:36
lifelessits not finalised08:36
mwhudson_morning jelmer08:36
lifelessI may tweak the config if it doesn't fit well with our needs here08:36
jelmerhey Michael08:36
jelmermwhudson_: Thanks for landing that C introspection branch :-)08:37
wgrantBah, fixtures isn't packaged :(08:38
jelmerwgrant: it is in the testing-cabal PPA08:38
wgrantjelmer: Perfect, thanks.08:38
mwhudson_jelmer: thanks for the prods08:38
jelmerwgrant: ppa:testing-cabal/archive08:38
lifelesstip testr is in that ppa too08:42
lifelesswgrant: its also in Debian.08:42
wgrantMaybe I should go back to unstable for a while.08:44
lifelessor natty08:44
wgrantI trust unstable more.08:44
lifelessunstable is ~frozen08:44
wgrantBut it doesn't have Unity rewrite crack :)08:45
wgrantAnyway.08:45
* wgrant investigates testr --parallel08:45
al-maisanwgrant: you can use natty with the normal gnome desktop as well08:45
wgrantBut NM doesn't seem to really work :/08:46
StevenKlifeless: Off the top of your head, will the Hudson EC2 plugin just deal with and mount an EBS volume, or does that require changes? I can't find much TFM to R08:51
lifelessshould just work08:51
lifeless^-WAG08:52
bigjoolsmorning08:54
wgrantMorning bigjools.08:55
bigjoolsinbox most definitely not zero today08:56
wgrantI should clean up mine this week.08:57
wgrantBecause I guess I will acquire a new problematic one soon.08:58
wgrant:(08:58
mwhudson_i had a soul-crushing amount of email this morning08:58
bigjoolswgrant: you already subscribe to bugmail, that forms 80% of mine08:59
bigjoolsthen the oops reports08:59
bigjoolsthen warthogs on a bad day :)08:59
wgrantbigjools: There's a lot of bugmail, sure, but it's nice and quick to get through.08:59
bigjoolshahaha09:00
mrevellHello09:13
bigjoolsg'day mrevell09:22
thumperwgrant: just one week to go :)09:25
thumpermwhudson_: yeah, saw your tweet09:25
* thumper goes to make a cuppa tea09:25
thumperhi mrevell09:25
* thumper waves at StevenK09:25
* thumper thinks that is everyone now09:25
mrevellhaha09:25
mrevellGood work thumper09:25
wgrantthumper: Indeed!09:27
* bigjools pouts09:43
wgrantUhoh.09:44
bigjoolswgrant: https://bugs.launchpad.net/soyuz/+bug/68507609:44
_mup_Bug #685076: PPA deletion leaves unremoved publications <Soyuz:New> <https://launchpad.net/bugs/685076>09:44
bigjoolsARGH.09:44
wgrantbigjools: How is that ARGH-worthy?09:45
bigjoolsbecause I overlooked it09:45
wgrantHeh.09:45
bigjoolsthere's a pheasant outside my window.  Pie anyone?09:46
lifelessmmm09:46
lifelessPIE09:46
bigjools:D09:46
bigjoolswgrant: so all these bugs you're filing!09:48
bigjoolsshall I just assign you to them now? :)09:48
wgrantMaybe.09:49
bigjoolswgrant: I am not clearly following the timeline you wrote in bug 682692.  Could be my lack of caffeine though.09:50
_mup_Bug #682692: Some PPAs have duplicated builds <soyuz-build> <soyuz-publisher> <Soyuz:Triaged> <https://launchpad.net/bugs/682692>09:50
wgrantbigjools: What's ununderstandable?09:52
bigjoolsI am having trouble understanding where the problem is in that example timeline09:53
wgrantbigjools: The first copy was deleted 18 seconds before the second copy was performed.09:54
wgrantSo the status summary returned FULLYBUILT instead of FULLYBUILT_PENDING, so the copy was permitted with unpublished builds.09:54
bigjoolswgrant: can you condense that comment into a simple instruction to re-create the bug?09:55
bigjoolsit will double as a QA test when we fix it09:55
bigjoolscongrats on figuring it out09:55
wgrantbigjools: Done.09:58
bigjoolswgrant: rock!  thanks09:59
wgrantHmm. Should I expect an acknowledgement when I reply to an rt.admin.canonical.com ticket?09:59
bigjoolswgrant: https://answers.launchpad.net/soyuz/+question/13668510:06
bigjoolswgrant: RT should reply but since it doesn't know you I expect it won't10:06
wgrantbigjools: Have you looked at the end of that question?10:07
bigjoolsthe massive list of files you mean? :)10:07
wgrantI was thinking more about my response and reviewed branch.10:07
bigjoolsoh didn't get that far10:08
bigjoolscarry on10:08
bigjoolswgrant: any further comments re my bug comment: https://bugs.launchpad.net/soyuz/+bug/684180/comments/3 ?11:52
_mup_Bug #684180: Deleted sources can leave binaries hanging around forever <soyuz-core> <soyuz-publisher> <Soyuz:Triaged> <https://launchpad.net/bugs/684180>11:52
wgrantbigjools: "Argh" just about covers it.11:53
bigjoolsthat's my Word of the Day11:53
wgrantDeletion won't cover everything.11:53
wgrantWhat if the source is superseded instead?11:53
wgrantWe can't block that.11:53
wgrantWe can, instead, cry.11:53
bigjoolsit won't but it covers this case11:54
bigjoolswe can block superseding surely?11:54
bigjoolsit'll get caught later11:54
wgrantPllllllease don't.11:54
wgrantThat violates the very little layering we have :(11:54
bigjoolsthe layering sucks11:55
wgrantWhy?11:55
bigjoolswe don't have layers, we have something organised into modules11:55
wgrantMaking the publisher depend on build statuses is unprecedented.11:55
wgrantAnd slow.11:55
bigjoolsin that case we need to make domination do more work11:56
bigjoolswhich is equally as unpaletable11:56
wgrantWe need to think about how binaries and sources interact.11:56
wgrantFor copies, for deletions, for dominations.11:56
bigjools"badly" :)11:56
wgrantIt is all tied together.11:56
wgrantAnd it is only consistent in one respect: it is fucked.11:56
=== Ursinha-afk is now known as Ursinha
* bigjools loses mouthful of coffee11:57
bigjoolswgrant: we can't auto-supersede binaries if the source is superseded12:00
bigjoolsthey might be a dependent12:00
wgrantI know12:00
wgrantExcept that's not the reason.12:00
bigjoolsmy main issue is preventing the manual deletion that people do in PPAs12:00
bigjoolswhat reason?12:00
wgrantWe don't seem to care about breaking dependencies.12:01
deryckMorning, all.12:01
bigjoolshowdy deryck12:01
bigjoolswgrant: why do we have this then? http://people.canonical.com/~ubuntu-archive/NBS/12:02
wgrantI guess it is sort of dependency-related.12:02
wgrantBut not entirely.12:02
wgrantRight.12:02
wgrantNBS.12:02
wgrantI hate NBS.12:02
bigjoolsthat's why we can't auto-supersede the binaries12:02
wgrantI don't like introducing a hack to partially fix a problem without at least thinking about how to fix the whole thing.12:03
bigjoolsI contest your definition of a hack12:04
wgrantBlocking deletion because builds are in the wild is a hack.12:04
bigjoolsI disagree12:04
wgrantWe should just make them be created deleted. Or something like that.12:04
bigjoolsthat's a hack too then ;)12:05
wgrantIt's a less obtrusive one.12:05
bigjoolsGRRR what has a maverick update done to my fonts12:05
wgrantSo, we can't really reject the binaries if the source is deleted. Blocking deletion is undesirable. This leaves us with accepting binaries for a deleted source.12:07
bigjoolswe can just throw them away and mark the build superseded12:07
bigjoolsif the build is not active then we mark it superseded immediately12:07
wgrantWe could. But that seems like a waste.12:08
bigjoolssomeone wants the package deleted - we're just doing what they want.12:08
wgrantI think most of my problems with this area stem from the way we fail to treat sources and their binaries as a single unit.12:11
wgrantAnd the way this interacts with copies.12:11
bigjoolshow would you treat them as a single unit?12:13
wgrantWe already do. For copies. And in the UI. Everywhere except most of the code.12:14
bigjoolshow would you do that in the code?12:14
wgrantI don't know exactly. But it's terribly confusing the way it is now: the same source can have different builds in different series and different archives. Copy from Hardy to Lucid, and you can no longer copy from Lucid to anywhere else. Copy one of those to another archive, and it will have all the same archs, but exactly one of them is actually a different build. Copy to a non-virt archive, it now contains another superset of the builds ...12:18
wgrant... of the original archive.12:18
wgrantWant to do a security update? hppa is being slow. Let's copy without it.12:18
wgrantOh look, can't copy it any more.12:18
bigjoolsyou conveniently forgot about copy with binaries12:20
bigjoolsor mixed examples with it set/not set12:20
wgrantAll those cases are copying with binaries.12:20
wgrantWithout binaries it's boring.12:20
bigjools" one of them is actually a different build" - how?12:21
wgrantbigjools: Because I copied from Hardy in one PPA to Lucid in another. Lucid has armel, Hardy does not. Lucid gets a new armel build, different from the one in the origin archive.12:22
bigjoolswhy are we creating new builds for binary copies?12:23
wgrantBecause the target series or archive can have more architectures.12:23
bigjoolsthat's crack12:23
wgrantI think it probably is.12:23
bigjoolsit's a binary copy, I don't expect more builds12:23
wgrantIt's also really, really slow.12:23
wgrantBut inter-series it becomes awkward.12:24
wgrantMy intuition says that builds should live in a set. And only sets should be copiable. Atomically. Inter-archive that's fine, since if you want more archs you can just rebuild it all.12:25
wgrantBut inter-series it doesn't work.12:26
bigjoolsyup12:26
wgrantSo perhaps there should be an exception, where a set can be extended within an archive by copying to a new series.12:27
wgrantBut then when somebody copies someone else's set to another archive, they are screwed. Because they can't append to it, and they can no longer rebuild it.12:27
wgrant"Argh"12:28
bigjoolswgrant: but soyuz isn't that complicated, don't worry.12:29
bigjoolswe'll just throw more people at it12:30
wgrant:)12:31
wgrantYes, the model is somewhat overcomplicated... but it can't be simplified away.12:31
bigjoolsit's the nature of the beast12:32
=== mrevell is now known as mrevell-lunch
bigjoolswgrant: so I think I'll amend my suggestion on that bug to just throw away the builds that are in progress and mark the build record as superseded.13:04
wgrantbigjools: When they try to finish?13:04
bigjoolsI just need to decide whether to do it in p-a or the b-m13:04
bigjoolsyes13:04
bigjoolsif they are not dispatched yet then we can supersede immediately13:05
wgrantThey'll supersede automatically when they try to start.13:05
bigjoolsgood point, well made13:05
bigjoolsI was thinking only of deletions13:05
bigjoolswhat do you think of the cleanup SQL I propose to do?13:06
wgrantAs long as there are no Published sourcepubs remaining, sure.13:08
bigjoolsyeah, it's just superseding binaries where the source is already superseded without being published at all13:08
bigjoolsshould be safe13:08
wgrantIt's possible that it was revived later. But I guess we could ignore that.13:09
bigjoolssome of the sources date back to 200913:09
wgrantOnly?13:09
bigjools2009-10-06 is the earliest13:09
wgrantHmmmm.13:10
wgrantThat's pretty strange.13:10
bigjools245 SPPHes13:10
* bigjools -> fud13:21
=== mrevell-lunch is now known as mrevell
=== henninge_ is now known as henninge
=== Ursinha is now known as Ursinha-lunch
allenapI have a new frozenset subclass. When it gets wrapped in a Zope security proxy nothing is available, not even the usual frozenset methods. Does anyone know how to configure this in ZCML (or otherwise) without repeating the existing configuration. Right now I think I'm going to modify BasicTypes directly.14:26
flacosteallenap: that's weird because frozenset is already part of _default_checker in zope.security.checker14:34
allenapflacoste: Yeah :-/14:34
flacosteallenap: have you try the security debugger on it?14:35
=== matsubara is now known as matsubara-lunch
allenapflacoste: Is that ZOPE_WATCH_CHECKERS? Not yet.14:35
flacosteallenap: no, it's something else14:35
flacostehang on14:35
flacostelooking for it14:35
flacostesomething i write a long time ago14:36
flacosteit's in lazr.restful.debug14:36
flacostefrom lazr.restful.debug import debug_proxy14:36
flacosteprint debug_proxy(obj)14:36
flacosteallenap: it will show all layers of wrapping, along the checkers registerd for security proxy14:37
allenapflacoste: Cool, that sounds useful.14:37
allenapflacoste: It gets: zope.security._proxy._Proxy (using zope.security.checker.Checker)14:39
flacostehmm14:41
flacosteallenap: is that all?14:42
allenapflacoste: Yes! That's it :)14:42
flacosteallenap: it's weird, because it should be showing the permission declared on it14:42
flacosteso it's like if the read_permission and write_persmission were unset14:42
flacostebut the default _setChecker14:42
flacosteis a NamesCheker that grant access to all the set procotol14:42
flacostei suspect that something is chaning the default one14:43
flacostehand on, i'll try something14:43
allenapflacoste: Yeah, that's what's stumping me.14:43
flacosteshit, my tree is out of date14:44
flacosteallenap: try printing the checker in make harness on devel14:44
flacostesee if it is the same thing14:44
allenapflacoste: That's where I did it.14:44
flacosteallenap: ok, let me compare notes (after my standup)14:46
allenapflacoste: Okay, thanks.14:47
flacostemrevell, jml: http://leankitblog.com/2010/12/10-kanban-boards-leankit-kanban-style/14:51
=== salgado is now known as salgado-lunch
=== matsubara-lunch is now known as matsubara
=== Ursinha-lunch is now known as Ursinha
=== deryck is now known as deryck[lunch]
timrcWe're granting people/ teams upload rights to ppas via newComponentUploader() in launchpadlib.  Our problem is that, when we do this, the PPA does not show up in +archivesubscriptions -- seems to me this should be happening?  Am I off my rocker?16:04
bigjoolstimrc: sounds wrong.  Can you file a bug with the details to re-create that please.16:06
timrcbigjools: sure, np16:06
bigjoolstimrc: presumably it's a private PPA?16:06
timrcbigjools: yep16:06
timrcbigjools: I'll be sure to note that detail in the bug report16:07
bigjoolswell you can't have subscriptions on them otherwise :)16:07
timrcbigjools: the only way I know of to expose the subscription is to use Manage access via LP16:07
bigjoolsthought I'd check the blindingly obvious first16:07
bigjoolsyou can use the API as well16:07
timrcbigjools: but maybe +archivesubscriptions only lists archives that you can install from and not ones you can upload too? I dunno, I'm a newbie :)16:08
=== salgado-lunch is now known as salgado
timrcand not ones you can _only_ upload too?16:08
bigjoolstimrc: it's possible that's happened, yes, and it would be a bug if that's the case.16:09
=== beuno is now known as beuno-lunch
gary_postersinzui: might you know anything about https://bugs.launchpad.net/launchpad-foundations/+bug/683945 ?  I don't have anyone ATM who does.16:25
_mup_Bug #683945: disabled lists are never completing disabling <canonical-losa-lp> <Launchpad Foundations:New> <https://launchpad.net/bugs/683945>16:25
flacosteabentley: how's the release is going?16:32
abentleyflacoste: It's going fine.  We've nearly got QA complete.16:32
flacosteabentley: did we sort out the DKIM QA?16:33
abentleyflacoste: poolie was supposed to do it on (his) saturday, but he hasn't updated the tags.16:33
flacosteabentley: should we consider reverting it?16:34
flacosteabentley: and should we do a no-downtime today with bzr 2.2.2?16:34
abentleyflacoste: I am hoping that he's done it and just forgotten to update the tags.16:34
abentleyflacoste: I was going to chase it down today.16:35
flacosteabentley: ok, let's see what he says when he comes online later today16:35
flacosteabentley: what about bzr 2.2.2?16:35
abentleyflacoste: I don't feel strongly either way.16:36
flacostehmm, ok16:37
=== deryck[lunch] is now known as deryck
abentleyflacoste: If we do, we might as well do 1206 as well, so we can nix both cowboys.16:38
flacosteabentley: the cow-boys are not part of the nodowntime set16:40
flacosteso it's probably not that useful16:40
abentleyHmm, I thought one of them was not part of the nodowntime set because of the cowboys.16:41
=== mrevell_ is now known as mrevell
gary_posterlosas, https://bugs.launchpad.net/launchpad-foundations/+bug/684672 is dealt with, right?  The bug report doesn't give a status update17:01
_mup_Bug #684672: staging update broken on revno 10029 <canonical-losa-lp> <Launchpad Foundations:Triaged> <https://launchpad.net/bugs/684672>17:01
gary_postereh, sorry, going to ops17:02
=== beuno-lunch is now known as beuno
=== benji is now known as benji-lunch
timrcbigjools: So, I found a minute to play around.  You have to call newSubscription() to have the PPA appear in  +archivesubscriptions .. which is fine as being 'subscribed' to the archive simply means you can install from it.  I suppose there is a use case where you'd like to make the archive 'write-only' in that a person / team can upload to the PPA, but not install from it (e.g. the PPA is upstream to another archive that is inst17:40
timrcallable)17:40
bigjoolstimrc: actually I think we have a bug somewhere about having implicit subscriptions for PPA uploaders.17:41
timrcbigjools: so I think the takeaway here is that it'd be nice to see the access rights you have on a private PPA, regardless of subscription.. is there a way to do this from the UI?17:41
bigjoolsyou can query what rights someone has via the API17:41
timrcbigjools: I was hoping there'd already be something out there to support our users :) but I'll put this on my list of TODOs17:41
bigjoolsyeah some of this is a little clunky still17:42
timrcbigjools: well I appreciate your time, thanks17:44
bigjoolstimrc: no problem, let me know if you have more questions17:44
sinzuigary_poster, sorry I was OTP. I marked the bug as fix released. Chex and I resolved the stuck Mm + XMLRPC combo18:24
=== benji-lunch is now known as benji
mkanatHey hey. Any hope of getting my loggerhead update merged and staged?18:41
mkanatThis is the MP: https://code.launchpad.net/~mkanat/loggerhead/launchpad/+merge/4233818:43
lifelessthat won't get it deployed18:43
lifelessyou need to propose a source dep change to launchpad itself18:43
mkanatlifeless: Right, but first I have to merge to this branch, which I don't control.18:44
lifelessmmm, well both things I need18:44
lifelessanyhow, right now - we're frozen.18:44
mkanatlifeless: Okay. How long does that last?18:44
lifelessI'm not sure, i'm on leave - check the -dev list archive, dates should have been announced there18:44
mkanatlifeless: Okay, thanks.18:45
=== al-maisan is now known as almaisan-away
gary_posterack, thanks sinzui18:59
=== Ursinha is now known as Ursinha-brb
lifelesssigh, the python3 shenanigans are just plain tiring19:21
benjilifeless: which shenanigans in particular are getting you down?19:23
lifelessbenji: most currently the transform stuff19:23
lifelessbenji: but more generally the way its just so painful to do widespread compatibility with19:24
* lifeless things py3k was a mistake19:24
benjiunfortunately I have to agree; too much incompatability for such small gains19:26
benjiI still think someone who wanted to be BDFL 2.0 could take up Python 2 and run with it.19:26
mkanatbenji: What I hear is that 2.7 is the last 2.x release ever.19:36
benjimkanat: right, the last official one; the python-dev folks won't be working on any more releases, but someone else could take it up -- they'd have to use a name other than "Python" though19:37
mkanatbenji: Ah, yeah.19:37
lifelessbenji: they could use Compatython19:40
benjiheh19:40
lifelessbenji: hi19:45
lifelessbenji: can you join #launchpad ?19:45
lifelessbenji: (generally every dev should be in that channel btw)19:45
=== Ursinha-brb is now known as Ursinha
thumpermorning19:55
* thumper relocates19:55
=== matsubara is now known as matsubara-afk
mhall119question, if I have a launchpad username, can I get their openid identity_url from the launchpad API?20:13
thumpermhall119: what problem are you trying to solve?20:16
=== salgado is now known as salgado-afk
mhall119loco.ubuntu.com pre-creates Django users based on a team's admins20:21
mhall119however, when those admins go to log in to loco.u.c, django-openid-auth sees an existing user with that username, and instead assigns them to $username+120:22
mhall119if they already had their openid identity_url set, it would just log them into the correct account20:23
mhall119see https://code.launchpad.net/~mhall119/django-openid-auth/fixes-639772/+merge/3854520:23
EdwinGrubbslifeless: what do you think of this solution for specifying the store used by Set classes? http://pastebin.ubuntu.com/540423/20:51
lifelessEdwinGrubbs: that makes a context specific item a global behaviour: thats undesireable20:51
EdwinGrubbslifeless: what do you mean? The dbpolicy is only being used during the duration of the with-statement.20:53
lifelessright20:53
EdwinGrubbslifeless: Does that mean that the BaseDatabasePolicy shouldn't have had the __enter__() and __exit__() methods added so that someone could use "with MasterDatabasePolicy()"20:56
lifelessEdwinGrubbs: the with statement you wrote changes the selector for all threads at once AFAICT20:58
lifelessEdwinGrubbs: it may well mean that the __enter__ and __exit__ are bad ideas outside of the test suite.20:59
lifelessEdwinGrubbs: but even if it was threaded21:00
lifelessEdwinGrubbs: its still a risky idiom to change context-wide state when calling an object will a defined lifetime greater than ones own contect21:01
lifeless*context*21:01
EdwinGrubbslifeless: then, Person.getOrCreateByOpenIDIdentifier() should probably stop using  a policy as a context manager also.21:01
lifelessEdwinGrubbs: myself, I would have used the person_validity_queries static method that was on Person and not put anything onto PersonSet21:02
lifelessEdwinGrubbs: re Person.getOrCreateByOpenIDIdentifier - yes, I suspect so.21:02
lifelessEdwinGrubbs: I had a quick look through zope.component and zope.interfaces to see if I could find a threaded context in there - I couldn't, but its pretty convoluted code, so it could be anything.21:03
mhall119if I have a launchpad username, can I get their openid  identity_url from the launchpad API?21:03
james_wmhall119, it doesn't look like it21:04
lifelessmkanat: https://api.launchpad.net/+apidoc/ has the api docs21:04
lifelessbah21:04
lifelessmhall119: ^21:04
lifelessmhall119: That said, I think its probably undesirable to share that information about users to people other than the user themselves.21:04
mhall119lifeless: if you view the source of https://launchpad.net/~lifeless21:05
mhall119it's in there21:05
mhall119<link rel="openid2.local_id" href="https://login.launchpad.net/+id/kPbPBDC" />21:05
lifelessmhall119: hmm, we should nuke that21:06
lifelesswe use the ubuntu sso these days21:06
mhall119why?21:06
lifelesslaunchpad does not contain an authentication database.21:06
lifelesswe delegate to sso.ubuntu.com21:06
lifelesslogin.launchpad.net is just a skin on sso.ubuntu.com21:06
mhall119okay, any way to get the identity url from sso?21:07
mhall119given a launchpad username21:07
lifelesstry asking in #canonical-isd21:08
mhall119thanks21:08
lifelesspersonally though, I'd just stop preallocating things21:08
lifelessand do admin lookup on-demand.21:08
mhall119we setup "user profiles" for admins in a team, so we can display their real names21:08
mhall119but that requires that we have a django user account for them21:09
mhall119which throws off django-openid-auth once the admin actually tries to log in for the first time21:09
lifelessright21:11
lifelessso do that setup *after* they login21:11
lifelessits just a thought21:11
cody-somervilleIs this page cached? https://code.launchpad.net/~offspring-hackers21:23
cody-somervilleThat page says the branch was modified 1 minute ago - the branch was last modified on the 3rd.21:23
lifelessmetadata changes count too21:24
cody-somervilleah21:26
cody-somervilleright21:27
lifelessmwhudson: get a losa to set the review bits properly21:43
marsflacoste, pong?21:50
flacostehi mars21:50
=== Ursinha is now known as Ursinha-afk
benjigrr, the frequent firefox grey-screen-of-wait I'm getting lately is irritating22:01
poolielifeless/whoever: is there a bug or rt for live log feeds from production?22:39
poolieshould i file one?22:39
pooliewbn22:39
lifelessrealtime? possibly22:39
lifelessup to you22:39
wgrant17 minutes for DB changes!? I thought the last estimate was slightly under two hours...22:45
lifelessrecife branch was rolled back22:45
wgrantAhh.22:45
poolielifeless: well, at least frequently you don't have a multi-minute wait22:46
poolieor apparently a 24h wait22:46
wgrantAren22:47
wgrantAren't they synced every 5 minutes?22:47
lifelesspoolie: EPARSE22:47
poolienot from staging, apparently22:47
poolielifeless: if something i changed produces an error or debugging message on staging, i would like to be able to see that message22:48
poolieat the moment i need to either prod a losa, or wait 24h22:48
poolieiwbn if it was available to developers less than a minute after being emitted22:49
lifelesspoolie: sure, sounds nice to be better at22:49
lifelessI wouldn't aim at realtime just yet22:49
poolienear-22:49
lifelessstaging load is very high22:49
wgrantIs the scripts split going to happen at some point?22:50
lifelesspoolie: I don't want to think about this deeply on leave22:50
lifelesspoolie: so, as I said before, up to you22:50
pooliesure22:54
poolieabentley, i'm back to qa now23:43
pooliei think 643219 and bug 316272 are ok23:45
_mup_Bug #316272: launchpad should verify gmail or DomainKeys authenticators <dkim> <feature> <qa-needstesting> <Launchpad Registry:Fix Committed by mbp> <https://launchpad.net/bugs/316272>23:45
pooliespiv/wgrant/abentley: would you like to read my qa steps on those bugs and suggest any others?23:46
wgrantpoolie: Have you tried DKIM to an existing bug?23:47
pooliei have, and it worked23:48
wgrantSounds good then.23:48
poolieand gpg is still accepted23:48
wgrantYep, saw that.23:48
abentleypoolie: ideally, your reviewer will have seen your testing plan as part of your merge proposal, but I trust your judgement.23:55
pooliethat would be good23:55
poolienobody asked me for one23:55
pooliei'm just checking the last bug is ok23:55
poolieit doesn't seem to have regressed, at any rate23:55
pooliei should now manually retag them qa-ok?23:55
abentleypoolie: yes.23:56
poolieand they'll automatically go to fixreleased during rollout to lpnet?23:57
abentleypoolie: we have a standard template for proposals, which includes a section for testing it.  But very few people are using the template.  I should bring that up at the reviewer meeting.23:57
abentleypoolie: You don't have to worry about marking them fixreleased.  I believe sinzui runs a script to do that.23:58

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