/srv/irclogs.ubuntu.com/2010/10/26/#launchpad-dev.txt

LPCIBotProject devel build (151): STILL FAILING in 3 hr 37 min: https://hudson.wedontsleep.org/job/devel/151/00:05
LPCIBotLaunchpad Patch Queue Manager: [r=abentley][ui=none][bug=620615,00:05
LPCIBot659078] change Launchpad to use Storm 0.1800:05
=== matsubara-afk is now known as matsubara
wallyworldthumper: they can suck, you just have to know how to tune them :-) i think storm may not be as advanced as say hibernate in this area00:25
thumperwallyworld: I'm attacking a place in the code where there is simplistic thinking00:25
wallyworldhibernate has all sorts of goodies, including support for batch updates and mixing native sql with it's object query language (HQL)00:25
thumperwallyworld: so does storm00:25
* wallyworld needs to learn more storm00:26
* wallyworld wonders, does storm support per query fetching strategies and pre-fetch sizes, or lazy instantiation of collections?00:28
thumperwallyworld: kinda00:37
wallyworldit's stuff like all that which is needed to properly tune one's app of course00:38
LPCIBotProject db-devel build (98): SUCCESS in 3 hr 37 min: https://hudson.wedontsleep.org/job/db-devel/98/00:39
LPCIBotLaunchpad Patch Queue Manager: [rs=thumper][ui=none][no-qa] Manual merge of stable into db-devel.00:39
wallyworldhey thumper, i'm likely the last to realise - i suppose you know about http://blogs.computerworld.com/17224/ubuntu_changes_its_desktop_from_gnome_to_unity00:40
thumperno00:40
wallyworldsomeone emailed it to me00:40
* wallyworld gets out the tissues and blubbers over how unloved KDE is :-(00:43
jcsackettwallyworld: if it helps, i know in OS X circled KDE 4 got serious circulation. :-P00:43
jcsacketts/circled/circles/00:44
wallyworldit would help more if KDE had more support on Ubuntu :-)00:44
wallyworldat least maybe there's a growing realisation that Qt is good. one step forward...00:46
jcsackettit helps that Qts license situation got cleaned up.00:47
jcsackett(or the FUD around it got cleaned up; whatever your preference might be)00:47
wallyworldjcsackett: same thing really, given the impact it had...00:49
jcsackettwallyworld: fair point.00:49
jcsackettwallyworld: these days i'm a wmii man, if you want to talk no love. :-D00:51
wallyworld:-)00:51
* thumper afk for walk to shops and lunch 00:53
maxbDoes anyone have any idea why ~vcs-imports is associated with https://code.edge.launchpad.net/~brian-rogers/gnome-power/report-percentage/+merge/39329 ?01:19
jelmermaxb, it's probably the default reviewer for some reason01:19
* thumper needs coffee badly02:08
=== matsubara is now known as matsubara-afk
* thumper has coffee and is screaming at his screen02:25
* thumper dodges a tumbleweed02:25
thumperwhy is that damn test being run twice?02:26
thumperlifeless: ping02:26
* wgrant rolls some more tumbleweed in.02:26
lifelesshi02:26
wgrantIt's not one of those tests which is confusingly run with the same name for several object types?02:26
thumperlifeless: when running a doc test with "bin/test -vvt message-holds-xmlrpc.txt" it runs twice02:27
thumperlifeless: any idea?02:27
thumperlifeless: btw, this will fix the mailman xmlrpc call when it makes 5000 database calls02:27
lifelessnice02:28
lifelesswhat causes that ?02:28
thumpersimplistic iteration over a large result set02:28
lifelesslack of eager loading ?02:28
thumperand silly logic02:29
* thumper is enfixorating02:29
thumperI'm changing it slightly02:29
thumperso will just do 602:29
thumperI could attempt to make it 2, but I don't see the benefit for that02:29
thumperthe logic would be harder to follow if I did02:29
thumper5000 -> 6 is much better02:30
lifelessI think 6 queries is fine ;)02:30
lifelesssometimes less is worse too02:30
lifelessdepending on the related data density02:30
rockstarderyck, lp:~rockstar/launchpad/javascript-refresh02:30
thumperlifeless: this one: OOPS 1758XMLP34102:31
lifelessnice02:32
lifelessthumper: I see the double run in trunk02:34
lifelessthumper: its being loaded twice by whatever module constructs it02:34
thumperlifeless: is it all doc tests or just some?02:34
lifelesslp/registry/tests/test_doc.py02:34
lifelesswill be a buglet in there02:35
lifelessignore for this branch IMO02:35
thumperok02:36
thumperlifeless: are you going to file a bug for it?02:36
thumperlifeless: if it is simple, I could fix in this branch02:36
lifelessits late + jetlag02:36
lifelessI haven't looked02:37
thumperlifeless: ah, it seems to have two different setup methods02:38
thumperlifeless: so it is testing things twice02:38
thumperone with the real mailinglist api and another with a fake02:39
jcsackettthumper: you mean it's doing two tests in one via diff setUps?02:51
thumperkinda02:51
* jcsackett pulls up code.02:52
jcsackettor not; i forgot my dev machine doesn't multitask well when updating.02:53
jcsackettand here i thought i might be helpful. this is a test in registry?02:53
thumperit is, and there isn't a problem02:54
thumperjust my misunderstanding02:54
jcsackettthumper: ah, dig.02:55
* wgrant WTFs at launchpad-users.03:03
lifelessoh?03:04
wgrantThe last email to it.03:04
lifelesson oct 15th?03:05
wgrantAbout 47 minutes ago.03:05
lifelessswiss branded watches?03:06
jcsackettchain letter about returning vets, looks like.03:06
wgrantIt looks like it.03:07
jcsackettthat does seem about as Off-topic as you're going to get...03:07
lifeless\o/ edge bye bye03:07
wgrant:( poor edge03:07
lifelesshah03:09
jcsacketti had only just met poor edge.03:10
bacwgrant: the previous one about getting mugged in KL was mighty off topic too03:10
* wgrant remembers back when it was 'beta', restricted access, and had the shiny new UI.03:10
wgrantbac: Ah yeah, I forgot about that one.03:10
jcsackettbac: that one is however the result of a compromised account.03:14
jcsackettmy sister had some people get into her yahoo account a few months back and received (nearly) word for word that email. it's apparently a pretty popular scam.03:14
thumperlifeless: has edge gone?03:23
wgrantHeh. I was about to ask if someone had turned off the redirect.03:23
* thumper imagines some people getting a little miffed at no recipes ...03:23
wgrantThen I noticed the last rev.03:23
EdwinGrubbsthumper: ping03:41
thumperEdwinGrubbs: hi03:41
LPCIBotYippie, build fixed!03:42
LPCIBotProject devel build (152): FIXED in 3 hr 37 min: https://hudson.wedontsleep.org/job/devel/152/03:42
LPCIBot* Launchpad Patch Queue Manager: [r=lifeless][ui=none][no-qa] Refactor page-performance-report to use03:42
LPCIBotless memory by using a SQLite3 db to hold the requests and03:42
LPCIBotgenerating statistics for only one key at a time.03:42
LPCIBot* Launchpad Patch Queue Manager: [r=henninge][ui=none][bug=659085] Remove03:42
LPCIBotgetBugNotificationRecipients()03:42
LPCIBot* Launchpad Patch Queue Manager: [r=henninge][ui=none][bug=628762] Ditch property cache adapters in03:42
LPCIBotfavour of plain old Python. The IPropertyCache adapter is03:42
LPCIBotreplaced by get_property_cache() and the IPropertyCacheManager03:42
LPCIBotadapter is replaced by clear_property_cache().03:42
wgrant!03:42
thumpernow that is a bit of IRC spam03:43
wgrantDoes this mean we are green?03:43
wgrantYes!03:43
wgrantBoth are green!03:43
wgrantFor the first time!03:43
EdwinGrubbsthumper: according to bzr annotate, you worked on canonical.launchpad.webapp.sorting.sorted_version_numbers. I'm wondering why you the numbers are sorted in the reverse order as the letters. For example, [3, 2, 1, A, B, C], which looks wrong if a version combines letters and numbers like [5a, 4a, 3a, 3b, 3c].03:43
thumperholy crap that is quite a while ago...03:44
EdwinGrubbsthumper: I'm in the process of moving the sorting into the db so that the results can be batched. It looks like this is just used for the productseries.03:44
thumperEdwinGrubbs: I think the reasoning was we wanted newer version numbers first03:44
EdwinGrubbsthumper: ok, so there shouldn't be a problem in making the letters sort descending like the numbers?03:45
thumperahh...03:45
thumperfind out where it's used forst03:45
thumperfirst03:45
thumperI really don't remember03:45
EdwinGrubbsthumper: so far, it looks like it is just used for https://qastaging.launchpad.net/launchpad/+series03:46
EdwinGrubbsbut I'll double check with sinzui03:46
thumperEdwinGrubbs: in which case it is all yours :)03:46
thumperwho whatever you like03:46
thumpers/who/do/03:46
EdwinGrubbsthanks03:46
LPCIBotProject db-devel build (99): FAILURE in 3 hr 39 min: https://hudson.wedontsleep.org/job/db-devel/99/04:18
LPCIBot* Launchpad Patch Queue Manager: [rs=buildbot-poller] automatic merge from stable. Revisions: 1179104:18
LPCIBotincluded.04:18
LPCIBot* Launchpad Patch Queue Manager: [r=henninge][ui=none][no-qa] Fix the models and accompanying tests04:18
LPCIBotfor Branch Merge Queues to use the secured utilities, etc.04:18
rockstarHow does launchpad not depend on beautiful soup anymore?04:27
thumperrockstar: eh?04:53
rockstarthumper, launchpad used to use beautiful soup a lot, but apparently it doesn't anymore.04:54
thumperah ...04:54
thumperreally?04:54
rockstarAlso, I SHOULD go to bed, but deryck and I is watching teh football.04:54
rockstarthumper, yeah, I just added soupmatchers to Launchpad and it needed BeautifulSoup.04:54
thumperrockstar: still used by canonical.launchpad.testing.pages04:55
thumperrockstar: it may be an egg04:55
rockstarthumper, no, it wasn't in versions.cfg or setup.py04:55
rockstarWeird.04:55
rockstarAt least, buildout couldn't find it, even when I just installed the package.04:56
thumperrockstar: it is LP telling you to sleep04:57
thumper:)04:57
rockstarthumper, probably.04:57
bacmatsubara-afk: ping me when you arrive05:49
bacrockstar: who is playing?05:49
=== almaisan-away is now known as al-maisan
LPCIBotProject devel build (153): FAILURE in 3 hr 38 min: https://hudson.wedontsleep.org/job/devel/153/07:21
LPCIBot* Launchpad Patch Queue Manager: [r=julian-edwards][ui=none][bug=662912][incr] Add diagnostics for bug07:21
LPCIBot66291207:21
LPCIBot* Launchpad Patch Queue Manager: [r=henninge][ui=none][no-qa] Remove the beta redirect and is_edge facilities.07:21
LPCIBotYippie, build fixed!07:53
LPCIBotProject db-devel build (100): FIXED in 3 hr 34 min: https://hudson.wedontsleep.org/job/db-devel/100/07:53
LPCIBotLaunchpad Patch Queue Manager: [rs=buildbot-poller] automatic merge from stable. Revisions: 1179207:53
LPCIBotincluded.07:53
al-maisanHello! I cannot push branches to launchpad. Can somebody please look into this?08:12
al-maisanHere's the error I get: $ bzr push08:13
al-maisanUsing saved push location: bzr+ssh://bazaar.launchpad.net/~al-maisan/landscape/edit-computer08:13
al-maisanTraceback (most recent call last):08:13
al-maisan  File "/srv/bazaar.launchpad.net/production/launchpad-rev-9885/eggs/bzr-2.2.0-py2.6-linux-x86_64.egg/EGG-INFO/scripts/bzr", line 140, in <module>08:13
al-maisan    exit_val = bzrlib.commands.main()08:13
al-maisan  File "/srv/bazaar.launchpad.net/production/launchpad-rev-9885/eggs/bzr-2.2.0-py2.6-linux-x86_64.egg/bzrlib/commands.py", line 1191, in main08:13
al-maisan    _register_builtin_commands()08:13
al-maisan  File "/srv/bazaar.launchpad.net/production/launchpad-rev-9885/eggs/bzr-2.2.0-py2.6-linux-x86_64.egg/bzrlib/commands.py", line 182, in _register_builtin_commands08:13
al-maisan    import bzrlib.builtins08:13
al-maisanValueError: bad marshal data08:13
al-maisanbzr: ERROR: Connection closed: Unexpected end of message. Please check connectivity and permissions, and report a bug if problems persist.08:13
al-maisansorry08:13
al-maisanmeant to paste pastebin url08:13
adeuringgood  morning08:34
wgrantrockstar: BeautifulSoup is included via the Java Mentality.09:07
mrevellHi09:18
LPCIBotYippie, build fixed!10:57
LPCIBotProject devel build (154): FIXED in 3 hr 35 min: https://hudson.wedontsleep.org/job/devel/154/10:57
LPCIBot* Launchpad Patch Queue Manager: [r=lifeless,thumper][ui=none][no-qa] Slightly fix the IDSJob tests.10:57
LPCIBot* Launchpad Patch Queue Manager: [r=lifeless][ui=none][bug=664012] Fix the oopses raided by +branch10:57
LPCIBotnavigation when there is no referrer to be 404s.10:57
=== jtv1 is now known as jtv
=== al-maisan is now known as almaisan-away
=== matsubara-afk is now known as matsubara
=== almaisan-away is now known as al-maisan
bigjoolsgood morning13:21
wgrantMorning.13:22
mwhudsonjam: staging codehosting seems to be working14:23
mwhudsonthis surprises me :-)14:23
jammwhudson: well, we haven't updated the config for the new code yet14:23
mwhudsonoh of course14:23
lifelesspoolie: bug 666765 is also on my nice-to-have for scopes15:17
StevenKlifeless: Didn't you mention a fix for the "No module named mailman.monkeypatches.defaults" ? I seem to recall it involved wielding rm over a directory15:29
lifelessrm -rf lib/mailman; make15:31
jmlwhy is stdout a text stream rather than a bytes stream in Python 3?15:31
lifelessbecause python three15:31
lifelessjml: hows your session15:31
jmllifeless: not exactly riveting15:32
bigjoolslifeless: so, running tests locally I get a lot of errors about librarians either running when they should not be or vice versa15:35
bigjoolsthis Is Bad.15:36
rockstarjames_w, can we chat at some point about soupmatchers?15:36
bigjoolslifeless: looks like some layer issues15:36
rockstarbigjools, yeah, I've had those problems for about three weeks.15:37
rockstarbigjools, did you bin/kill-test-services ?15:37
bigjoolsyes15:38
rockstarbigjools, where is you?15:38
bigjoolsrockstar: boner 615:38
rockstarbigjools, is there a session in there, or are you loitering?15:38
bigjoolsrockstar: linaro session15:38
rockstarbigjools, ah.15:38
bigjoolsrockstar: but I may duck out RSN15:38
rockstarbigjools, okay.  I'm at the end of the arterial hallway.15:39
lifelessbigjools: last time I saw that I had a prod librarian running15:39
bigjoolsI am getting motivated to fix test_on_merge15:40
bigjoolsrockstar: wtf is the arterial hallway? :)15:45
lifelessantigua15:45
rockstarbigjools, the hallway that acts as artery to all the dead end hallways.15:45
bigjoolsrockstar: which end?15:46
poolielifeless, replied15:48
poolielifeless, please tag them so i can find them later15:49
bigjoolslifeless: it was a stale pidfile15:50
=== matsubara is now known as matsubara-lunch
=== al-maisan is now known as almaisan-away
lifelessjml: :(16:05
james_wrockstar, sure thing16:06
rockstarjames_w, are you going to be in a session after the one that just started?16:06
james_wrockstar, yes. I think my afternoon is quieter though16:07
james_wrockstar, or lunch?16:07
rockstarjames_w, lunch will work.16:08
StevenKIs it just me or is codehosting down?16:17
=== Ursinha is now known as Ursinha-lunch
fjlacostelifeless: around?16:40
lifelessys16:41
lifelessfjlacoste: yes16:44
fjlacostelifeless: i need your advice, i'm working on a version of the ppr report that computes everything using SQL, that will operate in constant memory16:44
fjlacostelifeless: i have all the stats covered except the median16:44
fjlacostei know how to compute it, but it is very expensive16:44
lifelessfjlacoste: btw, disk space on sodium - we had to kill your thing16:44
fjlacostei know16:45
fjlacostethat's fine16:45
fjlacostewe can sort that out later16:45
lifelessit was using 5% of the disk :)16:45
fjlacoste2.9G16:45
lifelessyeah16:45
lifelessthats ~ 5%16:45
fjlacosteit needs a bigger disk :-)16:45
fjlacosteanyway16:45
fjlacosteso the way i can compute the median is by sorting on the column and using LIMIT 1 OFFSET n/2 to get the value16:45
fjlacostebut this seems like very expensive to do given that we are taking about millions of records16:46
fjlacosteand we are computing 3 medians16:46
fjlacosteand we can only do that by key16:46
lifelessWHERE Rank = (SELECT (COUNT(*)+1) DIV 2 FROM16:46
fjlacosteah16:46
lifelessalso16:47
lifelesshttp://wiki.postgresql.org/wiki/Aggregate_Median16:47
lifelessthough I haven't looked into that implementation16:47
fjlacostelifeless: i'm using sqlite16:47
lifelessoh16:47
lifeless:P16:47
fjlacostei was asking if you mind dropping it :-)16:47
fjlacostebut your rank thing gave me an idea16:48
lifelessfjlacoste: hmm, so if the column is indexed grabbing the median should be darn cheap16:48
fjlacostethe question is how do I index the column relatively cheaply16:49
=== fjlacoste is now known as flacoste
lifelessinsert the data16:49
lifelessindex the column16:49
lifelessthen start querying16:49
lifelessoh16:50
lifelessyou'll want to increase the sqlite cache size too16:50
lifelessbecause its like 2MB or something insane by default16:50
lifelessgive it 200MB or so16:50
flacostecreate an index on category, time for example?16:50
flacosteok16:50
lifelessyeah16:50
lifelessremember that the query prefix has to match the index16:50
flacosteright16:51
lifelesscategory,time won't help time only sorts16:51
lifelessIIRC16:51
flacostewell i need the rank of time within each category16:51
flacoste(or url, or pageid)16:51
lifelessyack16:51
jcsackettlifeless: any chance you could help me think through bac's notes on bug 652156 and bug 652149?16:52
jcsackettbasically, it looks to me like the timeouts there aren't related to either bug/branch.16:52
lifelessqastaging has a 10second timeout16:52
abentleylifeless: do you mean yak (-shaving), yuck, or some combination?16:52
lifelessI meant 'acl'16:53
lifelessbah16:53
lifeless'ack'16:53
lifelessjcsackett: so any page that has a timeout bug open (https://bugs.edge.launchpad.net/launchpad-project/+bugs?field.tag=timeout) is almost certain to timeout on [qa]staging too16:53
lifelessjcsackett: I would not stress about incidental timeouts there (but I *would* fix the page to not timeout at all :P separately :) )16:54
mwhudsonmtaylor, jam: launchpad/bzr planning over lunch?16:54
jcsackettlifeless: that makes sense. so bac and i can mark ours qa-ok if the functionality is working (aside from the branchbug timeout)?16:54
lifelessjcsackett: as long as the page is already timing out with bug, production oops)16:55
flacostelifeless: actually, there is no Rank expression in sqlite16:56
flacosteand generating the rank seems as costly as getting the median using iterative limit/Offset16:56
lifelessflacoste: how slow is slow16:58
flacosteseveral hours16:58
flacosteactually, i don't know16:59
flacostejust guessing here16:59
flacostei'll measure and see16:59
flacosteassess16:59
jcsackettlifeless: so, there isn't a bug filed against project/+branches, but person/+branches has one (bug 627945) and the query is the same in both. i'm thinking either add detail to that bug or file a new bug for project linked to it. thoughts?16:59
mwhudsonmtaylor, jam: which isn't right now, of course17:01
* mwhudson was just getting hungry and optimistic17:01
flacostelifeless: http://oreilly.com/catalog/transqlcook/chapter/ch08.html#Calculating%20a%20Median has an actual elegant solution :-)17:02
jammwhudson: sounds good17:02
jamI'm pretty sure poolie is interested too17:02
lifelessjcsackett: add the page id for the other case to the title, done.17:04
jcsackettlifeless: fantastic. thanks.17:04
=== benji is now known as benji-lunch
flacostehmm, that query does look expensive17:11
=== matsubara-lunch is now known as matsubara
lifelessflacoste: up the cache size as a start17:12
flacostelifeless: ack17:12
lifelessyou could denormalise the counts per category17:12
lifelessgiven you're starting with a fresh db each run, right ?17:12
mtaylormwhudson: yes17:18
mwhudsonjam, mtaylor: i'll loiter by the registration desk17:20
mwhudsonpoolie: ^^17:20
lifelessjml: https://code.edge.launchpad.net/~lifeless/testtools/fixtures/+merge/3938817:55
=== benji-lunch is now known as benji
LPCIBotProject db-devel build (101): SUCCESS in 4 hr 4 min: https://hudson.wedontsleep.org/job/db-devel/101/18:14
=== Ursinha-lunch is now known as Ursinha
jmllifeless: reviewed19:09
lifelessgary_poster: http://pypi.python.org/pypi/storm - needs 0.18 ?19:13
marsthe PyPi version is almost a year old19:15
gary_posterlifeless, yes, remaining release of storm (per the release instructions) is stalled until Jamu gives me privs to upload docs or does it himself.  There are a number of small things remaining (also announcing it on ML, for instance).19:15
gary_posterFor PyPI, Storm didn't release 0.17 there and it's not on their release checklist, but I got privs to do it anyway, so I'll do it then.19:15
lifelessgary_poster: Jamu here, go ahead and release, I'll deploy docs later.19:16
gary_posterCool Jamu, thanks19:16
* mars loves UDS :)19:16
mwhudsoni am inpressed by the fact that people are just registering kernel imports, and they basically work19:22
mwhudsonit would be much better if we supported stacking for import branches sensibly but well19:23
jelmermwhudson: +119:24
jelmerit shouldn't really be all that hard if I understand correctly19:24
* mwhudson quickly reaches for his "not a launchpad developer" hat19:24
* jelmer reaches for his "not a launchpad-code developer" hat :-)19:24
mwhudsonjelmer: i think the bug report on this describes what to do fairly clearly19:25
mwhudsoni hope so,  becuase i'm fairly sure i knew what to do at one point and certainly don't any more19:26
jmllifeless: do you still thing that https://bugs.edge.launchpad.net/testtools/+bug/584824 is still worth doing?19:33
jmlthink19:33
jmllifeless: also, https://bugs.edge.launchpad.net/testtools/+bug/666923 wrt previous discussions19:36
jkakargary_poster: Hi!19:58
gary_posterjkakar: hi :-)19:58
lifelessderyck: hey20:05
deryckhi lifeless20:05
lifelesshttps://devpad.canonical.com/~lpqateam/qa_reports/deployment-stable.html20:05
derycklooking....20:06
deryckslow openid dance20:06
lifelessderyck: so https://bugs.qastaging.launchpad.net/ubuntu/+source/xorg/+bug/10000 looks ok20:09
lifelessderyck: I'm not entirely sure how to validate it further20:09
derycklifeless, yeah, I'm looking at the changes now to see.20:09
derycklifeless, I smell a bad rev.  https://bugs.qastaging.launchpad.net/bugs/10000/+bug-portlet-subscribers-content20:13
deryckhmmm, maybe that's not gmb's20:14
lifelessderyck: whats wrong with it ?20:18
derycklifeless, that link OOPS for me.20:18
deryckhttps://lp-oops.canonical.com/oops.py/?oopsid=OOPS-1760QS4620:19
deryckoh, that's security.cfg permissions again.  Like bug expiry script.  We missed something in pg 8.4 migration, I guess.20:21
derycklifeless, ok, so gmb's is qa-ok.  Visiting https://bugs.qastaging.launchpad.net/ubuntu/+source/xorg/+bug/10000/+subscribe confirms the new code is hit and works fine.20:22
lifelessderyck: will you mark it as such? the two bugs ?20:23
derycklifeless, indeed.20:23
lifelessthanks20:23
derycklifeless, do you not get an OOPS at the first URL I provided?  the portlet one.20:25
lifelessyes, I do, i thought it might be transient but its very quick20:25
derycklifeless, transient?  You mean timeout?  I get ProgrammingError.20:26
lifelessderyck: I meant I thought it might be a slow db or something20:26
derycklifeless, ah, ok20:27
derycklifeless, we should talk more about this after we get out of sessions, since I saw the same on expiry script test runs.20:27
bigjoolsTHE EAGLE HAS LANDED20:27
bigjoolswhere eagle == new buildd manager20:28
deryckbigjools, awesome!20:28
bigjools7k branch with no mechanical changes!20:28
thumperbigjools: what does it give us?20:29
bigjoolsasynchronous comms20:30
bigjoolsand far, far better failure detection and dealing with failures20:30
bigjoolslp:~julian-edwards/launchpad/builderslave-resume if you want gory details :)20:30
deryckhi thumper.  Being at UDS makes us overlap for once. :-)20:31
rockstarjames_w, are you going to have some bandwidth this afternoon?20:31
thumperderyck: hey20:31
james_wrockstar, the next session looks pretty good to me20:32
rockstarjames_w, okay, I think I need to make some hard choices here.20:34
james_wrockstar, 6pm would also work for me20:34
rockstarjames_w, yeah, that won't work at all.  Next session is good, I'll just chat with aq before the next session.20:34
james_wrockstar, well, that was the session I was thinking going to as well :-)20:37
james_wrockstar, are you here all week?20:37
rockstarjames_w, indeed I am, but I want to land some work before next week.  :)20:37
james_wrockstar, sure :-)20:37
james_wrockstar, where are you now?20:38
rockstarjames_w, I'm in the game development session, but it's a bust, so I can get out now if you can.20:38
james_wrockstar, I can20:38
james_wrockstar, outside the plenary room?20:38
rockstarjames_w, I'm in the hall.20:40
=== fjlacoste is now known as flacoste
=== Ursinha is now known as Ursinha-brb
jmlbigjools: grats!21:12
rockstar\o/ BrowserTestCase just got a whole lot better.21:13
bigjoolsjml: and thanks to you!21:13
bigjoolsjml: FWIW I hammered it on DF with a load of production builders that were borrowed and it didn't put a foot wrong.21:19
jmlbigjools: sweet.21:19
lifelessrockstar: what got better about it ?21:23
bigjoolsjml: of course, I just set myself up for a big fall :)21:23
jmlbigjools: sure. :)21:23
jmlbigjools: it's not going to work the first time. never does.21:23
rockstarlifeless, I'm teaching it about james_w's soupmatchers.21:23
bigjoolskiko's 1st law of software?21:23
jmlbigjools: exactly21:23
jmllifeless: https://code.launchpad.net/~jml/launchpad/atexit-warning/+merge/3940321:24
lifelessjml: I saw, i don't understand it. It seems wrong21:24
jmllifeless: that's why I wanted to talk about it :)21:24
lifelesswhy not just remove the atexit call21:24
lifelessand if things go screwy fix the code that isn't using a finally where needed.21:25
jmllifeless: I don't know why the atexit call was there. it seems like a belt-and-braces thing21:25
lifelessI think its because zope.testing stops teardown when a NotImplemented is enoucountered except in the last runner instance21:26
jmllifeless: I guess I should do some archeology and find out why it was added.21:26
=== matsubara is now known as matsubara-afk
jmllifeless: ok. I'm not sure what to do with that though.21:27
lifelessjml: change zope.testing to not mess about21:27
lifelessif a layer is setUp, it should tear it down21:28
lifelessperhaps we could use Fixture to do that ;)21:28
jmllifeless: I don't want to fix zope.testing to silence this warning21:28
lifelessjml: the warning is a symptom of the root cause21:28
lifelessthe root cause is that teardown is inconsistently called21:28
lifelessand because of that we have:21:29
jmllifeless: the root cause being that layers is a pos21:29
lifeless - ordering problems21:29
lifelessjml: well, what do you want to do?21:29
lifelessjml: I don't want the fixture code to silently accept teardowns when torn down21:29
lifelessthat seems prone to hiding things21:30
jmllifeless: it's not doing that, there's a special tear down for atexit, since there's no way of cancelling atexit calls21:30
lifelessjml: you can use atexit._exithandlers.remove()21:31
jmllifeless: you think that would be better than the code in the patch?21:31
lifelessyes21:31
lifelessbecause the ugliness would be outside the island of sanity I've created21:31
lifelessbetter would be fixing the root cause: zope.testing21:32
jmlI'm not going to fix zope.testing to address this issue21:32
=== henninge is now known as henninge_
lifelesswell, I may :)21:32
jmllifeless: you're welcome to it21:33
lifelessjml: I'm not terribly concerned about pushing it upstream, because I think layers is so poor, I want a good platform for migrating from though.21:33
lifelessjml: I want us to stop papering over things like this.21:33
jmllifeless: sure, me too. but to me this is an incremental improvement. we're using atexit today to paper over an issue, and it's generating a warning that has little to do with the underlying issue....21:36
jmllifeless: I'm patching it to not generate useless the warning.21:36
lifelessStevenK: https://devpad.canonical.com/~lpqateam/qa_reports/deployment-stable.html tag, you're it.21:36
lifelessjml: but, you're:21:38
lifeless - duplicating code21:38
lifeless - adding a silent-wrong code path21:38
lifeless - leaving the underlying issue open21:38
lifelessjml: I think its a much better use of your time to just fix the basic cause21:39
jmllifeless: no, it's not. because that's a significantly larger amount of work, and I've got other things I want to do.21:39
lifelessjml: I'd be happy with a change that didn't duplicate code and make the server.py module less clear21:39
jmllifeless: ok. I'm moving to atexit._exithandlers now21:40
jcsacketti may have missed an email on this topic, but does launchpadlib work on qastaging?21:44
lifelessit should, but it may need a glue update21:44
jmllifeless: patch updated.21:45
jcsacketthm. question the second: is lplib supposed to work with regular staging?21:48
LPCIBotProject db-devel build (102): SUCCESS in 3 hr 36 min: https://hudson.wedontsleep.org/job/db-devel/102/21:51
LPCIBotLaunchpad Patch Queue Manager: [rs=lamont][ui=none][no-qa] Update the changelog for launchpad-buildd.21:51
jmllifeless: yes.21:53
jmljcsackett: yes.21:53
jml(sorry lifeless)21:53
jcsackettjml: thanks. i must just have something wrong in my setup; staging is telling me that it can't connect b/c it can't forward a request.21:55
thumpermorning fellow hackers21:55
jcsacketti'll continue futzing with it.21:55
jmljcsackett: hmm.21:56
jmlthumper: hi21:56
thumperjml: do you have a few minutes?21:56
jmlthumper: I will very soon.21:56
jmlthumper: in a session right now21:56
thumperjml: well, I have the standup shortly, and a talk with flacoste after that21:57
jmlthumper: when are you next free?21:57
thumperjml: probably in 1.5 hours21:57
jcsackettjml: if that "hmm" indicates interest, the response is pasted here https://pastebin.canonical.com/39102/21:58
flacostethumper: i'm free whenever you are21:58
flacostei know that our call is only in 30 mins21:58
jmljcsackett: that looks like a bug that a losa ought to know about21:59
=== henninge_ is now known as henninge
jcsackettjml: yeah? okay. i was assuming my own screw up, but i'll go tell one.21:59
jmljcsackett: I don't know21:59
jcsackettjml: dig. i'm getting a losa now.22:00
jmlthumper: I can be around in 1.5hrs time22:01
thumperjml: ok22:02
wallyworld_thumper: abentley: standup?22:11
lifelessStevenK: ping22:20
StevenKlifeless: O hai22:21
lifelessI can has QA? https://devpad.canonical.com/~lpqateam/qa_reports/deployment-stable.html22:21
lifeless(Is it deployable)22:21
StevenKlifeless: Yes, but let me finish what I'm in the middle of22:22
StevenKI was unblocked to do so on Friday, and then there was a weekend and a plane trip22:23
LPCIBotProject devel build (155): SUCCESS in 3 hr 56 min: https://hudson.wedontsleep.org/job/devel/155/22:38
lifelessStevenK: bigjools I've requested a review of what was qa'd. Please unblock the queue as soon as possible.23:18
lifelessor tell me enough to do it myself23:18
bigjoolslifeless: pardon?23:18
lifelesss/review/deploy/23:18
bigjoolslifeless: need moar context23:18
lifelessbigjools: https://devpad.canonical.com/~lpqateam/qa_reports/deployment-stable.html23:18
lifelessbigjools: 9 day old unqa'd patch23:19
lifelessbigjools: (which is to be expected, we're only just in the actually-use-it stage of RFWTAD23:19
bigjoolslifeless: Steve was in the middle of QAing that as far as I know23:20
lifelesscool23:21
thumperrockstar: ping23:24
jmlthumper: ping23:28
thumperjml: just finishing off with flacoste23:28
jmlthumper: ok.23:28
=== Ursinha-brb is now known as Ursinha
wgrant"The size of the diff (7194 lines) is larger than your specified limit of 1000 lines" D:23:32
jmlwgrant: it's a big patch.23:32
StevenKlifeless: Are you saying "Please QA this right now", in leiu of say, dinner?23:35
StevenKlifeless: And it would take me longer to explain how to QA it than to just do it myself. :-)23:36
flacostelifeless: what's the status of removing edge?23:38
flacostei actually need to go afk, let's talk about it tomorrow23:41

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