Friday, 2017-10-13

*** xinliang has quit IRC00:18
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Use sources.list.d for Debian with APT  https://review.openstack.org/51165800:27
openstackgerritMerged openstack-infra/zuul-jobs master: Use sources.list.d for Debian with APT  https://review.openstack.org/51165800:51
*** mgagne has quit IRC02:19
*** mgagne has joined #zuul02:22
*** mgagne is now known as Guest5368002:22
*** zhuli has quit IRC08:37
*** zhuli has joined #zuul08:37
*** electrofelix has joined #zuul08:56
*** sambetts|afk is now known as sambetts10:05
*** hashar has joined #zuul11:19
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Collapse job header lines into a single statement  https://review.openstack.org/51182111:57
*** hashar has quit IRC12:01
*** hashar has joined #zuul12:11
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Fix doc typo that missed important words  https://review.openstack.org/50990512:21
*** hashar has quit IRC12:25
*** hashar has joined #zuul12:32
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add role to ensure per-node output dirs exist  https://review.openstack.org/51182312:35
*** hashar has quit IRC12:40
*** hashar has joined #zuul12:45
*** hashar has quit IRC13:15
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add role to fetch output from nodes  https://review.openstack.org/51184313:22
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Update upload-logs to process docs as well  https://review.openstack.org/51185313:44
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Upload fetched artifacts to log server  https://review.openstack.org/51185814:09
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Only publish artifacts when zuul.change is not defined  https://review.openstack.org/51185914:09
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add role to fetch output from nodes  https://review.openstack.org/51184314:13
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Update upload-logs to process docs as well  https://review.openstack.org/51185314:13
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Upload fetched artifacts to log server  https://review.openstack.org/51185814:13
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Only publish artifacts when zuul.change is not defined  https://review.openstack.org/51185914:13
dmsimardmordred: some more comments on that output stack14:40
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Update upload-logs to process docs as well  https://review.openstack.org/51185315:03
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Upload fetched artifacts to log server  https://review.openstack.org/51185815:03
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Only publish artifacts when zuul.change is not defined  https://review.openstack.org/51185915:03
dmsimardmordred: so the output things will be added to the base job ?15:04
mordreddmsimard: yah - did you see my email?15:04
dmsimardmordred: on -infra ? Yeah, but I don't see a patch for the base job update yet :p15:05
mordredah - there should be patches - all in the zuulv3-output topic15:05
dmsimardmordred: oh, indeed15:05
dmsimardmordred: I also had a last question in https://review.openstack.org/#/c/511859/2/roles/publish-artifacts-to-fileserver/tasks/main.yaml@1215:06
mordreddmsimard: good question - answered there - but tl;dr is yes, periodic, post and release ... it's about not uploading artifacts to the artifacts server for check and gate changes (but instead uploading them to the log server if there are any artifacts - in case someone wants to emit an artifact to verify it before actually publishing it)15:09
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Better exception handling during autohold  https://review.openstack.org/51190215:12
dmsimardmordred: I replied with another question :D15:13
ShrewsI think we need a zuul cli command to output our current autohold list15:19
pabelanger++15:24
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Better exception handling during autohold  https://review.openstack.org/51190215:26
Shrewssilly pep815:26
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: WIP: Do not delete instances that are 'building'  https://review.openstack.org/51191716:25
*** sambetts is now known as sambetts|adk17:20
*** sambetts|adk is now known as sambetts|afk17:20
*** Guest53680 is now known as mgagne18:14
*** mgagne has quit IRC18:14
*** mgagne has joined #zuul18:14
*** electrofelix has quit IRC18:32
dmsimardmordred: I just noticed we have an include_role for set-zuul-log-path-fact in 'emit-job-header' which is in base. Should that be explicit (executed from the base playbook before emit-job-header) rather than implicit ?18:52
dmsimardI was writing a patch to run integration tests for emit-job-header but noticed that it would also be testing set-zuul-log-path-fact, I don't mind, just thought I would ask.18:53
dmsimardI've thought about it and leaving it like that makes sense, the role simply can't work without the fact role. It's probably better to have it as an include_role rather than a meta dependency.19:05
dmsimard¯\_(ツ)_/¯19:05
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove -e from requirements  https://review.openstack.org/51195720:15
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Remove -e from requirements  https://review.openstack.org/51195720:44
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Switch statsd config to zuul.conf  https://review.openstack.org/51101721:09
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some stats from executor  https://review.openstack.org/51107321:09
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some nodepool stats  https://review.openstack.org/51108521:09
jeblairShrews: re autohold: why did the node have no data?21:25
jeblairmordred: want to +3 511902 ?21:25
mordredjeblair: sure!21:28
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some nodepool stats  https://review.openstack.org/51108521:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Specify duplicate project defn behavior  https://review.openstack.org/51102521:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: experiment with late-binding inheritance  https://review.openstack.org/51135221:41
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Use weakref for change cache  https://review.openstack.org/51135521:44
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Better exception handling during autohold  https://review.openstack.org/51190221:54
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: experiment with late-binding inheritance  https://review.openstack.org/51135222:51
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some stats from executor  https://review.openstack.org/51107322:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some nodepool stats  https://review.openstack.org/51108522:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Re-order executor/job classes  https://review.openstack.org/51197722:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Make statsd import failure raise exception  https://review.openstack.org/51197823:12
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Switch statsd config to zuul.conf  https://review.openstack.org/51101723:22
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Re-order executor/job classes  https://review.openstack.org/51197723:31
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Emit some stats from executor  https://review.openstack.org/51107323:31
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Emit some nodepool stats  https://review.openstack.org/51108523:31

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