SpecialK|Canonhttps://code.launchpad.net/~doismellburning/launchpad/+git/launchpad/+merge/385644 please10:26
SpecialK|CanonThanks :)10:33
SpecialK|CanonAnd uh https://code.launchpad.net/~doismellburning/launchpad/+git/launchpad/+merge/385741 to fix the test I broke please...!12:04
SpecialK|Canonhttp://lpbuildbot.canonical.com/builders/lp-devel-xenial/builds/1334/steps/shell_9/logs/summary also shows TestRunMissingJobs.test_run_missing_ready_does_not_return_results failure but I can't reproduce it locally, can't see how my change would have impacted it, and have a suspicion that it's a transient timing issue?12:05
tomwardillthat's a celery / rabbit failure manifestation12:05
SpecialK|Canontomwardill: an actual proper persistent failure, it seems? http://lpbuildbot.canonical.com/builders/lp-devel-xenial/builds/1335/steps/shell_9/logs/summary13:16
ilascSpecialK|Canon: nope, that's transient13:16
tomwardillyou just have the fun of it happening multiple times :)13:16
ilascthat's one of "the usual"13:17
ilascright, as tomwardill said :)13:17
SpecialK|CanonAh ok I'm 2/2 so far - I recognise that's not a lot of data points but also figured safest to assume that I'd just broken stuff ;)13:17
SpecialK|CanonCan I kick off a restart somehow or do I just wait for someone else to merge something?13:17
SpecialK|Canon...it's even right there on the front page isn't it - thanks13:18
tomwardillit's non-ovious terminology13:18
tomwardillalso non-obvious place, you'd expect a 'retry' on the build page or something13:19
tomwardill(fwiw, our cancel also doesn't work)13:19
pappacenaDid it fail again? I've retried right before the standup...13:19
SpecialK|Canonpappacena: ah then I might have double-retried, sorry13:19
* pappacena not a problem :)13:19
tomwardilliirc, if it's in a failed state, a merge won't cause a new build13:19
tomwardillyou have to force it once the merge shows up on the left column13:20
SpecialK|Canonit definitely tried a new build of my merged test fixen13:20
tomwardillnot at that time difference it didn't, that was pappacena retrying it for you13:21
* SpecialK|Canon retracts the "definitely"13:21
SpecialK|CanonI assumed that there would be visible indication in the waterfall of someone doing that13:22
tomwardillforce is a bit of a weird hack in this version of buildbot, it's not natively implemented13:22
tomwardillbut there was a standard 'recipe' for people to extend the web ui to create it13:22
SpecialK|Canonno pressure re your buildbot work, but uh.... ;)13:22
tomwardillSpecialK|Canon: you can see it on http://lpbuildbot.canonical.com/builders/lp-devel-xenial/builds/1335 fwiw13:23
tomwardillunder 'Reason'13:23
tomwardillbut the waterfall doesn't show it13:24
pappacenailasc, tomwardill these are the MPs for projects as pillar for OCI projects:13:24
pappacena- https://code.launchpad.net/~pappacena/launchpad/+git/launchpad/+merge/384506 (create & edit page)13:24
pappacena- https://code.launchpad.net/~pappacena/launchpad/+git/launchpad/+merge/384514 (search & list)13:24
SpecialK|Canonhttp://lpbuildbot.canonical.com/builders/lp-devel-xenial/builds/1336/steps/shell_9/logs/summary - TestJobRunner.test_runAll_mails_oopses also a known flaky?14:07
pappacenatomwardill: on https://code.launchpad.net/~pappacena/launchpad/+git/launchpad/+merge/384514, you've mentioned a typo, but I cannot see your comment on the code :-(15:35
tomwardillpappacena: sigh, bet I forgotr to save it15:36
tomwardillI did15:36
tomwardillpappacena: L590 in the diff, s/contains/contain15:36
pappacenaAh! Thanks!15:36
tomwardillpappacena: fairly sure it was my typo to start with :)15:38
pappacenahahaha. Maybe... I will fix it there anyway :)15:38
tomwardillah poop, actual test failure15:54
* tomwardill gets to fixing15:54
* pappacena :-( 15:55
tomwardilltricky as we don't actually have the git repository that has been created to check against it's IP16:01
tomwardillmgiht just have to swap it for an assert 'id' in repo.keys()16:02
pappacenaMaybe using a matcher for the data type could do?16:04
pappacenaLike `'id': IsInstance(int)`16:06
tomwardillyeah, that's probably better16:06
* tomwardill does that16:06
tomwardillfix my whoopsie: https://code.launchpad.net/~twom/launchpad/+git/launchpad/+merge/38575916:09
tomwardillcurious as to how it got to be 'not 1' though, that does imply a test isolation failure somewhere16:10
* pappacena +116:10
pappacenaI guess the tests do not delete the database (or it depends on the layer, or whatever). And increasing a sequence has effect regardless on rolling back a transaction...16:11
tomwardillta, landing16:11
tomwardilland then poking buildbot16:12

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