| *** mschuppert has joined #openstack-infra | 00:05 | |
| *** agopi has joined #openstack-infra | 00:26 | |
| *** dklyle has quit IRC | 00:32 | |
| *** dklyle has joined #openstack-infra | 00:32 | |
| *** markvoelker has quit IRC | 00:35 | |
| *** slagle has quit IRC | 00:36 | |
| *** mtreinish has quit IRC | 00:45 | |
| *** mtreinish has joined #openstack-infra | 00:46 | |
| *** mriedem has quit IRC | 00:52 | |
| *** jtomasek_ has quit IRC | 00:58 | |
| *** hongbin has joined #openstack-infra | 01:00 | |
| *** mtreinish has quit IRC | 01:06 | |
| *** pcrews has quit IRC | 01:06 | |
| *** mtreinish has joined #openstack-infra | 01:08 | |
| *** ijw has joined #openstack-infra | 01:10 | |
| *** jamesmcarthur has joined #openstack-infra | 01:10 | |
| *** slaweq has joined #openstack-infra | 01:11 | |
| *** jamesmcarthur has quit IRC | 01:12 | |
| *** jamesmcarthur has joined #openstack-infra | 01:12 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: Add openafs-client role https://review.openstack.org/589334 | 01:15 |
|---|---|---|
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: Add kerberos-client role https://review.openstack.org/590591 | 01:15 |
| *** slaweq has quit IRC | 01:16 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: Test OpenAFS client install https://review.openstack.org/589335 | 01:16 |
| *** ijw has quit IRC | 01:17 | |
| *** zhurong has joined #openstack-infra | 01:22 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: fix job required-projects list https://review.openstack.org/590594 | 01:31 |
| *** yamahata has quit IRC | 01:32 | |
| *** rosmaita has quit IRC | 01:33 | |
| *** jamesmcarthur has quit IRC | 01:33 | |
| *** mschuppert has quit IRC | 01:36 | |
| *** ijw has joined #openstack-infra | 01:38 | |
| *** studarus has quit IRC | 01:45 | |
| *** jamesmcarthur has joined #openstack-infra | 01:48 | |
| *** jamesmcarthur has quit IRC | 01:53 | |
| *** jamesmcarthur has joined #openstack-infra | 01:55 | |
| *** Bhujay has joined #openstack-infra | 01:57 | |
| *** jamesmcarthur has quit IRC | 02:00 | |
| *** jamesmcarthur has joined #openstack-infra | 02:01 | |
| *** ijw has quit IRC | 02:02 | |
| *** ijw has joined #openstack-infra | 02:03 | |
| *** ijw has quit IRC | 02:06 | |
| *** e0ne has joined #openstack-infra | 02:16 | |
| *** e0ne has quit IRC | 02:18 | |
| *** jamesmcarthur has quit IRC | 02:18 | |
| *** jamesmcarthur has joined #openstack-infra | 02:22 | |
| *** zhurong has quit IRC | 02:23 | |
| *** yamamoto has joined #openstack-infra | 02:31 | |
| *** jamesmcarthur has quit IRC | 02:43 | |
| openstackgerrit | Merged openstack-infra/zuul master: Fix wrong matched project template https://review.openstack.org/588201 | 02:47 |
| *** jamesmcarthur has joined #openstack-infra | 02:47 | |
| *** psachin has joined #openstack-infra | 02:48 | |
| *** yamamoto has quit IRC | 02:54 | |
| *** jamesmcarthur has quit IRC | 03:04 | |
| *** armax has quit IRC | 03:07 | |
| *** jamesmcarthur has joined #openstack-infra | 03:09 | |
| *** ijw has joined #openstack-infra | 03:09 | |
| *** slaweq has joined #openstack-infra | 03:11 | |
| *** hoangcx has left #openstack-infra | 03:12 | |
| *** slaweq has quit IRC | 03:15 | |
| *** ijw has quit IRC | 03:16 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: Add kerberos-client role https://review.openstack.org/590591 | 03:19 |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: Add openafs-client role https://review.openstack.org/589334 | 03:19 |
| *** hongbin has quit IRC | 03:19 | |
| openstackgerrit | wu.chunyang proposed openstack-infra/puppet-apparmor master: fix tests_smoke link https://review.openstack.org/590621 | 03:21 |
| *** jamesmcarthur has quit IRC | 03:26 | |
| *** rh-jelabarre has quit IRC | 03:28 | |
| *** jamesmcarthur has joined #openstack-infra | 03:30 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Remove legacy-install-afs-with-puppet role https://review.openstack.org/590636 | 03:43 |
| *** udesale has joined #openstack-infra | 03:49 | |
| *** Bhujay has quit IRC | 03:51 | |
| *** ianychoi has joined #openstack-infra | 03:56 | |
| *** jamesmcarthur has quit IRC | 04:00 | |
| *** ykarel|away has joined #openstack-infra | 04:03 | |
| *** jamesmcarthur has joined #openstack-infra | 04:04 | |
| *** jamesmcarthur has quit IRC | 04:10 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Remove legacy-install-afs-with-puppet role https://review.openstack.org/590636 | 04:15 |
| openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Bump ansible for linters https://review.openstack.org/590641 | 04:15 |
| *** jamesmcarthur has joined #openstack-infra | 04:17 | |
| *** jamesmcarthur has quit IRC | 04:22 | |
| *** jamesmcarthur has joined #openstack-infra | 04:25 | |
| *** janki has joined #openstack-infra | 04:25 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: linters: Take a guess at the roles path https://review.openstack.org/590642 | 04:30 |
| ianw | tonyb/pabelanger/mordred: ^^ https://review.openstack.org/#/c/590636/ and it's chain of deps should remove the need for puppet installs on the wheel builders by installing kerberos and openafs via ansible. should be useful for mirror servers under ansible too | 04:32 |
| tonyb | ianw: Thanks! I'll look over them but I like where they're going ;P | 04:34 |
| *** yamahata has joined #openstack-infra | 04:37 | |
| *** ykarel|away is now known as ykarel | 04:39 | |
| *** jamesmcarthur has quit IRC | 04:42 | |
| *** Bhujay has joined #openstack-infra | 04:43 | |
| *** yamamoto has joined #openstack-infra | 04:51 | |
| *** slaweq has joined #openstack-infra | 05:11 | |
| *** jamesmcarthur has joined #openstack-infra | 05:11 | |
| *** slaweq has quit IRC | 05:16 | |
| *** janki has quit IRC | 05:19 | |
| *** jamesmcarthur has quit IRC | 05:20 | |
| *** jamesmcarthur has joined #openstack-infra | 05:22 | |
| *** ykarel_ has joined #openstack-infra | 05:26 | |
| *** jamesmcarthur has quit IRC | 05:26 | |
| *** ykarel has quit IRC | 05:28 | |
| *** janki has joined #openstack-infra | 05:28 | |
| *** ykarel__ has joined #openstack-infra | 05:28 | |
| *** ykarel_ has quit IRC | 05:31 | |
| *** quiquell has joined #openstack-infra | 05:42 | |
| *** yamamoto has quit IRC | 05:42 | |
| *** yamamoto has joined #openstack-infra | 05:46 | |
| *** nicolasbock has joined #openstack-infra | 05:50 | |
| openstackgerrit | YumengBao proposed openstack-infra/project-config master: Switch to publish-openstack-docs-pti for cyborg https://review.openstack.org/590650 | 05:53 |
| *** hwoarang has joined #openstack-infra | 05:56 | |
| *** mschuppert has joined #openstack-infra | 06:02 | |
| openstackgerrit | Merged openstack/diskimage-builder master: better handle existing keywords files/directories https://review.openstack.org/577633 | 06:07 |
| *** Emine has quit IRC | 06:12 | |
| dmellado | morning! could anyone please check what's going on with https://review.openstack.org/#/c/584377/ | 06:15 |
| dmellado | ? | 06:15 |
| *** yamahata has quit IRC | 06:15 | |
| dmellado | it seems that zuul didn't pick it properly and got stuck when going into gate queue | 06:15 |
| dmellado | tonyb: ianw AJaeger anyone around? ;) | 06:16 |
| tonyb | dmellado: I think it just needs a recheck | 06:16 |
| dmellado | oh, well I'll recheck it then I'll play the dictator and just +a+w it | 06:17 |
| tonyb | the chnage it 'Depends-On' was in the gate pipeline when it got it;s +W so at that time it wasn't valid to tranistion | 06:17 |
| dmellado | tonyb: ack, thanks! | 06:17 |
| tonyb | dmellado: I could be wrong but that's what I see. | 06:17 |
| tonyb | dmellado: Looks like you'll know in about an hour ... if it gies straint gtom check -> gate pipelines | 06:18 |
| dmellado | tonyb: yep | 06:18 |
| * dmellado hitting recheck xD | 06:18 | |
| tonyb | :) | 06:21 |
| *** yamamoto has quit IRC | 06:23 | |
| *** ykarel_ has joined #openstack-infra | 06:23 | |
| *** yamamoto has joined #openstack-infra | 06:24 | |
| *** ykarel__ has quit IRC | 06:26 | |
| *** jtomasek has joined #openstack-infra | 06:32 | |
| *** yamamoto has quit IRC | 06:32 | |
| *** larainema has quit IRC | 06:38 | |
| *** janki has quit IRC | 06:41 | |
| *** pcaruana has joined #openstack-infra | 06:43 | |
| *** hwoarang has quit IRC | 06:45 | |
| *** hwoarang has joined #openstack-infra | 06:50 | |
| *** ccamacho has joined #openstack-infra | 06:50 | |
| *** apetrich has joined #openstack-infra | 06:55 | |
| AJaeger | dmellado: indeed, those two do not share a queue - so you need to wait with +A until the first is merged. Adding another +A is all that was needed | 06:55 |
| dmellado | AJaeger: yep, looks like it's on its way now, thanks! | 06:56 |
| AJaeger | dmellado: or a toggle of the +A (remove, add back) | 06:56 |
| dmellado | AJaeger: heh, that's what I first tried | 06:56 |
| *** ykarel__ has joined #openstack-infra | 06:56 | |
| AJaeger | dmellado: too early ;) | 06:57 |
| AJaeger | dmellado: the other change was not merged when you did it | 06:57 |
| *** ykarel_ has quit IRC | 06:59 | |
| *** quiquell has quit IRC | 07:00 | |
| *** ykarel__ is now known as ykarel | 07:01 | |
| *** quiquell has joined #openstack-infra | 07:01 | |
| *** egonzalez has joined #openstack-infra | 07:01 | |
| *** jpich has joined #openstack-infra | 07:02 | |
| *** amoralej|off is now known as amoralej | 07:02 | |
| egonzalez | hi, ask.o.o is down | 07:03 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use new translation jobs templates https://review.openstack.org/590665 | 07:03 |
| egonzalez | nvm, it works now | 07:03 |
| AJaeger | eumel8: ^ | 07:03 |
| eumel8 | got it | 07:04 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use new translation jobs templates https://review.openstack.org/590665 | 07:05 |
| AJaeger | eumel8: more later, just wanted to get the ball rolling... | 07:05 |
| *** slaweq has joined #openstack-infra | 07:05 | |
| *** slaweq has quit IRC | 07:06 | |
| *** evrardjp has joined #openstack-infra | 07:06 | |
| *** slaweq has joined #openstack-infra | 07:06 | |
| *** rcernin has quit IRC | 07:07 | |
| *** holser_ has joined #openstack-infra | 07:10 | |
| *** jaosorior has quit IRC | 07:15 | |
| eumel8 | yes,thx | 07:17 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Add --check-config option to zuul scheduler https://review.openstack.org/542160 | 07:17 |
| *** janki has joined #openstack-infra | 07:27 | |
| *** ijw has joined #openstack-infra | 07:31 | |
| *** ijw has quit IRC | 07:35 | |
| *** jamesmcarthur has joined #openstack-infra | 07:38 | |
| *** ijw has joined #openstack-infra | 07:41 | |
| *** ginopc has joined #openstack-infra | 07:41 | |
| *** jamesmcarthur has quit IRC | 07:42 | |
| *** ykarel is now known as ykarel|lunch | 07:43 | |
| *** ijw has quit IRC | 07:45 | |
| *** ijw has joined #openstack-infra | 07:46 | |
| *** udesale has quit IRC | 07:51 | |
| *** udesale has joined #openstack-infra | 07:51 | |
| *** jpena|off is now known as jpena | 07:55 | |
| *** e0ne has joined #openstack-infra | 07:59 | |
| *** ijw has quit IRC | 08:08 | |
| *** ijw has joined #openstack-infra | 08:09 | |
| *** psachin has quit IRC | 08:13 | |
| *** priteau has joined #openstack-infra | 08:14 | |
| *** Bhujay has quit IRC | 08:23 | |
| *** psachin has joined #openstack-infra | 08:27 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix stuck job caused by exception during repo update https://review.openstack.org/590697 | 08:27 |
| *** ijw has quit IRC | 08:28 | |
| *** janki has quit IRC | 08:32 | |
| *** derekh has joined #openstack-infra | 08:33 | |
| *** janki has joined #openstack-infra | 08:43 | |
| *** ykarel|lunch is now known as ykarel | 08:45 | |
| *** e0ne has quit IRC | 08:49 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use new translation jobs templates https://review.openstack.org/590665 | 08:52 |
| *** xarses_ has quit IRC | 08:52 | |
| *** ssbarnea has quit IRC | 08:52 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Do not abort node launch if failed node cannot be deleted https://review.openstack.org/589854 | 08:59 |
| witek | hello, could someone please help me on https://review.openstack.org/590387 ? | 09:02 |
| witek | I tried to follow https://git.openstack.org/cgit/openstack/goal-tools/tree/README.rst | 09:02 |
| witek | but Zuul was misconfigured for stable branches on monasca-api prior to that | 09:03 |
| *** gfidente has joined #openstack-infra | 09:04 | |
| witek | Python 3 support was added in Rocky and enabling openstack-python35-jobs template blocked stable branches | 09:05 |
| openstackgerrit | Merged openstack-infra/irc-meetings master: Update the nova team meeting chair to melwitt https://review.openstack.org/590427 | 09:06 |
| AJaeger | witek: best wait until dhellmann is around... | 09:09 |
| witek | ok | 09:09 |
| AJaeger | witek: he's US based | 09:09 |
| *** e0ne has joined #openstack-infra | 09:14 | |
| *** pbourke has quit IRC | 09:16 | |
| *** Dobroslaw has joined #openstack-infra | 09:17 | |
| *** pbourke has joined #openstack-infra | 09:18 | |
| *** e0ne has quit IRC | 09:27 | |
| *** Bhujay has joined #openstack-infra | 09:31 | |
| *** ssbarnea has joined #openstack-infra | 09:31 | |
| *** e0ne has joined #openstack-infra | 09:37 | |
| *** rkukura_ has joined #openstack-infra | 09:46 | |
| *** rkukura has quit IRC | 09:46 | |
| *** rkukura_ is now known as rkukura | 09:46 | |
| *** jaosorior has joined #openstack-infra | 09:48 | |
| *** roman_g has quit IRC | 10:13 | |
| *** roman_g has joined #openstack-infra | 10:13 | |
| *** tosky has joined #openstack-infra | 10:19 | |
| *** e0ne has quit IRC | 10:21 | |
| *** ykarel_ has joined #openstack-infra | 10:28 | |
| *** ykarel__ has joined #openstack-infra | 10:30 | |
| *** ykarel has quit IRC | 10:30 | |
| *** ykarel_ has quit IRC | 10:33 | |
| *** panda|ruck|off is now known as panda|ruck | 10:33 | |
| *** tosky has quit IRC | 10:35 | |
| *** tosky has joined #openstack-infra | 10:36 | |
| *** udesale has quit IRC | 10:36 | |
| *** udesale has joined #openstack-infra | 10:47 | |
| *** dave-mccowan has joined #openstack-infra | 10:55 | |
| *** rpittau has quit IRC | 11:00 | |
| *** dave-mccowan has quit IRC | 11:00 | |
| *** dtantsur|afk is now known as dtantsur | 11:01 | |
| *** jpena is now known as jpena|lunch | 11:06 | |
| *** holser_ has quit IRC | 11:18 | |
| *** nicolasbock has quit IRC | 11:19 | |
| *** rh-jelabarre has joined #openstack-infra | 11:26 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Split base playbook into two plays https://review.openstack.org/590478 | 11:26 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remove old inactive users https://review.openstack.org/588553 | 11:32 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remove old inactive users https://review.openstack.org/588553 | 11:37 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Set mgmt_hieradata variable for bridge.openstack.org https://review.openstack.org/588627 | 11:37 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add exim role to base playbook https://review.openstack.org/589964 | 11:37 |
| mordred | cmurphy: I think https://review.openstack.org/#/c/588326/ is solid now - it has been properly running the ansible (and it has been failing because of ansible issues) | 11:37 |
| cmurphy | mordred: coolbeans | 11:40 |
| *** udesale has quit IRC | 11:41 | |
| *** udesale has joined #openstack-infra | 11:42 | |
| *** ykarel_ has joined #openstack-infra | 11:50 | |
| *** ykarel__ has quit IRC | 11:53 | |
| mordred | cmurphy: also, as you might can tell - having that in place is super helpful :) | 11:55 |
| *** boden has joined #openstack-infra | 11:56 | |
| cmurphy | :) | 11:57 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add system-config to roles path https://review.openstack.org/590752 | 11:58 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Move roles into roles dir https://review.openstack.org/590753 | 11:58 |
| mordred | cmurphy: I haz ? ... in setup_host, there is a "host.add_env_var" call - does that add that as an env var to all subsequent on host calls? | 12:01 |
| *** jpena|lunch is now known as jpena | 12:02 | |
| *** jaosorior has quit IRC | 12:03 | |
| *** mriedem has joined #openstack-infra | 12:05 | |
| cmurphy | mordred: it adds the variable to .ssh/environment which i believe has that effect yes | 12:07 |
| mordred | cmurphy: cool | 12:07 |
| openstackgerrit | Monty Taylor proposed openstack-infra/puppet-openstack_infra_spec_helper master: Add roles dir in system-config to roles path https://review.openstack.org/590755 | 12:09 |
| mordred | cmurphy: so that ^^ should work? | 12:09 |
| *** gary_perkins has quit IRC | 12:10 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Move roles into roles dir https://review.openstack.org/590753 | 12:10 |
| cmurphy | mordred: read the comment on line 24 - beaker keeps the ssh connection open which means changes in the environment won't take effect until you reinitialize the connection | 12:11 |
| mordred | oh! | 12:11 |
| cmurphy | i would probably suggest adding both vars within the setup_host function | 12:11 |
| mordred | yah. I agree | 12:12 |
| mordred | cmurphy: is host.ssh_permit_user_environment needed for the add_env_var thing to work? | 12:13 |
| AJaeger | config-core, could you review https://review.openstack.org/589372 - and https://review.openstack.org/590665 (will need recheck once the first one is in), please? The first check will make changes for translations at release time much easier for us... | 12:14 |
| openstackgerrit | Monty Taylor proposed openstack-infra/puppet-openstack_infra_spec_helper master: Add roles dir in system-config to roles path https://review.openstack.org/590755 | 12:14 |
| cmurphy | mordred: yes, it makes a change to /etc/ssh/sshd_config in order to allow it | 12:14 |
| mordred | cmurphy: so maybe like that then | 12:14 |
| cmurphy | mordred: yeah that looks right | 12:16 |
| *** mriedem has quit IRC | 12:17 | |
| *** kgiusti has joined #openstack-infra | 12:20 | |
| mordred | \o/ | 12:20 |
| *** quiquell has quit IRC | 12:20 | |
| *** rpittau has joined #openstack-infra | 12:23 | |
| pabelanger | dmsimard: mind a +3 on https://review.openstack.org/590577/ blocking the removal of fedora-27 images in nodepool | 12:25 |
| *** ansmith has joined #openstack-infra | 12:26 | |
| *** ykarel_ is now known as ykarel | 12:27 | |
| *** amoralej is now known as amoralej|lunch | 12:28 | |
| *** udesale has quit IRC | 12:34 | |
| *** janki has quit IRC | 12:35 | |
| *** holser_ has joined #openstack-infra | 12:35 | |
| *** psachin has quit IRC | 12:39 | |
| *** holser_ has quit IRC | 12:41 | |
| *** trown|outtypewww is now known as trown | 12:41 | |
| *** holser_ has joined #openstack-infra | 12:41 | |
| *** tpsilva has joined #openstack-infra | 12:44 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix permanently broken git cache https://review.openstack.org/590761 | 12:45 |
| *** jamesmcarthur has joined #openstack-infra | 12:47 | |
| *** jamesmcarthur has quit IRC | 12:51 | |
| *** witek has quit IRC | 12:52 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix permanently broken git cache https://review.openstack.org/590761 | 12:54 |
| *** jcoufal has joined #openstack-infra | 12:55 | |
| *** jamesmcarthur has joined #openstack-infra | 12:56 | |
| *** egonzalez has quit IRC | 13:03 | |
| boden | hi, I'm having some complications publishing a release for the vmware-nsx unofficial projects using the publish-to-pypi template; it appears my tag is being renamed... For example I pushed a signed tag of 13.0.0.0 to vmware-nsx, but it results in 2018.0.0.0rc2.dev48 being published to pypi | 13:03 |
| boden | the job log http://logs.openstack.org/e3/e300f9e7ad2d334adc8afad38c943d8ee76ec4ca/release/release-openstack-python/8cddd6a/job-output.txt.gz for https://pypi.org/project/vmware-nsx/#history | 13:04 |
| boden | any ideas what I'm doing wrong here? | 13:04 |
| *** rosmaita has joined #openstack-infra | 13:05 | |
| fungi | boden: for starters, 13.0.0.0 isn't valid semver, you have one .0 too many | 13:06 |
| boden | fungi: if I use 13.0.0 the dist created has three 0s... also that format seems to match what neutron is doing https://pypi.org/project/neutron/#history | 13:07 |
| fungi | boden: but also, looks like pbr might be finding a 2018.0.0.0rc2 tag and considering that the newest | 13:07 |
| boden | fungi: can I delete a signed tag? | 13:07 |
| fungi | boden: what neutron is doing where? | 13:08 |
| boden | fungi https://pypi.org/project/neutron/#history has 13.0.0.0rc1 that's three 0s | 13:08 |
| fungi | tag deletion is unreliable since it only propagates over push and not over pull | 13:08 |
| boden | fungi: hmm, ok so I'm now unsure how to resolve if I can't delete the tag | 13:09 |
| fungi | 13.0.0.0rc1 is what comes before 13.0.0 per pep 440 normalizaton rules | 13:09 |
| boden | fungi: ok I was mislead by what's on pypi... any suggestions on how to proceed? | 13:10 |
| fungi | yeah, the usual advice about testing locally and being sure before you push a tag isn't much help afterward | 13:11 |
| boden | fungi I did test locally, but frankly the process wasn't clear to me.. I thought everything was in order | 13:11 |
| fungi | infra-root: opinions on trying to delete the vmware-nsx 2018.0.0.0rc2 tag? do we anticipate that sticking around on zuul mergers? | 13:12 |
| fungi | boden: testing insofar as `python setup.py sdisd bdist_wheel` and then looking at the resulting filenames that creates in the dist/ subdir? | 13:13 |
| boden | fungi yes | 13:13 |
| fungi | s/sdisd/sdist/ | 13:13 |
| fungi | and it didn't create them with 2018.0.0.0rc2 in the filenames? interesting | 13:13 |
| *** jamesmcarthur has quit IRC | 13:14 | |
| *** eharney has joined #openstack-infra | 13:14 | |
| boden | fungi this is my first time though this process and was using mainly https://docs.openstack.org/infra/manual/creators.html#tagging-a-release there isn't much info there on such details, but in any case I think I just need to figure out how to proceed | 13:14 |
| fungi | i need to spend a few minutes taking a closer look at the repo and the job logs to see if there might be a simpler solution | 13:16 |
| boden | fungi I have similar problems with the other vmware-nsx projects too.. or maybe I'm just confused | 13:16 |
| fungi | mordred: you may also have a slightly better grasp of pbr behaviors in this situation ^ | 13:17 |
| fungi | boden: was your intent for this upcoming release to be 2018.0.0 or 13.0.0? if you're following neutron as an example, it stopped using year-based versions in 2015 | 13:18 |
| boden | 13.0.0 | 13:19 |
| mordred | fungi: reading | 13:20 |
| fungi | yeah, looking at earlier tags in vmware-nsx you had like 12.0.0.0rc1 followed by 12.0.0 which is what we usually expect | 13:20 |
| fungi | i have a feeling pbr is merely ignoring the malformed 13.0.0.0 tag and that if we just tag 13.0.0 it'll work | 13:22 |
| fungi | without trying to delete any tags | 13:22 |
| fungi | but running tox on the vmware-nsx repo is taking a few minutes | 13:22 |
| mordred | ok. just tag again with 13.0.0 | 13:23 |
| *** amoralej|lunch is now known as amoralej | 13:23 | |
| mordred | and it'll fix it | 13:23 |
| boden | fungi I can try... just hoping to not munk things up more.... sorry for this, I just didn't see such details noted in the docs. | 13:23 |
| fungi | ahh, so pbr does indeed ignore tags with an extra (non-semver) 0? | 13:23 |
| mordred | yes | 13:23 |
| * mordred has verified the behavior locally | 13:23 | |
| fungi | boden: well, the tagging a release document is generic for all sorts of projects, and you're specifically fighting pbr behaviors/expectations here | 13:24 |
| boden | understood, I didn't realize the can of worm lurking here :( | 13:24 |
| mordred | fungi: we should maybe make tagging a relase more explicit | 13:24 |
| fungi | so the information which would have helped you in this case would have come from reading the pbr docs | 13:24 |
| fungi | mordred: yeah, like a sentence or two mentioning and linking to things like pbr documentatin? | 13:25 |
| mordred | yeah | 13:25 |
| mordred | "there are specific requirements on tag format, etc ... please see" | 13:25 |
| boden | IMHO if there was a little mroe info/links on https://docs.openstack.org/infra/manual/creators.html#tagging-a-release in regards to versioning I think that would help... also it might be nice to note that users should test locally; for newbies like myself | 13:26 |
| *** dmsimard has quit IRC | 13:26 | |
| fungi | yep, though also how you test is going to be very language/ecosystem-specific too | 13:27 |
| fungi | for testing javascript artifacts build correctly you'd need completely different instructions than python | 13:27 |
| fungi | i suppose we can simply say that, and leave figuring out how to test a local tag as an exercise for the reader, assuming they're competent in whatever language ecosystem they're programming | 13:28 |
| boden | or WARNING: signed tags can't be deleted, make sure you know what the heck you're doing :) | 13:29 |
| fungi | boden: but yeah, i also confirmed that checking out your 13.0.0.0 tag and building an sdist and a wheel i get 2018.0.0.0rc2.dev48 but after tagging the same commit with 13.0.0 it works as expected | 13:30 |
| mordred | yah - I think we don't need to delete those tags | 13:30 |
| *** dpawlik has quit IRC | 13:30 | |
| mordred | they cease doing anything once a 13.0.0 signed tag is in place | 13:30 |
| mordred | they might look unclean - but such is life | 13:30 |
| *** dmsimard has joined #openstack-infra | 13:31 | |
| *** dansmith is now known as SteelyDan | 13:31 | |
| *** jamesmcarthur has joined #openstack-infra | 13:32 | |
| fungi | boden: oh, also, you were looking at the document on initial project creation, not the one for project maintainers pushing releases on existing projects. you probably wanted https://docs.openstack.org/infra/manual/drivers.html#tagging-a-release | 13:32 |
| fungi | the project creator's guide is merely intended as a walkthrough of the steps involved in starting a new project from scratch | 13:33 |
| fungi | which does include a note about pbr's semver+pep440 version formatting | 13:34 |
| *** efried is now known as fried_rice | 13:35 | |
| fungi | though it is still lacking a warning about our inability to delete unwanted tags effectively, or a recommendation to locally test release artifact generation | 13:35 |
| *** holser_ has quit IRC | 13:35 | |
| fungi | er, i should say, the tagging section in the driver's guide does include a note about pbr's semver+pep440 version formatting | 13:36 |
| fungi | (what i linked to) | 13:36 |
| *** holser_ has joined #openstack-infra | 13:36 | |
| fungi | while what you said you were following was the creator's guide which is a walkthrough for starting new projects | 13:36 |
| boden | fungi: ack, def my fault... I didn't see the latter link you sent until now | 13:39 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Make integration tests works https://review.openstack.org/590478 | 13:39 |
| *** ykarel is now known as ykarel|away | 13:41 | |
| boden | fungi: so I had thought the branch creation was done afterwards via gerrit https://review.openstack.org/#/admin/projects/openstack/vmware-nsx,branches but appears from the doc you sent that's not the case... assuming I get the signed tag can I just create and push the branch after-the-fact? | 13:43 |
| *** larainema has joined #openstack-infra | 13:43 | |
| *** jistr is now known as jistr|call | 13:46 | |
| boden | otherwise I guess I can just bump it up to version 14.0.0 + push again including the branch | 13:52 |
| mordred | boden: you can make the branch after the tag for sure | 13:54 |
| mordred | in fact, you can give the tag as the ref to create the branch from | 13:55 |
| boden | mordred would just just do that via git CLI? git checkout stable/rocky && git push gerrit 13.0.0 | 13:55 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Rework translation templates https://review.openstack.org/589372 | 13:56 |
| AJaeger | frickler: thanks for review, I fixed the typo everywhere ^ | 13:56 |
| AJaeger | any other config-cores for review, please? ^ | 13:56 |
| *** ykarel|away has quit IRC | 13:58 | |
| *** jistr|call is now known as jistr | 14:00 | |
| fungi | boden: i think you want `git push gerrit 13.0.0:stable/rocky` to create a stable/rocky branch from your 13.0.0 tag, but someone else should double-check me on that syntax | 14:00 |
| *** apetrich has quit IRC | 14:00 | |
| *** hongbin_ has joined #openstack-infra | 14:03 | |
| frickler | AJaeger: ah, I didn't notice that it was in the original templates already | 14:04 |
| smcginnis | Any idea why a releasenotes job would get a MODULE FAILURE on "Synchronize files to AFS"? | 14:04 |
| smcginnis | http://logs.openstack.org/5b/5b98bd8a702a8502ce7cb832c74b6a3ee05a33ed/tag/publish-openstack-releasenotes/ef70a94/job-output.txt.gz#_2018-08-10_03_21_54_363897 | 14:04 |
| smcginnis | I thought we've seen that MODULE FAILURE message before when there was a typo in the ansible bits. | 14:05 |
| frickler | smcginnis: http://logs.openstack.org/5b/5b98bd8a702a8502ce7cb832c74b6a3ee05a33ed/tag/publish-openstack-releasenotes/ef70a94/ara-report/result/8ebe070d-650d-461c-bbdc-99bb002790a1/ | 14:05 |
| frickler | not sure about the root cause for that error, though | 14:06 |
| smcginnis | Oh right, I forgot the ARA part would show more details. Thanks frickler. | 14:06 |
| AJaeger | frickler: a pasto - but still good to fix ;) | 14:06 |
| smcginnis | That looks like the issue we saw the other day with possible overlapping rsync operations. Wasn't that what we thought it was dhellmann? | 14:07 |
| frickler | that would at least sound plausible to me | 14:07 |
| dhellmann | smcginnis : that's what that error has been in the past, but I thought the new zuul changes to only run one of those jobs at a time was supposed to eliminate that | 14:08 |
| smcginnis | This would be two cases where we've seen that not to be the case. | 14:08 |
| smcginnis | So there may be something else going on. | 14:08 |
| fungi | unless the same job is being run for multiple projects but publishing into a common root | 14:08 |
| smcginnis | I guess they all would go to /afs/.openstack.org/docs/releasenotes/ | 14:09 |
| smcginnis | Then a project specific subdirectory under that root. | 14:10 |
| *** e0ne has joined #openstack-infra | 14:11 | |
| fungi | : rename "/afs/.openstack.org/docs/releasenotes/sahara/_static/.jquery.js.TqhjvC" -> "_static/jquery.js": No such file or directory (2) | 14:11 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Make integration tests works https://review.openstack.org/590478 | 14:12 |
| fungi | i don't think there should be two jobs fighting over files in /afs/.openstack.org/docs/releasenotes/sahara/_static/ | 14:12 |
| fungi | that looks fairly project-specific | 14:12 |
| smcginnis | Looks like the only failure is publishing the release notes. Should we just re-enqueue that one? | 14:13 |
| *** eharney has quit IRC | 14:13 | |
| fungi | hard to tell whether it's complaining because .jquery.js.TqhjvC is missing or _static/ is missing | 14:13 |
| smcginnis | Something else is missing here, so we should keep an eye on these failures. But they do seem to be transient at least. | 14:13 |
| smcginnis | fungi: Right, I wasn't sure which it meant either. | 14:14 |
| fungi | i don't think we should reenqueue that one since it's a tag ref and presumably already successfully uploaded to pypi | 14:14 |
| fungi | so will fail if we try to reenqueue | 14:14 |
| smcginnis | Oh? The only thing it showed as part of the failed jobs was the run of publish-openstack-releasenotes. Isn't this the post-tagging job? | 14:15 |
| boden | fungi any idea if I can just use gerrit web https://review.openstack.org/#/admin/projects/openstack/vmware-nsx,branches and just create branch name "stable/rocky" and use the 13.0.0 revision number for the "Initial REvision" of the branch? I'm tempted to try, but don't want to munk anything up | 14:16 |
| fungi | smcginnis: it was in "pipeline: tag" per http://logs.openstack.org/5b/5b98bd8a702a8502ce7cb832c74b6a3ee05a33ed/tag/publish-openstack-releasenotes/ef70a94/zuul-info/inventory.yaml (though you can also currently see the pipeline name as part of the log url path) | 14:17 |
| fungi | boden: i think the acl for create ref may not also cover the gerrit webui, but i honestly don't remember. if it lets you, that's perfectly safe | 14:18 |
| smcginnis | fungi: Ah, thanks. | 14:18 |
| fungi | boden: you can definitely put the tag in the revision field there, yes | 14:18 |
| boden | fungi: the tag (e.g. "13.0.0") or the revision number for it ("8076937814554a5ad1df148e7ddf34a336990e86") | 14:19 |
| boden | ? | 14:19 |
| fungi | the tag works fine there | 14:19 |
| boden | ok thanks.. fingers crossed | 14:19 |
| fungi | doesn't have to be a git object sha | 14:19 |
| fungi | smcginnis: dhellmann: also as far as the recent changes which help avoid contention over publication jobs with the supercedent pipeline manager, that's for things running in the post pipeline, not the tag pipeline | 14:20 |
| dhellmann | oh, hrm | 14:21 |
| dhellmann | I wonder if we should set that pipeline up the same way | 14:21 |
| dhellmann | although I don't know why 2 things would be publishing release notes to the same project at the same time | 14:21 |
| fungi | well, that wouldn't help if that's what this is | 14:21 |
| smcginnis | At least that explains it. | 14:21 |
| fungi | it's not like you're pushing two tags for the same project at the same time | 14:22 |
| dhellmann | right | 14:22 |
| dhellmann | I mean, we might, but that's not likely to be what happened here | 14:22 |
| dhellmann | we do sometimes approve 2 tags on different branches | 14:22 |
| fungi | but also the supercedent pipeline manager works by deduplicating multiple queue items | 14:23 |
| fungi | which i don't think you'd want for releases | 14:23 |
| fungi | according to the traceback, rsync was copying from /var/lib/zuul/builds/ef70a94325ff4891a1e325e3d566839b/work/artifacts/ on the executor to /afs/.openstack.org/docs/releasenotes/sahara/ | 14:25 |
| mordred | yeah - we DEFINITELY don't want supercedent for release pipeline | 14:25 |
| mordred | since the job/queue-item is "publish-openstack-releasenotes" | 14:25 |
| fungi | is it possible some other different job was also writing to /afs/.openstack.org/docs/releasenotes/sahara/ at the same time? | 14:26 |
| corvus | fungi: if we delete a tag, i expect it to remain on the mergers, so we would want to delete it from all 19 of them (and the git mirrors, etc) | 14:26 |
| fungi | corvus: yeah, i expected that since zuul v2 did the same | 14:26 |
| mordred | corvus: ++ ... I believe we figured out we do not need to delete it | 14:26 |
| fungi | thanks for confirming | 14:26 |
| *** eharney has joined #openstack-infra | 14:26 | |
| corvus | mordred: thanks... that was my next question. :) i thought i'd go ahead and answer anyway since it seemed important | 14:27 |
| corvus | infra-root: i just +3d https://review.openstack.org/590755 but i want to bring it to your attention as a subtle point about how testing ansible will work. it seems like it could be important information to have in the back of your mind when debugging in the future. | 14:35 |
| openstackgerrit | Javier Peña proposed openstack-infra/puppet-openstackci master: Add AFS mirror support https://review.openstack.org/529376 | 14:36 |
| openstackgerrit | Javier Peña proposed openstack-infra/puppet-openstackci master: Add AFS mirror support for RHEL/CentOS https://review.openstack.org/528739 | 14:36 |
| mordred | corvus: now if only I could get https://review.openstack.org/#/c/590478 to pass tests on centos ... | 14:36 |
| *** e0ne has quit IRC | 14:39 | |
| fungi | what's the upshot of adding the roles dir in system-config to the roles path? just that we can refer to those roles in the jobs? | 14:40 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Make integration tests works https://review.openstack.org/590478 | 14:40 |
| corvus | fungi: i think it makes the integration test configuration match the production configuration on bridge | 14:40 |
| mordred | yup | 14:41 |
| mordred | also | 14:41 |
| fungi | neat, that sounds useful | 14:41 |
| mordred | it's a stepping stone to ... https://review.openstack.org/#/c/590753/ - and would allow us to put the afs/krb roles ianw has been working on into there and use them for our test jobs AND in production | 14:41 |
| mordred | so - yes, what you said | 14:41 |
| *** rpioso|afk is now known as rpioso | 14:41 | |
| *** bobh has joined #openstack-infra | 14:43 | |
| *** ccamacho has quit IRC | 14:47 | |
| openstackgerrit | Merged openstack-infra/puppet-openstack_infra_spec_helper master: Run ansible base playbook for puppet tests https://review.openstack.org/588326 | 14:47 |
| openstackgerrit | Merged openstack-infra/puppet-openstack_infra_spec_helper master: Add roles dir in system-config to roles path https://review.openstack.org/590755 | 14:47 |
| openstackgerrit | Merged openstack-infra/zuul master: web: fix job required-projects list https://review.openstack.org/590594 | 14:47 |
| *** bobh has quit IRC | 14:49 | |
| clarkb | fungi: catcing up on tag deletion is plan to delete 2018* and 13* tags? | 14:51 |
| mordred | clarkb: nope | 14:51 |
| mordred | clarkb: no need to delete | 14:51 |
| clarkb | mordred: just superceding them with new tags or? | 14:52 |
| mordred | clarkb: adding the correct tag fixes the issue | 14:52 |
| mordred | yup | 14:52 |
| clarkb | neat | 14:52 |
| fungi | yeah, pushing a new correct tag was all they needed in the end | 14:52 |
| dhellmann | regarding that rsync error in the sahara release notes job, https://review.openstack.org/#/c/590619/ was approved a few hours ago and would also have triggered a release notes build, so that may have been the source of the second job that caused the race condition | 14:56 |
| fungi | aha, so yes it's possible we want to move release notes publication to release-post or something? | 14:57 |
| fungi | rather that doing it from the tag pipeline | 14:57 |
| dhellmann | maybe? it isn't always a release thing, but it might be worth doing that just to serialize them | 14:57 |
| *** hamzy_ is now known as hamzy | 14:57 | |
| fungi | or even just to the regular post pipeline. somewhere with the supercedent manager | 14:58 |
| clarkb | I think we decided its mostly a non issue too right? maybe we can filter that as a failure condition? | 14:59 |
| clarkb | or scope the rsync to unique tmp dirs? | 14:59 |
| *** Bhujay has quit IRC | 14:59 | |
| *** Bhujay has joined #openstack-infra | 15:00 | |
| dhellmann | I'd be fine with moving the job into a pipeline that uses the supercedent manager. Does it matter which one? The job does run sometimes just when a regular patch merges. | 15:00 |
| fungi | yeah i don't think the job running when a regular patch merges would be impacted by that, since it's already happening from another pipeline but maybe having them be the same pipeline would help (so regular post?) | 15:01 |
| dhellmann | hmm | 15:02 |
| fungi | mmm, though the ref-updated event in that case would be the one for the releases repo | 15:02 |
| fungi | not for the individual projects | 15:02 |
| dhellmann | the pipeline doesn't have anything to do with what triggers the job, right? it's just how it's queued up to run? | 15:02 |
| fungi | basically we have an idempotent publication task we want to take advantage of supercedent pipeline manager behavior, but triggered from a tag push event | 15:03 |
| *** tosky has quit IRC | 15:03 | |
| *** e0ne has joined #openstack-infra | 15:03 | |
| fungi | the tag, pre-release and release pipelines are triggered by a ref-updated event with a ref pattern match of ^refs/tags/something | 15:04 |
| openstackgerrit | Javier Peña proposed openstack-infra/puppet-openstackci master: Add AFS mirror support https://review.openstack.org/529376 | 15:05 |
| fungi | the post pipeline is triggered by a ref-updated event matching ^refs/heads/something | 15:05 |
| *** dpawlik has joined #openstack-infra | 15:05 | |
| dhellmann | so it sounds like we need it in both pipelines | 15:06 |
| fungi | well, i think we have it in post and tag pipelines currently | 15:06 |
| dhellmann | yes, we do | 15:06 |
| fungi | post covers when changes merge to change the release notes, tag covers when tags are pushed adjusting what versions exist | 15:06 |
| dhellmann | and we need it to run when something is tagged and we need it to run when changes to notes files are merged | 15:06 |
| dhellmann | right | 15:06 |
| dhellmann | hence my question about whether the pipeline was tied to the trigger | 15:07 |
| dhellmann | I couldn't remember if that was defined in the job or the pipeline | 15:07 |
| openstackgerrit | Javier Peña proposed openstack-infra/puppet-openstackci master: Add AFS mirror support for RHEL/CentOS https://review.openstack.org/528739 | 15:07 |
| fungi | it's almost like we want a different pipeline entirely which uses the supercedent pipeline manager and triggers on all ref-updated events regardless of ref pattern | 15:07 |
| fungi | corvus: ^ you might find this an interesting puzzle | 15:08 |
| dhellmann | the way we built the release notes was special. I think I got too clever. | 15:08 |
| fungi | i don't think it's all that special | 15:08 |
| dhellmann | the other thing that is special about it is that it always checks out master to run the build, regardless of the commit that actually triggered the job | 15:09 |
| fungi | we also have projects which want to update their documentation both when changes merge (to update branch tip content which documents the next release) and on tags (so that version numbers can be corrected in published docs) | 15:09 |
| *** dpawlik has quit IRC | 15:09 | |
| dhellmann | ah, true | 15:09 |
| fungi | we've sort of punted i the past and told projects to do one or the other | 15:10 |
| fungi | for various logistical reasons | 15:10 |
| dhellmann | so if we create an "any-ref" pipeline as you describe we could put the release notes job into it but we could also put the doc jobs into it | 15:10 |
| *** e0ne has quit IRC | 15:10 | |
| fungi | but i think the use case is still a valid one | 15:10 |
| fungi | potentially. that's what i'm brainstorming at least | 15:11 |
| dhellmann | I like it. | 15:11 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Make integration tests works https://review.openstack.org/590478 | 15:11 |
| mordred | corvus, fungi, clarkb: ^^ the most recent change to that makes me want to consider making bindep files instead of dealing with package lists in ansible variables | 15:12 |
| mordred | I don't think it's urgent right now - but I think it might be easier to maintain long-term | 15:12 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Add nodepool info to test-emit-job-header https://review.openstack.org/557947 | 15:15 |
| clarkb | mordred: fwiw I'm kind of amazed that puppet and ansible and friends don't all have a bindep like thing to solve this problem | 15:16 |
| clarkb | I guess most shops are single distro | 15:16 |
| clarkb | and now with containers you just encode it into a dockerfile | 15:16 |
| clarkb | mordred: re https://review.openstack.org/#/c/590478/9/playbooks/roles/base-server/tasks/main.yaml what does the paths: 'vars' do there? | 15:18 |
| clarkb | is that a magical input to the first found lookup? | 15:20 |
| corvus | fungi: yeah, i think if you want something to run on post and tag and be assured not to run simultaneously, a supercedent any is what you need -- except, i think supercedent collapses items by project-branch, which could cause you to end up with different queues for something that entered via refs/heads and something that entered via refs/tags. so we may need a variant of supercedent which | 15:22 |
| corvus | collapses by project only. | 15:22 |
| corvus | dhellmann: ^ | 15:22 |
| *** hwoarang has quit IRC | 15:23 | |
| corvus | clarkb: yes, "paths" is a parameter to the first_found lookup | 15:23 |
| corvus | clarkb: means "look in these paths" | 15:23 |
| fungi | corvus: great point. it's not even currently safe in post since the job publishes to the same paths regardless of the branch which triggered it | 15:25 |
| fungi | so i agree, a project-supercedent pipeline manager implementation in zuul would be required if we want that | 15:26 |
| fungi | could we achieve something similar with a mutex? | 15:27 |
| * fungi can't remember if zuul has support for a project+job mutex or if it's only explicitly by job | 15:28 | |
| corvus | fungi: yeah, but you'd have to make a mutex for each project (or otherwise share a global mutex) | 15:28 |
| corvus | fungi: well, mutexes are global singletons -- then you attach them to jobs. you could do that at the job definition (so all such jobs and their descendents use that single mutex), or at the project-pipeline level (so only that project's run of the job uses it) | 15:29 |
| aspiers | are there any docs on use of publish-openstack-sphinx-docs? I'm probably looking in the wrong places | 15:29 |
| corvus | project-collapsing supercedent still sounds like the winner :) | 15:30 |
| *** mriedem has joined #openstack-infra | 15:30 | |
| aspiers | I'd like to start publishing the likes of http://logs.openstack.org/06/590406/7/check/build-openstack-sphinx-docs/8b8ade4/html/ on docs.openstack.org, if that's appropriate | 15:31 |
| fungi | yeah, seems like it would be complicated to apply project-specific mutexes to all the projects since they're getting the job from a template | 15:31 |
| corvus | aspiers: https://docs.openstack.org/infra/openstack-zuul-jobs/project-templates.html#project_template-build-openstack-sphinx-docs | 15:31 |
| AJaeger | aspiers: if you have an official repo, just use it ;) | 15:32 |
| aspiers | corvus: thanks! | 15:32 |
| aspiers | AJaeger: I'm guessing self-healing-sig is official by virtue of being a SIG, but I'm not 100% sure | 15:33 |
| AJaeger | aspiers: it is - listed in governance repo under reference/sigs-repos.yaml | 15:33 |
| fungi | aspiers: make sure you list that repo in the sigs repos list (i think reference/sigs-repos.yaml in openstack/governance is the right place) | 15:34 |
| AJaeger | fungi: he did ^ | 15:34 |
| aspiers | yes, I know it's there - just not sure what that means from a governance perspective | 15:34 |
| fungi | just a means of tracking | 15:35 |
| AJaeger | aspiers: it's a repo of an official project/SIG - and only official ones can use docs.o.o | 15:35 |
| aspiers | so I can just change the .zuul.yaml from build-openstack-sphinx-docs to publish-openstack-sphinx-docs? | 15:35 |
| fungi | i still wonder whether that info should move to the sigs.yaml in openstack/governance-sigs at some point | 15:35 |
| AJaeger | aspiers: yes, just do it | 15:35 |
| aspiers | ok thanks | 15:35 |
| AJaeger | aspiers: and afterwards sent a change to openstack-manuals/www/... to add to index file. That is covered in the docs contributor guide | 15:35 |
| aspiers | got it! | 15:36 |
| AJaeger | I mean: after you published content - we only link to content thta exists ;) | 15:36 |
| aspiers | sure :) | 15:36 |
| corvus | AJaeger: maybe we should link the project-template doc to the contributor guide? | 15:36 |
| *** Bhujay has quit IRC | 15:36 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Make integration tests works https://review.openstack.org/590478 | 15:36 |
| *** Bhujay has joined #openstack-infra | 15:37 | |
| *** rajinir has joined #openstack-infra | 15:37 | |
| corvus | mordred: that sounds like a great idea ^ :) | 15:37 |
| AJaeger | corvus: so, link to https://docs.openstack.org/doc-contrib-guide/doc-index.html ? | 15:37 |
| corvus | AJaeger: yeah? that way someone who finds all this by way of the project-template can get there? | 15:38 |
| mordred | corvus: bah. working tests are so legacy | 15:38 |
| * AJaeger works on a quick change... | 15:38 | |
| AJaeger | config-core, could you review https://review.openstack.org/589372 - and https://review.openstack.org/590665 (will need recheck once the first one is in), please? The first check will make changes for translations at release time much easier for us... | 15:38 |
| mordred | corvus: in cloud native, everything just magically works even if it's broken | 15:38 |
| mordred | corvus: at least as long as you don't touch files in any way | 15:39 |
| corvus | mordred: is npm cloud native? because i've never been able to run "npm build" two days in a row and have it work without having to reinstall everything. | 15:41 |
| mordred | corvus: it must be! | 15:41 |
| clarkb | corvus: mordred: nodeenv was my solution for that | 15:42 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: enhance publish-openstack-sphinx-docs description https://review.openstack.org/590808 | 15:42 |
| AJaeger | corvus, aspiers ^ | 15:42 |
| clarkb | then I just make a new one | 15:42 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/infra-manual master: Clarify inability to delete tags https://review.openstack.org/590809 | 15:44 |
| fungi | boden: would that ^ have helped? | 15:44 |
| mordred | clarkb, corvus: I mean, actually I haven't had that problem because yarn has the yarn.lock file ... so I was mostly taking the corvus comment as snark. if that's an actual problem I'd love to learn more about it | 15:44 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/infra-manual master: Clarify inability to delete tags https://review.openstack.org/590809 | 15:45 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Add nodepool info to test-emit-job-header https://review.openstack.org/557947 | 15:46 |
| clarkb | mordred: I've not tried things with lock files but that would likely help. The problems tend to be unpinned deps that are incompatible with either other libraries or older node | 15:46 |
| *** yamahata has joined #openstack-infra | 15:46 | |
| clarkb | mordred: so you build a new one which gets you newer node and newer other libs | 15:47 |
| mordred | oh - yeah - using yarn to install there isn't really unpinned deps anymore | 15:48 |
| *** rpittau has quit IRC | 15:48 | |
| mordred | but that's easier for zuul-web where we control it - and harder for etherpad-lite | 15:48 |
| mordred | clarkb: oh wow. I just looked at the etherpad-lite repo | 15:49 |
| mordred | I will not make that mistake again | 15:49 |
| *** sthussey has joined #openstack-infra | 15:49 | |
| aspiers | AJaeger: is https://docs.openstack.org/infra/manual/creators.html#add-links-to-your-project-documentation a poor duplicate of https://docs.openstack.org/doc-contrib-guide/doc-index.html ? | 15:51 |
| aspiers | perhaps the former should just redirect to the latter? | 15:51 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Add nodepool info to test-emit-job-header https://review.openstack.org/557947 | 15:51 |
| AJaeger | aspiers: it should link to the same page | 15:53 |
| AJaeger | aspiers: and it does, both link to the same page... | 15:53 |
| *** ginopc has quit IRC | 15:56 | |
| aspiers | AJaeger: yes but they contain different info | 15:56 |
| aspiers | https://docs.openstack.org/doc-contrib-guide/doc-index.html has a lot more info | 15:56 |
| AJaeger | aspiers: both link to the same page for the template. But if you want to align them more, please send a change. Happy to review! | 15:57 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Add nodepool info to test-emit-job-header https://review.openstack.org/557947 | 15:57 |
| aspiers | it just generally feels wrong to have two sections in two manuals covering the same topic | 15:57 |
| aspiers | that's a DRY violation just waiting for someone to modify one place but not the other | 15:58 |
| aspiers | it's not so much about the current disparity as the potential for endless divergence | 15:58 |
| openstackgerrit | Akihiro Motoki proposed openstack-infra/project-config master: Update horizon grafana https://review.openstack.org/590813 | 16:03 |
| * fungi is gonna go grab some lunch, back soon | 16:03 | |
| mordred | clarkb: do we have any hosts, other than puppetmaster, that are still trusty? | 16:07 |
| clarkb | mordred: yes, quite a few actually :/ floating in the back of my head is the need to rebuild them but wanting to be slushy for release means now is not greatest time | 16:07 |
| mordred | ++ | 16:08 |
| mordred | kk. so fighting through my trusty gate issues is worthwhile | 16:08 |
| clarkb | mordred: likely | 16:08 |
| *** fried_rice is now known as fried_rolls | 16:08 | |
| clarkb | I'm thinking post PTG we can plan to work through those again | 16:08 |
| clarkb | (sorting out the config mgmt stuff first at ptg and otherwise seems useful as we may do conversions of config management too) | 16:09 |
| clarkb | mordred: we got through quite a few service last winter. Then ran up agains tthose that needed work. The upside to cmurphy's puppet 4 stuff is that we get testing on xenial so the config management issues at least are being worked out | 16:12 |
| clarkb | review.o.o and review-dev.o.o are also done leaving us with things like lists and ask and similar as the likely "difficult" nodes | 16:13 |
| clarkb | hosts like nodepool.o.o can hopefully be deleted in favor of new zk cluster, etherpad and similar srevices just needed the puppet updates I think | 16:14 |
| *** Bhujay has quit IRC | 16:15 | |
| aspiers | hrm http://zuul.openstack.org/stream.html?uuid=1a2c0b80c3e34b5bb0f62a7342038660&logfile=console.log | 16:16 |
| aspiers | "msg": "SSH Error: data could not be sent to remote host \"147.75.38.184\". Make sure this host can be reached over ssh" | 16:17 |
| aspiers | IIRC it's retrying now | 16:17 |
| clarkb | aspiers: if that happens in the pre-run playbook it should retry on a newly scheduled instance | 16:17 |
| clarkb | that looks like a packethost IP which I thought John tracked back to a bad hypervisor which was being replaced | 16:18 |
| aspiers | clarkb: thanks. watching to see if the retry succeeds | 16:19 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Make integration tests works https://review.openstack.org/590478 | 16:21 |
| *** kei-ichi has quit IRC | 16:22 | |
| *** jpena is now known as jpena|off | 16:22 | |
| *** mriedem has quit IRC | 16:27 | |
| mordred | clarkb: ++ | 16:27 |
| *** fabo has quit IRC | 16:31 | |
| aspiers | clarkb: it finished ok, although I'm not sure where the resulting docs are published to | 16:34 |
| aspiers | https://docs.openstack.org/self-healing-sig/ is still 404 | 16:34 |
| clarkb | aspiers: have a link to the job logs? | 16:35 |
| aspiers | http://logs.openstack.org/71/71871b9922c135fcd9c0ce68712c118fea044e4f/post/publish-openstack-sphinx-docs/3abb7df/ara-report/ | 16:35 |
| *** jpich has quit IRC | 16:35 | |
| *** armax has joined #openstack-infra | 16:35 | |
| clarkb | we publish to afs which is on a 5 minute publish cycle iirc | 16:35 |
| clarkb | which should've happened about a minute ago | 16:36 |
| aspiers | ahah | 16:36 |
| clarkb | yup docs are there now | 16:36 |
| aspiers | yup! | 16:36 |
| aspiers | awesome :) | 16:36 |
| clarkb | mordred: speaking of fun upgrades ^ afs will be a dance. The good news is we can do that without downtime assuming no unforeseen circumstances | 16:37 |
| corvus | mumble mumble in place manual upgrade mumble | 16:38 |
| mordred | clarkb: also - ianw has the afs/krb modules - so we might not be far-off from getting the afs servers to just be pure-ansible | 16:38 |
| mordred | s/modules/roles/ | 16:38 |
| openstackgerrit | Merged openstack-infra/zuul master: Cache branches in connections/sources https://review.openstack.org/589975 | 16:39 |
| *** diablo_rojo has joined #openstack-infra | 16:41 | |
| clarkb | infra-root the stack at https://review.openstack.org/#/c/590023/ updates puppetry for etherpad, if we get that stuff in I will work on an upgrade for etherpad-dev | 16:42 |
| *** holser_ has quit IRC | 16:43 | |
| clarkb | (may as well keep picking them off as we can) | 16:43 |
| mordred | clarkb: done | 16:44 |
| *** panda|ruck is now known as panda|ruck|off | 16:48 | |
| *** rmcall has quit IRC | 16:50 | |
| pabelanger | clarkb: did john say anything about stuck deleting nodes? Wonder if we need to have an admin set the state again to something besides deleting | 16:51 |
| pabelanger | for packethost | 16:51 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Tolerate missing project https://review.openstack.org/579872 | 16:52 |
| clarkb | pabelanger: he was looking at why they were unhappy. That was the neutron file limit problem I think | 16:56 |
| openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Add digit support to etherpad servers https://review.openstack.org/591017 | 16:59 |
| clarkb | now does that change conflict with anyone else's changes to show me someone else was already working on that and I should revie wtheir stuff instead | 16:59 |
| *** derekh has quit IRC | 17:01 | |
| *** dtantsur is now known as dtantsur|afk | 17:01 | |
| clarkb | while I wait for etherpad stuff to merge, it looks like ethercalc may be in an upgradeable spot already. It isn't a heavily used service, any objection to me trying to spin up a new 02 instance on xenial for it? | 17:03 |
| *** jamesmcarthur has quit IRC | 17:03 | |
| *** jamesmcarthur has joined #openstack-infra | 17:04 | |
| clarkb | mordred: ^ and I'm going to spin that up from puppetmaster right? | 17:05 |
| clarkb | mordred: or should I run launch node from bridge at this point? | 17:05 |
| *** jcoufal has quit IRC | 17:06 | |
| mordred | clarkb: that's a great question - I think from puppetmaster is a better choice - although hang on one sec | 17:07 |
| mordred | clarkb: ok. it should just work | 17:08 |
| mordred | the one thing to keep an eye out for is make_swap, which now calls python3 explicitly | 17:08 |
| clarkb | mordred: it being using bridge for launch node for puppet nodes? | 17:09 |
| mordred | but I think python3 is already installed on xenial, so it should still work- if not, we might need to make that smarter | 17:09 |
| mordred | clarkb: oh - sorry, run launch_node.py from puppetmaster | 17:09 |
| *** trown is now known as trown|lunch | 17:09 | |
| mordred | we haven't tested running puppet on nodes from bridge yet (hopefully very soon, but we havne't done it yet) | 17:09 |
| clarkb | oh hrm https://review.openstack.org/#/c/527144/ never merged and the noop tests don't cover the things it fixes | 17:10 |
| clarkb | I guess I need to work on reviews first | 17:10 |
| *** e0ne has joined #openstack-infra | 17:11 | |
| *** jamesmcarthur has quit IRC | 17:11 | |
| mordred | zomg | 17:12 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Make integration tests works https://review.openstack.org/590478 | 17:12 |
| mordred | clarkb: you're going to enjoy the diff between PS11 and PS12 there ^^ :) | 17:13 |
| clarkb | mordred: ha | 17:17 |
| boden | we recently updated our tempest project to publish to https://pypi.org/project/vmware-nsx-tempest-plugin/ instead of https://pypi.org/project/vmware-nsx-tempest/ as this matches our project setup/pkgs.. is there someway to delete the old https://pypi.org/project/vmware-nsx-tempest/ PYPI project as its not used and adds confusion | 17:18 |
| boden | the person who created/owns https://pypi.org/project/vmware-nsx-tempest/ is no long reachable, but this proj has openstackci as a maintainer | 17:19 |
| *** gyee has joined #openstack-infra | 17:21 | |
| clarkb | boden: generally, I think its considered bad manners to fully delete things from pypi because then anyone depending on that would be sad | 17:22 |
| *** gfidente has quit IRC | 17:22 | |
| clarkb | boden: would it make sense to push a release that maybe points at the new location as a dependency and also updates the readme docs for the old one to point at new one? | 17:22 |
| *** jamesmcarthur has joined #openstack-infra | 17:22 | |
| clarkb | mordred: ^ how has sdk/shade/etc handled this sort of thing | 17:22 |
| clarkb | as a heads up I'm approving the ethercalc series of changes | 17:23 |
| mordred | like that | 17:23 |
| boden | clarkb as far as I know no one is depending on it, but its possible I suppose... I can push a release with new location if needed | 17:23 |
| mordred | clarkb: it's also how we did the quantum/neutron migration | 17:23 |
| *** jcoufal has joined #openstack-infra | 17:24 | |
| mordred | we made a release of python-quantumclient that just had a dependency on python-neutronclient | 17:24 |
| clarkb | frickler: ianw cmurphy I notice though that other people have been thinking about this far more than me. I will defer on booting a new instance if someone else intended on doing that, but will work to get the stack merged and check the existing instance is still happy after for today | 17:24 |
| mordred | clarkb: ++ | 17:24 |
| *** yamahata has quit IRC | 17:25 | |
| *** eharney_ has joined #openstack-infra | 17:30 | |
| *** eharney has quit IRC | 17:33 | |
| *** amoralej is now known as amoralej|off | 17:34 | |
| *** jamesmcarthur has quit IRC | 17:36 | |
| *** jcoufal has quit IRC | 17:39 | |
| *** jamesmcarthur has joined #openstack-infra | 17:42 | |
| AJaeger | corvus, config-core, can we merge https://review.openstack.org/#/c/557946/1 now? Not sure why we did not merge it last time... | 17:43 |
| AJaeger | I see https://review.openstack.org/#/c/557947/7 on top of it just approved... | 17:43 |
| pabelanger | I'm ready for testing, it will allow us to first do https://review.openstack.org/557948/ | 17:44 |
| pabelanger | but also can wait unitl next week | 17:44 |
| clarkb | cmurphy: can you check the comment on https://review.openstack.org/#/c/585292/3 I wnat to make sure I didn't misparse that | 17:45 |
| AJaeger | config-core, could you review https://review.openstack.org/589372 - and https://review.openstack.org/590665 (will need recheck once the first one is in), please? The first check will make changes for translations at release time much easier for us... | 17:46 |
| *** jamesmcarthur has quit IRC | 17:47 | |
| clarkb | AJaeger: I'll take a look once through the ethercalc stack | 17:47 |
| *** jamesmcarthur has joined #openstack-infra | 17:47 | |
| AJaeger | clarkb: thanks | 17:47 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: enhance publish-openstack-sphinx-docs description https://review.openstack.org/590808 | 17:49 |
| mordred | infra-root: I need to AFK for a little bit to go pick up some scuba tanks | 17:49 |
| clarkb | mordred: watch your toes | 17:49 |
| clarkb | cmurphy: also comment on https://review.openstack.org/#/c/590489/1 | 17:50 |
| *** priteau has quit IRC | 17:52 | |
| *** xarses_ has joined #openstack-infra | 17:53 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove double docs job https://review.openstack.org/591026 | 17:54 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Move openstack-zuul-jobs completely in-repo https://review.openstack.org/591027 | 17:55 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove double docs job https://review.openstack.org/591026 | 17:55 |
| AJaeger | config-core, we run an extra docs jobs on openstack-zuul-jobs, the two changes above clean this up ^ | 17:56 |
| AJaeger | clarkb: thanks for reviewing the translation changes | 17:57 |
| clarkb | AJaeger: no problem | 17:57 |
| *** yamahata has joined #openstack-infra | 17:58 | |
| AJaeger | corvus, pabelanger:I'll go ahead with https://review.openstack.org/#/c/557946 now | 17:58 |
| openstackgerrit | Merged openstack-infra/project-config master: Update horizon grafana https://review.openstack.org/590813 | 17:58 |
| *** yamahata has quit IRC | 18:02 | |
| *** xarses_ has quit IRC | 18:03 | |
| *** yamahata has joined #openstack-infra | 18:03 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Rework translation templates https://review.openstack.org/589372 | 18:08 |
| *** ianychoi has quit IRC | 18:10 | |
| *** ianychoi has joined #openstack-infra | 18:11 | |
| * fungi returns to discover that he should have decided to take the afternoon for diving | 18:12 | |
| mordred | fungi: that's always the right choice | 18:13 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remove old inactive users https://review.openstack.org/588553 | 18:13 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Set mgmt_hieradata variable for bridge.openstack.org https://review.openstack.org/588627 | 18:13 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add exim role to base playbook https://review.openstack.org/589964 | 18:14 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add system-config to roles path https://review.openstack.org/590752 | 18:14 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Move roles into roles dir https://review.openstack.org/590753 | 18:14 |
| mordred | infra-root: ok that ^^ is finally green (or at least the first patch is) | 18:15 |
| mordred | they should all be safe to land up until the lsat one which I marked WIP | 18:15 |
| *** trown|lunch is now known as trown | 18:15 | |
| mordred | I will NOW go AFK to pick up scuba tanks, whilst being careful of my toes :) | 18:15 |
| fungi | instead i went to a beach bar and had a dolphin sandwich that was listed on the menu as mahi-mahi (rage rage) | 18:15 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use translations-jobs-master-stable https://review.openstack.org/591032 | 18:16 |
| fungi | the atlantic ocean has no such fish as mahi-mahi. we have dolphin | 18:16 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add test-emit-job-header roles https://review.openstack.org/557946 | 18:16 |
| * fungi swears at tourists some more | 18:16 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add nodepool info to test-emit-job-header https://review.openstack.org/557947 | 18:16 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use translations-jobs-master-stable https://review.openstack.org/591032 | 18:18 |
| AJaeger | dhellmann: could you review https://review.openstack.org/590666 , please? The dependency is in now... | 18:23 |
| openstackgerrit | Merged openstack-infra/project-config master: Use new translation jobs templates https://review.openstack.org/590665 | 18:24 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use translation-jobs-master-only https://review.openstack.org/591033 | 18:24 |
| dhellmann | AJaeger : thanks | 18:24 |
| dhellmann | approved | 18:24 |
| AJaeger | thanks! | 18:24 |
| dhellmann | AJaeger : did you see my comment on that monasca patch? | 18:24 |
| AJaeger | dhellmann: I see that 588842 (the olso jobs) has a merge conflict. Do you mind if I push the translation job change in and then rebase the oslo job on top? | 18:25 |
| dhellmann | AJaeger : please go right ahead | 18:25 |
| dhellmann | AJaeger : I proposed an approach for https://review.openstack.org/#/c/590387/ to unblock them for now. Let me know what you think. | 18:26 |
| AJaeger | dhellmann: yes, I did; thanks. I was this morning in a hurry and couldn't dig into monasca, just thought they do something really odd... | 18:26 |
| AJaeger | checking... | 18:26 |
| dhellmann | oh, that's the monasca patch, in case you didn't have the link handy | 18:26 |
| AJaeger | dhellmann: yep, read it. | 18:26 |
| dhellmann | is that approach ok with you? | 18:27 |
| AJaeger | dhellmann: yes, it is. commented | 18:28 |
| *** apetrich has joined #openstack-infra | 18:29 | |
| dhellmann | great, thanks for pulling me in on that one | 18:30 |
| * AJaeger thanks dhellmann | 18:30 | |
| *** roman_g has quit IRC | 18:32 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use translation-jobs-master-only https://review.openstack.org/591033 | 18:34 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: remove job settings for Oslo repositories https://review.openstack.org/588842 | 18:34 |
| AJaeger | dhellmann: wnat to WIP this ^ yourself again - or shall I? | 18:35 |
| dhellmann | on it | 18:35 |
| AJaeger | thanks | 18:35 |
| AJaeger | config-core, could you review https://review.openstack.org/#/c/591032 and https://review.openstack.org/#/c/591033, please? I like to get these quickly in to avoid conflicts and rebases | 18:37 |
| pabelanger | infra-root: maybe sometime next week we could either consider deleting the /var/lib/zuul/executor-git directories on zuul-executors, so they are recloned with non 0777 permissions directories, or doing a mass chmod on them. In an effort to fix ansible warning in tox linter jobs: | 18:38 |
| pabelanger | [WARNING] Ansible is in a world writable directory (/home/zuul/src/git.openstack.org/openstack/ansible-role-nodepool), ignoring it as an ansible.cfg source. | 18:38 |
| pabelanger | http://logs.openstack.org/14/584614/17/check/tox-linters/c584cbe/job-output.txt.gz#_2018-07-28_02_15_24_873989 for more info | 18:39 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use translation-jobs-master-only https://review.openstack.org/591033 | 18:39 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: remove job settings for Oslo repositories https://review.openstack.org/588842 | 18:39 |
| fungi | pabelanger: sounds good. i could just mass-chmod them now if that's safe, but maybe friday during release candidate week is poor timing | 18:39 |
| AJaeger | dhellmann: sorry, found a problem and needed a rebase | 18:39 |
| dhellmann | AJaeger : got it | 18:40 |
| * dhellmann goes afk for a bit | 18:40 | |
| pabelanger | fungi: agree, after release week and non-friday :) | 18:40 |
| pabelanger | err, RC week | 18:40 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: enhance publish-openstack-sphinx-docs description https://review.openstack.org/590808 | 18:44 |
| AJaeger | aspiers, dhellmann, updated the descriptions for docs templates ^ | 18:44 |
| *** apetrich has quit IRC | 18:46 | |
| *** fried_rolls is now known as fried_rice | 18:47 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Setup missing translation jobs for neutron-lbaas-dashboard https://review.openstack.org/590364 | 18:48 |
| *** diablo_rojo has quit IRC | 18:50 | |
| *** eharney_ is now known as eharney | 18:52 | |
| *** diablo_rojo has joined #openstack-infra | 18:54 | |
| dhellmann | AJaeger : I wonder if we want to make that job fail if it can't find the repo name in the list in the governance repo | 19:02 |
| dhellmann | maybe that's too pedantic | 19:03 |
| dhellmann | let's wait and see if it's a problem | 19:03 |
| AJaeger | config-core, could you review https://review.openstack.org/#/c/591032 , https://review.openstack.org/#/c/591033 and https://review.openstack.org/590364, please? I like to get these quickly in to avoid conflicts and rebases | 19:03 |
| AJaeger | dhellmann: let's wait... | 19:04 |
| dhellmann | yeah | 19:04 |
| clarkb | infra-root can we get reviews on https://review.openstack.org/#/c/590478/12 to fix our config management tests? All the puppe tthings I tried approving have failed on the fixes there | 19:08 |
| clarkb | AJaeger: I'm looking at your list of changes now | 19:08 |
| AJaeger | thanks, clarkb - and have a great weekend. I'm off now... | 19:09 |
| fungi | enjoy your weekend, AJaeger! | 19:09 |
| *** jamesmcarthur has quit IRC | 19:18 | |
| *** pcaruana has quit IRC | 19:19 | |
| *** dpawlik has joined #openstack-infra | 19:20 | |
| *** kaiokmo has quit IRC | 19:22 | |
| *** dpawlik has quit IRC | 19:24 | |
| *** kaiokmo has joined #openstack-infra | 19:24 | |
| *** kaiokmo has quit IRC | 19:25 | |
| openstackgerrit | Tim Burke proposed openstack-infra/project-config master: Drop legacy-swift*-dsvm-functional-identity-v3-only jobs https://review.openstack.org/591064 | 19:27 |
| *** diablo_rojo has quit IRC | 19:28 | |
| openstackgerrit | Merged openstack-infra/system-config master: Make integration tests works https://review.openstack.org/590478 | 19:30 |
| *** jamesmcarthur has joined #openstack-infra | 19:33 | |
| *** jamesmcarthur has quit IRC | 19:35 | |
| *** sthussey has quit IRC | 19:39 | |
| *** jamesmcarthur has joined #openstack-infra | 19:40 | |
| *** eharney has quit IRC | 19:45 | |
| *** priteau has joined #openstack-infra | 19:47 | |
| *** e0ne has quit IRC | 19:51 | |
| *** xe1gyq has quit IRC | 19:55 | |
| *** eharney has joined #openstack-infra | 19:57 | |
| *** jamesmcarthur has quit IRC | 19:58 | |
| mordred | clarkb, corvus, fungi, pabelanger: can I interest you in a nice shiny https://review.openstack.org/#/c/588553 ? | 19:58 |
| clarkb | I thought I had +2'd that when you asked about the comment the other day guess I didn't | 19:59 |
| *** boden has quit IRC | 19:59 | |
| *** jamesmcarthur has joined #openstack-infra | 20:00 | |
| pabelanger | +2 | 20:06 |
| corvus | mordred: wfm, thx | 20:06 |
| mordred | clarkb: oh - I had a thought about the puppet 4 parser thing and the new groups inventory on bridge .. | 20:06 |
| mordred | corvus: \o/ | 20:06 |
| *** ansmith has quit IRC | 20:06 | |
| cmurphy | clarkb: i'm gonna double check https://review.openstack.org/585292 tomorrow before beer but i'm pretty sure you're wrong | 20:08 |
| mordred | I'm not sure how to feel about that. neither cmurphy or clarkb are ever wrong | 20:09 |
| clarkb | cmurphy: damn | 20:09 |
| cmurphy | mordred: i know it's confusing | 20:09 |
| mordred | cmurphy: I'm completely questioning reality now | 20:09 |
| cmurphy | i've had a couple beers though so clarkb has an edge | 20:09 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/puppet-ethercalc master: Use versioncmp function https://review.openstack.org/590489 | 20:10 |
| clarkb | cmurphy: fwiw I always do get confused about what item is which when symlinks are involved | 20:10 |
| clarkb | I read `man ln` every time I use it | 20:10 |
| cmurphy | clarkb: same here | 20:10 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add exim role to base playbook https://review.openstack.org/589964 | 20:10 |
| mordred | corvus: if you have a sec ^^ I just updated that slightly | 20:10 |
| cmurphy | you would kind of think a two letter command would be easier to use | 20:10 |
| mordred | nope | 20:11 |
| clarkb | cmurphy: but my specific concern here is that on existing server we have a dir with content already but we are replacing it with a symlink to the location that newer npm will install it to | 20:11 |
| mordred | totally hard | 20:11 |
| clarkb | cmurphy: which doesn't exist yet | 20:11 |
| clarkb | because existing server uses old npm | 20:11 |
| openstackgerrit | Merged openstack-infra/puppet-ethercalc master: Update ethercalc to work with Ubuntu Xenial https://review.openstack.org/527144 | 20:11 |
| cmurphy | hmmm yeah this is why nodejs is scary | 20:11 |
| *** priteau has quit IRC | 20:12 | |
| *** diablo_rojo has joined #openstack-infra | 20:13 | |
| cmurphy | clarkb: it's doing ln -s /opt/ethercalc/node_modules/ethercalc/node_modules/socialcalc /opt/ethercalc/node_modules/socialcalc <- the second one doesn't exist according to your example so it should be safe to create that link | 20:15 |
| fungi | far from being the only reason nodejs is scary | 20:15 |
| openstackgerrit | Merged openstack-infra/puppet-ethercalc master: Add basic rspec test https://review.openstack.org/527822 | 20:16 |
| openstackgerrit | Merged openstack-infra/puppet-ethercalc master: Turn off binary symlink in nodejs module https://review.openstack.org/585291 | 20:16 |
| clarkb | cmurphy: if that is the case then puppet has reversed the name and target values according to man ln | 20:16 |
| clarkb | target = /opt/ethercalc/node_modules/socialcalc name = /opt/ethercalc/node_modules/ethercalc/node_modules/socialcalc according to the puppet I think | 20:16 |
| clarkb | which works if you have a new npm install but not if you have an old I think | 20:17 |
| cmurphy | clarkb: yeah looking at https://puppet.com/docs/puppet/5.5/types/file.html#file-attribute-target i think they did | 20:17 |
| *** sthussey has joined #openstack-infra | 20:20 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Map file comment line numbers (alternate) https://review.openstack.org/590442 | 20:22 |
| clarkb | cmurphy: I think we may want `ln -s /opt/ethercalc/node_modules/ethercalc/node_modules/socialcalc /opt/ethercalc/node_modules/socialcalc` for old npm and `ln -s /opt/ethercalc/node_modules/socialcalc ules/ethercalc/node_modules/socialcalc` for new npm beacuse the dir npm installs to changes :/ | 20:23 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Map file comment line numbers https://review.openstack.org/590442 | 20:23 |
| clarkb | cmurphy: we might be able to switch on the nodejs version to decide which we want to do | 20:23 |
| cmurphy | clarkb: what version is currently running? | 20:25 |
| clarkb | npm --version is 2.15.11 | 20:26 |
| clarkb | node --version is v4.8.7 | 20:26 |
| cmurphy | hmm okay i guess xenial is 6.x so i guess that makes sense | 20:27 |
| mordred | yah. the upstream docker image is built using node:4.8 base image - so I'm guessing the upstream isn't doing much with 6.x yet | 20:31 |
| mordred | and thus the confusion with the paths | 20:31 |
| clarkb | mordred: https://review.openstack.org/#/c/527144/18 indicates 4.x doesn't work on newer ubuntu | 20:32 |
| corvus | i'd like to restart the zuul scheduler with the fix to the job api | 20:32 |
| corvus | though... | 20:32 |
| corvus | maybe we ought to wait a few more mins and see if the line mapping change is ready to go in too | 20:33 |
| mordred | ++ | 20:34 |
| mordred | corvus: it'll make it easier to iterate on that patch | 20:34 |
| mordred | which - btw - I ahve realized I need to completely rewrite | 20:34 |
| mordred | pabelanger: hey - you know things about centos, right? | 20:37 |
| clarkb | pabelanger: quick run away | 20:37 |
| mordred | pabelanger: http://logs.openstack.org/64/589964/6/check/openstackci-beaker-centos-7/c9adc6a/job-output.txt.gz#_2018-08-10_20_23_29_608260 | 20:37 |
| mordred | pabelanger: "No package matching 'exim' found available, installed or updated" | 20:37 |
| mordred | if I do a yum search for exim on git01.openstack.org it totally finds one | 20:37 |
| clarkb | mordred: is the package in epel? we disable epel by default on our test nodes iirc | 20:38 |
| mordred | ah - yes, it is | 20:38 |
| mordred | so I need to make sure that the code to enable epel is running | 20:38 |
| corvus | clarkb: you know things about centos! | 20:38 |
| mordred | aha! we enable epel in install_puppet.sh - but not in any of our puppet | 20:39 |
| mordred | so I didn't translate it in to ansibles | 20:39 |
| clarkb | corvus: I'll never admit to it | 20:39 |
| mordred | I has question | 20:40 |
| mordred | in install_puppet.sh, there is this note: | 20:40 |
| mordred | # NOTE(pabelanger): In openstack-infra, we already have epel on | 20:40 |
| mordred | # our nodes, properly configured for mirrors and gpg keys; thus | 20:40 |
| mordred | # check to see if the package is installed first. | 20:40 |
| mordred | I assume that is talking about on build nodes | 20:41 |
| corvus | do we have it installed but disabled? | 20:41 |
| clarkb | corvus: yes that | 20:41 |
| mordred | how do we have it installed? | 20:41 |
| mordred | sorry - first question - that is referring to build nodes and not infra control plane nodes, right? | 20:42 |
| clarkb | mordred: its a config file in /etc/yum.d/ iirc with the flag that says disabled set | 20:42 |
| clarkb | mordred: ya | 20:42 |
| clarkb | mordred: you can yum install --enable-repo epel with our config I think | 20:42 |
| mordred | cool. so for control plane nodes we _don't_ have it installed already, but then we run instlal_puppet.sh so we do have it installed | 20:42 |
| mordred | clarkb: well, not if I didn't run install_puppet.sh I can't | 20:42 |
| corvus | (the 'epel' element is in the nodepool config) | 20:43 |
| clarkb | right its gonna be different on the test nodes and our actual control plane nodes | 20:43 |
| mordred | awesome. I can work with that | 20:44 |
| mordred | wow. we have some important logic in that install_puppet.sh script :) | 20:44 |
| corvus | in case this needs clarification: if epel is installed (but disabled), "yum install --enable-repo epel" will install a package from epel (the --enable-repo means to temporarily enable a disabled repo just for that command) | 20:44 |
| *** eharney has quit IRC | 20:46 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Map file comment line numbers https://review.openstack.org/590442 | 20:48 |
| openstackgerrit | Witold Bedyk proposed openstack-infra/project-config master: Revert "Make monasca-api py35 job voting" https://review.openstack.org/591086 | 20:54 |
| *** jamesmcarthur has quit IRC | 20:55 | |
| *** trown is now known as trown|outtypewww | 21:01 | |
| *** diablo_rojo has quit IRC | 21:02 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add exim role to base playbook https://review.openstack.org/589964 | 21:03 |
| mordred | corvus, clarkb: ^^ that adds the epel logic from install_puppet.sh - and the systemd restart logic - to base-repos role | 21:04 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add exim role to base playbook https://review.openstack.org/589964 | 21:08 |
| mordred | sorry - realized we should put a trap on there so that we only restart systemd if we updated it | 21:08 |
| clarkb | corvus: left a comment on the line mappings change (actually two but one is sort of hand wavy). Unfortunately unable to do a comprehensive review so far but figured you might address the testing -1's and catch my comment in the process | 21:09 |
| *** jamesmcarthur has joined #openstack-infra | 21:09 | |
| pabelanger | mordred: clarkb: corvus: everything you said is correct about epel :) | 21:11 |
| mordred | pabelanger: yay! | 21:11 |
| mordred | pabelanger: wanna look at https://review.openstack.org/589964 and see if that looks right? | 21:11 |
| pabelanger | sure | 21:12 |
| corvus | clarkb: responded | 21:15 |
| corvus | and yeah, i'm fixing up the test errors now | 21:15 |
| openstackgerrit | Merged openstack-infra/system-config master: Remove old inactive users https://review.openstack.org/588553 | 21:16 |
| openstackgerrit | Merged openstack-infra/system-config master: Set mgmt_hieradata variable for bridge.openstack.org https://review.openstack.org/588627 | 21:16 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Fix web content copying in multi dashboard job https://review.openstack.org/591089 | 21:17 |
| clarkb | corvus: ah I just noticed that FETCH_HEAD is passed as ref where that function is called then we don't use ref at all and hard code FETCH_HEAD in the funcion | 21:17 |
| pabelanger | mordred: +2, left comment / suggestion | 21:17 |
| mordred | pabelanger: yay! thanks | 21:18 |
| corvus | clarkb: oh, ha, i misunderstood. will fix :) | 21:18 |
| mordred | pabelanger: do you know why you had the external repo thing in the install_puppet.sh then? | 21:18 |
| pabelanger | mordred: I don't maybe from a time of centos6? | 21:19 |
| mordred | wow. fun | 21:19 |
| mordred | well - that'll certainly make the file easier to deal with :) | 21:19 |
| pabelanger | mordred: ianw might also know | 21:19 |
| mordred | pabelanger: ianw knows everything | 21:19 |
| mordred | pabelanger: I can confirm - yum install epel-release works without anything else in a centos docker | 21:20 |
| pabelanger | yah, pretty sure legacy for centos-6, looking at git blame | 21:21 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Map file comment line numbers https://review.openstack.org/590442 | 21:21 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add exim role to base playbook https://review.openstack.org/589964 | 21:22 |
| mordred | pabelanger: ^^ there we go then | 21:22 |
| pabelanger | mordred: original commit: http://git.openstack.org/cgit/openstack-infra/system-config/commit/?id=d45bddc133ddd07733e9d0db4ebc1891c1fd0558 | 21:23 |
| pabelanger | :) | 21:23 |
| pabelanger | +2 | 21:23 |
| mordred | pabelanger: that was so many years ago | 21:23 |
| pabelanger | indeed | 21:23 |
| *** jamesmcarthur has quit IRC | 21:24 | |
| *** AhmadMahmoudi has joined #openstack-infra | 21:24 | |
| *** dave-mccowan has joined #openstack-infra | 21:26 | |
| openstackgerrit | Fatema Khalid Sherif proposed openstack-infra/storyboard master: Exposing search for comments by substring https://review.openstack.org/591092 | 21:31 |
| *** anticw has joined #openstack-infra | 21:41 | |
| anticw | hi there, quick gerrit question ... is there some sort of 'bot' we can get to close/abanadon old patchsets over a certain date or with some other trigger? | 21:41 |
| anticw | i can do it manually but a bot doing it is less 'personal' so it often causes less offense | 21:41 |
| fungi | anticw: closing people's proposed changes sounds pretty personal already | 21:46 |
| clarkb | anticw: we had one a while back but then gerrit grew the ability for us to delegate that to our users (all change owners and core reviewers should be able to do it) | 21:46 |
| clarkb | anticw: so we killed the bot in favor of individuals doing it when appropriate | 21:46 |
| fungi | also, gerrit has great features which allow you to just ignore old, stale reviews if that's your concern | 21:47 |
| fungi | i have open reviews i've been waiting for projects to merge since 2015 | 21:47 |
| fungi | (no joke) | 21:48 |
| fungi | i'm sure others in here have some even older than that | 21:48 |
| anticw | fungi: some are a year old, people have moved on ... the issues are likely stale | 21:48 |
| anticw | fungi: what's the syntax / query you use to ignore older stuff? | 21:49 |
| fungi | and close any bugs you've had open for a year. who knows if they're still valid, right? ;) | 21:49 |
| fungi | anticw: after:YYYY-MM-DD | 21:49 |
| fungi | or you can express it as age:Xy | 21:49 |
| fungi | the gerrit search documentation has a lot of detail | 21:50 |
| anticw | thanks, i looked at the docs a while ago trying to see if i could detect what's in merge conflict | 21:50 |
| fungi | it has that too, just a sec and i can find it | 21:50 |
| fungi | https://review.openstack.org/Documentation/user-search.html#search-operators | 21:51 |
| fungi | is:mergeable | 21:51 |
| anticw | yeah, trying that now | 21:51 |
| anticw | thanks | 21:51 |
| fungi | which you can also invert | 21:51 |
| anticw | that's what i did | 21:53 |
| anticw | doesn't look like you can alter the sort order at present though | 21:54 |
| fungi | yeah, except maybe on the consumer end of the api | 21:55 |
| fungi | for example, gertty has several sorting toggles (by number, by last update, reverse...) | 21:56 |
| fungi | but it's basically providing those at the presentation layer | 21:57 |
| fungi | i thought gerrit had sorting options in its webui, but i'm not finding them. maybe they dropped those before 2.13 | 21:59 |
| anticw | i see others asking for it, perhaps a later Web UI might suck more into the dom and allow client-side-sorting as it quite common now | 22:01 |
| *** yamamoto has joined #openstack-infra | 22:02 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add exim role to base playbook https://review.openstack.org/589964 | 22:13 |
| *** dhill_ has quit IRC | 22:13 | |
| *** dave-mccowan has quit IRC | 22:19 | |
| fungi | "later webui" is the polyglot-based total rewrite "polygerrit" interface i, 2.14 (or is it 2.15?) which we're not running yet | 22:25 |
| fungi | anticw: ^ | 22:25 |
| fungi | find it in action at https://gerrit-review.googlesource.com/ | 22:27 |
| *** jamesmcarthur has joined #openstack-infra | 22:37 | |
| openstackgerrit | Fatema Khalid Sherif proposed openstack-infra/storyboard-webclient master: Expose ordering criteria in search UI https://review.openstack.org/577870 | 22:41 |
| openstackgerrit | Fatema Khalid Sherif proposed openstack-infra/storyboard-webclient master: Exposing task creator name in search results https://review.openstack.org/583085 | 22:41 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Map file comment line numbers https://review.openstack.org/590442 | 22:42 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Fix race in test_crd_check_unknown https://review.openstack.org/591106 | 22:42 |
| corvus | i think the line mapping change is ready, but it hit a racy test (fix ^). i think i'll just restart the scheduler with the job api fix for now. | 22:43 |
| *** rpioso is now known as rpioso|afk | 22:44 | |
| corvus | and, well, lots of other changes :) | 22:45 |
| *** jamesmcarthur has quit IRC | 22:59 | |
| *** rosmaita has quit IRC | 22:59 | |
| *** hongbin_ has quit IRC | 23:00 | |
| corvus | dhellmann, AJaeger, tristanC, mordred: progress! http://zuul.openstack.org/api/job/devstack | 23:04 |
| *** yamamoto has quit IRC | 23:05 | |
| *** elod has quit IRC | 23:15 | |
| *** elod has joined #openstack-infra | 23:22 | |
| *** yamamoto has joined #openstack-infra | 23:38 | |
| *** jamesmcarthur has joined #openstack-infra | 23:39 | |
| *** jamesmcarthur has quit IRC | 23:44 | |
| *** sthussey has quit IRC | 23:49 | |
| pabelanger | clarkb: ianw: tonyb: corvus: do you mind adding https://review.openstack.org/590785/ and https://review.openstack.org/590769/ to your review pipeline, they are devstack changes to stable/queens for fedora-28. needed for barbican | 23:52 |
| pabelanger | then we can remove fedora-27 from nodepool | 23:52 |
| pabelanger | also Sigyn didn't like that ping | 23:53 |
| pabelanger | just got a warning | 23:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!