Thursday, 2018-08-30

fungicould be00:00
fungihttps://sks-keyservers.net/pks/lookup?op=vindex&search=0x27023b1ffccd8e3ae9a5ce95d943d5d270273ada&fingerprint=on is what i'll be linking on the release site anyway00:01
fungiit's not only got the stein key but also signatures from the rocky key and from me00:01
clarkbhttp://keyserver.ubuntu.com:11371/pks/lookup?search=0xD943D5D270273ADA&op=vindex shows my signature too00:02
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for neutron repositories  https://review.openstack.org/59786500:05
fungiAn 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_LONG00:13
pabelangerclarkb: I'm trying to get testinfra to work in https://review.openstack.org/593029/ first00:13
pabelangerneed to figure out pytest fixtures to run tests against a specific host00:14
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Bug on endpoint /api/v1/summits/{id}/events/published/empty-spots  https://review.openstack.org/59796500:19
openstackgerritMerged openstack-infra/openstackid-resources master: Bug on endpoint /api/v1/summits/{id}/events/published/empty-spots  https://review.openstack.org/59796500:20
*** dingyichen has joined #openstack-infra00:25
*** longkb has joined #openstack-infra00:26
openstackgerritIan Wienand proposed openstack-infra/project-config master: wheel-build: only install afs client for publish jobs  https://review.openstack.org/59796600:26
ianwtonyb: ^ so i think it's that easy00:26
ianwpabelanger: i did that for the logrotate work00:26
ianwyou can inspect the ansible host variables00:26
tonybianw: 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
ianwtonyb: i'd say that makes sense, it makes sure the wheel builds will still work00:30
tonybianw: Okay.00:32
tonybianw: I have updated the 2 chnages that touch bindep.txt in openstack/requirements to depnds-on your change00:32
ianwtonyb: 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 jobs00:34
tonybianw: Oh :( I though all the zuulness was dynamic like that :/00:35
tonybianw: If you're okay with a quick merge I am ;P00:35
ianwit just get complicated where secrets and post-only jobs get involved.  this straddles it all so i dunno :)00:37
openstackgerritPaul Belanger proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk  https://review.openstack.org/59302900:38
pabelangerianw: thanks, one last test then I'll try that. But likely morning now, based on size of check pipeline00:42
ianwpabelanger: decorator looks good too if it works :)00:43
pabelangerlets hope00:43
ianwwhat's in the queue that we have 800 centos-7 nodes going?00:49
fungitonyb: 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 automatically00:49
*** efried1 has joined #openstack-infra00:50
tonybfungi: Yup I can see that.  I00:50
tonyb'll probably need to be told more than once befoer I remember it ;P00:51
*** efried has quit IRC00:51
*** efried1 is now known as efried00:51
fungihowever, 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 up00:51
fungiso the less we put in there, the better00:51
tonybfungi: cool.  That makes a lot of sense00:54
clarkbianw: tripleo?00:55
ianwyeah, bits of the afs build could conceivably be moved into o-z-j00:55
ianwclarkb: 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
clarkbhrm that doesnt render great on a phone00:58
*** efried has quit IRC00:58
clarkbso no idea :)00:58
ianwi'm thinking might just need to rm those stats on disk if they snuck in there at some point00:59
ianw"Remove limestone specific labels from nl02.o.o"01:00
ianwyeah, looks like they were just for testing at one point01:00
*** signed8bit_Zzz is now known as signed8bit01:07
ianwok, manually removed those, and fedora-26.  graphs are much less busy now01:08
ianwoh there's a few more old fedoras might as well clear while here01:09
openstackgerritNguyen Hai proposed openstack-infra/project-config master: remove job settings for designate repositories  https://review.openstack.org/59286501:12
openstackgerritNguyen Hai proposed openstack-infra/project-config master: remove job settings for designate repositories  https://review.openstack.org/59286501:13
openstackgerritNguyen Hai proposed openstack-infra/project-config master: remove job settings for designate repositories  https://review.openstack.org/59286501:13
*** rh-jelabarre has quit IRC01:16
openstackgerritDavid Moreau Simard proposed openstack-infra/nodepool master: WIP: Implement an Amazon EC2 driver  https://review.openstack.org/53555801:18
*** bobh has joined #openstack-infra01:19
*** anteaya has joined #openstack-infra01:19
*** efried has joined #openstack-infra01:20
*** gyee has quit IRC01:22
*** jiapei has joined #openstack-infra01:23
*** signed8bit is now known as signed8bit_Zzz01:25
openstackgerritNguyen Hai proposed openstack-infra/project-config master: remove job settings for swift repositories  https://review.openstack.org/59622401:33
*** rlandy|bbl is now known as rlandy01:37
*** bobh has quit IRC01:41
*** rlandy has quit IRC01:42
*** ijw has joined #openstack-infra01:46
*** ijw has quit IRC01:51
*** hongbin has joined #openstack-infra01:54
*** jamesdenton has quit IRC01:59
*** larainema has quit IRC02:00
*** slaweq has joined #openstack-infra02:11
*** bobh has joined #openstack-infra02:15
*** slaweq has quit IRC02:16
*** bobh has quit IRC02:16
*** apetrich has quit IRC02:16
*** bobh has joined #openstack-infra02:16
*** anteaya has quit IRC02:19
*** bobh has quit IRC02:20
mnaserso it looks like things have gone up to 5002:41
mnaserand no fires are up?02:41
*** ijw has joined #openstack-infra02:43
*** ijw has quit IRC02:47
*** psachin has joined #openstack-infra02:50
*** bhavikdbavishi has joined #openstack-infra02:54
*** signed8bit_Zzz is now known as signed8bit03:12
*** signed8bit is now known as signed8bit_Zzz03:12
*** ykarel has joined #openstack-infra03:24
*** ykarel_ has joined #openstack-infra03:31
*** jiapei has quit IRC03:33
*** ykarel has quit IRC03:33
*** rosmaita has quit IRC03:34
*** bhavikdbavishi has quit IRC03:34
*** nicolasbock has quit IRC03:40
*** graphene has quit IRC03:49
*** ykarel__ has joined #openstack-infra03:58
*** ykarel_ has quit IRC03:58
*** janki has joined #openstack-infra04:05
openstackgerritMerged openstack-infra/project-config master: wheel-build: only install afs client for publish jobs  https://review.openstack.org/59796604:09
ianwmnaser: things seem ok!  gate is busy04:09
*** ramishra has joined #openstack-infra04:12
openstackgerritMerged openstack-infra/irc-meetings master: Update Searchlight team meeting  https://review.openstack.org/59753704:14
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Fixes on member affiliation API  https://review.openstack.org/59798404:15
openstackgerritMerged openstack-infra/openstackid-resources master: Fixes on member affiliation API  https://review.openstack.org/59798404:16
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Infra repo retirement of 18 repos - step 3  https://review.openstack.org/59763604:17
openstackgerritMerged openstack-infra/zuul-packaging master: Retire zuul-packaging  https://review.openstack.org/59753104:18
*** ykarel__ has quit IRC04:22
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Infra repo retirement of 18 repos - step 3  https://review.openstack.org/59763604:28
*** markvoelker has joined #openstack-infra04:34
AJaegerconfig-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
AJaegerconfig-core, and a couple of grafana updates: https://review.openstack.org/597168  https://review.openstack.org/59576304:49
*** hongbin has quit IRC04:56
*** ykarel__ has joined #openstack-infra04:58
*** ykarel__ is now known as ykarel05:00
*** jbadiapa has quit IRC05:01
*** ijw has joined #openstack-infra05:06
*** ijw has quit IRC05:10
*** slaweq has joined #openstack-infra05:11
*** shabeeb has joined #openstack-infra05:11
openstackgerritMerged openstack-infra/project-config master: remove job settings for designate repositories  https://review.openstack.org/59286505:15
*** slaweq has quit IRC05:15
*** srp has joined #openstack-infra05:23
*** shabeeb has quit IRC05:23
*** e0ne has joined #openstack-infra05:31
openstackgerritMerged openstack-infra/project-config master: Neutron failure rate - include TIMED_OUT jobs on graphs  https://review.openstack.org/59576305:34
*** dingyichen has quit IRC05:35
*** dingyichen has joined #openstack-infra05:37
*** ccamacho has joined #openstack-infra05:38
openstackgerritIan Wienand proposed openstack-infra/project-config master: wheel-build: ignore existing files and fix error log  https://review.openstack.org/59798905:38
ianwtonyb: ^05:38
tonybianw: +1 Thanks!05:40
tonybianw: and you fixed the off by 2 error ;P05:40
ianwoh now i see what you mean :)05:40
ianwi think we should merge that and try again.  i think there might be other fails?05:41
*** ijw has joined #openstack-infra05:42
ianwovs/_json.c:2:30: fatal error: openvswitch/json.h: No such file or directory05:43
ianwstuff like that ... although that wasn't a hard failure05:43
openstackgerritMerged openstack-infra/project-config master: Add openstack/os-ken to neutron  https://review.openstack.org/58835505:46
*** ijw has quit IRC05:47
tonybianw: Yup I'm good to watch aother run or 305:50
ianwactually05:51
tonybianw: Some of the failures are more or less expected (on bionic) but xenial and centos shoudl be fine05:51
ianwthe 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
tonybianw: Oh interesting05:52
openstackgerritMerged openstack-infra/project-config master: wheel-build: ignore existing files and fix error log  https://review.openstack.org/59798905:53
*** udesale has joined #openstack-infra05:53
* tonyb deosn't really see how that's possible unless there was a network snafu05:54
ianw... or the openafs client doesn't work05:55
ianwno that's not right, it's actually working in the afs directory05:55
*** snapiri has joined #openstack-infra05:59
ianwoh, i bet the python 2 & 3 updates raced?06:00
tonybianw: Yeah that could be it06:02
ianwi 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 enough06:03
*** signed8bit_Zzz is now known as signed8bit06:06
*** openstackgerrit has quit IRC06:07
*** ramishra has quit IRC06:07
*** ijw has joined #openstack-infra06:07
*** udesale has quit IRC06:07
*** udesale has joined #openstack-infra06:08
*** ijw has quit IRC06:12
*** openstackgerrit has joined #openstack-infra06:13
openstackgerritOpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/59799606:13
openstackgerritIan Wienand proposed openstack-infra/project-config master: wheel-index: only create index once  https://review.openstack.org/59799706:14
*** ramishra has joined #openstack-infra06:14
ianwtonyb: ^ best guess.  think should run that by pabelanger06:14
tonybianw: Looks okay to me and I can see how the commit message macthes the code but that's about the depth of my review06:16
tonybianw: it's waay deeper ansible mojo that I grok ;p06:16
*** e0ne has quit IRC06:19
*** e0ne has joined #openstack-infra06:20
*** e0ne has quit IRC06:20
*** alexchadin has joined #openstack-infra06:23
*** srp has quit IRC06:25
*** srp has joined #openstack-infra06:26
mpetersondhellmann: 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
AJaegermpeterson: where exactly?06:29
AJaegerOh, I see it...06:29
AJaegermpeterson: want to fix it manually yourself? Good catch!06:29
*** janki has quit IRC06:30
*** jchhatbar has joined #openstack-infra06:30
AJaegermpeterson: and py35 is now there three times - I left a -1 for both06:31
mpetersonAJaeger: yeah, I'll fix it manually. I'm just leaving him a message so he can review the tool before proceeding to generate more patches06:31
mpetersonAJaeger: yup, I saw that one as well, thanks!06:32
AJaegermpeterson: yep, that's an important heads-up06:32
*** shardy has joined #openstack-infra06:32
AJaegermpeterson: and that py35 can be abandoned I guess - and then rebase the py36 on the other one06:32
*** signed8bit has quit IRC06:35
*** srp has quit IRC06:36
*** srp has joined #openstack-infra06:37
mpetersonAJaeger: will do06:38
openstackgerritMerged openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/59799606:38
*** shabeeb has joined #openstack-infra06:39
*** shabeeb has quit IRC06:40
*** srp has quit IRC06:42
*** dklyle has quit IRC06:43
AJaegerconfig-core, https://review.openstack.org/#/c/593264 is next python3-first in-repo migration ready for review, please06:43
*** ijw has joined #openstack-infra06:43
*** dklyle has joined #openstack-infra06:44
*** ginopc has joined #openstack-infra06:46
AJaegerianw: 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 IRC06:47
openstackgerritDobroslaw Zybort proposed openstack-infra/project-config master: Remove jobs settings for monasca-notification  https://review.openstack.org/59806106:54
*** alexchadin has quit IRC07:04
*** alexchadin has joined #openstack-infra07:04
*** alexchadin has quit IRC07:05
*** florianf has joined #openstack-infra07:05
*** apetrich has joined #openstack-infra07:09
*** slaweq has joined #openstack-infra07:11
*** jbadiapa has joined #openstack-infra07:14
*** pcaruana has joined #openstack-infra07:14
*** slaweq has quit IRC07:16
*** rcernin has quit IRC07:20
*** rtjure has quit IRC07:20
*** slaweq has joined #openstack-infra07:23
mpetersonAJaeger: do you know if there are neutron flavored versions of openstack-tox-cover and openstack-tox-lower-constraints ?07:27
AJaegermpeterson: check openstack-infra/openstack-zuul-jobs repo, if they are not there, they don't exist07:29
*** hashar has joined #openstack-infra07:35
mpetersonAJaeger: 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-infra07:38
AJaegermpeterson: 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
mpetersonAJaeger: currently only jobs and no template for them exists07:39
AJaegerright now we have no cover/lower-constraints templates... but could add those as well07:39
AJaegermpeterson: no time right now to dig deeper - but happy to review a proposal and then iterate on it.07:40
mpetersonAJaeger: sure will do, I'll add you to the reviewers07:41
*** janki has joined #openstack-infra07:43
*** jchhatbar has quit IRC07:44
*** olivierb_ has joined #openstack-infra07:47
*** jchhatbar has joined #openstack-infra07:49
*** hwoarang has joined #openstack-infra07:49
AJaegermpeterson: no need - if it's in ozj, I'll have my queries ;)07:49
*** janki has quit IRC07:49
*** gema has joined #openstack-infra07:50
*** jpena|off is now known as jpena07:54
*** jpich has joined #openstack-infra07:56
*** ijw has joined #openstack-infra08:06
*** ijw has quit IRC08:11
*** pbourke has quit IRC08:15
*** pbourke has joined #openstack-infra08:17
*** electrofelix has joined #openstack-infra08:21
*** Emine has joined #openstack-infra08:23
openstackgerritTony Breeds proposed openstack-infra/project-config master: Use $NF to get the last field from the wheel build log file  https://review.openstack.org/59808308:24
tonybianw: ^^08:25
*** rabel has left #openstack-infra08:25
*** ykarel is now known as ykarel|lunch08:30
*** Bhujay has joined #openstack-infra08:37
*** markvoelker has quit IRC08:38
*** ijw has joined #openstack-infra08:42
*** ijw has quit IRC08:47
*** e0ne has joined #openstack-infra08:48
*** dtantsur|afk is now known as dtantsur08:49
*** panda is now known as panda|rover08:52
openstackgerritMerged openstack-infra/project-config master: Use $NF to get the last field from the wheel build log file  https://review.openstack.org/59808308:53
*** noama has joined #openstack-infra09:00
*** ramishra has quit IRC09:00
*** holser_ has joined #openstack-infra09:02
*** dingyichen has quit IRC09:02
*** holser_ has quit IRC09:02
*** holser_ has joined #openstack-infra09:03
*** ijw has joined #openstack-infra09:06
*** dpawlik has quit IRC09:06
*** dpawlik has joined #openstack-infra09:07
*** ijw has quit IRC09:11
*** markvoelker has joined #openstack-infra09:13
*** ramishra has joined #openstack-infra09:13
*** markvoelker has quit IRC09:15
*** sambetts|afk is now known as sambetts09:18
*** ykarel|lunch is now known as ykarel09:18
*** priteau has joined #openstack-infra09:20
*** udesale has quit IRC09:26
*** jesusaur has quit IRC09:27
*** jesusaur has joined #openstack-infra09:33
*** flaper87 has quit IRC09:34
*** flaper87 has joined #openstack-infra09:36
*** ccamacho has quit IRC09:39
*** ccamacho has joined #openstack-infra09:39
openstackgerritMichel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-lower-constraints  https://review.openstack.org/59810309:40
openstackgerritMichel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-cover  https://review.openstack.org/59810409:41
*** ijw has joined #openstack-infra09:42
*** ijw has quit IRC09:46
*** ginopc has quit IRC09:48
*** ginopc has joined #openstack-infra09:52
*** alexchadin has joined #openstack-infra09:57
*** ijw has joined #openstack-infra10:05
*** ijw has quit IRC10:10
*** hashar is now known as hasharLunch10:11
*** jaosorior has quit IRC10:25
*** jaosorior has joined #openstack-infra10:27
*** udesale has joined #openstack-infra10:33
*** dave-mccowan has joined #openstack-infra10:34
*** ijw has joined #openstack-infra10:41
*** udesale has quit IRC10:44
*** udesale has joined #openstack-infra10:44
*** ijw has quit IRC10:46
*** annp has quit IRC10:46
*** nicolasbock has joined #openstack-infra10:47
*** shardy is now known as shardy_lunch10:57
*** priteau has quit IRC11:00
openstackgerritJens Harbott (frickler) proposed openstack-infra/openstack-zuul-jobs master: Add openstack-python36-jobs-horizon variant  https://review.openstack.org/59813111:00
*** gfidente has joined #openstack-infra11:04
*** ijw has joined #openstack-infra11:05
openstackgerritMichel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-lower-constraints  https://review.openstack.org/59810311:07
*** priteau has joined #openstack-infra11:08
openstackgerritMichel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-cover  https://review.openstack.org/59810411:08
*** ijw has quit IRC11:10
*** ansmith has quit IRC11:13
*** holser_ has quit IRC11:14
*** holser_ has joined #openstack-infra11:15
*** hwoarang has quit IRC11:20
*** jcoufal has joined #openstack-infra11:20
*** hwoarang has joined #openstack-infra11:20
*** hasharLunch is now known as hashar11:21
*** jpena is now known as jpena|lunch11:27
*** sshnaidm is now known as sshnaidm|afk11:28
AJaegermpeterson: 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
AJaegermordred: what's your take? ^11:30
*** rnm has joined #openstack-infra11:34
mpetersonAJaeger: AFAIU tox_install_siblings still needs the required-projects directive to be present, as it depends on Zuul checking out the project11:35
mpetersonAJaeger: http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/tox/tasks/siblings.yaml11:35
*** aojea has joined #openstack-infra11:36
*** rnm is now known as rmart0411:36
*** aojea has quit IRC11:36
*** aojea has joined #openstack-infra11:37
*** threestrands has quit IRC11:38
*** longkb has quit IRC11:38
*** rh-jelabarre has joined #openstack-infra11:39
*** rmart04 has quit IRC11:40
*** rnm has joined #openstack-infra11:40
*** ijw has joined #openstack-infra11:41
*** rnm has quit IRC11:42
*** rnm has joined #openstack-infra11:42
*** rnm is now known as rmart0411:43
*** hwoarang has quit IRC11:46
*** ijw has quit IRC11:46
*** hwoarang has joined #openstack-infra11:46
*** gouthamr has quit IRC11:49
*** dmellado has quit IRC11:50
*** stevebaker has quit IRC11:51
*** boden has joined #openstack-infra11:53
*** alexchadin has quit IRC11:59
openstackgerritTristan Cacqueray proposed openstack-infra/gerritlib master: Read all Gerrit events from poll interruption  https://review.openstack.org/41275712:01
*** ijw has joined #openstack-infra12:06
*** trown|outtypewww is now known as trown12:06
*** udesale has quit IRC12:08
*** dpawlik has quit IRC12:08
*** dpawlik has joined #openstack-infra12:09
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add config-errors notifications drawer  https://review.openstack.org/59714712:09
*** dpawlik has quit IRC12:10
*** dpawlik has joined #openstack-infra12:10
*** ijw has quit IRC12:11
*** udesale has joined #openstack-infra12:11
AJaegermpeterson: ah, indeed. so, the comment for those project could use updating to mention tox-siblings.12:14
*** shardy_lunch is now known as shardy12:18
florianfEmilienM: 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 containers12:20
smcginnisInfra folks - are things looking good for doing our final Rocky release within the next hour or two?12:21
smcginnisJust want to make sure there aren't any known issues going on before we hit the go button.12:21
AJaegersmcginnis: nobody around - and no problems know. Did you coordinate with docs team for docs.o.o page?12:22
smcginnisAJaeger: I did not. That is part of our process doc though.12:23
AJaegersmcginnis: I just pinged pkovar on #openstack-doc - I haven't seen anything yet and can't help ;(12:23
smcginnisAJaeger: Thanks.12:24
florianfEmilienM: oops, I replied in the wrong channel ;) sorry...12:28
*** alexchadin has joined #openstack-infra12:28
*** oanson has joined #openstack-infra12:30
*** jpena|lunch is now known as jpena|off12:31
*** jpena|off is now known as jpena12:32
*** slaweq has quit IRC12:34
*** slaweq has joined #openstack-infra12:34
*** rlandy has joined #openstack-infra12:37
*** alexchadin has quit IRC12:38
*** udesale has quit IRC12:40
*** signed8bit has joined #openstack-infra12:45
*** vivsoni has quit IRC12:49
*** vivsoni_ has joined #openstack-infra12:49
*** udesale has joined #openstack-infra12:53
*** kgiusti has joined #openstack-infra12:54
*** mriedem has joined #openstack-infra12:54
*** rosmaita has joined #openstack-infra12:55
*** ansmith has joined #openstack-infra12:57
*** tpsilva has joined #openstack-infra13:01
*** jbadiapa has quit IRC13:03
*** bobh has joined #openstack-infra13:04
*** sthussey has joined #openstack-infra13:05
openstackgerritMerged openstack-infra/pypi-mirror master: Retire pypi-mirror  https://review.openstack.org/59740113:06
openstackgerritThierry Carrez proposed openstack/ptgbot master: Identify with SASL  https://review.openstack.org/59815513:09
openstackgerritMichel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-cover  https://review.openstack.org/59810413:10
openstackgerritSimon Westphahl proposed openstack-infra/nodepool master: Update static nodes in Zookeeper on config change  https://review.openstack.org/59815613:11
openstackgerritMerged openstack-infra/project-config master: remove job settings for mistral repositories  https://review.openstack.org/59326413:11
openstackgerritMichel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-lower-constraints  https://review.openstack.org/59810313:11
mpetersonAJaeger: did it, as per your comment ^^13:12
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove obsolete legacy-js-generator-openstack-nodejs6-npm-docs  https://review.openstack.org/59737113:13
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove puppet-featuretracker  https://review.openstack.org/59765113:13
*** alexchadin has joined #openstack-infra13:13
openstackgerritSimon Westphahl proposed openstack-infra/nodepool master: Update static nodes in Zookeeper on config change  https://review.openstack.org/59815613:18
*** jbadiapa has joined #openstack-infra13:22
*** yamamoto has joined #openstack-infra13:37
*** jamesdenton has joined #openstack-infra13:38
*** aojea has quit IRC13:40
*** eharney has joined #openstack-infra13:40
*** shabeeb has joined #openstack-infra13:42
*** shabeeb has quit IRC13:43
*** toabctl has joined #openstack-infra13:44
*** Bhujay has quit IRC13:47
*** shabeeb has joined #openstack-infra13:48
*** udesale has quit IRC13:49
*** alexchadin has quit IRC13:50
*** udesale has joined #openstack-infra13:50
*** psachin has quit IRC13:51
*** jistr is now known as jistr|call13:51
*** aojea has joined #openstack-infra13:55
*** yamamoto has quit IRC13:56
fungiand so it begins13:59
fungifollow rocky release madness unfolding at http://zuul.openstack.org/13:59
*** ykarel is now known as ykarel|away14:00
fungithe tag-releases job in the release-post pipeline has just gotten an allocation14:00
fungishould see the release pipeline start filling up momentarily14:01
fungior it had an allocation, then got dequeued and requeued14:02
*** ykarel|away has quit IRC14:06
*** sshnaidm|afk has quit IRC14:10
AJaegerfungi, 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-infra14:19
AJaegermpeterson: thanks!14:19
*** sshnaidm|afk is now known as sshnaidm14:20
*** alexchadin has joined #openstack-infra14:20
AJaegerdhellmann: what do you think of  https://review.openstack.org/598103 ? Is there a special reason this wasn't done earlier?14:20
dhellmannI wonder if we're only running that job as a check and not in the gate?14:21
*** jistr|call is now known as jistr14:21
dhellmannAJaeger : seems OK to do14:22
AJaegerdhellmann: looking at http://codesearch.openstack.org/?q=openstack-tox-lower-constraints&i=nope&files=&repos= , it's in both check and gate14:23
AJaegerif you want, your script could convert to use the template... That would be nice - not essential ;)14:24
dhellmannthe migration script?14:24
dhellmannhmm14:24
dhellmannyeah, I don't know about adding that level of complexity, esp. if we have variants14:24
AJaegerdhellmann: understood14:25
AJaegermpeterson: https://review.openstack.org/598184 uses your new template...14:31
mpetersonAJaeger: nice! Btw, the template is called openstack-lower-constraints-jobs14:32
*** srp has joined #openstack-infra14:34
*** shabeeb has quit IRC14:35
AJaegermpeterson: thanks...14:35
*** ykarel has joined #openstack-infra14:35
openstackgerritPaul Belanger proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk  https://review.openstack.org/59302914:42
*** alexchadin has quit IRC14:42
*** jchhatbar has quit IRC14:43
mpetersonAJaeger, 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 IRC14:43
*** alexchadin has joined #openstack-infra14:44
AJaegermpeterson: if you want to script it, go for it - but then I suggest to also send an announcement out ;)14:45
AJaegerjust a different announcement...14:46
AJaegermpeterson: announcement in any case14:46
*** r-daneel has joined #openstack-infra14:47
mpetersonAJaeger: makes sense. okey, I'm wrapping up for this week. Have a good weekend.14:50
*** shabeeb has joined #openstack-infra14:51
*** vivsoni has joined #openstack-infra14:51
AJaegermpeterson: enjoy yours as well!14:52
*** srp has quit IRC14:52
*** ykarel has quit IRC14:54
corvusfungi, 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_56688214:56
corvusbased 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 process14:58
fungihuh, like ansible task parallelization at work?14:59
corvusfungi: well, test.py in this case14:59
corvuswe have test.py set up to use 5 parallel workers15:00
fungiohhh15:00
*** sdoran has joined #openstack-infra15:02
*** ykarel has joined #openstack-infra15:03
*** udesale has quit IRC15:04
*** zul has quit IRC15:05
*** ginopc has quit IRC15:08
*** zul has joined #openstack-infra15:08
*** ginopc has joined #openstack-infra15:08
*** ginopc has quit IRC15:11
*** ginopc has joined #openstack-infra15:12
*** pcaruana has quit IRC15:12
*** r-daneel has quit IRC15:13
*** r-daneel_ has joined #openstack-infra15:13
*** r-daneel_ is now known as r-daneel15:16
openstackgerritPaul Belanger proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk  https://review.openstack.org/59302915:16
fungionly one release seems to have failed its job (networking-bgpvpn 9.0.0) so i've reenqueued it15:17
*** dpawlik has quit IRC15:18
*** olivier__ has joined #openstack-infra15:21
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add system-config-run-eavesdrop  https://review.openstack.org/59761715:21
*** olivierb_ has quit IRC15:22
*** eglute has joined #openstack-infra15:25
*** ykarel has quit IRC15:25
*** e0ne has quit IRC15:27
*** ykarel has joined #openstack-infra15:29
*** Emine has quit IRC15:29
*** bobh has quit IRC15:35
AJaegerconfig-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 IRC15:38
*** ykarel_ has joined #openstack-infra15:38
*** ykarel has quit IRC15:39
*** gyee has joined #openstack-infra15:42
*** anteaya has joined #openstack-infra15:43
clarkbexpect a slow day from me, I've learned UPSes are really loud at 1am during power outages.15:44
*** olivier__ has quit IRC15:46
*** olivierb_ has joined #openstack-infra15:48
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: switch release job for networking-bgpvpn  https://review.openstack.org/59821315:52
fungii get reminded of that remarkably frequently here15:52
fungiand i have some dozen very loud upses scattered around the house due to the instability of our power grid here on the island15:53
fungineed to just give in and budget for whole-house ups and generator combo15:53
fungithey're pretty popular here so easy to find a contractor familiar with installing them15:53
corvusthen you can sell excess capacity to your neighbors....15:53
fungiexcept that my neighbors don't live in their houses, so they're vacant most of storm season15:54
clarkbfungi: I honeslty thought it was the fire alarms when they first woke me15:55
clarkbwas not a fun way to wake up15:55
*** ykarel_ is now known as ykarel15:55
*** trown is now known as trown|lunch15:55
dhellmannconfig-core: https://review.openstack.org/598213 should unblock the last release deliverable for the rocky release today15:56
*** ginopc has quit IRC15:59
pabelangerclarkb: 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 evening16:00
*** hamzy has quit IRC16:00
*** ykarel has quit IRC16:00
clarkbpabelanger: 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 me16:01
clarkbmay just be nice to have to avoid splitting the management of that data?16:01
dansmithcan anyone tell me if our xenial worker nodes have cloud-archive:pike already configured?\16:03
dansmithdevstack attempts to do it,16:03
clarkbdhellmann: approved16:04
dansmithbut I think it might not be updating qemu-utils from it after configured when I run it locally16:04
dansmithbut the gate jobs all work, so..16:04
clarkbdansmith: they should not have UCA preconfigured16:04
dansmithhmm16:04
dansmithI get 100% fail when I run devstack from queens locally, because it has qemu-utils from base instead of cloud-archive16:04
dansmithmaybe something I'm installing beforehand is pulling it in and it's not getting updated16:05
pabelangerclarkb: we need access to limestone from bridge.o.o, if we want to create a mirror16:05
* dansmith keeps digging16:05
pabelangerbut yah, it also create /etc/openstack folders16:05
*** srp has joined #openstack-infra16:05
corvusfungi, clarkb: testinfra uses hacking16:06
fungium, wow16:06
fungias in openstack hacking?16:06
corvusayup16:06
fungifunny to see how tools like that have propagated (in sometimes questionable ways)16:07
clarkbwoah16:08
clarkbpabelanger: aha that is the missing link bridge doesnt puppet16:08
*** zxiiro-away is now known as zxiiro16:09
*** dpawlik has joined #openstack-infra16:09
*** srp has quit IRC16:09
*** shabeeb has quit IRC16:09
*** jpich has quit IRC16:10
*** shabeeb has joined #openstack-infra16:11
*** shachar has joined #openstack-infra16:12
openstackgerritSorin Sbarnea proposed openstack-infra/zuul master: Display correct cursor when hovering over patchset header  https://review.openstack.org/59822216:12
*** alexchadin has quit IRC16:13
*** dpawlik has quit IRC16:13
*** ykarel has joined #openstack-infra16:14
*** snapiri has quit IRC16:14
openstackgerritMerged openstack-infra/project-config master: switch release job for networking-bgpvpn  https://review.openstack.org/59821316:17
*** ykarel has quit IRC16:17
*** ccamacho has quit IRC16:19
*** rlandy is now known as rlandy|afk16:19
*** anteaya has quit IRC16:23
*** ykarel has joined #openstack-infra16:25
*** anteaya has joined #openstack-infra16:25
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add system-config-run-eavesdrop  https://review.openstack.org/59761716:27
*** signed8bit is now known as signed8bit_Zzz16:27
*** eharney has quit IRC16:29
*** ykarel_ has joined #openstack-infra16:29
*** noama has quit IRC16:30
*** signed8bit_Zzz is now known as signed8bit16:31
*** jamesmcarthur has joined #openstack-infra16:31
*** ykarel has quit IRC16:31
*** sambetts is now known as sambetts|afk16:31
openstackgerritJames E. Blair proposed openstack-infra/system-config master: run-base: collect ansible configuration  https://review.openstack.org/59822616:32
corvusclarkb: we ready for https://review.openstack.org/596913 ?16:33
clarkbcorvus: yes I think so, it worked out great manually applying it to the sjc1 mirror16:34
*** jbadiapa has quit IRC16:35
*** bnemec has quit IRC16:35
corvusso 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
openstackgerritThierry Carrez proposed openstack/ptgbot master: Identify with SASL  https://review.openstack.org/59815516:36
clarkbcorvus: huh, do we log the written inventory?16:40
*** agopi has quit IRC16:40
corvusclarkb: i add that in https://review.openstack.org/59822616:40
*** rmart04 has quit IRC16:41
clarkbcorvus: cool, I'll hvae to review in a bit. Going to double check on power outage fallout now. zfs scrub here I come16:42
*** agopi has joined #openstack-infra16:42
*** r-daneel_ has joined #openstack-infra16:47
*** r-daneel has quit IRC16:47
*** r-daneel_ is now known as r-daneel16:47
*** gouthamr has joined #openstack-infra16:49
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Provide per-project ssh key to executor  https://review.openstack.org/59772716:51
*** ykarel_ has quit IRC16:53
corvusoh for crying out loud.  iptables in trusty doesn't support "-w"16:53
*** ykarel has joined #openstack-infra16:55
*** ykarel has quit IRC16:55
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove release job variants for networking-bgpvpn  https://review.openstack.org/59823216:56
*** shardy has quit IRC16:56
openstackgerritMerged openstack-infra/system-config master: Install python2.7 on xenial nodes  https://review.openstack.org/59691316:57
*** ccamacho has joined #openstack-infra16:58
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add system-config-run-eavesdrop  https://review.openstack.org/59761717:02
corvusi'm just going to turn of parallel execution for now17:02
*** anteaya has quit IRC17:03
*** dtantsur is now known as dtantsur|afk17:04
fungilooks like release activity has settled down, so i'm going to go find lunch17:06
*** diablo_rojo has joined #openstack-infra17:06
*** jamesmcarthur has quit IRC17:09
*** jamesmcarthur has joined #openstack-infra17:10
*** mriedem is now known as mriedem_afk17:11
*** aojea has quit IRC17:11
*** agopi has quit IRC17:11
*** trown|lunch is now known as trown17:13
*** abelur has joined #openstack-infra17:16
*** bnemec has joined #openstack-infra17:17
*** shabeeb has quit IRC17:21
*** ccamacho has quit IRC17:24
*** jamesmcarthur has quit IRC17:24
openstackgerritJames E. Blair proposed openstack-infra/system-config master: run-base: collect ansible configuration  https://review.openstack.org/59822617:25
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add system-config-run-eavesdrop  https://review.openstack.org/59761717:25
corvusokay i think that's going to do it17:25
*** eharney has joined #openstack-infra17:25
corvus(everything passed the last round; that just drops an rsync trailing slash)17:26
*** jpena is now known as jpena|away17:27
*** Emine has joined #openstack-infra17:28
*** hamzy has joined #openstack-infra17:28
*** anteaya has joined #openstack-infra17:29
*** jamesmcarthur has joined #openstack-infra17:31
clarkband I'm back to computer now. Will review17:32
EmilienMhi 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
EmilienMe.g. http://mirror.gra1.ovh.openstack.org/centos/17:33
pabelangerEmilienM: that isn't a mirror, but reverse proxy cache17:34
clarkbcorrect that won't show up in the index17:35
*** holser_ has quit IRC17:35
pabelangerhttp://mirror.gra1.ovh.openstack.org:8080/cbs.centos/17:35
pabelangeris right url17:35
clarkbit 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
clarkbpabelanger: jinx17:35
EmilienMoh gotcha :)17:35
EmilienMthank you clarkb / pabelanger17:35
*** shabeeb has joined #openstack-infra17:43
*** jamesdenton has quit IRC17:44
*** dmellado has joined #openstack-infra17:44
openstackgerritEmilien Macchi proposed openstack-infra/openstack-zuul-jobs master: mirror-info: add variables for CBS proxy  https://review.openstack.org/59825017:45
*** anteaya has quit IRC17:45
openstackgerritEmilien Macchi proposed openstack-infra/openstack-zuul-jobs master: mirror-info: add variable for CBS proxy  https://review.openstack.org/59825017:45
clarkbcorvus: left a suggestion on https://review.openstack.org/#/c/597617/12 which should help with further debugging17:46
*** eharney has quit IRC17:46
*** shabeeb has quit IRC17:47
corvusclarkb: i was just implementing that17:47
*** stevebaker has joined #openstack-infra17:48
*** agopi has joined #openstack-infra17:48
*** shabeeb has joined #openstack-infra17:49
*** electrofelix has quit IRC17:50
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Refactor run-base jobs  https://review.openstack.org/59825217:50
corvusclarkb: as a followup ^17:50
corvushappy to do another ps though if you want17:50
*** jamesmcarthur has quit IRC17:51
AJaegerconfig-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
clarkbcorvus: followup change works, reviewing thatnow then will apply votes to stack based on its entire status17:51
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for neutron repositories  https://review.openstack.org/59786517:55
*** r-daneel has quit IRC17:55
clarkbcorvus: stack lgtm another obversation for a cleanup on https://review.openstack.org/#/c/598226/2 that can go in a followup too I Think17:55
*** gyee has quit IRC17:56
*** eharney has joined #openstack-infra17:56
*** zxiiro is now known as zxiiro-away17:57
openstackgerritMerged openstack-infra/system-config master: Generate junit.xml file for testinfra  https://review.openstack.org/59718517:58
*** jamesdenton has joined #openstack-infra17:59
*** r-daneel has joined #openstack-infra17:59
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for neutron repositories  https://review.openstack.org/59786517:59
*** hashar is now known as hasharDinner18:00
Shrewscorvus: 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
clarkbShrews: ya that was my comment too18:02
clarkbI think for now its a single file (or two if you count the .py) that we probably don't want18:03
Shrewsoh, oops18:03
clarkbso not urgent to change and can do i na followup18:03
corvusso maybe "--exclude=__pycache__" ?18:05
clarkbya18:05
openstackgerritMerged openstack-infra/project-config master: remove job settings for rally repositories  https://review.openstack.org/59751718:05
openstackgerritJames E. Blair proposed openstack-infra/system-config master: run-base: exclude __pycache__ from logs  https://review.openstack.org/59825618:06
EmilienMsimple patch to review (for CBS proxy): https://review.openstack.org/598250 - thanks!18:07
openstackgerritMerged openstack-infra/project-config master: remove job settings for cyborg repositories  https://review.openstack.org/59284918:08
*** r-daneel_ has joined #openstack-infra18:08
openstackgerritMerged openstack-infra/project-config master: Remove jobs settings for monasca-notification  https://review.openstack.org/59806118:09
*** r-daneel has quit IRC18:09
*** r-daneel_ is now known as r-daneel18:09
*** rosmaita has quit IRC18:24
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for neutron repositories  https://review.openstack.org/59786518:24
*** bobh has joined #openstack-infra18:27
openstackgerritMerged openstack-infra/system-config master: run-base: collect ansible configuration  https://review.openstack.org/59822618:29
*** jamesmcarthur has joined #openstack-infra18:30
clarkbwoo scrub came back clean, I can go worry about other things now18:30
*** signed8bit is now known as signed8bit_Zzz18:31
*** shabeeb has quit IRC18:33
*** Emine has quit IRC18:35
openstackgerritMerged openstack-infra/system-config master: Add system-config-run-eavesdrop  https://review.openstack.org/59761718:38
*** signed8bit_Zzz is now known as signed8bit18:38
*** jamesmcarthur has quit IRC18:40
clarkbpabelanger: https://review.openstack.org/#/c/593029/20/playbooks/configure-openstack.yaml should that file be removed?18:41
pabelangerclarkb: yah, we can delete that18:42
pabelanger1 sec18:42
clarkbpabelanger: 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.yaml18:43
*** jamesmcarthur has joined #openstack-infra18:43
clarkbmaybe just make that a non default option that we set on bridge and it writes the extra file18:44
pabelangerclarkb: yah, was thinking we'd call the role twice, just with difference clouds.yaml values and locations18:44
clarkbmakes 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
clarkbso we need to specify thattemplate either way I think18:45
openstackgerritPaul Belanger proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk  https://review.openstack.org/59302918:46
clarkband we should be able to just take the existing erb file and make it a jinja2 template and the existing lookups to hiera should work18:47
*** Emine has joined #openstack-infra18:47
clarkbok this is starting to make sense. Hopeflly have time to take a stab at it later today18:47
*** jamesmcarthur has quit IRC18:48
pabelangerclarkb: 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 hieradata18:48
openstackgerritMerged openstack-infra/system-config master: Refactor run-base jobs  https://review.openstack.org/59825218:48
pabelangeransible will error if it tries to reference a non existing variable, unless we use default(foo)18:48
*** imacdonn has joined #openstack-infra18:49
pabelangeror moved the whole yaml structure for clouds.yaml into a dict in hiera18:49
clarkbwe'd have to serialize that back to yaml though right? does ansible make that easy? that might be a good followup18:50
pabelangeryah, there are yaml filters we could use18:50
clarkbpabelanger: also did you see my change for cloud launcher? https://review.openstack.org/#/c/597728/18:51
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for Release Management repositories  https://review.openstack.org/59831418:51
*** toabctl has quit IRC18:51
clarkbone thing I like about having the files in their resulting format is its easier to use them in othre contexts18:51
clarkblike showing someone an example working clouds.yaml for $cloud18:51
clarkbsimilar with apache config files18:51
clarkbyou lose a lot of that readability and reusability when you store them in $configmgmt_tool intermediate format18:51
clarkbnot to mention the actual act of going from one management system to another as we are doing18:52
*** gfidente has quit IRC18:54
pabelangeryah, 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 work18:55
pabelangerthen we can template things out properly in test environment, same variables, just different data18:56
openstackgerritMerged openstack-infra/system-config master: run-base: exclude __pycache__ from logs  https://review.openstack.org/59825618:57
clarkbif 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-infra18:59
imacdonnhi guys .. anything going on that would cause this (neutron-grenade for a backport to glance stable/queens):19:00
imacdonn2018-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
imacdonn2018-08-30 18:41:27.320663 | primary |     from ansible.config.manager import find_ini_config_file19:00
imacdonn2018-08-30 18:41:27.320704 | primary | ImportError: No module named manager19:00
clarkbara just made a release19:00
pabelangerclarkb: yah, relative to playbook IIRC19:00
clarkbdmsimard: ^ see imacdonn's comment19:01
clarkbpabelanger: thanks19:01
dmsimardimacdonn: have a link to the job ?19:01
dmsimardclarkb: thanks, I'll look19:01
imacdonndmsimard: http://logs.openstack.org/00/598300/1/check/neutron-grenade/e303134/19:02
*** jamesmcarthur has quit IRC19:02
dmsimardthat looks like devstack-gate19:02
dmsimardhttp://logs.openstack.org/00/598300/1/check/neutron-grenade/e303134/ara-report/result/d6b7948c-e626-4d7a-9503-5e1d90090171/19:02
*** jamesmcarthur has joined #openstack-infra19:02
clarkbdmsimard: it is devstack-gate running ara I think?19:03
dmsimardclarkb: 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 IRC19:04
dmsimardbut ara also has ansible>=2.4 in it's req.txt19:04
*** ijw has joined #openstack-infra19:04
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: switch all official python projects to python3 publishing job  https://review.openstack.org/59832319:04
pabelangerdmsimard: clarkb: https://review.openstack.org/591527/19:04
pabelangerdevstack-gate is still using 2.319:04
openstackgerritDavid Moreau Simard proposed openstack-infra/devstack-gate master: Bump ansible to latest 2.5 version  https://review.openstack.org/59152719:05
dmsimardpabelanger: yup, it was pinned to 2.3, thanks19:05
*** rlandy|afk is now known as rlandy19:05
dmsimardI've been out of the loop for a while, devstack-gate is not dead yet? :(19:05
pabelangerno, a lot of legacy jobs still using it19:06
clarkbwe can probably bump that ansible version19:06
corvusdmsimard: what was the new patchset?19:06
clarkbits pinned because ansible releases are not backward compat19:06
dmsimardclarkb: pabelanger's patchset already does that19:06
clarkbah19:06
dmsimardcorvus: I don't understand the question19:06
corvusdmsimard: you just pushed a new patchset of 527.  why?19:06
dmsimardcorvus: it's a rebase to ensure that it really does fix the problem19:07
*** toabctl has joined #openstack-infra19:07
*** SteffanW has joined #openstack-infra19:08
pabelangerI have to run and do errands now, will loop back on clouds.yaml later this evening19:09
dmsimardSorry about that folks, I tested TripleO, OSA, Kolla-Ansible and Zuul.. I'll add devstack-gate, it's a gap :(19:09
corvusdmsimard: 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
dmsimardcorvus: true, I erred on the side of safety19:11
*** mriedem has joined #openstack-infra19:11
*** mriedem_afk has quit IRC19:12
corvusdmsimard: 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
corvusclarkb: i'm +2 on 591527 if you want to +319:13
dmsimardcorvus: totally agree19:13
*** ansmith has quit IRC19:15
corvusopenstack 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 IRC19:17
openstackgerritClark Boylan proposed openstack-infra/system-config master: WIP: Manage clouds.yaml files in ansible  https://review.openstack.org/59832919:18
clarkbpabelanger: ^ 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 chnage19:19
*** olivier__ has joined #openstack-infra19:19
clarkbcorvus: done19:20
*** olivierb_ has quit IRC19:20
*** SteffanW has quit IRC19:21
imacdonnthanks 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
clarkbimacdonn: devtack-gate is "branchless" the master branch applies to all runs19:26
clarkbso it will take effect once merged19:26
imacdonncool, thanks19:26
*** jamesmcarthur has joined #openstack-infra19:29
fungiokay, i'm back catching up and seeing how many people i've managed to annoy19:30
*** dpawlik has joined #openstack-infra19:34
*** e0ne has joined #openstack-infra19:34
*** cfriesen has joined #openstack-infra19:34
cfriesenhas 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
fungii've definitely not seen that19:36
fungithen again i've never seen a browser that had a feature to auto-translate content without user intervention asking for it19:37
cfriesenI 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
fungialso, that reminds me, i need to restart the etherpad-lite service in production now that the change merged to upgrade it to the latest release19:38
*** jpena|away is now known as jpena|off19:38
*** cfriesen has left #openstack-infra19:40
*** ijw has quit IRC19:42
smcginnisAny 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
clarkbsmcginnis: yes that is part of the motivation19:45
fungiyou can test it on etherpad-dev.o.o already, but it seems to be fixed19:45
fungias soon as the restart happens on etherpad.o.o we'll see it fixed there too, i believe19:45
fungii wanted to make sure we had it in production a week or more in advance of the ptg19:46
smcginnisGood plan.19:49
*** olivier__ has quit IRC19:54
*** hamzy has quit IRC19:54
*** r-daneel has joined #openstack-infra19:55
*** hamzy has joined #openstack-infra19:55
*** hasharDinner is now known as hashar19:56
fungii'll try to make sure to restart it later in my evening once our usual community activity levels die down19:56
*** bobh has quit IRC20:06
*** bobh has joined #openstack-infra20:08
*** otherwiseguy has joined #openstack-infra20:10
corvusclarkb: can you help clean up my oops?  https://review.openstack.org/59724220:13
notmynameare 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 advise20:15
clarkbcorvus: done20:17
corvusnotmyname: 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
clarkbcorvus: 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 approach20:18
*** kgiusti has left #openstack-infra20:18
notmynamecorvus: 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
corvuspabelanger: i don't think https://review.openstack.org/597241 is saving a lot of time, but i like it anyway.  <-- clarkb20:18
corvusclarkb: will do20:18
corvusnotmyname: 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
dhellmannnotmyname : I tried to explain why in https://governance.openstack.org/tc/goals/stein/python3-first.html#moving-settings-from-project-config20:20
clarkbcorvus: 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
corvusclarkb: theoretically run-base20:22
corvusclarkb: though if you think there's a problem, it's probably worth working through whether any failures there would actually be detected...20:23
clarkbmostly 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 down20:23
corvusclarkb: 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 IRC20:24
clarkbya it would likely help other users debugging nodepool to not have to su to that user20:24
*** mugsie has quit IRC20:25
clarkbok I think I understand ansible package node, its a meta module that becomes apt or yum etc, and apt at least accepts a list20:25
clarkbinterestingly the yum docs show both methods of installing a list of pakcages20:27
clarkbcorvus: ok the install puppet role uses this too and we definitely check that the puppet package sstuff is installed20:28
corvusclarkb: approach in 598329 lgtm20:28
clarkbwhich is good enough for me so approved20:28
*** e0ne has quit IRC20:29
notmynamedhellmann: 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
dhellmannnotmyname : yes, that is the plan20:31
dhellmannif you'd like I will review the patches you have, but Nguyen has been doing well with the other projects so far.20:31
fungialso the "extended maintenance" plan where we agreed to just stop deleting branches and possibly let them lie fallow instead plays into this a bit20:33
fungiharder to know when a branch has truly "expired" at this point20:33
notmynamefungi: did the extended maintenance discussion get to a point of a formal plan? I remember the conversations, but not a resolution20:36
notmynamedhellmann: 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 histor20:36
notmynameical patches like this), and I need to understand the nuances so that it can be maintained.20:36
dhellmannnotmyname : 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-infra20:37
dhellmann there's a list of people I expect to be proposing these patches in that doc, for just this reason20:37
openstackgerritMerged openstack-infra/system-config master: unbound: fix dhclient.conf copy  https://review.openstack.org/59724220:38
notmynameyeah, 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 implied20:39
funginotmyname: 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 change20:39
notmyname...especially where the implied things differ from standard or historic practice20:40
notmynametangental 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 projects20:41
pabelangerclarkb: yah, that's kinda what I was thinking for clouds.yaml20:42
dhellmannnotmyname : older ones? where is it saying that?20:42
clarkbpabelanger: corvus thanks I'll work on sorting out what the hieradata to group_vars mapping is here20:42
dhellmannfungi : I keep failing to convince tonyb to just leave all the old branches alone when they aren't maintained.20:42
notmynamedhellmann: "this release is under development. current supported release is rocky" <-- I paraphrased :-)20:43
dhellmannah20:43
dhellmannthe paraphrase is definitely inaccurate; is the actual statement? :-)20:43
notmynametoday, that's somewhat misleading. yesterday it was wrong (when it said queens)20:43
corvusdhellmann, fungi: i don't think branches should exist if patches aren't accepted20:43
*** harlowja has joined #openstack-infra20:43
notmynameand as soon as a -with-intermediary project does a release, the statement becomes wrong again20:43
dhellmannnotmyname: ok, I guess that's fair. the docs team manages the theme20:44
fungicorvus: i agree with you20:44
dhellmanncorvus : 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
fungithe 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
fungiwe know from experience that hasn't been a realistic strategy in th epast though20:45
toskyfungi: oh, thanks (wiki)20:45
dhellmannI'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
dhellmannI guess I don't consider an open branch an invitation to propose patches.20:45
fungitosky: i think i'm the only person still actively patrolling wiki edits and adding users to the verified group when they lame leditimate edits20:45
fungier, legitimate20:46
dhellmannI haven't observed that behavior in the past myself, but maybe it does happen.20:46
fungis/lame/make/20:46
*** bobh has quit IRC20:46
corvusdhellmann: believe it is an invitation to do that -- that's part of why we started doing that.20:46
fungifreudian typo maybe20:46
*** rmart04 has joined #openstack-infra20:46
dhellmanncorvus : Ok20:46
corvusi 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-infra20:47
fungiyes, we can post all the notices we like in documentation and readmes but people proposing patches to open branches tend not to see them20:47
dhellmannI 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
corvusif 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
corvusdhellmann: i think we should only delete them after people have stopped working on them :)20:47
fungithe next gerrit release is supposed to make it finally possible to control the branch deletion api call with an explicit acl permission20:48
dhellmannwell, at least we agree on that20:48
fungi(next after the one we're running, i mean... 2.14)20:48
dhellmannthat should make it possible to automate closing the branches20:48
corvussounds like a good time to revisit20:48
dhellmannyep20:48
fungithe 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
fungier, contrived20:50
openstackgerritMerged openstack-infra/system-config master: Don't use loops with package task  https://review.openstack.org/59724120:53
*** hamzy has quit IRC20:59
openstackgerritSorin Sbarnea proposed openstack-infra/devstack-gate master: Bump default Ansible version from 2.3.2 to 2.5.8  https://review.openstack.org/59835121:03
*** trown is now known as trown|outtypewww21:03
tonybdhellmann: tagging the branches as EOL was a compromise to get mor peopel on board with the general plan21:06
openstackgerritSorin Sbarnea proposed openstack-infra/zuul master: Display correct cursor when hovering over patchset header  https://review.openstack.org/59822221:13
*** rmart04 has quit IRC21:16
*** dpawlik has quit IRC21:17
*** eharney has joined #openstack-infra21:17
*** holser_ has joined #openstack-infra21:19
*** jcoufal has quit IRC21:20
openstackgerritClark Boylan proposed openstack-infra/system-config master: WIP: Manage clouds.yaml files in ansible  https://review.openstack.org/59832921:21
clarkbcorvus: 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 today21:22
corvusclarkb: we can put fake secret values in playbooks/zuul/templates/...21:25
corvuswe do that with all.yaml to add the root private key for ansible21:26
*** dmellado has quit IRC21:31
*** dmellado has joined #openstack-infra21:31
dansmithcan someone tell me if this is my fault?21:34
dansmithhttp://logs.openstack.org/53/598353/1/check/neutron-grenade/14e0da6/21:34
dansmithno logs dir,21:34
dansmithvery little of use in the output.txt and not sure if that's an infra fail or what21:34
pabelangerwe 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 today21:34
dansmithoh, looks like there are other jobs hitting that right now21:35
pabelangerdansmith: 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_90411421:35
dansmithlike all grenade-y jobs, nova-next, etc21:35
pabelangerpatch up to bump ansible 2.5 in devstack-gate21:35
dansmithpabelanger: ack okay hadn't seen it yet, sorry21:35
pabelangerhttps://review.openstack.org/591527/ should be the fix21:36
fungii guess the grenade jobs are still legacy-based21:36
*** stevebaker has quit IRC21:36
corvuspabelanger: that's exactly what the run-base job does21:36
dansmithfungi: and all our tree-defined ones I think?21:36
corvuspabelanger: that's what i suggested to clarkb21:36
pabelangercorvus: ah, great. I've missed that21:36
*** holser_ has quit IRC21:37
corvuspabelanger, clarkb: these tasks: http://git.openstack.org/cgit/openstack-infra/system-config/tree/playbooks/zuul/run-base.yaml#n2621:37
corvuswe 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
pabelangeryah, I think that is a great idea21:38
*** agopi has quit IRC21:40
clarkbcorvus: thanks I will take a look at doing that21:41
*** stevebaker has joined #openstack-infra21:41
*** holser_ has joined #openstack-infra21:42
*** eernst has quit IRC21:42
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for SIGs repositories  https://review.openstack.org/59836021:46
*** cdent has joined #openstack-infra21:46
*** dkehn is now known as dkehn_21:46
*** priteau has quit IRC21:47
*** rcernin has joined #openstack-infra21:49
corvusclarkb, 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 deployment21:51
corvusi'll tag that update-cfg-mgmt21:51
fungilooking! (sorry, i've been sucked into other black holes all day/week)21:52
corvusnp, i figured you figured i'd let you know when it was worth looking at :)21:52
fungiooh, i remember this one!21:53
*** diablo_rojo has quit IRC21:53
fungi"it's a unix system, i know this!"21:53
fungisomething like that ;)21:53
corvusfungi: several "1"s have been changed to "0"s.  :)21:54
*** florianf has quit IRC21:54
fungiseems like that's always the case21:54
*** dpawlik has joined #openstack-infra21:54
openstackgerritClark Boylan proposed openstack-infra/system-config master: WIP: Manage clouds.yaml files in ansible  https://review.openstack.org/59832921:55
corvusso tell me, are any *other* jurrasic park movies any good?  or is it like the matrix where they only made one?21:55
clarkbcorvus: its mostly where the only made one21:55
clarkbthe other films are decent action movies but nothing on par with the original imo21:55
*** boden has quit IRC21:55
clarkbalso we now have testing, I just have to figure out how I want to handle puppet managing nodepool clouds.yaml now21:56
clarkbI guess we just make that optional in the puppet21:56
fungii don't think i ever saw any of the jp sequels either21:58
*** dpawlik has quit IRC21:59
fungithough where the matrix was concerned, i did actually rather enjoy the animatrix shorts21:59
fungiunlike the sequels21:59
fungithough animatrix was mostly prequel material (in addition to being highly stylized)21:59
clarkbthe book is really good if you haven't read it22:01
SotKI've not seen the modern jp films, but my experience of the first 3 is that it is like the matrix22:01
fungiwhich book? if you mean jp, i found it to be so-so22:01
openstackgerritChris Dent proposed openstack-infra/project-config master: Create an openstack/placement project  https://review.openstack.org/59836222:02
cdentfungi, clarkb etc I followed along in https://docs.openstack.org/infra/manual/creators.html to create ^ but I'm not filled with confidence22:02
fungithen again, i find most of crichton's work that i've tried to read to be poorly-researched22:02
cdentIt's at once "oh yeah that's simple" and fraught22:03
fungicdent: our ci jobs and reviewers will fill you with any confidence you may be lacking22:03
cdent22:03
* corvus pours another shot of confidence22:03
clarkbfungi: ya jp, you have to treat it as half fantasy, it is entertaining but not kim stanley robinson levels of sifi22:03
dmsimardcorvus: zuul triggered deployment ?22:04
fungiclarkb: yeah, i read it. it was fun enough that i didn't regret reading it. but it didn't exactly blow me away22:04
corvusdmsimard: 3rd pgraph in http://specs.openstack.org/openstack-infra/infra-specs/specs/update-config-management.html#ansible22:04
clarkbfungi: have you read red mars? if not you'd probably like it if you want well researched scifi22:05
cdentred mars++22:05
funginot yet, but i've been meaning to22:05
fungigood to know22:05
* corvus adds to list22:05
fungii like a lot of sci with just enough fi to keep me on my toes22:05
dmsimardcorvus: 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
cdentwhat is the gold-standard prefered form of Depends-On these days?22:07
corvuscdent: url22:08
cdentcorvus: thanks22:08
cdentaw bummer, I broked it22:09
slaweqhi22:10
slaweqare 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
corvuscdent: 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_79719922:10
slaweqI see it now in many neutron related jobs22:11
cdentcorvus: yeah, already on it22:11
*** ijw has joined #openstack-infra22:11
corvusslaweq: https://review.openstack.org/591527 should fix22:11
dmsimardslaweq: it'll be fixed once https://review.openstack.org/591527 lands22:11
dmsimardMy fault!22:11
slaweqthx corvus and dmsimard for info22:11
*** jamesmcarthur has quit IRC22:13
*** jamesmcarthur has joined #openstack-infra22:14
*** ijw has quit IRC22:16
*** hashar has quit IRC22:16
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for InteropWG repositories  https://review.openstack.org/59837722:16
*** mriedem is now known as mriedem_lawnboy22:17
*** gema has quit IRC22:19
openstackgerritClark Boylan proposed openstack-infra/puppet-openstackci master: Make oscc_file_contents optional  https://review.openstack.org/59837822:20
openstackgerritClark Boylan proposed openstack-infra/system-config master: Manage clouds.yaml files in ansible  https://review.openstack.org/59832922:21
clarkbcorvus: pabelanger ^ I think that does it22:21
*** tosky has quit IRC22:23
*** anteaya has joined #openstack-infra22:28
*** holser_ has quit IRC22:30
openstackgerritDean Troyer proposed openstack-infra/project-config master: Add stx-distcloud and stx-distcloud-client repos  https://review.openstack.org/59838322:34
*** sshnaidm is now known as sshnaidm|off22:35
corvusclarkb: i'm really interested in the openstacksdf.  i'm assuming that's the self-defence force.22:35
clarkbcorvus: that was typo in mordreds change that I decided to fix :)22:36
fungispace defense force22:36
openstackgerritDean Troyer proposed openstack-infra/project-config master: Add Zuul jobs for new StarlingX repos  https://review.openstack.org/59838622:36
openstackgerritDean Troyer proposed openstack-infra/project-config master: Add stx-distcloud and stx-distcloud-client repos  https://review.openstack.org/59838322:37
*** dbecker has quit IRC22:37
clarkbif 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 followup22:41
clarkbthough 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 right22:42
openstackgerritDean Troyer proposed openstack-infra/project-config master: Add stx-distcloud and stx-distcloud-client repos  https://review.openstack.org/59838322:43
openstackgerritChris Dent proposed openstack-infra/project-config master: Create an openstack/placement project  https://review.openstack.org/59836222:46
*** anteaya has quit IRC22:47
openstackgerritMerged openstack-infra/zuul master: Add missing parentheses to title invocation  https://review.openstack.org/59536122:48
corvusclarkb: er, switched to -1 on https://review.openstack.org/59832922:48
clarkbcorvus: 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
corvusclarkb: (though, also, i think maybe we should refactor those out into a new host group)22:49
corvusclarkb: completeness -- i was about to build a nodepool job22:50
clarkbcool22:50
corvusclarkb: would you mind rebasing that stack?  that'll make the nodepool job easier to build22:50
clarkbcorvus: rebase to master?22:51
corvusyep22:51
clarkbsure22:51
openstackgerritClark Boylan proposed openstack-infra/system-config master: Install limestone CA on hosts using openstacksdk  https://review.openstack.org/59302922:51
openstackgerritClark Boylan proposed openstack-infra/system-config master: Manage clouds.yaml files in ansible  https://review.openstack.org/59832922:51
clarkbrebased, now to address the review comment22:51
cdentcorvus: 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
fungicdent: lately could be up to an hour22:54
cdentfungi: thanks. that's suitably and nicely fast22:54
fungiwe're in the midst of a transition to a new way of rolling out our continuous deployment and are still fine-tuning it for performance22:54
*** anteaya has joined #openstack-infra22:54
openstackgerritClark Boylan proposed openstack-infra/system-config master: Manage clouds.yaml files in ansible  https://review.openstack.org/59832922:55
corvuswe're aiming for minutes :)22:55
clarkbcorvus: ^ comment addressed22:56
cdentI 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 IRC22:56
cdentit's happy in ci now, pointing out the review to nova folk22:58
clarkbcdent: did you delete the openwrt image in nova as part of the filter branching? it clones so quickly :)22:58
cdentha22:58
cdented did some before and after measurements: the repo goes from abourt 511M to 11322:59
*** eharney has quit IRC22:59
corvusoh good.  that's much appreciated.23:00
clarkbthe change lgtm mechanically23:00
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for User Committee repositories  https://review.openstack.org/59840023:00
*** jamesmcarthur has joined #openstack-infra23:00
clarkband now I've got to step out for a bit23:01
*** tpsilva has quit IRC23:02
*** jamesmcarthur has quit IRC23:03
*** jamesmcarthur has joined #openstack-infra23:03
*** jamesmcarthur has quit IRC23:06
*** dbecker has joined #openstack-infra23:06
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add a run-nodepool job  https://review.openstack.org/59840323:07
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add testinfra tests for bridge  https://review.openstack.org/59840423:07
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add a run-nodepool job  https://review.openstack.org/59840323:10
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add testinfra tests for bridge  https://review.openstack.org/59840423:10
*** ijw has joined #openstack-infra23:12
*** agopi has joined #openstack-infra23:12
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Zuul: increase max-nodes-per-job  https://review.openstack.org/59840623:12
ianwcorvus: https://review.openstack.org/#/c/595920/ -- should i update that change about variable naming guidelines to say "always prefix"?23:12
corvusianw: seems like it might be a good idea, yeah23:13
ianwcorvus: 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 linters23:13
ianwcorvus: 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 other23:15
ianwhttps://review.openstack.org/#/c/597374/ is a pretty trivial typo fix too for kerberos role variables23:16
*** ijw has quit IRC23:17
*** cdent has quit IRC23:22
*** adriant has quit IRC23:23
*** adriant has joined #openstack-infra23:23
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for User Committee repositories  https://review.openstack.org/59840023:26
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: Add guidelines on variable naming  https://review.openstack.org/59592023:28
clarkbianw: 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
ianwclarkb: 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 ATM23:31
ianwat 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 to23:32
clarkbok I didn't approve it since you specifically asked for corvus' input but lgtm23:32
ianwthanks, since it needs a release to be effective probably covus' bailiwick23:36
*** r-daneel has quit IRC23:36
* ianw is unsure if bailiwick is a Queen's english thing that will confuse everyone else23:37
clarkbI'm not familiar with it :)23:37
*** anteaya has quit IRC23:44
*** signed8b_ has joined #openstack-infra23:44
*** signed8bit has quit IRC23:47
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Refactor Project Group editing to match other objects  https://review.openstack.org/59840723:51
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Support linking to Project Groups by name  https://review.openstack.org/59840823:51
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Link to Project Groups by name  https://review.openstack.org/59840923:51
*** dingyichen has joined #openstack-infra23:52
*** rcernin has quit IRC23:56
*** rcernin has joined #openstack-infra23:56
openstackgerritClark Boylan proposed openstack-infra/system-config master: Manage clouds.yaml files in ansible  https://review.openstack.org/59832923:58
clarkbcorvus: ^ minor file name derp I had to fix23:58
clarkbcorvus: should I rebase your changes?23:58

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