| SamYaple | SpamapS: ive do deterministic and reproducible builds whereever possible. i even have a side projects to build deterministic container images. huge fan of reproducible-builds | 00:00 |
|---|---|---|
| *** baoli has joined #openstack-infra | 00:00 | |
| SamYaple | 90% of the work is just timestamp bits anyway | 00:00 |
| clarkb | constraints actually goes a long way to addressing a major reproduceable build issue in python packagin (and pipfile is the next generation/future of that) | 00:01 |
| SamYaple | looking forward to pipfiles for sure | 00:01 |
| SpamapS | SamYaple: yep | 00:01 |
| clarkb | but anytime you can run arbitrary code to intsall packages well ya | 00:01 |
| clarkb | SamYaple: check out `pip install --no-binary rdalal` :) | 00:01 |
| *** jdg has quit IRC | 00:02 | |
| *** jaypipes has quit IRC | 00:02 | |
| SamYaple | im confused | 00:02 |
| SamYaple | oh wow | 00:03 |
| clarkb | so now if the package can do that when installing from source, it can bsaically do whatever it wants whne building a wheel too :) | 00:04 |
| *** tosky has quit IRC | 00:04 | |
| SamYaple | ugh fair enough | 00:05 |
| SamYaple | does infra rebuild the wheels they have often? | 00:05 |
| *** jdg has joined #openstack-infra | 00:06 | |
| clarkb | SamYaple: whenever requirements updates it looks like | 00:08 |
| SamYaple | awesome. thats what i was hoping for | 00:08 |
| *** jdg has quit IRC | 00:11 | |
| *** jdg has joined #openstack-infra | 00:11 | |
| *** gildub has joined #openstack-infra | 00:12 | |
| *** mrunge has quit IRC | 00:12 | |
| pabelanger | Yah, I still need to make sure the wheel-builds are working for zuulv3 | 00:13 |
| *** mrunge has joined #openstack-infra | 00:14 | |
| SpamapS | mordred: do you know if there is already an os_loadbalancer ansible module in the works? | 00:16 |
| SpamapS | https://github.com/ansible/ansible/issues/30098 | 00:17 |
| SpamapS | n/m, google provides | 00:17 |
| *** slaweq_ has quit IRC | 00:20 | |
| *** slaweq_ has joined #openstack-infra | 00:25 | |
| *** ijw has joined #openstack-infra | 00:26 | |
| *** xarses has joined #openstack-infra | 00:30 | |
| *** wolverineav has joined #openstack-infra | 00:34 | |
| *** felipemonteiro__ has quit IRC | 00:38 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix Gearman UnknownJob handler https://review.openstack.org/508992 | 00:39 |
| *** wolverineav has quit IRC | 00:39 | |
| *** Swami has quit IRC | 00:40 | |
| openstackgerrit | Mohammed Naser proposed openstack-infra/openstack-zuul-jobs master: Refactor tox to use the role https://review.openstack.org/509559 | 00:46 |
| openstackgerrit | Mohammed Naser proposed openstack-infra/openstack-zuul-jobs master: Switch translated release notes conditionals https://review.openstack.org/509560 | 00:46 |
| fungi | #status log updated openids in the storyboard.openstack.org database from login.launchpad.net to login.ubuntu.com | 00:51 |
| openstackstatus | fungi: finished logging | 00:51 |
| fungi | mordred: ^ | 00:51 |
| *** mriedem has quit IRC | 00:51 | |
| pabelanger | mordred: clarkb: so, I think the first thing we'd need to do is update zuul-mergers and support cloning from HTTP(s), then we'd be able to use gitconfig limits for bandwidth | 00:52 |
| pabelanger | we also can use GitPython but I've found a few bugs i think in the kill_after_timeout options, we'll need to fix that upstream also | 00:53 |
| pabelanger | or roll our own timeout handling | 00:53 |
| *** sree has joined #openstack-infra | 00:58 | |
| *** slaweq_ has quit IRC | 00:58 | |
| *** jdg has quit IRC | 00:59 | |
| *** slaweq_ has joined #openstack-infra | 01:00 | |
| *** cshastri has joined #openstack-infra | 01:05 | |
| *** stakeda has joined #openstack-infra | 01:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 01:09 |
| *** sree has quit IRC | 01:10 | |
| *** sree has joined #openstack-infra | 01:11 | |
| *** Swami has joined #openstack-infra | 01:14 | |
| *** sree has quit IRC | 01:15 | |
| *** jgriffith_ has joined #openstack-infra | 01:19 | |
| *** hongbin has joined #openstack-infra | 01:20 | |
| *** ijw has quit IRC | 01:21 | |
| *** erlon has quit IRC | 01:22 | |
| *** jgriffith_ has quit IRC | 01:23 | |
| *** jgriffith_ has joined #openstack-infra | 01:23 | |
| *** bnemec-backlater has quit IRC | 01:24 | |
| *** bnemec has joined #openstack-infra | 01:24 | |
| *** jgriffith_ has quit IRC | 01:26 | |
| *** SumitNaiksatam has joined #openstack-infra | 01:27 | |
| *** jgriffith_ has joined #openstack-infra | 01:27 | |
| *** xarses has quit IRC | 01:32 | |
| *** slaweq_ has quit IRC | 01:32 | |
| *** jgriffith_ has quit IRC | 01:34 | |
| openstackgerrit | Mohammed Naser proposed openstack-infra/openstack-zuul-jobs master: Avoid moving release notes after complete execution https://review.openstack.org/509675 | 01:35 |
| *** slaweq_ has joined #openstack-infra | 01:39 | |
| *** Douhet has quit IRC | 01:43 | |
| *** yamahata has quit IRC | 01:44 | |
| *** Douhet has joined #openstack-infra | 01:44 | |
| *** iyamahat has quit IRC | 01:44 | |
| *** xarses has joined #openstack-infra | 01:45 | |
| *** andreww has joined #openstack-infra | 01:48 | |
| *** SumitNaiksatam has quit IRC | 01:48 | |
| *** xarses has quit IRC | 01:50 | |
| *** cuongnv has joined #openstack-infra | 01:50 | |
| *** jhesketh has quit IRC | 01:51 | |
| *** jhesketh has joined #openstack-infra | 01:51 | |
| *** dave-mcc_ has quit IRC | 01:57 | |
| *** katyafervent_awa has quit IRC | 01:59 | |
| *** adreznec has quit IRC | 02:00 | |
| *** dteselkin has quit IRC | 02:00 | |
| *** kiennt26 has joined #openstack-infra | 02:00 | |
| *** lukebrowning_ has quit IRC | 02:01 | |
| *** igormarnat has quit IRC | 02:01 | |
| *** timothyb89 has quit IRC | 02:01 | |
| *** afazekas has quit IRC | 02:02 | |
| *** lukebrowning has joined #openstack-infra | 02:02 | |
| *** afazekas has joined #openstack-infra | 02:02 | |
| *** adreznec has joined #openstack-infra | 02:02 | |
| *** timothyb89 has joined #openstack-infra | 02:02 | |
| *** andreww has quit IRC | 02:04 | |
| *** katyafervent_awa has joined #openstack-infra | 02:05 | |
| *** dteselkin has joined #openstack-infra | 02:05 | |
| *** igormarnat has joined #openstack-infra | 02:06 | |
| *** slaweq_ has quit IRC | 02:09 | |
| *** hemna_ has joined #openstack-infra | 02:11 | |
| *** bobh has joined #openstack-infra | 02:13 | |
| *** slaweq_ has joined #openstack-infra | 02:18 | |
| *** Rockyg has joined #openstack-infra | 02:18 | |
| *** owalsh_ has joined #openstack-infra | 02:28 | |
| *** owalsh has quit IRC | 02:31 | |
| *** hemna_ has quit IRC | 02:39 | |
| *** ramishra has joined #openstack-infra | 02:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 02:41 |
| *** jgriffith_ has joined #openstack-infra | 02:42 | |
| *** nicolasbock has quit IRC | 02:43 | |
| *** jkilpatr has quit IRC | 02:46 | |
| SamYaple | what exactly happens between pre-run and run playbooks? are they run back to back without any envrionement changes between them? or is there any code that runs between those two playbooks? | 02:48 |
| SamYaple | im trying to track down an issue with how things are behaving different from the legacy jobs | 02:48 |
| SamYaple | coming up blank | 02:48 |
| *** yamamoto has joined #openstack-infra | 02:48 | |
| *** slaweq_ has quit IRC | 02:51 | |
| openstackgerrit | Mohammed Naser proposed openstack-infra/openstack-zuul-jobs master: Remove trailing slash from releasenote job sphinx_output_src https://review.openstack.org/509678 | 02:52 |
| openstackgerrit | Mohammed Naser proposed openstack-infra/openstack-zuul-jobs master: Drop zuul_executor_dest from release notes job variables https://review.openstack.org/509679 | 02:52 |
| *** ijw has joined #openstack-infra | 02:53 | |
| *** slaweq_ has joined #openstack-infra | 02:55 | |
| *** ijw has quit IRC | 02:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 02:57 |
| *** sflanigan has joined #openstack-infra | 02:59 | |
| *** sflanigan has joined #openstack-infra | 02:59 | |
| *** bobh has quit IRC | 03:00 | |
| openstackgerrit | Mohammed Naser proposed openstack-infra/project-config master: Switch release note publish jobs to use the correct path https://review.openstack.org/509680 | 03:00 |
| *** ijw has joined #openstack-infra | 03:02 | |
| *** baoli has quit IRC | 03:02 | |
| *** claudiub has quit IRC | 03:07 | |
| *** ijw has quit IRC | 03:08 | |
| *** jgriffith_ has quit IRC | 03:08 | |
| *** ijw has joined #openstack-infra | 03:11 | |
| *** lbragstad has joined #openstack-infra | 03:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 03:13 |
| *** ijw has quit IRC | 03:16 | |
| *** baoli has joined #openstack-infra | 03:17 | |
| clarkb | SamYaple: should be nothing | 03:19 |
| *** larainema has joined #openstack-infra | 03:25 | |
| *** baoli has quit IRC | 03:26 | |
| *** slaweq_ has quit IRC | 03:28 | |
| *** sree has joined #openstack-infra | 03:30 | |
| *** udesale has joined #openstack-infra | 03:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 03:33 |
| *** slaweq_ has joined #openstack-infra | 03:34 | |
| *** markvoelker has quit IRC | 03:36 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Use the infra version of sphinx jobs for infra https://review.openstack.org/509683 | 03:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Use infra docs jobs for nodepool https://review.openstack.org/509684 | 03:38 |
| *** tonyb has quit IRC | 03:42 | |
| *** hongbin has quit IRC | 03:42 | |
| *** zzzeek has quit IRC | 03:46 | |
| *** links has joined #openstack-infra | 03:48 | |
| *** lbragstad has quit IRC | 03:50 | |
| *** dbecker has quit IRC | 03:54 | |
| *** tonyb has joined #openstack-infra | 03:56 | |
| *** vhosakot has quit IRC | 04:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 04:03 |
| *** dbecker has joined #openstack-infra | 04:07 | |
| *** SumitNaiksatam has joined #openstack-infra | 04:10 | |
| *** jgriffith has quit IRC | 04:11 | |
| *** sflanigan has quit IRC | 04:11 | |
| *** jgriffith has joined #openstack-infra | 04:11 | |
| *** armax has quit IRC | 04:12 | |
| *** kiennt26 has quit IRC | 04:12 | |
| *** armax has joined #openstack-infra | 04:13 | |
| *** armax has quit IRC | 04:13 | |
| *** armax has joined #openstack-infra | 04:13 | |
| *** armax has quit IRC | 04:14 | |
| *** isviridov_away has quit IRC | 04:14 | |
| *** vaidy has quit IRC | 04:14 | |
| *** jamielennox has quit IRC | 04:14 | |
| *** armax has joined #openstack-infra | 04:14 | |
| *** armax has quit IRC | 04:14 | |
| openstackgerrit | Merged openstack-infra/project-config master: Fix project-config-frozen-files https://review.openstack.org/509513 | 04:15 |
| *** armax has joined #openstack-infra | 04:15 | |
| *** armax has quit IRC | 04:15 | |
| *** armax has joined #openstack-infra | 04:16 | |
| *** armax has quit IRC | 04:16 | |
| *** cburgess has quit IRC | 04:16 | |
| *** vaidy has joined #openstack-infra | 04:17 | |
| *** isviridov_away has joined #openstack-infra | 04:17 | |
| *** jamielennox has joined #openstack-infra | 04:18 | |
| *** cburgess has joined #openstack-infra | 04:19 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 04:19 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Print environment for legacy-grenade-dsvm-ironic https://review.openstack.org/509007 | 04:33 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Make openstackci-beaker-ubuntu-trusty and logstash-filters non-voting https://review.openstack.org/509182 | 04:33 |
| *** coolsvap has joined #openstack-infra | 04:33 | |
| *** Rockyg has quit IRC | 04:35 | |
| *** zzzeek has joined #openstack-infra | 04:38 | |
| *** ekcs has joined #openstack-infra | 04:40 | |
| *** slaweq_ has quit IRC | 04:41 | |
| *** bhavik1 has joined #openstack-infra | 04:44 | |
| *** slaweq_ has joined #openstack-infra | 04:49 | |
| *** chlong_ has joined #openstack-infra | 04:51 | |
| *** stakeda has quit IRC | 04:51 | |
| *** gmann is now known as gmann_lunch | 04:58 | |
| *** psachin has joined #openstack-infra | 05:01 | |
| *** gouthamr has quit IRC | 05:06 | |
| *** jtomasek has joined #openstack-infra | 05:08 | |
| *** eumel8 has joined #openstack-infra | 05:12 | |
| openstackgerrit | Hiroaki Kobayashi proposed openstack-infra/project-config master: Add required-projects to the blazar-dashboard jobs https://review.openstack.org/509078 | 05:19 |
| *** slaweq_ has quit IRC | 05:22 | |
| *** dhajare has joined #openstack-infra | 05:26 | |
| *** slaweq_ has joined #openstack-infra | 05:26 | |
| openstackgerrit | Hiroaki Kobayashi proposed openstack-infra/project-config master: Add required-projects to the blazar-dashboard jobs https://review.openstack.org/509078 | 05:27 |
| openstackgerrit | Hiroaki Kobayashi proposed openstack-infra/project-config master: Add required-projects to the blazar-dashboard jobs https://review.openstack.org/509078 | 05:31 |
| *** markvoelker has joined #openstack-infra | 05:37 | |
| *** Rockyg has joined #openstack-infra | 05:38 | |
| *** claudiub has joined #openstack-infra | 05:40 | |
| *** udesale__ has joined #openstack-infra | 05:42 | |
| *** gmann_lunch is now known as gmann | 05:43 | |
| *** udesale has quit IRC | 05:45 | |
| *** threestrands has quit IRC | 05:45 | |
| *** gildub has quit IRC | 05:48 | |
| *** ekcs has quit IRC | 05:53 | |
| *** dhellmann has quit IRC | 05:56 | |
| *** dhellmann has joined #openstack-infra | 05:58 | |
| *** slaweq_ has quit IRC | 05:59 | |
| *** pgadiya has joined #openstack-infra | 06:00 | |
| *** pgadiya has quit IRC | 06:04 | |
| *** slaweq_ has joined #openstack-infra | 06:05 | |
| *** ykarel has joined #openstack-infra | 06:09 | |
| *** ykarel is now known as ykarel|pto | 06:10 | |
| *** markvoelker has quit IRC | 06:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 06:11 |
| AJaeger | clarkb, mnaser : Releasenotes are published from master. But you need to check that they build if you add/touch them on a branch | 06:13 |
| *** chlong_ has quit IRC | 06:13 | |
| *** andreas_s has joined #openstack-infra | 06:15 | |
| *** ykarel_ has joined #openstack-infra | 06:16 | |
| *** pgadiya has joined #openstack-infra | 06:17 | |
| *** ykarel_ has quit IRC | 06:18 | |
| *** ykarel|pto has quit IRC | 06:19 | |
| *** jbadiapa has joined #openstack-infra | 06:19 | |
| *** rcernin has joined #openstack-infra | 06:24 | |
| *** kiennt26 has joined #openstack-infra | 06:26 | |
| *** martinkopec has joined #openstack-infra | 06:28 | |
| *** iyamahat has joined #openstack-infra | 06:28 | |
| *** jascott1_ has joined #openstack-infra | 06:31 | |
| *** jascott1 has quit IRC | 06:31 | |
| *** vsaienk0 has joined #openstack-infra | 06:32 | |
| *** spectr has quit IRC | 06:34 | |
| *** spectr has joined #openstack-infra | 06:34 | |
| *** kiennt26 has quit IRC | 06:37 | |
| *** slaweq_ has quit IRC | 06:38 | |
| *** Swami has quit IRC | 06:43 | |
| *** slaweq_ has joined #openstack-infra | 06:44 | |
| *** udesale has joined #openstack-infra | 06:46 | |
| *** udesale__ has quit IRC | 06:48 | |
| *** kiennt26 has joined #openstack-infra | 06:48 | |
| *** shu-mutou-AWAY is now known as shu-mutou | 06:49 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci master: Collect /etc/snmp in quickstart jobs https://review.openstack.org/509525 | 06:50 |
| AJaeger | ianw, yolanda, we can merge again - yesterday just the website is broken. | 06:50 |
| AJaeger | could you put https://review.openstack.org/#/c/508998/ and https://review.openstack.org/#/c/509578 on your review queue, please? And then https://review.openstack.org/#/c/508757/ (can only merge after 508998 is in), I can +A later. | 06:53 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy static- and releases- jobs https://review.openstack.org/508999 | 06:58 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove obsolete validate jobs https://review.openstack.org/509579 | 07:00 |
| AJaeger | and these two as well, please ^ | 07:00 |
| *** tesseract has joined #openstack-infra | 07:07 | |
| *** markvoelker has joined #openstack-infra | 07:08 | |
| *** pcaruana has joined #openstack-infra | 07:08 | |
| kklimonda | is there an easy way to test zuulv3 jobs/additional ansible modules without zuul scheduler/executor setup? alternatively, what's the workflow that most people use right now? | 07:13 |
| numans | AJaeger, Hi | 07:13 |
| AJaeger | kklimonda: I'm not aware - you might want to ask the main developers later - they are US based | 07:14 |
| AJaeger | numans: feel free to leave a message here... | 07:14 |
| numans | AJaeger, its regarding https://review.openstack.org/#/c/502899/ | 07:14 |
| *** mwarad has joined #openstack-infra | 07:15 | |
| numans | AJaeger, in the comment you asked to check in this channel on how to remove the duplicated code. do you have any comments on that ? | 07:15 |
| AJaeger | numans: I suggested to ask the rest of the team ;) Best ask during US office hours again... | 07:16 |
| numans | AJaeger, ok. sure. Are you fine with that present state of the patch ? I can follow up with another patch to remove the duplicate code. | 07:17 |
| numans | after checking with other folks. | 07:17 |
| *** slaweq_ has quit IRC | 07:17 | |
| numans | AJaeger, thanks. | 07:18 |
| *** slaweq_ has joined #openstack-infra | 07:21 | |
| AJaeger | numans: let's wait for that discussion, please | 07:23 |
| numans | AJaeger, ok. sure. | 07:24 |
| * AJaeger has no time for detailed review now and needs to go offline again... | 07:24 | |
| *** stakeda has joined #openstack-infra | 07:24 | |
| *** gongysh has joined #openstack-infra | 07:24 | |
| *** gildub has joined #openstack-infra | 07:27 | |
| matbu | o/ hi folks, im just passing here to say that we have strange behavior in the tripleo verified gate | 07:28 |
| matbu | the verified gate is running several times with success (like a loop) and finally merged the review | 07:29 |
| matbu | my review take 14hours to get merged (with a lot of useless verified jobs) | 07:29 |
| *** iyamahat has quit IRC | 07:34 | |
| *** e0ne has joined #openstack-infra | 07:34 | |
| *** alexchadin has joined #openstack-infra | 07:37 | |
| *** jascott1_ has quit IRC | 07:41 | |
| *** jascott1 has joined #openstack-infra | 07:41 | |
| *** markvoelker has quit IRC | 07:42 | |
| *** jascott1 has quit IRC | 07:46 | |
| *** lucas-afk is now known as lucasagomes | 07:46 | |
| *** vsaienk0 has quit IRC | 07:48 | |
| *** vsaienk0 has joined #openstack-infra | 07:51 | |
| *** jpena|off is now known as jpena | 07:52 | |
| *** eranrom__ has quit IRC | 07:52 | |
| *** ociuhandu has joined #openstack-infra | 07:53 | |
| *** spiffxp_ has quit IRC | 07:54 | |
| openstackgerrit | Duong Ha-Quang proposed openstack-infra/project-config master: Remove legacy jobs from tacker-horizon https://review.openstack.org/509706 | 07:54 |
| *** vipul has quit IRC | 07:55 | |
| openstackgerrit | Merged openstack-infra/project-config master: add ec2-api to unified doc build jobs https://review.openstack.org/508880 | 07:56 |
| *** egonzalez has joined #openstack-infra | 07:58 | |
| *** armaan has quit IRC | 08:00 | |
| *** armaan has joined #openstack-infra | 08:00 | |
| openstackgerrit | Merged openstack-infra/project-config master: neutron-fwaas: Add requirements for tox jobs https://review.openstack.org/509137 | 08:02 |
| openstackgerrit | Merged openstack-infra/project-config master: networking-odl: Remove cover job from post https://review.openstack.org/509138 | 08:06 |
| openstackgerrit | Merged openstack-infra/project-config master: Remove centrally defined jobs for openstack/releases https://review.openstack.org/508998 | 08:06 |
| openstackgerrit | Merged openstack-infra/project-config master: Change the docs job to a deploy-publish-job https://review.openstack.org/508298 | 08:06 |
| *** spectr has quit IRC | 08:09 | |
| *** spectr has joined #openstack-infra | 08:09 | |
| *** Rockyg has quit IRC | 08:11 | |
| *** spectr has quit IRC | 08:17 | |
| *** spectr has joined #openstack-infra | 08:18 | |
| *** shardy has joined #openstack-infra | 08:19 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Add another missing repo to legacy-openstackci-beaker jobs https://review.openstack.org/509358 | 08:20 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove auto-converted legacy shade jobs https://review.openstack.org/508757 | 08:20 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Use the infra version of sphinx jobs for infra https://review.openstack.org/509683 | 08:20 |
| *** dingyichen has quit IRC | 08:21 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Really only copy logs dir for chef-rake-integration https://review.openstack.org/508841 | 08:24 |
| openstackgerrit | Merged openstack-infra/project-config master: Add separate coverage job for ironic-inspector https://review.openstack.org/508129 | 08:24 |
| *** slaweq_ has quit IRC | 08:27 | |
| *** ricky_ has joined #openstack-infra | 08:28 | |
| *** ralonsoh has joined #openstack-infra | 08:28 | |
| *** slaweq_ has joined #openstack-infra | 08:31 | |
| openstackgerrit | Merged openstack-infra/project-config master: Return neutron functional and api jobs back to voting for zuulv3 https://review.openstack.org/509287 | 08:33 |
| openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/nodepool feature/zuulv3: Bring back per label groups in Openstack https://review.openstack.org/509620 | 08:33 |
| *** hashar has joined #openstack-infra | 08:34 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Use same flake8 config as in zuul https://review.openstack.org/509715 | 08:34 |
| *** shu-mutou is now known as shu-mutou-AWAY | 08:35 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add neutron to required-projects for dragonflow https://review.openstack.org/508785 | 08:38 |
| *** vsaienk0 has quit IRC | 08:38 | |
| *** markvoelker has joined #openstack-infra | 08:38 | |
| *** vsaienk0 has joined #openstack-infra | 08:41 | |
| *** tmorin has joined #openstack-infra | 08:41 | |
| tmorin | hi yolanda... about https://review.openstack.org/#/c/509167 : I've Cc'd neutron liaison, but I would be glad if we can merge as is to unbreak our CI, the change does not introduce anything we hadn't before, it's only a zuulv3 bug fix... | 08:42 |
| tmorin | yolanda: same for https://review.openstack.org/#/c/508825 | 08:43 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Fix nodesets for tricircle multi-region job https://review.openstack.org/508824 | 08:43 |
| yolanda | tmorin, ok , let's ask AJaeger about the opinion ^ | 08:43 |
| tmorin | yolanda: https://review.openstack.org/#/c/508961 / https://review.openstack.org/#/c/508822 which do the equivalent for more neutron stadium projects, didn't even had neutron liaison Cc'd | 08:45 |
| openstackgerrit | Merged openstack-infra/storyboard master: Handle Branches from Launchpad in Migration https://review.openstack.org/437469 | 08:45 |
| yolanda | tmorin, i understand the issue, but before merging myself i'd like to have the agreement of at least another core | 08:46 |
| AJaeger | yolanda: yeah, I would merge those that just fix the v3 jobs... | 08:46 |
| yolanda | good | 08:47 |
| AJaeger | yolanda: but yamamoto has just +1'ed as neutron liaison ;) | 08:47 |
| yolanda | it got it yes, so no discussion :) | 08:47 |
| tmorin | AJaeger: yamamoto is quick ! :) | 08:47 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Add release-notes-jobs-neutron https://review.openstack.org/509165 | 08:47 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Fix puppet jobs by adding integration parent https://review.openstack.org/509173 | 08:47 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: legacy-cross-networking-midonet-python35: Add missing requirements https://review.openstack.org/509153 | 08:47 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Add diskimage-builder to legacy-rally-dsvm-verify jobs https://review.openstack.org/509171 | 08:47 |
| AJaeger | yolanda: could you +2 https://review.openstack.org/#/c/508999/ and https://review.openstack.org/#/c/509579 - those should be ready after recheck | 08:47 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Modified "required_projects" for openstack/omni project https://review.openstack.org/509159 | 08:47 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove legacy-rally-dsvm-keystone-v2api-rally definition https://review.openstack.org/508850 | 08:47 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove legacy Packstack jobs https://review.openstack.org/508855 | 08:47 |
| *** dfflanders has joined #openstack-infra | 08:48 | |
| AJaeger | wow! nice to see those merges... | 08:48 |
| *** owalsh_ is now known as owalsh | 08:50 | |
| AJaeger | yolanda: you've been busy - thanks! | 08:50 |
| SpamapS | AJaeger: yeah, along with the memory leak fixes, feels like maybe the retry will go smoother. ;) | 08:51 |
| * SpamapS really should be asleep | 08:51 | |
| yolanda | indeed | 08:51 |
| AJaeger | yolanda: could you also +2A https://review.openstack.org/#/c/509578/ , please? | 08:56 |
| AJaeger | SpamapS: yes, it looks really nice today ;) | 08:57 |
| AJaeger | Have a well deserved sleep, SpamapS | 08:57 |
| AJaeger | yolanda: with the depends-on between project-config and openstack-zuul-jobs, some changes need several changes ;( | 08:58 |
| yolanda | yep, with that project separation, but looks more clear now | 08:59 |
| AJaeger | it does... | 09:02 |
| AJaeger | thanks, yolanda | 09:02 |
| *** sree_ has joined #openstack-infra | 09:02 | |
| *** sree_ is now known as Guest68095 | 09:02 | |
| *** slaweq_ has quit IRC | 09:03 | |
| *** jbadiapa has quit IRC | 09:03 | |
| *** namnh has joined #openstack-infra | 09:04 | |
| *** sree has quit IRC | 09:05 | |
| AJaeger | mordred, jeblair, pabelanger one question on moving projects to their own repos: Will they move *all* their templates including merge-check and publish-to-pypi to their .zuul.yaml? Or is there anything that needs to stay in project-config? | 09:07 |
| *** markvoelker has quit IRC | 09:12 | |
| *** inc0 has quit IRC | 09:13 | |
| *** Swimingly has quit IRC | 09:13 | |
| *** greghaynes has quit IRC | 09:14 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove un-used legacy-releasenotes job https://review.openstack.org/508766 | 09:16 |
| *** alexchadin has quit IRC | 09:16 | |
| *** yamamoto has quit IRC | 09:16 | |
| *** alexchadin has joined #openstack-infra | 09:16 | |
| *** jbadiapa has joined #openstack-infra | 09:17 | |
| *** ccamacho has joined #openstack-infra | 09:18 | |
| *** yamamoto has joined #openstack-infra | 09:20 | |
| *** bhavik1 has quit IRC | 09:23 | |
| *** vipul has joined #openstack-infra | 09:24 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add projects to required-projects for networking-(bagpipe|bgpvpn) https://review.openstack.org/508825 | 09:24 |
| *** spiffxp has joined #openstack-infra | 09:25 | |
| *** haypo has joined #openstack-infra | 09:27 | |
| haypo | hello. the legacy-tempest-dsvm-neutron-src job failed on installing oslo.versionedobjects from the Git repository. pip complains with "!! Could not determine repository location". did someone already see this error? see my report on the latest comment of https://review.openstack.org/#/c/508423/ | 09:27 |
| openstackgerrit | Merged openstack-infra/project-config master: Use release-notes-jobs-neutron for networking-ovn https://review.openstack.org/509176 | 09:27 |
| openstackgerrit | Merged openstack-infra/project-config master: v3: Use openstack-tox-validate https://review.openstack.org/509578 | 09:27 |
| *** slaweq_ has joined #openstack-infra | 09:28 | |
| *** armaan has quit IRC | 09:28 | |
| *** armaan has joined #openstack-infra | 09:29 | |
| *** alexchadin has quit IRC | 09:30 | |
| *** Swimingly has joined #openstack-infra | 09:30 | |
| *** Swimingly has quit IRC | 09:30 | |
| *** Swimingly has joined #openstack-infra | 09:30 | |
| *** stakeda has quit IRC | 09:30 | |
| openstackgerrit | Jens Harbott (frickler) proposed openstack-infra/project-config master: Fix neutron-dynamic-routing python jobs https://review.openstack.org/509726 | 09:30 |
| frickler | haypo: that was an issue that got fixed in devstack, see https://review.openstack.org/508344 and https://review.openstack.org/508907 | 09:30 |
| frickler | haypo: we are also back to running with zuul v2 for now, I did a recheck so we can see what jenkins has to say about it | 09:31 |
| haypo | frickler: oh! it's nice to hear that a bug is already fixed :) | 09:32 |
| *** inc0 has joined #openstack-infra | 09:35 | |
| *** greghaynes has joined #openstack-infra | 09:38 | |
| *** kiennt26 has quit IRC | 09:38 | |
| *** tosky has joined #openstack-infra | 09:40 | |
| *** gildub has quit IRC | 09:44 | |
| *** jascott1 has joined #openstack-infra | 09:45 | |
| *** geguileo has joined #openstack-infra | 09:46 | |
| *** derekh has joined #openstack-infra | 09:47 | |
| *** mwarad has quit IRC | 09:47 | |
| geguileo | AJaeger: hi, is it normal that now with a recheck I get +1 from Jenkins but Zuul does't remove the -1 it had on the patch? | 09:47 |
| haypo | frickler: thanks | 09:48 |
| AJaeger | geguileo: Zuul will also recheck but might be slower since we have assigned fewer nodes to it. | 09:49 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Add required projects to a legacy n8g-bgpvpn job https://review.openstack.org/509167 | 09:49 |
| AJaeger | geguileo: if you check zuulv3.openstack.org, you should see your job still in the queue - if not, issue another "recheck". BUT you can merge a change that has a -1 by Zuul, so if you do not touch the Zuul v3 files, go ahead and merge the change... | 09:50 |
| geguileo | AJaeger: but the patch doesn't appear on the status web | 09:50 |
| AJaeger | geguileo: which ones? | 09:50 |
| geguileo | AJaeger: https://review.openstack.org/#/c/508167/ | 09:50 |
| geguileo | AJaeger: Problem is that people don't look at patches with -1 :-( | 09:50 |
| AJaeger | geguileo: they will... And a recheck might not help if the jobs are broken | 09:54 |
| AJaeger | But looking at your change: You got results after your recheck - since those are post failures, I issued another recheck | 09:54 |
| AJaeger | geguileo: Jay looked at it ;) | 09:54 |
| geguileo | AJaeger: OK, thanks | 09:54 |
| openstackgerrit | Chandan Kumar proposed openstack-infra/project-config master: Add python-tempestconf project https://review.openstack.org/508502 | 09:56 |
| AJaeger | geguileo: and it shows now up on zuulv3.o.o | 09:58 |
| geguileo | AJaeger: Awesome, thanks! | 09:58 |
| *** sdague has joined #openstack-infra | 10:00 | |
| *** pbourke has quit IRC | 10:00 | |
| *** Guest68095 has quit IRC | 10:00 | |
| AJaeger | yolanda: https://review.openstack.org/#/c/508766/ is ready - if you have time for a quick cleanup review, please | 10:01 |
| *** sree has joined #openstack-infra | 10:01 | |
| *** slaweq_ has quit IRC | 10:01 | |
| *** pbourke has joined #openstack-infra | 10:01 | |
| *** yamamoto has quit IRC | 10:02 | |
| openstackgerrit | Chandan Kumar proposed openstack-infra/project-config master: Add python-tempestconf project https://review.openstack.org/508502 | 10:02 |
| *** tikitavi has joined #openstack-infra | 10:03 | |
| chandankumar | AJaeger: need help on this review https://review.openstack.org/#/c/508502/ not sure how to make both gates passing | 10:04 |
| *** tikitavi_ has joined #openstack-infra | 10:04 | |
| *** sree has quit IRC | 10:05 | |
| *** iyamahat has joined #openstack-infra | 10:05 | |
| *** sree has joined #openstack-infra | 10:05 | |
| *** pblaho has quit IRC | 10:05 | |
| tikitavi_ | Hi, it’s me with the problem in https://review.openstack.org/#/c/500091/ in stable/ocata. In a month or longer we have problems with instances that don’t start. Looking for example to instance 05a8ce7e-6ca6-40eb-adaf-7da260b67077 in nova-api logs and is fully absent in nova-cpu and nova-sch logs (logs: http://logs.openstack.org/91/500091/7/check/gate-functional-neutron-dsvm-ec2api-ubuntu-xenial/b05f013/) The same pic | 10:05 |
| AJaeger | chandankumar: let'S ask others for help - mordred, pabelanger, could you review 508502 when you're online and guide chandankumar on what to do, please? | 10:06 |
| *** yamamoto has joined #openstack-infra | 10:06 | |
| *** tikitavi has quit IRC | 10:07 | |
| openstackgerrit | Duong Ha-Quang proposed openstack-infra/project-config master: Remove trivial jobs in murano deliverables https://review.openstack.org/509746 | 10:08 |
| *** tikitavi has joined #openstack-infra | 10:09 | |
| *** pblaho has joined #openstack-infra | 10:09 | |
| chandankumar | AJaeger: thanks :-) | 10:09 |
| *** markvoelker has joined #openstack-infra | 10:09 | |
| *** sree has quit IRC | 10:10 | |
| *** iyamahat has quit IRC | 10:11 | |
| *** slaweq_ has joined #openstack-infra | 10:11 | |
| *** yamamoto has quit IRC | 10:12 | |
| *** alexchadin has joined #openstack-infra | 10:12 | |
| *** tikitavi_ has quit IRC | 10:13 | |
| *** tikitavi_ has joined #openstack-infra | 10:15 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove legacy static- and releases- jobs https://review.openstack.org/508999 | 10:17 |
| frickler | tikitavi_: I don't think that this is an issue with infra, did you try to reproduce this locally? also there are critical errors in n-cond that seem relevant: http://logs.openstack.org/91/500091/7/check/gate-functional-neutron-dsvm-ec2api-ubuntu-xenial/b05f013/logs/screen-n-cond.txt.gz?level=ERROR | 10:18 |
| *** tikitavi has quit IRC | 10:18 | |
| *** yamamoto has joined #openstack-infra | 10:19 | |
| *** tikitavi has joined #openstack-infra | 10:19 | |
| *** tikitavi_ has quit IRC | 10:21 | |
| *** tikitavi_ has joined #openstack-infra | 10:23 | |
| tikitavi_ | frickler: the problem with instances are in different tests every time. Once for a month all of them have passed | 10:29 |
| *** wolverineav has joined #openstack-infra | 10:29 | |
| tikitavi_ | We suppose that it may be a problem with resources... | 10:30 |
| *** alexchadin has quit IRC | 10:35 | |
| *** tikitavi_ has quit IRC | 10:35 | |
| *** udesale has quit IRC | 10:38 | |
| *** yamamoto has quit IRC | 10:41 | |
| *** markvoelker has quit IRC | 10:42 | |
| *** dtantsur|afk is now known as dtantsur | 10:44 | |
| *** tikitavi_ has joined #openstack-infra | 10:45 | |
| *** tikitavi has quit IRC | 10:47 | |
| *** tikitavi has joined #openstack-infra | 10:48 | |
| *** slaweq_ has quit IRC | 10:50 | |
| *** cuongnv has quit IRC | 10:52 | |
| *** tikitavi_ has quit IRC | 10:52 | |
| *** alexchadin has joined #openstack-infra | 10:54 | |
| *** tikitavi_ has joined #openstack-infra | 10:54 | |
| *** spectr has quit IRC | 10:54 | |
| *** andreas_s_ has joined #openstack-infra | 10:56 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/os-testr master: init testr repo when running with --pdb https://review.openstack.org/509752 | 10:56 |
| *** ghostg has joined #openstack-infra | 10:57 | |
| *** andreas_s has quit IRC | 10:58 | |
| *** tikitavi has quit IRC | 10:58 | |
| *** jpena is now known as jpena|lunch | 10:58 | |
| *** nicolasbock has joined #openstack-infra | 10:59 | |
| *** tikitavi has joined #openstack-infra | 10:59 | |
| *** ricky_ has quit IRC | 10:59 | |
| *** yamamoto has joined #openstack-infra | 11:01 | |
| *** tikitavi_ has quit IRC | 11:01 | |
| *** tikitavi_ has joined #openstack-infra | 11:02 | |
| tikitavi_ | by the way, the last patch didn't pass, because the instance didn't started even during devstack deploy - http://logs.openstack.org/91/500091/8/check/gate-functional-neutron-dsvm-ec2api-ubuntu-xenial/206b664/ | 11:03 |
| *** dave-mccowan has joined #openstack-infra | 11:04 | |
| *** nicolasbock has quit IRC | 11:05 | |
| *** nicolasbock has joined #openstack-infra | 11:05 | |
| *** priteau has joined #openstack-infra | 11:05 | |
| *** yamamoto has quit IRC | 11:06 | |
| *** tikitavi has quit IRC | 11:06 | |
| *** tikitavi has joined #openstack-infra | 11:07 | |
| *** ghostg has left #openstack-infra | 11:07 | |
| *** tikitavi_ has quit IRC | 11:10 | |
| *** tikitavi_ has joined #openstack-infra | 11:11 | |
| *** tikitavi has quit IRC | 11:14 | |
| *** alexchadin has quit IRC | 11:14 | |
| *** alexchadin has joined #openstack-infra | 11:15 | |
| *** ociuhandu has quit IRC | 11:15 | |
| *** jkilpatr has joined #openstack-infra | 11:15 | |
| *** slaweq_ has joined #openstack-infra | 11:15 | |
| *** tikitavi has joined #openstack-infra | 11:16 | |
| *** hemna_ has joined #openstack-infra | 11:17 | |
| *** lucasagomes is now known as lucas-hungry | 11:17 | |
| *** tikitavi_ has quit IRC | 11:18 | |
| *** tikitavi_ has joined #openstack-infra | 11:20 | |
| *** tikitavi__ has joined #openstack-infra | 11:22 | |
| *** tikitavi has quit IRC | 11:23 | |
| *** jkilpatr has quit IRC | 11:24 | |
| *** tikitavi_ has quit IRC | 11:25 | |
| *** alexchadin has quit IRC | 11:27 | |
| AJaeger | tikitavi__: better ask on #openstack-qa and dig into the devstack logs | 11:27 |
| *** alexchadin has joined #openstack-infra | 11:35 | |
| *** tikitavi__ has quit IRC | 11:36 | |
| *** jkilpatr has joined #openstack-infra | 11:38 | |
| *** haypo has left #openstack-infra | 11:39 | |
| *** ldnunes has joined #openstack-infra | 11:39 | |
| *** tikitavi has joined #openstack-infra | 11:39 | |
| *** markvoelker has joined #openstack-infra | 11:39 | |
| *** cshastri has quit IRC | 11:40 | |
| *** ianychoi has quit IRC | 11:46 | |
| *** slaweq_ has quit IRC | 11:51 | |
| *** priteau has quit IRC | 11:53 | |
| *** priteau has joined #openstack-infra | 11:53 | |
| *** priteau has quit IRC | 11:54 | |
| *** priteau has joined #openstack-infra | 11:54 | |
| *** bobh has joined #openstack-infra | 11:54 | |
| *** yamamoto has joined #openstack-infra | 12:00 | |
| *** dprince has joined #openstack-infra | 12:08 | |
| *** namnh has quit IRC | 12:10 | |
| *** alexchadin has quit IRC | 12:10 | |
| *** markvoelker has quit IRC | 12:12 | |
| *** jpena|lunch is now known as jpena | 12:13 | |
| *** edmondsw has joined #openstack-infra | 12:13 | |
| *** edmondsw_ has joined #openstack-infra | 12:16 | |
| *** armaan has quit IRC | 12:16 | |
| *** armaan has joined #openstack-infra | 12:16 | |
| *** tpsilva has joined #openstack-infra | 12:16 | |
| *** edmondsw has quit IRC | 12:17 | |
| *** lucas-hungry is now known as lucasagomes | 12:17 | |
| *** slaweq_ has joined #openstack-infra | 12:18 | |
| mnaser | morning infra, release note jobs are fixed! stack at https://review.openstack.org/#/q/topic:zuulv3-releasenotes-job can be reviewed .. i split it into smaller easy-to-review patches to make things easier, there's also a job in that topic that depends on the changes to show that its working | 12:18 |
| *** vsaienk0 has quit IRC | 12:18 | |
| *** vsaienk0 has joined #openstack-infra | 12:20 | |
| *** ldnunes has quit IRC | 12:21 | |
| *** slaweq_ has quit IRC | 12:22 | |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: set neutron mtu to match system mtu https://review.openstack.org/509761 | 12:22 |
| *** edmondsw_ has quit IRC | 12:22 | |
| *** bobh has quit IRC | 12:24 | |
| *** trown|outtypewww is now known as trown | 12:25 | |
| *** jcoufal has joined #openstack-infra | 12:27 | |
| *** alexchadin has joined #openstack-infra | 12:27 | |
| *** markvoelker has joined #openstack-infra | 12:28 | |
| *** edmondsw has joined #openstack-infra | 12:29 | |
| *** vsaienk0 has quit IRC | 12:31 | |
| *** vsaienk0 has joined #openstack-infra | 12:32 | |
| *** slaweq_ has joined #openstack-infra | 12:33 | |
| *** ldnunes has joined #openstack-infra | 12:33 | |
| *** edmondsw has quit IRC | 12:33 | |
| *** edmondsw has joined #openstack-infra | 12:36 | |
| *** pgadiya has quit IRC | 12:38 | |
| *** yamamoto has quit IRC | 12:39 | |
| *** yamamoto has joined #openstack-infra | 12:41 | |
| *** SotK_ has joined #openstack-infra | 12:41 | |
| *** links has quit IRC | 12:44 | |
| *** yamamoto has quit IRC | 12:46 | |
| *** SotK_ has left #openstack-infra | 12:48 | |
| *** camunoz has joined #openstack-infra | 12:50 | |
| *** yamamoto has joined #openstack-infra | 12:51 | |
| *** bobh has joined #openstack-infra | 12:56 | |
| *** kgiusti has joined #openstack-infra | 12:56 | |
| *** udesale has joined #openstack-infra | 12:58 | |
| AJaeger | mnaser: great! I hope mordred will review the stack... | 12:58 |
| *** mriedem has joined #openstack-infra | 12:59 | |
| jzerebecki | could someone with the permission remove zuul from this patch as it doesn't react and thus will remain with vote -1 (zuul is different from jenkins for which users have the permission to remove it) https://review.openstack.org/#/c/508261/ | 13:02 |
| *** spectr has joined #openstack-infra | 13:02 | |
| *** Goneri has joined #openstack-infra | 13:03 | |
| AJaeger | jzerebecki: a core can merge 508261 - even with a -1 by zuul. | 13:03 |
| AJaeger | jzerebecki: you need a +1 by jenkins - and I see none on 508261 | 13:04 |
| jzerebecki | AJaeger: made a thinko and removed jenkins instead of zuul :( the last vote of jekins was a +1 | 13:05 |
| AJaeger | jzerebecki: why did you remove that? The rule with Zuul v2 is: You can merge if there's a +1 by Jenkins and cores +2A | 13:05 |
| AJaeger | jzerebecki: so, recheck to get back the Jenkins +1 - and then somebody else can review and +2A | 13:05 |
| jzerebecki | AJaeger: I would assume nobody looks at it with a zuul -1 | 13:05 |
| *** yamamoto has quit IRC | 13:06 | |
| eumel8 | can someone check why the build job for the releasenotes failed: https://review.openstack.org/#/c/509498/ | 13:06 |
| AJaeger | infra-root, I think we need to do some education here - this is the third+ time this came up ^ | 13:06 |
| *** slaweq_ has quit IRC | 13:06 | |
| *** lbragstad has joined #openstack-infra | 13:06 | |
| AJaeger | jzerebecki: that's a perception problem, let's see what the rest of the team suggest for messaging. | 13:07 |
| AJaeger | eumel8: POST_FAILURE on zuul v3 - we know those are broken. mnaser has fixes for them. Ignore for now. | 13:07 |
| eumel8 | AJaeger: thx! | 13:08 |
| AJaeger | eumel8: some comment as to jzerebecki : a core can merge these changes without problems... | 13:08 |
| eumel8 | k | 13:09 |
| *** esberglu has joined #openstack-infra | 13:09 | |
| fungi | jzerebecki: until we roll back forward onto zuul v3, verify votes provided by the "zuul" account are purely advisory in nature just like for a third-party ci system | 13:12 |
| *** eharney has joined #openstack-infra | 13:12 | |
| fungi | and we do want it voting so we can see which v3 jobs are still broken and give people a (brief) opportunity to work on fixing them before we roll back to v3 and they become authoritative again | 13:13 |
| AJaeger | fungi: I see people "panicking" about such an advisory vote. Perhaps we might need to send out another email? | 13:13 |
| fungi | i did a moment ago on one of the threads | 13:13 |
| fungi | but honestly i have no time for people who panic over such nonsense | 13:13 |
| AJaeger | fungi, cool. | 13:13 |
| AJaeger | yeah ;( | 13:13 |
| tikitavi | I really need help, for more than a month the review https://review.openstack.org/#/c/500091/ didn't pass because one or more instances cannot start because of the errors in n-cond | 13:14 |
| mordred | AJaeger: which stack? | 13:14 |
| *** eumel8 has quit IRC | 13:15 | |
| fungi | tikitavi: that doesn't sound like a question for the infra team, you probably want to discuss it with the nova or qa teams | 13:15 |
| AJaeger | mordred: https://review.openstack.org/#/q/topic:zuulv3-releasenotes-job | 13:16 |
| AJaeger | fungi, mordred, here's a cleanup patch that I'd like to see in, could you review that as well as time permits, please? - needs one +2A https://review.openstack.org/#/c/509579/ | 13:18 |
| AJaeger | and also https://review.openstack.org/#/c/508766/ | 13:20 |
| mordred | AJaeger: reviewing ... | 13:20 |
| *** slaweq_ has joined #openstack-infra | 13:20 | |
| mordred | AJaeger: also, when we have a little time, I'd love to figure out how to make a sphinx plugin or something that can do the translation steps in the releasenotes job | 13:21 |
| alexchadin | hi, we have two legacy jobs for watcher project - multinode and watcherclient-dsvm-functional. Where should I move watcherclient-dsvm-functional: to watcher repo or to python-watcherclient repo? | 13:21 |
| *** baoli has joined #openstack-infra | 13:21 | |
| AJaeger | mordred: best talk with dhellmann about that | 13:22 |
| mordred | alexchadin: either one will work - whichever one makes more sense to you - since watcherclient-dsvm-functional is a watcherclient named job, perhaps python-watcherclient ? | 13:22 |
| mordred | AJaeger: oh yah - he's the master of such things :) | 13:22 |
| mordred | alexchadin: you can use it in both repos no matter which repo you put it in | 13:22 |
| AJaeger | fungi, mordred , thanks for reviews | 13:22 |
| alexchadin | mordred: if I place watcherclient-dsvm-functional into watcher repo, how will python-watcherclient know where to find this job? | 13:24 |
| *** gouthamr has joined #openstack-infra | 13:25 | |
| mnaser | alexchadin the answer is magic | 13:27 |
| mnaser | zuulv3 locates jobs across all repos | 13:27 |
| fungi | well, across all repos it knows about and is configured to expect job configuration in, which in our case is all repos in our gerrit at the moment | 13:28 |
| *** erlon has joined #openstack-infra | 13:29 | |
| alexchadin | wow | 13:29 |
| alexchadin | thanks, mnaser, fungi it makes a little bit more sense now :D | 13:29 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove obsolete validate jobs https://review.openstack.org/509579 | 13:30 |
| stephenfin | dhellmann: I found out why the reno patch was running on my machine but failing on the gate - I was using tox under Python 3 https://review.openstack.org/#/c/475033/ | 13:35 |
| *** jaosorior has quit IRC | 13:35 | |
| *** coolsvap has quit IRC | 13:36 | |
| stephenfin | dhellmann: Looks like it is indeed a bug in upstream distutils, which sucks :( | 13:36 |
| beisner | hi all - apologies if this is a dup question. i'm having trouble getting things to land. here's an example: https://review.openstack.org/#/c/509068/ | 13:38 |
| *** alexchadin has quit IRC | 13:38 | |
| *** martinkopec has quit IRC | 13:39 | |
| *** alexchadin has joined #openstack-infra | 13:39 | |
| *** eranrom has joined #openstack-infra | 13:39 | |
| *** spectr has quit IRC | 13:40 | |
| AJaeger | beisner: you need a +1 by *Jenkins* with the partial rollback we did - add a "recheck" comment and you should be good to go... | 13:41 |
| beisner | ack thx AJaeger | 13:42 |
| AJaeger | mnaser: do you want to rebase https://review.openstack.org/#/c/508766/ to resolve the merge conflict? | 13:43 |
| openstackgerrit | Merged openstack-infra/storyboard master: Check story permissions when populating automatic worklists https://review.openstack.org/509601 | 13:43 |
| mnaser | AJaeger sure 1 sec | 13:44 |
| numans | Hello everyone, I have a patch here - https://review.openstack.org/#/c/502899/. In the review comments AJaeger suggested to check here to see if the patch can be improved. Basically, there is lot of duplication in this patch. I wanted to know if there is a way to get rid of this duplication. | 13:44 |
| *** mriedem is now known as ronlund | 13:44 | |
| AJaeger | pabelanger, mordred, could you help numans, please? ^ | 13:45 |
| numans | AJaeger, thanks. | 13:45 |
| *** alexchad_ has joined #openstack-infra | 13:46 | |
| mordred | mnaser: great patches for releasenotes - thank you!!! | 13:46 |
| *** chlong_ has joined #openstack-infra | 13:46 | |
| *** alexchadin has quit IRC | 13:47 | |
| AJaeger | +2! Thanks! | 13:48 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 13:49 |
| AJaeger | mordred: will you +2 https://review.openstack.org/#/c/509680/ as well? I know it has a depends, so needs to wait... | 13:50 |
| openstackgerrit | Mohammed Naser proposed openstack-infra/openstack-zuul-jobs master: Remove un-used legacy-releasenotes job https://review.openstack.org/508766 | 13:50 |
| mnaser | mordred thank you so much :) also there's the rebase AJaeger ^ | 13:50 |
| AJaeger | mnaser: thanks, reviewed with -1 | 13:52 |
| *** jaypipes has joined #openstack-infra | 13:52 | |
| mnaser | oops i dun goof'd | 13:53 |
| AJaeger | mnaser: and remove the playbook - see the previous revision that I pushed | 13:53 |
| *** bramleya has joined #openstack-infra | 13:53 | |
| *** slaweq_ has quit IRC | 13:54 | |
| openstackgerrit | Mohammed Naser proposed openstack-infra/openstack-zuul-jobs master: Remove un-used legacy-releasenotes job https://review.openstack.org/508766 | 13:55 |
| mnaser | AJaeger sorry, that should be better | 13:55 |
| mordred | tonyb, fungi, prometheanfire, clarkb, pabelanger, dims, dhellmann: https://review.openstack.org/#/c/508891 https://review.openstack.org/#/c/508894/ if you get a sec? I'd like to finish getting new requirements-check jobs rolled out | 13:55 |
| *** slaweq_ has joined #openstack-infra | 13:57 | |
| *** spectr has joined #openstack-infra | 13:57 | |
| *** ronlund is now known as mriedem | 13:58 | |
| *** srobert has joined #openstack-infra | 13:58 | |
| *** hongbin has joined #openstack-infra | 13:59 | |
| AJaeger | mordred: those kind of changes are now cross-repo - changing it everywhere gets complicated ;( Still, I love the benefits ;) | 14:00 |
| *** alexchad_ has quit IRC | 14:00 | |
| AJaeger | mnaser: fine, +2! Thanks | 14:00 |
| *** shardy has quit IRC | 14:00 | |
| *** shardy has joined #openstack-infra | 14:00 | |
| tosky | I just saw a revision in sahara that tried to import the zuulv3 legacy jobs as an intermediate step for the migration - and I voted -2. Are there official guidelines about this? I'd rather have the time to write proper new jobs, instead of importing them and then changing them | 14:00 |
| AJaeger | tosky: https://docs.openstack.org/infra/manual/zuulv3.html | 14:01 |
| tosky | AJaeger: yes, that's the "how do the migration" | 14:02 |
| pabelanger | morning | 14:02 |
| pabelanger | catching up on backscroll | 14:02 |
| *** spzala has joined #openstack-infra | 14:02 | |
| AJaeger | mordred, pabelanger a related one question on moving projects to their own repos: Will they move *all* their templates including merge-check and publish-to-pypi to their .zuul.yaml? Or is there anything that needs to stay in project-config? | 14:02 |
| tosky | the question is more: should we copy the legacy job immediately and remove from project-config, and then convert them, or can we take the time to rewrite them properly? | 14:02 |
| *** srobert has quit IRC | 14:03 | |
| tosky | I'm basically questioning the reason for https://review.openstack.org/#/q/topic:migrate-to-zuulv3 | 14:03 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Refactor releasenote job to have one path for tox building https://review.openstack.org/509558 | 14:03 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Refactor tox to use the role https://review.openstack.org/509559 | 14:03 |
| *** zzzeek has quit IRC | 14:03 | |
| pabelanger | AJaeger: there should be anything that is needed to say, but we might want to do _something_ for operations. But right now, I don't know what that would be | 14:03 |
| AJaeger | tosky, at the bottom of the docs patch is a step by step guide | 14:03 |
| tosky | not all of them just the one that copy the legacy jobs | 14:03 |
| pabelanger | AJaeger: I think time will tell | 14:03 |
| tosky | AJaeger: yes, but it describe something different than what's done by https://review.openstack.org/#/c/509702/ - or at least I read it differently | 14:04 |
| AJaeger | tosky: indeed, it misses the "rename" bit | 14:05 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Switch translated release notes conditionals https://review.openstack.org/509560 | 14:05 |
| * AJaeger is in a meeting now, will be back later | 14:05 | |
| tosky | ok, fine; but still, can I -2 that patch and instead working on writing native jobs directly? | 14:06 |
| *** gongysh has quit IRC | 14:06 | |
| *** yamamoto has joined #openstack-infra | 14:06 | |
| *** zzzeek has joined #openstack-infra | 14:07 | |
| *** gongysh has joined #openstack-infra | 14:07 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Avoid moving release notes after complete execution https://review.openstack.org/509675 | 14:07 |
| *** spectr has quit IRC | 14:07 | |
| AJaeger | tosky: quickly commented | 14:07 |
| *** spectr has joined #openstack-infra | 14:08 | |
| dims | mordred : +2A on both | 14:08 |
| tosky | AJaeger: thanks, but... I'd still prefer to not do that in that way | 14:08 |
| *** jaosorior has joined #openstack-infra | 14:10 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove trailing slash from releasenote job sphinx_output_src https://review.openstack.org/509678 | 14:10 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Drop zuul_executor_dest from release notes job variables https://review.openstack.org/509679 | 14:10 |
| *** alexchadin has joined #openstack-infra | 14:11 | |
| mordred | dims: thanks! | 14:12 |
| mordred | AJaeger: I believe we should, by policy, keep merge-check in project-config ... also, for projects subject to the PTI, I believe we should keep PTI job definitions in project-config, since those aren't actually supposed to be optional | 14:13 |
| mordred | AJaeger: but we should probably write up some docs on that | 14:13 |
| *** yamamoto has quit IRC | 14:13 | |
| jeblair | tosky, AJaeger: i think rewriting new native v3 jobs in-repo while the legacy ones stay in project-config is fine -- as long as you don't need to change the legacy jobs very much. (and then obviously removing the legacy jobs later.) | 14:15 |
| *** gongysh has quit IRC | 14:15 | |
| tosky | jeblair: that was the idea; I just need to fix the existing jobs first, probably something related to (surprise surprise) keystone v2 removal | 14:16 |
| *** jheroux has joined #openstack-infra | 14:16 | |
| mordred | tosky: :) | 14:16 |
| jeblair | tosky: yeah, i imagine we're going to have a window where we make lots of changes to the legacy jobs, then when they mostly work, we're going to want to stop changing them. | 14:17 |
| AJaeger | mordred: yes, let's document that, please. We have first repos moving stuff over and some move *everything*. | 14:17 |
| cloudnull | qq - we're seeing Zuul review on a few of our jobs (example: https://review.openstack.org/#/c/501054) is that expected? should we see both Jenkins and Zuul, & are those blocking votes from Zuul ATM? | 14:18 |
| evrardjp | cloudnull: yes | 14:19 |
| frickler | cloudnull: yes that is expected and no, you can regard zuul like a 3rd party ci for now | 14:19 |
| evrardjp | see the dev ML | 14:19 |
| cloudnull | ok. thanks! | 14:19 |
| AJaeger | mordred: https://review.openstack.org/#/c/509746/1/zuul.d/projects.yaml | 14:19 |
| jeblair | apparently zuulv3 is working well enough to leave votes now. :) | 14:20 |
| jlvillal | Any known issue with log.o.o? I'm seeing a timeout error for: logs.openstack.org/12/453012/9/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial/ed418e3/ | 14:20 |
| jlvillal | never mind. it works now | 14:20 |
| AJaeger | jeblair: you did some miracles yesterday, today I'm really happy with zuul v3's performance! | 14:20 |
| AJaeger | ... and stability! | 14:21 |
| *** nikhil_k has joined #openstack-infra | 14:22 | |
| *** nikhil_k is now known as nikhil | 14:23 | |
| dansmith | is logs.o.o down? | 14:23 |
| frickler | jlvillal: dansmith: seems pretty loaded currently | 14:23 |
| frickler | http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=311&rra_id=all | 14:24 |
| dansmith | heh, sorry I just noticed jlvillal's question my bad | 14:24 |
| dansmith | I looked, but skipped the very recent history :/ | 14:24 |
| jeblair | http://cacti.openstack.org/cacti/graph_view.php?action=tree&tree_id=1&leaf_id=13 | 14:24 |
| jeblair | recent outbound traffic spike, and connection spike | 14:25 |
| jeblair | there are a lot of requests for /kolla/images/centos-binary-registry-ocata.tar.gz | 14:26 |
| fungi | it's possible those just finished rebuilding and all our proxies are updating their caches? | 14:26 |
| fungi | though in the past it's been random people on the internet starting to download them over and over (likely runaway ci or deployment systems) | 14:27 |
| fungi | in this case, tons of random addresses in china, it looks like | 14:30 |
| jeblair | ya, that's what i'm seeing too | 14:30 |
| *** slaweq_ has quit IRC | 14:30 | |
| fungi | some making as many as 500 requests for that file | 14:30 |
| pabelanger | possible 3rd party CI out of china? | 14:31 |
| fungi | i count 2030 distinct ip addresses requesting that file between 14:00-14:10z | 14:32 |
| fungi | 99930 requests in all for that file over a 10 minute span | 14:32 |
| fungi | inc0: ^ any clue what could have triggered this? | 14:32 |
| sdague | someone doing a cluster deploy with it? | 14:32 |
| fungi | that would be a huge cluster | 14:32 |
| fungi | 1k requests per minute for the image | 14:33 |
| fungi | we've had to mark the kolla images non-world-readable in the past for similar denial of service conditions. may need to do that again | 14:33 |
| fungi | i don't fancy adding 2k+ distinct entries to iptables | 14:34 |
| *** trown is now known as trown|brb | 14:34 | |
| *** vhosakot has joined #openstack-infra | 14:34 | |
| sdague | yeh, or figure out a way to segment them onto a dedicated system so they don't impact other services | 14:34 |
| jeblair | what's our bw limit for this host? | 14:34 |
| fungi | judging from cacti, likely 500mbps | 14:35 |
| fungi | though also possible rackspace's rate-limiting has a bit of a delay before it kicks in | 14:35 |
| *** gmann is now known as gmann_sleep | 14:35 | |
| fungi | server show says the flavor is performance1-8 | 14:36 |
| *** eranrom has quit IRC | 14:36 | |
| jeblair | fungi: well, i wonder if we're hitting apache limits rather than bw limits, and if we could tune up | 14:37 |
| jeblair | | rxtx_factor | 1600.0 | | 14:37 |
| fungi | is it the rxtx_factor in flavor show which is what tells us the bandwidth limit? | 14:37 |
| sdague | yeh, it would be nice if it wasn't contending against the logs though, which being slow slows down interactive development | 14:37 |
| *** felipemonteiro_ has joined #openstack-infra | 14:37 | |
| *** alexchadin has quit IRC | 14:37 | |
| jeblair | fungi: yeah, i forget how to decode that | 14:37 |
| *** slaweq_ has joined #openstack-infra | 14:38 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Handle double node locking snafu https://review.openstack.org/509603 | 14:38 |
| fungi | sdague: agreed, we've talked about moving tarballs.o.o off of static.o.o for a while for a variety of reasons | 14:38 |
| fungi | and also moving logs.o.o off static.o.o | 14:38 |
| *** felipemonteiro__ has joined #openstack-infra | 14:38 | |
| jeblair | help is welcome :) | 14:39 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove un-used legacy-releasenotes job https://review.openstack.org/508766 | 14:39 |
| jeblair | 40% of aggregate | 14:40 |
| jeblair | so 640mbps | 14:40 |
| *** trown|brb is now known as trown | 14:40 | |
| mriedem | jhesketh: mikal: we can probably remove turbo-hipster from nova-ci now right? https://review.openstack.org/#/admin/groups/511,members i don't think that's run on nova changes in a long time, and no one is maintaining it at rax anymore are they? | 14:40 |
| *** armax has joined #openstack-infra | 14:41 | |
| *** felipemonteiro_ has quit IRC | 14:42 | |
| fungi | infra-root: i've temoprarily installed apachetop on static.o.o to see if it's much help | 14:44 |
| fungi | neat, /kolla/images/centos-binary-registry-ocata.tar.gz pops right to the top | 14:45 |
| pabelanger | which is about 4GB, last I looked | 14:45 |
| *** ccamacho has left #openstack-infra | 14:45 | |
| fungi | clocking around 3 requests per second for it | 14:45 |
| jeblair | server-status seems broken on that host :( | 14:45 |
| openstackgerrit | Telles Mota Vidal Nóbrega proposed openstack-infra/irc-meetings master: Changing Sahara meeting time https://review.openstack.org/509796 | 14:46 |
| fungi | jeblair: broken as in available to everyone you mean, right? | 14:47 |
| jeblair | fungi: er, no i wasn't seeing it at all | 14:47 |
| jeblair | did i mistype it? | 14:47 |
| fungi | oh, i'm getting to it via security.o.o/server-status | 14:47 |
| jeblair | oh, ha, the irony | 14:48 |
| fungi | yeah :/ | 14:48 |
| fungi | i've yet to figure out the cause, but i think it's that you have to explicitly exclude access for it in every vhost definition on named-based multihost systems | 14:48 |
| fungi | teh apache docs are unfortunately vague about restricting it | 14:48 |
| *** yamamoto has joined #openstack-infra | 14:48 | |
| *** yamamoto has quit IRC | 14:48 | |
| jeblair | that's actually looking pretty good. of course, logs.o.o is working for me now, so... | 14:49 |
| *** eranrom has joined #openstack-infra | 14:49 | |
| fungi | right, i think they're probably causing a denial of service against the gfwoc | 14:49 |
| fungi | actual data transfer for tarballs.o.o seems to be spiking up around 4MB/sec according to apachetop (so i guess 35Mbps) | 14:51 |
| mordred | AJaeger: ok - I commented on that murano- patch | 14:51 |
| openstackgerrit | Merged openstack-infra/nodepool feature/zuulv3: Bring back per label groups in Openstack https://review.openstack.org/509620 | 14:51 |
| fungi | oh, though bandwidth utilization per cacti has also trailed back down to nearly nothing | 14:52 |
| fungi | ~60mbps outbound | 14:52 |
| mordred | pabelanger, rcarrillocruz I put up a patch last night to switch to the infra-docs job | 14:52 |
| jeblair | fungi: yay we fixed it | 14:52 |
| jeblair | pabelanger: regarding http://logs.openstack.org/20/509620/2/check/build-openstack-sphinx-docs/c6a1d72/job-output.txt.gz#_2017-10-04_22_33_06_558917 | 14:53 |
| fungi | anyway, at least i determined that apachetop is a way useful piece of kit | 14:53 |
| mordred | pabelanger, rcarrillocruz: https://review.openstack.org/#/c/509684/ | 14:53 |
| jeblair | pabelanger: do we not want to run docs build jobs on all changes to all branches? even though openstack doesn't publish feature branches? | 14:53 |
| fungi | packaged in ubuntu, so we can just apt-get install it as needed. invoking `sudo apachetop -f /var/log/apache2/tarballs.openstack.org_access | 14:53 |
| fungi | .log | 14:53 |
| fungi | ` | 14:53 |
| fungi | yay stray newlines | 14:53 |
| jeblair | fungi: cool, thanks | 14:53 |
| pabelanger | jeblair: I think we want to be using build-openstack-infra-sphinx-docs job | 14:53 |
| fungi | tails the access log and tracks utilization across a number of axes | 14:54 |
| jeblair | pabelanger: why is it infra-specific? | 14:54 |
| pabelanger | at least, that is how we talked about doing it a few weeks ago | 14:54 |
| pabelanger | jeblair: when we originally did this, there was a difference on how openstack docs and infra docs were published. | 14:54 |
| pabelanger | so we created 2 jobs | 14:54 |
| jeblair | pabelanger: yes, but this isn't *publishing* docs, it's *building* docs | 14:54 |
| jeblair | i don't understand what needs to be different about that | 14:55 |
| jeblair | this is a job that runs in the check pipeline. its purpose is to verify that docs can be built, and then to upload them to logs.o.o (only) so that people can see them. | 14:55 |
| *** vsaienk0 has quit IRC | 14:56 | |
| pabelanger | jeblair: both build / publish jobs use the same roles, which sets up some path things. The only difference when I last checked, if just changed where they published too between them. | 14:56 |
| mordred | jeblair: the difference is the prepare-docs-for-afs role | 14:56 |
| pabelanger | so, i think if we want to make all jobs use build-openstack-sphinx-docs, we need to refactor a bit | 14:56 |
| mordred | jeblair: which we run in the build- job | 14:56 |
| mordred | the infra prepare-docs-for-afs is different, as it allows docs on feature branches | 14:57 |
| jeblair | mordred: i feel like that shouldn't be in the build job -- the build job doesn't publish to afs | 14:57 |
| jeblair | the build job should just publish the docs as built | 14:57 |
| mordred | yah- I think the thinking was so that people could verify that stamps were prepared properly and whatnot | 14:57 |
| mordred | jeblair: but I'm also fine with just removing the afs prep from thebuild jobs and collapsing back down to one build-openstack-sphinx-docs job | 14:58 |
| pabelanger | Sure, i'd be okay with that | 14:58 |
| jeblair | i think the key thing for me is -- how does an openstack project avoid breaking their docs on feature branches? | 14:58 |
| *** Swami has joined #openstack-infra | 14:58 | |
| mordred | however - I think also we were keeping "shift infra to using the normal openstack docs jobs" as a post-v3 thing | 14:59 |
| mordred | jeblair: yah -that's a great point | 14:59 |
| mordred | let's do the remove afs prep from build jobs | 14:59 |
| * mordred can take that real quick | 14:59 | |
| jeblair | ok, thanks | 14:59 |
| *** eranrom has quit IRC | 15:01 | |
| *** gouthamr_ has joined #openstack-infra | 15:03 | |
| *** gouthamr has quit IRC | 15:04 | |
| *** gouthamr_ is now known as gouthamr | 15:04 | |
| *** mat128 has joined #openstack-infra | 15:05 | |
| *** jaosorior has quit IRC | 15:07 | |
| openstackgerrit | Mohammed Naser proposed openstack-infra/openstack-zuul-jobs master: Add openstack/puppet-openstack-cookiecutter to puppet jobs https://review.openstack.org/509802 | 15:08 |
| openstackgerrit | Merged openstack-infra/project-config master: Switch release note publish jobs to use the correct path https://review.openstack.org/509680 | 15:09 |
| *** slaweq_ has quit IRC | 15:11 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Stop doing afs prep in docs build jobs https://review.openstack.org/509804 | 15:13 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Put doc build jobs into doc publication project-template https://review.openstack.org/509805 | 15:13 |
| *** slaweq_ has joined #openstack-infra | 15:14 | |
| mordred | jeblair, pabelanger, AJaeger: ^^ does the afs change - and then the second is a little bit of a reorg (moves doc build to publish template instead of python-jobs template) | 15:14 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Remove build-openstack-sphinx-docs-infra https://review.openstack.org/509808 | 15:18 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add release-openstack-sphinx-docs-infra template https://review.openstack.org/509809 | 15:18 |
| *** andreas_s_ has quit IRC | 15:21 | |
| *** iyamahat has joined #openstack-infra | 15:24 | |
| *** hashar is now known as hasharAway | 15:24 | |
| *** iyamahat has quit IRC | 15:31 | |
| *** cody-somerville has joined #openstack-infra | 15:32 | |
| pabelanger | mordred: -1 on 509804, otherwise looks good | 15:35 |
| mordred | pabelanger: oh great catch! | 15:36 |
| *** srobert has joined #openstack-infra | 15:39 | |
| *** jogo has quit IRC | 15:39 | |
| *** hemna__ has joined #openstack-infra | 15:39 | |
| *** pcaruana has quit IRC | 15:39 | |
| *** srobert has quit IRC | 15:40 | |
| *** iyamahat has joined #openstack-infra | 15:42 | |
| *** hemna_ has quit IRC | 15:42 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Stop doing afs prep in docs build jobs https://review.openstack.org/509804 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Put doc build jobs into doc publication project-template https://review.openstack.org/509805 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Remove build-openstack-sphinx-docs-infra https://review.openstack.org/509808 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add release-openstack-sphinx-docs-infra template https://review.openstack.org/509809 | 15:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add project-template for python jobs without constraints https://review.openstack.org/509815 | 15:42 |
| *** hemna_ has joined #openstack-infra | 15:43 | |
| *** hemna__ has quit IRC | 15:44 | |
| *** jbadiapa has quit IRC | 15:45 | |
| *** hemna__ has joined #openstack-infra | 15:46 | |
| *** slaweq_ has quit IRC | 15:46 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Put doc build jobs into doc publication project-template https://review.openstack.org/509805 | 15:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Remove build-openstack-sphinx-docs-infra https://review.openstack.org/509808 | 15:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add release-openstack-sphinx-docs-infra template https://review.openstack.org/509809 | 15:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add project-template for python jobs without constraints https://review.openstack.org/509815 | 15:46 |
| mordred | pabelanger, AJaeger, fungi, jeblair: ^^ ok - that stack should be solid now | 15:46 |
| *** srobert has joined #openstack-infra | 15:47 | |
| *** hemna_ has quit IRC | 15:48 | |
| *** tikitavi has quit IRC | 15:49 | |
| *** yamamoto has joined #openstack-infra | 15:49 | |
| pabelanger | looking | 15:49 |
| *** slaweq_ has joined #openstack-infra | 15:51 | |
| *** spectr has quit IRC | 15:51 | |
| *** cody-somerville has quit IRC | 15:53 | |
| jeblair | mordred: what would use https://review.openstack.org/509809 ? | 15:54 |
| *** tmorin has quit IRC | 15:54 | |
| *** cody-somerville has joined #openstack-infra | 15:55 | |
| *** eumel8 has joined #openstack-infra | 15:56 | |
| *** nicolasbock_ has joined #openstack-infra | 15:56 | |
| *** bhavik1 has joined #openstack-infra | 15:56 | |
| *** yamamoto has quit IRC | 15:58 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Switch infra projects to using docs templates https://review.openstack.org/509820 | 15:59 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Switch infra repos to not use constraints https://review.openstack.org/509821 | 15:59 |
| mordred | jeblair: bindep, git-restack, git-review, python-storyboardclient and storyboard | 15:59 |
| *** baoli has quit IRC | 15:59 | |
| *** baoli has joined #openstack-infra | 16:00 | |
| *** egonzalez has quit IRC | 16:00 | |
| mordred | jeblair: also, https://review.openstack.org/509820 and https://review.openstack.org/509821 apply the various templates | 16:01 |
| jeblair | infra-root, AJaeger, jlk, dmsimard, mnaser: can we spend a few minutes checking in on zuulv3 etherpad progress? | 16:02 |
| *** dprince has quit IRC | 16:03 | |
| pabelanger | sure | 16:04 |
| jeblair | pabelanger: did you put the 100% swap item on the debug list? | 16:04 |
| pabelanger | jeblair: I did, line 95 | 16:04 |
| *** jpena is now known as jpena|off | 16:05 | |
| jeblair | pabelanger: okay, we did that intentionally on friday | 16:05 |
| pabelanger | ah, okay | 16:05 |
| *** dprince has joined #openstack-infra | 16:05 | |
| jeblair | we'll probably re-allocate that at some point, but we didn't have an immediate need for it, so... | 16:05 |
| pabelanger | jeblair: we are up to 53% capacity on / for zuulv3.o.o, something to keep an eye on | 16:06 |
| jeblair | pabelanger: yeah, and is likely to get worse. maybe when we reallocate it, we should mount it at /logs | 16:06 |
| jeblair | i'm just going to remove it from the debug list for now | 16:06 |
| pabelanger | sure | 16:06 |
| jeblair | pabelanger: did you ever figure out why ze03 was stopped? | 16:07 |
| pabelanger | jeblair: no smoking gun yet | 16:07 |
| pabelanger | I can spend more time today looking | 16:08 |
| *** vhosakot has quit IRC | 16:09 | |
| fungi | any idea if zk needs a restart to pick up changes to /etc/zookeeper/conf/zoo.cfg ? | 16:10 |
| fungi | i'm noticing that the intended config options are present there now to start purging snapshots, but it hasn't purged any yet | 16:10 |
| jeblair | pabelanger: if you have time, that would be great; but if you're still working on the git timeout thing, i'd say that's priority | 16:10 |
| jeblair | fungi: i would guess it does but don't know for certain | 16:10 |
| jeblair | pabelanger: are you still looking at git timeouts? should i put your name on that? | 16:11 |
| fungi | what's the impact of a zk restart like? | 16:11 |
| pabelanger | jeblair: yes, for git timeout | 16:11 |
| *** jpena|off is now known as jpena | 16:12 | |
| jeblair | pabelanger: it sounded like the git config approach may be superior to the gitpython timeout anyway | 16:12 |
| jeblair | pabelanger: okay i put your name on that in the etherpad | 16:12 |
| pabelanger | jeblair: ya, if we do .gitconfig approach, we'll need to clone over HTTP(s) to get that support. i don't believe git protocol supported. I am trying to test it locally | 16:12 |
| jeblair | fungi: it will cause a full delete/build cycle on nodepool v3 | 16:12 |
| pabelanger | and I think we talked about doing HTTP(s) clones at PTG too | 16:13 |
| jeblair | pabelanger: oh, hrm. i think we mostly use ssh now. | 16:13 |
| *** rcernin has quit IRC | 16:13 | |
| pabelanger | yah | 16:13 |
| jeblair | pabelanger: we may need to just proceed with the gitpython option for now then, and solve as_process problem. | 16:13 |
| jeblair | pabelanger: because switching protocols is a bigger change | 16:13 |
| pabelanger | sure | 16:13 |
| *** rcernin has joined #openstack-infra | 16:13 | |
| *** psachin has quit IRC | 16:14 | |
| jeblair | fungi: not entirely sure what the builders will do | 16:14 |
| jeblair | fungi: but aside from being very disruptive, everything should handle it. :) | 16:14 |
| *** rcernin has quit IRC | 16:14 | |
| fungi | heh "aside from being very disruptive" | 16:15 |
| fungi | well, anyway we've got a fair amount of breathing room on the rootfs for nodepool.o.o since i manually expired ancient accumulated snapshots last week | 16:16 |
| fungi | so can probably wait for some point when we need to restart zk anyway for other reasons | 16:16 |
| jeblair | mordred: i'm not sure if you saw "attempt to write outside of work/ caught by bubblewrap (not ansible zuul plugins)" in the debuging section. you may be interested in that. | 16:16 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Stop doing afs prep in docs build jobs https://review.openstack.org/509804 | 16:17 |
| jeblair | fungi: okay, or we can try to do it during a zuulv3 restart | 16:17 |
| *** e0ne has quit IRC | 16:17 | |
| *** Rockyg has joined #openstack-infra | 16:18 | |
| *** Apoorva has joined #openstack-infra | 16:19 | |
| fungi | wfm. mostly just wanting to confirm the purging configuration actually does what it states on the tin, before we reach a point weeks down the road where we've forgotten about it and we fill up the disk again | 16:19 |
| jeblair | so my plan is to work on the memory and config loading tasks some more; i'm currently looking into whether we still have memory leaks (smaller ones which may have been masked by the big one). then i'd like to do a few more things to improve dynamic config loading (i think we can further halve the time we spend there in many cases). | 16:20 |
| jeblair | i will continue to help pabelanger on git timeouts as needed, and Shrews on the zk issues | 16:20 |
| jeblair | who else is working on stuff in https://etherpad.openstack.org/p/zuulv3-issues today? | 16:21 |
| fungi | i've mostly been reviewing but will look for something unclaimed i think i can make progress on and snag that | 16:22 |
| jeblair | it looks like the jobs section got a lot shorter, but still some things there | 16:22 |
| *** srobert_ has joined #openstack-infra | 16:23 | |
| fungi | i'll start by combing back over the list and seeing what's actually fixed which we haven't noticed can come off | 16:23 |
| openstackgerrit | Ricardo Noriega proposed openstack-infra/project-config master: Add new repo for Neutron L2GW Tempest Plugin https://review.openstack.org/509823 | 16:23 |
| jeblair | fungi: thanks | 16:24 |
| *** slaweq_ has quit IRC | 16:24 | |
| clarkb | I think the tripleo job issues are largely sorted out by 508660 now | 16:24 |
| clarkb | they passed last night except for one job | 16:24 |
| jeblair | clarkb: are you around today or traveling? | 16:25 |
| clarkb | Mostly traveling | 16:25 |
| jeblair | ack | 16:25 |
| clarkb | I hav to pack and then drive | 16:25 |
| jeblair | fungi, pabelanger: i'd like to have another quick meeting to review etherpad tomorrow, is this an okay time? | 16:25 |
| *** slaweq_ has joined #openstack-infra | 16:26 | |
| *** srobert has quit IRC | 16:26 | |
| fungi | jeblair: yep, wfm. there's a release team meeting 15:00-16:00 that i'l be trying to pay attention to, but after that my schedule is pretty open | 16:26 |
| dmsimard | Sorry I'm having lunch with the kids, I'll follow up on issues in 45ish mins | 16:27 |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: set neutron mtu to match system mtu https://review.openstack.org/509761 | 16:27 |
| pabelanger | jeblair: also wfm | 16:28 |
| fungi | conversation for tc thursday office hour ran over, so i'm still catching up on the issues etherpad state | 16:28 |
| jeblair | dmsimard: cool, enjoy lunch, we'll catch up later with mordred and mnaser | 16:29 |
| mordred | jeblair: yes - the attempt to write outside of work/ is on my list - and I already know why it's broken | 16:29 |
| *** tesseract has quit IRC | 16:29 | |
| mordred | jeblair: fixing it is a little bit wonkier - and I think I'd like to upgrade us to 2.4 because in 2.4 there is a facility for adding arbitrary config to ansible.cfg and having plugins and whatnot be able to read that config | 16:30 |
| chandankumar | mordred: need help on this https://review.openstack.org/#/c/508502/ to make it passing. | 16:31 |
| mordred | jeblair: the issue is that we're checking the validity of the path relative to os.curdir ... | 16:31 |
| *** ralonsoh has quit IRC | 16:31 | |
| Shrews | mordred: we cannot use 2.4. we need >=2 .4.1 | 16:31 |
| mordred | Shrews: yah- that's what I really meant | 16:31 |
| Shrews | >= 2.4.1 | 16:31 |
| mordred | jeblair: but os.curdir is the directory in-which the playbook resides | 16:31 |
| jeblair | mordred: yeah, that sounds like it matches | 16:31 |
| mordred | jeblair: so we need to be able to communicate to our paths filtering what the workdir is and we cannot rely on curdir to do that | 16:32 |
| *** jascott1 has quit IRC | 16:32 | |
| *** jascott1 has joined #openstack-infra | 16:32 | |
| mordred | jeblair: doing that with >= 2.4.1 will be straightforward as we can add the workdir to ansible.cfg when we write it out, and then read that value in our paths path matcher | 16:32 |
| mordred | Shrews: do we have a timeframe for 2.4.1 release? | 16:32 |
| Shrews | mordred: i don't | 16:33 |
| *** gouthamr_ has joined #openstack-infra | 16:33 | |
| jeblair | mordred: we could env-var until then? | 16:33 |
| rajinir | what does this mean? There is a duplicated hook in your elements: /install.d/40-install-bindep | 16:35 |
| mordred | jeblair: yah - we could envvar - in fact, DUH - how about I do that real quick | 16:35 |
| *** gouthamr has quit IRC | 16:35 | |
| rajinir | diskimagebuilder is throwing errors, new setup and the seeing all kinds of issues. Need help | 16:35 |
| jeblair | mordred: it looks like the requirements fixes are moving, but not landed yet, is that right? are we just waiting for those to land at this point? | 16:37 |
| *** Swami has quit IRC | 16:37 | |
| mordred | jeblair: yes- that's right | 16:37 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Put doc build jobs into doc publication project-template https://review.openstack.org/509805 | 16:37 |
| clarkb | rajinir: it means two elements provide that same file. You'll have to find the duplicate and rename or remove it | 16:38 |
| rajinir | clarkb: ok, thanks, will see | 16:38 |
| jeblair | jlk: what's the status of translation jobs? get a +1 from zuul on 502208 then reviews? or more to it than that? | 16:39 |
| *** baoli has quit IRC | 16:39 | |
| *** trown is now known as trown|lunch | 16:40 | |
| jeblair | mordred: it looks like zuul is still grumpy about one of your docs patches -- 509808 | 16:40 |
| jeblair | okay, i'm going to go back to memory debugging now, feel free to ping me for further etherpad discussion | 16:41 |
| AJaeger | jeblair, I'm back online again, will read backlog... | 16:42 |
| *** baoli has joined #openstack-infra | 16:44 | |
| *** spzala has quit IRC | 16:44 | |
| mordred | jeblair: oh - piddle. thanks | 16:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Use normal docs build jobs https://review.openstack.org/509833 | 16:49 |
| mordred | jeblair: ^^ | 16:49 |
| *** udesale has quit IRC | 16:49 | |
| openstackgerrit | Merged openstack-infra/irc-meetings master: Changing Sahara meeting time https://review.openstack.org/509796 | 16:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Remove build-openstack-sphinx-docs-infra https://review.openstack.org/509808 | 16:50 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add release-openstack-sphinx-docs-infra template https://review.openstack.org/509809 | 16:50 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add project-template for python jobs without constraints https://review.openstack.org/509815 | 16:50 |
| mordred | jeblair: and added a depends-on | 16:50 |
| *** bhavik1 has quit IRC | 16:50 | |
| *** tesseract has joined #openstack-infra | 16:50 | |
| mordred | pabelanger, clarkb, fungi ^^ | 16:50 |
| *** derekh has quit IRC | 16:51 | |
| inc0 | hey, reading scrollback - someone tried to ddos us? | 16:52 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Add translation jobs https://review.openstack.org/502208 | 16:54 |
| AJaeger | jlk, jeblair, I rebased the translation change and resolved conflicts so that we can review ^ | 16:54 |
| mnaser | jeblair: the release note jobs are fixed and all merged (thanks to mordred and AJaeger for their reviews). we could move it to fixed issues. the publish jobs for release notes is not something i can test unfortunately but i believe it should work | 16:55 |
| mnaser | i'm not sure *how* to test release note publish jobs but the ones to build work well | 16:56 |
| AJaeger | mordred, do you want to test releasenotes publishing with shade? ^ | 16:56 |
| *** csomerville has joined #openstack-infra | 16:57 | |
| *** slaweq_ has quit IRC | 16:58 | |
| AJaeger | mordred: you abandoned https://review.openstack.org/#/c/508898/ - what happens with the playbooks/legacy/requirements? And the check-requirements template? I think we still need parts of that change. Are you working on it or shall I? | 16:58 |
| fungi | inc0: no idea whether it was an intentional ddos, but several requests a second for a kolla centos image from a couple thousand different ip addresses registered to chniese internet service providers | 16:58 |
| *** cody-somerville has quit IRC | 16:59 | |
| inc0 | duh | 16:59 |
| fungi | inc0: it started just prior to 14:00 utc and there were several spikes over the course of the hour | 16:59 |
| fungi | http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=311&rra_id=all | 16:59 |
| mordred | AJaeger: ah - good point - I just need to reparent it don't I? | 17:00 |
| *** jamesmcarthur has joined #openstack-infra | 17:00 | |
| fungi | inc0: with the ipv4 resale market in china being what it is these days, it's hard to actually know whether they were all originating from the same organization or not | 17:00 |
| mordred | AJaeger, mnaser I'm working on the publish-releasenotes job right now ... and now I remember why we moved releastenotes/ to doc/ in the original version ... | 17:00 |
| mordred | AJaeger: if you have a sec and want to take that over, that's cool by me - or I can get it as soon as I'm done with this one | 17:01 |
| AJaeger | mordred: I can do it - currently will document project migration and then go for it | 17:02 |
| *** srobert_ has quit IRC | 17:02 | |
| *** srobert has joined #openstack-infra | 17:03 | |
| mordred | pabelanger: I'm looking at prepare-docs-for-afs - and we seem to be doing the rearranging of directories twice | 17:03 |
| mordred | pabelanger: once in the ansible-ified section, and then in a shell task that looks like the old version from before we ansible-ified the other bits | 17:04 |
| mordred | pabelanger: do you know if we did that for a reason? or did we jus mess up a merge somewhere? | 17:04 |
| mordred | AJaeger: (or do you happen to know?) | 17:04 |
| *** ihrachys has quit IRC | 17:07 | |
| *** slaweq_ has joined #openstack-infra | 17:07 | |
| *** ijw has joined #openstack-infra | 17:07 | |
| pabelanger | mordred: a lot of that we legacy stuff, where it was moving directories multiple times. I think now we can simplify it a lot based on the final layout | 17:07 |
| *** ihrachys has joined #openstack-infra | 17:07 | |
| pabelanger | but AJaeger might know more | 17:07 |
| mordred | pabelanger: ok. I'll leave that to a follow up step | 17:07 |
| *** iyamahat has quit IRC | 17:08 | |
| *** Swami has joined #openstack-infra | 17:09 | |
| *** srobert has quit IRC | 17:09 | |
| *** jamesmcarthur has quit IRC | 17:09 | |
| AJaeger | that's tricky code, don't remember the details - hope there's history to check ;( | 17:10 |
| *** srobert has joined #openstack-infra | 17:10 | |
| AJaeger | fungi did some of that as well, he might remember ^ | 17:10 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/infra-manual master: Document which jobs should *not* get migrated https://review.openstack.org/509839 | 17:11 |
| AJaeger | mordred, this is based on your comments in https://review.openstack.org/#/c/509746 ^ | 17:11 |
| *** lucasagomes is now known as lucas-afk | 17:11 | |
| *** yamamoto has joined #openstack-infra | 17:11 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Allow prepare-docs-for-afs to be used with other dirs https://review.openstack.org/509840 | 17:13 |
| fungi | AJaeger: mordred: i believe it was because jenkins scp publisher didn't allow us to strip leading paths, so we needed to move all the content to the root of the workspace if we wanted to publish to the root of the target, and that meant temporarily putting the content somewhere identifiable while we blew away all other files in the workspace | 17:14 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Add the run playbook for releasenotes and fix path https://review.openstack.org/509843 | 17:16 |
| mordred | mnaser, fungi, AJaeger, jeblair: ^^ that and https://review.openstack.org/509840 should finish off releasenotes | 17:16 |
| fungi | so we copied from doc/build/html/... to some path starting with a "," on the assumption that nothing else in the workspace would start with a comma, then deleted everything not starting with a comma (so we could also get rid of dotfiles) and then moved the content from the temporary path to the root of the workspace and explicitly removed the tempdir | 17:16 |
| fungi | i think that's the odd dance you're asking about, at any rate | 17:17 |
| fungi | basically, it's something we ought to be able to do in a much more straightforward way with rsync | 17:17 |
| AJaeger | fungi: looking at the code - yes we did that but not for docs publishing in current revision | 17:18 |
| *** jascott1 has quit IRC | 17:18 | |
| AJaeger | what we do with the moves nowaday, is move to the right location, so that we publish either to docs.o.o/REPO/latest or REPO/pike or repo/TAG | 17:19 |
| AJaeger | and that's what we need to ensure still works | 17:19 |
| fungi | oh, well i was talking about infra-specific publishing jobs, not general docs jobs | 17:19 |
| *** ramishra has quit IRC | 17:19 | |
| fungi | i don't recall working on any multiple reshuffling in a regular docs job | 17:19 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Replace legacy-requirements with new requirements-check https://review.openstack.org/508898 | 17:21 |
| AJaeger | fungi, ah - I was talking about the generic stuff ;) | 17:21 |
| *** ramishra has joined #openstack-infra | 17:21 | |
| fungi | no recollection. i don't think i worked on that | 17:21 |
| AJaeger | fungi, that code seems to be gone by now, I don't see it anymore ;) | 17:21 |
| *** panda is now known as panda|off | 17:22 | |
| *** tesseract has quit IRC | 17:22 | |
| mordred | AJaeger: left a comment on https://review.openstack.org/#/c/509839/1 about why it failed | 17:22 |
| mordred | AJaeger: maybe we shold define that project-template in the requirements repo too .... | 17:23 |
| *** SumitNaiksatam has quit IRC | 17:24 | |
| *** iyamahat has joined #openstack-infra | 17:25 | |
| AJaeger | mordred: I'm currently updating project-config and will propose that next. | 17:25 |
| mordred | AJaeger: remote: https://review.openstack.org/509845 Add check-requirements project-template | 17:26 |
| mordred | fungi, clarkb, dhellmann, dims: ^^ quick +A perhaps? | 17:26 |
| mordred | AJaeger: I can respin https://review.openstack.org/508898 real quick ... | 17:26 |
| *** markmcclain has quit IRC | 17:26 | |
| AJaeger | mordred: will that work? You define it in two places... | 17:26 |
| AJaeger | mordred: I'll clean up... | 17:27 |
| AJaeger | mordred: we should add files section to 508898... | 17:27 |
| mnaser | mordred: https://review.openstack.org/#/c/509843/1/playbooks/publish/releasenotes.yaml merged but https://review.openstack.org/#/c/509680/1/playbooks/publish/releasenotes.yaml does some of the same, will that cause a git conflict or itll be okay? | 17:27 |
| mnaser | other htan that it seems good for me :> | 17:27 |
| AJaeger | mordred: I'll update 508898... | 17:28 |
| mordred | AJaeger: lemme fix 509845 real quick with files | 17:28 |
| AJaeger | mordred: go ahead... | 17:28 |
| * AJaeger waits - and fixes project-config | 17:28 | |
| *** markmcclain has joined #openstack-infra | 17:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 17:29 |
| mordred | AJaeger, dims, prometheanfire, fungi: https://review.openstack.org/509845 is good now | 17:30 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/openstack-zuul-jobs master: Require requirements for legacy-bandit-integration https://review.openstack.org/509846 | 17:30 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Replace legacy-requirements with new requirements-check https://review.openstack.org/508898 | 17:30 |
| *** dtantsur is now known as dtantsur|afk | 17:32 | |
| *** jpena is now known as jpena|off | 17:32 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Add the run playbook for releasenotes and fix path https://review.openstack.org/509843 | 17:32 |
| mordred | mnaser: I rebased it on master ^^ to make sure the merge worked right | 17:33 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use requirements-check everywhere https://review.openstack.org/509847 | 17:34 |
| AJaeger | mordred: project-config side ^ | 17:34 |
| *** e0ne has joined #openstack-infra | 17:35 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/infra-manual master: Document which jobs should *not* get migrated https://review.openstack.org/509839 | 17:36 |
| *** trown|lunch is now known as trown | 17:36 | |
| *** lbragstad has quit IRC | 17:37 | |
| *** tosky has quit IRC | 17:38 | |
| AJaeger | mordred: I think we cannot move requirements-check template from one repo to another - or define in both as we do now. So, we might need to do: Create new requirements-check-new template, use that in project-config, remove requirements-check from ozr and add to requirements, and then use it in project-config | 17:38 |
| fungi | mordred: regarding 509845 i think we want to make sure zuulv3 gives it a +1 before we approve, right? otherwise we risk breaking things on the v3 side? | 17:38 |
| AJaeger | fungi, mordred, please ^ | 17:39 |
| AJaeger | fungi, see my comment above - I doubt 509845 will work as is | 17:39 |
| fungi | AJaeger: i agree | 17:39 |
| fungi | let's get the removal merged first | 17:40 |
| *** slaweq_ has quit IRC | 17:40 | |
| fungi | or do the rename dance you describe | 17:40 |
| AJaeger | let's see what mordred things - and then coordinate who does it... | 17:40 |
| dmsimard | I don't know who pasted review links on the zuulv3-issues etherpad but this format doesn't work for me: https://review.openstack.org/#/c/508850/ | 17:41 |
| dmsimard | the "/#/c/id/" | 17:41 |
| fungi | given v3 isn't gating projects running that job right now, we could remove all uses of the template, add it to the requirements repo, then revert the change which stopped it being used | 17:41 |
| AJaeger | dmsimard: I did - works fine for me | 17:42 |
| AJaeger | fungi, yeah, that works as well... | 17:42 |
| *** e0ne has quit IRC | 17:42 | |
| mnaser | mordred publishing stuff looks good to me :> | 17:43 |
| *** slaweq_ has joined #openstack-infra | 17:43 | |
| fungi | dmsimard: if you go to https://review.openstack.org/508850 it will redirect to https://review.openstack.org/#/c/508850/ so if that doesn't work something's really odd | 17:43 |
| *** yamahata has joined #openstack-infra | 17:43 | |
| dmsimard | hmm, weird, I had a proxy error earlier | 17:44 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 17:45 |
| *** bramleya has quit IRC | 17:46 | |
| fungi | dmsimard: that could just be "gerrit fell over momentarily" rather than a bad url format. wouldn't surprise me in the least | 17:46 |
| AJaeger | fungi, I think we can merge 509847 to make the diff smaller, then do that dance | 17:46 |
| fungi | wfm | 17:47 |
| dmsimard | fungi: yeah, I was reviewing a few links back to back with that format and none of them worked and made a wrong assumption :) | 17:47 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Add translation jobs https://review.openstack.org/502208 | 17:48 |
| AJaeger | fungi, could you +2 https://review.openstack.org/509847 then, please? Anybody else around for review? | 17:49 |
| fungi | yeah, i was just looking at it | 17:49 |
| AJaeger | great | 17:49 |
| mordred | AJaeger: so - I think we haven't landed the patch adding the projet-template yet - so I just removed it from the patch that defines it in ozj | 17:50 |
| fungi | AJaeger: 509847 is actually adding use of the check-requirements project-template. isn't that making it harder to move? | 17:50 |
| mordred | AJaeger: otherwise you are quite correct - we can't double-define it | 17:50 |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: DNM testing 509841 https://review.openstack.org/509853 | 17:50 |
| fungi | it doesn't look like we're using the check-requirements project-template yet until 509847 | 17:50 |
| *** SumitNaiksatam has joined #openstack-infra | 17:51 | |
| AJaeger | fungi, it was used already before | 17:52 |
| mordred | AJaeger, fungi: I listed the patches: https://etherpad.openstack.org/p/zuulv3-issues line 83 | 17:53 |
| AJaeger | fungi, http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/projects.yaml#n2051 | 17:53 |
| mordred | fungi, AJaeger in the order they need to land | 17:53 |
| mordred | AJaeger: oh - wait - it was? where does it exist? | 17:53 |
| AJaeger | mordred: let me update the list | 17:54 |
| mordred | AJaeger: OH PIDDLE | 17:54 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Disable check-requirements template https://review.openstack.org/509854 | 17:54 |
| mordred | we defined it in legacy ... I understand now | 17:54 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Revert "Disable check-requirements template" https://review.openstack.org/509855 | 17:54 |
| mordred | AJaeger: YES | 17:55 |
| mordred | AJaeger: yes. you are 100% correct - thank you :) | 17:55 |
| AJaeger | ;) | 17:56 |
| AJaeger | then let's go ahead and merge 509847 | 17:56 |
| mordred | yes - fungi you wanna +A that one? | 17:57 |
| *** hamzy has quit IRC | 17:57 | |
| *** claudiub has quit IRC | 17:57 | |
| fungi | yup | 17:57 |
| AJaeger | jeblair, fungi, clarkb, pabelanger, mordred, could you review my infra-manual change - since that's a policy update for zuulv3 migration based on a comment that mordred did in https://review.openstack.org/#/c/509746 | 17:58 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Remove legacy-requirements jobs https://review.openstack.org/508898 | 17:58 |
| fungi | AJaeger: will do | 17:58 |
| mordred | AJaeger, fungi: ^^ I updated the depends-on there so that we can land it in the righ tsequence | 17:59 |
| AJaeger | jlk, jeblair, the translation job is passing now - see https://review.openstack.org/502208 | 17:59 |
| AJaeger | mordred: thanks! | 17:59 |
| mordred | AJaeger: gah. I have a weird nitpick on the infra-manual change ... | 18:00 |
| AJaeger | mordred, fungi, if the two of you +2 the changes any of us could +A at the right time... | 18:00 |
| AJaeger | mordred: happy to fix ;) | 18:00 |
| *** SumitNaiksatam has quit IRC | 18:00 | |
| *** tosky has joined #openstack-infra | 18:01 | |
| mordred | AJaeger: I have this up: https://review.openstack.org/#/c/508692/ to governance to change Consistent Testing Interface to Project Testing Interface (since that's how we talk about it and what the file is name) | 18:01 |
| *** SumitNaiksatam has joined #openstack-infra | 18:01 | |
| AJaeger | mordred: I always considered PTI = python testing interface ;( | 18:02 |
| * AJaeger updates | 18:02 | |
| mordred | AJaeger: many problems result in having two overlapping names :( | 18:02 |
| mordred | AJaeger: one more comment ... | 18:02 |
| AJaeger | yeah ;( | 18:02 |
| AJaeger | ok, waiting... | 18:03 |
| mordred | AJaeger: before you respin - could you also add a sentence or something indicating that the guidance about PTI jobs being central is for *official* openstack projects | 18:03 |
| mordred | AJaeger: non-official projects can do whatever they want- other than leaving merge-check in project-config | 18:03 |
| numans | AJaeger, Hi. If in case there is no opinion on this patch https://review.openstack.org/#/c/502899/, is it possible to consider the present state of the patch. We need this patch to unblock another patch in tripleo related to OVN. thanks | 18:03 |
| fungi | if it helps (it probably doesn't) i try to refer to it as the cti because the document title is consistent testing interface | 18:04 |
| jlk | AJaeger: nice! I wish there was a way to test the job itself. But I think we have to merge it and try to get something to use it | 18:04 |
| AJaeger | jlk: I was wrong - zuul runs in post_failure;misread jenkins+1 ;( | 18:04 |
| mordred | fungi: I notice you have not voted on https://review.openstack.org/#/c/508692 | 18:05 |
| jlk | ah | 18:05 |
| AJaeger | mordred: could you help numans, please? | 18:05 |
| fungi | mordred: indeed, i'm still trying to catch up on older governance reviews | 18:06 |
| fungi | trying to get through the masakari project addition proposal next when i juggle back to governance reviewing | 18:06 |
| openstackgerrit | Merged openstack-infra/project-config master: Use requirements-check everywhere https://review.openstack.org/509847 | 18:07 |
| AJaeger | fungi, do you want to +2A https://review.openstack.org/#/c/509854 , please? Next requirements check change | 18:08 |
| *** lbragstad has joined #openstack-infra | 18:10 | |
| AJaeger | mordred: looking at https://review.openstack.org/#/c/509840/1/roles/prepare-docs-for-afs/tasks/main.yaml - for releasenotes, we do not want those directory changes | 18:13 |
| *** slaweq_ has quit IRC | 18:16 | |
| *** ijw has quit IRC | 18:16 | |
| *** e0ne has joined #openstack-infra | 18:18 | |
| *** electrofelix has quit IRC | 18:20 | |
| *** slaweq_ has joined #openstack-infra | 18:20 | |
| AJaeger | jlk: now the translatin change passed ;) | 18:21 |
| *** e0ne has quit IRC | 18:23 | |
| *** yamamoto has quit IRC | 18:24 | |
| AJaeger | numans: such a change needs also tripleo liaison +1 | 18:24 |
| openstackgerrit | Merged openstack-infra/project-config master: Disable check-requirements template https://review.openstack.org/509854 | 18:25 |
| dmsimard | jeblair: okay, I'm caught up with zuulv3-issues and I also took the liberty of triaging open/merged reviews | 18:26 |
| dmsimard | I'm not currently working on anything specific, if not waiting for an executor reload to test elastic-recheck v3 things https://review.openstack.org/#/c/509313/ | 18:27 |
| *** yamamoto has joined #openstack-infra | 18:27 | |
| dmsimard | I'll go ahead and add that to the pad | 18:27 |
| jeblair | dmsimard: cool; let's get pabelanger's git timeout patch landed and then restart the executors for both | 18:27 |
| mordred | AJaeger: which directory changes? | 18:29 |
| clarkb | have the console stream issues been looked at? | 18:29 |
| jeblair | clarkb: not that i'm aware of | 18:30 |
| jeblair | though i did restart zuul-web the other day. somethig to keep in mind if situation has changed. | 18:30 |
| mordred | yah - I'd like to collect more info on that one - there were a bunch of other things in flux | 18:31 |
| AJaeger | mordred: commented on https://review.openstack.org/#/c/509840/ - that playbook cannot be used for releasenotes | 18:31 |
| mordred | AJaeger: AH - do we need to do prepare-docs-for-afs for releasenotes at all? | 18:32 |
| mordred | AJaeger: or do we just need to write the stamp file? | 18:32 |
| *** nikhil has quit IRC | 18:32 | |
| AJaeger | mordred: just write the stamp file | 18:32 |
| *** yamamoto has quit IRC | 18:32 | |
| AJaeger | mordred: and releasenotes always get published from *master* branch | 18:32 |
| AJaeger | even if you commit on stable | 18:32 |
| mordred | kk. coming right up - I'll abandon that patch and just fix the publish job | 18:32 |
| *** sambetts|afk has quit IRC | 18:33 | |
| mordred | jeblair: ^^ releasenotes always getting published from master branch even for a commit to stable ... does override-branch help us here? | 18:33 |
| fungi | right, if memory serves reno considers the master branch content primary, but inspects all stable branches for additional notes missing from master | 18:33 |
| fungi | so the job doesn't care what branch triggered the job, and starts from master regardless | 18:34 |
| AJaeger | argh, mistral uses check-requirements ;((( | 18:34 |
| fungi | AJaeger: in their repo? | 18:34 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Handle double node locking snafu https://review.openstack.org/509603 | 18:34 |
| AJaeger | yes | 18:34 |
| jeblair | looking | 18:35 |
| AJaeger | working on a patch | 18:35 |
| fungi | check-requirements addition for official deliverables should likely remain in project-config | 18:35 |
| *** sambetts_ has joined #openstack-infra | 18:36 | |
| mordred | jeblair: we could also just add "git checkout master" to the reno role | 18:36 |
| AJaeger | fungi, yes - see 509839. | 18:36 |
| fungi | AJaeger: yep, i was just confirming that it was in there | 18:37 |
| AJaeger | https://review.openstack.org/509869 is the change | 18:37 |
| jeblair | AJaeger: are you aware of https://docs.openstack.org/infra/zuul/feature/zuulv3/user/config.html#attr-job.allowed-projects ? | 18:37 |
| AJaeger | jeblair: I wasn't. but it wouldn't help since I move a *template*, not a job | 18:38 |
| jeblair | mordred: i have a mental block on the reno question... | 18:38 |
| jeblair | AJaeger: the template includes a job, right? | 18:38 |
| AJaeger | but double checking: This means, I could blacklist a repo using this job | 18:38 |
| mordred | jeblair: its ok - I think we don't need anything specific | 18:38 |
| *** ijw has joined #openstack-infra | 18:39 | |
| AJaeger | jeblair: yes, it includes a job - zuul complaints aobut the template in https://review.openstack.org/#/c/508898/ | 18:39 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/infra-manual master: Document which jobs should *not* get migrated https://review.openstack.org/509839 | 18:39 |
| jeblair | mordred: i looked at that change, but i need more background. i guess we can a) tell me to fugetaboutit b) ask me a really specific question or c) give me the whole story :) | 18:40 |
| AJaeger | thanks, fungi for fixing my typo | 18:40 |
| fungi | np | 18:40 |
| openstackgerrit | Paul Belanger proposed openstack-infra/system-config master: zuulv3: add .gitconfig support to zuul-executors / zuul-mergers https://review.openstack.org/509871 | 18:41 |
| fungi | thanks for writing a much-needed addition to that document | 18:41 |
| jeblair | AJaeger: oh, you're doing a repo move; i thought you were trying to prevent projects from using a template | 18:41 |
| *** ijw has quit IRC | 18:41 | |
| AJaeger | jeblair: I'm doing a move of one template from openstack-zuul-jobs to requirements | 18:41 |
| pabelanger | jeblair: mordred: fungi: clarkb: ^ adds .gitconfig files to zuul-executors / zuul-mergers for zuulv3 to try and fix git timeouts | 18:41 |
| *** hamzy has joined #openstack-infra | 18:41 | |
| jeblair | pabelanger: this should be a zuul change i think | 18:42 |
| pabelanger | jeblair: kk, add them as zuul.conf setting? | 18:42 |
| AJaeger | jeblair: and as part of that move, I need to disable usage of the template. | 18:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Always build releasenotes from master https://review.openstack.org/509872 | 18:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Add the run playbook for releasenotes and fix afs stamping https://review.openstack.org/509843 | 18:43 |
| mordred | AJaeger: ^^ ok, I think that does it | 18:43 |
| jeblair | pabelanger: or hardcode it (for now at least) | 18:43 |
| pabelanger | okay, I think we can use the env we pass to GitPython, let me test quickly | 18:43 |
| AJaeger | argh, no mistral-core around . rakhmerov, really offline? | 18:43 |
| *** chlong_ has quit IRC | 18:44 | |
| mordred | AJaeger: I'm going to vote that it's ok for us to merge that change for them | 18:44 |
| AJaeger | mordred: +1 | 18:44 |
| AJaeger | wait, they did this in several repos, will find them all ;) | 18:45 |
| AJaeger | https://review.openstack.org/509869 is mistral | 18:45 |
| mordred | fungi, clarkb, jeblair: ^^ I'm going to merge the zuulv3 job removal on the mistral repo(s) unless anyone screams in objection | 18:45 |
| AJaeger | let's tell rakhmerov ^ | 18:46 |
| rajinir | clarkeb: Was able to build the dib image, but I see this error with the new image "VFS: Cannot open root device "LABEL=cloudimg-rootfs" or unknown-block(0,0): error -6" | 18:46 |
| mordred | AJaeger, rakhmerov: ++ | 18:46 |
| mordred | also - mistral has the old broken versions of tarball/pypi - so this will fix them in more than one way | 18:47 |
| AJaeger | mordred: https://review.openstack.org/#/c/509873/ | 18:47 |
| mordred | AJaeger: ok. I have merged both | 18:49 |
| *** jascott1 has joined #openstack-infra | 18:50 | |
| AJaeger | mordred: https://review.openstack.org/#/c/509874/ | 18:50 |
| AJaeger | mordred: and https://review.openstack.org/#/c/509875/ | 18:51 |
| AJaeger | mordred: that's all... I used codesearch | 18:52 |
| mordred | woot | 18:52 |
| mordred | AJaeger: all three merged | 18:52 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add git timeout for HTTP(S) operations https://review.openstack.org/509876 | 18:53 |
| AJaeger | thanks, mordred | 18:53 |
| fungi | let's hope they didn't backport it to their stable branches (which won't show up in codesearch) | 18:53 |
| rajinir | Was able to build the dib image, but I see this error with the new image "VFS: Cannot open root device "LABEL=cloudimg-rootfs" or unknown-block(0,0): error -6" | 18:53 |
| *** slaweq_ has quit IRC | 18:53 | |
| AJaeger | mordred: https://review.openstack.org/#/c/509875/ is missing... | 18:53 |
| mordred | AJaeger: done | 18:53 |
| AJaeger | thanks! | 18:54 |
| AJaeger | Now rechecking the removal change to see whether they backported it or whether we missed anything else... | 18:54 |
| mordred | AJaeger: it's nice that zuul will tell us that :) | 18:54 |
| openstackgerrit | Merged openstack-infra/infra-manual master: Document which jobs should *not* get migrated https://review.openstack.org/509839 | 18:55 |
| fungi | rajinir: looks similar to the report in bug 1661292 (did you look into any of the things mentioned there already?) | 18:56 |
| openstack | bug 1661292 in OpenStack Compute (nova) "VFS: Cannot open root device "LABEL=cloudimg-rootfs" or unknown-block(0,0): error -6" [Undecided,Expired] https://launchpad.net/bugs/1661292 | 18:56 |
| *** slaweq_ has joined #openstack-infra | 18:56 | |
| AJaeger | mordred: yes, indeed | 18:57 |
| rajinir | fungi: I looked at it, but nothing useful to solve the issue | 18:58 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 18:58 |
| rajinir | fungi: the yaml to build the image is simple | 18:59 |
| rajinir | https://www.irccloud.com/pastebin/FU6y3XxS/nodepoolyaml | 18:59 |
| *** chlong_ has joined #openstack-infra | 19:00 | |
| fungi | rajinir: also https://stackoverflow.com/questions/46327787 and https://www.queryxchange.com/q/1_722761/google-compute-engine-not-booting-up-properly-unable-to-ssh/ look similar | 19:00 |
| fungi | ahh, actually not so much the latter one | 19:01 |
| *** baoli has quit IRC | 19:02 | |
| *** vhosakot has joined #openstack-infra | 19:02 | |
| rajinir | fungi, ya, nothing that will solve my issue. This is a brand new image built today | 19:02 |
| *** baoli has joined #openstack-infra | 19:02 | |
| fungi | rajinir: seems like an issue with the resulting image being built... you might consider asking in #openstack-dib | 19:02 |
| rajinir | fungi, sure thanks | 19:03 |
| fungi | issues finding the rootfs at boot is unlikely to be specific to our custom elements, and is probably a deeper problem | 19:03 |
| rajinir | fungi :( ok | 19:05 |
| fungi | rajinir: you should probably first test that you can create and boot a very simple, generic image of some kind with dib | 19:06 |
| fungi | just to rule out anything higher up | 19:06 |
| rajinir | ok | 19:06 |
| *** MasterOfBugs has joined #openstack-infra | 19:11 | |
| *** Rockyg has quit IRC | 19:12 | |
| AJaeger | mordred: http://logs.openstack.org/10/103aaff48e334faa5627052224837de6f572bfde/infra-post/publish-infra-index/d1cc482/ara/ - that fails, still in fetch-tox-output | 19:14 |
| AJaeger | mordred: and this one as well - http://logs.openstack.org/10/103aaff48e334faa5627052224837de6f572bfde/infra-post/publish-openstack-specs-site/5feae39/ara/ - the venv fix did not help | 19:14 |
| *** markvoelker_ has joined #openstack-infra | 19:15 | |
| *** markvoelker has quit IRC | 19:15 | |
| *** armaan has quit IRC | 19:16 | |
| *** armaan has joined #openstack-infra | 19:17 | |
| eumel8 | repo names with dot - is this a known issue: http://logs.openstack.org/20/509520/2/check/legacy-tempest-dsvm-neutron-src/9dd44fe/job-output.txt.gz#_2017-10-05_11_39_23_351438 | 19:18 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Fetch logs from infra-docs, not venv https://review.openstack.org/509882 | 19:19 |
| mordred | AJaeger: ^^ that should fix the infra-index one | 19:19 |
| *** slaweq_ has quit IRC | 19:22 | |
| *** slaweq_ has joined #openstack-infra | 19:23 | |
| fungi | eumel8: looks like the arithmetic was added when https://review.openstack.org/476440 merged in june | 19:23 |
| fungi | eumel8: did you only just start encountering that failure on oslo.versionedobjects? | 19:24 |
| fungi | is it not happening in the v2 jobs? | 19:25 |
| eumel8 | saw this for the first time | 19:26 |
| fungi | might want to set -x in devstack-vm-gate-wrap.sh and then depends-on that from a dummy change ni the project where that job is running | 19:26 |
| fungi | to see what the actual expansion of that line is in the script | 19:26 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Update specs-site and infra-index publication jobs https://review.openstack.org/509882 | 19:27 |
| mordred | AJaeger, fungi: ^^ actually just reworked the previous patch to handle both things | 19:27 |
| fungi | eumel8: i have confirmed that if $DEVSTACK_PROJECT_FROM_GIT is a string including a "." character then bash will raise that error inside a [[ ]] expression | 19:29 |
| fungi | eumel8: i believe the expression there should be using = instead of -eq (the latter is intended for arithmetic comparisons) | 19:30 |
| fungi | eumel8: i'll push up a fix, but i _am_ curious how this has been hiding in devstack-gate for over 3 months undetected | 19:30 |
| AJaeger | thanks, mordred | 19:31 |
| AJaeger | yeah, infra-manual publishing works - https://docs.openstack.org/infra/manual/zuulv3.html#what-not-to-convert is online... | 19:31 |
| eumel8 | fungi: sorry, this error caused not the failure. it's also in previous builds | 19:33 |
| eumel8 | but I have a new one: http://logs.openstack.org/20/509520/2/check/legacy-tempest-dsvm-neutron-src/9dd44fe/job-output.txt.gz#_2017-10-05_12_29_18_222234 | 19:33 |
| fungi | eumel8: thanks for pointing it out anyway! | 19:33 |
| fungi | i almost have the fix for it pushed | 19:33 |
| eumel8 | it seems volume build was not sucessful | 19:33 |
| eumel8 | fungi: you are too fast ;) | 19:34 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/devstack-gate master: Use = for string comparisons https://review.openstack.org/509885 | 19:36 |
| fungi | eumel8: that's the fix for the mostly-benign error ^ | 19:36 |
| fungi | looking at your next error now | 19:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Align other tox-based project-config publish jobs https://review.openstack.org/509886 | 19:38 |
| mordred | AJaeger: there's a follow up wich does the same thing for the other similar jobs | 19:38 |
| *** dhajare has quit IRC | 19:39 | |
| fungi | eumel8: your new error is probably more appropriate for #openstack-qa (likely nondeterminism of some sort) | 19:39 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Update publish-service-types to work like other jobs https://review.openstack.org/509887 | 19:40 |
| eumel8 | fungi: ok, thx, will looking there | 19:41 |
| mtreinish | eumel8: http://logs.openstack.org/20/509520/2/check/legacy-tempest-dsvm-neutron-src/9dd44fe/logs/screen-c-vol.txt.gz?level=TRACE#_Oct_05_12_24_34_708115 is the error you're hitting there | 19:42 |
| AJaeger | mordred: looking at https://review.openstack.org/#/c/509809/4/zuul.d/project-templates.yaml - for openstack docs, we always publish nowadays, not only on release. Do you really want to handle infra projects differently - or should we switch over to always publish docs? | 19:43 |
| *** chlong_ has quit IRC | 19:44 | |
| mordred | AJaeger: there are 5 projects that were set up to only publish on releases | 19:45 |
| AJaeger | mordred: do we want to keep it - or change it? | 19:46 |
| AJaeger | mordred: you entered recheck at the same time Zuul gave a +1 on https://review.openstack.org/#/c/508898/ | 19:46 |
| mordred | AJaeger: BAH I suck | 19:46 |
| AJaeger | fungi, do you want to +2A https://review.openstack.org/#/c/508898/ ? | 19:46 |
| mordred | AJaeger: bindep, git-restack, git-review, python-storyboardclient and storyboard | 19:48 |
| AJaeger | mordred: so, should we keep status quo - and discus later? | 19:48 |
| mordred | AJaeger: I think bindep, git-review and git-restack are likley better off only publishing on release (we sometimes go a LONG time between git-review releases) | 19:48 |
| *** baoli has quit IRC | 19:48 | |
| mordred | AJaeger: I think we could certainly shift storyboard and python-storyboardclient to publishing on every commit | 19:48 |
| mordred | AJaeger: https://review.openstack.org/#/c/509820 is the consumption of that- I can do a followup real quick that switches storyboard | 19:49 |
| fungi | in this case there are 2 years of unreleased changes on git-review master (and we don't generally encourage people to run unreleased versions of git-review) | 19:49 |
| AJaeger | mordred: +2A for the new templates | 19:49 |
| *** baoli has joined #openstack-infra | 19:49 | |
| *** hasharAway is now known as hashar | 19:50 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Publish docs for storyboard in post https://review.openstack.org/509888 | 19:51 |
| mordred | AJaeger: there's the stryboard followup | 19:51 |
| mordred | fungi: yah | 19:51 |
| mordred | fungi, pabelanger: feel like a +A on https://review.openstack.org/#/c/509808/ ? | 19:52 |
| pabelanger | +3 | 19:53 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Add openstack/puppet-openstack-cookiecutter to puppet jobs https://review.openstack.org/509802 | 19:53 |
| mordred | jeblair: in the v2 status page, when you filter on a project, the change queues are only printed for the projects matching the filter - but in v3 we see all of the change queues regardless of whether the project matches the filter ... I'm assuming we'd like it work like v2 did yeah? | 19:54 |
| fungi | i noticed the same, but considered it was likely a lowish priority for now. still would be good to track | 19:55 |
| * mordred is hoping that mnaser will become annoyed by the disparity and use his magical javascript prowess to fix it | 19:55 | |
| *** srobert has quit IRC | 19:57 | |
| fungi | mordred: actually, it's only our custom status page where it's working as expected; v2's built-in status frontend has the same defect so this isn't really a zuul regression | 20:00 |
| fungi | just a long-standing (and mostly cosmetic) bug | 20:00 |
| *** ianw|pto is now known as ianw | 20:00 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Handle double node locking snafu https://review.openstack.org/509603 | 20:01 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Handle double node locking snafu https://review.openstack.org/509603 | 20:02 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Handle double node locking snafu https://review.openstack.org/509603 | 20:03 |
| *** jkilpatr has quit IRC | 20:03 | |
| *** vhosakot has quit IRC | 20:04 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Create git_http_low_speed_limit / git_http_low_speed_time https://review.openstack.org/509893 | 20:06 |
| *** eharney has quit IRC | 20:07 | |
| *** edmondsw has quit IRC | 20:07 | |
| *** edmondsw has joined #openstack-infra | 20:08 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove legacy-requirements jobs https://review.openstack.org/508898 | 20:08 |
| *** edmondsw_ has joined #openstack-infra | 20:09 | |
| AJaeger | we can now merge the requirements change https://review.openstack.org/#/c/509845/ . fungi, do you want to +A? | 20:10 |
| fungi | AJaeger: should we recheck it and wait for zuulv3 feedback? | 20:12 |
| fungi | oh, i was looking at an old cached version because i don't subscribe to that repo | 20:12 |
| AJaeger | that's what I was wondering as well - zuulv3 will finish in 12 hours or so... | 20:12 |
| *** edmondsw has quit IRC | 20:12 | |
| fungi | i mean, we _did_ ask other teams to refrain from merging changes to job configuration without waiting for v3 feedback | 20:13 |
| fungi | i would feel sort of bad for not setting an example there | 20:13 |
| AJaeger | yeah ;( | 20:13 |
| fungi | especially if it broke :/ | 20:13 |
| AJaeger | let's wait - and if we restart Zuulv3, let's reenqueue | 20:14 |
| *** edmondsw_ has quit IRC | 20:14 | |
| fungi | agreed | 20:14 |
| hashar | (I am not so active anymore, but I have been introduced to a sphinx module that parse argparse to generate doc : https://github.com/ribozz/sphinx-argparse ) | 20:15 |
| hashar | that might be a good replacement for sphinxcontrib-programoutput | 20:15 |
| *** vhosakot has joined #openstack-infra | 20:15 | |
| fungi | hashar: thanks! that does look promising (i think i looked at it previously for something else too) | 20:16 |
| hashar | fungi: it has a few bugs here and there though. But yeah that sounds like a good idea | 20:17 |
| * AJaeger updated the etherpad - removed merged jobs, added review requests | 20:17 | |
| hashar | I havent looked at the code though and have no clue how it generates the material. But it is potentially worth checking | 20:17 |
| *** jcoufal has quit IRC | 20:18 | |
| fungi | http://git.openstack.org/cgit/openstack/timmy/ seems to use it | 20:18 |
| *** jtomasek has quit IRC | 20:20 | |
| fungi | or maybe they're just planning to use it. i don't see it in their extensions list in the sphinx conf | 20:20 |
| mnaser | mordred i can try hacking on it | 20:21 |
| mnaser | if you want to point me at where the status page html is in the zuul codebase... :x | 20:21 |
| fungi | oh, maybe sphinx-argparse works off entrypoint discovery, because they seem to be leveraging it in http://git.openstack.org/cgit/openstack/timmy/tree/doc/source/cli.rst | 20:22 |
| jeblair | mnaser: etc/status | 20:22 |
| *** jkilpatr has joined #openstack-infra | 20:22 | |
| mnaser | jeblair cool thanks, looking | 20:29 |
| *** lbragstad has quit IRC | 20:30 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Create git_http_low_speed_limit / git_http_low_speed_time https://review.openstack.org/509893 | 20:33 |
| * AJaeger waves good night | 20:34 | |
| *** kgiusti has left #openstack-infra | 20:34 | |
| fungi | have a good night AJaeger! thanks for all the help again today | 20:34 |
| *** mat128 has quit IRC | 20:35 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Add git timeout for HTTP(S) operations https://review.openstack.org/509876 | 20:35 |
| *** wolverineav has quit IRC | 20:37 | |
| *** edmondsw has joined #openstack-infra | 20:40 | |
| *** lbragstad has joined #openstack-infra | 20:40 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/elastic-recheck master: Re-add query for lvm create vol from snap bug 1642111 https://review.openstack.org/509899 | 20:41 |
| openstack | bug 1642111 in Cinder "create lvm volume from snapshot fails with "device-mapper: reload ioctl on (252:4) failed: Invalid argument"" [Critical,Confirmed] https://launchpad.net/bugs/1642111 | 20:41 |
| tonyb | mordred: Looks they they're all merged. and the project-template change is also close. Thanks for doing that. | 20:41 |
| tonyb | so general question we can add a project-template to any repo? does that repo need to be listed as a required-project? or is there another mechanisim? | 20:42 |
| *** gouthamr_ has quit IRC | 20:44 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/elastic-recheck master: Re-add query for lvm create vol from snap bug 1642111 https://review.openstack.org/509899 | 20:45 |
| openstack | bug 1642111 in Cinder "create lvm volume from snapshot fails with "device-mapper: reload ioctl on (252:4) failed: Invalid argument"" [Critical,Confirmed] https://launchpad.net/bugs/1642111 | 20:45 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/elastic-recheck master: Re-add query for lvm create vol from snap bug 1642111 https://review.openstack.org/509899 | 20:46 |
| clarkb | tonyb for cross role usage I had to require the project iirc | 20:47 |
| ihrachys | is there a way to trigger new zuul for a patch? | 20:47 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Fix path exclusions https://review.openstack.org/509901 | 20:47 |
| ihrachys | because recheck now triggers Jenkins | 20:48 |
| *** camunoz has quit IRC | 20:48 | |
| ihrachys | and I would like to see how neutron repos behave with v3 | 20:48 |
| clarkb | ihrachys: recheck will run it in both | 20:49 |
| *** priteau has quit IRC | 20:49 | |
| *** priteau has joined #openstack-infra | 20:50 | |
| ihrachys | I think I did that | 20:51 |
| ihrachys | but ok, I will retry | 20:51 |
| ihrachys | is it in all branches? | 20:51 |
| ihrachys | because I am looking at stable | 20:51 |
| *** hamzy has quit IRC | 20:52 | |
| clarkb | yes should be any branch | 20:52 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove references to pipelines, queues, and layouts on dequeue https://review.openstack.org/509903 | 20:54 |
| mordred | tonyb, clarkb: you don't have todo anything special to consume project-templates in other repos - it's a global set of project-templates | 20:55 |
| *** trown is now known as trown|outtypewww | 20:56 | |
| mordred | tonyb, clarkb: roles are a little different because you have to tell zuul to prepare git repos on the executor so that the roles are available to ansible when ansible-playbook is run | 20:56 |
| *** priteau has quit IRC | 20:59 | |
| *** slaweq_ has quit IRC | 21:00 | |
| *** slaweq_ has joined #openstack-infra | 21:00 | |
| *** hashar has quit IRC | 21:04 | |
| mordred | tonyb: also - yes! we're in good shape (as soon as https://review.openstack.org/#/c/509845/ merges) - I'm excited for the requirements team to own the requirements jobs! | 21:07 |
| fungi | ihrachys: we've only allotted ~20% of our overall quota to zuulv3, so it's running a backlog of epic proportions (675 changes in check and climbing, some over 15 hours old). could take a while to catch up to the point where you get results back | 21:07 |
| SamYaple | yea i havent been able to test the nova-func fix because of the slow zuulv3 so i just stopped trying | 21:08 |
| SamYaple | for requiremetns | 21:08 |
| mordred | it's an impressive backlog :) | 21:09 |
| pabelanger | most impressive | 21:09 |
| ihrachys | fungi, ack, thanks, then I will wait | 21:09 |
| SamYaple | im more impressed superdans dash.py script is holding up | 21:09 |
| SamYaple | i had to switch to it because my browser stopped working for zuulv3.o.o | 21:09 |
| *** rajinir is now known as rajinir-afk | 21:11 | |
| *** jheroux has quit IRC | 21:11 | |
| *** eumel8 has quit IRC | 21:11 | |
| *** aeng has joined #openstack-infra | 21:14 | |
| *** armaan has quit IRC | 21:15 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Fix doc typo that inverts logic https://review.openstack.org/509905 | 21:17 |
| *** gridinv has quit IRC | 21:18 | |
| *** Goneri has quit IRC | 21:19 | |
| *** baoli has quit IRC | 21:19 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Add some allowed-projects restrictions to publish jobs https://review.openstack.org/509906 | 21:20 |
| *** rcernin has joined #openstack-infra | 21:22 | |
| *** dprince has quit IRC | 21:23 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Handle double node locking snafu https://review.openstack.org/509603 | 21:27 |
| *** ldnunes has quit IRC | 21:34 | |
| *** esberglu has quit IRC | 21:36 | |
| *** edmondsw has quit IRC | 21:41 | |
| *** edmondsw has joined #openstack-infra | 21:41 | |
| *** gouthamr has joined #openstack-infra | 21:43 | |
| *** edmondsw has quit IRC | 21:46 | |
| *** MasterOfBugs has quit IRC | 21:46 | |
| *** MasterOfBugs has joined #openstack-infra | 21:46 | |
| *** felipemonteiro__ has quit IRC | 21:46 | |
| *** armaan has joined #openstack-infra | 21:52 | |
| *** slaweq_ has quit IRC | 21:55 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix early processing of merge-pending items on reconfig https://review.openstack.org/509912 | 21:59 |
| tonyb | mordred, clarkb: okay. I appreciate you trying to educate me. It may need to wait until we're in a rom togther. I still don't understand how the project-template in openstack/requirements is used for jobs in openstack/neutron *without* zuul knowing / pulling all repos to get the global set of templates. | 22:01 |
| jeblair | tonyb: that's exactly what zuul does | 22:03 |
| *** pramodrj07 has joined #openstack-infra | 22:04 | |
| tonyb | jeblair: Oh. well then that makes sense then | 22:04 |
| tonyb | jeblair: isn't that really expensive for ~1600 repos? | 22:04 |
| jeblair | tonyb: to put it in perspective, when zuul cold starts, it reads its config files from every branch of every project in the system. then, each time it sees a change lands to a repo, it updates its config. | 22:04 |
| tonyb | jeblair: Oh okay so the cold start is expensive but the rest in incremental, okay | 22:05 |
| jeblair | tonyb: that's about 6000 repo-branch combinations it inspects. it currently does it in about 6 minutes, from cold-start. when we finish the migration, i expect it to be under 5 minutes. once it's going, it can update its config very quickly -- just a few seconds. | 22:05 |
| *** rcernin has quit IRC | 22:06 | |
| tonyb | Oh okay, I had foolishly included clone time, but of course it has all the data local. | 22:07 |
| tonyb | jeblair: Thanks. I know I don' | 22:07 |
| *** MasterOfBugs has quit IRC | 22:07 | |
| tonyb | t really *need* to know this but it all helps when trying to write jobs / debug what they're doing ;P | 22:07 |
| *** baoli has joined #openstack-infra | 22:08 | |
| jeblair | tonyb: ++ | 22:09 |
| *** aeng has quit IRC | 22:10 | |
| *** signed8bit has joined #openstack-infra | 22:11 | |
| *** baoli has quit IRC | 22:12 | |
| *** threestrands has joined #openstack-infra | 22:12 | |
| *** baoli has joined #openstack-infra | 22:12 | |
| *** yamamoto has joined #openstack-infra | 22:14 | |
| *** bobh has quit IRC | 22:15 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Fix doc typo that missed important words https://review.openstack.org/509905 | 22:27 |
| thingee | clarkb, mordred, fungi: so I do have a project coming down the pipeline which is the contributor-guide. It's a sphinx project to hold the general documentation for the Contributor Portal effort. Do I have it right that I can submit this still but it has to be for v2 and v3, whatever that means in project-config? | 22:34 |
| fungi | thingee: either that or go v3-only if it can wait until we're back on v3 _and_ sure we don't need to roll back to v2 again | 22:35 |
| thingee | fungi: thanks | 22:36 |
| fungi | if we had managed to avoid a rollback we could have plowed forward without needing redundant configuration, and we're still hoping to keep the period of parallel configuration changes to a minimum | 22:37 |
| *** lbragstad has quit IRC | 22:38 | |
| thingee | fungi: understood. as long as there are options I'm happy as a clam. I'm look up documentation now on what this all means :) | 22:38 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Use yarn and webpack to manage status javascript https://review.openstack.org/487538 | 22:39 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Migrate console streaming to webpack/yarn https://review.openstack.org/487539 | 22:39 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: alien-list: use provider name https://review.openstack.org/508788 | 22:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Migrate console streaming to webpack/yarn https://review.openstack.org/487539 | 22:52 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: WIP Use yarn and webpack to manage status javascript https://review.openstack.org/487538 | 22:54 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: WIP Migrate console streaming to webpack/yarn https://review.openstack.org/487539 | 22:54 |
| mordred | added WIP prefixes so that it's clear what the state is - and that'll all I'll do on those for now | 22:54 |
| mordred | jeblair: https://review.openstack.org/#/c/509901 is shown as in merge conflict, but is totally not ... should I chalk that up to general weirdness? | 22:56 |
| mordred | also - jeblair, clarkb, fungi: if you have a sec, https://review.openstack.org/#/c/509905 is ready | 22:57 |
| mtreinish | clarkb, mordred, fungi: can I get reviews on: https://review.openstack.org/#/q/status:open+project:openstack-infra/puppet-subunit2sql+branch:master+topic:restore-name-sanity so we're prepared for the next v3 switchover | 22:59 |
| jeblair | mordred: i'll take a look | 23:00 |
| *** gmann_sleep is now known as gmann | 23:01 | |
| *** erlon has quit IRC | 23:01 | |
| *** tpsilva has quit IRC | 23:02 | |
| *** mrunge has quit IRC | 23:06 | |
| *** iyamahat_ has joined #openstack-infra | 23:06 | |
| mordred | mtreinish: looking | 23:07 |
| *** hongbin has quit IRC | 23:07 | |
| mtreinish | mordred: thanks | 23:07 |
| *** mrunge has joined #openstack-infra | 23:07 | |
| *** iyamahat has quit IRC | 23:08 | |
| mordred | infra-root: I just took a pass through https://etherpad.openstack.org/p/zuulv3-issues and updated the job section - there's 4 lists of changes thatshould fix the associated issues, but we have to shepherd them through the gate lke we did with the requirements jobs today | 23:08 |
| mordred | mtreinish: both look fine to me - I didn't hit +A on the first one cause I wasnt' sure if that was a thing you wanted to do now or just wanted it primed for rollout | 23:10 |
| thingee | hey all, I'm following project creator documentation and I'm so lost on what I'm suppose to do for zuul jobs. https://docs.openstack.org/infra/manual/creators.html#add-basic-zuul-jobs | 23:10 |
| thingee | is there a list of existing jobs and their descriptions of what they do? | 23:11 |
| *** aviau has quit IRC | 23:11 | |
| *** gildub has joined #openstack-infra | 23:11 | |
| mtreinish | mordred: I'm not sure what would be better. I think the if checks should prevent it from ever being triggered until we do the rollouts | 23:11 |
| *** aviau has joined #openstack-infra | 23:11 | |
| thingee | I looked at this document and it made me more confused https://docs.openstack.org/infra/manual/drivers.html#naming-with-zuul-v3 | 23:11 |
| mtreinish | which would make it safe to just land now | 23:11 |
| mtreinish | but I haven't checked every job name out there | 23:11 |
| mtreinish | there might be non-dsvm jobs that don't have a project name in them | 23:12 |
| mtreinish | but that's something we'll hit at the rollout anyway | 23:12 |
| thingee | all I want to do is have sphinx docs built in check and merge | 23:12 |
| thingee | I'm guessing I create a .zuul.yml in my project file, just not sure what the approriate names are for jobs. I tried digging through project-config and I think I'm getting confused on what is v2 and what is v3 | 23:14 |
| mordred | thingee: SO ... | 23:15 |
| mordred | thingee: the https://docs.openstack.org/infra/manual/creators.html#add-basic-zuul-jobs comment about main.yaml is up to date and correct | 23:15 |
| mordred | wow - the next section tha tI updated a couple of days ago is totally wrong :) | 23:16 |
| mordred | thingee: next thing is to add an entry for the project in project-config zuul.d/projects.yaml that looks like this: http://paste.openstack.org/show/622796/ | 23:17 |
| thingee | mordred: please use my naive eyes. I think I've created one project before. I've been struggling with creating a project for a couple days. I know the infra team is busy. | 23:17 |
| *** sdague has quit IRC | 23:18 | |
| *** claudiub has joined #openstack-infra | 23:19 | |
| mordred | thingee: well - unfortuntely things are in a lovely half-way state for you - but this is good, it's helping see where the docs are broken :) | 23:19 |
| jeblair | mordred: that merge conflict was due to a git hang. | 23:19 |
| mordred | jeblair: cool. so recheck is safe | 23:20 |
| mordred | thingee: I think what I'm seeing is that there is a bunch of additoinal info after "Add Basic Zuul Jobs" where there should be a very prescriptive "do this" | 23:20 |
| jeblair | mordred: ya. i think we'll probably want to add some auto-retry stuff for merger jobs if we can detect that situation. | 23:20 |
| mordred | jeblair: ++ | 23:20 |
| mordred | thingee: if you have a sec, lemme just make a quick patch to that doc and see if it clears anything up | 23:21 |
| *** baoli has quit IRC | 23:24 | |
| thingee | mordred: sure | 23:25 |
| *** edmondsw has joined #openstack-infra | 23:28 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove references to pipelines, queues, and layouts on dequeue https://review.openstack.org/509903 | 23:30 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix early processing of merge-pending items on reconfig https://review.openstack.org/509912 | 23:30 |
| *** tosky has quit IRC | 23:31 | |
| *** edmondsw has quit IRC | 23:32 | |
| openstackgerrit | Mike Perez proposed openstack-infra/project-config master: Adding the Contributor Guide Project https://review.openstack.org/509937 | 23:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/infra-manual master: Be a bit more prescriptive about getting started https://review.openstack.org/509939 | 23:39 |
| mordred | thingee: ok - I tried to reorganize a little bit and also be a bit more direct/specific | 23:40 |
| mordred | oh! this is fun ... | 23:40 |
| thingee | mordred: hi zuul admin, my project is not accepted :) | 23:41 |
| mordred | thingee: :) | 23:41 |
| mordred | SO - you are the first to exercise what I have just realized is going to be a rather interesting workflow ... | 23:41 |
| * thingee puts on dancing shoes | 23:41 | |
| jeblair | i think the other thing that we could add (later) is an index of common templates/jobs. ie, "if you're adding a python project, you probably want these templates which do the following..." | 23:42 |
| mordred | jeblair: yup. I was thinking the same thing | 23:42 |
| thingee | jeblair: +1000 | 23:42 |
| thingee | jeblair: exactly what I was looking for | 23:42 |
| mordred | jeblair, thingee: SO - step one is "write patch adding project to main.yaml" | 23:43 |
| mordred | then that patch must land before adding the project to zuul.d/projects.yaml | 23:43 |
| *** hamzy has joined #openstack-infra | 23:43 | |
| *** SumitNaiksatam has quit IRC | 23:44 | |
| jeblair | mordred: yeah. that patch can/should also include the addition to gerrit's projects.yaml | 23:44 |
| *** gildub has quit IRC | 23:44 | |
| mordred | jeblair: yah- I guess that's the sequence- everything but zuul.d/projects.yaml - then projects.yaml yeah? | 23:44 |
| jlk | Can't you just make the second patch (adding the project to zuul.d/projects.yaml) depend on the first one (adding it to main.yaml) ? | 23:44 |
| mordred | jlk: you totally can - the second patch is just gonna fail until the first one is landed | 23:45 |
| thingee | jlk: yup that's what I plan to do | 23:45 |
| jlk | nod | 23:45 |
| jeblair | so we can put a "..note::" or whatever letting folks know to expect a temporary error message | 23:45 |
| mordred | yah | 23:45 |
| mordred | it's getting late in the day here - I'm going to make a note to update the project creation guide when I wake up ... | 23:46 |
| mordred | but thingee I believe if you split that into two patches with the zuul.d/projects.yaml bit coming second, you'll be off to a good start | 23:46 |
| openstackgerrit | Mike Perez proposed openstack-infra/project-config master: Adding Contributor Guide to zuul main config https://review.openstack.org/509937 | 23:46 |
| mordred | woot | 23:47 |
| mordred | thingee: oh - sorry, I may have rambled and given you bad advice - if you re-do that to be the same as patch 1 except just without zuul.d/projects.yaml - then have patch 2 have zuul.d/projects.yaml, that'd be best | 23:48 |
| *** gildub has joined #openstack-infra | 23:50 | |
| *** markvoelker_ has quit IRC | 23:51 | |
| *** vhosakot has quit IRC | 23:52 | |
| openstackgerrit | Mike Perez proposed openstack-infra/project-config master: Adding Contributor Guide to zuul main config https://review.openstack.org/509937 | 23:55 |
| openstackgerrit | Mike Perez proposed openstack-infra/project-config master: Adding the Contributor Guide Project https://review.openstack.org/509943 | 23:55 |
| *** pramodrj07 has quit IRC | 23:56 | |
| thingee | mordred: doc build failed for your enhancement | 23:56 |
| mordred | thingee: yah - it's because I'm bad at things | 23:56 |
| mordred | thingee: left a note on the review - one pair of files is flipped, other than that looks great | 23:56 |
| thingee | or are computers just so logical that they can't compensate on their own for your mistakes, hmmm? | 23:57 |
| mordred | thingee: I think you are very correct | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!