openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add an authoritative hidden master https://review.openstack.org/528438 | 00:00 |
---|---|---|
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 00:00 |
*** jcoufal has joined #openstack-infra | 00:00 | |
jeblair | clarkb, fungi, dmsimard: ^ those changes are manually-puppet-applied on adns1.openstack.org and ns1.openstack.org. values are in hiera. | 00:01 |
jeblair | i'm sure the whitespace is wrong, but i think the functionality is there | 00:01 |
clarkb | jeblair: can I dig zuul-ci.org against ns1.openstack.org yet? | 00:01 |
clarkb | dig worked but zone isn't there yet | 00:02 |
clarkb | oh wait | 00:02 |
clarkb | I asked for an A record | 00:02 |
clarkb | nice it totally owrk | 00:02 |
jeblair | clarkb: yes. lists is a cname in there | 00:02 |
jeblair | should be able to get soa or lists | 00:02 |
clarkb | yup works | 00:03 |
clarkb | (I asked for NS) | 00:03 |
*** xarses_ has joined #openstack-infra | 00:04 | |
*** d0ugal has joined #openstack-infra | 00:04 | |
*** jcoufal has quit IRC | 00:05 | |
*** andreww has joined #openstack-infra | 00:07 | |
*** andreww has quit IRC | 00:07 | |
jeblair | clarkb: dig +dnssec zuul-ci.org @ns1.openstack.org should also work | 00:07 |
*** xarses_ has quit IRC | 00:08 | |
*** rcernin has quit IRC | 00:09 | |
*** rcernin_ has joined #openstack-infra | 00:09 | |
*** andreww has joined #openstack-infra | 00:09 | |
clarkb | jeblair: in your docs it says run dnssec-keygen -fx, reading the manpage -f takes KSK or REVOKE? | 00:09 |
jeblair | typo, should be -fk | 00:10 |
jeblair | will fix | 00:10 |
clarkb | is that short for ksk? | 00:11 |
jeblair | i'm guessing so. it's what all the tutorials say to do to generate a ksk | 00:11 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add an authoritative hidden master https://review.openstack.org/528438 | 00:12 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 00:12 |
jeblair | clarkb: you can examine the output on adns1 or in hiera. the generated key does say that it is a ksk | 00:12 |
*** ijw has joined #openstack-infra | 00:13 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 00:14 |
clarkb | jeblair: left one more comment on the old patchset | 00:17 |
fungi | okay, back and catching up | 00:17 |
*** ijw has quit IRC | 00:18 | |
jeblair | clarkb: ah yep | 00:18 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add an authoritative hidden master https://review.openstack.org/528438 | 00:21 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 00:21 |
fungi | jeblair: ahh, yeah, if we authenticate axfr then i don't think we need to bother with ip address identification anyway. i mean it would be nice to have both i guess so that it's harder for someone to try to brute-force your key, but not really critical | 00:22 |
clarkb | jeblair: lgtm | 00:22 |
*** lukebrowning has joined #openstack-infra | 00:24 | |
fungi | reviewing now, then i'll catch up on the zuul meeting log | 00:24 |
jeblair | fungi: we do have iptables rules on adns1, so we have a form of ip protection + tsig. | 00:24 |
clarkb | jeblair: isn't it in the wrong direction though? | 00:25 |
fungi | oh, that's a great point. i kinda trust iptables to do a better job at that anyway | 00:25 |
clarkb | (I mean its probably a good idea to have the rule regardless but I think tcp connection will initiate the other direction won't it?) | 00:25 |
*** ijw has joined #openstack-infra | 00:26 | |
jeblair | clarkb: the axfr requests come from ns* to adns1, and adns1 only allows tcp/53 from ns* | 00:26 |
fungi | clarkb: the silent master doesn't need to allow 53 )tcp or udp) from anybody except the slaves | 00:26 |
fungi | or what jeblair managed to say in far fewer typos | 00:26 |
clarkb | but the notify is the other way? I guess the transfer is what matters ultimately | 00:27 |
fungi | notifies go from adns1 to ns*, and then axfr queries come from ns* to adns1 | 00:27 |
jeblair | the slaves receive notify requests from the master, which are not tsig-signed, since puppet-nsd doesn't expose that option. but nsd is set up to do ip acls on those. | 00:27 |
clarkb | gotcha | 00:27 |
fungi | worst case someone spoofs a bunch of notifies to the slaves and they axfr to the silent master more often | 00:28 |
*** lukebrowning has quit IRC | 00:28 | |
jeblair | (nsd does support requiring tsig on notifies, but for some reason, the puppet module doesn't expose that. that *might* be different in the latest version (requires puppet4). but a lot has changed and i didn't trace it all through. | 00:28 |
jeblair | fungi: ya | 00:28 |
jeblair | i was a bit surprised -- the puppet-nsd module is otherwise amazing | 00:29 |
*** ijw has quit IRC | 00:30 | |
fungi | the also-notify is probably also not strictly necessary since i'm pretty sure bind's default behavior is to notify any nameservers with ns records for the zone being updated | 00:31 |
fungi | but it can't hurt | 00:32 |
jeblair | fungi: yes i thought so too -- but fore some reason i wasn't seeing it happen in my testing. i could try again, though i don't believe i did anything to "fix" that other than add also-notify | 00:32 |
fungi | meh, i wouldn't sweat it then | 00:33 |
fungi | jeblair: i love the implementation here. we get the flexibility of bind's vast featureset coupled with the more limited risk profile of nsd | 00:34 |
*** marst has quit IRC | 00:35 | |
clarkb | jeblair: oh, one question what tells bind to pick up the new dns zones in the first place? doesn't it need a service reload for that (or the more specific managmeent command I can't remember because service reload worked on $distro) | 00:36 |
jeblair | fungi: yeah, i'm digging it! i think it would also be easy to do a bind slave -- and the hidden master approach makes that nice. | 00:36 |
clarkb | oh there is a puppet notify service thing in there | 00:37 |
jeblair | clarkb: yep | 00:37 |
clarkb | so I guess puppet will do a service restart but since bind isn't the public facing ns thats fine | 00:37 |
clarkb | and as long as we update zones via puppet nad not some other system too there won't be a race on those restarts so should be fine | 00:37 |
jeblair | and restarts send out notifies | 00:37 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Xenial: add a service file https://review.openstack.org/528626 | 00:37 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test https://review.openstack.org/528130 | 00:37 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: [WIP] try a later version https://review.openstack.org/528156 | 00:37 |
*** bobh has joined #openstack-infra | 00:39 | |
*** d0ugal has quit IRC | 00:40 | |
fungi | and i'm caught up on the happy holidays edition of the zuul meeting log as well now | 00:43 |
*** xarses has joined #openstack-infra | 00:45 | |
*** pvaneck has quit IRC | 00:50 | |
*** dhill_ has quit IRC | 00:50 | |
*** rcernin_ has quit IRC | 00:50 | |
*** d0ugal has joined #openstack-infra | 00:51 | |
*** jamesmcarthur has joined #openstack-infra | 00:52 | |
*** psachin has joined #openstack-infra | 00:52 | |
*** annp has quit IRC | 00:53 | |
*** annp has joined #openstack-infra | 00:53 | |
*** daidv has quit IRC | 00:53 | |
*** daidv has joined #openstack-infra | 00:54 | |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Xenial: add a service file https://review.openstack.org/528626 | 00:55 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test https://review.openstack.org/528130 | 00:55 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: [WIP] try a later version https://review.openstack.org/528156 | 00:55 |
*** jamesmcarthur has quit IRC | 00:56 | |
*** catintheroof has joined #openstack-infra | 00:56 | |
fungi | oh, interesting... minor hole in gerrit's commit message update detection | 00:57 |
fungi | i have two changes in a series. moving some of the delta from the parent change to the child results in gerrit thinking i only updated the commit message of the child because the diff of the resulting code between patchsets didn't change | 00:58 |
*** edmondsw has joined #openstack-infra | 00:58 | |
clarkb | ha | 00:58 |
fungi | 526140 for an example | 00:58 |
fungi | patchset 4 includes half a dozen lines moved into it from the parent change's tox.ini | 00:59 |
*** gyee has quit IRC | 01:01 | |
*** kiennt26 has joined #openstack-infra | 01:03 | |
*** lukebrowning has joined #openstack-infra | 01:04 | |
*** cuongnv has joined #openstack-infra | 01:05 | |
fungi | doing it again | 01:07 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-base-jobs master: Initial boilerplate, packaging and testing https://review.openstack.org/526139 | 01:07 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-base-jobs master: Add generic base and base-test jobs/playbooks https://review.openstack.org/526140 | 01:07 |
fungi | yep. "Uploaded patch set 5: Commit message was updated." | 01:08 |
fungi | this one even moved some entire files from the parent change to the child | 01:08 |
*** lukebrowning has quit IRC | 01:09 | |
openstackgerrit | Cao Xuan Hoang proposed openstack-infra/project-config master: Add neutron-vpnaas/vpnaas-dashboard to gerritbot https://review.openstack.org/527867 | 01:10 |
*** lukebrowning has joined #openstack-infra | 01:10 | |
clarkb | fungi: does this make you the gerrit upgrade tester? | 01:11 |
*** andreww has quit IRC | 01:11 | |
*** salv-orlando has quit IRC | 01:13 | |
*** salv-orlando has joined #openstack-infra | 01:14 | |
*** bobh has quit IRC | 01:14 | |
*** bobh has joined #openstack-infra | 01:14 | |
*** lukebrowning has quit IRC | 01:15 | |
*** lukebrowning has joined #openstack-infra | 01:17 | |
*** iyamahat has joined #openstack-infra | 01:17 | |
*** salv-orlando has quit IRC | 01:18 | |
*** Swimingly has quit IRC | 01:20 | |
*** Swimingly has joined #openstack-infra | 01:20 | |
*** catintheroof has quit IRC | 01:21 | |
*** lukebrowning has quit IRC | 01:21 | |
*** lukebrowning has joined #openstack-infra | 01:23 | |
*** ijw has joined #openstack-infra | 01:26 | |
*** lukebrowning has quit IRC | 01:28 | |
*** lukebrowning has joined #openstack-infra | 01:29 | |
*** ijw has quit IRC | 01:32 | |
*** Apoorva_ has joined #openstack-infra | 01:32 | |
*** linkmark has quit IRC | 01:33 | |
*** lukebrowning has quit IRC | 01:34 | |
*** lukebrowning has joined #openstack-infra | 01:35 | |
*** Apoorva has quit IRC | 01:36 | |
*** Apoorva_ has quit IRC | 01:37 | |
*** lukebrowning has quit IRC | 01:40 | |
*** lukebrowning has joined #openstack-infra | 01:42 | |
*** lukebrowning has quit IRC | 01:46 | |
*** liujiong has joined #openstack-infra | 01:47 | |
*** lukebrowning has joined #openstack-infra | 01:48 | |
*** claudiub has joined #openstack-infra | 01:48 | |
*** lukebrowning has quit IRC | 01:53 | |
*** edmondsw has quit IRC | 01:53 | |
*** ijw has joined #openstack-infra | 01:54 | |
*** lukebrowning has joined #openstack-infra | 01:54 | |
*** andreas_s has joined #openstack-infra | 01:55 | |
*** jamesmcarthur has joined #openstack-infra | 01:56 | |
*** wolverineav has joined #openstack-infra | 01:56 | |
*** lukebrowning has quit IRC | 01:59 | |
*** andreas_s has quit IRC | 01:59 | |
*** wolverineav has quit IRC | 02:01 | |
*** jamesmcarthur has quit IRC | 02:01 | |
*** ijw has quit IRC | 02:01 | |
*** lukebrowning has joined #openstack-infra | 02:02 | |
*** lukebrowning has quit IRC | 02:06 | |
*** ijw has joined #openstack-infra | 02:06 | |
*** lukebrowning has joined #openstack-infra | 02:08 | |
*** rcernin has joined #openstack-infra | 02:11 | |
*** lukebrowning has quit IRC | 02:13 | |
*** lukebrowning has joined #openstack-infra | 02:14 | |
*** salv-orlando has joined #openstack-infra | 02:14 | |
*** ijw has quit IRC | 02:15 | |
*** r-daneel has quit IRC | 02:17 | |
*** psachin has quit IRC | 02:18 | |
*** lukebrowning has quit IRC | 02:19 | |
*** salv-orlando has quit IRC | 02:20 | |
*** bobh has quit IRC | 02:20 | |
*** lukebrowning has joined #openstack-infra | 02:20 | |
*** daidv has quit IRC | 02:20 | |
*** bobh has joined #openstack-infra | 02:21 | |
*** daidv has joined #openstack-infra | 02:21 | |
*** bobh has quit IRC | 02:24 | |
*** bobh has joined #openstack-infra | 02:24 | |
*** lukebrowning has quit IRC | 02:25 | |
*** claudiub has quit IRC | 02:25 | |
*** lukebrowning has joined #openstack-infra | 02:27 | |
*** gcb has joined #openstack-infra | 02:27 | |
*** Apoorva has joined #openstack-infra | 02:29 | |
*** lukebrowning has quit IRC | 02:31 | |
*** lukebrowning has joined #openstack-infra | 02:33 | |
*** psachin has joined #openstack-infra | 02:36 | |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Xenial: add a service file https://review.openstack.org/528626 | 02:36 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test https://review.openstack.org/528130 | 02:36 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: [WIP] try a later version https://review.openstack.org/528156 | 02:36 |
*** ramishra has joined #openstack-infra | 02:37 | |
*** lukebrowning has quit IRC | 02:38 | |
*** lukebrowning has joined #openstack-infra | 02:39 | |
*** ramishra has quit IRC | 02:40 | |
*** liujiong has quit IRC | 02:42 | |
*** ramishra has joined #openstack-infra | 02:43 | |
*** lukebrowning has quit IRC | 02:44 | |
*** lukebrowning has joined #openstack-infra | 02:45 | |
*** jascott1_ has joined #openstack-infra | 02:47 | |
*** jascott1 has quit IRC | 02:47 | |
lbragstad | o/ i'm seeing some documentation failures that i can't recreate locally https://review.openstack.org/#/c/512468/6 | 02:48 |
lbragstad | is that possibly related to http://lists.openstack.org/pipermail/openstack-dev/2017-December/125710.html ? | 02:48 |
*** liujiong has joined #openstack-infra | 02:48 | |
*** lukebrowning has quit IRC | 02:50 | |
*** lukebrowning has joined #openstack-infra | 02:52 | |
*** bobh has quit IRC | 02:52 | |
*** bobh has joined #openstack-infra | 02:53 | |
*** ijw has joined #openstack-infra | 02:54 | |
*** liujiong has quit IRC | 02:54 | |
*** bmjen has quit IRC | 02:56 | |
*** Hunnur has quit IRC | 02:56 | |
*** mriedem has quit IRC | 02:56 | |
*** lukebrowning has quit IRC | 02:56 | |
*** liujiong has joined #openstack-infra | 02:56 | |
*** ijw has quit IRC | 02:59 | |
*** lukebrowning has joined #openstack-infra | 03:03 | |
*** lukebrowning has quit IRC | 03:07 | |
*** lukebrowning has joined #openstack-infra | 03:09 | |
*** rlandy|rover has quit IRC | 03:10 | |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Xenial: add a service file https://review.openstack.org/528626 | 03:11 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test https://review.openstack.org/528130 | 03:11 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: [WIP] try a later version https://review.openstack.org/528156 | 03:11 |
*** lukebrowning has quit IRC | 03:13 | |
clarkb | lbragstad: http://logs.openstack.org/68/512468/6/check/build-openstack-sphinx-docs/0773382/job-output.txt.gz#_2017-12-19_00_23_32_716523 is the error | 03:14 |
*** edmondsw has joined #openstack-infra | 03:15 | |
clarkb | lbragstad: which should've been fixed by https://review.openstack.org/#/c/528796 I think | 03:15 |
*** ramishra has quit IRC | 03:15 | |
*** salv-orlando has joined #openstack-infra | 03:15 | |
*** bobh has quit IRC | 03:17 | |
*** yamahata has joined #openstack-infra | 03:17 | |
clarkb | sphinx_pbr_autodoc was set to false according to ara | 03:18 |
clarkb | I see why. I'll get a patch up shortly | 03:19 |
*** lukebrowning has joined #openstack-infra | 03:19 | |
*** edmondsw has quit IRC | 03:19 | |
*** salv-orlando has quit IRC | 03:20 | |
lbragstad | clarkb: aha - so hold off on a recheck then? | 03:20 |
*** armax has quit IRC | 03:22 | |
clarkb | ya I don't think a recheck will work | 03:23 |
*** markvoelker has joined #openstack-infra | 03:24 | |
*** lukebrowning has quit IRC | 03:25 | |
openstackgerrit | Clark Boylan proposed openstack-infra/zuul-jobs master: Support autodoc_tree_index_modules in sphinx role https://review.openstack.org/528946 | 03:26 |
clarkb | lbragstad: AJaeger mordred config-core ^ I think taht should do it | 03:26 |
*** lukebrowning has joined #openstack-infra | 03:26 | |
*** yamahata has quit IRC | 03:27 | |
*** iyamahat has quit IRC | 03:27 | |
*** iyamahat has joined #openstack-infra | 03:27 | |
*** lukebrowning has quit IRC | 03:30 | |
*** lukebrowning has joined #openstack-infra | 03:32 | |
ianw | wow, there's no end of options, are there | 03:34 |
*** iyamahat has quit IRC | 03:35 | |
clarkb | ianw: configure all the things | 03:37 |
clarkb | I'm still not sure I understand the difference reading pbr's docs | 03:38 |
clarkb | but keystone definitely uses the one I Just added into the ansible role | 03:38 |
ianw | yeah, i have fought with those before | 03:38 |
*** lukebrowning has quit IRC | 03:41 | |
*** e0ne has joined #openstack-infra | 03:42 | |
*** lukebrowning has joined #openstack-infra | 03:42 | |
*** ramishra has joined #openstack-infra | 03:43 | |
*** e0ne has quit IRC | 03:47 | |
*** lukebrowning has quit IRC | 03:47 | |
ianw | 2017-12-19 03:30:50.967916 | TASK [sphinx : Attempt to get warning-is-error from config file] | 03:48 |
ianw | 2017-12-19 03:30:51.954655 | ubuntu-xenial | MODULE FAILURE | 03:48 |
ianw | that doesn't give much help | 03:48 |
clarkb | did my change fail? | 03:48 |
ianw | clarkb: http://logs.openstack.org/46/528946/1/check/build-openstack-sphinx-docs/a5ff336/job-output.txt.gz#_2017-12-19_03_30_50_967916 | 03:48 |
clarkb | I probably made a python fail | 03:49 |
*** lukebrowning has joined #openstack-infra | 03:49 | |
clarkb | yup | 03:50 |
clarkb | ara has details I will get a patch up | 03:50 |
openstackgerrit | Clark Boylan proposed openstack-infra/zuul-jobs master: Support autodoc_tree_index_modules in sphinx role https://review.openstack.org/528946 | 03:51 |
*** dave-mccowan has quit IRC | 03:51 | |
clarkb | ianw: ^ needed to set the variable first as without that in your setup.cfg the code to set the value won't be run | 03:51 |
ianw | ahh, that makes sense | 03:52 |
*** lukebrowning has quit IRC | 03:54 | |
*** ijw has joined #openstack-infra | 03:55 | |
*** lukebrowning has joined #openstack-infra | 03:55 | |
*** markvoelker has quit IRC | 03:58 | |
*** ijw has quit IRC | 03:59 | |
*** lukebrowning has quit IRC | 04:00 | |
*** bobh has joined #openstack-infra | 04:00 | |
*** e0ne has joined #openstack-infra | 04:01 | |
*** armax has joined #openstack-infra | 04:01 | |
*** lukebrowning has joined #openstack-infra | 04:03 | |
*** bobh has quit IRC | 04:04 | |
*** Apoorva has quit IRC | 04:04 | |
*** lukebrowning has quit IRC | 04:08 | |
*** lukebrowning has joined #openstack-infra | 04:09 | |
*** e0ne has quit IRC | 04:13 | |
*** lukebrowning has quit IRC | 04:14 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: zk: automatically retry command when connection is lost https://review.openstack.org/523640 | 04:14 |
*** lukebrowning has joined #openstack-infra | 04:15 | |
*** e0ne has joined #openstack-infra | 04:16 | |
*** salv-orlando has joined #openstack-infra | 04:16 | |
*** gouthamr has quit IRC | 04:18 | |
*** lukebrowning has quit IRC | 04:20 | |
*** salv-orlando has quit IRC | 04:21 | |
*** e0ne has quit IRC | 04:21 | |
*** lukebrowning has joined #openstack-infra | 04:21 | |
*** lukebrowning has quit IRC | 04:26 | |
*** e0ne has joined #openstack-infra | 04:26 | |
*** lukebrowning has joined #openstack-infra | 04:28 | |
*** psachin has quit IRC | 04:30 | |
*** e0ne has quit IRC | 04:31 | |
*** coolsvap has joined #openstack-infra | 04:32 | |
*** jamesmcarthur has joined #openstack-infra | 04:32 | |
*** lukebrowning has quit IRC | 04:33 | |
*** lukebrowning has joined #openstack-infra | 04:34 | |
*** psachin has joined #openstack-infra | 04:35 | |
*** jamesmcarthur has quit IRC | 04:36 | |
*** lukebrowning has quit IRC | 04:39 | |
*** lukebrowning has joined #openstack-infra | 04:40 | |
*** andreas_s has joined #openstack-infra | 04:40 | |
*** e0ne has joined #openstack-infra | 04:41 | |
*** iyamahat has joined #openstack-infra | 04:42 | |
*** janki has joined #openstack-infra | 04:43 | |
*** ykarel has joined #openstack-infra | 04:44 | |
*** lukebrowning has quit IRC | 04:45 | |
*** andreas_s has quit IRC | 04:45 | |
*** e0ne has quit IRC | 04:46 | |
*** lukebrowning has joined #openstack-infra | 04:46 | |
*** annp has quit IRC | 04:51 | |
*** lukebrowning has quit IRC | 04:51 | |
*** annp has joined #openstack-infra | 04:52 | |
*** lukebrowning has joined #openstack-infra | 04:53 | |
*** armax has quit IRC | 04:53 | |
*** kiennt26 has quit IRC | 04:54 | |
*** daidv has quit IRC | 04:54 | |
*** cuongnv has quit IRC | 04:54 | |
*** daidv has joined #openstack-infra | 04:55 | |
*** cuongnv has joined #openstack-infra | 04:55 | |
*** kiennt26 has joined #openstack-infra | 04:55 | |
*** markvoelker has joined #openstack-infra | 04:55 | |
*** e0ne has joined #openstack-infra | 04:56 | |
*** gcb has quit IRC | 04:56 | |
*** yamamoto has joined #openstack-infra | 04:57 | |
*** lukebrowning has quit IRC | 04:58 | |
*** gcb has joined #openstack-infra | 04:58 | |
*** pgadiya has joined #openstack-infra | 05:01 | |
*** e0ne has quit IRC | 05:01 | |
*** e0ne_ has joined #openstack-infra | 05:01 | |
*** lukebrowning has joined #openstack-infra | 05:04 | |
*** jascott1_ has left #openstack-infra | 05:06 | |
*** e0ne_ has quit IRC | 05:07 | |
*** yamamoto has quit IRC | 05:07 | |
*** psachin has quit IRC | 05:07 | |
*** yamamoto has joined #openstack-infra | 05:08 | |
*** lukebrowning has quit IRC | 05:09 | |
*** janki has quit IRC | 05:09 | |
*** lukebrowning has joined #openstack-infra | 05:10 | |
*** e0ne has joined #openstack-infra | 05:10 | |
*** janki has joined #openstack-infra | 05:10 | |
*** lukebrowning has quit IRC | 05:15 | |
*** e0ne has quit IRC | 05:15 | |
*** psachin has joined #openstack-infra | 05:16 | |
*** lukebrowning has joined #openstack-infra | 05:16 | |
*** salv-orlando has joined #openstack-infra | 05:17 | |
*** lukebrowning has quit IRC | 05:21 | |
*** salv-orlando has quit IRC | 05:22 | |
*** armax has joined #openstack-infra | 05:22 | |
*** lukebrowning has joined #openstack-infra | 05:23 | |
*** e0ne has joined #openstack-infra | 05:23 | |
*** yamamoto has quit IRC | 05:27 | |
*** lukebrowning has quit IRC | 05:28 | |
*** e0ne has quit IRC | 05:28 | |
*** markvoelker has quit IRC | 05:28 | |
*** armax has quit IRC | 05:28 | |
*** lukebrowning has joined #openstack-infra | 05:29 | |
*** yamamoto has joined #openstack-infra | 05:30 | |
*** e0ne has joined #openstack-infra | 05:32 | |
*** lukebrowning has quit IRC | 05:34 | |
*** lukebrowning has joined #openstack-infra | 05:35 | |
*** yamamoto has quit IRC | 05:36 | |
*** e0ne has quit IRC | 05:37 | |
*** lukebrowning has quit IRC | 05:40 | |
*** {aborted} is now known as chandankumar | 05:43 | |
*** e0ne has joined #openstack-infra | 05:44 | |
*** links has joined #openstack-infra | 05:47 | |
*** lukebrowning has joined #openstack-infra | 05:48 | |
*** e0ne has quit IRC | 05:49 | |
*** lukebrowning has quit IRC | 05:53 | |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Update default version of nodejs https://review.openstack.org/528156 | 05:53 |
openstackgerrit | Ian Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test https://review.openstack.org/528130 | 05:53 |
*** iyamahat_ has joined #openstack-infra | 05:53 | |
*** iyamahat has quit IRC | 05:53 | |
*** lukebrowning has joined #openstack-infra | 05:54 | |
*** e0ne has joined #openstack-infra | 05:54 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/irc-meetings master: Move irc meetings tox job in-tree https://review.openstack.org/528498 | 05:56 |
openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Update etherpad versions for Xenial https://review.openstack.org/528959 | 05:57 |
AJaeger | tonyb: figured it out, the check_chairs.py call parsed .zuul.yaml ;( ^ | 05:57 |
*** wolverineav has joined #openstack-infra | 05:57 | |
AJaeger | clarkb: thanks, +3 | 05:57 |
*** lukebrowning has quit IRC | 05:59 | |
*** e0ne has quit IRC | 05:59 | |
*** e0ne has joined #openstack-infra | 06:01 | |
ianw | clarkb: ^^ i think etherpad is getting close, but it's going to need a lot of babysitting | 06:02 |
*** wolverineav has quit IRC | 06:02 | |
*** lukebrowning has joined #openstack-infra | 06:04 | |
tonyb | AJaeger: Oh :( That'd do it sounds like a bug to me ;P | 06:04 |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Support autodoc_tree_index_modules in sphinx role https://review.openstack.org/528946 | 06:06 |
*** annp has quit IRC | 06:08 | |
*** annp has joined #openstack-infra | 06:09 | |
*** lukebrowning has quit IRC | 06:09 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/528963 | 06:10 |
*** lukebrowning has joined #openstack-infra | 06:10 | |
*** afazekas has quit IRC | 06:11 | |
*** afazekas has joined #openstack-infra | 06:11 | |
*** e0ne has quit IRC | 06:13 | |
*** namnh has joined #openstack-infra | 06:13 | |
*** lukebrowning has quit IRC | 06:15 | |
*** e0ne has joined #openstack-infra | 06:16 | |
*** lukebrowning has joined #openstack-infra | 06:16 | |
*** armax has joined #openstack-infra | 06:21 | |
*** lukebrowning has quit IRC | 06:21 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix docs building https://review.openstack.org/528822 | 06:21 |
*** e0ne has quit IRC | 06:21 | |
*** kiennt26 has quit IRC | 06:22 | |
*** kiennt26 has joined #openstack-infra | 06:22 | |
*** janki has quit IRC | 06:22 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/irc-meetings master: Move irc meetings tox job in-tree https://review.openstack.org/528498 | 06:23 |
AJaeger | tonyb: now really fixed ^ | 06:23 |
* AJaeger forgot to push a change | 06:24 | |
*** lukebrowning has joined #openstack-infra | 06:24 | |
AJaeger | ianw, frickler could you review two job removal changes for openstack-zuul-jobs: https://review.openstack.org/528581 and https://review.openstack.org/#/c/522471/ , please? | 06:25 |
*** markvoelker has joined #openstack-infra | 06:25 | |
AJaeger | lbragstad: the fix merged, I'll recheck your change | 06:26 |
*** armax has quit IRC | 06:28 | |
*** CrayZee has joined #openstack-infra | 06:28 | |
*** e0ne has joined #openstack-infra | 06:28 | |
*** lukebrowning has quit IRC | 06:29 | |
openstackgerrit | Merged openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/528963 | 06:30 |
AJaeger | ianw: dmsimard asked that you review https://review.openstack.org/525658 - could you do so, please? His -1 is procedural | 06:30 |
*** lukebrowning has joined #openstack-infra | 06:30 | |
AJaeger | tonyb: 528498 passes testsuite now | 06:32 |
*** e0ne has quit IRC | 06:33 | |
*** yamamoto has joined #openstack-infra | 06:35 | |
*** lukebrowning has quit IRC | 06:35 | |
*** lukebrowning has joined #openstack-infra | 06:36 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/devstack-gate master: Move commands to the base testenv https://review.openstack.org/528819 | 06:38 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/devstack-gate master: Remove run-tests and py3-run-tests https://review.openstack.org/528817 | 06:38 |
*** lukebrowning has quit IRC | 06:41 | |
*** yamamoto has quit IRC | 06:41 | |
*** e0ne has joined #openstack-infra | 06:42 | |
*** lukebrowning has joined #openstack-infra | 06:43 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: config: add statsd-server config parameter https://review.openstack.org/528969 | 06:44 |
*** yolanda has quit IRC | 06:44 | |
*** yolanda has joined #openstack-infra | 06:47 | |
*** e0ne has quit IRC | 06:47 | |
*** lukebrowning has quit IRC | 06:48 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Git driver https://review.openstack.org/525614 | 06:48 |
*** lukebrowning has joined #openstack-infra | 06:49 | |
*** edmondsw has joined #openstack-infra | 06:51 | |
*** e0ne has joined #openstack-infra | 06:53 | |
*** lukebrowning has quit IRC | 06:54 | |
*** armaan has quit IRC | 06:55 | |
*** lukebrowning has joined #openstack-infra | 06:55 | |
*** armaan has joined #openstack-infra | 06:55 | |
*** edmondsw has quit IRC | 06:55 | |
*** e0ne has quit IRC | 06:58 | |
*** markvoelker has quit IRC | 06:59 | |
*** e0ne has joined #openstack-infra | 06:59 | |
*** jchhatbar has joined #openstack-infra | 07:00 | |
*** lukebrowning has quit IRC | 07:00 | |
*** e0ne has quit IRC | 07:04 | |
*** lukebrowning has joined #openstack-infra | 07:04 | |
*** lukebrowning has quit IRC | 07:09 | |
*** armax has joined #openstack-infra | 07:10 | |
*** lukebrowning has joined #openstack-infra | 07:10 | |
*** armax has quit IRC | 07:11 | |
*** e0ne has joined #openstack-infra | 07:14 | |
*** lukebrowning has quit IRC | 07:15 | |
*** lukebrowning has joined #openstack-infra | 07:17 | |
*** dsariel has quit IRC | 07:17 | |
*** kjackal has joined #openstack-infra | 07:17 | |
*** andreas_s has joined #openstack-infra | 07:18 | |
*** salv-orlando has joined #openstack-infra | 07:19 | |
*** e0ne has quit IRC | 07:20 | |
*** claudiub has joined #openstack-infra | 07:20 | |
*** lukebrowning has quit IRC | 07:21 | |
*** lukebrowning has joined #openstack-infra | 07:22 | |
*** salv-orlando has quit IRC | 07:23 | |
*** lukebrowning has quit IRC | 07:27 | |
*** e0ne has joined #openstack-infra | 07:28 | |
*** rcernin has quit IRC | 07:31 | |
*** lukebrowning has joined #openstack-infra | 07:32 | |
*** e0ne has quit IRC | 07:33 | |
*** e0ne has joined #openstack-infra | 07:35 | |
*** lukebrowning has quit IRC | 07:36 | |
*** shardy has joined #openstack-infra | 07:37 | |
*** shardy has quit IRC | 07:37 | |
*** lukebrowning has joined #openstack-infra | 07:38 | |
*** e0ne has quit IRC | 07:41 | |
*** shardy has joined #openstack-infra | 07:41 | |
*** lukebrowning has quit IRC | 07:42 | |
openstackgerrit | Chandan Kumar proposed openstack-infra/project-config master: Add new repo for Blazar tempest plugin https://review.openstack.org/528672 | 07:43 |
*** yamamoto has joined #openstack-infra | 07:45 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement an OpenContainer driver https://review.openstack.org/468753 | 07:46 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement a Kubernetes driver https://review.openstack.org/521356 | 07:46 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement an Amazon EC2 driver https://review.openstack.org/528982 | 07:46 |
*** slaweq has joined #openstack-infra | 07:46 | |
*** mmethot_ has joined #openstack-infra | 07:49 | |
*** lukebrowning has joined #openstack-infra | 07:50 | |
*** mmethot has quit IRC | 07:51 | |
*** e0ne has joined #openstack-infra | 07:51 | |
*** salv-orlando has joined #openstack-infra | 07:54 | |
*** e0ne has quit IRC | 07:55 | |
*** lukebrowning has quit IRC | 07:57 | |
*** e0ne has joined #openstack-infra | 07:58 | |
*** lukebrowning has joined #openstack-infra | 07:59 | |
*** rcernin has joined #openstack-infra | 08:02 | |
*** d0ugal has quit IRC | 08:03 | |
*** e0ne has quit IRC | 08:03 | |
*** lukebrowning has quit IRC | 08:04 | |
*** lukebrowning has joined #openstack-infra | 08:05 | |
*** d0ugal has joined #openstack-infra | 08:07 | |
*** e0ne has joined #openstack-infra | 08:07 | |
*** slaweq_ has joined #openstack-infra | 08:09 | |
*** HeOS has joined #openstack-infra | 08:09 | |
*** ralonsoh has joined #openstack-infra | 08:10 | |
*** lukebrowning has quit IRC | 08:10 | |
*** cuongnv has quit IRC | 08:10 | |
*** daidv has quit IRC | 08:10 | |
*** daidv has joined #openstack-infra | 08:11 | |
*** cuongnv has joined #openstack-infra | 08:11 | |
*** lukebrowning has joined #openstack-infra | 08:11 | |
*** slaweq_ has quit IRC | 08:14 | |
*** e0ne has quit IRC | 08:16 | |
*** lukebrowning has quit IRC | 08:16 | |
*** lukebrowning has joined #openstack-infra | 08:18 | |
*** xinliang has quit IRC | 08:22 | |
*** alexchadin has joined #openstack-infra | 08:22 | |
*** lukebrowning has quit IRC | 08:22 | |
*** lukebrowning has joined #openstack-infra | 08:23 | |
*** amoralej|off is now known as amoralej | 08:25 | |
*** e0ne has joined #openstack-infra | 08:28 | |
*** lukebrowning has quit IRC | 08:28 | |
*** andreas__ has joined #openstack-infra | 08:30 | |
openstackgerrit | megan guiney proposed openstack-infra/openstack-zuul-jobs master: [WIP] docs sitemap generation automation https://review.openstack.org/524862 | 08:30 |
*** hashar has joined #openstack-infra | 08:32 | |
*** jamesmcarthur has joined #openstack-infra | 08:33 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack-infra/openstack-zuul-jobs master: Remove neutron-dynamic-routing legacy jobs https://review.openstack.org/528988 | 08:33 |
*** lukebrowning has joined #openstack-infra | 08:33 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack-infra/project-config master: Remove neutron-dynamic-routing legacy jobs https://review.openstack.org/528989 | 08:33 |
*** yolanda__ has joined #openstack-infra | 08:34 | |
*** andreas_s has quit IRC | 08:34 | |
*** xinliang has joined #openstack-infra | 08:34 | |
*** yolanda has quit IRC | 08:36 | |
*** florianf has joined #openstack-infra | 08:37 | |
*** jamesmcarthur has quit IRC | 08:37 | |
*** lukebrowning has quit IRC | 08:38 | |
*** ccamacho has joined #openstack-infra | 08:39 | |
*** lukebrowning has joined #openstack-infra | 08:39 | |
*** edmondsw has joined #openstack-infra | 08:39 | |
*** jtomasek has joined #openstack-infra | 08:40 | |
*** lukebrowning has quit IRC | 08:44 | |
*** edmondsw has quit IRC | 08:44 | |
*** salv-orlando has quit IRC | 08:45 | |
*** lukebrowning has joined #openstack-infra | 08:45 | |
*** salv-orlando has joined #openstack-infra | 08:46 | |
*** alexchadin has quit IRC | 08:47 | |
*** alexchadin has joined #openstack-infra | 08:48 | |
*** priteau has joined #openstack-infra | 08:50 | |
*** lukebrowning has quit IRC | 08:50 | |
*** jpena|off is now known as jpena | 08:51 | |
*** salv-orlando has quit IRC | 08:51 | |
*** lukebrowning has joined #openstack-infra | 08:52 | |
*** hashar has quit IRC | 08:54 | |
*** markvoelker has joined #openstack-infra | 08:56 | |
*** lukebrowning has quit IRC | 08:57 | |
*** lukebrowning has joined #openstack-infra | 08:58 | |
*** salv-orlando has joined #openstack-infra | 08:58 | |
*** hashar has joined #openstack-infra | 08:59 | |
chandankumar | AJaeger: hello | 08:59 |
chandankumar | AJaeger: anything more to add in this review https://review.openstack.org/#/c/528672/ | 08:59 |
AJaeger | hi chandankumar , let me check | 08:59 |
chandankumar | as per last dmsimard comment i have again fixed it | 08:59 |
*** e0ne has quit IRC | 09:00 | |
AJaeger | chandankumar: sorry, you misunderstood my comment - and we fixed the problem properly, so we're back to where we started with the two changes | 09:00 |
*** andreas_s has joined #openstack-infra | 09:00 | |
chandankumar | AJaeger: cool, thanks :-) | 09:01 |
AJaeger | chandankumar: so, change is fine, +2. Anything else open from you that I can check or clarify? | 09:02 |
chandankumar | AJaeger: both are done from your side :-) | 09:02 |
*** lukebrowning has quit IRC | 09:03 | |
chandankumar | i hope blazar would be the last one from tempest plugin goal repo creation | 09:03 |
AJaeger | yeah ;) | 09:04 |
*** andreas__ has quit IRC | 09:04 | |
AJaeger | chandankumar: thanks a lot for driving this! | 09:04 |
*** lukebrowning has joined #openstack-infra | 09:05 | |
*** lukebrowning has quit IRC | 09:09 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack-infra/devstack-gate master: Add neutron-tempest-plugin-api job https://review.openstack.org/529000 | 09:09 |
*** lukebrowning has joined #openstack-infra | 09:11 | |
*** lukebrowning has quit IRC | 09:16 | |
*** lukebrowning has joined #openstack-infra | 09:17 | |
openstackgerrit | Javier Peña proposed openstack-infra/puppet-openstackci master: Add AFS mirror support https://review.openstack.org/528739 | 09:23 |
*** lukebrowning has quit IRC | 09:28 | |
*** lukebrowning has joined #openstack-infra | 09:29 | |
*** markvoelker has quit IRC | 09:29 | |
*** jpich has joined #openstack-infra | 09:29 | |
AJaeger | frickler: could you review https://review.openstack.org/#/c/521557/ , please? We now have updated the check/gate job and should update the post job the same way... | 09:32 |
*** lucas-afk is now known as lucasagomes | 09:32 | |
AJaeger | mordred: ^ | 09:32 |
*** armaan has quit IRC | 09:33 | |
*** armaan has joined #openstack-infra | 09:34 | |
*** lukebrowning has quit IRC | 09:34 | |
*** lukebrowning has joined #openstack-infra | 09:35 | |
*** derekh has joined #openstack-infra | 09:36 | |
*** lukebrowning has quit IRC | 09:40 | |
*** lukebrowning has joined #openstack-infra | 09:42 | |
*** armaan has quit IRC | 09:45 | |
*** armaan has joined #openstack-infra | 09:46 | |
*** lukebrowning has quit IRC | 09:46 | |
*** lukebrowning has joined #openstack-infra | 09:48 | |
*** wolverineav has joined #openstack-infra | 09:50 | |
*** jascott1 has joined #openstack-infra | 09:52 | |
*** lukebrowning has quit IRC | 09:53 | |
*** liujiong has quit IRC | 09:53 | |
*** lukebrowning has joined #openstack-infra | 09:54 | |
*** wolverineav has quit IRC | 09:55 | |
*** jascott1 has quit IRC | 09:57 | |
*** calbers has quit IRC | 09:57 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add swagger api documentation https://review.openstack.org/529013 | 09:58 |
*** lukebrowning has quit IRC | 09:59 | |
*** Shrews has quit IRC | 10:00 | |
*** afazekas has quit IRC | 10:01 | |
*** namnh has quit IRC | 10:01 | |
*** kiennt26 has quit IRC | 10:02 | |
*** calbers has joined #openstack-infra | 10:03 | |
frickler | AJaeger: is there a test for https://review.openstack.org/521557 somewhere? otherwise, how about adding the new job first and verifying it? I'd really like to stop breaking things every other day | 10:04 |
*** lukebrowning has joined #openstack-infra | 10:05 | |
*** afazekas has joined #openstack-infra | 10:07 | |
*** lukebrowning has quit IRC | 10:09 | |
*** cuongnv has quit IRC | 10:09 | |
*** Shrews has joined #openstack-infra | 10:10 | |
*** bhavik has joined #openstack-infra | 10:10 | |
AJaeger | frickler: no test - and it runs as post job ;( | 10:12 |
AJaeger | frickler: this should mirror what we do in build-sphinx-docs since yesterday - and is needed due to the changes we merged yesterday | 10:13 |
AJaeger | frickler: we can wait with merging until mordred is around in case of problems - so, a +2 alone without the +W. | 10:14 |
*** psachin has quit IRC | 10:24 | |
*** bhavik has quit IRC | 10:26 | |
*** markvoelker has joined #openstack-infra | 10:26 | |
*** edmondsw has joined #openstack-infra | 10:27 | |
*** lukebrowning has joined #openstack-infra | 10:29 | |
frickler | AJaeger: I'm not convinced yet. what prevents you from changing the post job only for a single project and test it there? | 10:30 |
*** e0ne has joined #openstack-infra | 10:31 | |
*** edmondsw has quit IRC | 10:32 | |
*** lukebrowning has quit IRC | 10:34 | |
*** lukebrowning has joined #openstack-infra | 10:36 | |
gcb | AJaeger: about comments in https://review.openstack.org/#/c/528970/ should we adjust the proposal change in http://lists.openstack.org/pipermail/openstack-dev/2017-December/125710.html ? | 10:37 |
*** jtomasek has quit IRC | 10:38 | |
*** sambetts|afk is now known as sambetts | 10:39 | |
*** jtomasek has joined #openstack-infra | 10:39 | |
*** jpich has quit IRC | 10:40 | |
*** lukebrowning has quit IRC | 10:41 | |
*** jchhatbar is now known as janki | 10:41 | |
*** niska has quit IRC | 10:42 | |
*** lukebrowning has joined #openstack-infra | 10:42 | |
*** stakeda has quit IRC | 10:44 | |
AJaeger | gcb: yes, we should - let's ask mordred once he's awake ^ | 10:46 |
*** electrofelix has joined #openstack-infra | 10:46 | |
AJaeger | gcb: just pushed a few changes to remove tox_install.sh from remaining oslo projects, those are needed as well | 10:46 |
AJaeger | gcb, see topic:rm-tox_install | 10:46 |
*** lukebrowning has quit IRC | 10:47 | |
gcb | AJaeger: thanks will look at them | 10:47 |
*** lukebrowning has joined #openstack-infra | 10:48 | |
AJaeger | frickler: let's talk with mordred later... | 10:49 |
*** niska has joined #openstack-infra | 10:52 | |
*** lukebrowning has quit IRC | 10:53 | |
*** yamamoto has quit IRC | 10:55 | |
*** lukebrowning has joined #openstack-infra | 10:55 | |
*** yamamoto has joined #openstack-infra | 10:55 | |
*** yamamoto has quit IRC | 10:56 | |
*** yamamoto has joined #openstack-infra | 10:56 | |
*** lukebrowning has quit IRC | 10:59 | |
*** markvoelker has quit IRC | 11:00 | |
ttx | I'm trying to fix the governance doc builds -- I can't really reproduce errors locally though. | 11:00 |
ttx | Running tox -evenv -- sphinx-build -W -b html doc/source doc/build/html | 11:01 |
ttx | Things just work | 11:01 |
ttx | Is there a better way to simulate the new thing ? | 11:01 |
AJaeger | ttx, governance needs python3. | 11:01 |
AJaeger | but our new job by default uses python2. Let me check with mordred how to fix this best later today... | 11:02 |
ttx | AJaeger: ok | 11:02 |
AJaeger | gcb: better wait with removing the requirements.txt line, I'll wait for mordred to comment | 11:03 |
gcb | AJaeger: okay | 11:03 |
AJaeger | gcb: i see you just updated - let's see what mordred says | 11:04 |
AJaeger | mordred: I updated https://etherpad.openstack.org/p/merge-updated-pti-topic with some comments | 11:04 |
AJaeger | bbl | 11:04 |
ttx | AJaeger: the releases repo is in the same situation.... smcginnis included some workarounds for open() calls now being run under python 2.7 and failing | 11:04 |
ttx | so let's see if there is a more general way to solve this and allow the doc job to run under py3 | 11:05 |
*** lukebrowning has joined #openstack-infra | 11:05 | |
*** salv-orlando has quit IRC | 11:09 | |
*** salv-orlando has joined #openstack-infra | 11:09 | |
*** yamamoto has quit IRC | 11:09 | |
openstackgerrit | Javier Peña proposed openstack-infra/system-config master: Move AFS mirror code to puppet-openstackci https://review.openstack.org/529032 | 11:10 |
*** lukebrowning has quit IRC | 11:10 | |
*** andreas_s has quit IRC | 11:10 | |
*** andreas_s has joined #openstack-infra | 11:11 | |
*** lukebrowning has joined #openstack-infra | 11:11 | |
*** yamamoto has joined #openstack-infra | 11:13 | |
*** salv-orlando has quit IRC | 11:14 | |
*** lukebrowning has quit IRC | 11:17 | |
*** yamamoto has quit IRC | 11:18 | |
*** bhavik has joined #openstack-infra | 11:19 | |
*** ldnunes has joined #openstack-infra | 11:22 | |
*** pgadiya has quit IRC | 11:22 | |
*** andreas_s has quit IRC | 11:24 | |
*** andreas_s has joined #openstack-infra | 11:26 | |
*** armaan has quit IRC | 11:28 | |
*** armaan has joined #openstack-infra | 11:28 | |
*** jtomasek has quit IRC | 11:29 | |
*** andreas_s has quit IRC | 11:30 | |
*** jamesmcarthur has joined #openstack-infra | 11:33 | |
*** armaan has quit IRC | 11:33 | |
*** armaan has joined #openstack-infra | 11:33 | |
*** jamesmcarthur has quit IRC | 11:38 | |
*** lukebrowning has joined #openstack-infra | 11:39 | |
*** andreas_s has joined #openstack-infra | 11:40 | |
*** e0ne has quit IRC | 11:41 | |
*** lukebrowning has quit IRC | 11:43 | |
*** alexchadin has quit IRC | 11:44 | |
*** armaan has quit IRC | 11:45 | |
*** lukebrowning has joined #openstack-infra | 11:45 | |
*** pbourke has joined #openstack-infra | 11:47 | |
*** andreas_s has quit IRC | 11:50 | |
*** yolanda__ is now known as yolanda | 11:50 | |
*** lukebrowning has quit IRC | 11:50 | |
*** andreas_s has joined #openstack-infra | 11:51 | |
AJaeger | ttx: there is - mordred did the ground work already. I just wonder how to configure in the best way. So, let's wait for him and discuss later today... | 11:51 |
*** e0ne has joined #openstack-infra | 11:51 | |
*** lukebrowning has joined #openstack-infra | 11:51 | |
*** andreas_s has quit IRC | 11:53 | |
*** andreas_s has joined #openstack-infra | 11:53 | |
*** lukebrowning has quit IRC | 11:56 | |
*** smatzek has joined #openstack-infra | 11:56 | |
*** markvoelker has joined #openstack-infra | 11:57 | |
*** lukebrowning has joined #openstack-infra | 11:58 | |
*** bhavik has quit IRC | 12:02 | |
*** lukebrowning has quit IRC | 12:03 | |
*** lukebrowning has joined #openstack-infra | 12:06 | |
*** yamamoto has joined #openstack-infra | 12:06 | |
*** alexchadin has joined #openstack-infra | 12:06 | |
*** salv-orlando has joined #openstack-infra | 12:10 | |
*** yamamoto has quit IRC | 12:10 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Small refactor of translation scripts https://review.openstack.org/529047 | 12:14 |
*** salv-orlando has quit IRC | 12:14 | |
*** lukebrowning has quit IRC | 12:15 | |
*** edmondsw has joined #openstack-infra | 12:16 | |
*** annp has quit IRC | 12:17 | |
*** rkukura has quit IRC | 12:18 | |
*** edmondsw has quit IRC | 12:20 | |
*** e0ne has quit IRC | 12:22 | |
*** links has quit IRC | 12:23 | |
*** katkapilatova has joined #openstack-infra | 12:27 | |
*** amoralej is now known as amoralej|off | 12:29 | |
*** markvoelker has quit IRC | 12:29 | |
*** yamamoto has joined #openstack-infra | 12:38 | |
*** yamamoto has quit IRC | 12:39 | |
*** lucasagomes is now known as lucas-hungry | 12:42 | |
*** alexchadin has quit IRC | 12:46 | |
*** jpena is now known as jpena|lunch | 12:48 | |
*** weshay_pto is now known as weshay | 12:48 | |
*** hichihara has joined #openstack-infra | 12:50 | |
*** aarefiev has joined #openstack-infra | 12:50 | |
*** bobh has joined #openstack-infra | 12:52 | |
*** bobh has quit IRC | 12:53 | |
*** ralonsoh has quit IRC | 12:53 | |
*** jascott1 has joined #openstack-infra | 12:54 | |
*** claudiub|2 has joined #openstack-infra | 12:57 | |
*** jascott1 has quit IRC | 12:59 | |
*** zhurong has joined #openstack-infra | 13:00 | |
*** claudiub has quit IRC | 13:00 | |
*** zhurong has quit IRC | 13:02 | |
*** wolverineav has joined #openstack-infra | 13:02 | |
*** zhurong has joined #openstack-infra | 13:03 | |
*** claudiub has joined #openstack-infra | 13:04 | |
openstackgerrit | Fabien Boucher proposed openstack-infra/zuul feature/zuulv3: Do not exit early tenant configuration in case of config issue https://review.openstack.org/529060 | 13:04 |
*** jcoufal has joined #openstack-infra | 13:06 | |
*** claudiub|2 has quit IRC | 13:07 | |
*** salv-orlando has joined #openstack-infra | 13:11 | |
*** janki has quit IRC | 13:11 | |
*** janki has joined #openstack-infra | 13:11 | |
*** catintheroof has joined #openstack-infra | 13:11 | |
*** catintheroof has quit IRC | 13:12 | |
*** catintheroof has joined #openstack-infra | 13:12 | |
*** lukebrowning has joined #openstack-infra | 13:13 | |
*** d0ugal has quit IRC | 13:14 | |
*** salv-orlando has quit IRC | 13:15 | |
*** rossella_s has joined #openstack-infra | 13:18 | |
*** lukebrowning has quit IRC | 13:18 | |
*** links has joined #openstack-infra | 13:18 | |
*** r-daneel has joined #openstack-infra | 13:19 | |
*** rcernin has quit IRC | 13:20 | |
*** lukebrowning has joined #openstack-infra | 13:20 | |
*** r-daneel has quit IRC | 13:20 | |
*** d0ugal has joined #openstack-infra | 13:23 | |
*** lukebrowning has quit IRC | 13:24 | |
*** zhurong has quit IRC | 13:25 | |
*** dprince has joined #openstack-infra | 13:25 | |
*** hashar has quit IRC | 13:25 | |
*** hashar has joined #openstack-infra | 13:26 | |
*** lukebrowning has joined #openstack-infra | 13:26 | |
*** markvoelker has joined #openstack-infra | 13:27 | |
*** markvoelker has quit IRC | 13:28 | |
*** fultonj has quit IRC | 13:28 | |
*** rlandy has joined #openstack-infra | 13:29 | |
*** markvoelker has joined #openstack-infra | 13:29 | |
*** rlandy is now known as rlandy|rover | 13:29 | |
*** dsariel has joined #openstack-infra | 13:30 | |
*** lukebrowning has quit IRC | 13:31 | |
*** coolsvap has quit IRC | 13:32 | |
*** lukebrowning has joined #openstack-infra | 13:32 | |
*** armaan has joined #openstack-infra | 13:34 | |
*** r-daneel has joined #openstack-infra | 13:34 | |
*** lucas-hungry is now known as lucasagomes | 13:34 | |
*** trown|outtypewww is now known as trown|ruck | 13:36 | |
*** rkukura has joined #openstack-infra | 13:37 | |
andreaf | AJaeger, frickler: could you have a look at if you have a sec? https://review.openstack.org/#/c/528748/ - it's a very simple change and a long list of approved devstack patches depends on it :) | 13:37 |
*** d0ugal has quit IRC | 13:37 | |
*** lukebrowning has quit IRC | 13:37 | |
*** lukebrowning has joined #openstack-infra | 13:39 | |
*** ralonsoh has joined #openstack-infra | 13:39 | |
*** fultonj has joined #openstack-infra | 13:39 | |
*** d0ugal has joined #openstack-infra | 13:39 | |
*** yamamoto has joined #openstack-infra | 13:40 | |
frickler | andreaf: done | 13:43 |
*** lukebrowning has quit IRC | 13:43 | |
*** lukebrowning has joined #openstack-infra | 13:44 | |
*** armaan has quit IRC | 13:45 | |
*** dave-mccowan has joined #openstack-infra | 13:45 | |
*** kgiusti has joined #openstack-infra | 13:46 | |
*** mihalis68 has joined #openstack-infra | 13:48 | |
*** yamamoto has quit IRC | 13:48 | |
*** ykarel has quit IRC | 13:48 | |
*** jpena|lunch is now known as jpena | 13:49 | |
*** lukebrowning has quit IRC | 13:50 | |
openstackgerrit | Merged openstack-infra/devstack-gate master: Import tox jobs https://review.openstack.org/528553 | 13:50 |
*** rkukura has quit IRC | 13:50 | |
*** lukebrowning has joined #openstack-infra | 13:51 | |
*** makowals has joined #openstack-infra | 13:53 | |
*** dsariel has quit IRC | 13:53 | |
*** annp has joined #openstack-infra | 13:54 | |
*** lukebrowning has quit IRC | 13:55 | |
*** lukebrowning has joined #openstack-infra | 13:57 | |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Dereference symlinks when staging artefacts https://review.openstack.org/528748 | 13:58 |
*** mriedem has joined #openstack-infra | 13:58 | |
*** bh526r has joined #openstack-infra | 13:58 | |
*** lukebrowning has quit IRC | 14:01 | |
*** rkukura has joined #openstack-infra | 14:03 | |
*** links has quit IRC | 14:03 | |
*** dhill_ has joined #openstack-infra | 14:06 | |
*** salv-orlando has joined #openstack-infra | 14:11 | |
openstackgerrit | Akihiro Motoki proposed openstack-infra/project-config master: Drop django_openstack_auth master branch jobs https://review.openstack.org/529080 | 14:13 |
*** katkapilatova has quit IRC | 14:14 | |
*** salv-orlando has quit IRC | 14:16 | |
*** links has joined #openstack-infra | 14:17 | |
*** lukebrowning has joined #openstack-infra | 14:17 | |
*** lukebrowning has quit IRC | 14:22 | |
*** dhill_ has quit IRC | 14:22 | |
andreykurilin | hi folks! does anyone can explain me about writting custom roles. I mean how can I assign the custom role to task? | 14:22 |
*** salv-orlando has joined #openstack-infra | 14:23 | |
*** lukebrowning has joined #openstack-infra | 14:23 | |
*** jtomasek has joined #openstack-infra | 14:23 | |
*** lukebrowning has quit IRC | 14:28 | |
*** dhill_ has joined #openstack-infra | 14:29 | |
*** smatzek has quit IRC | 14:29 | |
*** lukebrowning has joined #openstack-infra | 14:30 | |
*** andreas_s has quit IRC | 14:30 | |
*** andreas_s has joined #openstack-infra | 14:30 | |
lbragstad | clarkb: AJaeger thanks for the assist | 14:33 |
*** lukebrowning has quit IRC | 14:34 | |
*** lukebrowning has joined #openstack-infra | 14:36 | |
*** kiennt26 has joined #openstack-infra | 14:37 | |
*** abelur_ has quit IRC | 14:37 | |
*** lukebrowning has quit IRC | 14:41 | |
*** lukebrowning has joined #openstack-infra | 14:42 | |
andreaf | frickler: cool, thanks | 14:43 |
*** andreas_s has quit IRC | 14:45 | |
*** andreas_s has joined #openstack-infra | 14:45 | |
*** sshnaidm has quit IRC | 14:46 | |
*** gouthamr has joined #openstack-infra | 14:47 | |
*** ramishra has quit IRC | 14:47 | |
*** lukebrowning has quit IRC | 14:47 | |
*** felipemonteiro has joined #openstack-infra | 14:47 | |
*** sshnaidm has joined #openstack-infra | 14:48 | |
*** andreas_s has quit IRC | 14:48 | |
*** andreas_s has joined #openstack-infra | 14:48 | |
*** hichihara has quit IRC | 14:52 | |
*** witek has joined #openstack-infra | 14:54 | |
*** lukebrowning has joined #openstack-infra | 14:55 | |
*** jascott1 has joined #openstack-infra | 14:56 | |
*** smatzek has joined #openstack-infra | 14:58 | |
*** lukebrowning has quit IRC | 14:59 | |
*** jascott1 has quit IRC | 15:01 | |
*** lukebrowning has joined #openstack-infra | 15:01 | |
*** andreas_s has quit IRC | 15:03 | |
*** andreas_s has joined #openstack-infra | 15:03 | |
*** salv-orlando has quit IRC | 15:04 | |
*** salv-orlando has joined #openstack-infra | 15:04 | |
*** lukebrowning has quit IRC | 15:06 | |
cmurphy | AJaeger: keystone doc builds are still broken, can you tell us if https://review.openstack.org/#/c/528960 is the way to go? basically our setup.cfg [extras] aren't getting installed | 15:07 |
*** lukebrowning has joined #openstack-infra | 15:08 | |
*** efoley has joined #openstack-infra | 15:08 | |
*** salv-orlando has quit IRC | 15:09 | |
AJaeger | cmurphy: just started loooking at that patch ;) give me a few mins | 15:09 |
cmurphy | :) | 15:10 |
*** marst has joined #openstack-infra | 15:10 | |
*** kiennt26 has quit IRC | 15:11 | |
cmurphy | AJaeger: recent failure is https://review.openstack.org/#/c/528866/ btw | 15:11 |
*** kiennt26 has joined #openstack-infra | 15:11 | |
*** andreas_s has quit IRC | 15:12 | |
AJaeger | mordred: master debugger of docs related odditities, are you already around? | 15:12 |
*** lukebrowning has quit IRC | 15:12 | |
AJaeger | change 528960 just move all the extras from setup.cfg into doc/requirements.txt so that the build actually works. Quite a hack ;( But it generates the documentation ;) | 15:13 |
*** andreas_s has joined #openstack-infra | 15:13 | |
AJaeger | cmurphy: I agree, tox.ini should use the previous python.py setup.py build_sphinx | 15:14 |
*** lukebrowning has joined #openstack-infra | 15:14 | |
AJaeger | cmurphy: let me update 528960 with that single line change and wait for mordred on whether he has a better idea. This works but I'm unhappy ;( | 15:15 |
*** salv-orlando has joined #openstack-infra | 15:15 | |
*** gmmaha has left #openstack-infra | 15:16 | |
cmurphy | AJaeger: cool :) | 15:16 |
AJaeger | cmurphy: pushed the change - it will take some time until it passes, I suggest we hear whether mordred has a better idea - but if he's not reacting, go ahead and merge to unblock keystone and we can revert/adjust later. | 15:18 |
andreaf | AJaeger, frickler: any idea how to trigger the gate on https://review.openstack.org/#/c/527106? The parent and depends-on patches are merged but nothing happens | 15:18 |
cmurphy | AJaeger: alright thank you | 15:19 |
*** lukebrowning has quit IRC | 15:19 | |
AJaeger | andreaf: checking... | 15:19 |
AJaeger | andreaf: a recheck should be enough - or another +A. You're core there, aren't you? Just +A to generate the event | 15:20 |
*** aarefiev has quit IRC | 15:20 | |
*** lukebrowning has joined #openstack-infra | 15:20 | |
*** armax has joined #openstack-infra | 15:20 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate master: Handle a stestr results directory https://review.openstack.org/523646 | 15:21 |
andreaf | AJaeger ok thx | 15:21 |
*** andreas_s has quit IRC | 15:23 | |
AJaeger | andreaf: the two repos - the dependent one and the one waiting - do not share a queue. And then there's no notification when dependent job merged... | 15:24 |
AJaeger | andreaf: a recheck would work as well, but then you would need to wait for the job to run through check again, so the extra +A triggers the gate here | 15:24 |
*** lukebrowning has quit IRC | 15:25 | |
AJaeger | config-core, could you review https://review.openstack.org/#/c/528518 and https://review.openstack.org/#/c/529047/1 for translations, please? Fixes one specific cornercase (only one repo has the situation). | 15:25 |
*** bobh has joined #openstack-infra | 15:26 | |
*** lukebrowning has joined #openstack-infra | 15:26 | |
*** abelur_ has joined #openstack-infra | 15:27 | |
*** andreas_s has joined #openstack-infra | 15:27 | |
*** kiennt26 has quit IRC | 15:27 | |
*** rbrndt has joined #openstack-infra | 15:28 | |
*** felipemonteiro has quit IRC | 15:29 | |
*** felipemonteiro has joined #openstack-infra | 15:30 | |
*** lukebrowning has quit IRC | 15:32 | |
*** lukebrowning has joined #openstack-infra | 15:33 | |
*** andreas_s has quit IRC | 15:34 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Create project-templates for sphinx building with python3 https://review.openstack.org/529093 | 15:34 |
jeblair | andreykurilin: are you asking about ansible roles? | 15:34 |
*** andreas_s has joined #openstack-infra | 15:34 | |
andreykurilin | jeblair: yes. I already tried guessing, but it looks like wasting compute resources without any progress. I have a patch - https://review.openstack.org/#/c/524637 . It adds one role to existing job | 15:36 |
*** ykarel has joined #openstack-infra | 15:37 | |
AJaeger | ttx, for releases repo, do something similar to 529093 | 15:37 |
AJaeger | ttx, or reuse the template | 15:37 |
*** lukebrowning has quit IRC | 15:38 | |
*** eharney has joined #openstack-infra | 15:38 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Create project-templates for sphinx building with python3 https://review.openstack.org/529093 | 15:39 |
jeblair | andreykurilin: looks like you've got it running -- what's the problem? | 15:39 |
andreykurilin | jeblair: I have post_failure status | 15:40 |
* andreykurilin jeblair: http://logs.openstack.org/37/524637/6/check/rally-install-ubuntu-xenial/f4aff39/ | 15:40 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use python3 sphinx building for governance https://review.openstack.org/529095 | 15:41 |
andreykurilin | also, there is nothing at ara | 15:41 |
AJaeger | ttx, and 529095 fixes governance ^ | 15:42 |
jeblair | andreykurilin: hrm, i agree, there should be more output in that log telling you what went wrong (cc mordred) | 15:42 |
jeblair | andreykurilin: we have some longstanding bugs about ansible error output not making it into the logs | 15:43 |
jeblair | andreykurilin: i'll see if i can find something on the executor | 15:43 |
andreykurilin | jeblair: and I expect to see custom index.html at the top dir of logs (we did it before in legacy jobs), but there is nothing, so it means that my role is wrong or it was not executed correctly | 15:43 |
andreykurilin | jeblair: thanks | 15:43 |
*** lukebrowning has joined #openstack-infra | 15:44 | |
*** liusheng has quit IRC | 15:45 | |
*** liusheng has joined #openstack-infra | 15:45 | |
*** lukebrowning has quit IRC | 15:48 | |
*** lukebrowning has joined #openstack-infra | 15:50 | |
ttx | AJaeger: looks like it fails due to a recursion in job definition in https://review.openstack.org/#/c/529093/2 ? | 15:50 |
*** jtomasek has quit IRC | 15:50 | |
ttx | smcginnis: see for fixing the releases repo ^ | 15:51 |
*** edmondsw has joined #openstack-infra | 15:52 | |
*** salv-orlando has quit IRC | 15:52 | |
*** salv-orlando has joined #openstack-infra | 15:53 | |
smcginnis | Watching | 15:53 |
ttx | smcginnis: probably needing something like https://review.openstack.org/#/c/529095/ | 15:53 |
ttx | But 529093 needs to be fixed first | 15:53 |
*** jamesmcarthur has joined #openstack-infra | 15:54 | |
*** lukebrowning has quit IRC | 15:55 | |
jeblair | andreykurilin: this is the error: http://paste.openstack.org/show/629322/ | 15:55 |
jeblair | andreykurilin: the problem is subtle | 15:55 |
jeblair | andreykurilin: if you look at the first line, it has: ...ansible/post_playbook_0/role_0: | 15:55 |
jeblair | andreykurilin: followed by: ansible/post_playbook_0/role_1/project-config/roles: | 15:56 |
*** edmondsw has quit IRC | 15:56 | |
jeblair | andreykurilin: the second element (role_1) has a project name in it | 15:56 |
*** lukebrowning has joined #openstack-infra | 15:56 | |
jeblair | andreykurilin: the first element, which is the element in the path for this job (it orders them backwards with this repo first, and others later), has no project name | 15:57 |
*** salv-orlando has quit IRC | 15:57 | |
*** xarses_ has joined #openstack-infra | 15:57 | |
jeblair | andreykurilin: that means that zuul thought this repo was a "bare role" -- explained at https://docs.openstack.org/infra/zuul/feature/zuulv3/user/config.html#attr-job.roles | 15:57 |
jeblair | andreykurilin: it thought that because there is a tasks/ directory at the root of the repo | 15:57 |
andreykurilin | jeblair: hm... based on the first like (PATH), I can put the roles in openstack/rally/tests/ci/playbooks/base_new/roles and it should become discoverable. right? | 15:59 |
jeblair | andreykurilin: in other words, this repo is not structured to be an ansible shared role | 15:59 |
jeblair | andreykurilin: yes | 15:59 |
jeblair | andreykurilin: in that case, the role would only be able to be used by that playbook | 15:59 |
andreykurilin | jeblair: ok. so `zuul:openstack/rally` become redundant | 16:00 |
jeblair | andreykurilin: yes -- it's actually redundant either way -- the job's repo is always automatically added as a role | 16:00 |
jeblair | andreykurilin: (it's harmless to have it) | 16:00 |
jeblair | (the last pgraph of that doc describes that) | 16:01 |
*** lukebrowning has quit IRC | 16:01 | |
andreykurilin | I read the docs, but it looks like I missed this. | 16:01 |
jeblair | they are... dense. | 16:01 |
andreykurilin | jeblair: one more question. if such role will be used by a job and some other project will try to use such job, what will happen? | 16:02 |
jeblair | andreykurilin: should be fine | 16:02 |
*** lukebrowning has joined #openstack-infra | 16:02 | |
andreykurilin | jeblair: cool, thanks | 16:03 |
openstackgerrit | Thierry Carrez proposed openstack-infra/openstack-zuul-jobs master: Create project-templates for sphinx building with python3 https://review.openstack.org/529093 | 16:03 |
AJaeger | ttx, yeah, that's the usual dance with project-config - we need to merge one change, recheck others, merge, recheck... | 16:04 |
ttx | Ah/ | 16:04 |
andreykurilin | jeblair: the last question, I hope. Does it possible to write something "abort check" to stop all the jobs for a patch? | 16:05 |
ttx | I thought you added -python3 to the publish job erroneously and pushed a new patchset... please check | 16:05 |
ttx | AJaeger: https://review.openstack.org/#/c/529093/ | 16:05 |
jeblair | andreykurilin: not at the moment | 16:05 |
jeblair | andreykurilin: a new patchset will abort them (but of course, start them again with the new patchset) | 16:06 |
AJaeger | ttx, your change is wrong - the depends-on defines that job. and since project-config is special, Zuul does not know about it... | 16:06 |
ttx | damn | 16:06 |
ttx | I'll revert | 16:06 |
jeblair | andreykurilin: if all your jobs are defined in the zuul.yaml file in the repo you're changing, you can temporarily remove jobs you aren't interested in while you work on a change | 16:06 |
AJaeger | ttx, so the change will pass with that one - but then once the project-config change is in, we have to do change it again... | 16:07 |
andreykurilin | jeblair: :( ok. hope that in near future, we will port all the jobs to be hosted in our repo, so it will be possible to specify what jobs to launch while experimenting. | 16:07 |
jeblair | andreykurilin: yeah, once you get to that point, experimenting becomes a lot easier | 16:07 |
andreykurilin | jeblair: and less expensive :) | 16:07 |
*** lukebrowning has quit IRC | 16:07 | |
AJaeger | ttx, so if we don't merge https://review.openstack.org/#/c/521557/ quickly, I would use your variant - better a half-working solution (publishing broken but check/gate working) than everything broken as is now. | 16:07 |
*** apetrich has joined #openstack-infra | 16:08 | |
*** lukebrowning has joined #openstack-infra | 16:09 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add an authoritative hidden master https://review.openstack.org/528438 | 16:11 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 16:11 |
*** ykarel has quit IRC | 16:11 | |
ttx | AJaeger: so should I revert it? | 16:12 |
AJaeger | ttx, I still have it locally here - let me push... | 16:12 |
ttx | ok! That other patch looks close enough | 16:13 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Create project-templates for sphinx building with python3 https://review.openstack.org/529093 | 16:13 |
*** lukebrowning has quit IRC | 16:13 | |
AJaeger | ttx, want me to do a change for releases repo? | 16:14 |
*** lukebrowning has joined #openstack-infra | 16:15 | |
ttx | AJaeger: if you're on a roll, please do. Otherwise I expect smcginnis to catch it (he has another change in flight he needs to abandon) | 16:17 |
clarkb | lbragstad: AJaeger mordred http://logs.openstack.org/68/512468/6/check/build-openstack-sphinx-docs/e724ee3/ara/ still not happy, in this case looks like it wants some ldap library for autodocing that isn't installed | 16:17 |
clarkb | may need to add a dependency? | 16:17 |
*** hashar is now known as hasharAway | 16:18 | |
lbragstad | clarkb: https://review.openstack.org/#/c/528960/5 | 16:18 |
AJaeger | clarkb: as lbragstad says. While I'm not happy with it, I have no other solution. Waiting for mordred to comment | 16:19 |
AJaeger | clarkb: it's all this [extra] referencing setup.cfg | 16:19 |
AJaeger | ttx, ok, will fix | 16:20 |
*** lukebrowning has quit IRC | 16:20 | |
clarkb | AJaeger: lbragstad was the dep missing from the normal project requirements? | 16:20 |
lbragstad | i think it's optional | 16:21 |
lbragstad | from a keystone perspective | 16:21 |
*** lukebrowning has joined #openstack-infra | 16:21 | |
AJaeger | lbragstad: I wonder why it's optional from keystone but required for docs building... | 16:22 |
AJaeger | lbragstad: I know you can do this, just looks odd | 16:22 |
lbragstad | we don't have a hard dependency on it since keystone can be run without ldap | 16:24 |
AJaeger | frickler: rereading 521557 again: It does not use the new sphinx role everywhere and just adds this for a few specific repos as you suggested. 525734 is intended as the change to switch the rest. | 16:24 |
*** salv-orlando has joined #openstack-infra | 16:25 | |
*** andreas_s has quit IRC | 16:26 | |
AJaeger | ttx, https://review.openstack.org/529104 | 16:26 |
*** jamesmcarthur has quit IRC | 16:26 | |
*** lukebrowning has quit IRC | 16:26 | |
*** andreas_s has joined #openstack-infra | 16:26 | |
*** jamesmcarthur has joined #openstack-infra | 16:27 | |
ttx | AJaeger: should we just do a similar change in governance ? i.e. using zuul.d ? | 16:28 |
*** lukebrowning has joined #openstack-infra | 16:28 | |
AJaeger | ttx, 529095 fixes governance | 16:29 |
ttx | AJaeger: ok, was just wondering if that would avoid the dependency and allow to quickfix | 16:30 |
ttx | (to do a local .zuul.yaml change) | 16:31 |
*** janki has quit IRC | 16:31 | |
*** jamesmcarthur has quit IRC | 16:31 | |
ttx | avoids using the template and all | 16:31 |
AJaeger | ttx, yes, it would. So, you could do as hotfix | 16:32 |
ttx | AJaeger: ok thanks. I'd like to unblock asap | 16:32 |
*** lukebrowning has quit IRC | 16:32 | |
AJaeger | ttx but publishing will not work this way! | 16:32 |
ttx | Ah! again | 16:32 |
ttx | I'm confused | 16:32 |
smcginnis | ttx, AJaeger: Looks like for releases we will need to get sphinxocntrib.datatemplates added to g-r first, then get that landed in releases to include it in doc/requirements.txt. | 16:33 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add an authoritative hidden master https://review.openstack.org/528438 | 16:33 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 16:33 |
smcginnis | I will un-squash that change with the io.open one. | 16:33 |
openstackgerrit | Jens Harbott (frickler) proposed openstack-infra/devstack-gate master: Add neutron-tempest-plugin-api job https://review.openstack.org/529000 | 16:33 |
ttx | Oh well, probably too late for me to fix this today | 16:33 |
AJaeger | smcginnis: argh ;( | 16:33 |
ttx | I'll wait until those changes make it to project-config | 16:33 |
*** lukebrowning has joined #openstack-infra | 16:34 | |
AJaeger | ttx, I'll advocate to push this through today. | 16:34 |
*** armaan has joined #openstack-infra | 16:37 | |
AJaeger | jeblair: for translations, I want to change the templates. Can I have two post jobs like in http://paste.openstack.org/show/629326/ - I might then add translation-jobs-master to all translated repos; and have translation-jobs-ocata to some added. So, that we run the push job only for master and stable/ocata (if both templates are used). | 16:37 |
AJaeger | clarkb: that's basically implementing your suggestion ^ | 16:37 |
*** r-daneel has quit IRC | 16:38 | |
*** lukebrowning has quit IRC | 16:39 | |
AJaeger | clarkb: and will not use a single line of change to project-config and how we do branching ;) | 16:39 |
*** andreas_s has quit IRC | 16:40 | |
*** jamesmcarthur has joined #openstack-infra | 16:41 | |
clarkb | AJaeger: went with different josb instead? | 16:41 |
*** gyee has joined #openstack-infra | 16:42 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Rework translation templates https://review.openstack.org/529111 | 16:43 |
AJaeger | clarkb: different *templates* ;) Please review ^ | 16:43 |
fungi | do we actually enforce requirements matching on doc/requirements.txt? | 16:43 |
fungi | wondering why sphinxcontrib.datatemplates would need to go in global reqs | 16:43 |
AJaeger | fungi: we currently do - I was too quick ;( Some discussed here to skip that one... | 16:43 |
*** annp has quit IRC | 16:44 | |
AJaeger | fungi, what we might want to do: sync but not enforce. That's a bit more work for the scripts... | 16:44 |
AJaeger | fungi: meaning: I don't think they can handle this case today, so needs extending of scripts. | 16:45 |
smcginnis | Seems like it would make sense to skip docs requirements. | 16:45 |
fungi | i guess for the specific purpose of ensuring runtime compatibility between projects we could scale back to only top-level requirements.txt matching global-reqs even, though anything not in the transitive global requirements set also misses out on constraints pinning | 16:45 |
clarkb | fungi: wouldn't we want to enforce those requirements for all the of the same reasons we enforce them otherwise? | 16:45 |
clarkb | (ensures that packagers know what to package and don't have to package a bunch of redundant/duplicate functionality, license checks, and so on) | 16:45 |
fungi | clarkb: depends on whether we consider those reasons to only be important for run-time dependencies | 16:46 |
clarkb | docs somewhat complicate that as html/js/css are runtime in many cases | 16:47 |
jeblair | AJaeger: i think your proposed change would work. you could also move the use of the translation-jobs template in-repo, and only put it on the branches you want. | 16:47 |
clarkb | so even if we said its only run time deps we wouldn't be avoided that in docs builds | 16:48 |
*** lukebrowning has joined #openstack-infra | 16:48 | |
clarkb | *avoiding | 16:48 |
AJaeger | jeblair: I don't want that yet - the translators tranlate only master and last branch, so having a central place to disable/enable branches is important | 16:49 |
AJaeger | jeblair: could you review then how I did this in https://review.openstack.org/529111, please? | 16:49 |
*** lukebrowning has quit IRC | 16:52 | |
*** andreas_s has joined #openstack-infra | 16:53 | |
*** lukebrowning has joined #openstack-infra | 16:54 | |
*** lucasagomes is now known as lucas-afk | 16:56 | |
*** lukebrowning has quit IRC | 16:59 | |
* mordred waves to everyone | 16:59 | |
AJaeger | mordred: do you have time to chat? | 16:59 |
AJaeger | mordred: https://etherpad.openstack.org/p/merge-updated-pti-topic is updated, we need https://review.openstack.org/#/c/521557 to go in for post jobs, then https://review.openstack.org/529093 and https://review.openstack.org/529095 for repos with python3 for sphinx building. Could you review the later two - and should we ask for merging of 521557? | 17:01 |
*** salv-orlando has quit IRC | 17:01 | |
*** salv-orl_ has joined #openstack-infra | 17:01 | |
frickler | just noticed that eavesdrop is posting unexpected links: http://eavesdrop01.openstack.org/meetings/neutron_ci/2017/neutron_ci.2017-12-19-16.01.html | 17:02 |
frickler | shouldn't that be eavesdrop.o.o? | 17:02 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add an authoritative hidden master https://review.openstack.org/528438 | 17:02 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 17:02 |
*** andreas_s has quit IRC | 17:03 | |
*** Goneri has joined #openstack-infra | 17:03 | |
AJaeger | frickler: I agree, it should. | 17:04 |
*** iyamahat_ has quit IRC | 17:04 | |
AJaeger | frickler: links work but once we setup eavesdrop02, it might not... | 17:04 |
*** jascott1 has joined #openstack-infra | 17:05 | |
mordred | AJaeger: yes - I think https://review.openstack.org/#/c/521557 is ready to go in | 17:06 |
openstackgerrit | Fabien Boucher proposed openstack-infra/zuul feature/zuulv3: Attempt to improve tenant configuration in case of config issue https://review.openstack.org/529060 | 17:06 |
*** r-daneel has joined #openstack-infra | 17:07 | |
*** slaweq has quit IRC | 17:08 | |
*** bh526r has quit IRC | 17:08 | |
jeblair | infra-root: frickler's issue above is an important error we should fix asap | 17:09 |
AJaeger | infra-root, can we merge 521557 , please? This is one step in bringing post jobs and check/gate together | 17:09 |
mordred | AJaeger: the other two lgtm | 17:09 |
clarkb | jeblair: frickler its just from the irc bot side right? apache isn't redirecting you from eavesdrop.o. to eavesdrop01.o.o is it? | 17:10 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Set eavesdrop vhost https://review.openstack.org/529118 | 17:10 |
jeblair | infra-root: ^ i think that will fix future occurances. we will need to edit files on disk to correct content generated with wrong links. | 17:11 |
jeblair | if there is any -- i don't know | 17:11 |
jeblair | it's possible this only affects what the bot says in channel? | 17:11 |
*** ccamacho has left #openstack-infra | 17:11 | |
clarkb | looks like there is maybe one link written to disk that is affected, the link back to the full logs | 17:12 |
jeblair | clarkb: oh i think that's relative | 17:12 |
AJaeger | mordred: thanks. can you help pushing those in? I'll be offline now for an hour or two | 17:12 |
mordred | AJaeger: yup | 17:13 |
mordred | jeblair: +A | 17:13 |
*** jascott1 has quit IRC | 17:13 | |
openstackgerrit | Fabien Boucher proposed openstack-infra/zuul feature/zuulv3: Attempt to improve tenant configuration in case of config issue https://review.openstack.org/529060 | 17:14 |
fungi | do we need someone to check the files on disk for incorrect hyperlinks and fix them? i can take care of that once the fix is in production | 17:15 |
openstackgerrit | Fabien Boucher proposed openstack-infra/zuul feature/zuulv3: Attempt to improve tenant config loading in case of config issue https://review.openstack.org/529060 | 17:17 |
openstackgerrit | Merged openstack-infra/elastic-recheck master: Track tripleo timeout or canceled deployments https://review.openstack.org/513524 | 17:17 |
clarkb | fungi: I'm not finding any cases of that through my web browser | 17:18 |
clarkb | I think it is just what the bot says in channel | 17:18 |
clarkb | so the meeting logs themselves may have the wrong links in the log | 17:18 |
fungi | cool. i'm doing a recursive grep just in case | 17:19 |
frickler | jeblair: does your patch change /var/lib/meetbot/openstack.org/ircmeeting/meetingLocalConfig.py ? cause that contains the bad logUrlPrefix | 17:20 |
fungi | clarkb: bad news... http://eavesdrop.openstack.org/meetings/neutron_ci/2017/neutron_ci.2017-12-12-16.00.txt | 17:21 |
clarkb | frickler: yes https://git.openstack.org/cgit/openstack-infra/puppet-meetbot/tree/templates/meetingLocalConfig.py.erb | 17:21 |
fungi | er, wrong link | 17:21 |
clarkb | fungi: ok I was confused for a second | 17:22 |
mordred | fungi: I was about to say - that seemed like good news :) | 17:22 |
fungi | http://eavesdrop.openstack.org/meetings/neutron_ci/2017/neutron_ci.2017-12-19-16.01.txt is the one i meant :( | 17:22 |
*** jascott1 has joined #openstack-infra | 17:22 | |
*** trown|ruck is now known as trown|lunch | 17:22 | |
fungi | i can work on categorizing places where it leaked in and stream edit the affected files | 17:22 |
*** dsariel has joined #openstack-infra | 17:22 | |
fungi | but that's the first one that jumped out at me in my grep | 17:23 |
clarkb | ah ok its the same message printed in irc that is printed in those logs | 17:23 |
mordred | frickler, clarkb, fungi, jeblair: when you have a sec, could you also land https://review.openstack.org/#/c/521557/7 - we need it to unbreak governance repo builds | 17:23 |
*** andreas_s has joined #openstack-infra | 17:26 | |
*** ralonsoh has quit IRC | 17:30 | |
pabelanger | mordred: when do you expect the next shade release? | 17:32 |
openstackgerrit | Alex Schultz proposed openstack-infra/elastic-recheck master: Query for 1719469 https://review.openstack.org/507294 | 17:32 |
mordred | pabelanger: I need to land one revert patch (fixing merge conflict right now) then will cut the release | 17:32 |
pabelanger | mordred: ack, thanks | 17:33 |
mordred | pabelanger: while I've got you, could I get you to review https://review.openstack.org/#/c/521557/7 please? | 17:33 |
pabelanger | looking | 17:33 |
*** lukebrowning has joined #openstack-infra | 17:34 | |
*** jamesmcarthur has quit IRC | 17:34 | |
pabelanger | mordred: +3 | 17:35 |
*** jamesmcarthur has joined #openstack-infra | 17:35 | |
mordred | pabelanger: thank you | 17:35 |
clarkb | mordred: pabelanger we need to restart nodepool launchers to pick up my bug fix and the quota stuff | 17:35 |
clarkb | do we want to lump shade into that too? or should we restart now then do shade after things settle in? | 17:35 |
mordred | clarkb: yah - let's lump the shade release into that too | 17:36 |
clarkb | ok | 17:36 |
pabelanger | ++ | 17:36 |
pabelanger | my afteroon is clear so able to support | 17:36 |
mordred | awesome | 17:36 |
clarkb | alright, I'm likely to be afk ish this afternoon to help smoke salmon | 17:36 |
pabelanger | Going to also look at getting kdc03 online (new master) and see what is need to migrate kdc01 to it | 17:36 |
clarkb | but will have laptop too | 17:36 |
* fungi wants smoked salmon now | 17:37 | |
fungi | i candied and smoked salmon cheeks here a few months ago and they came out amazingly | 17:38 |
fungi | er, no, it was salmon bellies | 17:39 |
*** jamesmcarthur has quit IRC | 17:39 | |
*** lukebrowning has quit IRC | 17:39 | |
mordred | fungi: mmm. salmon bellies | 17:40 |
*** edmondsw has joined #openstack-infra | 17:40 | |
clarkb | this is a freezer full of the filets that didn't get eaten after fishing trips. I expect a bunch of it will be given away at family gatherings over the next week or so | 17:40 |
mordred | nice | 17:40 |
clarkb | Problem is I'm so sick of salmon now :) | 17:40 |
fungi | you do get a lot more of them there than we do on this end of the rock | 17:41 |
mordred | clarkb: I'll trade you - I've got plenty of beef in these parts | 17:41 |
fungi | but atlantic salmon are still pretty awesome | 17:41 |
openstackgerrit | Merged openstack-infra/project-config master: Use the new sphinx PTI for special publish jobs https://review.openstack.org/521557 | 17:42 |
*** andreas_s has quit IRC | 17:43 | |
*** Apoorva has joined #openstack-infra | 17:43 | |
*** Apoorva has quit IRC | 17:43 | |
*** Apoorva has joined #openstack-infra | 17:44 | |
andreykurilin | folks. what is the right way to copy some file <project_src>/foo.file to the results of jobs? I'm trying to use copy task with `src: "{{ zuul.executor.src_root }}foo.file"` but it doesn't work. Any suggestions? | 17:44 |
*** edmondsw has quit IRC | 17:44 | |
mordred | andreykurilin: the file is created as part of the job? | 17:44 |
andreykurilin | mordred: no. just constant file | 17:45 |
mordred | ah. hrm. got a link? | 17:45 |
*** lukebrowning has joined #openstack-infra | 17:45 | |
andreykurilin | mordred: https://review.openstack.org/#/c/524637/12/tests/ci/playbooks/base_new/roles/fetch-rally-reports/tasks/main.yaml | 17:46 |
andreykurilin | mordred: https://github.com/openstack/rally/blob/master/rally/ui/templates/ci/index.html | 17:46 |
andreykurilin | mordred: and the results of that role: http://logs.openstack.org/37/524637/12/check/rally-install-ubuntu-xenial/f28d3fd/ara/ | 17:46 |
*** derekh has quit IRC | 17:47 | |
*** aviau has quit IRC | 17:47 | |
*** aviau has joined #openstack-infra | 17:47 | |
mordred | andreykurilin: k. so - the copy task is for copying files between computers - and that's going to be going in the wrong direction ... | 17:48 |
*** andreas_s has joined #openstack-infra | 17:48 | |
mordred | andreykurilin: I think instead you should maybe to synchronize: mode: pull src: "{{ zuul.project.src_dir }}/rally/ui/templates/ci/index.html" dest: "{{ zuul.executor.log_root }}/index.html" | 17:49 |
andreykurilin | mordred: so it should look something like https://review.openstack.org/#/c/524637/9/tests/ci/playbooks/base_new/roles/fetch-rally-reports/tasks/main.yaml | 17:49 |
*** catintheroof has quit IRC | 17:50 | |
andreykurilin | it was several attempts before | 17:50 |
*** lukebrowning has quit IRC | 17:50 | |
mordred | andreykurilin: yes - but zuul.executor.src_root shoujld be zuul.project.src_dir | 17:50 |
mordred | andreykurilin: there is a stack of stuff we'll hopefully get landed soon that should make this easier - but for now I think if you just do it like that patch but wiht zuul.project.src_dir it should work for you | 17:51 |
*** catintheroof has joined #openstack-infra | 17:51 | |
*** lukebrowning has joined #openstack-infra | 17:52 | |
andreykurilin | mordred: thanks. last time, it found `--ignore-missing-args` option unknown and I started to find another solution. | 17:52 |
mordred | andreykurilin: weird - we use --ignore-missing-args like that in other places | 17:53 |
fungi | clarkb: the good news is that the only occurrences i'm finding are in the plaintext versions of the meeting minutes, not in the html minutes nor either version of the meeting logs | 17:56 |
mnaser | i have some interesting questions to the folks who previously operated osic, does anyone in infra know who i can throw some interesting questions about the deployment? | 17:56 |
fungi | well, also the channel logs, which are probably not worth fixing | 17:56 |
andreykurilin | mordred: yes, I copied it from somewhere. unfortunately, I'm trying to resubmitte new patch-sets as soon as target job failed to reduce the cost of other jobs, so there is no posted link to results of failure with `--ignore-missing-args` :( | 17:56 |
fungi | mnaser: you want cloudnull | 17:56 |
*** lukebrowning has quit IRC | 17:56 | |
mordred | pabelanger: wanna do another one? https://review.openstack.org/#/c/529093 | 17:57 |
mnaser | fungi: thank you, cloudnull, i have a question in #openstack-operators that i'd be interested in hearing more from you about (or free free to pm, tia :D) | 17:57 |
*** jtomasek has joined #openstack-infra | 17:58 | |
*** lukebrowning has joined #openstack-infra | 17:58 | |
pabelanger | mordred: +3 | 17:59 |
*** raissa has joined #openstack-infra | 18:00 | |
*** raissa has quit IRC | 18:01 | |
*** r-daneel_ has joined #openstack-infra | 18:01 | |
*** raissa has joined #openstack-infra | 18:01 | |
*** r-daneel has quit IRC | 18:01 | |
*** r-daneel_ is now known as r-daneel | 18:01 | |
*** raissa has quit IRC | 18:01 | |
*** andreas_s has quit IRC | 18:02 | |
*** lukebrowning has quit IRC | 18:03 | |
mordred | pabelanger: \o/ | 18:03 |
*** sambetts is now known as sambetts|afk | 18:03 | |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Create project-templates for sphinx building with python3 https://review.openstack.org/529093 | 18:03 |
*** efoley has quit IRC | 18:04 | |
*** catintheroof has quit IRC | 18:07 | |
*** catintheroof has joined #openstack-infra | 18:08 | |
*** dsariel has quit IRC | 18:09 | |
*** lukebrowning has joined #openstack-infra | 18:09 | |
*** raissa has joined #openstack-infra | 18:10 | |
clarkb | fungi: ya the actual channel logs probably don't want to be changed | 18:14 |
kmalloc | o/ | 18:14 |
*** lukebrowning has quit IRC | 18:14 | |
kmalloc | oh wait nvm | 18:14 |
kmalloc | ugh | 18:14 |
fungi | kmalloc: you're welcome here any time! ;) | 18:14 |
*** harlowja has joined #openstack-infra | 18:15 | |
kmalloc | :P | 18:15 |
*** lukebrowning has joined #openstack-infra | 18:15 | |
*** dsariel has joined #openstack-infra | 18:17 | |
clarkb | mwhahaha: for the e-r query change that just went up, dropping that into kibana returns no results for me | 18:18 |
clarkb | mwhahaha: 507294 specifically | 18:18 |
*** lukebrowning has quit IRC | 18:20 | |
*** links has quit IRC | 18:23 | |
AJaeger | one more change needed to unbreak governance building. clarkb, pabelanger, please review https://review.openstack.org/#/c/529095 | 18:24 |
*** armaan has quit IRC | 18:24 | |
AJaeger | mordred: do you want to keep your IRC priviledges? It's awkward to see "@mordred" here ;) | 18:24 |
AJaeger | mordred: did you see https://review.openstack.org/528960 ? Do you have a better solution for that one? | 18:26 |
*** dsariel has quit IRC | 18:27 | |
*** rbrndt has quit IRC | 18:28 | |
AJaeger | dmsimard: tristanC is fine with https://review.openstack.org/527313 as one step - could you change your -1 to a +2A, please? | 18:30 |
*** rmcall has joined #openstack-infra | 18:30 | |
*** ChanServ sets mode: -o mordred | 18:31 | |
*** lukebrowning has joined #openstack-infra | 18:31 | |
mordred | AJaeger: thanks for the reminder :) | 18:31 |
*** bobh has quit IRC | 18:32 | |
*** jpena is now known as jpena|off | 18:33 | |
mordred | AJaeger: what's the issue with that patch? (also, wow that's a LOT of doc requirements) | 18:33 |
AJaeger | mordred: doc/requirements.txt includes the [extra] dependencies so that keystone can be installed. | 18:35 |
AJaeger | mordred: I hate those [extra] dependencies via setup.cfg but we cannot just ignore them. So, the change works. But I wonder whether there's a superior solution than putting all optional parts into doc/requirement if you want to autodoc? | 18:36 |
*** lukebrowning has quit IRC | 18:36 | |
mordred | AJaeger: ah - yah. I don't have a better solution there - I totally see the benefit of having the requirements broken out into different extras for optional depends, but based on how sphinx imports things you really do have to install the union of them all ... | 18:37 |
AJaeger | mordred: and I'm out of wits on why https://review.openstack.org/#/c/528735/ fails. do you have an idea? | 18:37 |
AJaeger | mordred: ok, so be it - thanks | 18:37 |
mordred | AJaeger: OR - you have to do *something* in the code itself to perhaps protect against import errors | 18:37 |
mordred | AJaeger: so that sphinx can import and doc the code - maybe that's a thing we can play with | 18:37 |
*** lukebrowning has joined #openstack-infra | 18:37 | |
*** andreas_s has joined #openstack-infra | 18:39 | |
*** iyamahat has joined #openstack-infra | 18:40 | |
AJaeger | maybe ;) | 18:40 |
*** trown|lunch is now known as trown | 18:40 | |
*** trown is now known as trown|ruck | 18:40 | |
AJaeger | mordred: do you hae any idea on taskflow? I'm not seeing why it fails - and works without the change ;( | 18:42 |
AJaeger | I can reproduce locally with tox -e docs | 18:42 |
mordred | AJaeger: cool - I'm doing the same | 18:42 |
*** lukebrowning has quit IRC | 18:42 | |
*** andreas_s has quit IRC | 18:43 | |
*** lukebrowning has joined #openstack-infra | 18:44 | |
AJaeger | mordred: pydotplus is not in doc/requirements.txt - checking whether adding that one helps | 18:47 |
*** inc0 has quit IRC | 18:48 | |
* AJaeger run a diff between the pip freeze output and looked for hints | 18:48 | |
AJaeger | does not help ;8 | 18:48 |
*** inc0 has joined #openstack-infra | 18:48 | |
mwhahaha | clarkb: right so that's a bug when the ovb the clouds have problems spinning up envs. We don't have to necessarily merge that but it's one of those nice to know when it happens | 18:48 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 18:48 |
*** lukebrowning has quit IRC | 18:48 | |
clarkb | mwhahaha: I see so no current results isn't suprising? | 18:48 |
mwhahaha | clarkb: no it's not | 18:49 |
*** lukebrowning has joined #openstack-infra | 18:50 | |
clarkb | Infra meeting in ~10 minutes over in #openstack-meeting | 18:50 |
AJaeger | mordred: does not help ;( I'm puzzled | 18:51 |
openstackgerrit | Paul Belanger proposed openstack-infra/nodepool feature/zuulv3: requirements: remove paramiko <2.0 cap https://review.openstack.org/528719 | 18:54 |
clarkb | mwhahaha: I'll let mriedem weigh in since he already did a review on it, but looks fine to me if that is expected. | 18:55 |
*** lukebrowning has quit IRC | 18:55 | |
mriedem | ? | 18:56 |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: requirements: remove paramiko <2.0 cap https://review.openstack.org/528729 | 18:56 |
*** dtantsur is now known as dtantsur|afk | 18:56 | |
clarkb | mriedem: https://review.openstack.org/#/c/507294/2 | 18:56 |
*** catintheroof has quit IRC | 18:56 | |
*** wolverineav has quit IRC | 18:56 | |
*** lukebrowning has joined #openstack-infra | 18:56 | |
AJaeger | mordred: adding *all* missing requirements fixes it ;( So, same bad change as for keystone.... | 18:56 |
*** r-daneel has quit IRC | 18:57 | |
AJaeger | mordred: so, I'll have an IMHO ugly patch in a bit... | 18:57 |
*** armaan has joined #openstack-infra | 18:57 | |
*** wolverineav has joined #openstack-infra | 18:58 | |
mriedem | clarkb: mwhahaha: why would we merge a query that doesn't have any hits? | 18:58 |
clarkb | mriedem: because the issue happens occasionally and knowing when that happens is nice? | 18:58 |
mwhahaha | so when it happens again we know why | 18:58 |
mwhahaha | ie regressions | 18:58 |
mwhahaha | or enviromental failures | 18:58 |
mwhahaha | so folks don't spin their wheels on a known problem that has happened in the past | 18:59 |
fungi | and now is the time on infra when we meet | 19:00 |
*** jokke_ has quit IRC | 19:00 | |
*** lukebrowning has quit IRC | 19:01 | |
*** jokke_ has joined #openstack-infra | 19:01 | |
*** andreas_s has joined #openstack-infra | 19:01 | |
AJaeger | ianw, good morning: dmsimard asked that you review https://review.openstack.org/525658 - could you do so, please? His -1 is procedural... | 19:02 |
*** sbezverk has joined #openstack-infra | 19:02 | |
mordred | AJaeger: ok. seems like a thing for us to consider | 19:03 |
*** r-daneel has joined #openstack-infra | 19:03 | |
*** lukebrowning has joined #openstack-infra | 19:05 | |
ianw | AJaeger: done ... i'll try to update the testing on the other job ... but it's my last day until early jan so don't hold your breath :) not urgent, anyway | 19:06 |
AJaeger | dmsimard: ianw is happy with https://review.openstack.org/#/c/525658/ - could you remove your -1, please? ianw ,thanks for reviewing | 19:06 |
AJaeger | yes, not urgent... | 19:06 |
AJaeger | just nice ;) | 19:06 |
*** lukebrowning has quit IRC | 19:10 | |
mordred | AJaeger: what if we added an optional zuul job variable that is a list of extras ... so similar to how taskflow has in its original tox.ini file: | 19:10 |
mordred | AJaeger: deps = .[test,workers,zookeeper,database,redis,eventlet,doc] | 19:10 |
mordred | AJaeger: for projects that have decided to organize their dependencies in this way, the could just say "vars: some_good_name: workers,zookeeper,database,redis,eventlet,doc" | 19:11 |
*** lukebrowning has joined #openstack-infra | 19:11 | |
*** iyamahat has quit IRC | 19:11 | |
mordred | AJaeger: and then in install-if-python role, instead of just pip install . - we could do pip install .[{{some_good_name}}] | 19:12 |
clarkb | dmsimard: can you join us in #openstack-meeting to talk about your meeting agenda items | 19:12 |
AJaeger | mordred: that will not work for post jobs ;/ the docs publishing job is final | 19:12 |
mordred | AJaeger: oh - right | 19:13 |
mordred | AJaeger: well - it'll work for post jobs once we get the base job rework finished | 19:14 |
*** shardy has quit IRC | 19:14 | |
mordred | AJaeger: so maybe for now we just make really big doc/requirements.txt files, but with a TODO to shrink them again once we've got the base job rework done | 19:14 |
*** andreas_s has quit IRC | 19:15 | |
*** jamesmcarthur has joined #openstack-infra | 19:15 | |
AJaeger | mordred: yeah | 19:15 |
*** lukebrowning has quit IRC | 19:16 | |
*** lukebrowning has joined #openstack-infra | 19:18 | |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck master: Remove old queries for fixed bugs https://review.openstack.org/529136 | 19:18 |
*** jamesmcarthur has quit IRC | 19:20 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/infra-manual master: Better explain needed patchsets https://review.openstack.org/528375 | 19:21 |
AJaeger | jeblair: thanks for spotting, fixed ^ | 19:21 |
openstackgerrit | Merged openstack-infra/system-config master: Set eavesdrop vhost https://review.openstack.org/529118 | 19:22 |
*** lukebrowning has quit IRC | 19:23 | |
*** lukebrowning has joined #openstack-infra | 19:24 | |
jeblair | AJaeger: now the question is -- should we merge that, or WIP it again pending the merge of 513368? :) | 19:24 |
jeblair | AJaeger: documentation driven development! if you document the terrible process, it gets better :) | 19:25 |
AJaeger | jeblair: I'm fine if my documents results in great ideas and better code ;) | 19:25 |
fungi | true that writing down the process makes it easier to see how terrible it is ;) | 19:25 |
AJaeger | so, if we merge that, let's abandon my change - once we *use* that change. | 19:25 |
* AJaeger volunteers to add the implicit target and remove system-required and update docs... | 19:26 | |
*** jamesmcarthur has joined #openstack-infra | 19:27 | |
*** edmondsw has joined #openstack-infra | 19:28 | |
*** iyamahat has joined #openstack-infra | 19:28 | |
*** lukebrowning has quit IRC | 19:29 | |
*** catintheroof has joined #openstack-infra | 19:29 | |
*** rbrndt has joined #openstack-infra | 19:29 | |
*** lukebrowning has joined #openstack-infra | 19:31 | |
*** edmondsw has quit IRC | 19:32 | |
AJaeger | jeblair: depends also how quickly you merge 513368 and apply it... | 19:35 |
*** lukebrowning has quit IRC | 19:36 | |
*** lukebrowning has joined #openstack-infra | 19:37 | |
*** edmondsw has joined #openstack-infra | 19:37 | |
*** openstack has joined #openstack-infra | 19:42 | |
*** ChanServ sets mode: +o openstack | 19:42 | |
*** edmondsw has quit IRC | 19:44 | |
*** gouthamr has quit IRC | 19:45 | |
openstackgerrit | Merged openstack-infra/irc-meetings master: Move irc meetings tox job in-tree https://review.openstack.org/528498 | 19:45 |
AJaeger | config-core https://review.openstack.org/#/c/528500/ removes a converted job, would appreciate if you put this on your review list for some time this week. thanks | 19:47 |
*** yamahata has joined #openstack-infra | 19:47 | |
*** tdasilva has quit IRC | 19:47 | |
*** hasharAway is now known as hashar | 19:47 | |
*** tdasilva has joined #openstack-infra | 19:47 | |
*** claudiub|2 has joined #openstack-infra | 19:48 | |
*** electrofelix has quit IRC | 19:48 | |
jlvillal | fungi: So did your script only remove -1 Jenkins and not +1 Jenkins? https://review.openstack.org/#/c/429770/ | 19:48 |
*** rlandy|rover is now known as rlandy|rover|bia | 19:49 | |
AJaeger | jlvillal: might be - those are not bad... | 19:49 |
AJaeger | I see no reason to update 1000s of reviews for that - do you? | 19:50 |
*** claudiub has quit IRC | 19:50 | |
inc0 | hey, qq about https://review.openstack.org/#/c/526469/10 - latest ps has sytrax error, I assume that's about vars: in line 13...question, how can I pass varible to job that is imported from another project?:) | 19:53 |
fungi | jlvillal: yes | 19:53 |
jlvillal | AJaeger: fungi thanks. Just curious after seeing the run that removed the Jenkins votes. I wasn't sure if it only did -1s | 19:54 |
fungi | jlvillal: it specifically addressed "nobody is reviewing my change because there's still a -1 from jenkins i can't clear without rebasing or otherwise creating a new patchset" | 19:54 |
AJaeger | inc0: missing ":" in line 14 | 19:54 |
jlvillal | fungi: Makes sense. | 19:55 |
*** dprince has quit IRC | 19:55 | |
inc0 | AJaeger: ah..right.. thank you. | 19:55 |
clarkb | dmsimard: maybe you want to send a thred to the infra mailing list about the puppet modules for new services topic? | 19:59 |
AJaeger | infra-root, one more change needed to unbreak governance building, please review https://review.openstack.org/#/c/529095 | 20:00 |
*** gouthamr has joined #openstack-infra | 20:00 | |
*** lukebrowning has joined #openstack-infra | 20:00 | |
AJaeger | fungi, pabelanger , could you help, please? ttx has a large pile of changes waiting for the merge ^ | 20:01 |
*** gouthamr has quit IRC | 20:01 | |
fungi | in _good_ news, looks like we may have only impacted ourselves by triggering a meetbot restart. there was only one other meeting starting at 19:00 and it ended early, before the bot departed | 20:01 |
fungi | AJaeger: yes, i started looking earlier. will go through that one now | 20:02 |
AJaeger | thanks, fungi | 20:02 |
fungi | thanks for the reminder | 20:02 |
*** lukebrowning has quit IRC | 20:05 | |
*** caphrim007 has joined #openstack-infra | 20:10 | |
*** lukebrowning has joined #openstack-infra | 20:14 | |
*** jtomasek has quit IRC | 20:15 | |
AJaeger | team, one more docs change for review, please: https://review.openstack.org/#/c/525735 | 20:16 |
openstackgerrit | Alex Schultz proposed openstack-infra/elastic-recheck master: Query for 1719469 https://review.openstack.org/507294 | 20:17 |
*** lukebrowning has quit IRC | 20:19 | |
*** claudiub|2 has quit IRC | 20:23 | |
*** dprince has joined #openstack-infra | 20:23 | |
AJaeger | config-core, and two review request for translations, please: https://review.openstack.org/#/c/528518 and https://review.openstack.org/#/c/529047 - and then https://review.openstack.org/529111 | 20:23 |
* AJaeger cannot count until three ;( | 20:24 | |
openstackgerrit | Merged openstack-infra/project-config master: Use python3 sphinx building for governance https://review.openstack.org/529095 | 20:24 |
*** ldnunes has quit IRC | 20:26 | |
openstackgerrit | Merged openstack-infra/elastic-recheck master: Remove old queries for fixed bugs https://review.openstack.org/529136 | 20:26 |
*** armaan has quit IRC | 20:27 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/system-config master: Add subunit2sql check database server and workers https://review.openstack.org/520192 | 20:27 |
*** slaweq_ has joined #openstack-infra | 20:27 | |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config master: Remove jenkins related puppet manifests https://review.openstack.org/529145 | 20:30 |
*** lukebrowning has joined #openstack-infra | 20:30 | |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config master: Remove jenkins related puppet manifests https://review.openstack.org/529145 | 20:32 |
*** lukebrowning has quit IRC | 20:34 | |
AJaeger | mordred: did you see my -1 on https://review.openstack.org/#/c/525734 ? | 20:35 |
*** shardy has joined #openstack-infra | 20:36 | |
*** lukebrowning has joined #openstack-infra | 20:36 | |
*** armaan has joined #openstack-infra | 20:38 | |
*** rlandy|rover|bia is now known as rlandy|rover | 20:38 | |
fungi | #status log Manually repaired eavesdrop URLs in recent plaintext meeting minutes after https://review.openstack.org/529118 merged | 20:39 |
openstackstatus | fungi: finished logging | 20:39 |
clarkb | jeblair: I'm looking at https://review.openstack.org/#/c/528846/8/manifests/site.pp and puppet typically doesn't let you redefine a variable. Does it let yuo go from undef to defined value (tsig_name in particular has me asking that) | 20:40 |
*** lukebrowning has quit IRC | 20:41 | |
clarkb | jeblair: also maybe now is a good time to approve https://review.openstack.org/#/c/527498/ ? | 20:41 |
*** abelur_ has quit IRC | 20:43 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/system-config master: Add subunit2sql check database server and workers https://review.openstack.org/520192 | 20:48 |
mtreinish | clarkb, fungi: ^^^ I think that should work, I'm just fighting the puppet syntax. Before I commit too much time to that though, do you think the apprach is basically what we discussed in syd? | 20:48 |
pabelanger | infra-root: https://review.openstack.org/529145/ deletes a whole lot of jenkins code in system-config, figured might be a good time to land it :D | 20:49 |
*** Apoorva has quit IRC | 20:51 | |
AJaeger | mordred: http://logs.openstack.org/03/03478673435a7e852529c4132d2fe050dd83471f/post/publish-governance-tc/249e2fb fails ;( | 20:51 |
AJaeger | mordred: I'll sign off for the day now, do you have time to look into that one, please? | 20:51 |
*** abelur_ has joined #openstack-infra | 20:52 | |
AJaeger | mordred: that's still using pyhon2 instead of python3 - did we not convert the job properly? | 20:52 |
clarkb | ianw: I've reviewed the etherpad stack, left comments on them as I went but only -1 on https://review.openstack.org/#/c/528625/2 | 20:54 |
*** smatzek has quit IRC | 20:54 | |
AJaeger | mordred: at least check/gate jobs worked ;) | 20:54 |
*** lukebrowning has joined #openstack-infra | 20:54 | |
* AJaeger waves good night | 20:55 | |
pabelanger | clarkb: so, should we run yum update on a single git server first, before proposing puppet changes? or just let puppet handle it | 20:55 |
clarkb | mtreinish: one question I have is whether or not we need different workers for this and insteast can have a common worker pool that write to the correct location. I guess check could slow the gate workers if we do that though | 20:55 |
clarkb | pabelanger: probably not a bad idea though I hope that cgit, git and httpd don't change so much they stop working | 20:55 |
clarkb | pabelanger: one thing to test is shallow clones | 20:56 |
*** jamesmcarthur has quit IRC | 20:56 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 20:56 |
jeblair | clarkb: ^ fixed | 20:56 |
pabelanger | clarkb: do we have a process for taking a server out of rotation? | 20:56 |
jeblair | clarkb: the replacement that we do on hiera values in site.pp makes it difficult to use complex values :( | 20:56 |
mtreinish | clarkb: It's possible to do it all in one worker, I was just trying to minimize the complexity in the log capture mechanism, it's already complicated enough | 20:57 |
clarkb | pabelanger: I'm not sure its written down but you disable the three backends for the node (http, https, git://) in haproxy | 20:57 |
clarkb | pabelanger: you do that via the command socket | 20:57 |
mtreinish | clarkb: and yeah performance is an unknown quantity for it right now, the check queue has a lot more stuff. The owrkers can keep up with gate well enough, but with gate and check it's not clear | 20:58 |
mordred | pabelanger: hah. I actually wrote that same patch yesterday but forgot to push it up | 20:58 |
mtreinish | I also don't know how big the check db will gate even with it's much shorter duration | 20:58 |
mtreinish | s/gate/get/g | 20:58 |
jeblair | mtreinish, clarkb: what about the database do we need to manually control? | 20:59 |
clarkb | jeblair: I think thats largely a mordred thing? | 20:59 |
clarkb | (wanting to get away from trove) | 20:59 |
mtreinish | jeblair: well the pain point for subunit2sql in the past is the temp filesystem size | 20:59 |
jeblair | clarkb: yes, i recall some discussion about that in the past, mostly around backups, etc... | 20:59 |
*** lukebrowning has quit IRC | 20:59 | |
fungi | pabelanger: i may have some example socat calls for that in my command history on git.o.o... checking now | 21:00 |
mordred | yah - and being able to do schema upgrades on larger datasets like subunit2sql without downtime by applying the update on a slave, then promoting that slave to master then apply the schema update to the new slave | 21:00 |
mtreinish | we can't run certain operations in trove given the size of the db because the tempdir is a ramdisk and the actual os is hidden | 21:00 |
*** lukebrowning has joined #openstack-infra | 21:01 | |
cmurphy | AJaeger: mordred i guess our other keystone* docs jobs are broken too, is there a non-hacky fix on the way for including [extras] requirements or should i do the same as https://review.openstack.org/#/c/528960/ for our other projects | 21:01 |
*** wolverineav has quit IRC | 21:01 | |
cmurphy | like we have http://logs.openstack.org/31/526631/2/gate/build-openstack-sphinx-docs/153f89b/job-output.txt.gz broken because oslo_messaging is only required in setup.cfg here http://git.openstack.org/cgit/openstack/keystonemiddleware/tree/setup.cfg#n27 | 21:01 |
*** CrayZee has quit IRC | 21:02 | |
jeblair | mtreinish: ack. the word "manually" in the commit message threw me for a loop, but looking further at the change, i see you mean "run using configuration management". | 21:02 |
fungi | pabelanger: no such luck. looks like i probably haven't done it since we switched back off the split frontend servers | 21:02 |
fungi | haproxy docs are your friend | 21:02 |
*** catintheroof has quit IRC | 21:02 | |
mordred | cmurphy: we've discussed some non-hacky solutions, but for now I think you should just do the same as https://review.openstack.org/#/c/528960/ - as i tmight take a few days to get the non-hacky fix in place | 21:02 |
mtreinish | jeblair: ah yeah it's manual as opposed to trove. Heh, I guess it's amatter of degrees | 21:02 |
cmurphy | mordred: kk ty | 21:03 |
*** catintheroof has joined #openstack-infra | 21:03 | |
fungi | mtreinish: actually _less_ manual for us than trove. spinning up trove instances is not something we have orchestrated in our automation | 21:03 |
clarkb | pabelanger: https://cbonte.github.io/haproxy-dconv/1.6/management.html#9.2 has info | 21:04 |
jeblair | but more manual for server migrations :| | 21:04 |
fungi | fair | 21:04 |
*** rossella_s has quit IRC | 21:04 | |
mtreinish | fungi: heh, I assumed there was a playbook or something for the roots to just say give me a trove | 21:04 |
fungi | though putting the database on a cinder volume probably also makes server migrations pretty painless | 21:04 |
clarkb | pabelanger: if you look at git.openstack.org:/etc/haproxy/haproxy.cfg there are three listen directives | 21:04 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Get rid of references to Jenkins slaves https://review.openstack.org/529148 | 21:05 |
fungi | mtreinish: nope, we generally use the command-line client or the service provider's (non-horizon) dashboard | 21:05 |
mordred | pabelanger: ^^ that goes along with your system-config patch | 21:05 |
clarkb | pabelanger: each listen directive names a backend then the servers lsited are all the servers you can disable | 21:05 |
*** lukebrowning has quit IRC | 21:05 | |
fungi | granted, i'm told there's a non-trove implementation behind that particular trove api anyway | 21:05 |
*** rossella_s has joined #openstack-infra | 21:06 | |
clarkb | pabelanger: so I think you can do `disable server balance_git_daemon/git08.openstack.org` against the command socket and do that for the other two backend agaist git08.openstack.org then use git08.openstack.org for testing | 21:06 |
mtreinish | fungi: well it used to be a fork from a release several years old at this point. Not sure if that changed though | 21:06 |
mtreinish | it's not like I want to use that service, my rax account lapsed a long time ago too | 21:07 |
clarkb | pabelanger: echo "disable server balance_git_daemon/git08.openstack.org" | socat /var/lib/haproxy/stats stdio | 21:07 |
*** catintheroof has quit IRC | 21:07 | |
mtreinish | while I don't see an trove openstack ansible module, I do see: https://docs.ansible.com/ansible/latest/rax_cdb_database_module.html | 21:08 |
openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remove additional dead jenkins related code https://review.openstack.org/529150 | 21:08 |
clarkb | pabelanger: I've just tested `echo "show stat" | socat /var/lib/haproxy/stats stdio` and that works so ya I think the command above and two more for the other backends should work | 21:09 |
mordred | mtreinish: yah - we haven't added any trove ansible modules yet | 21:09 |
*** lukebrowning has joined #openstack-infra | 21:12 | |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Set python3 on publish-governance-tc https://review.openstack.org/529155 | 21:12 |
mordred | AJaeger: ^^ that's what we missed | 21:12 |
mordred | clarkb, pabelanger, fungi: could I get a quick +3 on that? | 21:12 |
clarkb | pabelanger: also if you feed it something it doesn't understand it errors gracefully | 21:14 |
jeblair | fungi: can you review https://review.openstack.org/526483 ? it should be okay to merge now (indeed, i'd like to merge it before the big change to lists.o.o) | 21:14 |
mordred | jeblair: you wrote a lot of ruby | 21:15 |
jeblair | mordred: i took one for the team | 21:15 |
*** kgiusti has left #openstack-infra | 21:15 | |
jeblair | sadly, i wrote a lot more ruby than actually made it into that change :| | 21:16 |
fungi | yup, going back over what's changed in the last few patchsets now | 21:16 |
ianw | jeblair: puppet's failing to connect to adns1 & ns1 due to not knowing their host keys. is it still wip or should i just accept them? | 21:16 |
fungi | jeblair: but the good news is that we ended up needing far less ruby? ;) | 21:16 |
*** armaan has quit IRC | 21:16 | |
*** lukebrowning has quit IRC | 21:17 | |
openstackgerrit | Merged openstack-infra/puppet-exim master: Move user-supplied routers to the end https://review.openstack.org/527498 | 21:17 |
openstackgerrit | Merged openstack-infra/system-config master: Add an authoritative hidden master https://review.openstack.org/528438 | 21:17 |
fungi | ianw: the dns server changes merged a few minutes ago | 21:17 |
*** armaan_ has joined #openstack-infra | 21:17 | |
fungi | or were approved at least | 21:17 |
*** edmondsw has joined #openstack-infra | 21:17 | |
fungi | maybe gating now | 21:17 |
ianw | ok ... there's actually an odd selection of things failing due to host keys -> http://paste.openstack.org/show/629377/ | 21:18 |
ianw | was just looking at the logs | 21:18 |
*** lukebrowning has joined #openstack-infra | 21:18 | |
jeblair | ianw: i will accept them once the patches merge | 21:18 |
jeblair | (for adns1 and ns1) | 21:19 |
clarkb | for paste.o.o and grafana.o.o I think that implies the old servers are still in the inventory list and the cname points at the new servers which then fails? | 21:19 |
clarkb | either the servers didn't get deleted or ansible hasn't caught up? | 21:19 |
*** Apoorva has joined #openstack-infra | 21:20 | |
pabelanger | clarkb: fungi: okay, let me read up on it and see how to do it, I am not sure I'll start now, but maybe in the morning tomorrow | 21:20 |
jeblair | clarkb, ianw: paste.o.o was deleted. i don't know why it would show up | 21:20 |
ianw | yeah, we have both | 21:20 |
ianw | 017-12-19 21:12:24,409 p=16350 u=root | grafana.openstack.org : ok=0 changed=0 unreachable=1 failed=0 | 21:20 |
ianw | 2017-12-19 21:12:24,410 p=16350 u=root | grafana01.openstack.org : ok=8 changed=2 unreachable=0 failed=0 | 21:20 |
fungi | stale inventory cache? | 21:21 |
ianw | oh, i wonder if this goes back to the stale credentials in all-clouds ... did we fix that? | 21:21 |
jeblair | grafana.o.o was also deleted | 21:22 |
ianw | can we remove the infra-files-[rw|ro] entries from all-clouds? | 21:22 |
*** lukebrowning has quit IRC | 21:22 | |
*** edmondsw has quit IRC | 21:22 | |
fungi | that was for swift-based log publishing, right? | 21:23 |
clarkb | ianw: thats the swift account we were using | 21:23 |
clarkb | I think at somepoint we will want to look at using it again but isn't necessary now | 21:23 |
pabelanger | ianw: yah, I think we use it for swift things, but maybe others know | 21:24 |
*** lukebrowning has joined #openstack-infra | 21:24 | |
ianw | well we can't be updating anything with it because the -rw credentials are wrong | 21:24 |
ianw | this was a problem when i was using /etc/ansible/hosts/openstack --list for updating the keypairs | 21:25 |
clarkb | ah in that case I think we can just remove them | 21:25 |
clarkb | and sort out swift later if we need to (though why would they suddenly be wrong?) | 21:25 |
*** jascott1 has quit IRC | 21:25 | |
ianw | alright, i'll remove them | 21:26 |
openstackgerrit | Merged openstack-infra/project-config master: Set python3 on publish-governance-tc https://review.openstack.org/529155 | 21:26 |
fungi | possible we never used clouds.yaml for that so the entries for them have always been incorrect in it? | 21:26 |
*** armaan_ has quit IRC | 21:28 | |
*** armaan has joined #openstack-infra | 21:29 | |
*** lukebrowning has quit IRC | 21:29 | |
ianw | yeah, grafana doesn't show up in the openstack --list, only grafana01 | 21:30 |
*** lukebrowning has joined #openstack-infra | 21:30 | |
jeblair | mordred: can you approve https://review.openstack.org/528138 ? | 21:31 |
jeblair | clarkb: want to restart those nodepool launchers now? | 21:31 |
ianw | fungi: oh right, all-clouds v clouds | 21:31 |
clarkb | jeblair: sure I'm here for about another hour before I have to salmon | 21:31 |
mordred | jeblair: done | 21:32 |
clarkb | did new shade release get installed /me checks | 21:32 |
jeblair | shade==1.25.0 | 21:32 |
clarkb | I think 1.26.0 is the new shade | 21:33 |
clarkb | mordred: ^ do you still want us to wait for you on that? | 21:33 |
*** slaweq_ is now known as slaweq | 21:33 | |
jeblair | that's the latest release, november 29 | 21:33 |
smcginnis | We can't release shade until we get docs builds working. :/ | 21:33 |
clarkb | ah maybe release hasn't happened yet then | 21:33 |
clarkb | gotcha | 21:33 |
clarkb | in that case I'm inclined to restart now and pick up the bug fixes and features that we can | 21:33 |
clarkb | jeblair: ^ | 21:33 |
jeblair | clarkb: ya. shall i type or you want to? | 21:34 |
clarkb | I can type. Also should we do both at the same time or stagger them? | 21:34 |
pabelanger | smcginnis: is there a patch up to fix the docs? | 21:34 |
jeblair | clarkb: one at a time should be fine i think | 21:34 |
clarkb | I'll run sudo service nodepool-launcher stop then start on nl01 now, check it starts then go on to 02 | 21:34 |
*** myoung is now known as myoung|bbl | 21:35 | |
smcginnis | pabelanger: We need this one to stop hitting random timeouts: https://review.openstack.org/528823 then this one to update the doc requirements https://review.openstack.org/528824 | 21:35 |
*** lukebrowning has quit IRC | 21:35 | |
smcginnis | And I think AJaeger has one yet to switch the jobs to py3. | 21:36 |
clarkb | jeblair: it seems ot haev started and is reconciling its state | 21:36 |
pabelanger | smcginnis: k, I think python3 docs landed already | 21:37 |
clarkb | 2017-12-19 21:36:20,338 WARNING nodepool.CleanupWorker: Unable to grab lock to deallocate node 0001518581 from request 100-0001655889 | 21:37 |
*** lukebrowning has joined #openstack-infra | 21:37 | |
smcginnis | pabelanger: No, I think this is the next step for that: https://review.openstack.org/#/c/529104/ | 21:37 |
*** HeOS has quit IRC | 21:38 | |
pabelanger | smcginnis: ah, ya. to convert it | 21:38 |
*** slaweq has quit IRC | 21:38 | |
smcginnis | pabelanger: And I think I will need to squash that with my requirements.txt update for either of them to pass. I'll do that now and hope the last recheck on the o/requirements patch goes through. | 21:38 |
clarkb | ah ok that is for cleaning up lost requests which is probably normal when service restarts | 21:38 |
clarkb | jeblair: ready for 02 to be restarted? | 21:38 |
mordred | smcginnis: I actually just got that squash done locally | 21:39 |
mordred | smcginnis: but if you'd rather do it, totally cool | 21:39 |
pabelanger | smcginnis: yah, think you are right: No module named 'sphinxcontrib.datatemplates' | 21:39 |
smcginnis | mordred: No, feel free. | 21:39 |
mordred | smcginnis: https://review.openstack.org/528824 Add doc job requirements.txt | 21:39 |
mordred | smcginnis: updated | 21:39 |
jeblair | clarkb: yep | 21:39 |
clarkb | jeblair: ok done | 21:40 |
smcginnis | mordred: Thanks, I'll keep my fingers crossed. | 21:40 |
clarkb | jeblair: it also appears to be happy and is going through state reconciliation | 21:41 |
jeblair | clarkb: cool. we'll want to check graphs periodically and also watch out for errors near quota | 21:41 |
*** lukebrowning has quit IRC | 21:41 | |
jeblair | we may want to do a zuul restart to (a) pick up pending changes and (b) drive us closer to quota :) | 21:41 |
clarkb | ha | 21:42 |
clarkb | how to drive up load, restart everything | 21:42 |
clarkb | I'll leave the zuul restart to you if it is something you want to do | 21:42 |
* mordred can always submit some patch bombs ... | 21:42 | |
*** lukebrowning has joined #openstack-infra | 21:43 | |
*** trown|ruck is now known as trown|outtypewww | 21:43 | |
clarkb | jeblair: 02 complains about citycloud quota which was preexisting situation | 21:44 |
clarkb | jeblair: I was somewhat hoping this would address that, but I'm beginning to think their nova may have the wrong quota calculation for us | 21:44 |
clarkb | (asking about that was in the email I sent them though) | 21:44 |
*** ilbelkyr has left #openstack-infra | 21:45 | |
jeblair | clarkb: what's the complaint? | 21:45 |
*** wolverineav has joined #openstack-infra | 21:46 | |
*** priteau_ has joined #openstack-infra | 21:46 | |
clarkb | jeblair: shade.exc.OpenStackCloudHTTPError: (403) Client Error for url: https://lon1.citycloud.com:8774/v2/bed89257500340af8d0fbe7141b1bfd6/servers Quota exceeded for cores, instances: Requested 8, 1, but already used 400, 50 of 400, 50 cores, instances | 21:46 |
jeblair | clarkb, mordred: there is a shade-related exception: http://paste.openstack.org/show/629380/ | 21:47 |
clarkb | jeblair: but that error was preexisting so not a regression | 21:47 |
mordred | jeblair's exception seems bad though | 21:47 |
mordred | I agree with the exception - that is not a real thing | 21:48 |
clarkb | the good news is its an exception raising and exception | 21:48 |
openstackgerrit | Merged openstack-infra/system-config master: Make ns*.openstack.org authoritative slaves https://review.openstack.org/528846 | 21:48 |
clarkb | so we probably handle it pretty similarly :) | 21:48 |
*** lukebrowning has quit IRC | 21:48 | |
*** armaan has quit IRC | 21:48 | |
*** armaan has joined #openstack-infra | 21:48 | |
mordred | I betcha I can squeeze a fix for that into 1.26 :) | 21:48 |
openstackgerrit | Merged openstack-infra/puppet-mailman master: Add multi-site support https://review.openstack.org/526483 | 21:49 |
jeblair | clarkb: we do have an error in the new code: http://paste.openstack.org/show/629381/ | 21:49 |
*** lukebrowning has joined #openstack-infra | 21:49 | |
mordred | https://review.openstack.org/529167 Throw OpenStackCloudCreateException on create errors | 21:49 |
*** priteau has quit IRC | 21:50 | |
clarkb | jeblair: OpenStackProvider has that method | 21:50 |
jeblair | clarkb: but not OpenStackProviderConfig | 21:51 |
jeblair | we need a test | 21:51 |
clarkb | oh I see _provider isn't a provider | 21:51 |
clarkb | this was something I ran into with the lock stuff, the names we've used in places are tricky | 21:51 |
*** lukebrowning has quit IRC | 21:54 | |
*** slaweq has joined #openstack-infra | 21:55 | |
*** lukebrowning has joined #openstack-infra | 21:56 | |
*** ijw has joined #openstack-infra | 21:56 | |
openstackgerrit | Merged openstack-infra/puppet-zuul master: Update zuulv3 rewrite rules for keys https://review.openstack.org/528138 | 21:56 |
smcginnis | Anyone know how much disk space is allocated on our test nodes? | 21:59 |
smcginnis | Seeing this error: http://logs.openstack.org/38/523138/3/check/legacy-tempest-dsvm-full-sheepdog/13c4781/logs/screen-c-sch.txt.gz#_Nov_27_16_31_22_134848 | 21:59 |
*** slaweq has quit IRC | 22:00 | |
smcginnis | Not sure if it's the test environment or something with this drivers devstack plugin and its configuration. | 22:00 |
clarkb | smcginnis: it varies based on cloud but should be ~80GB | 22:00 |
*** rossella_s has quit IRC | 22:00 | |
*** lukebrowning has quit IRC | 22:01 | |
smcginnis | clarkb: OK, thanks. That should be more than plenty then. | 22:01 |
*** lukebrowning has joined #openstack-infra | 22:02 | |
clarkb | jeblair: I think we need to call invalidateQuotaCache on self._manager | 22:04 |
*** eharney has quit IRC | 22:05 | |
jeblair | clarkb: yeah. i'm writing a test to make sure that code is exercised | 22:05 |
*** armaan has quit IRC | 22:05 | |
clarkb | jeblair: the terminology here is confusing but self._provider is actually just the provider configuration and self._manager is I think the provider (by way of the provider_manager whcih gives you an OpenStackProvider) | 22:06 |
clarkb | jeblair: cool, it might be nice to also rename things so that self._provider is self._provider_config and self._manager is self._provider? | 22:06 |
*** armaan has joined #openstack-infra | 22:06 | |
clarkb | that may be a fairly large change though so probably good for followup | 22:06 |
jeblair | clarkb: yes, i would be happy to have variables more reflective of what they are | 22:06 |
*** lukebrowning has quit IRC | 22:07 | |
clarkb | oh ya ProviderManager only has static methods so its not really something we have multiple objects of | 22:07 |
*** priteau_ has quit IRC | 22:09 | |
*** pcichy has joined #openstack-infra | 22:09 | |
*** priteau has joined #openstack-infra | 22:09 | |
ianw | so is /etc/ansible/hosts/generated-groups *only* updated when we run http://git.openstack.org/cgit/openstack-infra/system-config/tree/launch/launch-node.py#n152 ? | 22:10 |
clarkb | I've got to go help with the salmon shortly so won't dive into updating names now, but will keep an eye out for your patch to help get that reviewed | 22:10 |
ianw | that would explain why we have old hosts | 22:10 |
clarkb | ianw: oh ya since we don't delete the old host until later | 22:10 |
ianw | seems like we should probably slip a cron job in to regenerate that occasionally, to kick deleted hosts before the next node launch | 22:12 |
ianw | not that it matters much ... just a bit confusing | 22:12 |
*** claudiub|2 has joined #openstack-infra | 22:12 | |
*** jcoufal has quit IRC | 22:13 | |
*** priteau has quit IRC | 22:14 | |
*** jascott1 has joined #openstack-infra | 22:19 | |
ianw | no that's not it | 22:21 |
ianw | http://paste.openstack.org/show/629391/ | 22:21 |
ianw | ansible --list shows it, /etc/ansible/hosts/openstack --list doesn't | 22:22 |
*** florianf has quit IRC | 22:22 | |
ianw | grafana.o.o does not appear in /var/cache/ansible-inventory/ansible-inventory.cache | 22:23 |
*** rcernin has joined #openstack-infra | 22:28 | |
*** HeOS has joined #openstack-infra | 22:32 | |
*** slaweq has joined #openstack-infra | 22:32 | |
*** lukebrowning has joined #openstack-infra | 22:35 | |
*** salv-orl_ has quit IRC | 22:36 | |
*** salv-orlando has joined #openstack-infra | 22:37 | |
*** shardy has quit IRC | 22:37 | |
*** slaweq has quit IRC | 22:37 | |
*** pcichy has quit IRC | 22:38 | |
*** lukebrowning has quit IRC | 22:40 | |
*** marst has quit IRC | 22:41 | |
*** salv-orlando has quit IRC | 22:41 | |
*** lukebrowning has joined #openstack-infra | 22:42 | |
*** salv-orlando has joined #openstack-infra | 22:42 | |
*** lukebrowning has quit IRC | 22:47 | |
*** lukebrowning has joined #openstack-infra | 22:48 | |
ianw | you know what ... ansible reads /etc/ansible/hosts/generated-groups as it's creating generated-groups. if the regex matches in t/usr/local/bin/expand-groups.sh, the host gets put back into it indefinitely | 22:50 |
*** lukebrowning has quit IRC | 22:53 | |
*** slaweq has joined #openstack-infra | 22:53 | |
*** lukebrowning has joined #openstack-infra | 22:54 | |
openstackgerrit | James E. Blair proposed openstack-infra/nodepool feature/zuulv3: Fail on quota-exceeded (partial revert) https://review.openstack.org/529173 | 22:56 |
jeblair | clarkb, mordred, tobiash: ^ | 22:56 |
*** myoung|bbl is now known as myoung | 22:56 | |
*** slaweq has quit IRC | 22:58 | |
*** lennyb has quit IRC | 22:58 | |
*** lukebrowning has quit IRC | 22:59 | |
mordred | jeblair: reading | 22:59 |
jeblair | hrm. the nodepool graphs are discontinuitous after the restart | 23:00 |
*** lukebrowning has joined #openstack-infra | 23:00 | |
*** lennyb has joined #openstack-infra | 23:00 | |
mordred | jeblair: patch lgtm | 23:03 |
jeblair | mordred: does get_complute_limits use task manager? | 23:05 |
jeblair | get_compute_limits | 23:05 |
mordred | jeblair: everything uses task manager | 23:05 |
*** lukebrowning has quit IRC | 23:05 | |
jeblair | ah there it is, sorry, i missed the task | 23:05 |
*** edmondsw has joined #openstack-infra | 23:06 | |
mordred | jeblair: yah- all the rest calls go through task manager at the adapter layer - it would take a massive effort at this point to avoid task manager :) | 23:06 |
jeblair | i can't actually tell what nodepool thinks the quotas are | 23:07 |
jeblair | it outputs stuff to the log, but does not include the provider name | 23:07 |
jeblair | i suspect that there are problems | 23:07 |
jeblair | for instance, i don't think we've spun up a node on rackspace since the changes went in | 23:07 |
mordred | jeblair: oh - you know what - I think rax reports bogus quota values | 23:08 |
jeblair | 2017-12-19 23:06:21,619 DEBUG nodepool.driver.openstack.OpenStackProvider: Available nodepool quota: {'compute': {'ram': 2572288, 'instances': 219, 'cores': -25}} | 23:09 |
jeblair | it's possible that's rax-dfw, but i don't know for sure | 23:09 |
mordred | jeblair: nod. I just checked a rax dfw acount... | 23:10 |
*** edmondsw has quit IRC | 23:10 | |
mordred | jeblair: http://paste.openstack.org/raw/629397/ - which does sure return max_total_cores -1 | 23:11 |
mordred | although 'total_cores_used': 629 - so it does report *something(* | 23:11 |
jeblair | nodepool has 72 cores in use in dfw right now | 23:12 |
jeblair | -1 - 72 = -73 != -25 ... so i don't know where it got the -25 from | 23:13 |
jeblair | but maybe that wasn't ord | 23:13 |
jeblair | er dfw | 23:13 |
jeblair | anyway, it seems problematic | 23:13 |
mordred | yah | 23:13 |
jeblair | perhaps -1 means "not limited" ? | 23:14 |
mordred | maybe? | 23:14 |
*** lukebrowning has joined #openstack-infra | 23:14 | |
mordred | mriedem: ^^ does '-1' have a meaning for nova for quota/usage? | 23:14 |
jeblair | i think i'll start with 2 changes: a) log the provider name :) b) assume -1 means unlimited | 23:15 |
jeblair | and see where that gets us | 23:15 |
mordred | jeblair: sounds good to me | 23:15 |
*** hashar has quit IRC | 23:15 | |
mriedem | mordred: unlimited | 23:16 |
mordred | jeblair: you win | 23:16 |
mordred | mriedem: thanks | 23:17 |
openstackgerrit | Kendall Nelson proposed openstack-infra/storyboard master: Cleanup Migration Docs https://review.openstack.org/529177 | 23:18 |
*** lukebrowning has quit IRC | 23:19 | |
*** lukebrowning has joined #openstack-infra | 23:20 | |
openstackgerrit | James E. Blair proposed openstack-infra/nodepool feature/zuulv3: Log provider names with quota https://review.openstack.org/529178 | 23:21 |
clarkb | jeblair: reading nodepool fix test, why do we modify the client max instances when we have the quota mocked? | 23:22 |
clarkb | jeblair: is it caching the quota value from the mock then not asking again? | 23:23 |
*** felipemonteiro has quit IRC | 23:24 | |
*** felipemonteiro has joined #openstack-infra | 23:24 | |
*** lukebrowning has quit IRC | 23:25 | |
*** lukebrowning has joined #openstack-infra | 23:26 | |
openstackgerrit | James E. Blair proposed openstack-infra/nodepool feature/zuulv3: Assume a quota limit of -1 means unlimited https://review.openstack.org/529180 | 23:27 |
*** andreas_s has joined #openstack-infra | 23:28 | |
jeblair | clarkb: yes, the mock is used only when the provider is constructed. | 23:28 |
jeblair | clarkb: that's not a mock of a real function -- it's basically just a way to pass a parameter to the provider creation out of band. | 23:29 |
jeblair | clarkb, mordred: both changes up ^ | 23:30 |
jeblair | infra-root: ^ recent quota handling changes to nodepool have caused us to stop using rax; fixes in progress; if they don't merge soon, we should revert. | 23:31 |
*** lukebrowning has quit IRC | 23:31 | |
*** andreas_s has quit IRC | 23:32 | |
jeblair | i would revert to a20d81c32c7caeb330bbc6b7373ddb975c533e20 if necessary | 23:32 |
*** lukebrowning has joined #openstack-infra | 23:33 | |
clarkb | jeblair: the -1 change us what we need for rax? | 23:33 |
jeblair | clarkb: that's my understanding, based on some, i think, reasonable assumptions i'm making from the incomplete log output | 23:34 |
clarkb | ok reviewing the rest of stack now | 23:34 |
*** HeOS has quit IRC | 23:34 | |
*** stakeda has joined #openstack-infra | 23:35 | |
clarkb | jeblair changes lgtm | 23:37 |
clarkb | I approved the first one should I remove approval in case we want to revert? | 23:37 |
clarkb | (though getting fixes in seems like a good idea) | 23:37 |
jeblair | clarkb: nah, let's see if we can roll forward | 23:37 |
*** lukebrowning has quit IRC | 23:38 | |
openstackgerrit | Merged openstack-infra/project-config master: Improve zanata-cli download https://review.openstack.org/525658 | 23:38 |
openstackgerrit | Merged openstack-infra/project-config master: Add log-classify check and gate jobs https://review.openstack.org/527313 | 23:38 |
*** lukebrowning has joined #openstack-infra | 23:39 | |
*** pahuang has joined #openstack-infra | 23:42 | |
*** catinthe_ has joined #openstack-infra | 23:43 | |
*** lukebrowning has quit IRC | 23:44 | |
*** lukebrowning has joined #openstack-infra | 23:45 | |
*** rbrndt has quit IRC | 23:46 | |
openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Avoid generated-groups when creating generated-groups https://review.openstack.org/529186 | 23:48 |
openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Run expand-groups.sh occasionaly under cron https://review.openstack.org/529187 | 23:48 |
openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Run expand-groups.sh occasionally under cron https://review.openstack.org/529187 | 23:49 |
*** lukebrowning has quit IRC | 23:50 | |
pabelanger | jeblair: clarkb: +3 on rax fixes | 23:51 |
jeblair | pabelanger: great, thanks! hopefully those will go in and we don't have to do a manual revert | 23:51 |
openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Fix codesearch stderr capturing https://review.openstack.org/529188 | 23:52 |
*** catintheroof has joined #openstack-infra | 23:52 | |
*** catintheroof has quit IRC | 23:52 | |
*** catintheroof has joined #openstack-infra | 23:53 | |
*** catinthe_ has quit IRC | 23:55 | |
ianw | #status log removed infra-files-ro and infra-files-rw from all-clouds.yaml as they are invalid, and cause issues deploying new keys. saved in a backup file on puppetmaster.o.o if required | 23:56 |
openstackstatus | ianw: finished logging | 23:56 |
*** lukebrowning has joined #openstack-infra | 23:56 | |
*** catintheroof has quit IRC | 23:58 | |
pabelanger | ianw: is there a patch up for all-clouds.yaml in system-config? | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!