fungi | could be | 00:00 |
---|---|---|
fungi | https://sks-keyservers.net/pks/lookup?op=vindex&search=0x27023b1ffccd8e3ae9a5ce95d943d5d270273ada&fingerprint=on is what i'll be linking on the release site anyway | 00:01 |
fungi | it's not only got the stein key but also signatures from the rocky key and from me | 00:01 |
clarkb | http://keyserver.ubuntu.com:11371/pks/lookup?search=0xD943D5D270273ADA&op=vindex shows my signature too | 00:02 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove job settings for neutron repositories https://review.openstack.org/597865 | 00:05 |
fungi | An error occurred during a connection to keyserver.ubuntu.com:11371. SSL received a record that exceeded the maximum permissible length. Error code: SSL_ERROR_RX_RECORD_TOO_LONG | 00:13 |
pabelanger | clarkb: I'm trying to get testinfra to work in https://review.openstack.org/593029/ first | 00:13 |
pabelanger | need to figure out pytest fixtures to run tests against a specific host | 00:14 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Bug on endpoint /api/v1/summits/{id}/events/published/empty-spots https://review.openstack.org/597965 | 00:19 |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Bug on endpoint /api/v1/summits/{id}/events/published/empty-spots https://review.openstack.org/597965 | 00:20 |
*** dingyichen has joined #openstack-infra | 00:25 | |
*** longkb has joined #openstack-infra | 00:26 | |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: wheel-build: only install afs client for publish jobs https://review.openstack.org/597966 | 00:26 |
ianw | tonyb: ^ so i think it's that easy | 00:26 |
ianw | pabelanger: i did that for the logrotate work | 00:26 |
ianw | you can inspect the ansible host variables | 00:26 |
tonyb | ianw: Looks good to me. I'm still unsure why we run those jobs when we update bindep.txt but that's a discussion for another time. | 00:29 |
ianw | tonyb: i'd say that makes sense, it makes sure the wheel builds will still work | 00:30 |
tonyb | ianw: Okay. | 00:32 |
tonyb | ianw: I have updated the 2 chnages that touch bindep.txt in openstack/requirements to depnds-on your change | 00:32 |
ianw | tonyb: unfortunately it won't pull in project-config changes. but i think we could just quick merge that, and i can trigger a periodic job and we can check it out. otherwise we're in a multi-day debug cycle anyway waiting for jobs | 00:34 |
tonyb | ianw: Oh :( I though all the zuulness was dynamic like that :/ | 00:35 |
tonyb | ianw: If you're okay with a quick merge I am ;P | 00:35 |
ianw | it just get complicated where secrets and post-only jobs get involved. this straddles it all so i dunno :) | 00:37 |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk https://review.openstack.org/593029 | 00:38 |
pabelanger | ianw: thanks, one last test then I'll try that. But likely morning now, based on size of check pipeline | 00:42 |
ianw | pabelanger: decorator looks good too if it works :) | 00:43 |
pabelanger | lets hope | 00:43 |
ianw | what's in the queue that we have 800 centos-7 nodes going? | 00:49 |
fungi | tonyb: ianw: yeah, since project-config contains a pile of sensitive configuration and secrets, the way those get protected from disclosure or dangerous fiddling is by only executing them on changes which have been reviewed and merged. allowing anybody to propose a job change which dumped access credentials for our services to the console log would _not_ be good to just run automatically | 00:49 |
*** efried1 has joined #openstack-infra | 00:50 | |
tonyb | fungi: Yup I can see that. I | 00:50 |
tonyb | 'll probably need to be told more than once befoer I remember it ;P | 00:51 |
*** efried has quit IRC | 00:51 | |
*** efried1 is now known as efried | 00:51 | |
fungi | however, it makes the case that everything we _can_ safely move out of that repository is something which can be tested in real time and not hold people up | 00:51 |
fungi | so the less we put in there, the better | 00:51 |
tonyb | fungi: cool. That makes a lot of sense | 00:54 |
clarkb | ianw: tripleo? | 00:55 |
ianw | yeah, bits of the afs build could conceivably be moved into o-z-j | 00:55 |
ianw | clarkb: yeah. do you know what the limestone-* nodes are doing in http://grafana.openstack.org/d/rZtIH5Imz/nodepool?panelId=13&fullscreen&orgId=1 ? | 00:55 |
clarkb | hrm that doesnt render great on a phone | 00:58 |
*** efried has quit IRC | 00:58 | |
clarkb | so no idea :) | 00:58 |
ianw | i'm thinking might just need to rm those stats on disk if they snuck in there at some point | 00:59 |
ianw | "Remove limestone specific labels from nl02.o.o" | 01:00 |
ianw | yeah, looks like they were just for testing at one point | 01:00 |
*** signed8bit_Zzz is now known as signed8bit | 01:07 | |
ianw | ok, manually removed those, and fedora-26. graphs are much less busy now | 01:08 |
ianw | oh there's a few more old fedoras might as well clear while here | 01:09 |
openstackgerrit | Nguyen Hai proposed openstack-infra/project-config master: remove job settings for designate repositories https://review.openstack.org/592865 | 01:12 |
openstackgerrit | Nguyen Hai proposed openstack-infra/project-config master: remove job settings for designate repositories https://review.openstack.org/592865 | 01:13 |
openstackgerrit | Nguyen Hai proposed openstack-infra/project-config master: remove job settings for designate repositories https://review.openstack.org/592865 | 01:13 |
*** rh-jelabarre has quit IRC | 01:16 | |
openstackgerrit | David Moreau Simard proposed openstack-infra/nodepool master: WIP: Implement an Amazon EC2 driver https://review.openstack.org/535558 | 01:18 |
*** bobh has joined #openstack-infra | 01:19 | |
*** anteaya has joined #openstack-infra | 01:19 | |
*** efried has joined #openstack-infra | 01:20 | |
*** gyee has quit IRC | 01:22 | |
*** jiapei has joined #openstack-infra | 01:23 | |
*** signed8bit is now known as signed8bit_Zzz | 01:25 | |
openstackgerrit | Nguyen Hai proposed openstack-infra/project-config master: remove job settings for swift repositories https://review.openstack.org/596224 | 01:33 |
*** rlandy|bbl is now known as rlandy | 01:37 | |
*** bobh has quit IRC | 01:41 | |
*** rlandy has quit IRC | 01:42 | |
*** ijw has joined #openstack-infra | 01:46 | |
*** ijw has quit IRC | 01:51 | |
*** hongbin has joined #openstack-infra | 01:54 | |
*** jamesdenton has quit IRC | 01:59 | |
*** larainema has quit IRC | 02:00 | |
*** slaweq has joined #openstack-infra | 02:11 | |
*** bobh has joined #openstack-infra | 02:15 | |
*** slaweq has quit IRC | 02:16 | |
*** bobh has quit IRC | 02:16 | |
*** apetrich has quit IRC | 02:16 | |
*** bobh has joined #openstack-infra | 02:16 | |
*** anteaya has quit IRC | 02:19 | |
*** bobh has quit IRC | 02:20 | |
mnaser | so it looks like things have gone up to 50 | 02:41 |
mnaser | and no fires are up? | 02:41 |
*** ijw has joined #openstack-infra | 02:43 | |
*** ijw has quit IRC | 02:47 | |
*** psachin has joined #openstack-infra | 02:50 | |
*** bhavikdbavishi has joined #openstack-infra | 02:54 | |
*** signed8bit_Zzz is now known as signed8bit | 03:12 | |
*** signed8bit is now known as signed8bit_Zzz | 03:12 | |
*** ykarel has joined #openstack-infra | 03:24 | |
*** ykarel_ has joined #openstack-infra | 03:31 | |
*** jiapei has quit IRC | 03:33 | |
*** ykarel has quit IRC | 03:33 | |
*** rosmaita has quit IRC | 03:34 | |
*** bhavikdbavishi has quit IRC | 03:34 | |
*** nicolasbock has quit IRC | 03:40 | |
*** graphene has quit IRC | 03:49 | |
*** ykarel__ has joined #openstack-infra | 03:58 | |
*** ykarel_ has quit IRC | 03:58 | |
*** janki has joined #openstack-infra | 04:05 | |
openstackgerrit | Merged openstack-infra/project-config master: wheel-build: only install afs client for publish jobs https://review.openstack.org/597966 | 04:09 |
ianw | mnaser: things seem ok! gate is busy | 04:09 |
*** ramishra has joined #openstack-infra | 04:12 | |
openstackgerrit | Merged openstack-infra/irc-meetings master: Update Searchlight team meeting https://review.openstack.org/597537 | 04:14 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Fixes on member affiliation API https://review.openstack.org/597984 | 04:15 |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Fixes on member affiliation API https://review.openstack.org/597984 | 04:16 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Infra repo retirement of 18 repos - step 3 https://review.openstack.org/597636 | 04:17 |
openstackgerrit | Merged openstack-infra/zuul-packaging master: Retire zuul-packaging https://review.openstack.org/597531 | 04:18 |
*** ykarel__ has quit IRC | 04:22 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Infra repo retirement of 18 repos - step 3 https://review.openstack.org/597636 | 04:28 |
*** markvoelker has joined #openstack-infra | 04:34 | |
AJaeger | config-core, https://review.openstack.org/597371 and https://review.openstack.org/597651 are two small cleanups for openstack-zuul-jobs due to infra-retire. rewiews would be nice. | 04:48 |
AJaeger | config-core, and a couple of grafana updates: https://review.openstack.org/597168 https://review.openstack.org/595763 | 04:49 |
*** hongbin has quit IRC | 04:56 | |
*** ykarel__ has joined #openstack-infra | 04:58 | |
*** ykarel__ is now known as ykarel | 05:00 | |
*** jbadiapa has quit IRC | 05:01 | |
*** ijw has joined #openstack-infra | 05:06 | |
*** ijw has quit IRC | 05:10 | |
*** slaweq has joined #openstack-infra | 05:11 | |
*** shabeeb has joined #openstack-infra | 05:11 | |
openstackgerrit | Merged openstack-infra/project-config master: remove job settings for designate repositories https://review.openstack.org/592865 | 05:15 |
*** slaweq has quit IRC | 05:15 | |
*** srp has joined #openstack-infra | 05:23 | |
*** shabeeb has quit IRC | 05:23 | |
*** e0ne has joined #openstack-infra | 05:31 | |
openstackgerrit | Merged openstack-infra/project-config master: Neutron failure rate - include TIMED_OUT jobs on graphs https://review.openstack.org/595763 | 05:34 |
*** dingyichen has quit IRC | 05:35 | |
*** dingyichen has joined #openstack-infra | 05:37 | |
*** ccamacho has joined #openstack-infra | 05:38 | |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: wheel-build: ignore existing files and fix error log https://review.openstack.org/597989 | 05:38 |
ianw | tonyb: ^ | 05:38 |
tonyb | ianw: +1 Thanks! | 05:40 |
tonyb | ianw: and you fixed the off by 2 error ;P | 05:40 |
ianw | oh now i see what you mean :) | 05:40 |
ianw | i think we should merge that and try again. i think there might be other fails? | 05:41 |
*** ijw has joined #openstack-infra | 05:42 | |
ianw | ovs/_json.c:2:30: fatal error: openvswitch/json.h: No such file or directory | 05:43 |
ianw | stuff like that ... although that wasn't a hard failure | 05:43 |
openstackgerrit | Merged openstack-infra/project-config master: Add openstack/os-ken to neutron https://review.openstack.org/588355 | 05:46 |
*** ijw has quit IRC | 05:47 | |
tonyb | ianw: Yup I'm good to watch aother run or 3 | 05:50 |
ianw | actually | 05:51 |
tonyb | ianw: Some of the failures are more or less expected (on bionic) but xenial and centos shoudl be fine | 05:51 |
ianw | the thing that caused the trusty job to fail was http://logs.openstack.org/f4/f4f63270332e196e6df47f96e773614b149a405b/periodic/publish-wheel-mirror-ubuntu-trusty/d796d92/ara-report/result/375d42c6-7441-4f14-9925-a4c90138181c/ | 05:51 |
tonyb | ianw: Oh interesting | 05:52 |
openstackgerrit | Merged openstack-infra/project-config master: wheel-build: ignore existing files and fix error log https://review.openstack.org/597989 | 05:53 |
*** udesale has joined #openstack-infra | 05:53 | |
* tonyb deosn't really see how that's possible unless there was a network snafu | 05:54 | |
ianw | ... or the openafs client doesn't work | 05:55 |
ianw | no that's not right, it's actually working in the afs directory | 05:55 |
*** snapiri has joined #openstack-infra | 05:59 | |
ianw | oh, i bet the python 2 & 3 updates raced? | 06:00 |
tonyb | ianw: Yeah that could be it | 06:02 |
ianw | i don't know how to get around that :/ it does a cmp first and then a mv, but i don't think even that's atomic enough | 06:03 |
*** signed8bit_Zzz is now known as signed8bit | 06:06 | |
*** openstackgerrit has quit IRC | 06:07 | |
*** ramishra has quit IRC | 06:07 | |
*** ijw has joined #openstack-infra | 06:07 | |
*** udesale has quit IRC | 06:07 | |
*** udesale has joined #openstack-infra | 06:08 | |
*** ijw has quit IRC | 06:12 | |
*** openstackgerrit has joined #openstack-infra | 06:13 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/597996 | 06:13 |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: wheel-index: only create index once https://review.openstack.org/597997 | 06:14 |
*** ramishra has joined #openstack-infra | 06:14 | |
ianw | tonyb: ^ best guess. think should run that by pabelanger | 06:14 |
tonyb | ianw: Looks okay to me and I can see how the commit message macthes the code but that's about the depth of my review | 06:16 |
tonyb | ianw: it's waay deeper ansible mojo that I grok ;p | 06:16 |
*** e0ne has quit IRC | 06:19 | |
*** e0ne has joined #openstack-infra | 06:20 | |
*** e0ne has quit IRC | 06:20 | |
*** alexchadin has joined #openstack-infra | 06:23 | |
*** srp has quit IRC | 06:25 | |
*** srp has joined #openstack-infra | 06:26 | |
mpeterson | dhellmann: hey, just a heads up that the tool for python3-first is duplicating the same block of code (e.g. https://review.openstack.org/#/c/597873/1/.zuul.d/project.yaml ) | 06:27 |
AJaeger | mpeterson: where exactly? | 06:29 |
AJaeger | Oh, I see it... | 06:29 |
AJaeger | mpeterson: want to fix it manually yourself? Good catch! | 06:29 |
*** janki has quit IRC | 06:30 | |
*** jchhatbar has joined #openstack-infra | 06:30 | |
AJaeger | mpeterson: and py35 is now there three times - I left a -1 for both | 06:31 |
mpeterson | AJaeger: yeah, I'll fix it manually. I'm just leaving him a message so he can review the tool before proceeding to generate more patches | 06:31 |
mpeterson | AJaeger: yup, I saw that one as well, thanks! | 06:32 |
AJaeger | mpeterson: yep, that's an important heads-up | 06:32 |
*** shardy has joined #openstack-infra | 06:32 | |
AJaeger | mpeterson: and that py35 can be abandoned I guess - and then rebase the py36 on the other one | 06:32 |
*** signed8bit has quit IRC | 06:35 | |
*** srp has quit IRC | 06:36 | |
*** srp has joined #openstack-infra | 06:37 | |
mpeterson | AJaeger: will do | 06:38 |
openstackgerrit | Merged openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/597996 | 06:38 |
*** shabeeb has joined #openstack-infra | 06:39 | |
*** shabeeb has quit IRC | 06:40 | |
*** srp has quit IRC | 06:42 | |
*** dklyle has quit IRC | 06:43 | |
AJaeger | config-core, https://review.openstack.org/#/c/593264 is next python3-first in-repo migration ready for review, please | 06:43 |
*** ijw has joined #openstack-infra | 06:43 | |
*** dklyle has joined #openstack-infra | 06:44 | |
*** ginopc has joined #openstack-infra | 06:46 | |
AJaeger | ianw: could you put https://review.openstack.org/#/c/597371/ and https://review.openstack.org/#/c/597651/ on your review queue, please? | 06:46 |
*** ijw has quit IRC | 06:47 | |
openstackgerrit | Dobroslaw Zybort proposed openstack-infra/project-config master: Remove jobs settings for monasca-notification https://review.openstack.org/598061 | 06:54 |
*** alexchadin has quit IRC | 07:04 | |
*** alexchadin has joined #openstack-infra | 07:04 | |
*** alexchadin has quit IRC | 07:05 | |
*** florianf has joined #openstack-infra | 07:05 | |
*** apetrich has joined #openstack-infra | 07:09 | |
*** slaweq has joined #openstack-infra | 07:11 | |
*** jbadiapa has joined #openstack-infra | 07:14 | |
*** pcaruana has joined #openstack-infra | 07:14 | |
*** slaweq has quit IRC | 07:16 | |
*** rcernin has quit IRC | 07:20 | |
*** rtjure has quit IRC | 07:20 | |
*** slaweq has joined #openstack-infra | 07:23 | |
mpeterson | AJaeger: do you know if there are neutron flavored versions of openstack-tox-cover and openstack-tox-lower-constraints ? | 07:27 |
AJaeger | mpeterson: check openstack-infra/openstack-zuul-jobs repo, if they are not there, they don't exist | 07:29 |
*** hashar has joined #openstack-infra | 07:35 | |
mpeterson | AJaeger: nope, do you think it makes sense to create project templates for them like in http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/zuul.d/project-templates.yaml#n432 ? | 07:37 |
*** tosky has joined #openstack-infra | 07:38 | |
AJaeger | mpeterson: if there enough users, we can add them. Question is whether those are jobs (openstack-tox-lower-constraints-neutron for example) or templates? | 07:39 |
mpeterson | AJaeger: currently only jobs and no template for them exists | 07:39 |
AJaeger | right now we have no cover/lower-constraints templates... but could add those as well | 07:39 |
AJaeger | mpeterson: no time right now to dig deeper - but happy to review a proposal and then iterate on it. | 07:40 |
mpeterson | AJaeger: sure will do, I'll add you to the reviewers | 07:41 |
*** janki has joined #openstack-infra | 07:43 | |
*** jchhatbar has quit IRC | 07:44 | |
*** olivierb_ has joined #openstack-infra | 07:47 | |
*** jchhatbar has joined #openstack-infra | 07:49 | |
*** hwoarang has joined #openstack-infra | 07:49 | |
AJaeger | mpeterson: no need - if it's in ozj, I'll have my queries ;) | 07:49 |
*** janki has quit IRC | 07:49 | |
*** gema has joined #openstack-infra | 07:50 | |
*** jpena|off is now known as jpena | 07:54 | |
*** jpich has joined #openstack-infra | 07:56 | |
*** ijw has joined #openstack-infra | 08:06 | |
*** ijw has quit IRC | 08:11 | |
*** pbourke has quit IRC | 08:15 | |
*** pbourke has joined #openstack-infra | 08:17 | |
*** electrofelix has joined #openstack-infra | 08:21 | |
*** Emine has joined #openstack-infra | 08:23 | |
openstackgerrit | Tony Breeds proposed openstack-infra/project-config master: Use $NF to get the last field from the wheel build log file https://review.openstack.org/598083 | 08:24 |
tonyb | ianw: ^^ | 08:25 |
*** rabel has left #openstack-infra | 08:25 | |
*** ykarel is now known as ykarel|lunch | 08:30 | |
*** Bhujay has joined #openstack-infra | 08:37 | |
*** markvoelker has quit IRC | 08:38 | |
*** ijw has joined #openstack-infra | 08:42 | |
*** ijw has quit IRC | 08:47 | |
*** e0ne has joined #openstack-infra | 08:48 | |
*** dtantsur|afk is now known as dtantsur | 08:49 | |
*** panda is now known as panda|rover | 08:52 | |
openstackgerrit | Merged openstack-infra/project-config master: Use $NF to get the last field from the wheel build log file https://review.openstack.org/598083 | 08:53 |
*** noama has joined #openstack-infra | 09:00 | |
*** ramishra has quit IRC | 09:00 | |
*** holser_ has joined #openstack-infra | 09:02 | |
*** dingyichen has quit IRC | 09:02 | |
*** holser_ has quit IRC | 09:02 | |
*** holser_ has joined #openstack-infra | 09:03 | |
*** ijw has joined #openstack-infra | 09:06 | |
*** dpawlik has quit IRC | 09:06 | |
*** dpawlik has joined #openstack-infra | 09:07 | |
*** ijw has quit IRC | 09:11 | |
*** markvoelker has joined #openstack-infra | 09:13 | |
*** ramishra has joined #openstack-infra | 09:13 | |
*** markvoelker has quit IRC | 09:15 | |
*** sambetts|afk is now known as sambetts | 09:18 | |
*** ykarel|lunch is now known as ykarel | 09:18 | |
*** priteau has joined #openstack-infra | 09:20 | |
*** udesale has quit IRC | 09:26 | |
*** jesusaur has quit IRC | 09:27 | |
*** jesusaur has joined #openstack-infra | 09:33 | |
*** flaper87 has quit IRC | 09:34 | |
*** flaper87 has joined #openstack-infra | 09:36 | |
*** ccamacho has quit IRC | 09:39 | |
*** ccamacho has joined #openstack-infra | 09:39 | |
openstackgerrit | Michel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-lower-constraints https://review.openstack.org/598103 | 09:40 |
openstackgerrit | Michel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-cover https://review.openstack.org/598104 | 09:41 |
*** ijw has joined #openstack-infra | 09:42 | |
*** ijw has quit IRC | 09:46 | |
*** ginopc has quit IRC | 09:48 | |
*** ginopc has joined #openstack-infra | 09:52 | |
*** alexchadin has joined #openstack-infra | 09:57 | |
*** ijw has joined #openstack-infra | 10:05 | |
*** ijw has quit IRC | 10:10 | |
*** hashar is now known as hasharLunch | 10:11 | |
*** jaosorior has quit IRC | 10:25 | |
*** jaosorior has joined #openstack-infra | 10:27 | |
*** udesale has joined #openstack-infra | 10:33 | |
*** dave-mccowan has joined #openstack-infra | 10:34 | |
*** ijw has joined #openstack-infra | 10:41 | |
*** udesale has quit IRC | 10:44 | |
*** udesale has joined #openstack-infra | 10:44 | |
*** ijw has quit IRC | 10:46 | |
*** annp has quit IRC | 10:46 | |
*** nicolasbock has joined #openstack-infra | 10:47 | |
*** shardy is now known as shardy_lunch | 10:57 | |
*** priteau has quit IRC | 11:00 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack-infra/openstack-zuul-jobs master: Add openstack-python36-jobs-horizon variant https://review.openstack.org/598131 | 11:00 |
*** gfidente has joined #openstack-infra | 11:04 | |
*** ijw has joined #openstack-infra | 11:05 | |
openstackgerrit | Michel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-lower-constraints https://review.openstack.org/598103 | 11:07 |
*** priteau has joined #openstack-infra | 11:08 | |
openstackgerrit | Michel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-cover https://review.openstack.org/598104 | 11:08 |
*** ijw has quit IRC | 11:10 | |
*** ansmith has quit IRC | 11:13 | |
*** holser_ has quit IRC | 11:14 | |
*** holser_ has joined #openstack-infra | 11:15 | |
*** hwoarang has quit IRC | 11:20 | |
*** jcoufal has joined #openstack-infra | 11:20 | |
*** hwoarang has joined #openstack-infra | 11:20 | |
*** hasharLunch is now known as hashar | 11:21 | |
*** jpena is now known as jpena|lunch | 11:27 | |
*** sshnaidm is now known as sshnaidm|afk | 11:28 | |
AJaeger | mpeterson: I wonder whether we need these for still for master - with tox-siblings feature we might be able to do this differently. I'll think a bit about those - also in context of frickler's change 598131. | 11:30 |
AJaeger | mordred: what's your take? ^ | 11:30 |
*** rnm has joined #openstack-infra | 11:34 | |
mpeterson | AJaeger: AFAIU tox_install_siblings still needs the required-projects directive to be present, as it depends on Zuul checking out the project | 11:35 |
mpeterson | AJaeger: http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/tox/tasks/siblings.yaml | 11:35 |
*** aojea has joined #openstack-infra | 11:36 | |
*** rnm is now known as rmart04 | 11:36 | |
*** aojea has quit IRC | 11:36 | |
*** aojea has joined #openstack-infra | 11:37 | |
*** threestrands has quit IRC | 11:38 | |
*** longkb has quit IRC | 11:38 | |
*** rh-jelabarre has joined #openstack-infra | 11:39 | |
*** rmart04 has quit IRC | 11:40 | |
*** rnm has joined #openstack-infra | 11:40 | |
*** ijw has joined #openstack-infra | 11:41 | |
*** rnm has quit IRC | 11:42 | |
*** rnm has joined #openstack-infra | 11:42 | |
*** rnm is now known as rmart04 | 11:43 | |
*** hwoarang has quit IRC | 11:46 | |
*** ijw has quit IRC | 11:46 | |
*** hwoarang has joined #openstack-infra | 11:46 | |
*** gouthamr has quit IRC | 11:49 | |
*** dmellado has quit IRC | 11:50 | |
*** stevebaker has quit IRC | 11:51 | |
*** boden has joined #openstack-infra | 11:53 | |
*** alexchadin has quit IRC | 11:59 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/gerritlib master: Read all Gerrit events from poll interruption https://review.openstack.org/412757 | 12:01 |
*** ijw has joined #openstack-infra | 12:06 | |
*** trown|outtypewww is now known as trown | 12:06 | |
*** udesale has quit IRC | 12:08 | |
*** dpawlik has quit IRC | 12:08 | |
*** dpawlik has joined #openstack-infra | 12:09 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add config-errors notifications drawer https://review.openstack.org/597147 | 12:09 |
*** dpawlik has quit IRC | 12:10 | |
*** dpawlik has joined #openstack-infra | 12:10 | |
*** ijw has quit IRC | 12:11 | |
*** udesale has joined #openstack-infra | 12:11 | |
AJaeger | mpeterson: ah, indeed. so, the comment for those project could use updating to mention tox-siblings. | 12:14 |
*** shardy_lunch is now known as shardy | 12:18 | |
florianf | EmilienM: Yes. Listing works, but validation runs don't. I wonder if the issue can be solved by installing the openstack-tripleo-validations package in the mistral containers | 12:20 |
smcginnis | Infra folks - are things looking good for doing our final Rocky release within the next hour or two? | 12:21 |
smcginnis | Just want to make sure there aren't any known issues going on before we hit the go button. | 12:21 |
AJaeger | smcginnis: nobody around - and no problems know. Did you coordinate with docs team for docs.o.o page? | 12:22 |
smcginnis | AJaeger: I did not. That is part of our process doc though. | 12:23 |
AJaeger | smcginnis: I just pinged pkovar on #openstack-doc - I haven't seen anything yet and can't help ;( | 12:23 |
smcginnis | AJaeger: Thanks. | 12:24 |
florianf | EmilienM: oops, I replied in the wrong channel ;) sorry... | 12:28 |
*** alexchadin has joined #openstack-infra | 12:28 | |
*** oanson has joined #openstack-infra | 12:30 | |
*** jpena|lunch is now known as jpena|off | 12:31 | |
*** jpena|off is now known as jpena | 12:32 | |
*** slaweq has quit IRC | 12:34 | |
*** slaweq has joined #openstack-infra | 12:34 | |
*** rlandy has joined #openstack-infra | 12:37 | |
*** alexchadin has quit IRC | 12:38 | |
*** udesale has quit IRC | 12:40 | |
*** signed8bit has joined #openstack-infra | 12:45 | |
*** vivsoni has quit IRC | 12:49 | |
*** vivsoni_ has joined #openstack-infra | 12:49 | |
*** udesale has joined #openstack-infra | 12:53 | |
*** kgiusti has joined #openstack-infra | 12:54 | |
*** mriedem has joined #openstack-infra | 12:54 | |
*** rosmaita has joined #openstack-infra | 12:55 | |
*** ansmith has joined #openstack-infra | 12:57 | |
*** tpsilva has joined #openstack-infra | 13:01 | |
*** jbadiapa has quit IRC | 13:03 | |
*** bobh has joined #openstack-infra | 13:04 | |
*** sthussey has joined #openstack-infra | 13:05 | |
openstackgerrit | Merged openstack-infra/pypi-mirror master: Retire pypi-mirror https://review.openstack.org/597401 | 13:06 |
openstackgerrit | Thierry Carrez proposed openstack/ptgbot master: Identify with SASL https://review.openstack.org/598155 | 13:09 |
openstackgerrit | Michel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-cover https://review.openstack.org/598104 | 13:10 |
openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: Update static nodes in Zookeeper on config change https://review.openstack.org/598156 | 13:11 |
openstackgerrit | Merged openstack-infra/project-config master: remove job settings for mistral repositories https://review.openstack.org/593264 | 13:11 |
openstackgerrit | Michel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-lower-constraints https://review.openstack.org/598103 | 13:11 |
mpeterson | AJaeger: did it, as per your comment ^^ | 13:12 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove obsolete legacy-js-generator-openstack-nodejs6-npm-docs https://review.openstack.org/597371 | 13:13 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove puppet-featuretracker https://review.openstack.org/597651 | 13:13 |
*** alexchadin has joined #openstack-infra | 13:13 | |
openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: Update static nodes in Zookeeper on config change https://review.openstack.org/598156 | 13:18 |
*** jbadiapa has joined #openstack-infra | 13:22 | |
*** yamamoto has joined #openstack-infra | 13:37 | |
*** jamesdenton has joined #openstack-infra | 13:38 | |
*** aojea has quit IRC | 13:40 | |
*** eharney has joined #openstack-infra | 13:40 | |
*** shabeeb has joined #openstack-infra | 13:42 | |
*** shabeeb has quit IRC | 13:43 | |
*** toabctl has joined #openstack-infra | 13:44 | |
*** Bhujay has quit IRC | 13:47 | |
*** shabeeb has joined #openstack-infra | 13:48 | |
*** udesale has quit IRC | 13:49 | |
*** alexchadin has quit IRC | 13:50 | |
*** udesale has joined #openstack-infra | 13:50 | |
*** psachin has quit IRC | 13:51 | |
*** jistr is now known as jistr|call | 13:51 | |
*** aojea has joined #openstack-infra | 13:55 | |
*** yamamoto has quit IRC | 13:56 | |
fungi | and so it begins | 13:59 |
fungi | follow rocky release madness unfolding at http://zuul.openstack.org/ | 13:59 |
*** ykarel is now known as ykarel|away | 14:00 | |
fungi | the tag-releases job in the release-post pipeline has just gotten an allocation | 14:00 |
fungi | should see the release pipeline start filling up momentarily | 14:01 |
fungi | or it had an allocation, then got dequeued and requeued | 14:02 |
*** ykarel|away has quit IRC | 14:06 | |
*** sshnaidm|afk has quit IRC | 14:10 | |
AJaeger | fungi, want to +2A next python3-first changes: https://review.openstack.org/#/c/597517/ and https://review.openstack.org/#/c/592849/ , please? | 14:10 |
*** sshnaidm|afk has joined #openstack-infra | 14:19 | |
AJaeger | mpeterson: thanks! | 14:19 |
*** sshnaidm|afk is now known as sshnaidm | 14:20 | |
*** alexchadin has joined #openstack-infra | 14:20 | |
AJaeger | dhellmann: what do you think of https://review.openstack.org/598103 ? Is there a special reason this wasn't done earlier? | 14:20 |
dhellmann | I wonder if we're only running that job as a check and not in the gate? | 14:21 |
*** jistr|call is now known as jistr | 14:21 | |
dhellmann | AJaeger : seems OK to do | 14:22 |
AJaeger | dhellmann: looking at http://codesearch.openstack.org/?q=openstack-tox-lower-constraints&i=nope&files=&repos= , it's in both check and gate | 14:23 |
AJaeger | if you want, your script could convert to use the template... That would be nice - not essential ;) | 14:24 |
dhellmann | the migration script? | 14:24 |
dhellmann | hmm | 14:24 |
dhellmann | yeah, I don't know about adding that level of complexity, esp. if we have variants | 14:24 |
AJaeger | dhellmann: understood | 14:25 |
AJaeger | mpeterson: https://review.openstack.org/598184 uses your new template... | 14:31 |
mpeterson | AJaeger: nice! Btw, the template is called openstack-lower-constraints-jobs | 14:32 |
*** srp has joined #openstack-infra | 14:34 | |
*** shabeeb has quit IRC | 14:35 | |
AJaeger | mpeterson: thanks... | 14:35 |
*** ykarel has joined #openstack-infra | 14:35 | |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk https://review.openstack.org/593029 | 14:42 |
*** alexchadin has quit IRC | 14:42 | |
*** jchhatbar has quit IRC | 14:43 | |
mpeterson | AJaeger, dhellmann: should we send automated patches that make use of the new templates or just a mail announcement when they are merged? | 14:43 |
*** vivsoni_ has quit IRC | 14:43 | |
*** alexchadin has joined #openstack-infra | 14:44 | |
AJaeger | mpeterson: if you want to script it, go for it - but then I suggest to also send an announcement out ;) | 14:45 |
AJaeger | just a different announcement... | 14:46 |
AJaeger | mpeterson: announcement in any case | 14:46 |
*** r-daneel has joined #openstack-infra | 14:47 | |
mpeterson | AJaeger: makes sense. okey, I'm wrapping up for this week. Have a good weekend. | 14:50 |
*** shabeeb has joined #openstack-infra | 14:51 | |
*** vivsoni has joined #openstack-infra | 14:51 | |
AJaeger | mpeterson: enjoy yours as well! | 14:52 |
*** srp has quit IRC | 14:52 | |
*** ykarel has quit IRC | 14:54 | |
corvus | fungi, clarkb: ever seen this? http://logs.openstack.org/17/597617/8/check/system-config-run-eavesdrop/c086730/job-output.txt.gz#_2018-08-29_23_14_31_566882 | 14:56 |
corvus | based on the test logs a few lines above, it looks plausible that test_base.test_iptables() ran at the same time as test_eavesdrop(), but in a different process | 14:58 |
fungi | huh, like ansible task parallelization at work? | 14:59 |
corvus | fungi: well, test.py in this case | 14:59 |
corvus | we have test.py set up to use 5 parallel workers | 15:00 |
fungi | ohhh | 15:00 |
*** sdoran has joined #openstack-infra | 15:02 | |
*** ykarel has joined #openstack-infra | 15:03 | |
*** udesale has quit IRC | 15:04 | |
*** zul has quit IRC | 15:05 | |
*** ginopc has quit IRC | 15:08 | |
*** zul has joined #openstack-infra | 15:08 | |
*** ginopc has joined #openstack-infra | 15:08 | |
*** ginopc has quit IRC | 15:11 | |
*** ginopc has joined #openstack-infra | 15:12 | |
*** pcaruana has quit IRC | 15:12 | |
*** r-daneel has quit IRC | 15:13 | |
*** r-daneel_ has joined #openstack-infra | 15:13 | |
*** r-daneel_ is now known as r-daneel | 15:16 | |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk https://review.openstack.org/593029 | 15:16 |
fungi | only one release seems to have failed its job (networking-bgpvpn 9.0.0) so i've reenqueued it | 15:17 |
*** dpawlik has quit IRC | 15:18 | |
*** olivier__ has joined #openstack-infra | 15:21 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add system-config-run-eavesdrop https://review.openstack.org/597617 | 15:21 |
*** olivierb_ has quit IRC | 15:22 | |
*** eglute has joined #openstack-infra | 15:25 | |
*** ykarel has quit IRC | 15:25 | |
*** e0ne has quit IRC | 15:27 | |
*** ykarel has joined #openstack-infra | 15:29 | |
*** Emine has quit IRC | 15:29 | |
*** bobh has quit IRC | 15:35 | |
AJaeger | config-core, could you help reviewing next python3-first changes: https://review.openstack.org/#/c/597517/ and https://review.openstack.org/#/c/592849/ , please? | 15:35 |
*** jtomasek has quit IRC | 15:38 | |
*** ykarel_ has joined #openstack-infra | 15:38 | |
*** ykarel has quit IRC | 15:39 | |
*** gyee has joined #openstack-infra | 15:42 | |
*** anteaya has joined #openstack-infra | 15:43 | |
clarkb | expect a slow day from me, I've learned UPSes are really loud at 1am during power outages. | 15:44 |
*** olivier__ has quit IRC | 15:46 | |
*** olivierb_ has joined #openstack-infra | 15:48 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: switch release job for networking-bgpvpn https://review.openstack.org/598213 | 15:52 |
fungi | i get reminded of that remarkably frequently here | 15:52 |
fungi | and i have some dozen very loud upses scattered around the house due to the instability of our power grid here on the island | 15:53 |
fungi | need to just give in and budget for whole-house ups and generator combo | 15:53 |
fungi | they're pretty popular here so easy to find a contractor familiar with installing them | 15:53 |
corvus | then you can sell excess capacity to your neighbors.... | 15:53 |
fungi | except that my neighbors don't live in their houses, so they're vacant most of storm season | 15:54 |
clarkb | fungi: I honeslty thought it was the fire alarms when they first woke me | 15:55 |
clarkb | was not a fun way to wake up | 15:55 |
*** ykarel_ is now known as ykarel | 15:55 | |
*** trown is now known as trown|lunch | 15:55 | |
dhellmann | config-core: https://review.openstack.org/598213 should unblock the last release deliverable for the rocky release today | 15:56 |
*** ginopc has quit IRC | 15:59 | |
pabelanger | clarkb: https://review.openstack.org/593029 is a dependancy for clouds.yaml, however think I am getting pulled downstream for something today. If you want to write a ptach atop of it, I'm okay with that. Otherwise, I can look to push on it later this evening | 16:00 |
*** hamzy has quit IRC | 16:00 | |
*** ykarel has quit IRC | 16:00 | |
clarkb | pabelanger: I'm not quite sure I understand why that is a dep (we can have puppet manage the limestone cert while we get up clouds.yaml), but I'll review it more closely in a bit to see if it starts to make more sense to me | 16:01 |
clarkb | may just be nice to have to avoid splitting the management of that data? | 16:01 |
dansmith | can anyone tell me if our xenial worker nodes have cloud-archive:pike already configured?\ | 16:03 |
dansmith | devstack attempts to do it, | 16:03 |
clarkb | dhellmann: approved | 16:04 |
dansmith | but I think it might not be updating qemu-utils from it after configured when I run it locally | 16:04 |
dansmith | but the gate jobs all work, so.. | 16:04 |
clarkb | dansmith: they should not have UCA preconfigured | 16:04 |
dansmith | hmm | 16:04 |
dansmith | I get 100% fail when I run devstack from queens locally, because it has qemu-utils from base instead of cloud-archive | 16:04 |
dansmith | maybe something I'm installing beforehand is pulling it in and it's not getting updated | 16:05 |
pabelanger | clarkb: we need access to limestone from bridge.o.o, if we want to create a mirror | 16:05 |
* dansmith keeps digging | 16:05 | |
pabelanger | but yah, it also create /etc/openstack folders | 16:05 |
*** srp has joined #openstack-infra | 16:05 | |
corvus | fungi, clarkb: testinfra uses hacking | 16:06 |
fungi | um, wow | 16:06 |
fungi | as in openstack hacking? | 16:06 |
corvus | ayup | 16:06 |
fungi | funny to see how tools like that have propagated (in sometimes questionable ways) | 16:07 |
clarkb | woah | 16:08 |
clarkb | pabelanger: aha that is the missing link bridge doesnt puppet | 16:08 |
*** zxiiro-away is now known as zxiiro | 16:09 | |
*** dpawlik has joined #openstack-infra | 16:09 | |
*** srp has quit IRC | 16:09 | |
*** shabeeb has quit IRC | 16:09 | |
*** jpich has quit IRC | 16:10 | |
*** shabeeb has joined #openstack-infra | 16:11 | |
*** shachar has joined #openstack-infra | 16:12 | |
openstackgerrit | Sorin Sbarnea proposed openstack-infra/zuul master: Display correct cursor when hovering over patchset header https://review.openstack.org/598222 | 16:12 |
*** alexchadin has quit IRC | 16:13 | |
*** dpawlik has quit IRC | 16:13 | |
*** ykarel has joined #openstack-infra | 16:14 | |
*** snapiri has quit IRC | 16:14 | |
openstackgerrit | Merged openstack-infra/project-config master: switch release job for networking-bgpvpn https://review.openstack.org/598213 | 16:17 |
*** ykarel has quit IRC | 16:17 | |
*** ccamacho has quit IRC | 16:19 | |
*** rlandy is now known as rlandy|afk | 16:19 | |
*** anteaya has quit IRC | 16:23 | |
*** ykarel has joined #openstack-infra | 16:25 | |
*** anteaya has joined #openstack-infra | 16:25 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add system-config-run-eavesdrop https://review.openstack.org/597617 | 16:27 |
*** signed8bit is now known as signed8bit_Zzz | 16:27 | |
*** eharney has quit IRC | 16:29 | |
*** ykarel_ has joined #openstack-infra | 16:29 | |
*** noama has quit IRC | 16:30 | |
*** signed8bit_Zzz is now known as signed8bit | 16:31 | |
*** jamesmcarthur has joined #openstack-infra | 16:31 | |
*** ykarel has quit IRC | 16:31 | |
*** sambetts is now known as sambetts|afk | 16:31 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: run-base: collect ansible configuration https://review.openstack.org/598226 | 16:32 |
corvus | clarkb: we ready for https://review.openstack.org/596913 ? | 16:33 |
clarkb | corvus: yes I think so, it worked out great manually applying it to the sjc1 mirror | 16:34 |
*** jbadiapa has quit IRC | 16:35 | |
*** bnemec has quit IRC | 16:35 | |
corvus | so i believe i've got all the issues worked out for the eavesdrop job, except the last run of the run-base job (not eavesdrop) for some reason put 'bionic' in the puppet group. i don't understand why. i've (hopefully) added a group debug statement to the playbook. but i'm entertaining the possibility there's something unreliable happening in the constructed inventory plugin. | 16:36 |
openstackgerrit | Thierry Carrez proposed openstack/ptgbot master: Identify with SASL https://review.openstack.org/598155 | 16:36 |
clarkb | corvus: huh, do we log the written inventory? | 16:40 |
*** agopi has quit IRC | 16:40 | |
corvus | clarkb: i add that in https://review.openstack.org/598226 | 16:40 |
*** rmart04 has quit IRC | 16:41 | |
clarkb | corvus: cool, I'll hvae to review in a bit. Going to double check on power outage fallout now. zfs scrub here I come | 16:42 |
*** agopi has joined #openstack-infra | 16:42 | |
*** r-daneel_ has joined #openstack-infra | 16:47 | |
*** r-daneel has quit IRC | 16:47 | |
*** r-daneel_ is now known as r-daneel | 16:47 | |
*** gouthamr has joined #openstack-infra | 16:49 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Provide per-project ssh key to executor https://review.openstack.org/597727 | 16:51 |
*** ykarel_ has quit IRC | 16:53 | |
corvus | oh for crying out loud. iptables in trusty doesn't support "-w" | 16:53 |
*** ykarel has joined #openstack-infra | 16:55 | |
*** ykarel has quit IRC | 16:55 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove release job variants for networking-bgpvpn https://review.openstack.org/598232 | 16:56 |
*** shardy has quit IRC | 16:56 | |
openstackgerrit | Merged openstack-infra/system-config master: Install python2.7 on xenial nodes https://review.openstack.org/596913 | 16:57 |
*** ccamacho has joined #openstack-infra | 16:58 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add system-config-run-eavesdrop https://review.openstack.org/597617 | 17:02 |
corvus | i'm just going to turn of parallel execution for now | 17:02 |
*** anteaya has quit IRC | 17:03 | |
*** dtantsur is now known as dtantsur|afk | 17:04 | |
fungi | looks like release activity has settled down, so i'm going to go find lunch | 17:06 |
*** diablo_rojo has joined #openstack-infra | 17:06 | |
*** jamesmcarthur has quit IRC | 17:09 | |
*** jamesmcarthur has joined #openstack-infra | 17:10 | |
*** mriedem is now known as mriedem_afk | 17:11 | |
*** aojea has quit IRC | 17:11 | |
*** agopi has quit IRC | 17:11 | |
*** trown|lunch is now known as trown | 17:13 | |
*** abelur has joined #openstack-infra | 17:16 | |
*** bnemec has joined #openstack-infra | 17:17 | |
*** shabeeb has quit IRC | 17:21 | |
*** ccamacho has quit IRC | 17:24 | |
*** jamesmcarthur has quit IRC | 17:24 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: run-base: collect ansible configuration https://review.openstack.org/598226 | 17:25 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add system-config-run-eavesdrop https://review.openstack.org/597617 | 17:25 |
corvus | okay i think that's going to do it | 17:25 |
*** eharney has joined #openstack-infra | 17:25 | |
corvus | (everything passed the last round; that just drops an rsync trailing slash) | 17:26 |
*** jpena is now known as jpena|away | 17:27 | |
*** Emine has joined #openstack-infra | 17:28 | |
*** hamzy has joined #openstack-infra | 17:28 | |
*** anteaya has joined #openstack-infra | 17:29 | |
*** jamesmcarthur has joined #openstack-infra | 17:31 | |
clarkb | and I'm back to computer now. Will review | 17:32 |
EmilienM | hi infra, we merged https://review.openstack.org/#/c/592695/ which was supposed to create mirror for cbs.centos but I don't see the mirror alive yet, did I miss something? | 17:33 |
EmilienM | e.g. http://mirror.gra1.ovh.openstack.org/centos/ | 17:33 |
pabelanger | EmilienM: that isn't a mirror, but reverse proxy cache | 17:34 |
clarkb | correct that won't show up in the index | 17:35 |
*** holser_ has quit IRC | 17:35 | |
pabelanger | http://mirror.gra1.ovh.openstack.org:8080/cbs.centos/ | 17:35 |
pabelanger | is right url | 17:35 |
clarkb | it should be up and running at this point, you can test it with your brwoser at http://mirror.gra1.ovh.openstack.org:8080/cbs.centos/ | 17:35 |
clarkb | pabelanger: jinx | 17:35 |
EmilienM | oh gotcha :) | 17:35 |
EmilienM | thank you clarkb / pabelanger | 17:35 |
*** shabeeb has joined #openstack-infra | 17:43 | |
*** jamesdenton has quit IRC | 17:44 | |
*** dmellado has joined #openstack-infra | 17:44 | |
openstackgerrit | Emilien Macchi proposed openstack-infra/openstack-zuul-jobs master: mirror-info: add variables for CBS proxy https://review.openstack.org/598250 | 17:45 |
*** anteaya has quit IRC | 17:45 | |
openstackgerrit | Emilien Macchi proposed openstack-infra/openstack-zuul-jobs master: mirror-info: add variable for CBS proxy https://review.openstack.org/598250 | 17:45 |
clarkb | corvus: left a suggestion on https://review.openstack.org/#/c/597617/12 which should help with further debugging | 17:46 |
*** eharney has quit IRC | 17:46 | |
*** shabeeb has quit IRC | 17:47 | |
corvus | clarkb: i was just implementing that | 17:47 |
*** stevebaker has joined #openstack-infra | 17:48 | |
*** agopi has joined #openstack-infra | 17:48 | |
*** shabeeb has joined #openstack-infra | 17:49 | |
*** electrofelix has quit IRC | 17:50 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Refactor run-base jobs https://review.openstack.org/598252 | 17:50 |
corvus | clarkb: as a followup ^ | 17:50 |
corvus | happy to do another ps though if you want | 17:50 |
*** jamesmcarthur has quit IRC | 17:51 | |
AJaeger | config-core, could you help reviewing next python3-first changes: https://review.openstack.org/#/c/597517/ and https://review.openstack.org/#/c/592849/ and https://review.openstack.org/598061, please? | 17:51 |
clarkb | corvus: followup change works, reviewing thatnow then will apply votes to stack based on its entire status | 17:51 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove job settings for neutron repositories https://review.openstack.org/597865 | 17:55 |
*** r-daneel has quit IRC | 17:55 | |
clarkb | corvus: stack lgtm another obversation for a cleanup on https://review.openstack.org/#/c/598226/2 that can go in a followup too I Think | 17:55 |
*** gyee has quit IRC | 17:56 | |
*** eharney has joined #openstack-infra | 17:56 | |
*** zxiiro is now known as zxiiro-away | 17:57 | |
openstackgerrit | Merged openstack-infra/system-config master: Generate junit.xml file for testinfra https://review.openstack.org/597185 | 17:58 |
*** jamesdenton has joined #openstack-infra | 17:59 | |
*** r-daneel has joined #openstack-infra | 17:59 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove job settings for neutron repositories https://review.openstack.org/597865 | 17:59 |
*** hashar is now known as hasharDinner | 18:00 | |
Shrews | corvus: hrm, should we be more explicit about collecting the ansible config? pulling back __pycache__ dirs in your change http://logs.openstack.org/26/598226/2/check/system-config-run-base/1ff758a/ansible/inventory_plugins/ | 18:02 |
clarkb | Shrews: ya that was my comment too | 18:02 |
clarkb | I think for now its a single file (or two if you count the .py) that we probably don't want | 18:03 |
Shrews | oh, oops | 18:03 |
clarkb | so not urgent to change and can do i na followup | 18:03 |
corvus | so maybe "--exclude=__pycache__" ? | 18:05 |
clarkb | ya | 18:05 |
openstackgerrit | Merged openstack-infra/project-config master: remove job settings for rally repositories https://review.openstack.org/597517 | 18:05 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: run-base: exclude __pycache__ from logs https://review.openstack.org/598256 | 18:06 |
EmilienM | simple patch to review (for CBS proxy): https://review.openstack.org/598250 - thanks! | 18:07 |
openstackgerrit | Merged openstack-infra/project-config master: remove job settings for cyborg repositories https://review.openstack.org/592849 | 18:08 |
*** r-daneel_ has joined #openstack-infra | 18:08 | |
openstackgerrit | Merged openstack-infra/project-config master: Remove jobs settings for monasca-notification https://review.openstack.org/598061 | 18:09 |
*** r-daneel has quit IRC | 18:09 | |
*** r-daneel_ is now known as r-daneel | 18:09 | |
*** rosmaita has quit IRC | 18:24 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove job settings for neutron repositories https://review.openstack.org/597865 | 18:24 |
*** bobh has joined #openstack-infra | 18:27 | |
openstackgerrit | Merged openstack-infra/system-config master: run-base: collect ansible configuration https://review.openstack.org/598226 | 18:29 |
*** jamesmcarthur has joined #openstack-infra | 18:30 | |
clarkb | woo scrub came back clean, I can go worry about other things now | 18:30 |
*** signed8bit is now known as signed8bit_Zzz | 18:31 | |
*** shabeeb has quit IRC | 18:33 | |
*** Emine has quit IRC | 18:35 | |
openstackgerrit | Merged openstack-infra/system-config master: Add system-config-run-eavesdrop https://review.openstack.org/597617 | 18:38 |
*** signed8bit_Zzz is now known as signed8bit | 18:38 | |
*** jamesmcarthur has quit IRC | 18:40 | |
clarkb | pabelanger: https://review.openstack.org/#/c/593029/20/playbooks/configure-openstack.yaml should that file be removed? | 18:41 |
pabelanger | clarkb: yah, we can delete that | 18:42 |
pabelanger | 1 sec | 18:42 |
clarkb | pabelanger: any idea how we want ot handle clouds.yaml and all-clouds.yaml within that new role? I sort of see where the role is headed writing the clouds.yaml file which should be straightforward, but not sure how to handle all-clouds.yaml | 18:43 |
*** jamesmcarthur has joined #openstack-infra | 18:43 | |
clarkb | maybe just make that a non default option that we set on bridge and it writes the extra file | 18:44 |
pabelanger | clarkb: yah, was thinking we'd call the role twice, just with difference clouds.yaml values and locations | 18:44 |
clarkb | makes esnse, also remembering that nodepool has a different clouds.yaml (its the other half of all-clouds.yaml from bridge's clouds.yaml) | 18:45 |
clarkb | so we need to specify thattemplate either way I think | 18:45 |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk https://review.openstack.org/593029 | 18:46 |
clarkb | and we should be able to just take the existing erb file and make it a jinja2 template and the existing lookups to hiera should work | 18:47 |
*** Emine has joined #openstack-infra | 18:47 | |
clarkb | ok this is starting to make sense. Hopeflly have time to take a stab at it later today | 18:47 |
*** jamesmcarthur has quit IRC | 18:48 | |
pabelanger | clarkb: yah, other issue, I am not sure how we are going to test this with testinfra, because I think ansible is going to fail in the gate, unless we add in a all the same variables from hieradata | 18:48 |
openstackgerrit | Merged openstack-infra/system-config master: Refactor run-base jobs https://review.openstack.org/598252 | 18:48 |
pabelanger | ansible will error if it tries to reference a non existing variable, unless we use default(foo) | 18:48 |
*** imacdonn has joined #openstack-infra | 18:49 | |
pabelanger | or moved the whole yaml structure for clouds.yaml into a dict in hiera | 18:49 |
clarkb | we'd have to serialize that back to yaml though right? does ansible make that easy? that might be a good followup | 18:50 |
pabelanger | yah, there are yaml filters we could use | 18:50 |
clarkb | pabelanger: also did you see my change for cloud launcher? https://review.openstack.org/#/c/597728/ | 18:51 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove job settings for Release Management repositories https://review.openstack.org/598314 | 18:51 |
*** toabctl has quit IRC | 18:51 | |
clarkb | one thing I like about having the files in their resulting format is its easier to use them in othre contexts | 18:51 |
clarkb | like showing someone an example working clouds.yaml for $cloud | 18:51 |
clarkb | similar with apache config files | 18:51 |
clarkb | you lose a lot of that readability and reusability when you store them in $configmgmt_tool intermediate format | 18:51 |
clarkb | not to mention the actual act of going from one management system to another as we are doing | 18:52 |
*** gfidente has quit IRC | 18:54 | |
pabelanger | yah, agree a little. I think we could actually write a test inventory, beside our production inventory for ansible. Which production data on bridge.o.o, and test in-tree. But means, we duplicate all the variables name, which maybe is good, but a little more work | 18:55 |
pabelanger | then we can template things out properly in test environment, same variables, just different data | 18:56 |
openstackgerrit | Merged openstack-infra/system-config master: run-base: exclude __pycache__ from logs https://review.openstack.org/598256 | 18:57 |
clarkb | if you specify a relative template source in a role and that file does not exist at the role level with it look relative to the playbook? | 18:58 |
*** jamesmcarthur has joined #openstack-infra | 18:59 | |
imacdonn | hi guys .. anything going on that would cause this (neutron-grenade for a backport to glance stable/queens): | 19:00 |
imacdonn | 2018-08-30 18:41:27.320605 | primary | File "/tmp/ansible/local/lib/python2.7/site-packages/ara/config/compat.py", line 21, in <module> | 19:00 |
imacdonn | 2018-08-30 18:41:27.320663 | primary | from ansible.config.manager import find_ini_config_file | 19:00 |
imacdonn | 2018-08-30 18:41:27.320704 | primary | ImportError: No module named manager | 19:00 |
clarkb | ara just made a release | 19:00 |
pabelanger | clarkb: yah, relative to playbook IIRC | 19:00 |
clarkb | dmsimard: ^ see imacdonn's comment | 19:01 |
clarkb | pabelanger: thanks | 19:01 |
dmsimard | imacdonn: have a link to the job ? | 19:01 |
dmsimard | clarkb: thanks, I'll look | 19:01 |
imacdonn | dmsimard: http://logs.openstack.org/00/598300/1/check/neutron-grenade/e303134/ | 19:02 |
*** jamesmcarthur has quit IRC | 19:02 | |
dmsimard | that looks like devstack-gate | 19:02 |
dmsimard | http://logs.openstack.org/00/598300/1/check/neutron-grenade/e303134/ara-report/result/d6b7948c-e626-4d7a-9503-5e1d90090171/ | 19:02 |
*** jamesmcarthur has joined #openstack-infra | 19:02 | |
clarkb | dmsimard: it is devstack-gate running ara I think? | 19:03 |
dmsimard | clarkb: the new release drops a compat layer that existed for ansible 2.3 (which has been EOL for a while) | 19:04 |
*** jamesmcarthur has quit IRC | 19:04 | |
dmsimard | but ara also has ansible>=2.4 in it's req.txt | 19:04 |
*** ijw has joined #openstack-infra | 19:04 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: switch all official python projects to python3 publishing job https://review.openstack.org/598323 | 19:04 |
pabelanger | dmsimard: clarkb: https://review.openstack.org/591527/ | 19:04 |
pabelanger | devstack-gate is still using 2.3 | 19:04 |
openstackgerrit | David Moreau Simard proposed openstack-infra/devstack-gate master: Bump ansible to latest 2.5 version https://review.openstack.org/591527 | 19:05 |
dmsimard | pabelanger: yup, it was pinned to 2.3, thanks | 19:05 |
*** rlandy|afk is now known as rlandy | 19:05 | |
dmsimard | I've been out of the loop for a while, devstack-gate is not dead yet? :( | 19:05 |
pabelanger | no, a lot of legacy jobs still using it | 19:06 |
clarkb | we can probably bump that ansible version | 19:06 |
corvus | dmsimard: what was the new patchset? | 19:06 |
clarkb | its pinned because ansible releases are not backward compat | 19:06 |
dmsimard | clarkb: pabelanger's patchset already does that | 19:06 |
clarkb | ah | 19:06 |
dmsimard | corvus: I don't understand the question | 19:06 |
corvus | dmsimard: you just pushed a new patchset of 527. why? | 19:06 |
dmsimard | corvus: it's a rebase to ensure that it really does fix the problem | 19:07 |
*** toabctl has joined #openstack-infra | 19:07 | |
*** SteffanW has joined #openstack-infra | 19:08 | |
pabelanger | I have to run and do errands now, will loop back on clouds.yaml later this evening | 19:09 |
dmsimard | Sorry about that folks, I tested TripleO, OSA, Kolla-Ansible and Zuul.. I'll add devstack-gate, it's a gap :( | 19:09 |
corvus | dmsimard: for what it's worth, since it previously had passed check, we could have enqueued it in gate immediately and perhaps fixed the problem sooner. if it didn't fix the problem, it would be rejected in gate. | 19:09 |
dmsimard | corvus: true, I erred on the side of safety | 19:11 |
*** mriedem has joined #openstack-infra | 19:11 | |
*** mriedem_afk has quit IRC | 19:12 | |
corvus | dmsimard: the gate is safety :) if the patch failed (including with the current error, or something new in the intervening time) the gate will reject it. it can take a while to get used to that level of freedom, but it's worth remembering it and thinking strategically about changes with long-running jobs. | 19:13 |
corvus | clarkb: i'm +2 on 591527 if you want to +3 | 19:13 |
dmsimard | corvus: totally agree | 19:13 |
*** ansmith has quit IRC | 19:15 | |
corvus | openstack does have the clean-check requirement to keep bogus changes from clogging up the gate queue. so the flip side of what i said is true too. :) but this change seems less likely to have bit-rotted in the intervening time, plus things are broken anyway, so there's not much of a downside. | 19:15 |
*** r-daneel has quit IRC | 19:17 | |
openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: WIP: Manage clouds.yaml files in ansible https://review.openstack.org/598329 | 19:18 |
clarkb | pabelanger: ^ is more of an outline than anything else, if you hvae a momment can you check that that looks sane? I don't want to remove the old stuff and convert the tmeplate files over until we have general happyness with that chnage | 19:19 |
*** olivier__ has joined #openstack-infra | 19:19 | |
clarkb | corvus: done | 19:20 |
*** olivierb_ has quit IRC | 19:20 | |
*** SteffanW has quit IRC | 19:21 | |
imacdonn | thanks for the quick response! Will this take effect when merged to devstack-gate master, or does it have to get to some stable branch? | 19:26 |
clarkb | imacdonn: devtack-gate is "branchless" the master branch applies to all runs | 19:26 |
clarkb | so it will take effect once merged | 19:26 |
imacdonn | cool, thanks | 19:26 |
*** jamesmcarthur has joined #openstack-infra | 19:29 | |
fungi | okay, i'm back catching up and seeing how many people i've managed to annoy | 19:30 |
*** dpawlik has joined #openstack-infra | 19:34 | |
*** e0ne has joined #openstack-infra | 19:34 | |
*** cfriesen has joined #openstack-infra | 19:34 | |
cfriesen | has anyone else run into issues with etherpads being auto-translated to other languages by browsers set to auto-translate? is there any way to prevent it? | 19:35 |
fungi | i've definitely not seen that | 19:36 |
fungi | then again i've never seen a browser that had a feature to auto-translate content without user intervention asking for it | 19:37 |
cfriesen | I think the idea is that the browser is supposed to be translating for their user, but is somehow writing the translated content back to the etherpad. | 19:38 |
fungi | also, that reminds me, i need to restart the etherpad-lite service in production now that the change merged to upgrade it to the latest release | 19:38 |
*** jpena|away is now known as jpena|off | 19:38 | |
*** cfriesen has left #openstack-infra | 19:40 | |
*** ijw has quit IRC | 19:42 | |
smcginnis | Any idea if the etherpad update gets rid of the undocumented feature of changing the whole line's color when you make a single character change? | 19:44 |
clarkb | smcginnis: yes that is part of the motivation | 19:45 |
fungi | you can test it on etherpad-dev.o.o already, but it seems to be fixed | 19:45 |
fungi | as soon as the restart happens on etherpad.o.o we'll see it fixed there too, i believe | 19:45 |
fungi | i wanted to make sure we had it in production a week or more in advance of the ptg | 19:46 |
smcginnis | Good plan. | 19:49 |
*** olivier__ has quit IRC | 19:54 | |
*** hamzy has quit IRC | 19:54 | |
*** r-daneel has joined #openstack-infra | 19:55 | |
*** hamzy has joined #openstack-infra | 19:55 | |
*** hasharDinner is now known as hashar | 19:56 | |
fungi | i'll try to make sure to restart it later in my evening once our usual community activity levels die down | 19:56 |
*** bobh has quit IRC | 20:06 | |
*** bobh has joined #openstack-infra | 20:08 | |
*** otherwiseguy has joined #openstack-infra | 20:10 | |
corvus | clarkb: can you help clean up my oops? https://review.openstack.org/597242 | 20:13 |
notmyname | are we supposed to backport zuul job imports to stable branches? on the one hand, that makes sense if it means we can sooner reduce the overhead in the suul_jobs repo or project_config. but it also seems odd to have programatically-generated patches proposed to stable branches that are only changing test running definitions. please advise | 20:15 |
clarkb | corvus: done | 20:17 |
corvus | notmyname: if you want to remove them from project-config, yes. consider it a one-time cost. generally in the future you'll only need to modify stable branch jobs if, well, you want to modify the stable branch jobs. :) | 20:17 |
clarkb | corvus: can you look at https://review.openstack.org/#/c/598329/ and its parent as part of the clouds.yaml fixing? that change is the outline I'm working with but don't want ot go too far down that path if its not the right approach | 20:18 |
*** kgiusti has left #openstack-infra | 20:18 | |
notmyname | corvus: yeah, it makes sense. just seems a bit outside of the norm, so I wanted make sure this is the OpenStack Way (tm) | 20:18 |
corvus | pabelanger: i don't think https://review.openstack.org/597241 is saving a lot of time, but i like it anyway. <-- clarkb | 20:18 |
corvus | clarkb: will do | 20:18 |
corvus | notmyname: my answer is kind of general; if you have specific questions about the py3 work and any auto-generated patches (or lack thereof), dhellmann and AJaeger may have further details. | 20:19 |
dhellmann | notmyname : I tried to explain why in https://governance.openstack.org/tc/goals/stein/python3-first.html#moving-settings-from-project-config | 20:20 |
clarkb | corvus: pabelanger the package module docs don't say that is valid, do we have test coverage that it is doing the right thing? | 20:20 |
corvus | clarkb: theoretically run-base | 20:22 |
corvus | clarkb: though if you think there's a problem, it's probably worth working through whether any failures there would actually be detected... | 20:23 |
clarkb | mostly just trying to convince myself that is valid anisble. The source code for package.py just has the documentation strings in it so trying to track that down | 20:23 |
corvus | clarkb: the limestone ca patch makes me think we ought to maybe rework the nodepool nodes to install things in standard location and tweak group membership. but that's for a rainy day. maybe we can rethink things when we containerize. | 20:24 |
*** eharney has quit IRC | 20:24 | |
clarkb | ya it would likely help other users debugging nodepool to not have to su to that user | 20:24 |
*** mugsie has quit IRC | 20:25 | |
clarkb | ok I think I understand ansible package node, its a meta module that becomes apt or yum etc, and apt at least accepts a list | 20:25 |
clarkb | interestingly the yum docs show both methods of installing a list of pakcages | 20:27 |
clarkb | corvus: ok the install puppet role uses this too and we definitely check that the puppet package sstuff is installed | 20:28 |
corvus | clarkb: approach in 598329 lgtm | 20:28 |
clarkb | which is good enough for me so approved | 20:28 |
*** e0ne has quit IRC | 20:29 | |
notmyname | dhellmann: I'm asking for confirmation that yes the plan is to do something non-standard (merge test-only, and perhaps unique, changes to a stable branch) in order to get to the end state faster (ie not waiting for stable branches to naturally expire). | 20:30 |
dhellmann | notmyname : yes, that is the plan | 20:31 |
dhellmann | if you'd like I will review the patches you have, but Nguyen has been doing well with the other projects so far. | 20:31 |
fungi | also the "extended maintenance" plan where we agreed to just stop deleting branches and possibly let them lie fallow instead plays into this a bit | 20:33 |
fungi | harder to know when a branch has truly "expired" at this point | 20:33 |
notmyname | fungi: did the extended maintenance discussion get to a point of a formal plan? I remember the conversations, but not a resolution | 20:36 |
notmyname | dhellmann: nah, that's not necessary. these patches very closely match the pattern of patches that reviewers normally explicitly try to ignore (mechanical changes from a non-standard contributor to many projects at once, especially when the commit message says they are programatically generated). I'm trying to do the right thing, but it's already very high on my "not worth it" meter (because of histor | 20:36 |
notmyname | ical patches like this), and I need to understand the nuances so that it can be maintained. | 20:36 |
dhellmann | notmyname : Ok, well, the plan is to remove all of the test job settings from project-config so if you want to keep testing you'll need these patches, or something like them. Please read through the goal document and let me know if you have any questions about the details there. | 20:37 |
*** mugsie has joined #openstack-infra | 20:37 | |
dhellmann | there's a list of people I expect to be proposing these patches in that doc, for just this reason | 20:37 |
openstackgerrit | Merged openstack-infra/system-config master: unbound: fix dhclient.conf copy https://review.openstack.org/597242 | 20:38 |
notmyname | yeah, I really like the goal, and the doc does a pretty good job of explaining things. I love the idea of having all the test job configs per-project. but as with any proposed code, we need to understand it before landing it, so my questions are just to help clarify or make explicit the things I'm seeing implied | 20:39 |
fungi | notmyname: hrm, yeah https://docs.openstack.org/project-team-guide/stable-branches.html#extended-maintenance does say branches can still be declared eol, tagged and deleted. tonyb ^ is any of that still planned to change | 20:39 |
notmyname | ...especially where the implied things differ from standard or historic practice | 20:40 |
notmyname | tangental question, I noticed the new banner on docs saying, essentially, "these docs aren't as correct as some older ones". is there a way to disable that. I think it's misleading at best (and wrong at worst) for cycle-with-intermediary projects | 20:41 |
pabelanger | clarkb: yah, that's kinda what I was thinking for clouds.yaml | 20:42 |
dhellmann | notmyname : older ones? where is it saying that? | 20:42 |
clarkb | pabelanger: corvus thanks I'll work on sorting out what the hieradata to group_vars mapping is here | 20:42 |
dhellmann | fungi : I keep failing to convince tonyb to just leave all the old branches alone when they aren't maintained. | 20:42 |
notmyname | dhellmann: "this release is under development. current supported release is rocky" <-- I paraphrased :-) | 20:43 |
dhellmann | ah | 20:43 |
dhellmann | the paraphrase is definitely inaccurate; is the actual statement? :-) | 20:43 |
notmyname | today, that's somewhat misleading. yesterday it was wrong (when it said queens) | 20:43 |
corvus | dhellmann, fungi: i don't think branches should exist if patches aren't accepted | 20:43 |
*** harlowja has joined #openstack-infra | 20:43 | |
notmyname | and as soon as a -with-intermediary project does a release, the statement becomes wrong again | 20:43 |
dhellmann | notmyname: ok, I guess that's fair. the docs team manages the theme | 20:44 |
fungi | corvus: i agree with you | 20:44 |
dhellmann | corvus : I guess I'm just always looking for ways we can stop piling maintenance work on the ever shrinking group of people willing to do it. | 20:44 |
fungi | the people who wanted us to stop deleting them used arguments like "but, but, they _would_ be maintained if you just left them open and gave us a chance" | 20:44 |
fungi | we know from experience that hasn't been a realistic strategy in th epast though | 20:45 |
tosky | fungi: oh, thanks (wiki) | 20:45 |
dhellmann | I'm not going to get in anyone's way if they want to do it; I just think we have other things we could use tonyb's time for. | 20:45 |
dhellmann | I guess I don't consider an open branch an invitation to propose patches. | 20:45 |
fungi | tosky: i think i'm the only person still actively patrolling wiki edits and adding users to the verified group when they lame leditimate edits | 20:45 |
fungi | er, legitimate | 20:46 |
dhellmann | I haven't observed that behavior in the past myself, but maybe it does happen. | 20:46 |
fungi | s/lame/make/ | 20:46 |
*** bobh has quit IRC | 20:46 | |
corvus | dhellmann: believe it is an invitation to do that -- that's part of why we started doing that. | 20:46 |
fungi | freudian typo maybe | 20:46 |
*** rmart04 has joined #openstack-infra | 20:46 | |
dhellmann | corvus : Ok | 20:46 |
corvus | i think it's a really bad experience for someone to push a patch to a stable branch and have it ignored or rejected after all their hard work. better to make it clear that it's "closed". | 20:46 |
*** eernst has joined #openstack-infra | 20:47 | |
fungi | yes, we can post all the notices we like in documentation and readmes but people proposing patches to open branches tend not to see them | 20:47 |
dhellmann | I can see that. It's a bit of a chicken-and-egg issue, though, because if we close the branches then why would anyone ever show up to work on them in the first place? | 20:47 |
corvus | if the mechanical process of deleting branches is bothersome, perhaps we could automate it more. i do think it's a worthwhile to do. | 20:47 |
corvus | dhellmann: i think we should only delete them after people have stopped working on them :) | 20:47 |
fungi | the next gerrit release is supposed to make it finally possible to control the branch deletion api call with an explicit acl permission | 20:48 |
dhellmann | well, at least we agree on that | 20:48 |
fungi | (next after the one we're running, i mean... 2.14) | 20:48 |
dhellmann | that should make it possible to automate closing the branches | 20:48 |
corvus | sounds like a good time to revisit | 20:48 |
dhellmann | yep | 20:48 |
fungi | the main concern which was raised is that if we re-open branches we've already tagged as eol and deleted, then the next time we eol them what do we tag them with? stein-eol-no-really-we-mean-it-this-time (but it's a mostly contribed argument, we've only ever had to re-eol branches of projects a couple times that i can recall) | 20:49 |
fungi | er, contrived | 20:50 |
openstackgerrit | Merged openstack-infra/system-config master: Don't use loops with package task https://review.openstack.org/597241 | 20:53 |
*** hamzy has quit IRC | 20:59 | |
openstackgerrit | Sorin Sbarnea proposed openstack-infra/devstack-gate master: Bump default Ansible version from 2.3.2 to 2.5.8 https://review.openstack.org/598351 | 21:03 |
*** trown is now known as trown|outtypewww | 21:03 | |
tonyb | dhellmann: tagging the branches as EOL was a compromise to get mor peopel on board with the general plan | 21:06 |
openstackgerrit | Sorin Sbarnea proposed openstack-infra/zuul master: Display correct cursor when hovering over patchset header https://review.openstack.org/598222 | 21:13 |
*** rmart04 has quit IRC | 21:16 | |
*** dpawlik has quit IRC | 21:17 | |
*** eharney has joined #openstack-infra | 21:17 | |
*** holser_ has joined #openstack-infra | 21:19 | |
*** jcoufal has quit IRC | 21:20 | |
openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: WIP: Manage clouds.yaml files in ansible https://review.openstack.org/598329 | 21:21 |
clarkb | corvus: pabelanger ^ ok I think that is mostly complete except for testing (which should fail due to bad lookups?) and puppet is still managing the nodepool clouds.yaml because that is a required piece of the puppetry today | 21:22 |
corvus | clarkb: we can put fake secret values in playbooks/zuul/templates/... | 21:25 |
corvus | we do that with all.yaml to add the root private key for ansible | 21:26 |
*** dmellado has quit IRC | 21:31 | |
*** dmellado has joined #openstack-infra | 21:31 | |
dansmith | can someone tell me if this is my fault? | 21:34 |
dansmith | http://logs.openstack.org/53/598353/1/check/neutron-grenade/14e0da6/ | 21:34 |
dansmith | no logs dir, | 21:34 |
dansmith | very little of use in the output.txt and not sure if that's an infra fail or what | 21:34 |
pabelanger | we could add our testing host_vars / group_vars into /etc/ansible/hosts, when running in the gate. since that is unmanaged on bridge.o.o today | 21:34 |
dansmith | oh, looks like there are other jobs hitting that right now | 21:35 |
pabelanger | dansmith: think it is a bug in ara, with ansible 2.3: http://logs.openstack.org/53/598353/1/check/neutron-grenade/14e0da6/job-output.txt.gz#_2018-08-30_21_30_48_904114 | 21:35 |
dansmith | like all grenade-y jobs, nova-next, etc | 21:35 |
pabelanger | patch up to bump ansible 2.5 in devstack-gate | 21:35 |
dansmith | pabelanger: ack okay hadn't seen it yet, sorry | 21:35 |
pabelanger | https://review.openstack.org/591527/ should be the fix | 21:36 |
fungi | i guess the grenade jobs are still legacy-based | 21:36 |
*** stevebaker has quit IRC | 21:36 | |
corvus | pabelanger: that's exactly what the run-base job does | 21:36 |
dansmith | fungi: and all our tree-defined ones I think? | 21:36 |
corvus | pabelanger: that's what i suggested to clarkb | 21:36 |
pabelanger | corvus: ah, great. I've missed that | 21:36 |
*** holser_ has quit IRC | 21:37 | |
corvus | pabelanger, clarkb: these tasks: http://git.openstack.org/cgit/openstack-infra/system-config/tree/playbooks/zuul/run-base.yaml#n26 | 21:37 |
corvus | we can template in things that should be variables (like the bastion host in the job), or just put static values in the templates (what we would probably do to create a "fake" clouds.yaml. | 21:38 |
pabelanger | yah, I think that is a great idea | 21:38 |
*** agopi has quit IRC | 21:40 | |
clarkb | corvus: thanks I will take a look at doing that | 21:41 |
*** stevebaker has joined #openstack-infra | 21:41 | |
*** holser_ has joined #openstack-infra | 21:42 | |
*** eernst has quit IRC | 21:42 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove job settings for SIGs repositories https://review.openstack.org/598360 | 21:46 |
*** cdent has joined #openstack-infra | 21:46 | |
*** dkehn is now known as dkehn_ | 21:46 | |
*** priteau has quit IRC | 21:47 | |
*** rcernin has joined #openstack-infra | 21:49 | |
corvus | clarkb, fungi, Shrews, pabelanger: when you have a moment, can you put https://review.openstack.org/592213 and children on your radar? having that in place before ptg could let us start to work on zuul-triggered deployment | 21:51 |
corvus | i'll tag that update-cfg-mgmt | 21:51 |
fungi | looking! (sorry, i've been sucked into other black holes all day/week) | 21:52 |
corvus | np, i figured you figured i'd let you know when it was worth looking at :) | 21:52 |
fungi | ooh, i remember this one! | 21:53 |
*** diablo_rojo has quit IRC | 21:53 | |
fungi | "it's a unix system, i know this!" | 21:53 |
fungi | something like that ;) | 21:53 |
corvus | fungi: several "1"s have been changed to "0"s. :) | 21:54 |
*** florianf has quit IRC | 21:54 | |
fungi | seems like that's always the case | 21:54 |
*** dpawlik has joined #openstack-infra | 21:54 | |
openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: WIP: Manage clouds.yaml files in ansible https://review.openstack.org/598329 | 21:55 |
corvus | so tell me, are any *other* jurrasic park movies any good? or is it like the matrix where they only made one? | 21:55 |
clarkb | corvus: its mostly where the only made one | 21:55 |
clarkb | the other films are decent action movies but nothing on par with the original imo | 21:55 |
*** boden has quit IRC | 21:55 | |
clarkb | also we now have testing, I just have to figure out how I want to handle puppet managing nodepool clouds.yaml now | 21:56 |
clarkb | I guess we just make that optional in the puppet | 21:56 |
fungi | i don't think i ever saw any of the jp sequels either | 21:58 |
*** dpawlik has quit IRC | 21:59 | |
fungi | though where the matrix was concerned, i did actually rather enjoy the animatrix shorts | 21:59 |
fungi | unlike the sequels | 21:59 |
fungi | though animatrix was mostly prequel material (in addition to being highly stylized) | 21:59 |
clarkb | the book is really good if you haven't read it | 22:01 |
SotK | I've not seen the modern jp films, but my experience of the first 3 is that it is like the matrix | 22:01 |
fungi | which book? if you mean jp, i found it to be so-so | 22:01 |
openstackgerrit | Chris Dent proposed openstack-infra/project-config master: Create an openstack/placement project https://review.openstack.org/598362 | 22:02 |
cdent | fungi, clarkb etc I followed along in https://docs.openstack.org/infra/manual/creators.html to create ^ but I'm not filled with confidence | 22:02 |
fungi | then again, i find most of crichton's work that i've tried to read to be poorly-researched | 22:02 |
cdent | It's at once "oh yeah that's simple" and fraught | 22:03 |
fungi | cdent: our ci jobs and reviewers will fill you with any confidence you may be lacking | 22:03 |
cdent | ✔ | 22:03 |
* corvus pours another shot of confidence | 22:03 | |
clarkb | fungi: ya jp, you have to treat it as half fantasy, it is entertaining but not kim stanley robinson levels of sifi | 22:03 |
dmsimard | corvus: zuul triggered deployment ? | 22:04 |
fungi | clarkb: yeah, i read it. it was fun enough that i didn't regret reading it. but it didn't exactly blow me away | 22:04 |
corvus | dmsimard: 3rd pgraph in http://specs.openstack.org/openstack-infra/infra-specs/specs/update-config-management.html#ansible | 22:04 |
clarkb | fungi: have you read red mars? if not you'd probably like it if you want well researched scifi | 22:05 |
cdent | red mars++ | 22:05 |
fungi | not yet, but i've been meaning to | 22:05 |
fungi | good to know | 22:05 |
* corvus adds to list | 22:05 | |
fungi | i like a lot of sci with just enough fi to keep me on my toes | 22:05 |
dmsimard | corvus: ah, I'm currently flirting the idea to trigger AWX/Tower jobs from Zuul's post pipeline (there's ansible modules to control awx/tower) | 22:07 |
cdent | what is the gold-standard prefered form of Depends-On these days? | 22:07 |
corvus | cdent: url | 22:08 |
cdent | corvus: thanks | 22:08 |
cdent | aw bummer, I broked it | 22:09 |
slaweq | hi | 22:10 |
slaweq | are You aware of issue like in http://logs.openstack.org/26/597426/1/check/neutron-functional/5568b6a/job-output.txt.gz#_2018-08-30_19_05_58_206102 ? | 22:10 |
corvus | cdent: should be an easy fix: http://logs.openstack.org/62/598362/1/check/project-config-gerrit/5882f8c/job-output.txt.gz#_2018-08-30_22_05_08_797199 | 22:10 |
slaweq | I see it now in many neutron related jobs | 22:11 |
cdent | corvus: yeah, already on it | 22:11 |
*** ijw has joined #openstack-infra | 22:11 | |
corvus | slaweq: https://review.openstack.org/591527 should fix | 22:11 |
dmsimard | slaweq: it'll be fixed once https://review.openstack.org/591527 lands | 22:11 |
dmsimard | My fault! | 22:11 |
slaweq | thx corvus and dmsimard for info | 22:11 |
*** jamesmcarthur has quit IRC | 22:13 | |
*** jamesmcarthur has joined #openstack-infra | 22:14 | |
*** ijw has quit IRC | 22:16 | |
*** hashar has quit IRC | 22:16 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove job settings for InteropWG repositories https://review.openstack.org/598377 | 22:16 |
*** mriedem is now known as mriedem_lawnboy | 22:17 | |
*** gema has quit IRC | 22:19 | |
openstackgerrit | Clark Boylan proposed openstack-infra/puppet-openstackci master: Make oscc_file_contents optional https://review.openstack.org/598378 | 22:20 |
openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Manage clouds.yaml files in ansible https://review.openstack.org/598329 | 22:21 |
clarkb | corvus: pabelanger ^ I think that does it | 22:21 |
*** tosky has quit IRC | 22:23 | |
*** anteaya has joined #openstack-infra | 22:28 | |
*** holser_ has quit IRC | 22:30 | |
openstackgerrit | Dean Troyer proposed openstack-infra/project-config master: Add stx-distcloud and stx-distcloud-client repos https://review.openstack.org/598383 | 22:34 |
*** sshnaidm is now known as sshnaidm|off | 22:35 | |
corvus | clarkb: i'm really interested in the openstacksdf. i'm assuming that's the self-defence force. | 22:35 |
clarkb | corvus: that was typo in mordreds change that I decided to fix :) | 22:36 |
fungi | space defense force | 22:36 |
openstackgerrit | Dean Troyer proposed openstack-infra/project-config master: Add Zuul jobs for new StarlingX repos https://review.openstack.org/598386 | 22:36 |
openstackgerrit | Dean Troyer proposed openstack-infra/project-config master: Add stx-distcloud and stx-distcloud-client repos https://review.openstack.org/598383 | 22:37 |
*** dbecker has quit IRC | 22:37 | |
clarkb | if we want to test the clouds.yaml change I can update it such that we only write all-clouds.yaml on bridge first since nothing currently uses it then write clouds.yaml on bridge and nodepool in a followup | 22:41 |
clarkb | though I did a text replacement on the .erb to make them .j2 so it should all line up as long as I got the group_vars names right | 22:42 |
openstackgerrit | Dean Troyer proposed openstack-infra/project-config master: Add stx-distcloud and stx-distcloud-client repos https://review.openstack.org/598383 | 22:43 |
openstackgerrit | Chris Dent proposed openstack-infra/project-config master: Create an openstack/placement project https://review.openstack.org/598362 | 22:46 |
*** anteaya has quit IRC | 22:47 | |
openstackgerrit | Merged openstack-infra/zuul master: Add missing parentheses to title invocation https://review.openstack.org/595361 | 22:48 |
corvus | clarkb: er, switched to -1 on https://review.openstack.org/598329 | 22:48 |
clarkb | corvus: currently the tests will only test it for bridge.o.o you ok with just setting it for host_vars/bridge.o.o ? or should we add group_vars/nodepool for completeness? | 22:49 |
corvus | clarkb: (though, also, i think maybe we should refactor those out into a new host group) | 22:49 |
corvus | clarkb: completeness -- i was about to build a nodepool job | 22:50 |
clarkb | cool | 22:50 |
corvus | clarkb: would you mind rebasing that stack? that'll make the nodepool job easier to build | 22:50 |
clarkb | corvus: rebase to master? | 22:51 |
corvus | yep | 22:51 |
clarkb | sure | 22:51 |
openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk https://review.openstack.org/593029 | 22:51 |
openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Manage clouds.yaml files in ansible https://review.openstack.org/598329 | 22:51 |
clarkb | rebased, now to address the review comment | 22:51 |
cdent | corvus: between when that request for an openstack/placement is approved and when the new repo is seeded from its upstream: is there much delay? | 22:52 |
fungi | cdent: lately could be up to an hour | 22:54 |
cdent | fungi: thanks. that's suitably and nicely fast | 22:54 |
fungi | we're in the midst of a transition to a new way of rolling out our continuous deployment and are still fine-tuning it for performance | 22:54 |
*** anteaya has joined #openstack-infra | 22:54 | |
openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Manage clouds.yaml files in ansible https://review.openstack.org/598329 | 22:55 |
corvus | we're aiming for minutes :) | 22:55 |
clarkb | corvus: ^ comment addressed | 22:56 |
cdent | I was mostly thinking in terms of "is this something I'm going to start working on tomorrow or next week" | 22:56 |
*** jamesmcarthur has quit IRC | 22:56 | |
cdent | it's happy in ci now, pointing out the review to nova folk | 22:58 |
clarkb | cdent: did you delete the openwrt image in nova as part of the filter branching? it clones so quickly :) | 22:58 |
cdent | ha | 22:58 |
cdent | ed did some before and after measurements: the repo goes from abourt 511M to 113 | 22:59 |
*** eharney has quit IRC | 22:59 | |
corvus | oh good. that's much appreciated. | 23:00 |
clarkb | the change lgtm mechanically | 23:00 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove job settings for User Committee repositories https://review.openstack.org/598400 | 23:00 |
*** jamesmcarthur has joined #openstack-infra | 23:00 | |
clarkb | and now I've got to step out for a bit | 23:01 |
*** tpsilva has quit IRC | 23:02 | |
*** jamesmcarthur has quit IRC | 23:03 | |
*** jamesmcarthur has joined #openstack-infra | 23:03 | |
*** jamesmcarthur has quit IRC | 23:06 | |
*** dbecker has joined #openstack-infra | 23:06 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add a run-nodepool job https://review.openstack.org/598403 | 23:07 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add testinfra tests for bridge https://review.openstack.org/598404 | 23:07 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add a run-nodepool job https://review.openstack.org/598403 | 23:10 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add testinfra tests for bridge https://review.openstack.org/598404 | 23:10 |
*** ijw has joined #openstack-infra | 23:12 | |
*** agopi has joined #openstack-infra | 23:12 | |
openstackgerrit | James E. Blair proposed openstack-infra/project-config master: Zuul: increase max-nodes-per-job https://review.openstack.org/598406 | 23:12 |
ianw | corvus: https://review.openstack.org/#/c/595920/ -- should i update that change about variable naming guidelines to say "always prefix"? | 23:12 |
corvus | ianw: seems like it might be a good idea, yeah | 23:13 |
ianw | corvus: also https://review.openstack.org/#/c/596014/ -- this warns when a README.rst isn't found, which triggers our -E and won't require extras steps in repo linters | 23:13 |
ianw | corvus: also https://review.openstack.org/#/c/596630/ -- for setting the ansible group. pabelanger isn't 100% convinced but i'd like to go one way or the other | 23:15 |
ianw | https://review.openstack.org/#/c/597374/ is a pretty trivial typo fix too for kerberos role variables | 23:16 |
*** ijw has quit IRC | 23:17 | |
*** cdent has quit IRC | 23:22 | |
*** adriant has quit IRC | 23:23 | |
*** adriant has joined #openstack-infra | 23:23 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove job settings for User Committee repositories https://review.openstack.org/598400 | 23:26 |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: Add guidelines on variable naming https://review.openstack.org/595920 | 23:28 |
clarkb | ianw: for the zuul-sphinx change any idea if that will force us to fix a bunch of roles after we merge the readme check? | 23:31 |
ianw | clarkb: it will require a release because zuul-sphinx isn't really co-installed, but i don't think so because we have a manual check in system-config ATM | 23:31 |
ianw | at any rate, it's pretty easy to drop in a README.rst even if it's just a stub; it's at least something someone can contribute to | 23:32 |
clarkb | ok I didn't approve it since you specifically asked for corvus' input but lgtm | 23:32 |
ianw | thanks, since it needs a release to be effective probably covus' bailiwick | 23:36 |
*** r-daneel has quit IRC | 23:36 | |
* ianw is unsure if bailiwick is a Queen's english thing that will confuse everyone else | 23:37 | |
clarkb | I'm not familiar with it :) | 23:37 |
*** anteaya has quit IRC | 23:44 | |
*** signed8b_ has joined #openstack-infra | 23:44 | |
*** signed8bit has quit IRC | 23:47 | |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Refactor Project Group editing to match other objects https://review.openstack.org/598407 | 23:51 |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Support linking to Project Groups by name https://review.openstack.org/598408 | 23:51 |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Link to Project Groups by name https://review.openstack.org/598409 | 23:51 |
*** dingyichen has joined #openstack-infra | 23:52 | |
*** rcernin has quit IRC | 23:56 | |
*** rcernin has joined #openstack-infra | 23:56 | |
openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Manage clouds.yaml files in ansible https://review.openstack.org/598329 | 23:58 |
clarkb | corvus: ^ minor file name derp I had to fix | 23:58 |
clarkb | corvus: should I rebase your changes? | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!