| ianw | how did ansible break testinfra? isn't is a separate process? | 00:00 |
|---|---|---|
| ianw | oohhh, no i get it, it talks to the hosts over ansible, that's right | 00:00 |
| clarkb | ianw: I read it as testinfra using ansible programmatically then breaking | 00:01 |
| clarkb | now ansible will probably say that isn't a stable interface | 00:01 |
| clarkb | (but it points to a need for such an interface and that people are doing it anyway) | 00:02 |
| clarkb | https://github.com/philpep/testinfra/blob/master/testinfra/utils/ansible_runner.py#L105-L117 is the code that is broken with newer ansible | 00:03 |
| *** wolverineav has quit IRC | 00:03 | |
| *** wolverineav has joined #openstack-infra | 00:04 | |
| *** hwoarang has quit IRC | 00:06 | |
| clarkb | https://github.com/ansible/ansible/commit/7e92ff823e2bf97ee33c1b7ff87bb8acecb5ae77#diff-bdd6c847fae8976ab8a7259d0b583f34 is what broke it. I think it is just called parser now not base_parser | 00:07 |
| *** hwoarang has joined #openstack-infra | 00:07 | |
| corvus | perhaps testinfra could/should use https://github.com/ansible/ansible-runner | 00:07 |
| clarkb | oh neat | 00:08 |
| clarkb | ya if the goal there is a stable itnerface it probably should | 00:08 |
| *** wolverineav has quit IRC | 00:08 | |
| corvus | yeah, that's what they wrote it for. that's what awx is using to run ansible now. zuul would have used it if it had been invented when we wrote 2.5. and if we ever have to overhaul how we run ansible, that might be the time to switch. | 00:09 |
| corvus | (note, btw, what's doing the merging on that repo) | 00:10 |
| *** yamamoto_ has quit IRC | 00:10 | |
| clarkb | is it bad that I immediately scream in horror when I see it uses expect? | 00:10 |
| * clarkb has bad memories of using perl expect to configure cisco switches | 00:10 | |
| clarkb | also neat zuul merging commits to that | 00:11 |
| *** slaweq has joined #openstack-infra | 00:11 | |
| clarkb | it is interesting that they went with the process boundary and licensed it apache2 | 00:11 |
| clarkb | I'll file a bug on testinfra if there isn't one already | 00:12 |
| *** yamamoto has joined #openstack-infra | 00:14 | |
| clarkb | https://github.com/philpep/testinfra/issues/401#issuecomment-468491634 and done | 00:15 |
| *** slaweq has quit IRC | 00:15 | |
| ianw | clarkb: hrm, what's with https://github.com/philpep/testinfra/pull/404 | 00:17 |
| ianw | [LOOKING FOR SOMEONE TO TAKEOVER :-] replace ansible base_parser with create_base_parser #404 | 00:18 |
| clarkb | ianw: the individual that wrote the initial PR is looking for someone to take over fixing the bug I think | 00:18 |
| *** tosky has quit IRC | 00:18 | |
| clarkb | they asked the maintainer for help | 00:18 |
| clarkb | (bceause it turned out the fix was more complicated than initially assumed) | 00:18 |
| ianw | yeah, more than just a rename | 00:18 |
| *** yamamoto has quit IRC | 00:20 | |
| ianw | oh dear, yeah that's getting right into the middle of ansible cli stuff | 00:20 |
| ianw | i'm not sure if the devel job pulls in testinfra from master to test any changes | 00:21 |
| clarkb | I don't think it does | 00:21 |
| openstackgerrit | Merged openstack-infra/infra-specs master: letsencrypt spec https://review.openstack.org/587283 | 00:22 |
| ianw | no, just ara, openstacksdk & ansible. i think that might be a good first step ... | 00:22 |
| openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Add philpep/testinfra to Zuul https://review.openstack.org/640229 | 00:26 |
| *** mriedem is now known as mriedem_away | 00:26 | |
| openstackgerrit | Merged openstack-infra/system-config master: Set ansible_python_interpreter for zuul-preview https://review.openstack.org/640221 | 00:27 |
| *** dpawlik has joined #openstack-infra | 00:30 | |
| *** wolverineav has joined #openstack-infra | 00:31 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Add testinfra master to -devel job https://review.openstack.org/640231 | 00:31 |
| *** hwoarang has quit IRC | 00:32 | |
| *** rascasoft has joined #openstack-infra | 00:32 | |
| *** hwoarang has joined #openstack-infra | 00:34 | |
| *** dpawlik has quit IRC | 00:34 | |
| *** trown is now known as trown|outtypewww | 00:39 | |
| *** rascasoft has quit IRC | 00:41 | |
| *** yamamoto has joined #openstack-infra | 00:46 | |
| *** yamamoto has quit IRC | 00:50 | |
| *** strigazi has quit IRC | 00:55 | |
| *** strigazi has joined #openstack-infra | 00:56 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/storyboard master: Update docs on how to run the tests locally https://review.openstack.org/640233 | 00:57 |
| openstackgerrit | Clark Boylan proposed openstack-infra/storyboard master: Update docs on how to run the tests locally https://review.openstack.org/640233 | 00:59 |
| *** dpawlik has joined #openstack-infra | 01:02 | |
| *** whoami-rajat has joined #openstack-infra | 01:04 | |
| *** strigazi has quit IRC | 01:05 | |
| *** strigazi has joined #openstack-infra | 01:05 | |
| *** dpawlik has quit IRC | 01:07 | |
| *** slaweq has joined #openstack-infra | 01:11 | |
| *** hamzy has joined #openstack-infra | 01:11 | |
| *** slaweq has quit IRC | 01:16 | |
| pabelanger | ianw: clarkb: yah, some folks in ansible org recently found out license concern around testinfra. Somebody hacked up ansible-runner as replacement, but don't think it was pushed up | 01:17 |
| pabelanger | will check tomorrow | 01:17 |
| clarkb | Apparently it was being worked on | 01:19 |
| pabelanger | this is an issue for ansible/molecule also, which is now operated by ansible | 01:19 |
| ianw | pabelanger: ok, cool, LMN, I just half-hacked ansible-runner in ... | 01:20 |
| ianw | i think one problem may be it's api wants a list of hosts from the inventory | 01:20 |
| clarkb | iamw there was a response to my comment | 01:20 |
| clarkb | saying it is almost done? | 01:20 |
| *** wolverineav has quit IRC | 01:21 | |
| pabelanger | ianw: just pinged somebody, but likely won't hear anything until tomorrow | 01:22 |
| ianw | oh, ok cool. can't see any code, but interested how "get_hosts" will work if it doesn't want to duplicate ansible's inventory parsing | 01:22 |
| ianw | it's easy enough to do with "from ansible.inventory.manager import InventoryManager" but i think that's the root of the problem anyway :) | 01:22 |
| pabelanger | ianw: https://github.com/ansible/ansible-runner/issues/200 | 01:23 |
| pabelanger | ah, jctanner already commenting | 01:23 |
| pabelanger | that was who did the initial work in ansible org | 01:24 |
| corvus | ianw: +3 on the add repo change | 01:24 |
| *** ijw has quit IRC | 01:24 | |
| *** ijw has joined #openstack-infra | 01:25 | |
| ianw | corvus: cool, well whatever comes out of the above, we can test it :) | 01:25 |
| *** wolverineav has joined #openstack-infra | 01:26 | |
| *** wolverineav has quit IRC | 01:26 | |
| *** wolverineav has joined #openstack-infra | 01:27 | |
| pabelanger | ianw: the PR above has a gist of the ansible-runner code for test-infra, I think once the ansible-runner bug is fixed, we can propose the update to test-infra | 01:27 |
| pabelanger | then you'd be the hero of ansible/molecule :) | 01:27 |
| *** rascasoft has joined #openstack-infra | 01:28 | |
| ianw | oh, right ... https://gist.github.com/jctanner/512ec07171d95e61a5b7fd5f9103a301 is the actual code | 01:29 |
| ianw | and the answer to my question seems to be "cmd = 'ansible-inventory -i %s' % self.host_list" | 01:30 |
| *** ijw has quit IRC | 01:30 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add philpep/testinfra to Zuul https://review.openstack.org/640229 | 01:30 |
| pabelanger | ianw: yup | 01:31 |
| *** jamesmcarthur has joined #openstack-infra | 01:32 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/puppet-graphite master: Fix django wsgi typo https://review.openstack.org/640237 | 01:34 |
| *** jamesmcarthur has quit IRC | 01:35 | |
| *** jamesmcarthur has joined #openstack-infra | 01:36 | |
| *** rascasoft has quit IRC | 01:40 | |
| *** jillr has left #openstack-infra | 01:51 | |
| *** wolverineav has quit IRC | 01:54 | |
| *** wolverineav has joined #openstack-infra | 01:57 | |
| *** wolverineav has quit IRC | 01:57 | |
| *** wolverineav has joined #openstack-infra | 01:57 | |
| *** ijw has joined #openstack-infra | 01:58 | |
| *** yamamoto has joined #openstack-infra | 02:00 | |
| *** jamesmcarthur has quit IRC | 02:01 | |
| *** tkajinam has quit IRC | 02:02 | |
| *** tkajinam has joined #openstack-infra | 02:03 | |
| *** jamesmcarthur has joined #openstack-infra | 02:04 | |
| *** tkajinam_ has joined #openstack-infra | 02:05 | |
| *** bhavikdbavishi has joined #openstack-infra | 02:06 | |
| *** roman_g has quit IRC | 02:07 | |
| *** tkajinam has quit IRC | 02:07 | |
| *** tkajinam_ has quit IRC | 02:10 | |
| *** tkajinam has joined #openstack-infra | 02:10 | |
| *** jamesmcarthur has quit IRC | 02:11 | |
| *** jamesmcarthur has joined #openstack-infra | 02:12 | |
| *** rascasoft has joined #openstack-infra | 02:16 | |
| *** jamesmcarthur has quit IRC | 02:20 | |
| *** jamesmcarthur has joined #openstack-infra | 02:20 | |
| *** ijw has quit IRC | 02:21 | |
| *** markvoelker has quit IRC | 02:26 | |
| *** rascasoft has quit IRC | 02:26 | |
| ianw | pabelanger: well, i got it sort of working, and made a common pull request i hope we can work from -> https://github.com/philpep/testinfra/pull/410 | 02:37 |
| ianw | i hope that can be updated by others but i get lost with github permissions :/ | 02:38 |
| *** jamesmcarthur has quit IRC | 02:39 | |
| *** mriedem_away has quit IRC | 02:40 | |
| *** wolverineav has quit IRC | 02:40 | |
| *** gyee has quit IRC | 02:42 | |
| *** hongbin has joined #openstack-infra | 02:46 | |
| *** rlandy has quit IRC | 02:49 | |
| openstackgerrit | Merged openstack-infra/puppet-graphite master: Fix django wsgi typo https://review.openstack.org/640237 | 02:53 |
| *** sdake has joined #openstack-infra | 02:55 | |
| *** dpawlik has joined #openstack-infra | 03:03 | |
| *** yamamoto has quit IRC | 03:04 | |
| *** dpawlik has quit IRC | 03:08 | |
| *** slaweq has joined #openstack-infra | 03:11 | |
| *** psachin has joined #openstack-infra | 03:11 | |
| *** ramishra has joined #openstack-infra | 03:14 | |
| *** apetrich has quit IRC | 03:15 | |
| *** hwoarang has quit IRC | 03:15 | |
| *** slaweq has quit IRC | 03:15 | |
| *** hwoarang has joined #openstack-infra | 03:16 | |
| *** dave-mccowan has joined #openstack-infra | 03:16 | |
| *** bhavikdbavishi has quit IRC | 03:18 | |
| *** harlowja has quit IRC | 03:27 | |
| *** markvoelker has joined #openstack-infra | 03:27 | |
| *** rascasoft has joined #openstack-infra | 03:27 | |
| *** dpawlik has joined #openstack-infra | 03:35 | |
| *** hwoarang has quit IRC | 03:36 | |
| *** hwoarang has joined #openstack-infra | 03:38 | |
| *** janki has joined #openstack-infra | 03:39 | |
| *** dpawlik has quit IRC | 03:40 | |
| *** yamamoto has joined #openstack-infra | 03:48 | |
| *** markvoelker has quit IRC | 04:00 | |
| *** sthussey has quit IRC | 04:03 | |
| *** dave-mccowan has quit IRC | 04:04 | |
| *** bhavikdbavishi has joined #openstack-infra | 04:06 | |
| *** sdake has quit IRC | 04:08 | |
| *** ramishra has quit IRC | 04:09 | |
| *** ramishra has joined #openstack-infra | 04:09 | |
| *** hwoarang has quit IRC | 04:09 | |
| *** yamamoto has quit IRC | 04:10 | |
| *** slaweq has joined #openstack-infra | 04:11 | |
| *** wolverineav has joined #openstack-infra | 04:11 | |
| *** hwoarang has joined #openstack-infra | 04:15 | |
| *** slaweq has quit IRC | 04:16 | |
| ianw | Tags: <, c, l, a, s, s, , ', a, n, s, i, b, l, e, ., u, t, i, l, s, ., s, e, n, t, i, n, e, l, ., S, e, n, t, i, n, e, l, ', > | 04:16 |
| ianw | dmsimard: ^ http://logs.openstack.org/31/640231/1/check/system-config-run-base-ansible-devel/9c80e90/hosts/bridge.openstack.org/ara-report/result/beb3c691-9a12-41b7-bcc6-a373d5921cd3/ | 04:17 |
| *** wolverineav has quit IRC | 04:17 | |
| ianw | infra-root: if you can poke at http://graphite.opendev.org/ ... I think it's ready for production. If I don't hear anything, on my Monday when it's nice and quiet I'll look at updating dns and doing a final sync to make it live | 04:26 |
| *** hwoarang has quit IRC | 04:30 | |
| *** hwoarang has joined #openstack-infra | 04:35 | |
| *** hongbin has quit IRC | 04:40 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: [dnm] run devel job against testinfra with ansible-runner https://review.openstack.org/640256 | 04:41 |
| *** wolverineav has joined #openstack-infra | 04:42 | |
| *** ykarel|away has joined #openstack-infra | 04:47 | |
| *** ykarel|away is now known as ykarel | 04:47 | |
| *** yamamoto has joined #openstack-infra | 04:49 | |
| *** yamamoto has quit IRC | 04:54 | |
| *** markvoelker has joined #openstack-infra | 04:57 | |
| *** raukadah is now known as chandankumar | 05:09 | |
| *** markvoelker has quit IRC | 05:31 | |
| *** dpawlik has joined #openstack-infra | 05:36 | |
| *** dpawlik has quit IRC | 05:41 | |
| *** wolverineav has quit IRC | 05:43 | |
| *** hwoarang has quit IRC | 05:44 | |
| *** hwoarang has joined #openstack-infra | 05:46 | |
| openstackgerrit | Ivoline Ngong proposed openstack-infra/storyboard master: Updated_at content modified https://review.openstack.org/638840 | 05:50 |
| *** yamamoto has joined #openstack-infra | 05:53 | |
| *** yamamoto has quit IRC | 05:58 | |
| *** wolverineav has joined #openstack-infra | 06:05 | |
| *** slaweq has joined #openstack-infra | 06:11 | |
| *** dpawlik has joined #openstack-infra | 06:12 | |
| *** slaweq has quit IRC | 06:15 | |
| *** dpawlik has quit IRC | 06:17 | |
| *** flaper87 has joined #openstack-infra | 06:21 | |
| *** markvoelker has joined #openstack-infra | 06:28 | |
| *** hwoarang has quit IRC | 06:50 | |
| *** hwoarang has joined #openstack-infra | 06:51 | |
| *** armax has quit IRC | 06:54 | |
| *** wolverineav has quit IRC | 06:57 | |
| *** markvoelker has quit IRC | 07:00 | |
| *** quiquell|off is now known as quiquell | 07:01 | |
| *** yamamoto has joined #openstack-infra | 07:08 | |
| *** slaweq has joined #openstack-infra | 07:11 | |
| *** yamamoto has quit IRC | 07:13 | |
| *** slaweq has quit IRC | 07:15 | |
| *** dpawlik has joined #openstack-infra | 07:15 | |
| *** pgaxatte has joined #openstack-infra | 07:21 | |
| ianw | pabelanger: packing it in now, but https://github.com/philpep/testinfra/pull/410/ is actually I think really close. only two tests fail, that don't use the shell module | 07:22 |
| ianw | i think it has something to do with the output chunking ansible-runner does when it captures stdout. it seems to overflow it's buffer or somehow drop stuff, and read() on the stdout fd it provides just starts half-way through the response | 07:23 |
| ianw | it's the type of thing mordred loves to debug ;) | 07:23 |
| *** kopecmartin|off is now known as kopecmartin | 07:23 | |
| *** wolverineav has joined #openstack-infra | 07:25 | |
| *** e0ne has joined #openstack-infra | 07:28 | |
| *** slaweq has joined #openstack-infra | 07:32 | |
| ianw | speaking of mordred .. possible tox siblings doesn't work for github projects? | 07:32 |
| ianw | http://logs.openstack.org/56/640256/1/check/system-config-run-base-ansible-devel/6d0eda6/ara-report/result/bffb27b7-af76-4662-9803-2ce02f4b6446/ | 07:32 |
| ianw | https://review.openstack.org/640256 -- adding testinfra master from github to the ansible-devel system-config job | 07:33 |
| *** jtomasek has joined #openstack-infra | 07:37 | |
| *** sdake has joined #openstack-infra | 07:44 | |
| *** yamamoto has joined #openstack-infra | 07:46 | |
| *** hwoarang has quit IRC | 07:49 | |
| *** hwoarang has joined #openstack-infra | 07:50 | |
| *** auristor has quit IRC | 07:53 | |
| *** quiquell is now known as quiquell|brb | 07:54 | |
| *** markvoelker has joined #openstack-infra | 07:57 | |
| *** kjackal has joined #openstack-infra | 08:09 | |
| *** wolverineav has quit IRC | 08:10 | |
| *** ginopc has joined #openstack-infra | 08:11 | |
| *** apetrich has joined #openstack-infra | 08:12 | |
| *** auristor has joined #openstack-infra | 08:15 | |
| *** tkajinam has quit IRC | 08:18 | |
| *** ccamacho has joined #openstack-infra | 08:19 | |
| *** sdake has quit IRC | 08:23 | |
| *** gfidente has joined #openstack-infra | 08:24 | |
| *** quiquell|brb is now known as quiquell | 08:26 | |
| *** markvoelker has quit IRC | 08:31 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Implied branch matching must use fullmatch https://review.openstack.org/640272 | 08:33 |
| *** iurygregory has joined #openstack-infra | 08:36 | |
| *** mcgiggler has joined #openstack-infra | 08:39 | |
| *** tosky has joined #openstack-infra | 08:42 | |
| *** jpena|off is now known as jpena | 08:48 | |
| *** ramishra has quit IRC | 08:52 | |
| *** jpich has joined #openstack-infra | 08:52 | |
| *** sdake has joined #openstack-infra | 08:54 | |
| *** yamamoto has quit IRC | 09:04 | |
| *** yamamoto has joined #openstack-infra | 09:05 | |
| *** panda|ruck|off is now known as panda|ruck | 09:05 | |
| openstackgerrit | Riju Khatri proposed openstack-infra/storyboard-webclient master: allows editing for any length of story title https://review.openstack.org/640187 | 09:09 |
| *** IvensZambrano has joined #openstack-infra | 09:10 | |
| *** florianf has joined #openstack-infra | 09:10 | |
| *** roman_g has joined #openstack-infra | 09:11 | |
| *** priteau has joined #openstack-infra | 09:11 | |
| *** kashyap has left #openstack-infra | 09:15 | |
| *** wolverineav has joined #openstack-infra | 09:17 | |
| openstackgerrit | Riju Khatri proposed openstack-infra/storyboard-webclient master: Adds task title to new story modal https://review.openstack.org/639962 | 09:17 |
| *** wolverineav has quit IRC | 09:22 | |
| *** ykarel is now known as ykarel|lunch | 09:27 | |
| *** markvoelker has joined #openstack-infra | 09:28 | |
| fnordahl | config-core: would anyone have a spare moment for creating and importing the repositories in https://review.openstack.org/#/c/639066/ ? | 09:30 |
| *** bhavikdbavishi has quit IRC | 09:32 | |
| *** bhavikdbavishi has joined #openstack-infra | 09:32 | |
| *** dtantsur|afk is now known as dtantsur | 09:37 | |
| *** derekh has joined #openstack-infra | 09:42 | |
| *** sshnaidm is now known as sshnaidm|off | 10:00 | |
| *** markvoelker has quit IRC | 10:01 | |
| *** electrofelix has joined #openstack-infra | 10:02 | |
| *** ykarel|lunch is now known as ykarel | 10:03 | |
| *** ttx has quit IRC | 10:07 | |
| *** ttx has joined #openstack-infra | 10:08 | |
| AJaeger | fnordahl: want to review https://review.openstack.org/639831 as well? That needs your PTL +1. | 10:15 |
| *** owalsh has quit IRC | 10:17 | |
| *** sdake has quit IRC | 10:20 | |
| *** eandersson has quit IRC | 10:20 | |
| *** e0ne has quit IRC | 10:22 | |
| *** e0ne has joined #openstack-infra | 10:23 | |
| zbr | AJaeger: ianw: can you please look at https://review.openstack.org/#/c/639951/ on bindep. | 10:30 |
| AJaeger | zbr: I'm not a bindep core... | 10:30 |
| *** owalsh has joined #openstack-infra | 10:31 | |
| zbr | ahh, right forgot. But i hope I addressed your original comment. | 10:34 |
| zbr | pabelanger: ^ | 10:34 |
| zbr | is kinda key feature that we need in bindep in order to install right packages on fedora/centos | 10:34 |
| *** yamamoto has quit IRC | 10:45 | |
| *** bhavikdbavishi has quit IRC | 10:46 | |
| *** kazsh has quit IRC | 10:46 | |
| *** kazsh has joined #openstack-infra | 10:52 | |
| *** ramishra has joined #openstack-infra | 10:57 | |
| fnordahl | AJaeger: looking | 10:58 |
| *** markvoelker has joined #openstack-infra | 10:58 | |
| *** luizbag has joined #openstack-infra | 11:00 | |
| fnordahl | AJaeger: re the review you linked I put a -1 on that as it is not ready to be added yet. We will reach out to the contributor and discuss further steps. | 11:05 |
| *** IvensZambrano has quit IRC | 11:05 | |
| AJaeger | fnordahl: fine with me - thanks. | 11:05 |
| *** e0ne has quit IRC | 11:11 | |
| *** IvensZambrano has joined #openstack-infra | 11:11 | |
| *** e0ne has joined #openstack-infra | 11:14 | |
| *** tbarron has joined #openstack-infra | 11:21 | |
| *** yamamoto has joined #openstack-infra | 11:21 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add ``ceph-rbd-mirror`` charm and dependencies https://review.openstack.org/639066 | 11:22 |
| fnordahl | AJaeger: awesome, thank you for your prompt response and help. Sorry for the inconvenience of the unreviewed governance and project-config changes | 11:24 |
| *** IvensZambrano is now known as snevi | 11:24 | |
| snevi | AJaeger: Hi, my team is working on a feature and we created a new repo change in https://review.openstack.org/#/c/638633/ not sure if we are missing another step to get the repo created. Is tat something you could help us with? | 11:27 |
| *** bhavikdbavishi has joined #openstack-infra | 11:29 | |
| *** markvoelker has quit IRC | 11:30 | |
| *** yamamoto has quit IRC | 11:32 | |
| *** yamamoto has joined #openstack-infra | 11:35 | |
| *** yamamoto has quit IRC | 11:35 | |
| *** yamamoto has joined #openstack-infra | 11:43 | |
| *** yamamoto has quit IRC | 11:47 | |
| *** e0ne has quit IRC | 11:49 | |
| *** kjackal has quit IRC | 11:49 | |
| *** EmilienM is now known as EvilienM | 11:58 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add rsd-virt-for-nova project https://review.openstack.org/638633 | 12:03 |
| *** jchhatbar has joined #openstack-infra | 12:06 | |
| *** janki has quit IRC | 12:07 | |
| *** dave-mccowan has joined #openstack-infra | 12:10 | |
| *** yamamoto has joined #openstack-infra | 12:11 | |
| *** dave-mccowan has quit IRC | 12:15 | |
| *** panda|ruck is now known as panda|ruck|lunch | 12:16 | |
| *** priteau has quit IRC | 12:17 | |
| *** ykarel is now known as ykarel|afk | 12:18 | |
| *** ldnunes has joined #openstack-infra | 12:25 | |
| *** markvoelker has joined #openstack-infra | 12:28 | |
| *** kjackal has joined #openstack-infra | 12:38 | |
| *** yamamoto has quit IRC | 12:50 | |
| *** e0ne has joined #openstack-infra | 12:50 | |
| *** jpena is now known as jpena|lunch | 12:51 | |
| *** wolverineav has joined #openstack-infra | 12:54 | |
| *** wolverineav has quit IRC | 12:59 | |
| *** markvoelker has quit IRC | 13:01 | |
| *** panda|ruck|lunch is now known as panda|ruck | 13:04 | |
| *** trown|outtypewww is now known as trown | 13:16 | |
| *** rh-jelabarre has joined #openstack-infra | 13:17 | |
| *** kgiusti has joined #openstack-infra | 13:26 | |
| *** yamamoto has joined #openstack-infra | 13:27 | |
| *** yamamoto has quit IRC | 13:30 | |
| *** yamamoto has joined #openstack-infra | 13:30 | |
| *** ekultails has joined #openstack-infra | 13:30 | |
| *** mriedem has joined #openstack-infra | 13:32 | |
| *** eharney has quit IRC | 13:33 | |
| *** rlandy has joined #openstack-infra | 13:34 | |
| dtantsur | hi folks! can someone give us some hints why publish jobs stopped running after https://github.com/openstack/ironic-python-agent/commit/cf1127e03c6069ca3b117ff0d05b9e083656fb99 ? | 13:35 |
| openstackgerrit | Roman Gorshunov proposed openstack-infra/irc-meetings master: Update Airship meeting time https://review.openstack.org/640359 | 13:35 |
| dtantsur | it's not that they fail, I can no longer see runs on master at all in http://status.openstack.org/openstack-health/#/job/ironic-python-agent-buildimage-tinyipa | 13:35 |
| dtantsur | oh, I think I see them in http://zuul.openstack.org/builds?job_name=ironic-python-agent-buildimage-tinyipa | 13:37 |
| AJaeger | snevi: the repo is created now and you can use it, see http://git.openstack.org/cgit/openstack/rsd-virt-for-nova/ . The only missing part is adding the original reporter to the ACLs, an infra-root will need to do this for you. Then it's all self-service ;) | 13:38 |
| mnaser | infra-root: there seems to be some NODE_FAILURE around the flavor we use for nested virt.. does anyone have a clue? | 13:38 |
| frickler | dtantsur: there was an issue with log processing, so the last runs may not be have made their way into logstash correctly | 13:39 |
| dtantsur | ah, I see. anyway, we know the problem now. | 13:39 |
| frickler | mnaser: where do you see this? happy to dig into some logs if you point me where to look | 13:40 |
| mnaser | im pinging the containers team.. appparently they've seen it often | 13:41 |
| *** efried is now known as fried_rice | 13:45 | |
| AJaeger | frickler: did you see snevi's request above? Could you help, please? | 13:47 |
| frickler | snevi: AJaeger: infra-root: added helena.mcgough@intel.com to rsd-virt-for-nova-core and rsd-virt-for-nova-release (re https://review.openstack.org/#/c/638633/) | 13:47 |
| frickler | AJaeger: I just did that ;) | 13:47 |
| AJaeger | thanks | 13:48 |
| pabelanger | frickler: adding it to statusbot is also helpful, gives a place we can look outside of irc | 13:48 |
| snevi | AJaeger: Frickler: Thanks you very much for the collaboration!! | 13:49 |
| *** jamesmcarthur has joined #openstack-infra | 13:51 | |
| *** ykarel|afk is now known as ykarel | 13:52 | |
| pabelanger | zbr: left comment on bindep review | 13:52 |
| frickler | pabelanger: I was tagging i-r only to make it easy to see that the earlier request has been resolved. the action itself should be traceable in the gerrit audit log for those groups, so I don't think duplicating that in the status log will be useful | 13:53 |
| *** jpena|lunch is now known as jpena | 13:54 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: [WIP] Add AuthorizationRule Configuration Object, claim validation https://review.openstack.org/639855 | 13:54 |
| openstackgerrit | Ivoline Ngong proposed openstack-infra/storyboard-webclient master: Sort search results by updated_at by default https://review.openstack.org/638690 | 13:55 |
| *** bhavikdbavishi has quit IRC | 13:55 | |
| zbr | pabelanger: replied. is not about presence of tool, is about official packager. | 13:56 |
| zbr | we should not care if optional package managers are installed, only to document which one is the distro one. | 13:57 |
| zbr | pabelanger: mainly to replicate what is exposed by ansible_pkg_mgr variable. | 13:57 |
| *** markvoelker has joined #openstack-infra | 13:58 | |
| *** sdake has joined #openstack-infra | 14:02 | |
| pabelanger | zbr: I think the official pkg manager on rhel8 is yum, thats why I asked. To keep backwards compat with rhel 7 things, trying to find docs but having a hard time | 14:02 |
| *** sthussey has joined #openstack-infra | 14:03 | |
| *** eernst has joined #openstack-infra | 14:08 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: [WIP] Add AuthorizationRule Configuration Object, claim validation https://review.openstack.org/639855 | 14:08 |
| *** TheJulia is now known as needssleep | 14:14 | |
| *** ykarel_ has joined #openstack-infra | 14:16 | |
| *** mcgiggler has quit IRC | 14:17 | |
| zbr | pabelanger: not really, yum executable is only a placeholder for dnf in r8, i double checked now. | 14:17 |
| *** snevi has quit IRC | 14:17 | |
| *** snevi has joined #openstack-infra | 14:17 | |
| *** cmurphy is now known as cmorpheus | 14:17 | |
| *** eernst has quit IRC | 14:17 | |
| pabelanger | zbr: https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/8-beta/html/8.0_beta_release_notes/new-features#software_management_2 | 14:18 |
| pabelanger | according to ansible human, yum is still the default, so I think we should keep bindep using that on rhel8 | 14:18 |
| pabelanger | (maybe centos-8 too) | 14:18 |
| *** ykarel has quit IRC | 14:19 | |
| *** jchhatbar has quit IRC | 14:19 | |
| *** sdake has quit IRC | 14:19 | |
| *** eharney has joined #openstack-infra | 14:19 | |
| zbr | pabelanger: what does ansible returns on it? | 14:20 |
| pabelanger | zbr: checking that now | 14:20 |
| evrardjp | can I have a few votes on https://review.openstack.org/#/c/639779/ ? | 14:21 |
| evrardjp | thank you! | 14:21 |
| evrardjp | :D | 14:21 |
| *** sdake has joined #openstack-infra | 14:21 | |
| *** sdake has quit IRC | 14:24 | |
| *** jpich has quit IRC | 14:24 | |
| *** chandankumar is now known as raukadah | 14:26 | |
| *** jpich has joined #openstack-infra | 14:27 | |
| *** ykarel_ is now known as ykarel | 14:29 | |
| *** markvoelker has quit IRC | 14:31 | |
| pabelanger | zbr: it seems ansible will use /usr/bin/yum, but facts will return dnf. It is pretty confusing right now. So, given the docs for rhel8 say yum is the default, I think we need to do the same in bindep | 14:31 |
| pabelanger | which, yum4 really is dnf | 14:32 |
| pabelanger | it is a mess :( | 14:32 |
| zbr | pabelanger: this will totally break the entire purpose of the feature. | 14:32 |
| zbr | pabelanger: we rely on getting "dnf" on it, not yum. so we rely on ansible behavior, we have tons of code relying on this. | 14:33 |
| *** ccamacho has quit IRC | 14:33 | |
| AJaeger | zbr, so, rename it - do you really need dnf/yum - or do you want "Recent Red Hat family member" | 14:33 |
| zbr | like "if dnf install python3-foo else install python-foo" kind of logic. | 14:33 |
| *** ccamacho has joined #openstack-infra | 14:34 | |
| pabelanger | but on rhel8, the default is yum install python3-foo | 14:36 |
| pabelanger | you optionally could use dnf for that | 14:36 |
| pabelanger | the main reason, was to keep backwards compat for all the user legacy bash scripts | 14:37 |
| pabelanger | I left a -1 for now, mostly to get ianw input. But happy to revist. | 14:37 |
| pabelanger | but, don't think we should update bindep to dnf, if not the default pkgmgr | 14:37 |
| *** kjackal has quit IRC | 14:37 | |
| zbr | pabelanger: can you do a quick bj session? | 14:38 |
| *** sdake has joined #openstack-infra | 14:39 | |
| pabelanger | not currently, have another meeting coming up | 14:39 |
| pabelanger | happy to keep discussing it here | 14:39 |
| AJaeger | pabelanger: the change is not using dnf for installation - bindep does not install anything. It's only an atom to add to bindep.txt. | 14:40 |
| *** roman_g has quit IRC | 14:40 | |
| pabelanger | AJaeger: right, but | 14:41 |
| pabelanger | # dnf covers CentOS>=8, Fedora>=28, RHEL>=8 | 14:41 |
| pabelanger | https://review.openstack.org/#/c/639951/8/bindep/tests/bindep.txt | 14:42 |
| *** roman_g has joined #openstack-infra | 14:42 | |
| pabelanger | that is not true, yum also covers RHEL>8 | 14:42 |
| pabelanger | so, we either need to support both yum / dnf for rhel8 | 14:42 |
| pabelanger | or maybe default to yum on rhel8 | 14:42 |
| AJaeger | pabelanger, zbr, I think saying dnf when you mean "CentOS>=8, Fedora>=28, RHEL>=8" is confusing. | 14:43 |
| AJaeger | we wouldn't have this discussion if the atom would be called "centos8_fedora_28_rhel_8_or_newer" | 14:44 |
| pabelanger | yah | 14:44 |
| AJaeger | but I'm bad in naming ;( | 14:44 |
| pabelanger | I basically think, platform:redhat platform:dnf and platform:redhat platform:yum would match RHEL8 | 14:45 |
| pabelanger | because, both pkgmgr are avaiable | 14:45 |
| *** bnemec is now known as beekneemech | 14:46 | |
| *** luizbag has quit IRC | 14:46 | |
| AJaeger | so, in other words: Let's rename... | 14:46 |
| zbr | pabelanger: lets make ansible do the first step, and fix the ansible_pkg_mgr,... we can follow with bindep after ;) | 14:49 |
| zbr | i am writing a document about the issue now, so we can all work on a good solution | 14:49 |
| *** ccamacho has quit IRC | 14:51 | |
| *** quiquell is now known as quiquell|off | 14:51 | |
| *** ccamacho has joined #openstack-infra | 14:53 | |
| *** snevi has quit IRC | 14:54 | |
| *** snevi has joined #openstack-infra | 14:54 | |
| *** tobias-urdin is now known as tobeass-urdin | 15:03 | |
| *** sdake has quit IRC | 15:10 | |
| *** ekultails has quit IRC | 15:14 | |
| *** wolverineav has joined #openstack-infra | 15:15 | |
| *** roman_g has quit IRC | 15:17 | |
| *** wolverineav has quit IRC | 15:20 | |
| *** bhavikdbavishi has joined #openstack-infra | 15:20 | |
| openstackgerrit | Camila Moura proposed openstack-infra/storyboard master: Fixes the Python 3 dependencies and fixes compatibility issues in migration https://review.openstack.org/639716 | 15:22 |
| *** dpawlik has quit IRC | 15:24 | |
| openstackgerrit | Camila Moura proposed openstack-infra/storyboard master: Fixes the Python 3 dependencies and iterator https://review.openstack.org/639716 | 15:28 |
| *** markvoelker has joined #openstack-infra | 15:28 | |
| *** ekultails has joined #openstack-infra | 15:31 | |
| openstackgerrit | Camila Moura proposed openstack-infra/storyboard master: Fixes the Python 3 dependencies and iterator https://review.openstack.org/639716 | 15:31 |
| *** chason_ has quit IRC | 15:36 | |
| *** jistr is now known as jistr|mtg | 15:38 | |
| *** e0ne has quit IRC | 15:38 | |
| *** e0ne has joined #openstack-infra | 15:39 | |
| *** chason has joined #openstack-infra | 15:41 | |
| *** armax has joined #openstack-infra | 15:44 | |
| *** chason has quit IRC | 15:46 | |
| *** betherly has joined #openstack-infra | 15:48 | |
| *** chason has joined #openstack-infra | 15:52 | |
| *** betherly has quit IRC | 15:53 | |
| *** kjackal has joined #openstack-infra | 15:55 | |
| *** sdake has joined #openstack-infra | 15:58 | |
| *** markvoelker has quit IRC | 16:01 | |
| *** jistr|mtg is now known as jistr | 16:02 | |
| *** ykarel is now known as ykarel|away | 16:03 | |
| *** sdake has quit IRC | 16:03 | |
| *** sdake has joined #openstack-infra | 16:04 | |
| *** kjackal has quit IRC | 16:13 | |
| *** e0ne has quit IRC | 16:14 | |
| *** e0ne has joined #openstack-infra | 16:15 | |
| *** e0ne has quit IRC | 16:17 | |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Open port 80 on zuul-preview https://review.openstack.org/640425 | 16:21 |
| *** kmalloc is now known as needscoffee | 16:22 | |
| *** roman_g has joined #openstack-infra | 16:22 | |
| *** yamamoto has quit IRC | 16:23 | |
| *** sdake has quit IRC | 16:23 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zone-opendev.org master: Switch www.opendev.org to load-balanced gitea farm https://review.openstack.org/640428 | 16:25 |
| corvus | mordred, clarkb, fungi: ^ those two very simple changes fill me with hope and anticipation | 16:26 |
| *** yamamoto has joined #openstack-infra | 16:26 | |
| *** roman_g has quit IRC | 16:26 | |
| *** roman_g has joined #openstack-infra | 16:27 | |
| clarkb | this just occurred to me reviewing the www CNAME change, do we need to worry about lack of ipv6 there for ipv6 only clouds (I suppose those clouds will ipv4 through nat and maybe that is good enough since zuul pushes code into them now and they shouldn't fetch from the gitfarm) | 16:28 |
| corvus | clarkb: yes, also, we expect ipv6 before we really go into production. | 16:29 |
| clarkb | ++ | 16:29 |
| clarkb | I'll work on a the change to have gerrit replicate to those new gitea servers shortly | 16:29 |
| corvus | clarkb: thanks! | 16:30 |
| *** yamamoto has quit IRC | 16:31 | |
| clarkb | looks like gitea-lb01 is redirecting to https://opendev.org? | 16:32 |
| corvus | clarkb: yeah, if you hit it on http | 16:33 |
| corvus | https should work as expected | 16:33 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Handle multiple docker images with the same repository https://review.openstack.org/639779 | 16:36 |
| *** psachin has quit IRC | 16:38 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Replicate to the standalone gitea severs https://review.openstack.org/640431 | 16:38 |
| clarkb | infra-root I think ^ is what we want for gitea stanalone server replication | 16:38 |
| corvus | clarkb: imma gonna edit the commit msg | 16:40 |
| clarkb | sure | 16:40 |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Replicate to the standalone gitea severs https://review.openstack.org/640431 | 16:41 |
| mordred | corvus: I was just about to start working on porting the project creation stuff - did that already get done? | 16:42 |
| corvus | clarkb: lgtm. i marked it wip while we wait on mordred's task to update project creation :) | 16:42 |
| mordred | ok. cool. :) | 16:42 |
| mordred | just making sure | 16:42 |
| corvus | ++ | 16:42 |
| *** pgaxatte has quit IRC | 16:45 | |
| *** ykarel|away has quit IRC | 16:45 | |
| *** ykarel|away has joined #openstack-infra | 16:46 | |
| openstackgerrit | Merged openstack-infra/zone-opendev.org master: Switch www.opendev.org to load-balanced gitea farm https://review.openstack.org/640428 | 16:48 |
| *** jtomasek has quit IRC | 16:52 | |
| *** ijw has joined #openstack-infra | 16:55 | |
| *** ijw has quit IRC | 16:55 | |
| *** ijw has joined #openstack-infra | 16:56 | |
| dmsimard | corvus: regarding ^ I noticed that www.opendev.org properly redirects to gitea but "opendev.org" redirects me to "https://38.108.68.64/", is that patch intended to fix that ? | 16:57 |
| *** ginopc has quit IRC | 16:57 | |
| *** markvoelker has joined #openstack-infra | 16:58 | |
| clarkb | dmsimard: that IP addr is the gitea running in k8s | 16:59 |
| openstackgerrit | James E. Blair proposed openstack-infra/zone-opendev.org master: Switch opendev.org to load-balanced gitea farm https://review.openstack.org/640438 | 16:59 |
| corvus | dmsimard: nope. but that one ^ is. thanks. :) | 16:59 |
| dmsimard | corvus: cool, ++ | 16:59 |
| corvus | and actually... we should probably have www.opendev.org point to something that redirects to opendev.org | 17:00 |
| clarkb | corvus: I don't think @ can be a cname | 17:00 |
| corvus | this is why we have review :) | 17:00 |
| corvus | i don't think we want to end up with people cloning https://www.opendev.org/openstack/nova/ | 17:00 |
| clarkb | corvus: haproxy may be able to do that, but we'd have to terminate ssl in haproxy | 17:01 |
| corvus | clarkb: re cname -- why not? (which rule does this violate) | 17:01 |
| *** kaiokmo has joined #openstack-infra | 17:01 | |
| corvus | clarkb: ah, it's the no other resource type rule | 17:02 |
| clarkb | corvus: its the CNAME uniquenss rule | 17:02 |
| corvus | can't have cname + ns | 17:02 |
| clarkb | yup | 17:02 |
| *** iurygregory has quit IRC | 17:02 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zone-opendev.org master: Switch opendev.org to load-balanced gitea farm https://review.openstack.org/640438 | 17:04 |
| *** gyee has joined #openstack-infra | 17:04 | |
| clarkb | mnaser: NODE_FAILURE means that all clouds that could provide a label failed to do so | 17:05 |
| clarkb | mnaser: we currently have our retries per cloud set to 3. So for each cloud region that can provide a label we try three times before moving on to the next region. If no regions were able to satisfy the request then you get a NODE_FAILURE | 17:06 |
| *** yamamoto has joined #openstack-infra | 17:06 | |
| *** ykarel|away has quit IRC | 17:06 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zone-opendev.org master: Remove www name https://review.openstack.org/640444 | 17:07 |
| corvus | clarkb: ^ i think maybe we'll need to point it to a static apache server somewhere; i say we pull it until that happens | 17:07 |
| clarkb | wfm +2'd both changes | 17:08 |
| clarkb | I did quickly pull up haproxies docs and it can do redirects for us if we want to have it terminate ssl (at least for that name) | 17:09 |
| *** luizbag has joined #openstack-infra | 17:09 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Create git repos on gitea servers before gerrit https://review.openstack.org/640445 | 17:10 |
| mordred | corvus: how does that look? ^^ | 17:10 |
| *** yamamoto has quit IRC | 17:11 | |
| *** bhavikdbavishi has quit IRC | 17:11 | |
| clarkb | infra-root https://review.openstack.org/#/c/630391/6 switches all our dev servers (and some names that I don't think actually exist but we can clean that up in a followup) to puppet4. I'm happy to watch that go in today if we can get reviews on it | 17:11 |
| openstackgerrit | Merged openstack-infra/zone-opendev.org master: Switch opendev.org to load-balanced gitea farm https://review.openstack.org/640438 | 17:12 |
| openstackgerrit | Merged openstack-infra/zone-opendev.org master: Remove www name https://review.openstack.org/640444 | 17:12 |
| mordred | corvus: also - do you have an env setup already that would make testing that locally easy? (I should get one, but I do not currently have one setup) | 17:13 |
| *** kopecmartin is now known as kopecmartin|off | 17:15 | |
| corvus | mordred: nothing special, i'm just running a gitea docker container like this: http://paste.openstack.org/show/746612/ | 17:17 |
| *** bhavikdbavishi has joined #openstack-infra | 17:17 | |
| corvus | mordred: though that should be "image: gitea/gitea" | 17:17 |
| clarkb | oh that reminds me port 222 is the ssh port on the new servers right? | 17:17 |
| clarkb | I need a new patch on my chagne then | 17:17 |
| corvus | clarkb: ha, yes :) | 17:18 |
| *** snevi has quit IRC | 17:18 | |
| mordred | corvus: that doesn't have a mysql with it though, right? just uses built-in sqlite or something? | 17:18 |
| mordred | I should probably docker-compose the docker-compose - cause it's the running of the sql query I'm the most worried about | 17:19 |
| mordred | because quoting | 17:19 |
| corvus | oh yeah. that's correct. | 17:19 |
| corvus | mordred: you can probably crib from the one in system-config. | 17:19 |
| corvus | mordred: tiny suggestion inline on patch | 17:21 |
| *** panda|ruck is now known as panda|ruck|off | 17:21 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Replicate to the standalone gitea severs https://review.openstack.org/640431 | 17:21 |
| corvus | mordred: er, actually, i updated that suggestion, and now i'm a little unsure :) | 17:22 |
| mordred | yeah - I was about to ask if we needed to hit the 3000 | 17:22 |
| corvus | i think so, and it's ssl, so i don't know if that's going to work | 17:22 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Create git repos on gitea servers before gerrit https://review.openstack.org/640445 | 17:22 |
| corvus | mordred: i'll dig into that if you want to keep working on the local mariadb testing | 17:23 |
| mordred | corvus: maybe it needs to be https://localhost:3000 | 17:23 |
| openstackgerrit | Merged openstack-infra/system-config master: Open port 80 on zuul-preview https://review.openstack.org/640425 | 17:25 |
| *** gfidente has quit IRC | 17:26 | |
| corvus | mordred: okay, i think we have to use "https://localhost:3000" and add "validate_certs: false" to all the uri entries | 17:26 |
| mordred | corvus: k. doing that. I'm also making one change to the sql command | 17:27 |
| corvus | i'll go update my rename patch to do the same :) | 17:28 |
| mordred | corvus: I tested the sql using ansible to run a different command against gitea01.opendev.org | 17:28 |
| clarkb | can we tell gitea to also listen on http but only do that on locahost so we don't expose it? I guess that isn't any better than https localhost with no cert validation | 17:28 |
| mordred | so the plumbing at least works | 17:28 |
| corvus | clarkb: i don't think so. and yeah, i don't think it's worth it. | 17:28 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Create git repos on gitea servers before gerrit https://review.openstack.org/640445 | 17:29 |
| mordred | corvus: I thnik that might actually be solid now | 17:29 |
| clarkb | what are repo types 2,3,5,7? | 17:30 |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add gitea to project rename playbook https://review.openstack.org/640218 | 17:30 |
| clarkb | the prime repos | 17:31 |
| *** markvoelker has quit IRC | 17:31 | |
| *** trown is now known as trown|lunch | 17:31 | |
| corvus | it's a repo_unit type | 17:31 |
| corvus | i don't remember exactly which is which, but it's things like wiki, issues, etc | 17:31 |
| clarkb | gotcha | 17:32 |
| corvus | 7 is issues, you can see because we're tweaking it there | 17:32 |
| corvus | so here's a thing... | 17:32 |
| *** roman_g has quit IRC | 17:32 | |
| corvus | the swagger api didn't support setting these things, so we used sql | 17:33 |
| corvus | the swagger api also doesn't support renaming or transferring. but instead of using sql for that, i hit the UI web routes | 17:33 |
| mordred | corvus: we should make the some patches to update the swagger api for all of those things :) | 17:34 |
| corvus | i think that's the better choice for renaming -- that's a complex procedure that touches lots of sql data. it will certainly be easier to keep up with any form parameter changes than it would be to keep up with schema changes | 17:34 |
| mordred | you know - next time we want to write some go | 17:34 |
| corvus | we could probably adapt the repo_unit sql queries to use the same ui web route approach that i did for renaming | 17:35 |
| corvus | mordred: yes, that is definitely the right thing to do long-term | 17:35 |
| clarkb | so they aren't defining the api with swagger just the api clients? | 17:35 |
| mordred | corvus: I think I'm comfortable with the sql route for this for now - it's already there and whatnot | 17:35 |
| clarkb | iirc swagger will do both sides if you let it then there isn't a mismatch | 17:35 |
| corvus | mordred: yeah. i think i am too. if it gives us problems, we could switch it, but if it works, it's fine, and we can just leave it until there's a swagger api. | 17:35 |
| mordred | ++ | 17:36 |
| corvus | clarkb: afaict, the swagger api is completely different than the UI | 17:36 |
| *** ramishra has quit IRC | 17:37 | |
| corvus | so when you push the "update" button on the settings page, it hits /org/repo/settings. when you use the swagger api, it's at /api/v1/repo/yadda/yadda | 17:37 |
| dtantsur | hi folks! can someone please help me understanding what we're doing wrong in http://zuul.openstack.org/build/4319b34ff84c4bc789b950214db650f3 ? | 17:37 |
| clarkb | corvus: huh | 17:37 |
| dtantsur | this is a part of the attempt to move our publish job to zuul v3 | 17:37 |
| mordred | dtantsur: you didn't pay me enough money | 17:38 |
| dtantsur | mordred: I can exchange the chocolate medal you promised me for reviewing the Image nightmare in the SDK :) | 17:38 |
| clarkb | dtantsur: http://logs.openstack.org/5b/5b6bf0b6c86aad352db271cf530a6321ad4248eb/post/ironic-python-agent-buildimage-tinyipa/4319b34/ara-report/reports/7178e260-15c1-4f00-9b5a-5dad4352f301.html shows zero tasks run and that playbook is what does the sync to the executor looks like | 17:39 |
| clarkb | dtantsur: I'm not sure why that is happening but would explain the dir not existing | 17:39 |
| dtantsur | clarkb: this also raised my attention.. what "hosts" should I put for it? maybe that's the reason? | 17:39 |
| dtantsur | the zuul v2 variant had hosts:primary, so we cargo-culted it | 17:40 |
| clarkb | dtantsur: your job specifies the nodeset it wants to use which sets the inventory names | 17:40 |
| *** betherly has joined #openstack-infra | 17:40 | |
| clarkb | in this case you might just want 'all' ? | 17:40 |
| fungi | dtantsur: for a single-node job you can usually just get by with "all" for anything running on the job node | 17:40 |
| corvus | erm | 17:41 |
| fungi | and "localhost" for anything running on the executor server | 17:41 |
| corvus | is that an auto-converted playbook? | 17:41 |
| dtantsur | corvus: not really. it was converted manually by people who have little clue about zuul v3 :) | 17:41 |
| corvus | dtantsur: throw it out :) | 17:41 |
| dtantsur | LOL | 17:42 |
| * dtantsur throws his laptop out the window | 17:42 | |
| corvus | now it works! | 17:42 |
| dtantsur | \o/ | 17:42 |
| dtantsur | so, if you don't value your eyes too much, the resulting post playbook is https://github.com/openstack/ironic-python-agent/blob/master/playbooks/ironic-python-agent-buildimage/post.yaml | 17:42 |
| corvus | dtantsur: that looks like it was at least based on something that the autoconvert script made. the thing to know about that is that it produces complete gibberish, and no one understands how it works (including its authors). and it can usually be replaced with one line of yaml anyway. | 17:43 |
| fungi | but yeah, "primary" is generally a reference to legacy nodesets where we used that to emulate the old style of nodepool multi-node groups which had a separate primary node | 17:43 |
| dtantsur | corvus: I'm eager to learn this one line of yaml | 17:44 |
| corvus | yeah, that's the trick | 17:44 |
| corvus | lemme dig around a bit and understand the job more | 17:44 |
| dtantsur | corvus: tl;dr build ironic-python-agent images and push them to tarballs.o.o | 17:45 |
| *** sdake has joined #openstack-infra | 17:45 | |
| clarkb | mordred: couple of thoughts for minor improvements on https://review.openstack.org/#/c/640445/3 but +2'd as should work as is | 17:46 |
| corvus | dtantsur: any reason you don't build the image in check? | 17:46 |
| dtantsur | corvus: and publish from check? | 17:46 |
| corvus | dtantsur: no, just to see if it buids | 17:47 |
| dtantsur | corvus: ah. yes, we do it as part of the CI runs. | 17:47 |
| dtantsur | but we only publish in post | 17:47 |
| * mordred needs to afk for a few real quick to get a sandwich before a phone call in an hour - will BRB | 17:49 | |
| *** betherly has quit IRC | 17:50 | |
| clarkb | frickler: if you are still around, can you review https://review.openstack.org/#/c/639740/ this is the edit secrets file command script change | 17:50 |
| corvus | dtantsur: so for that post playbook which isn't running -- http://logs.openstack.org/5b/5b6bf0b6c86aad352db271cf530a6321ad4248eb/post/ironic-python-agent-buildimage-tinyipa/4319b34/ara-report/ | 17:51 |
| corvus | dtantsur: can you walk me through that? | 17:51 |
| *** florianf has quit IRC | 17:51 | |
| corvus | dtantsur: like, what is it that the run playbook produces, and what is it you want to end up on the fileserver? | 17:51 |
| dtantsur | corvus: they key part is this bunch of files https://github.com/openstack/ironic-python-agent/blob/master/playbooks/ironic-python-agent-buildimage/run.yaml#L42-L44 that should end up in https://tarballs.openstack.org/ironic-python-agent/tinyipa/files/ | 17:52 |
| dtantsur | correction: UPLOAD_TAR should end up in https://tarballs.openstack.org/ironic-python-agent/tinyipa/, UPLOAD_RAW - in files | 17:53 |
| dtantsur | this was supposed to be done by https://github.com/openstack/ironic-python-agent/blob/master/playbooks/ironic-python-agent-buildimage/post.yaml#L10-L30 (and it worked with old style jobs) | 17:53 |
| *** betherly has joined #openstack-infra | 17:54 | |
| corvus | dtantsur: it's unfortunate that mordred left, because he and pabelanger were working on a simplified method of fetching artifacts from remote hosts, however, that's in the fetch-output role, which runs in the base job post playbook, which runs *after* the artifact upload playbook in openstack-artifacts.yaml. so i do not know how they intend for that to be used. | 17:55 |
| *** mriedem is now known as mriedem_lunch | 17:56 | |
| * dtantsur knew that mordred was to blame | 17:56 | |
| dtantsur | corvus: do you think that changing to hosts:all will at least provide a relief? | 17:56 |
| corvus | dtantsur: absent an explanation there, i think the shortest path to working is the direction clarkb was heading -- get that post playbook running again by using hosts:all | 17:56 |
| dtantsur | okay, let us try this | 17:57 |
| corvus | dtantsur: that's still a really scary playbook. i hope later we can update it so that it just moves some files into {{ansible_user_dir}}/artifacts and then somehow use fetch-output to pull it to the executor. | 17:58 |
| dtantsur | I'll be happy to, but some guidance will be required | 17:59 |
| corvus | yeah, i don't know the plan either :( | 17:59 |
| clarkb | infra-root https://review.openstack.org/#/c/636734/ is a change that I don't want to get completely lost (even if we decide not to go with it). It will reduce the number of github api roundtrips we make by stopping our constant retrieval of api rate limit data | 17:59 |
| corvus | i think maybe fetch-output works for logs, but probably hasn't been used for artifacts yet | 18:00 |
| dtantsur | thanks anyway corvus | 18:00 |
| * dtantsur has proposed a patch and now has to go | 18:00 | |
| *** dtantsur is now known as dtantsur|afk | 18:00 | |
| *** harlowja has joined #openstack-infra | 18:01 | |
| *** ykarel|away has joined #openstack-infra | 18:03 | |
| *** jpena is now known as jpena|off | 18:03 | |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Create git repos on gitea servers before gerrit https://review.openstack.org/640445 | 18:04 |
| corvus | mordred, clarkb: ^ fixed the docs error | 18:04 |
| *** jaosorior has quit IRC | 18:04 | |
| clarkb | infra-root Zhenguo Niu confirms that nimble is no longer needed and that we should feel free to do the pypi ownership swap. I'll be reaching out to devin directly to get details about pypi users and can add them as owner as soon as I have that info | 18:05 |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Clone project-config to /opt on gitea servers https://review.openstack.org/640479 | 18:06 |
| corvus | mordred, clarkb: ^ and that's a minor followup to mordred's change | 18:06 |
| *** IvensZambrano has joined #openstack-infra | 18:07 | |
| *** betherly has quit IRC | 18:07 | |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Create git repos on gitea servers before gerrit https://review.openstack.org/640445 | 18:07 |
| corvus | sorry, just adding task headers | 18:07 |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Clone project-config to /opt on gitea servers https://review.openstack.org/640479 | 18:08 |
| *** ijw has quit IRC | 18:08 | |
| *** eharney has quit IRC | 18:09 | |
| clarkb | cmorpheus: if you have a moment for a puppet review, your input on https://review.openstack.org/#/c/639442/ would be helpful. Not sure if there is a simpler way to do that in puppet | 18:09 |
| *** jpich has quit IRC | 18:10 | |
| *** wolverineav has joined #openstack-infra | 18:10 | |
| *** irclogbot_1 has joined #openstack-infra | 18:10 | |
| *** wolverineav has quit IRC | 18:10 | |
| *** wolverineav has joined #openstack-infra | 18:10 | |
| *** betherly has joined #openstack-infra | 18:11 | |
| *** IvensZambrano has quit IRC | 18:13 | |
| cmorpheus | clarkb: hmm it's weird that path => '/usr/local/bin:/usr/bin:/bin/ doesn't just work | 18:14 |
| clarkb | infra-root while we've got DNS context paged in https://review.openstack.org/#/c/622620/2 should make reboots of our name servers more reliable | 18:15 |
| clarkb | cmorpheus: ya I think the issue is in production pip is installed by puppet and by then puppet has already cached PATH contents? | 18:15 |
| openstackgerrit | Merged openstack-infra/system-config master: Document kerberos stash file requirement https://review.openstack.org/638779 | 18:15 |
| *** betherly has quit IRC | 18:15 | |
| fungi | something about how the pip provider decides how/where to invoke pip, i think? | 18:16 |
| fungi | i used to know some of the internals there because of our pip3 provider fork | 18:16 |
| *** betherly has joined #openstack-infra | 18:17 | |
| fungi | there were some fancy rules which determined the name and path to pip (since it also might not just be called "pip") | 18:17 |
| clarkb | fungi: in this case we exec pip so not going through that code | 18:17 |
| clarkb | but I think it likely is a similar problem | 18:17 |
| fungi | ahh | 18:18 |
| fungi | not using a pip package provider then | 18:18 |
| clarkb | corvus: thanks I'll reboot one of the ns servers after that is applied to triple check it works as freedesktop says | 18:20 |
| *** jamesmcarthur has quit IRC | 18:20 | |
| *** jamesmcarthur has joined #openstack-infra | 18:20 | |
| cmorpheus | i wonder if using the environment option to set PATH would work better | 18:20 |
| *** derekh has quit IRC | 18:22 | |
| mordred | corvus: do you want me to switch the defaults from tmp to opt still? | 18:22 |
| *** wolverineav has quit IRC | 18:23 | |
| mordred | corvus: oh - i see - you did that as a followup | 18:24 |
| *** jamesmcarthur has quit IRC | 18:24 | |
| *** wolverineav has joined #openstack-infra | 18:25 | |
| *** mrhillsman is now known as mrhillsman_lunch | 18:28 | |
| *** markvoelker has joined #openstack-infra | 18:28 | |
| *** betherly has quit IRC | 18:30 | |
| *** ijw has joined #openstack-infra | 18:31 | |
| clarkb | mwhahaha: looks like requirements tarball publishing jobs are running successfully on the master branch now | 18:34 |
| mwhahaha | yea saw that | 18:34 |
| clarkb | I don't see chagnes for the stable branches so I'll push them up now | 18:34 |
| mwhahaha | kolla got broke by some other neutron requirements thing | 18:34 |
| mwhahaha | https://review.openstack.org/#/c/639872/ | 18:34 |
| mwhahaha | seems to have broken a few other projects as well | 18:35 |
| clarkb | ocata, pike, queens and rocky branches in requirements all have that fix cherry picked and proposed now | 18:37 |
| *** trown|lunch is now known as trown | 18:37 | |
| *** betherly has joined #openstack-infra | 18:37 | |
| *** betherly has quit IRC | 18:42 | |
| *** yboaron_ has quit IRC | 18:42 | |
| fungi | gonna go grab lunch, then come back and hopefully get started writing up the git redirect sites change. bbiab | 18:43 |
| *** betherly has joined #openstack-infra | 18:45 | |
| *** ijw has quit IRC | 18:46 | |
| prometheanfire | clarkb: guess it was needed :P | 18:46 |
| *** bhavikdbavishi has quit IRC | 18:46 | |
| clarkb | prometheanfire: ya | 18:47 |
| *** ykarel|away has quit IRC | 18:48 | |
| *** ijw has joined #openstack-infra | 18:50 | |
| *** wolverineav has quit IRC | 18:50 | |
| openstackgerrit | Merged openstack-infra/system-config master: Update nsd systemd unit deps https://review.openstack.org/622620 | 18:51 |
| corvus | mordred: http://site.926bb0aaddad4bc3853269451e115dcb.openstack.zuul-preview.opendev.org/ | 18:51 |
| corvus | JpMaxMan: ^ | 18:51 |
| *** electrofelix has quit IRC | 18:52 | |
| JpMaxMan | woah! | 18:52 |
| JpMaxMan | nice :) | 18:52 |
| corvus | that came from https://review.openstack.org/635924 | 18:52 |
| corvus | still a little bit more to do to plug it all in so that zuul reports that url | 18:52 |
| *** Adri2000 has joined #openstack-infra | 18:53 | |
| corvus | but all the pieces are in place :) | 18:53 |
| JpMaxMan | that's great news | 18:53 |
| JpMaxMan | very cool little tool!! | 18:53 |
| *** wolverineav has joined #openstack-infra | 18:53 | |
| AJaeger | corvus: woot! | 18:53 |
| *** yamamoto has joined #openstack-infra | 18:54 | |
| *** betherly has quit IRC | 18:57 | |
| mordred | corvus: \o/ | 18:59 |
| *** yamamoto has quit IRC | 18:59 | |
| *** sdake has quit IRC | 19:00 | |
| clarkb | rereviwing the zuul-preview stuff, the zuul-preview image must install apache right? | 19:01 |
| corvus | clarkb: yes | 19:01 |
| clarkb | (we aren't running apache as a separte image) | 19:01 |
| *** betherly has joined #openstack-infra | 19:01 | |
| *** markvoelker has quit IRC | 19:01 | |
| *** betherly has quit IRC | 19:07 | |
| *** ijw has quit IRC | 19:08 | |
| *** e0ne has joined #openstack-infra | 19:12 | |
| *** wolverineav has quit IRC | 19:12 | |
| *** kjackal has joined #openstack-infra | 19:21 | |
| *** rfolco|rover has quit IRC | 19:21 | |
| *** luizbag has quit IRC | 19:24 | |
| *** ijw has joined #openstack-infra | 19:25 | |
| *** betherly has joined #openstack-infra | 19:26 | |
| *** mriedem_lunch is now known as mriedem | 19:28 | |
| *** mrhillsman_lunch is now known as mrhillsman | 19:29 | |
| *** wolverineav has joined #openstack-infra | 19:30 | |
| *** betherly has quit IRC | 19:30 | |
| *** e0ne has quit IRC | 19:31 | |
| openstackgerrit | Riju Khatri proposed openstack-infra/storyboard-webclient master: resolves task 26745 of story 2003883 : fixed project group view checkboxes https://review.openstack.org/640496 | 19:32 |
| *** ijw has quit IRC | 19:33 | |
| *** auristor has quit IRC | 19:34 | |
| *** eharney has joined #openstack-infra | 19:35 | |
| *** auristor has joined #openstack-infra | 19:36 | |
| *** betherly has joined #openstack-infra | 19:39 | |
| *** tosky has quit IRC | 19:42 | |
| *** tosky has joined #openstack-infra | 19:43 | |
| mordred | clarkb: that's right - because apache actually spawns zuul-preview as a subprocess and talks to it over stdin/stdout | 19:43 |
| mordred | clarkb: so apache is actually the main entrypoint of the container / what the container is running | 19:44 |
| mordred | and zuul-preview is part of the confif | 19:44 |
| mordred | config | 19:44 |
| *** dmellado has quit IRC | 19:44 | |
| *** dmellado has joined #openstack-infra | 19:45 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Return npm site location as artifact https://review.openstack.org/640498 | 19:45 |
| corvus | mordred: https://opendev.org/ exists! | 19:46 |
| corvus | mordred: i've approved https://review.openstack.org/640445 | 19:47 |
| mordred | corvus: fingers crossed! | 19:48 |
| *** betherly has quit IRC | 19:48 | |
| *** irclogbot_1 has quit IRC | 19:50 | |
| *** ijw has joined #openstack-infra | 19:55 | |
| openstackgerrit | Merged openstack-infra/puppet-etherpad_lite master: Bump eplite version in rspec tests https://review.openstack.org/611000 | 19:55 |
| *** wolverineav has quit IRC | 19:56 | |
| *** rlandy is now known as rlandy|brb | 19:57 | |
| *** wolverineav has joined #openstack-infra | 19:57 | |
| *** markvoelker has joined #openstack-infra | 19:58 | |
| *** ijw has quit IRC | 20:00 | |
| *** irclogbot_1 has joined #openstack-infra | 20:01 | |
| *** wolverineav has quit IRC | 20:02 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remove Sign In link from gitea https://review.openstack.org/640503 | 20:02 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remove Sign In link from gitea https://review.openstack.org/640503 | 20:02 |
| dmsimard | corvus: gitea is empty for me, it wasn't earlier today though | 20:02 |
| clarkb | dmsimard: yes 640445 will address that (as well as my followup to replicate to the new servers) | 20:03 |
| dmsimard | ++ you guys are way ahead of me :p | 20:03 |
| mordred | corvus: ^^ I think that should do | 20:03 |
| mordred | it | 20:04 |
| *** e0ne has joined #openstack-infra | 20:05 | |
| fungi | okay, i'm back and probably ready to get some stuff done. trying to catch up now | 20:06 |
| fungi | ooh, netlify progress! | 20:07 |
| clarkb | infra-root ns2 has bene updated with the systemd fix. Any opposition to me rebootingthat server now to triple check ti comes up with a running named? | 20:07 |
| fungi | all the exciting stuff i almost miss by being a slave to sustenance | 20:07 |
| mordred | clarkb: ++ | 20:07 |
| mordred | fungi: stupid sustenance | 20:07 |
| * fungi notes https://opendev.org/ is one character longer than the equivalent gh url. hopefully properly free and open software is worth typing one more character | 20:10 | |
| mordred | fungi: my biggest issue is that when my fingers start typing open they have a habit of then typing stack | 20:11 |
| corvus | yep. i am going through retraining now. | 20:11 |
| fungi | there was some astronomical cost related to registering open.dev or the osf would have considered that as an option for us | 20:11 |
| *** EvilienM is now known as EmilienM | 20:14 | |
| *** betherly has joined #openstack-infra | 20:16 | |
| mordred | oh I'm sure that one was spendy | 20:16 |
| openstackgerrit | Merged openstack-infra/system-config master: Create git repos on gitea servers before gerrit https://review.openstack.org/640445 | 20:16 |
| mordred | BOOM | 20:16 |
| openstackgerrit | Merged openstack-infra/system-config master: Clone project-config to /opt on gitea servers https://review.openstack.org/640479 | 20:17 |
| *** sdake has joined #openstack-infra | 20:18 | |
| clarkb | my big issue is tab is far less useful for me | 20:18 |
| *** sdake has quit IRC | 20:18 | |
| clarkb | ok rebooting ns2.opendev.org now | 20:18 |
| fungi | clarkb: we can probably register opentab.org if it helps you | 20:20 |
| *** betherly has quit IRC | 20:21 | |
| mordred | fungi: could we just start selling Tab instead of building clouds? | 20:21 |
| fungi | the 80s will thank us | 20:22 |
| mordred | fungi: you may or may not find it amusing that your nick is currently Tab colored in my irc client | 20:22 |
| clarkb | hrm nsd didn't start because it couldn't bind to those interfaces | 20:22 |
| clarkb | now I wonder if network-online.target is just fubar on ubuntu | 20:23 |
| *** sdake has joined #openstack-infra | 20:24 | |
| clarkb | according to the logs ens3 was configured before nsd starts | 20:26 |
| *** derekh has joined #openstack-infra | 20:26 | |
| clarkb | oh but the ens3 logs only show ipv4 being online | 20:27 |
| clarkb | I wonder if network-online is any ip version is up and running | 20:27 |
| clarkb | and then we fail when we try to bind to ipv6 address | 20:27 |
| *** wolverineav has joined #openstack-infra | 20:28 | |
| clarkb | maybe it would be simpler to not unbound on those servers and listen globally? | 20:28 |
| clarkb | I'll have to think about that more | 20:28 |
| fungi | mordred: come back to me when it's moxie-colored | 20:29 |
| clarkb | Devin says they aren't ready to take over the nimble name now but will reach out to me when they are | 20:30 |
| *** markvoelker has quit IRC | 20:31 | |
| fungi | clarkb: hrm, is unbound trying to bind to interfaces other than loopback? or nsd trying to bind to loopback? | 20:31 |
| fungi | an alternative would be to configure one of them to not use actual port 53 and redirect on the relevant interface with iptables (1:1 nat basically) | 20:32 |
| clarkb | fungi: nsd is trying to bind to our actual public interfaces because nsd is on localhost so you can't bind :: as nsd | 20:32 |
| clarkb | er because unbound is on localhost | 20:32 |
| fungi | right | 20:32 |
| clarkb | and if we could bind to :: we wouldn't have to worry about network startup because :: is magical in the linux kernel and just works (tm) | 20:33 |
| fungi | i know that was the original idea. curious where the conflict is | 20:33 |
| clarkb | the conflict is nsd starts before we have networking fully up | 20:33 |
| clarkb | so fails to start | 20:33 |
| *** wolverineav has quit IRC | 20:33 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Return npm site location as artifact https://review.openstack.org/640498 | 20:33 |
| *** betherly has joined #openstack-infra | 20:33 | |
| clarkb | To address this I switched the unit to start after network-online.target and reading the docs my undertanding is that is only waiting for ipv4 to compe up | 20:34 |
| fungi | oh, i see, it tries to start before our public interface is configured? | 20:34 |
| clarkb | possibly because we wait for the RA to happen every 30 seconds or whatever it is so our ipv6 addr isn't there yet | 20:34 |
| clarkb | fungi: yes | 20:34 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-preview master: Use artifact name rather than log_url https://review.openstack.org/640509 | 20:35 |
| *** rfolco has joined #openstack-infra | 20:37 | |
| *** betherly has quit IRC | 20:37 | |
| *** betherly has joined #openstack-infra | 20:38 | |
| fungi | clarkb: do we listen by interface name or by address? | 20:40 |
| mnaser | eh | 20:41 |
| mnaser | as someone who has been transforming all infra to ipv | 20:41 |
| mnaser | ipv6 | 20:41 |
| clarkb | fungi: ipaddress | 20:42 |
| *** sdake has quit IRC | 20:42 | |
| mnaser | i strongly suggest configuring things as static and disabling autoconf | 20:42 |
| mnaser | it has been so flakey for me :( | 20:42 |
| clarkb | mnaser: via config drive data? | 20:42 |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Add windmill-config project https://review.openstack.org/640510 | 20:42 |
| fungi | glean can do that for us at least, right? | 20:42 |
| *** yamamoto has joined #openstack-infra | 20:42 | |
| mnaser | clarkb: yeah. plop it right into /etc/network/interfaces or whatever | 20:43 |
| fungi | on an unrelated note, were we expecting to do something to hide the "sign in" link at https://opendev.org/ ? | 20:43 |
| mnaser | what we do is actually we have an ansible role that gets ansible_ipv4_default or whatever and creates an ifcfg file from it | 20:43 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-preview master: Use artifact name rather than log_url https://review.openstack.org/640509 | 20:43 |
| clarkb | fungi: it can if we boot with config drive. I think up to date cloud init can too but that isn't present on xenail and older by default. unsure of bionic | 20:43 |
| clarkb | fungi: yes there are some chagnes above | 20:43 |
| clarkb | fungi: https://review.openstack.org/640503 | 20:43 |
| fungi | thanks!!! | 20:44 |
| * fungi feels so out of touch lately | 20:44 | |
| fungi | wow, "Remove Sign In link from gitea" involves adding 126 lines of html. clearly they didn't intend to make this optional :/ | 20:46 |
| *** betherly has quit IRC | 20:47 | |
| *** yamamoto has quit IRC | 20:48 | |
| clarkb | fungi: I think we are replacing the html | 20:48 |
| clarkb | and then we'll bake it in via an image rebuild | 20:48 |
| fungi | yeah | 20:49 |
| fungi | that's what it looks like to me anyway | 20:50 |
| fungi | hence my "not optional" comment | 20:50 |
| *** rlandy|brb is now known as rlandy | 20:51 | |
| *** wolverineav has joined #openstack-infra | 20:51 | |
| *** kgiusti has left #openstack-infra | 21:03 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Fix typo in docker push role https://review.openstack.org/640515 | 21:06 |
| *** jamesmcarthur has joined #openstack-infra | 21:07 | |
| corvus | fungi: yes, but they *did* intend for us to replace the html as needed :) | 21:08 |
| *** jamesmcarthur has quit IRC | 21:09 | |
| *** jamesmcarthur has joined #openstack-infra | 21:10 | |
| *** betherly has joined #openstack-infra | 21:11 | |
| corvus | clarkb: can you re-review https://review.openstack.org/638794 ? | 21:11 |
| clarkb | corvus: done. And now my turn for food | 21:12 |
| fungi | corvus: fair point | 21:13 |
| *** jamesmcarthur has quit IRC | 21:14 | |
| *** ldnunes has quit IRC | 21:15 | |
| openstackgerrit | Merged opendev/base-jobs master: Save registry logs https://review.openstack.org/638794 | 21:19 |
| *** betherly has quit IRC | 21:23 | |
| *** betherly has joined #openstack-infra | 21:27 | |
| *** markvoelker has joined #openstack-infra | 21:28 | |
| *** jamesmcarthur has joined #openstack-infra | 21:29 | |
| *** ijw has joined #openstack-infra | 21:32 | |
| mordred | clarkb, corvus: what am I missing: | 21:34 |
| mordred | 2019-03-01 21:24:47,587 p=11773 u=root | fatal: [gitea04.opendev.org]: FAILED! => {"msg": "An unhandled exception occurred while running the lookup plugin 'file'. Error was a <class 'ansible.errors.AnsibleError'>, original message: could not locate file in lookup: /opt/project_config/gerrit/projects.yaml"} | 21:34 |
| corvus | mordred: my first thought is: file lookups are local | 21:34 |
| mordred | but I see /opt/project_config ... | 21:35 |
| mordred | OH | 21:35 |
| mordred | yeah | 21:35 |
| mordred | so really this can skip cloning project_config and can just read from the one on bridge | 21:35 |
| corvus | mordred: yeah, i think so | 21:35 |
| mordred | fix coming | 21:35 |
| mordred | corvus: tahnks - I would not have gotten there for a while | 21:35 |
| corvus | mordred: i've hit it enough it's a reflex | 21:36 |
| mordred | corvus: while you're fixing thigns for me - do you know where my other cast iron skillet is? | 21:36 |
| *** reed has joined #openstack-infra | 21:36 | |
| *** betherly has quit IRC | 21:36 | |
| *** irclogbot_1 has quit IRC | 21:36 | |
| corvus | mordred: check in the fire | 21:36 |
| reed | it's been a while :) | 21:36 |
| * reed waves | 21:36 | |
| corvus | reed: howdy! | 21:37 |
| *** betherly has joined #openstack-infra | 21:39 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remember that lookups are local https://review.openstack.org/640537 | 21:39 |
| mordred | corvus: ^^ | 21:40 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Fix typo in docker push role https://review.openstack.org/640515 | 21:40 |
| corvus | mordred: is that going to update it to master each time it runs? | 21:41 |
| mordred | corvus: yes? | 21:47 |
| mordred | corvus: we might want force: yes added like we do for system-config | 21:48 |
| *** tobiash has quit IRC | 21:49 | |
| *** tobiash has joined #openstack-infra | 21:51 | |
| *** betherly has quit IRC | 21:52 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remember that lookups are local https://review.openstack.org/640537 | 21:55 |
| mordred | corvus: how about that instead? | 21:55 |
| *** betherly has joined #openstack-infra | 21:55 | |
| *** markvoelker has quit IRC | 22:01 | |
| *** wolverineav has quit IRC | 22:04 | |
| *** wolverineav has joined #openstack-infra | 22:05 | |
| *** jamesmcarthur has quit IRC | 22:07 | |
| *** betherly has quit IRC | 22:08 | |
| *** jamesmcarthur has joined #openstack-infra | 22:08 | |
| *** rlandy has quit IRC | 22:10 | |
| mordred | clarkb, fungi: ^^ feel like a +A? | 22:11 |
| clarkb | mordred: looking (sorry I've been nerd sniped using zuul-preview rust version to learn about moving variables and borrowing and all that fun stuff. It turns out that closures are a really good playground for that) | 22:12 |
| clarkb | mordred: did the other hosts need that clone to not be local? | 22:13 |
| clarkb | oh no we just needed the ref | 22:13 |
| mordred | clarkb: yah. so we didnt' clone because we didnt' need to | 22:14 |
| clarkb | mordred: left a comment you might want to address (but +2'd if you want to followup) | 22:15 |
| mordred | clarkb: I'll just followup | 22:16 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Update playbook doc comment for correctness https://review.openstack.org/640545 | 22:17 |
| *** rh-jelabarre has quit IRC | 22:18 | |
| *** kjackal has quit IRC | 22:19 | |
| openstackgerrit | Josef Wells proposed openstack-infra/zuul master: quickstart: web and others wait on mysql to start https://review.openstack.org/640548 | 22:22 |
| *** e0ne has quit IRC | 22:24 | |
| *** ijw has quit IRC | 22:25 | |
| *** ijw has joined #openstack-infra | 22:29 | |
| *** yamamoto has joined #openstack-infra | 22:31 | |
| *** wolverineav has quit IRC | 22:35 | |
| *** yamamoto has quit IRC | 22:35 | |
| *** jamesmcarthur has quit IRC | 22:40 | |
| *** jamesmcarthur has joined #openstack-infra | 22:44 | |
| *** ajo_ has joined #openstack-infra | 22:45 | |
| *** sadin999_ has joined #openstack-infra | 22:45 | |
| *** jamespage_ has joined #openstack-infra | 22:45 | |
| *** zaro_ has joined #openstack-infra | 22:45 | |
| *** guilhermesp__ has joined #openstack-infra | 22:45 | |
| *** abelur_ has joined #openstack-infra | 22:45 | |
| *** odyssey4me_ has joined #openstack-infra | 22:45 | |
| *** beisner_ has joined #openstack-infra | 22:45 | |
| *** mwhahaha_ has joined #openstack-infra | 22:45 | |
| *** jbryce_ has joined #openstack-infra | 22:45 | |
| *** seongsoocho_ has joined #openstack-infra | 22:45 | |
| *** betherly has joined #openstack-infra | 22:46 | |
| clarkb | should I remove the WIP from https://review.openstack.org/#/c/640431/ ? | 22:46 |
| clarkb | that won't take effect until we restart gerrit so might want to get that in place concurrently with the work to create the projects? | 22:47 |
| corvus | clarkb: we need the projects to be created before we replicate | 22:47 |
| corvus | same as git.o.o | 22:47 |
| *** ijw_ has joined #openstack-infra | 22:47 | |
| *** Tengu_ has joined #openstack-infra | 22:47 | |
| mordred | corvus: yeah - but I think he's saying we have to restart gerrit for it to go live - so we could go ahead and land the patch | 22:47 |
| corvus | and if gerrit is restarted before then, we have a big cleanup job. i'd rather wait | 22:48 |
| *** ijw has quit IRC | 22:48 | |
| mordred | ++ | 22:48 |
| clarkb | I'm fine with waiting. | 22:48 |
| *** jamesmcarthur has quit IRC | 22:48 | |
| mordred | I can go ahead and review it though | 22:48 |
| *** wolverineav has joined #openstack-infra | 22:50 | |
| *** eharney has quit IRC | 22:52 | |
| *** Tengu has quit IRC | 22:52 | |
| *** zaro has quit IRC | 22:52 | |
| *** HenryG has quit IRC | 22:52 | |
| *** odyssey4me has quit IRC | 22:52 | |
| *** nicolasbock has quit IRC | 22:52 | |
| *** mwhahaha has quit IRC | 22:52 | |
| *** vkmc has quit IRC | 22:52 | |
| *** xgerman has quit IRC | 22:52 | |
| *** eglute has quit IRC | 22:52 | |
| *** sadin999 has quit IRC | 22:52 | |
| *** lamt has quit IRC | 22:52 | |
| *** seongsoocho has quit IRC | 22:52 | |
| *** guilhermesp has quit IRC | 22:52 | |
| *** jbryce has quit IRC | 22:52 | |
| *** melwitt has quit IRC | 22:52 | |
| *** beisner has quit IRC | 22:52 | |
| *** ajo has quit IRC | 22:52 | |
| *** Ng has quit IRC | 22:52 | |
| *** abelur has quit IRC | 22:52 | |
| *** jamespage has quit IRC | 22:52 | |
| *** cjloader has quit IRC | 22:52 | |
| *** sadin999_ is now known as sadin999 | 22:52 | |
| *** guilhermesp__ is now known as 18WAAFJGJ | 22:52 | |
| *** abelur_ is now known as abelur | 22:52 | |
| *** beisner_ is now known as beisner | 22:52 | |
| *** ajo_ is now known as ajo | 22:52 | |
| *** seongsoocho_ is now known as seongsoocho | 22:52 | |
| *** odyssey4me_ is now known as odyssey4me | 22:52 | |
| *** zaro_ is now known as zaro | 22:52 | |
| *** jbryce_ is now known as jbryce | 22:52 | |
| *** jamespage_ is now known as jamespage | 22:52 | |
| *** melwitt has joined #openstack-infra | 22:53 | |
| *** vkmc has joined #openstack-infra | 22:53 | |
| *** mwhahaha_ is now known as mwhahaha | 22:53 | |
| *** kazsh has quit IRC | 22:55 | |
| *** Ng has joined #openstack-infra | 22:58 | |
| *** kazsh has joined #openstack-infra | 22:59 | |
| *** eharney has joined #openstack-infra | 22:59 | |
| *** markvoelker has joined #openstack-infra | 22:59 | |
| *** betherly has quit IRC | 22:59 | |
| *** jesusaur has quit IRC | 23:03 | |
| fungi | yeah, safe to review, not really safe to approve far in advance | 23:05 |
| fungi | but we have some control over when it takes effect | 23:05 |
| clarkb | 640537 should head to the gate soonish and when that merges we should create projects in gitea aiui | 23:06 |
| clarkb | then we can either restart gerrit late today or do it on monday? | 23:06 |
| *** betherly has joined #openstack-infra | 23:06 | |
| *** frickler has quit IRC | 23:06 | |
| *** frickler has joined #openstack-infra | 23:06 | |
| *** rascasoft has quit IRC | 23:07 | |
| clarkb | infra-root anyone else want to review https://review.openstack.org/#/c/630391/6 to do more puppet 4, https://review.openstack.org/639740 script for passwords/secrets editing, https://review.openstack.org/636734 reduce number of github requests that zuul does | 23:07 |
| fungi | oh, wow, i took an unvoluntary friday nap and nearly missed a wayward reed! | 23:08 |
| *** mriedem is now known as mriedem_afk | 23:08 | |
| *** jesusaur has joined #openstack-infra | 23:08 | |
| clarkb | I'm going to pop out for a bit now but will be back to watch project creation and/or that set of chagnes above | 23:08 |
| fungi | (for the record, clarkb's rust excuse was waaay better) | 23:09 |
| clarkb | fungi I took the script and removed the to_string I thought wasnt necessary then went down the rabbit hole of learning why it was necessary | 23:10 |
| *** betherly has quit IRC | 23:11 | |
| fungi | sounds like a marvellous lesson i won't learn until i attempt to do the same at some point | 23:12 |
| *** jesusaur has quit IRC | 23:14 | |
| *** betherly has joined #openstack-infra | 23:16 | |
| *** ekultails has quit IRC | 23:20 | |
| mordred | it's important to know why to_string is necessary | 23:23 |
| *** jesusaur has joined #openstack-infra | 23:23 | |
| *** betherly has quit IRC | 23:25 | |
| clarkb | 17 minutes to merging the project creation fix | 23:29 |
| *** betherly has joined #openstack-infra | 23:29 | |
| *** markvoelker has quit IRC | 23:31 | |
| *** hwoarang has quit IRC | 23:39 | |
| *** betherly has quit IRC | 23:43 | |
| openstackgerrit | Merged openstack-infra/system-config master: Remember that lookups are local https://review.openstack.org/640537 | 23:45 |
| clarkb | mordred: ^ ansible run after the current should include that | 23:48 |
| clarkb | will start at 00:00UTC | 23:48 |
| *** hwoarang has joined #openstack-infra | 23:49 | |
| *** wolverineav has quit IRC | 23:55 | |
| *** wolverineav has joined #openstack-infra | 23:55 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Fix buildset registry https://review.openstack.org/640557 | 23:56 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Fix docker image retag https://review.openstack.org/640558 | 23:56 |
| corvus | clarkb, fungi, mordred: ^ that's a bit of a redo on the buildset registry thing. it turns out that the second issue (640558) caused the buildset registry proxy to fail to serve an image at all, which is what alerted me to the issue corrected by 640557. | 23:57 |
| corvus | we need to land both of those, and then i can delete the current zuul-preview image on dockerhub, then we should be able to land that change and push it up. | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!