Thursday, 2017-02-02

SpamapSjeblair: is it possible 'timer' isn't implemented in v3's layout validator yet?00:03
SpamapSor rather, configloader?00:13
SpamapSah, looks like we never load the timer driver00:22
SpamapSShrews: I get infinite streams of this now from my test runs:00:22
jeblairSpamapS: cncf: cool! thanks, nice msg :)00:22
SpamapS2017-02-01 16:22:25,218 kazoo.client                     DEBUG    Sending request(xid=228): GetChildren(path='/nodepool_test/BALPatqw_2913/nodepool/requests', watcher=None)00:22
SpamapS2017-02-01 16:22:25,219 kazoo.client                     DEBUG    Received error(xid=228) NoNodeError((), {})00:22
jeblairSpamapS: yeah, that's the test running until timeout00:23
SpamapSah00:23
SpamapSnoisy buger00:23
jeblairSpamapS: (i have current work to mask that logging)00:23
jeblairSpamapS: i think the configparser hands off parsing drivers such as timer to the driver itself... i *think* that's implemented00:24
SpamapSIt does00:25
SpamapSand timer hasn't ever been registered00:25
SpamapSregistering it 'splodes in .reconfigure(tenant)00:25
SpamapSworking on that now00:25
SpamapSalright, now it's just failing the actual test. :)00:42
jeblairmordred, Spamaps: with 427946, test_dependent_behind_dequeue always fails with a stringexception/processreturncode error (that usually means some kind of timeout).  but that test is skipped.  when i run it locally, it skips as expected.  i have no idea how to proceed.00:43
*** openstackgerrit has quit IRC01:02
mordredjeblair: uhm01:14
SpamapSjeblair: looking01:42
SpamapSjeblair: you meant some other number. :)01:44
*** Shuo has quit IRC02:11
mordredSpamapS: I think he means https://review.openstack.org/42794602:13
mordredsuch as http://logs.openstack.org/46/427946/2/check/gate-zuul-python27-ubuntu-xenial/208c06b/testr_results.html.gz02:13
mordredwhich does not give me the pleasings02:14
SpamapShm I must have typo'd it twice :-P02:15
SpamapSt3h weirds02:16
*** openstackgerrit has joined #zuul02:17
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Re-enable test_abandoned_not_timer  https://review.openstack.org/42798502:17
SpamapSjeblair: ^^ I'm a bit stuck on this one... it doesn't seem to see the jobs in layout-idle.02:17
openstackgerritJamie Lennox proposed openstack-infra/zuul feature/zuulv3: Re-enable multiple gerrit connection test  https://review.openstack.org/40669904:06
*** saneax-_-|AFK is now known as saneax04:16
*** saneax is now known as saneax-_-|AFK04:40
*** bhavik1 has joined #zuul05:20
*** Zara has quit IRC06:12
*** Zara has joined #zuul06:13
*** yolanda has quit IRC06:13
*** yolanda has joined #zuul06:25
*** abregman has joined #zuul06:41
*** yolanda has quit IRC06:46
*** saneax-_-|AFK is now known as saneax07:05
*** yolanda has joined #zuul08:12
*** yolanda has quit IRC08:52
*** yolanda has joined #zuul09:03
*** hashar has joined #zuul09:06
*** yolanda has quit IRC09:10
*** yolanda has joined #zuul09:41
*** bhavik1 has quit IRC10:59
*** wznoinsk has left #zuul11:07
openstackgerritAlexander Evseev proposed openstack-infra/zuul master: Use ssh-agent instead of ugly ssh-wrapper scripts  https://review.openstack.org/33114811:08
*** openstackgerrit has quit IRC12:03
*** yolanda has quit IRC14:15
*** yolanda has joined #zuul14:15
*** abregman has quit IRC15:20
Shrewsjeblair: wow, that failure is... wow15:25
*** saneax is now known as saneax-_-|AFK15:26
*** Cibo_ has joined #zuul15:45
jeblairSpamapS: that change lgtm -- i don't immediately see what's wrong.  i'll take a closer look after i wrestle with testtools.15:46
*** hashar is now known as hasharAway16:24
*** abregman has joined #zuul16:38
*** abregman is now known as abregman|afk17:13
*** hasharAway has quit IRC17:29
jeblairaaah.  dependent_behind_dequeue was recently enabled, so my local branch doesn't have it (it's actually not skipped).  i'm guessing its runtime is close enough to the limit that the change in how the tests are run pushed it over.17:33
*** openstackgerrit has joined #zuul17:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Tests: store debug logs on error  https://review.openstack.org/42794617:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add SourceContext class  https://review.openstack.org/42784617:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Use playbooks defined in repos  https://review.openstack.org/42355017:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Put Zuul vars in an ansible vars file  https://review.openstack.org/42436717:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add playbooks for all test jobs  https://review.openstack.org/42581017:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Implement job aborting  https://review.openstack.org/42630617:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Move job running into own class  https://review.openstack.org/38596417:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add infrastructure for pre/post playbooks  https://review.openstack.org/42598917:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Run pre and post playbooks  https://review.openstack.org/42749417:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix test_live_configuration  https://review.openstack.org/42783817:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Handle multiple playbook extensions  https://review.openstack.org/42545017:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix nodeset return on skipped jobs  https://review.openstack.org/42586517:39
jeblairi rebased, increased the timeout, and switched to gentle timeouts.17:39
mordredjeblair: yay!17:40
*** abregman|afk has quit IRC17:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Re-enable test_abandoned_not_timer  https://review.openstack.org/42798518:06
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove layout.projects  https://review.openstack.org/42830618:06
jeblairSpamapS: ^ i found the sticking point and fixed it in 428306.  427985 is now failing for new unrelated reasons.  do you want to continue with that, or should i pick it up?18:07
jeblairi've just done a full testr run.  something like 10 tests failed.  they don't fail in isolation, so i can't just run one of them to see the logs.  does anyone know how to extract the logs for just one test out of the subunit file?18:34
jeblairtestr failing --subunit | subunit-filter -m tests.unit.test_scheduler.TestScheduler.test_file_head | subunit-2to118:36
jeblairthat approximates what i want.... surely there's a better way?  (especially the subunit-2to1 bit?)18:37
Shrewsthat stuff is all magic to me18:39
jeblairyeah, ttrun is my friend18:39
Shrewsi was never certain of what advantage ttrun provides over tox. isn't 'ttrun -epy27 x.y.z' the same as 'tox -epy27 x.y.z' ?18:41
SpamapSIIRC there's a subunit cli18:42
SpamapSah yeah you found it ;)18:42
SpamapSthat's how I'd have done it18:42
SpamapSthat's more or less how subunit2sql works too :-P18:43
*** hashar has joined #zuul18:46
SpamapSShrews: ttrun runs without subunit wrapping it, so you can run a debugger18:48
SpamapSsince tox is just feeding your args into testr18:48
jeblairand it streams to the console in real time, so you can diagnose hanging tests18:49
ShrewsSpamapS: ah, that would be handy, indeed18:49
SpamapSRight that too18:49
* Shrews pip installs18:49
SpamapSI wonder if one could just have a --no-subunit arg added to testr18:50
mordredShrews: ttrun skips the discover phase - so shaves several seconds off of a run18:55
SpamapSjeblair: I looked at the interdiff from my patch to yours. I'm not sure I understand why those changes were needed.18:55
mordredSpamapS: the thing is - all testr really is is a program that knows how to invoke subunit producing things and store their results into a database18:55
SpamapSmordred: in parallel18:56
mordredyah18:56
SpamapSwhich is the fun part :)18:56
jeblairSpamapS: did you look at my standalone change and the commit message? https://review.openstack.org/42830618:56
jeblairSpamapS: layout.projects is empty now18:57
mordredjeblair: and yes - I do not think there is a 'better' way to do what you're doing with the above pipeline18:57
SpamapSjeblair: OH no I did not. :)18:58
SpamapSthat 'splains it18:58
SpamapSjeblair: the failure still looks the same in the end... no builds.18:59
jeblairSpamapS: sorry, my ping probably made more sense in the context of the two-line irc patch upload notice19:00
SpamapShttp://logs.openstack.org/85/427985/2/check/gate-zuul-python27-ubuntu-xenial/1a3ad39/console.html#_2017-02-02_18_11_22_54462019:00
SpamapSactually I think it's a simple logic error19:00
*** bhavik1 has joined #zuul19:09
*** bhavik1 has quit IRC19:22
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Tests: store debug logs on error  https://review.openstack.org/42794619:34
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add SourceContext class  https://review.openstack.org/42784619:34
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Use playbooks defined in repos  https://review.openstack.org/42355019:34
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Put Zuul vars in an ansible vars file  https://review.openstack.org/42436719:34
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add playbooks for all test jobs  https://review.openstack.org/42581019:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Implement job aborting  https://review.openstack.org/42630619:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Move job running into own class  https://review.openstack.org/38596419:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add infrastructure for pre/post playbooks  https://review.openstack.org/42598919:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Run pre and post playbooks  https://review.openstack.org/42749419:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix test_live_configuration  https://review.openstack.org/42783819:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Handle multiple playbook extensions  https://review.openstack.org/42545019:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix nodeset return on skipped jobs  https://review.openstack.org/42586519:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Increase individual test timeout  https://review.openstack.org/42838919:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix race in fakenodepool  https://review.openstack.org/42839019:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Set TMPDIR to ZUUL_TEST_ROOT  https://review.openstack.org/42839119:35
jeblairokay, i'm not ready to switch to non-gentle timeouts -- i still see problems with that in some cases, so i removed that.19:35
jeblairSpamapS: i added 428391 to help my ssd.  i also moved my zookeeper to a tmpfs.  with those changes (i don't know if both were necessary, but i think at least one was) i can actually run "tox -epy27" on my workstation.19:36
jeblairi also added 428390 which seems to be the cause of some spurious test failures after the test logging change (timings changed everywhere, so it brings out the race conditions)19:36
jeblairi put the test race related changes at the top of the stack19:37
*** abregman|afk has joined #zuul19:48
*** abregman|afk has quit IRC19:55
*** abregman has joined #zuul19:56
SpamapSjeblair: weee20:33
*** harlowja has quit IRC21:00
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: Fix storeNode() for updates  https://review.openstack.org/42842521:06
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: WIP: Assign node set to node requests  https://review.openstack.org/42842821:14
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: WIP: Assign node set to node requests  https://review.openstack.org/42842821:15
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: WIP: Assign node set to node requests  https://review.openstack.org/42842821:44
*** harlowja has joined #zuul22:25
*** harlowja has quit IRC22:32
*** harlowja has joined #zuul22:33
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Tests: store debug logs on error  https://review.openstack.org/42794622:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add SourceContext class  https://review.openstack.org/42784622:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Use playbooks defined in repos  https://review.openstack.org/42355022:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Put Zuul vars in an ansible vars file  https://review.openstack.org/42436722:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add playbooks for all test jobs  https://review.openstack.org/42581022:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Implement job aborting  https://review.openstack.org/42630622:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Move job running into own class  https://review.openstack.org/38596422:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add infrastructure for pre/post playbooks  https://review.openstack.org/42598922:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Run pre and post playbooks  https://review.openstack.org/42749422:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Handle multiple playbook extensions  https://review.openstack.org/42545022:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix nodeset return on skipped jobs  https://review.openstack.org/42586522:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Avoid multiple build cancelations  https://review.openstack.org/42847722:46
*** hashar has quit IRC22:53
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Put Zuul vars in an ansible vars file  https://review.openstack.org/42436722:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Move job running into own class  https://review.openstack.org/38596422:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add infrastructure for pre/post playbooks  https://review.openstack.org/42598922:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix race in fakenodepool  https://review.openstack.org/42839022:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix test_live_configuration  https://review.openstack.org/42783822:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Set TMPDIR to ZUUL_TEST_ROOT  https://review.openstack.org/42839122:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Handle multiple playbook extensions  https://review.openstack.org/42545022:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Increase individual test timeout  https://review.openstack.org/42838922:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix nodeset return on skipped jobs  https://review.openstack.org/42586522:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Tests: store debug logs on error  https://review.openstack.org/42794622:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add SourceContext class  https://review.openstack.org/42784622:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Use playbooks defined in repos  https://review.openstack.org/42355022:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add playbooks for all test jobs  https://review.openstack.org/42581022:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Implement job aborting  https://review.openstack.org/42630622:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Avoid multiple build cancelations  https://review.openstack.org/42847722:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Run pre and post playbooks  https://review.openstack.org/42749422:57
jeblairi'm basically just making it look like i'm really busy22:58
jeblair(sorry about all that, but i'm on a tear fixing up test stability)22:58
jeblairpabelanger, mordred: thanks for the +3s; if you can add some more to that stack, that would be nice.  :)23:07
pabelangerjeblair: Ya, doing so now23:08
pabelangerjeblair: left trivial comment on 425989. And question on 42749423:22
jeblairpabelanger: re 989 agreed, i'll fix.  responded on 49423:26
pabelangerjeblair: ack23:26
*** abregman has quit IRC23:40
SpamapSjeblair: crikey, you're so busy!23:50
SpamapSjeblair: If you'd like for me to pick 427985 back up I will happily do so. I've been distracted mostly by yet another winter/travel cold sapping my ability to be awake :-P23:54
SpamapSjeblair: in playing with it earlier, it kind of feels like this test is covering a lot of code that was previously uncovered.23:55
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Tests: store debug logs on error  https://review.openstack.org/42794623:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add SourceContext class  https://review.openstack.org/42784623:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Use playbooks defined in repos  https://review.openstack.org/42355023:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Put Zuul vars in an ansible vars file  https://review.openstack.org/42436723:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add playbooks for all test jobs  https://review.openstack.org/42581023:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Implement job aborting  https://review.openstack.org/42630623:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Move job running into own class  https://review.openstack.org/38596423:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add infrastructure for pre/post playbooks  https://review.openstack.org/42598923:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Avoid multiple build cancelations  https://review.openstack.org/42847723:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Run pre and post playbooks  https://review.openstack.org/42749423:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Set TMPDIR to ZUUL_TEST_ROOT  https://review.openstack.org/42839123:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Handle multiple playbook extensions  https://review.openstack.org/42545023:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix nodeset return on skipped jobs  https://review.openstack.org/42586523:59
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Report exceptions in launcher  https://review.openstack.org/42849923:59

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