| *** rh-jelabarre has quit IRC | 00:07 | |
| *** rh-jelabarre has joined #openstack-infra | 00:08 | |
| *** rh-jelabarre has quit IRC | 00:12 | |
| *** slaweq has joined #openstack-infra | 00:14 | |
| *** ijw has joined #openstack-infra | 00:16 | |
| *** markvoelker has joined #openstack-infra | 00:16 | |
| *** hongbin has joined #openstack-infra | 00:18 | |
| *** slaweq has quit IRC | 00:25 | |
| *** markvoelker has quit IRC | 00:49 | |
| *** jd_ has quit IRC | 00:50 | |
| *** jd_ has joined #openstack-infra | 00:51 | |
| *** weshay has quit IRC | 00:59 | |
| *** slaweq has joined #openstack-infra | 01:11 | |
| *** bgmccollum has quit IRC | 01:21 | |
| *** bgmccollum has joined #openstack-infra | 01:23 | |
| *** slaweq has quit IRC | 01:25 | |
| *** hongbin has quit IRC | 01:39 | |
| *** hongbin has joined #openstack-infra | 01:41 | |
| ianw | it seems 2019 is not going to be the year i can unplug my laptop from it's docking station, plug it back in and have it recognise the external monitor again. maybe 2020 ... | 01:42 |
|---|---|---|
| *** hongbin_ has joined #openstack-infra | 01:45 | |
| *** hongbin has quit IRC | 01:45 | |
| *** markvoelker has joined #openstack-infra | 01:46 | |
| *** bgmccollum has quit IRC | 01:50 | |
| *** bgmccollum has joined #openstack-infra | 01:53 | |
| *** rh-jelabarre has joined #openstack-infra | 02:08 | |
| *** slaweq has joined #openstack-infra | 02:16 | |
| *** markvoelker has quit IRC | 02:19 | |
| *** slaweq has quit IRC | 02:24 | |
| *** ijw has quit IRC | 02:57 | |
| *** ykarel has joined #openstack-infra | 03:01 | |
| *** psachin has joined #openstack-infra | 03:04 | |
| *** slaweq has joined #openstack-infra | 03:14 | |
| *** markvoelker has joined #openstack-infra | 03:16 | |
| *** slaweq has quit IRC | 03:24 | |
| *** Bhujay has joined #openstack-infra | 03:30 | |
| *** Bhujay has quit IRC | 03:35 | |
| *** ramishra has joined #openstack-infra | 03:41 | |
| *** armstrong has quit IRC | 03:45 | |
| *** wolverineav has joined #openstack-infra | 03:47 | |
| *** markvoelker has quit IRC | 03:48 | |
| *** wolverineav has quit IRC | 03:51 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: executor: properly format error exception https://review.openstack.org/630928 | 03:52 |
| *** udesale has joined #openstack-infra | 04:00 | |
| *** udesale has quit IRC | 04:00 | |
| *** udesale has joined #openstack-infra | 04:01 | |
| *** bhavikdbavishi has joined #openstack-infra | 04:15 | |
| *** slaweq has joined #openstack-infra | 04:16 | |
| *** bhavikdbavishi has quit IRC | 04:20 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add neutron-tempest-iptables_hybrid-fedora job to Neutron dashboard https://review.openstack.org/631806 | 04:23 |
| *** bhavikdbavishi has joined #openstack-infra | 04:24 | |
| *** slaweq has quit IRC | 04:24 | |
| *** bobh has joined #openstack-infra | 04:41 | |
| *** lpetrut has joined #openstack-infra | 04:42 | |
| *** Bhujay has joined #openstack-infra | 04:43 | |
| *** jamesmcarthur has joined #openstack-infra | 04:44 | |
| *** ijw has joined #openstack-infra | 04:45 | |
| *** jamesmcarthur has quit IRC | 04:45 | |
| *** jamesmcarthur has joined #openstack-infra | 04:45 | |
| *** bhavikdbavishi has quit IRC | 04:59 | |
| *** ramishra has quit IRC | 04:59 | |
| *** jamesmcarthur has quit IRC | 04:59 | |
| *** jamesmcarthur has joined #openstack-infra | 05:02 | |
| openstackgerrit | Merged openstack/diskimage-builder master: support cracklib in pam for Gentoo's musl profile https://review.openstack.org/631009 | 05:03 |
| *** rh-jelabarre has quit IRC | 05:04 | |
| *** lpetrut has quit IRC | 05:05 | |
| *** ramishra has joined #openstack-infra | 05:07 | |
| *** bobh has quit IRC | 05:07 | |
| *** ijw has quit IRC | 05:12 | |
| *** slaweq has joined #openstack-infra | 05:15 | |
| *** ykarel has quit IRC | 05:19 | |
| *** bramwelt has quit IRC | 05:19 | |
| *** bhavikdbavishi has joined #openstack-infra | 05:23 | |
| *** slaweq has quit IRC | 05:24 | |
| *** jamesmcarthur has quit IRC | 05:27 | |
| *** bhavikdbavishi has quit IRC | 05:28 | |
| *** jamesmcarthur has joined #openstack-infra | 05:29 | |
| *** jamesmcarthur has quit IRC | 05:34 | |
| *** bhavikdbavishi has joined #openstack-infra | 05:35 | |
| *** spsurya has joined #openstack-infra | 05:41 | |
| *** ykarel has joined #openstack-infra | 05:47 | |
| *** ramishra_ has joined #openstack-infra | 05:48 | |
| *** ramishra has quit IRC | 05:51 | |
| *** lpetrut has joined #openstack-infra | 06:10 | |
| *** markvoelker has joined #openstack-infra | 06:16 | |
| *** bhavikdbavishi has quit IRC | 06:20 | |
| *** hongbin has joined #openstack-infra | 06:25 | |
| *** hongbin has quit IRC | 06:25 | |
| *** hongbin_ has quit IRC | 06:27 | |
| *** ijw has joined #openstack-infra | 06:37 | |
| *** portdirect has quit IRC | 06:42 | |
| *** egonzalez has quit IRC | 06:42 | |
| *** adrianreza has quit IRC | 06:42 | |
| *** jiapei has quit IRC | 06:42 | |
| *** mrhillsman has quit IRC | 06:42 | |
| *** corvus has quit IRC | 06:42 | |
| *** johnsom has quit IRC | 06:42 | |
| *** rabel has quit IRC | 06:42 | |
| *** icey has quit IRC | 06:42 | |
| *** Shrews has quit IRC | 06:42 | |
| *** ijw has quit IRC | 06:42 | |
| *** rabel has joined #openstack-infra | 06:42 | |
| *** Shrews has joined #openstack-infra | 06:42 | |
| *** johnsom has joined #openstack-infra | 06:42 | |
| *** mrhillsman has joined #openstack-infra | 06:42 | |
| *** portdirect has joined #openstack-infra | 06:42 | |
| *** icey has joined #openstack-infra | 06:43 | |
| *** jiapei has joined #openstack-infra | 06:43 | |
| *** adrianreza has joined #openstack-infra | 06:43 | |
| *** egonzalez has joined #openstack-infra | 06:43 | |
| *** quiquell|off is now known as quiquell | 06:44 | |
| *** jtomasek has joined #openstack-infra | 06:48 | |
| *** markvoelker has quit IRC | 06:48 | |
| *** ramishra_ has quit IRC | 06:50 | |
| *** ijw has joined #openstack-infra | 06:58 | |
| *** ramishra has joined #openstack-infra | 06:59 | |
| *** rcernin has quit IRC | 07:00 | |
| AJaeger | ianw, frickler, could you put https://review.openstack.org/631817 and https://review.openstack.org/631818 to retire puppet-stackalytics on your review queue, please? | 07:02 |
| *** roman_g has joined #openstack-infra | 07:02 | |
| AJaeger | And https://review.openstack.org/631861 as well... | 07:02 |
| *** larainema has quit IRC | 07:02 | |
| *** ijw has quit IRC | 07:03 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Set your-readthedocs-job-requires-attention host localhost https://review.openstack.org/632004 | 07:06 |
| *** wolverineav has joined #openstack-infra | 07:11 | |
| *** wolverineav has quit IRC | 07:12 | |
| *** dpawlik has joined #openstack-infra | 07:13 | |
| *** wolverineav has joined #openstack-infra | 07:14 | |
| *** Dobroslaw has joined #openstack-infra | 07:16 | |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: WIP: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 07:20 |
| *** ccamacho has joined #openstack-infra | 07:28 | |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 07:32 |
| *** AJaeger has quit IRC | 07:33 | |
| *** AJaeger has joined #openstack-infra | 07:36 | |
| *** pgaxatte has joined #openstack-infra | 07:42 | |
| *** slaweq has joined #openstack-infra | 07:43 | |
| *** markvoelker has joined #openstack-infra | 07:46 | |
| *** kopecmartin|off is now known as kopecmartin | 07:47 | |
| *** jamesmcarthur has joined #openstack-infra | 07:48 | |
| *** apetrich has joined #openstack-infra | 07:49 | |
| *** jamesmcarthur has quit IRC | 07:52 | |
| *** rpittau has joined #openstack-infra | 07:54 | |
| *** lpetrut has quit IRC | 07:58 | |
| *** ykarel is now known as ykarel|lunch | 07:58 | |
| *** e0ne has joined #openstack-infra | 08:02 | |
| *** ramishra has quit IRC | 08:03 | |
| *** Bhujay has quit IRC | 08:09 | |
| *** ramishra has joined #openstack-infra | 08:09 | |
| *** aojea has joined #openstack-infra | 08:10 | |
| *** ginopc has joined #openstack-infra | 08:14 | |
| *** tkajinam has quit IRC | 08:16 | |
| *** wolverineav has quit IRC | 08:18 | |
| *** markvoelker has quit IRC | 08:19 | |
| *** e0ne has quit IRC | 08:24 | |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 08:26 |
| *** wolverineav has joined #openstack-infra | 08:26 | |
| *** ijw has joined #openstack-infra | 08:27 | |
| *** wolverineav has quit IRC | 08:28 | |
| openstackgerrit | Merged openstack-infra/project-config master: Retire puppet-stackalytics (part 1) https://review.openstack.org/631817 | 08:28 |
| *** wolverineav has joined #openstack-infra | 08:29 | |
| AJaeger | ianw: thanks for review of https://review.openstack.org/#/c/631818 - could you +A as well? I just asked for recheck, it should pass now... | 08:31 |
| openstackgerrit | Merged openstack-infra/puppet-stackalytics master: Retire pupet-stackalytics https://review.openstack.org/631818 | 08:31 |
| *** ykarel|lunch is now known as ykarel | 08:32 | |
| *** ijw has quit IRC | 08:32 | |
| ianw | AJaeger: thanks, LGTM. hosting stackalytics was always ... interesting :) | 08:32 |
| *** wolverineav has quit IRC | 08:33 | |
| *** kjackal has joined #openstack-infra | 08:35 | |
| *** tosky has joined #openstack-infra | 08:36 | |
| *** jpich has joined #openstack-infra | 08:41 | |
| *** xek_ has joined #openstack-infra | 08:42 | |
| *** electrofelix has joined #openstack-infra | 08:43 | |
| *** gfidente has joined #openstack-infra | 08:48 | |
| *** ijw has joined #openstack-infra | 08:48 | |
| *** ijw has quit IRC | 08:53 | |
| *** Bhujay has joined #openstack-infra | 08:53 | |
| *** Bhujay has quit IRC | 08:53 | |
| *** Bhujay has joined #openstack-infra | 08:54 | |
| *** Bhujay has quit IRC | 08:55 | |
| *** Bhujay has joined #openstack-infra | 08:56 | |
| *** jpena|off is now known as jpena | 09:03 | |
| *** wolverineav has joined #openstack-infra | 09:05 | |
| *** SotK_ has joined #openstack-infra | 09:10 | |
| *** markvoelker has joined #openstack-infra | 09:16 | |
| *** d0ugal has quit IRC | 09:21 | |
| *** d0ugal has joined #openstack-infra | 09:22 | |
| *** shardy has joined #openstack-infra | 09:24 | |
| *** wolverineav has quit IRC | 09:28 | |
| *** panda|off is now known as panda | 09:29 | |
| AJaeger | thanks, ianw . Yes, indeed... | 09:30 |
| *** ramishra has quit IRC | 09:38 | |
| *** derekh has joined #openstack-infra | 09:38 | |
| *** ramishra has joined #openstack-infra | 09:40 | |
| openstackgerrit | Merged openstack-infra/project-config master: Retire gantt, python-ganttclient https://review.openstack.org/630162 | 09:49 |
| *** markvoelker has quit IRC | 09:49 | |
| *** jaosorior has joined #openstack-infra | 09:53 | |
| *** dtantsur|afk is now known as dtantsur | 09:57 | |
| tobias-urdin | fungi: tobiash regarding the "ansible_os_family" fact not being available, was that fixed or did you know what caused it and how we can fix it? | 10:11 |
| *** yboaron has joined #openstack-infra | 10:12 | |
| tobiash | tobias-urdin: the reason for that is that your job tries that on 'localhost' which is the executor. And for security reasons 'localhost' suppresses all facts. | 10:12 |
| *** e0ne has joined #openstack-infra | 10:13 | |
| tobiash | tobias-urdin: so if you need to install stuff you need to run on a node | 10:13 |
| tobiash | this will have that fact then | 10:13 |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 10:15 |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 10:16 |
| tobias-urdin | tobiash: ah, so "hosts: localhost" to "hosts: all" would make it not run on the executor but in a cloud vm? | 10:18 |
| *** ijw has joined #openstack-infra | 10:18 | |
| tobiash | Yes | 10:19 |
| tobias-urdin | tobiash: thanks! | 10:19 |
| tobiash | yw | 10:20 |
| openstackgerrit | Tobias Urdin proposed openstack-infra/project-config master: Run puppetforge publish playbook in the clouds https://review.openstack.org/632060 | 10:20 |
| *** openstackgerrit has quit IRC | 10:21 | |
| *** openstackgerrit has joined #openstack-infra | 10:22 | |
| openstackgerrit | Tobias Urdin proposed openstack-infra/project-config master: Run puppetforge publish playbook in the clouds https://review.openstack.org/632060 | 10:22 |
| *** ijw has quit IRC | 10:22 | |
| openstackgerrit | Tobias Urdin proposed openstack-infra/project-config master: Run puppet release in the clouds https://review.openstack.org/632060 | 10:35 |
| openstackgerrit | Tobias Urdin proposed openstack-infra/project-config master: Run puppet release in the clouds https://review.openstack.org/632060 | 10:37 |
| *** ijw has joined #openstack-infra | 10:39 | |
| openstackgerrit | Tobias Urdin proposed openstack-infra/project-config master: Run puppetforge publish playbook in the clouds https://review.openstack.org/632060 | 10:40 |
| *** wolverineav has joined #openstack-infra | 10:41 | |
| *** ramishra has quit IRC | 10:41 | |
| *** ramishra has joined #openstack-infra | 10:41 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: WIP: runner: add support for depends-on https://review.openstack.org/632064 | 10:42 |
| *** ijw has quit IRC | 10:43 | |
| *** markvoelker has joined #openstack-infra | 10:46 | |
| *** markvoelker has quit IRC | 11:19 | |
| *** udesale has quit IRC | 11:20 | |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 11:24 |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 11:34 |
| AJaeger | infra-root, I saw a promote failed on change https://review.openstack.org/630162 - see http://logs.openstack.org/62/630162/2/promote/infra-prod-zuul_reconfigure/df1e3c7/job-output.txt.gz#_2019-01-21_09_51_20_209370 . Could you check this, please? | 11:42 |
| *** lpetrut has joined #openstack-infra | 11:47 | |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 11:48 |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 11:50 |
| *** ijw has joined #openstack-infra | 12:08 | |
| *** kjackal has quit IRC | 12:11 | |
| *** ijw has quit IRC | 12:13 | |
| *** panda is now known as panda|lunch | 12:14 | |
| *** zul has joined #openstack-infra | 12:15 | |
| *** markvoelker has joined #openstack-infra | 12:16 | |
| *** kjackal has joined #openstack-infra | 12:20 | |
| *** bhavikdbavishi has joined #openstack-infra | 12:21 | |
| *** udesale has joined #openstack-infra | 12:26 | |
| *** ijw has joined #openstack-infra | 12:28 | |
| *** rh-jelabarre has joined #openstack-infra | 12:29 | |
| *** ijw has quit IRC | 12:33 | |
| *** jistr is now known as jistr|afk | 12:38 | |
| *** bhavikdbavishi has quit IRC | 12:46 | |
| *** markvoelker has quit IRC | 12:49 | |
| *** dkehn has quit IRC | 12:55 | |
| *** weshay has joined #openstack-infra | 12:58 | |
| *** eharney has joined #openstack-infra | 13:04 | |
| openstackgerrit | Merged openstack-infra/storyboard-webclient master: Update the min version of tox to 2.0 https://review.openstack.org/615035 | 13:05 |
| *** jistr|afk is now known as jistr | 13:10 | |
| *** bhavikdbavishi has joined #openstack-infra | 13:29 | |
| *** rlandy has joined #openstack-infra | 13:32 | |
| fungi | infra-root: reminder, i'm mostly afk this week and entirely offline next week | 13:36 |
| *** panda|lunch is now known as panda | 13:38 | |
| *** priteau has joined #openstack-infra | 13:40 | |
| *** bhavikdbavishi has quit IRC | 13:42 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Only move directory contents once in merge-output-to-logs https://review.openstack.org/632092 | 13:47 |
| mordred | fungi, AJaeger :^^ I think we have an issue | 13:47 |
| mordred | http://logs.openstack.org/94/623294/16/check/openstack-infra-multinode-integration-ubuntu-bionic/51af205/ara-report/result/63b23c1f-c7b8-4ab8-b227-19f806dc60df/ | 13:48 |
| *** agopi has joined #openstack-infra | 13:56 | |
| *** kgiusti has joined #openstack-infra | 13:57 | |
| *** ijw has joined #openstack-infra | 13:58 | |
| *** ykarel is now known as ykarel|away | 13:59 | |
| *** jcoufal has joined #openstack-infra | 14:00 | |
| frickler | mordred: I think your patch is good, but not enough, see the stdout lines here: http://logs.openstack.org/94/623294/16/check/openstack-infra-multinode-integration-ubuntu-bionic/51af205/ara-report/result/7bb8174c-7f56-4115-ba24-285935cdfc88/ | 14:01 |
| frickler | it seems to me the "if ! $(ls ...)" thing is bad, too | 14:01 |
| frickler | I'd also use "[ -d ... ]" instead of "$(ls ...)" | 14:02 |
| *** mriedem has joined #openstack-infra | 14:02 | |
| *** ijw has quit IRC | 14:03 | |
| *** yboaron_ has joined #openstack-infra | 14:03 | |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 14:03 |
| *** yboaron has quit IRC | 14:05 | |
| *** udesale has quit IRC | 14:05 | |
| *** udesale has joined #openstack-infra | 14:06 | |
| mordred | frickler: I think what I was trying to accomplish originally was "does this directory have any files in it" | 14:08 |
| *** ykarel|away has quit IRC | 14:08 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: DNM: test https://review.openstack.org/631970 | 14:10 |
| frickler | mordred: ah, I think I understand the error now. the $(ls ...) expands to "primary" because that's the name of the file/directory in there. then it runs "if ! primary ..." and that fails. so what you really want is "if [ -n $(ls ...)]" | 14:11 |
| mordred | frickler: but then why is it giving 'mv:' in the error? | 14:12 |
| frickler | mordred: that's the concurrency thing, which is fixed by your "run_once". two different issues | 14:13 |
| mordred | ah | 14:13 |
| mordred | frickler: I totally agree with you | 14:14 |
| mordred | thanks! | 14:14 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Fix concurrency and syntax issue in merge-output-to-logs https://review.openstack.org/632092 | 14:16 |
| mordred | frickler: ^^ how's that look now? | 14:17 |
| frickler | mordred: lacking a ")" ... ;-) | 14:17 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Fix concurrency and syntax issue in merge-output-to-logs https://review.openstack.org/632092 | 14:17 |
| mordred | frickler: what are we writing in here - lisp? :) | 14:18 |
| *** eharney has quit IRC | 14:19 | |
| *** ijw has joined #openstack-infra | 14:19 | |
| frickler | mordred: that would need a lot more nesting ;) | 14:19 |
| mordred | frickler: hahaha | 14:23 |
| *** ijw has quit IRC | 14:24 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: DNM: test https://review.openstack.org/631970 | 14:25 |
| *** xek_ has quit IRC | 14:25 | |
| *** xek_ has joined #openstack-infra | 14:26 | |
| *** smarcet has joined #openstack-infra | 14:32 | |
| *** e0ne has quit IRC | 14:36 | |
| *** efried has quit IRC | 14:41 | |
| *** e0ne has joined #openstack-infra | 14:41 | |
| *** efried has joined #openstack-infra | 14:42 | |
| *** rkukura has joined #openstack-infra | 14:43 | |
| *** Bhujay has quit IRC | 14:43 | |
| *** kmalloc has joined #openstack-infra | 14:43 | |
| *** shardy has quit IRC | 14:46 | |
| openstackgerrit | Quique Llorente proposed openstack-infra/zuul-jobs master: Default private_ipv4 to use public_ipv4 address when null https://review.openstack.org/623294 | 14:46 |
| *** shardy has joined #openstack-infra | 14:46 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: DNM: test https://review.openstack.org/631970 | 14:48 |
| *** yboaron_ has quit IRC | 14:50 | |
| *** wolverineav has quit IRC | 14:53 | |
| *** efried is now known as efried_mlk | 14:55 | |
| *** dave-mccowan has joined #openstack-infra | 14:55 | |
| dmsimard | I think the tox-pep8 job isn't happy with py3 f strings because it's running on py2? :/ | 15:00 |
| mordred | dmsimard: you can tell it to run on py3 | 15:01 |
| mordred | dmsimard: you just need to set basepython in your tox.ini | 15:01 |
| mordred | dmsimard: basepython = python3 | 15:01 |
| mordred | dmsimard: you can put that in [testenv:pep8] - or if the project is py3 only, just put it in the main [testenv] section | 15:02 |
| dmsimard | mordred: already have py3 as basepython from testenv | 15:02 |
| mordred | then it shouldn't be using py27 | 15:02 |
| mordred | dmsimard: got a link? | 15:02 |
| dmsimard | mordred: http://logs.openstack.org/28/631828/1/check/tox-pep8/6ff2b08/ara-report/result/696f01c3-20b9-4dc4-8d00-1c29ee07b17b/ | 15:03 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: WIP: Manage ansible installation within zuul https://review.openstack.org/631930 | 15:04 |
| dmsimard | mordred: don't spend too much time on it, I've worked around it for the time being | 15:05 |
| mordred | dmsimard: it's python version | 15:06 |
| mordred | dmsimard: that's running on xenial which has python3.5 | 15:06 |
| mordred | dmsimard: f-strings don't exist until 3.6 | 15:06 |
| dmsimard | oh | 15:06 |
| dmsimard | didn't realize it was >= 3.6 | 15:07 |
| mordred | dmsimard: btw - I love f-strings | 15:07 |
| smarcet | fungi: mordred: clarkb: morning guys when you get a chance please review and merge https://review.openstack.org/#/c/631655/ and https://review.openstack.org/#/c/625640/ :) thx u | 15:07 |
| *** yboaron_ has joined #openstack-infra | 15:07 | |
| dmsimard | mordred: yeah they're great -- I try to get rid of .format and %s's when I can lol | 15:07 |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Add ansible-role-ansible project https://review.openstack.org/632109 | 15:08 |
| mordred | dmsimard: I am looking forward to when we bump zuul's minimum | 15:09 |
| *** dave-mccowan has quit IRC | 15:12 | |
| *** szaher has quit IRC | 15:13 | |
| *** dpawlik has quit IRC | 15:14 | |
| *** szaher has joined #openstack-infra | 15:17 | |
| *** rkukura has quit IRC | 15:21 | |
| *** Bhujay has joined #openstack-infra | 15:21 | |
| *** ykarel|away has joined #openstack-infra | 15:22 | |
| *** ykarel|away is now known as ykarel | 15:25 | |
| *** dpawlik has joined #openstack-infra | 15:26 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: DNM: test https://review.openstack.org/631970 | 15:29 |
| *** dpawlik has quit IRC | 15:30 | |
| *** mriedem has quit IRC | 15:32 | |
| *** dkehn has joined #openstack-infra | 15:41 | |
| *** Bhujay has quit IRC | 15:43 | |
| *** bhavikdbavishi has joined #openstack-infra | 15:45 | |
| openstackgerrit | Merged openstack-infra/puppet-openstackid master: Fixed scope of deploy resource type usage https://review.openstack.org/631655 | 15:47 |
| mordred | pabelanger: if you have a sec, mind looking at https://review.openstack.org/632092 ? | 15:48 |
| *** ijw has joined #openstack-infra | 15:48 | |
| openstackgerrit | Merged openstack-infra/system-config master: Migrate OpenStackID dev server to php7 https://review.openstack.org/625640 | 15:50 |
| pabelanger | mordred: +2, approve when ready | 15:52 |
| *** ijw has quit IRC | 15:53 | |
| *** whoami-rajat has quit IRC | 15:55 | |
| mordred | pabelanger: thanks! | 15:55 |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Add ansible-role-ansible project https://review.openstack.org/632109 | 15:56 |
| *** mriedem has joined #openstack-infra | 15:57 | |
| *** quiquell is now known as quiquell|off | 15:58 | |
| *** udesale has quit IRC | 16:01 | |
| AJaeger | clarkb, could you review the final puppet-stackalytics retire changes, please? https://review.openstack.org/631819 https://review.openstack.org/631820 https://review.openstack.org/631861 | 16:02 |
| *** bhavikdbavishi has quit IRC | 16:03 | |
| mordred | AJaeger: +2 on the ones I can +2 (+3 on the first one) | 16:06 |
| *** ijw has joined #openstack-infra | 16:09 | |
| *** psachin has quit IRC | 16:12 | |
| *** ijw has quit IRC | 16:14 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Retire puppet-stackalytics (part 2b) https://review.openstack.org/631861 | 16:14 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Fix concurrency and syntax issue in merge-output-to-logs https://review.openstack.org/632092 | 16:14 |
| *** yboaron_ has quit IRC | 16:15 | |
| *** hemnaaway is now known as hemna | 16:17 | |
| *** bhavikdbavishi has joined #openstack-infra | 16:17 | |
| *** ramishra has quit IRC | 16:24 | |
| *** dpawlik has joined #openstack-infra | 16:31 | |
| *** dpawlik has quit IRC | 16:35 | |
| *** eharney has joined #openstack-infra | 16:36 | |
| *** whoami-rajat has joined #openstack-infra | 16:41 | |
| *** corvus has joined #openstack-infra | 16:42 | |
| openstackgerrit | Mark Goddard proposed openstack-dev/pbr master: Fix white space handling in file names https://review.openstack.org/629161 | 16:44 |
| *** smcginnis has joined #openstack-infra | 16:47 | |
| *** ijw has joined #openstack-infra | 16:49 | |
| *** bhavikdbavishi has quit IRC | 16:50 | |
| *** ijw has quit IRC | 16:53 | |
| *** pgaxatte has quit IRC | 16:54 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add python-builder docker image https://review.openstack.org/631878 | 16:55 |
| corvus | pabelanger, tobiash: i think i disagree with the suggested default on 623927 -- i think to make things easier for new users, we should have the executor manage ansible by default. that doesn't stop advanced users with more sophisticated config management from doing otherwise. | 16:57 |
| tobiash | corvus: fine for me too | 16:57 |
| clarkb | AJaeger: Approved the removal from zuul and +1'd the governance change | 16:57 |
| *** priteau has quit IRC | 16:58 | |
| *** e0ne has quit IRC | 16:58 | |
| tobiash | corvus: we could run the zuul-quick-start like this | 16:58 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul-jobs master: Add a slack-notify role https://review.openstack.org/623594 | 16:59 |
| *** rpittau has quit IRC | 16:59 | |
| corvus | AJaeger: infra-prod-zuul_reconfigure is currently expected to fail; it's a WIP. it doesn't waste any resources though so i think we can just leave it there for now. | 16:59 |
| *** priteau has joined #openstack-infra | 17:00 | |
| *** aojea has quit IRC | 17:00 | |
| pabelanger | corvus: sure, default not a issue for me, more the advance user usecase was looking for | 17:00 |
| corvus | tobiash: the quick start uses the docker images though, so it'll have it already | 17:00 |
| corvus | also, sorry about starting this convo in the wrong channel :/ | 17:01 |
| *** dpawlik has joined #openstack-infra | 17:02 | |
| tobiash | ah right, this needs a different test then | 17:03 |
| AJaeger | mordred, clarkb , thanks! | 17:03 |
| AJaeger | corvus: ah, good | 17:03 |
| clarkb | as a reminder I too am "traveling" starting today through thursday. I actually don't have to metal tube but will be in meetings all day for a few days. We'll need someone to chair the meeting tomorrow as a result | 17:05 |
| corvus | clarkb: staff meeting at your local pub? nice :) | 17:05 |
| clarkb | corvus: I didn't even do any of the planning, it just worked out that way :) | 17:06 |
| *** dpawlik has quit IRC | 17:07 | |
| *** jpich has quit IRC | 17:09 | |
| AJaeger | mordred: merge-output-to-logs is failing for me in project-config, see http://logs.openstack.org/19/631819/2/gate/openstack-zuul-jobs-linters/87953dc/ | 17:10 |
| *** jpich has joined #openstack-infra | 17:10 | |
| *** ijw has joined #openstack-infra | 17:10 | |
| AJaeger | mordred: http://logs.openstack.org/19/631819/2/gate/openstack-zuul-jobs-linters/87953dc/ara-report/result/ea1b2d47-d504-4c00-b578-573775cff288/ | 17:10 |
| AJaeger | mordred: so, https://review.openstack.org/632092 is broken | 17:11 |
| mordred | AJaeger: that looks like the issue https://review.openstack.org/632092 was intended to fix | 17:11 |
| mordred | AJaeger: that did not run with the patch applied | 17:12 |
| AJaeger | mordred: should I recheck? | 17:12 |
| mordred | oh - yest it did | 17:12 |
| mordred | wait a sec | 17:12 |
| AJaeger | oh - rechecked already... | 17:12 |
| AJaeger | mordred: ok, waiting otherwise... | 17:12 |
| mordred | AJaeger: let's revert that patch asap while we work on it | 17:14 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Revert "Fix concurrency and syntax issue in merge-output-to-logs" https://review.openstack.org/632135 | 17:14 |
| mordred | AJaeger: it went from being an issue affecting a small number of jobs to potentially an issue affecting a larger set of jobs | 17:14 |
| mordred | corvus: if I could steal your attention for a sec ... | 17:14 |
| *** ijw has quit IRC | 17:15 | |
| *** kjackal has quit IRC | 17:15 | |
| mordred | corvus: any ideas on a shell snippet for "if files exist in a directory" | 17:15 |
| *** shardy has quit IRC | 17:15 | |
| AJaeger | mordred: thanks - +2 | 17:15 |
| *** ginopc has quit IRC | 17:15 | |
| mordred | corvus: oh - and also, +3 the revert patch there ^^ | 17:16 |
| *** dtantsur is now known as dtantsur|afk | 17:16 | |
| corvus | mordred: catching up | 17:16 |
| *** shardy has joined #openstack-infra | 17:17 | |
| clarkb | mordred: find path/to/dir -type f -0 | wc -0 or something | 17:17 |
| mordred | corvus: I think I've ... yeah ^^ | 17:17 |
| mordred | that actually works | 17:17 |
| mordred | patch incoin | 17:17 |
| mordred | clarkb: (turns out just testing the output of find works by itself) | 17:17 |
| clarkb | oh cool | 17:18 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Use find instead of ls for determining dir emptiness https://review.openstack.org/632136 | 17:18 |
| mordred | clarkb, corvus: ^^ that did the right thing for me in local testing - but obviously since this is try three, would love an extra sanity check | 17:18 |
| corvus | mordred: you want to pull the revert? | 17:19 |
| mordred | corvus: what do you think? just land the find? or rebase find on the revert as a revert-revert? | 17:20 |
| clarkb | mordred: it returns 0 if the find dir is empty or has files for me | 17:20 |
| mordred | clarkb: it does? | 17:20 |
| corvus | mordred: as written, the find conflicts with the revert; might be best to rebase on the revert so we're not rushed :) | 17:21 |
| *** kopecmartin is now known as kopecmartin|off | 17:21 | |
| clarkb | mordred: http://paste.openstack.org/show/743057/ | 17:21 |
| mordred | corvus: kk. will do | 17:21 |
| AJaeger | mordred: the if fails for me ;( | 17:21 |
| clarkb | seems to do so for me | 17:21 |
| *** jpich has quit IRC | 17:22 | |
| mordred | clarkb: http://paste.openstack.org/show/743058/ | 17:22 |
| *** jpich has joined #openstack-infra | 17:22 | |
| mordred | that's my local testing :( | 17:22 |
| AJaeger | mordred: http://paste.openstack.org/show/743059/ | 17:22 |
| mordred | oh - gotit | 17:23 |
| clarkb | mordred: oh the $() wrapper is going to try and execute the stdout of find? | 17:23 |
| clarkb | ya | 17:23 |
| mordred | it's a combo of the two | 17:24 |
| mordred | or - of the frickler fix from earlier - one sec | 17:24 |
| AJaeger | we really need to test with more than one file ;) | 17:24 |
| mordred | yeah | 17:24 |
| clarkb | I think you do want to check the output of find there as being non null | 17:25 |
| *** shardy has quit IRC | 17:25 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Use find instead of ls for determining dir emptiness https://review.openstack.org/632136 | 17:25 |
| mordred | like that ^^ | 17:25 |
| *** shardy has joined #openstack-infra | 17:25 | |
| clarkb | $(find path -mindepth 1) != "" | 17:25 |
| clarkb | ya -n same diff cool | 17:25 |
| mordred | would != "" be clearer to read? | 17:26 |
| clarkb | no | 17:26 |
| mordred | kk | 17:26 |
| clarkb | because then you have to figure out how that is different than -ne | 17:26 |
| clarkb | :) | 17:26 |
| *** jpich has quit IRC | 17:26 | |
| fungi | corvus: mordred: find has an -empty option which will indicate empty directories, and can be negated... not sure exactly how i'd go about making use of it but perhaps ever so slightly more elegant than using wc. but whatever works is fine by me | 17:26 |
| *** bhavikdbavishi has joined #openstack-infra | 17:26 | |
| clarkb | -n is the exact test for this so its clearer imo | 17:26 |
| mordred | clarkb: haha | 17:26 |
| AJaeger | mordred: LGTM; +3 | 17:26 |
| mordred | AJaeger: ok. should we block the revert? | 17:27 |
| AJaeger | mordred: I'm fine either way... | 17:27 |
| AJaeger | mordred: https://review.openstack.org/632135 is failing - not self-testing | 17:28 |
| AJaeger | so, we need to force merge 632135 or 632136 | 17:28 |
| mordred | if we're gonna force-merge - I think 136 is a better option to force-merge ... I think we've all manually verified that syntax does what we actually mean now yeah? | 17:29 |
| AJaeger | you have 3 +2s ;) | 17:29 |
| AJaeger | yes, tested locally | 17:29 |
| * mordred force-merge | 17:30 | |
| corvus | frickler, pabelanger, mordred: where was the testing for https://review.openstack.org/632092 performed? | 17:30 |
| AJaeger | mordred: I'll abandon 135 | 17:31 |
| mordred | corvus: locally, same as this - which was clearly a wrong decision | 17:32 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Use find instead of ls for determining dir emptiness https://review.openstack.org/632136 | 17:33 |
| corvus | that isn't our policy. we do one of two things when we land changes to base jobs or roles -- we update base-test, or we use integration test jobs. | 17:33 |
| corvus | frickler, pabelanger, mordred: what should we do to improve things to make sure that we follow that policy in the future? ^ | 17:33 |
| AJaeger | corvus: https://review.openstack.org/#/c/623294 is one change that I want to see tested ;) | 17:35 |
| pabelanger | ah, I wasnt' aware that role was in production yet | 17:35 |
| pabelanger | figured it was base-test | 17:35 |
| mordred | corvus: I think following the policy is key. I do not believe there are additional things that need to be done - it was bad judgement on my part | 17:36 |
| pabelanger | Yah, I also haven't need watching the refactor as close a possible, so likely should hold off giving a +2 in future until I can watch more closely | 17:37 |
| corvus | perhaps if we annotate roles indicating they are only used in base and therefore require base-test or integration changes, it would remind reviewers of that? | 17:38 |
| corvus | mordred: would it be possible to write an integration test for this role? | 17:38 |
| tosky | sorry for jumping in late: were you discussing about a change which lead to POST_FAILURE? If it was, can I safely recheck? | 17:38 |
| corvus | tosky: yes and yes | 17:38 |
| tosky | thanks! | 17:38 |
| mordred | corvus: yeah - I'll work on that today | 17:39 |
| mordred | corvus: (integratio ntest) | 17:39 |
| *** anteaya has joined #openstack-infra | 17:40 | |
| *** ekultails has joined #openstack-infra | 17:42 | |
| corvus | mordred: looks like the framework for that is here: http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/tests | 17:42 |
| *** ekultails has quit IRC | 17:42 | |
| corvus | mordred: base.yaml is the job playbook. | 17:42 |
| mordred | ++ | 17:42 |
| mordred | corvus: and I like the annotation idea ... although that might be a little openstack-specific for zuul-jobs. merge-output-to-logs is in our base job, but once we finish the pabelanger base-job rework I think it's one of the ones that can go into the untrusted base job - so I'm not sure what the correct annotation would be in zuul-jobs, you know? | 17:42 |
| *** ekultails has joined #openstack-infra | 17:43 | |
| mordred | maybe an annotation of "used in base jobs" - so at least the reviewer will want to see one of either integration test or depends-on test as appropriate | 17:43 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul master: Improve debug logging for fingerclient https://review.openstack.org/632140 | 17:43 |
| corvus | mordred: good point -- though i think when it comes down to it, it's about letting reviewers know that special testing is required -- so if it's not being tested any other way, if it's in *our* base job it needs that testing, regardless of whether it's in any others. but i'm fine delaying implementation of that idea for a little while until the set is minimized. | 17:44 |
| AJaeger | mordred: do you need a review for the next base-job rework change? | 17:46 |
| AJaeger | corvus: is it safe to approve https://review.openstack.org/631848 (docker: add ability to restrict repository names)? Then, I'll do so - I wondered whether it needed a restart or so... | 17:47 |
| corvus | AJaeger: it's safe, thanks | 17:48 |
| *** mriedem has quit IRC | 17:50 | |
| *** gfidente has quit IRC | 17:51 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add python-builder docker image https://review.openstack.org/631878 | 17:52 |
| AJaeger | mordred: jobs are passing again, your fix works! Thanks! | 17:52 |
| mordred | AJaeger: yay! | 17:52 |
| *** logan- has quit IRC | 17:52 | |
| mordred | sorry for having broken them :( | 17:52 |
| *** logan_ has joined #openstack-infra | 17:53 | |
| *** logan_ is now known as logan- | 17:53 | |
| openstackgerrit | Merged openstack-infra/project-config master: Run puppetforge publish playbook in the clouds https://review.openstack.org/632060 | 17:54 |
| openstackgerrit | Merged openstack-infra/project-config master: stx-test: Add new repo for test tools https://review.openstack.org/631884 | 17:55 |
| corvus | fungi, AJaeger, tobias-urdin: can you elaborate on the problem that https://review.openstack.org/632060 solves? | 17:55 |
| *** smarcet has quit IRC | 17:55 | |
| corvus | i can't figure out the full context from the commit message and i don't see any links | 17:56 |
| openstackgerrit | Merged openstack-infra/project-config master: Retire puppet-stackalytics (part 3) https://review.openstack.org/631819 | 17:56 |
| *** cristoph has joined #openstack-infra | 17:56 | |
| mordred | corvus: puppet publish needs ruby and other stuff installed | 17:57 |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Add ansible-role-ansible project https://review.openstack.org/632109 | 17:58 |
| mordred | corvus: it would be potentially possible to add the puppet publication stuff to the executors | 17:58 |
| fungi | corvus: apaprently publishing to puppetforge relies on a rake task | 17:58 |
| mordred | yeah | 17:58 |
| corvus | ok. i didn't get that from "suppresses the facts" | 17:58 |
| fungi | so needs a compiler, some headers... | 17:59 |
| mordred | the part of that commit message about facts is actually not the problem - it was the original failure symptom | 17:59 |
| fungi | yeah the fact suppression was what led us to realize it was trying to do that on the executor | 17:59 |
| fungi | the goal of using the distro family fact was to decide which packages to install to get ruby and compilers and make | 18:00 |
| fungi | but i agree the commit message was confused | 18:00 |
| *** bhavikdbavishi has quit IRC | 18:00 | |
| *** priteau has quit IRC | 18:01 | |
| corvus | i know we're in a phase in our community where we don't like to -1 things because of "inconsequential" things like commit messages. but maybe if they are very misleading we could correct them, or at least leave a comment. that way our commit history can be a repository of knowledge rather than of misinformation. :) | 18:01 |
| *** derekh has quit IRC | 18:01 | |
| corvus | (after seeing that, i really thought there was some bug that needed fixing) | 18:02 |
| corvus | i've left a comment on the change for posterity | 18:03 |
| fungi | as did i | 18:03 |
| fungi | thanks! | 18:03 |
| *** smarcet has joined #openstack-infra | 18:04 | |
| *** shardy has quit IRC | 18:05 | |
| *** electrofelix has quit IRC | 18:05 | |
| *** lpetrut has quit IRC | 18:06 | |
| fungi | corvus: bad news... i just got a bunch of bounces back to openstack-discuss complaining "Unauthenticated email from pnnl.gov is not accepted due to domain's DMARC policy." with regard to http://lists.openstack.org/pipermail/openstack-discuss/2019-January/001753.html | 18:06 |
| *** openstackgerrit has quit IRC | 18:07 | |
| corvus | fungi: bummer, who's rejecting it? | 18:07 |
| fungi | gmail and others | 18:07 |
| *** diablo_rojo has joined #openstack-infra | 18:08 | |
| fungi | the DKIM-Signature header includes h=from:to:subject:date:message-id:references:in-reply-to:content-transfer-encoding:mime-version; so i'm guessing mailman mangled one of those | 18:08 |
| *** kjackal has joined #openstack-infra | 18:09 | |
| corvus | fungi: 'content-transfer-encoding' and 'mime-version' are insane things to sign. | 18:11 |
| *** openstackgerrit has joined #openstack-infra | 18:11 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul master: Improve exception handling of fingerclient https://review.openstack.org/632140 | 18:11 |
| mordred | corvus: ++ | 18:11 |
| clarkb | fwiw we don't seem to touch from, to, subject, message-id based on a random discuss email I looked at | 18:12 |
| corvus | fungi: i wonder if there's any way for us to find the 'original' message? | 18:12 |
| mordred | otoh - they also seem like weird things for mailman to mangle | 18:12 |
| fungi | corvus: i agree, maybe kfox111 can speak to pnnl's decision to sign those | 18:12 |
| corvus | fungi: do you think we can ask kevin if he has a 'sent' copy of that message? | 18:12 |
| fungi | likely | 18:12 |
| corvus | maybe he can bounce us a copy so we can inspect it | 18:12 |
| corvus | mordred: i agree, which is why i'm inclined to continue a technical investigation a bit further | 18:13 |
| *** jpena is now known as jpena|off | 18:13 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Build zuul containers with dockerfile not pbrx https://review.openstack.org/631840 | 18:14 |
| mordred | corvus: ++ | 18:14 |
| *** diablo_rojo has quit IRC | 18:14 | |
| corvus | fungi: you want to ask kfox or shall i? | 18:14 |
| fungi | i probably have time before my flight boards | 18:14 |
| corvus | fungi: cool, feel free to cc me and i can try to pitch in | 18:15 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: docker: add ability to restrict repository names https://review.openstack.org/631848 | 18:15 |
| fungi | thanks. i'm hesitant to cc the infra ml lest we unsibscribe a bunch of people from it when he replies | 18:15 |
| corvus | fungi: yeah, i think this is best handled privately for technical reasons | 18:15 |
| anteaya | I never got that email in my inbox | 18:18 |
| fungi | it seems lots of people didn't | 18:19 |
| anteaya | the last email I got from kevin fox was this one: http://lists.openstack.org/pipermail/openstack-discuss/2018-December/000553.html | 18:21 |
| anteaya | seems something happened between then and now | 18:21 |
| *** ccamacho has quit IRC | 18:33 | |
| corvus | DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pnnl.gov; s=s20171204; h=from:to:subject:date:message-id:references:in-reply-to:content-transfer-encoding:mime-version; | 18:34 |
| mnaser | maybe the mail servers are off because they're not paid #toosoon | 18:35 |
| corvus | fungi, anteaya: that's the dkim header i have from that message. it appears unchanged from the current set of headers | 18:35 |
| corvus | mnaser: heh, that's actually a legit avenue to explore -- maybe a signing key expired? | 18:36 |
| mnaser | i know a few certs did expire over the past few weeks | 18:36 |
| mnaser | someone was talking about how nice HSTS locked them out | 18:37 |
| anteaya | I wonder what would have changed then | 18:37 |
| anteaya | that might do it | 18:38 |
| clarkb | would certainly explain how it failed even though the rules didn't change | 18:39 |
| *** ijw has joined #openstack-infra | 18:39 | |
| fungi | corvus: i've sent the request along with a copy of the relevant returned headers, and cc'd you | 18:40 |
| fungi | oh, yeah, a cert renewal dead-man's switch triggered by the usg shutdown might 'splain it | 18:41 |
| *** ijw has quit IRC | 18:44 | |
| *** redrobot has joined #openstack-infra | 18:44 | |
| *** ccamacho has joined #openstack-infra | 18:50 | |
| *** ykarel has quit IRC | 18:53 | |
| fungi | smarcet: so looking at the missing legacy-laravel-openstackid-release-branch build you were asking about from when https://git.openstack.org/cgit/openstack-infra/openstackid/commit/?id=cb3fee4 merged back on the 15th, we can try reenqueuing that ref | 18:56 |
| *** jcoufal_ has joined #openstack-infra | 18:57 | |
| fungi | i'm at a bit of a loss as to why it would have been skipped, since https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/projects.yaml#n500 looks pretty straightforward to me and doesn't seem to have changed between then and now | 18:57 |
| fungi | and digging in status and irc logs i don't see any mention of zuul scheduler restarts which would have lost it around the time it merged (there was a restart a couple hours before and another half a day later) | 18:58 |
| * prometheanfire wonders if there's an infra-status topic that could be set | 19:00 | |
| *** ijw has joined #openstack-infra | 19:00 | |
| *** jcoufal has quit IRC | 19:01 | |
| fungi | smarcet: looks like we do have zuul service logs back that far, so i'll take a look in there first to see if anything jumps out at me | 19:02 |
| *** dpawlik has joined #openstack-infra | 19:03 | |
| fungi | zuul.log.11.gz is where that'll be logged until the next logrotate runs | 19:04 |
| *** ijw has quit IRC | 19:04 | |
| fungi | no mention of legacy-laravel-openstackid-release-branch in there nor in debug.log.11.gz | 19:07 |
| fungi | prometheanfire: topic set for what? | 19:07 |
| *** dpawlik has quit IRC | 19:07 | |
| prometheanfire | just assumed something was off with zuul https://review.openstack.org/632026 | 19:08 |
| fungi | not that i've heard | 19:08 |
| prometheanfire | ah, ton of post fails | 19:08 |
| fungi | prometheanfire: oh, that, right | 19:08 |
| fungi | it was fixed a while back... checking | 19:08 |
| prometheanfire | :D | 19:09 |
| *** dkehn has quit IRC | 19:09 | |
| fungi | prometheanfire: yeah, fix merged at 17:33 utc | 19:09 |
| fungi | sorry, my brain had already moved on ;) | 19:09 |
| prometheanfire | :D | 19:10 |
| prometheanfire | rechecking things then | 19:10 |
| fungi | i need to drop offline for a bit and prepare to board, but can try to troubleshoot the missing legacy-laravel-openstackid-release-branch build more from the air | 19:10 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add python-builder docker image https://review.openstack.org/631878 | 19:10 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Add a minimal dockerignore file https://review.openstack.org/631929 | 19:10 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Build zuul containers with dockerfile not pbrx https://review.openstack.org/631840 | 19:10 |
| *** yboaron_ has joined #openstack-infra | 19:11 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Build zuul containers with dockerfile not pbrx https://review.openstack.org/631840 | 19:13 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use sudo -E in install-js-tools only if not root https://review.openstack.org/632148 | 19:13 |
| *** eernst has joined #openstack-infra | 19:13 | |
| clarkb | we could do a #status notice The error causing post failures on jobs has been corrected. It is safe to recheck these jobs. | 19:14 |
| prometheanfire | up to you | 19:14 |
| clarkb | prometheanfire: ^ is that what you are asking for? and mordred ^ is that accurate? if so I can send it | 19:14 |
| mordred | yes | 19:15 |
| * mordred supports sending it | 19:15 | |
| prometheanfire | I generally would expect it (why I asked) | 19:15 |
| clarkb | #status notice The error causing post failures on jobs has been corrected. It is safe to recheck these jobs. | 19:15 |
| openstackstatus | clarkb: sending notice | 19:15 |
| -openstackstatus- NOTICE: The error causing post failures on jobs has been corrected. It is safe to recheck these jobs. | 19:17 | |
| *** eernst has quit IRC | 19:17 | |
| *** eernst has joined #openstack-infra | 19:17 | |
| *** dpawlik has joined #openstack-infra | 19:19 | |
| openstackstatus | clarkb: finished sending notice | 19:19 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/project-config master: Add gate job for Slack notifier in zuul-jobs https://review.openstack.org/623593 | 19:19 |
| *** dpawlik has quit IRC | 19:23 | |
| *** ijw has joined #openstack-infra | 19:25 | |
| corvus | i'd like to perform a scheduler restart to pick up pass-to-parent | 19:26 |
| *** eernst has quit IRC | 19:26 | |
| corvus | though, perhaps i should restart the whole system in anticipation of making a release | 19:26 |
| corvus | #status log restarted zuul at 691b1bc17c77ebce5b2a568e586d19b77cebbc7b | 19:30 |
| openstackstatus | corvus: finished logging | 19:30 |
| *** rh-jelabarre has quit IRC | 19:31 | |
| *** mriedem has joined #openstack-infra | 19:35 | |
| corvus | restart/re-enqueue complete | 19:45 |
| *** markvoelker has joined #openstack-infra | 19:46 | |
| *** rh-jelabarre has joined #openstack-infra | 19:50 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add python-builder docker image https://review.openstack.org/631878 | 19:50 |
| pabelanger | corvus: I am struggling to figure out why zuul isn't allowing pipelines in untrusted projects: https://review.openstack.org/631983/ | 19:56 |
| pabelanger | http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/main.yaml#n27 should be the configuration it is running now | 19:56 |
| pabelanger | corvus: I wonder if it is because the name is check | 19:57 |
| *** dpawlik has joined #openstack-infra | 19:58 | |
| pabelanger | nope :( | 19:59 |
| *** dpawlik has quit IRC | 20:02 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use sudo -E in install-js-tools only if not root https://review.openstack.org/632148 | 20:03 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Build zuul containers with dockerfile not pbrx https://review.openstack.org/631840 | 20:03 |
| sean-k-mooney | hi so is the gate broken in some way? https://review.openstack.org/#/c/631829/ | 20:04 |
| sean-k-mooney | all of the test seam to have passed but all of them are in post_failure in the check pipline | 20:05 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add python-builder docker image https://review.openstack.org/631878 | 20:06 |
| *** xek_ has quit IRC | 20:09 | |
| mordred | sean-k-mooney: yeah - there was an issue earlier - it has been rectified and you can recheck those patches. sorry about that! | 20:10 |
| sean-k-mooney | mordred: no need to applogies | 20:10 |
| sean-k-mooney | i was just perplexed as the logs had no errors | 20:11 |
| mordred | yeah... this is what happens when you land a chage to the log collection code without proper testing :) | 20:11 |
| sean-k-mooney | mordred: debuging logging without logs is always well challaning | 20:12 |
| mordred | it's not idea to say the least! | 20:12 |
| sean-k-mooney | cool ill recheck and then the k8s job will proably blow up again. | 20:13 |
| *** whoami-rajat has quit IRC | 20:15 | |
| *** eharney has quit IRC | 20:16 | |
| corvus | pabelanger: see starting at pgraph 4 of https://zuul-ci.org/docs/zuul/user/config.html#pipeline | 20:19 |
| *** markvoelker has quit IRC | 20:20 | |
| pabelanger | corvus: Hmm, so even if excluded, there is no way to have it in untrusted. This is change up my 3pci testing a little for pipelines | 20:22 |
| pabelanger | I am a little curious why https://review.openstack.org/631990 appears to be allow it, I must have done something wrong with syntax | 20:23 |
| *** cristoph has quit IRC | 20:24 | |
| pabelanger | I suspect I didn't validate properly on 631990 | 20:24 |
| corvus | pabelanger: 631990 does not allow it, you excluded pipelines. | 20:24 |
| *** ianw is now known as ianw_pto | 20:26 | |
| pabelanger | corvus: sorry, misunderstanding you. Are you saying, excluded pipelines are allowed / not allowed in untrusted projects | 20:26 |
| corvus | pabelanger: if they are excluded, then they aren't there, so there's nothing to allow or disallow | 20:27 |
| pabelanger | corvus: okay, in that case http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/main.yaml#n27 should be valid syntax right? | 20:28 |
| corvus | pabelanger: "exclude" means "don't load this configuration object", so there's nothing for zuul to object to | 20:28 |
| corvus | pabelanger: sure that's fine | 20:29 |
| corvus | (why wouldn't it be?) | 20:29 |
| pabelanger | okay, so that case I don't know why zuul is rejecting 631983 | 20:29 |
| pabelanger | my brain isn't making the connection here | 20:29 |
| corvus | pabelanger: well *that* seems like a bug | 20:32 |
| *** jtomasek has quit IRC | 20:32 | |
| *** rh-jelabarre has quit IRC | 20:33 | |
| pabelanger | okay, I'll dig more into it locally and see why that is. I just wanted to make sure my syntax was correct in main.yaml | 20:33 |
| pabelanger | need to #dadops for a bit here | 20:33 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul-jobs master: Propose some job writing guidelines https://review.openstack.org/631507 | 20:35 |
| *** rh-jelabarre has joined #openstack-infra | 20:36 | |
| * tobias-urdin reading backlog | 20:40 | |
| tobias-urdin | corvus: sorry, that commit message was pretty bad, i didn't understand all details but wanted to keep the ball rolling, should have included IRC logs | 20:41 |
| tobias-urdin | now that it's merged, could we do a test release again to see if it works? digging through the logs fungi tested with the following to retest the 14.2.0 release | 20:41 |
| tobias-urdin | 19:40 < fungi> sudo zuul enqueue-ref --tenant=openstack --trigger=gerrit --pipeline=release --project=openstack/puppet-aodh --ref=refs/tags/14.2.0 --newrev=617ffad84b633618490ca1023f8a31d9694b31a9 | 20:41 |
| fungi | tobias-urdin: yep, just a sec | 20:44 |
| fungi | tobias-urdin: i've reenqueued that tag | 20:45 |
| tobias-urdin | *crossing my fingers* | 20:45 |
| *** priteau has joined #openstack-infra | 20:51 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Install python requirements directly https://review.openstack.org/632162 | 20:52 |
| mordred | tobiash: ^^ I think that takes care of your use case? | 20:54 |
| tobiash | trying out | 20:54 |
| tobias-urdin | fungi: http://logs.openstack.org/61/617ffad84b633618490ca1023f8a31d9694b31a9/release/release-openstack-puppet/c6e519d/job-output.txt.gz#_2019-01-21_20_53_17_207155 | 20:56 |
| fungi | tobias-urdin: http://logs.openstack.org/61/617ffad84b633618490ca1023f8a31d9694b31a9/release/release-openstack-puppet/c6e519d/ara-report/result/6311c199-c059-486a-be76-8dacf273819c/ | 20:57 |
| tobias-urdin | never seen that one before | 20:57 |
| *** e0ne has joined #openstack-infra | 20:57 | |
| fungi | does that need a "become" so it runs as root? | 20:59 |
| tobias-urdin | hm, i guess we need to | 21:00 |
| tobias-urdin | since it tries to install system packages | 21:00 |
| fungi | tobias-urdin: yeah, here's an example: https://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/bindep/tasks/packages.yaml#n6 | 21:01 |
| fungi | looks like the package task needs explicit elevation | 21:02 |
| *** jcoufal_ has quit IRC | 21:03 | |
| *** priteau has quit IRC | 21:04 | |
| tobias-urdin | hm yea, i assume it would be bad to "become" the whole publish playbook instead of changing the role | 21:04 |
| tobias-urdin | i'll try adding the same "become" part there to the role | 21:05 |
| fungi | okay, i've got a new puzzle: | 21:14 |
| fungi | Exception: Unable to modify final job <Job publish-tox-docs-infra branches: None source: openstack-infra/project-config/zuul.d/jobs.yaml@master#26> attribute nodeset=ubuntu-bionic with variant <Job publish-tox-docs-infra branches: None source: openstack-infra/project-config/zuul.d/projects.yaml@master#500> | 21:14 |
| fungi | https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/projects.yaml#n500 | 21:15 |
| *** ijw has quit IRC | 21:15 | |
| *** ssbarnea|bkp2 has joined #openstack-infra | 21:16 | |
| *** ijw has joined #openstack-infra | 21:16 | |
| fungi | oh! "final job" | 21:16 |
| *** yboaron_ has quit IRC | 21:17 | |
| *** markvoelker has joined #openstack-infra | 21:17 | |
| fungi | since publish-tox-docs-infra is marked final, the attempt to override its nodeset is disallowed, i guess? | 21:17 |
| openstackgerrit | Tobias Urdin proposed openstack-infra/zuul-jobs master: Use become for packages in upload-puppetforge https://review.openstack.org/632163 | 21:17 |
| *** ssbarnea|rover has quit IRC | 21:18 | |
| tobias-urdin | fungi: ^ tested the whole playbook as the unprivileged "vagrant" user in a vagrant box | 21:18 |
| fungi | thanks tobias-urdin! | 21:19 |
| tobias-urdin | right back at you, thanks for bearing with me :) | 21:20 |
| fungi | project-config-core: i wonder whether we ought to consider moving our infra docs jobs and some other low-hanging fruit to bionic anyway? | 21:21 |
| mordred | fungi: ++ | 21:22 |
| fungi | though with the job marked final i suppose we have to do it all at once or transition between version-specific jobs | 21:22 |
| mordred | fungi: I would hope that all of our infra docs jobs would build properly on bionic | 21:22 |
| fungi | yeah, the only per-project iffyness is if we have some repos with a bindep.txt which isn't viable on bionic for some reasons | 21:23 |
| *** kjackal has quit IRC | 21:23 | |
| mordred | fungi: great way to find that out :) | 21:24 |
| fungi | for openstackid at least, the tox-docs job is working with nodeset: ubuntu-bionic but publish-tox-docs-infra is marked final so can't say for sure it works on bionic though i'd be surprised | 21:26 |
| *** kgiusti has quit IRC | 21:28 | |
| *** ccamacho has quit IRC | 21:28 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Build zuul containers with dockerfile not pbrx https://review.openstack.org/631840 | 21:29 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use node v10 instead of node v8 https://review.openstack.org/632165 | 21:30 |
| fungi | hrm, though it (as most others obviously) inherits that nodeset from the base job | 21:31 |
| fungi | at some point we need to flip the switch on our base job i guess | 21:31 |
| fungi | i know this is a terrible couple weeks for me to try and discuss it, but what are people' thoughts there? i suppose we should coordinate with #openstack-qa? | 21:33 |
| clarkb | the -qa team already flipped devstack/tempest jobs | 21:33 |
| clarkb | I think its largely a matterof individual projects now. | 21:34 |
| clarkb | so maybe wecan just do it | 21:34 |
| clarkb | we'll want to pin the py35 job to xenial first if it doesnt do that already | 21:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Remove zuul-migrate and zuul-bwrap images https://review.openstack.org/632167 | 21:40 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config master: Add publish-tox-docs-infra-bionic https://review.openstack.org/632168 | 21:40 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul-jobs master: Propose some job writing guidelines https://review.openstack.org/631507 | 21:41 |
| fungi | smarcet: clarkb: 632168 is a possible temporary workaround for the openstackid jobs | 21:41 |
| fungi | if we don't think nowish is a suitable time to switch everything | 21:42 |
| clarkb | considering I wont be able to debug any fallout it probably is a bad time for me | 21:44 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Add docker image build jobs https://review.openstack.org/632172 | 21:47 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Switch to zuul-jobs docker jobs https://review.openstack.org/632173 | 21:48 |
| *** markvoelker has quit IRC | 21:49 | |
| clarkb | fungi do we know why the nodeset variant we had wasnt working? | 21:51 |
| fungi | clarkb: final: true | 21:51 |
| *** e0ne has quit IRC | 21:51 | |
| fungi | (see exception above) | 21:51 |
| clarkb | also insteadof redefining the whole job can we parent to the exiwting job and justchange the nodeset? ah tbat explains both questions | 21:52 |
| fungi | had to dig it out of the zuul scheduler debug log | 21:52 |
| fungi | i tried to cover the reasons in the job description | 21:52 |
| fungi | lmk if that's unclear | 21:52 |
| fungi | or otherwise lacks sufficient context | 21:52 |
| clarkb | nope its clear, Im just on a phone so slow at grokking | 21:52 |
| fungi | i'm hurtling through the air at 250 meters per second | 21:53 |
| fungi | so can sympathize | 21:53 |
| fungi | science! | 21:53 |
| *** rcernin has joined #openstack-infra | 21:54 | |
| fungi | relatively speaking, it's the surface of the earth which is moving at 250m/s and i'm just a fixed point in space with some odd rotational geometries to contend with | 21:54 |
| clarkb | riding the planet express? | 21:55 |
| *** sean-k-mooney has quit IRC | 21:56 | |
| fungi | hey, that engine tech is at least based in real science fiction | 21:57 |
| *** sean-k-mooney has joined #openstack-infra | 21:58 | |
| *** dpawlik has joined #openstack-infra | 21:59 | |
| *** dpawlik has quit IRC | 22:03 | |
| *** markvoelker has joined #openstack-infra | 22:08 | |
| *** spsurya has quit IRC | 22:09 | |
| *** dpawlik has joined #openstack-infra | 22:15 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Build zuul containers with dockerfile not pbrx https://review.openstack.org/631840 | 22:18 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Switch to zuul-jobs docker jobs https://review.openstack.org/632173 | 22:19 |
| *** agopi has quit IRC | 22:20 | |
| *** dpawlik has quit IRC | 22:20 | |
| corvus | mordred: do we have an openstack/ org on dockerhub? | 22:22 |
| mordred | corvus: yes | 22:23 |
| mordred | corvus: I squatted it a while back | 22:23 |
| mordred | corvus: should I add openstackzuul to it? | 22:24 |
| corvus | mordred: at some point we might want to set up a credential for it and make a generic openstack-upload-docker-image job... we can do that now, but i don't think there's any urgency... | 22:24 |
| corvus | i guess if we do, we can announce that | 22:25 |
| corvus | mordred: i think we may need a different credential for the zuul project | 22:25 |
| openstackgerrit | Merged openstack-infra/zuul master: Add a minimal dockerignore file https://review.openstack.org/631929 | 22:25 |
| mordred | corvus: yeah - I think the new tools we've got open the door to having multiple credentials instead of just one account for zuul | 22:26 |
| corvus | mordred: i think adding the credential to the zuul repo may have crossed a security domain (ie, you retrieved the credential using your opendev hat, and added it to the zuul repo wearing your zuul hat) | 22:26 |
| *** smarcet has quit IRC | 22:26 | |
| mordred | corvus: yes - I think that is an excellent point | 22:27 |
| corvus | i think we can trust the zuul project not to compromise it, but, strictly speaking, any zuul maintainer now does have access to it. | 22:27 |
| corvus | (i mean, we'd *know* if they decrypted it, it'd be right there in the git log) | 22:28 |
| mordred | corvus: yah - we should grow some additional more specific credentials | 22:28 |
| mordred | corvus: :) | 22:28 |
| mordred | corvus: oh - not quite related to that - but sort of (Because it's related to dockerhub) - in your dockerhub api explorations, you didn't find any undocumented API methods for adding a long description did you? so far I've only been able to find people saying that it only gets populated when dockerhub does builds from your github | 22:29 |
| corvus | mordred: not yet, though i only briefly looked into that | 22:29 |
| openstackgerrit | Merged openstack-infra/zuul master: Use sudo -E in install-js-tools only if not root https://review.openstack.org/632148 | 22:29 |
| openstackgerrit | Merged openstack-infra/zuul master: Improve exception handling of fingerclient https://review.openstack.org/632140 | 22:30 |
| corvus | mordred: so maybe let's keep 'openstackzuul' to be 'the zuul that uploads to projects in the openstack tenant', and make... erm... 'zuulzuul' for the zuul tenant? and what about opendev? should we make a third for that or share one of the others? | 22:32 |
| *** priteau has joined #openstack-infra | 22:34 | |
| corvus | mordred: (alternatively, we can share a credential, we'd just need to keep the zuul-tenant jobs in a central repo, and make new jobs for each tenant). | 22:36 |
| mordred | corvus: I think we should make a zuulzuul and an opendevzuul :) | 22:36 |
| mordred | corvus: but just because it means we can talk about zuulzuul | 22:36 |
| corvus | okay, i'm making zuulzuul | 22:37 |
| anteaya | I vote for zuulzuulzuul | 22:38 |
| anteaya | if that fits in anywhere | 22:38 |
| corvus | mordred: can you add zuulzuul to zuul/ and nodepool/ ? | 22:39 |
| anteaya | maybe that is the next level round | 22:39 |
| mordred | corvus: yah | 22:39 |
| mordred | corvus: zuulzuul added to zuul (which I think makes the zuulzuulzuul concept anteaya wants :) ) | 22:40 |
| *** dkehn has joined #openstack-infra | 22:40 | |
| anteaya | moar zuul | 22:41 |
| mordred | corvus: there is no nodepool/ - only zuul/ | 22:41 |
| anteaya | thank you | 22:41 |
| corvus | mordred: excellent | 22:41 |
| corvus | mordred: i'll go ahead and remove openstackzuul, even though it will break things briefly, ok? | 22:41 |
| mordred | corvus: ++ | 22:42 |
| corvus | mordred: i'm going to update 631840 with the new credential; make sure you check that out locally if you make any more updates to it, as it will fail in gate with the wrong cred | 22:44 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Build zuul containers with dockerfile not pbrx https://review.openstack.org/631840 | 22:46 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Remove zuul-migrate and zuul-bwrap images https://review.openstack.org/632167 | 22:46 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Switch to zuul-jobs docker jobs https://review.openstack.org/632173 | 22:46 |
| *** agopi has joined #openstack-infra | 22:48 | |
| *** mriedem has quit IRC | 22:51 | |
| *** agopi has quit IRC | 22:53 | |
| *** agopi has joined #openstack-infra | 22:53 | |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Update dockerhub credential https://review.openstack.org/632183 | 22:58 |
| mordred | corvus: ++ | 22:59 |
| corvus | mordred: i think you should go ahead and +3 623183 since i've revoked the other creds | 22:59 |
| mordred | corvus: openstackzuul has been added to openstack | 23:00 |
| corvus | mordred: thanks! | 23:00 |
| corvus | i've updated the password store with all the new info | 23:00 |
| mordred | corvus: woot! | 23:01 |
| corvus | fungi: if you have time to drop a +2 on https://review.openstack.org/632162 as well, that would be swell | 23:02 |
| fungi | you bet | 23:03 |
| corvus | once 183 lands, i'll approve those two | 23:03 |
| *** tkajinam has joined #openstack-infra | 23:03 | |
| fungi | though we're descending, so i may get it at my next gate if they cut me off | 23:03 |
| fungi | lgtm, +2! | 23:06 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Allow nodepool to use zuul-*-image jobs https://review.openstack.org/632186 | 23:06 |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Build images with Dockerfile instead of pbrx https://review.openstack.org/632187 | 23:15 |
| openstackgerrit | Merged openstack-infra/project-config master: Add Cinder Storage charm interface https://review.openstack.org/631252 | 23:16 |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Build images with Dockerfile instead of pbrx https://review.openstack.org/632187 | 23:16 |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Build images with Dockerfile instead of pbrx https://review.openstack.org/632187 | 23:23 |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Stop building an explicit nodepool-base image https://review.openstack.org/632188 | 23:23 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Stop building an explicit zuul-base image https://review.openstack.org/632189 | 23:26 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool master: Build images with Dockerfile instead of pbrx https://review.openstack.org/632187 | 23:28 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool master: Stop building an explicit nodepool-base image https://review.openstack.org/632188 | 23:30 |
| *** e0ne has joined #openstack-infra | 23:34 | |
| *** slaweq has quit IRC | 23:35 | |
| mordred | corvus: does the failure in https://review.openstack.org/#/c/632183 make any sense to you? | 23:36 |
| mordred | I'm not seeing it | 23:37 |
| *** rkukura has joined #openstack-infra | 23:37 | |
| *** e0ne has quit IRC | 23:39 | |
| corvus | mordred: i think there are ongoing issues with puppet on trusty? i think the 'failed to copy modules' thing is a known random failure | 23:40 |
| *** ekultails has quit IRC | 23:40 | |
| mordred | corvus: gtocha. so recheck is safe/fair for now | 23:42 |
| *** cristoph has joined #openstack-infra | 23:45 | |
| cristoph | Hi, can someone please add me as reviewer to the group "starlingx-test-core" ? I created the review: https://review.openstack.org/#/c/631884/ already merged | 23:51 |
| corvus | cristoph: done | 23:57 |
| cristoph | thanks a lot | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!