| *** ansmith has joined #openstack-infra | 00:00 | |
| *** eernst has joined #openstack-infra | 00:07 | |
| *** eernst has quit IRC | 00:08 | |
| *** eernst has joined #openstack-infra | 00:18 | |
| *** jamesmcarthur has joined #openstack-infra | 00:20 | |
| *** eernst has quit IRC | 00:20 | |
| *** jamesmcarthur has quit IRC | 00:24 | |
| *** gyee has quit IRC | 00:25 | |
| *** jamesmcarthur has joined #openstack-infra | 00:32 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [wip] validate-host: retry network tests and include unbound logs https://review.openstack.org/615698 | 00:35 |
|---|---|---|
| *** jamesmcarthur has quit IRC | 00:36 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [wip] validate-host: retry network tests and include unbound logs https://review.openstack.org/615698 | 00:39 |
| fungi | ianw: it seems gary_perkins is either up very late or very early, but either way we should be able to retry the armci cloud config without insecure: true | 00:53 |
| *** Swami has quit IRC | 00:56 | |
| ianw | fungi: and done! LGTM :) | 00:56 |
| ianw | fungi: i think it's ready to go? if you want to merge it, i can look at bringing up a mirror when it's all settled down with the cloud-launcher | 00:57 |
| *** felipemonteiro has joined #openstack-infra | 00:59 | |
| fungi | agreed, i just spent a moment to give it one more time over first | 00:59 |
| fungi | and approved | 00:59 |
| clarkb | follow up on the zuul usage stuff from before. tripleo is down to ~47.7% usage (which includes the same time period as before and the time since then) so I think the changes they are making are working | 01:01 |
| openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: [dnm] testing new network validation https://review.openstack.org/615703 | 01:01 |
| clarkb | its still the biggest consumer by far, but is now letting others use the resources which is nice | 01:01 |
| *** longkb has joined #openstack-infra | 01:03 | |
| clarkb | also yay more cloud regions | 01:04 |
| *** jamesmcarthur has joined #openstack-infra | 01:08 | |
| *** rlandy has quit IRC | 01:09 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [wip] validate-host: retry network tests and include unbound logs https://review.openstack.org/615698 | 01:13 |
| *** felipemonteiro has quit IRC | 01:15 | |
| openstackgerrit | Merged openstack-infra/system-config master: Add Arm64 CI cloud https://review.openstack.org/602436 | 01:21 |
| fungi | ianw: ^ it's in | 01:22 |
| *** rkukura has quit IRC | 01:36 | |
| *** rh-jelabarre has joined #openstack-infra | 01:37 | |
| *** xinliang has joined #openstack-infra | 01:39 | |
| *** xinliang has quit IRC | 01:42 | |
| ianw | \o/ | 01:43 |
| *** jamesmcarthur has quit IRC | 01:44 | |
| *** hongbin has joined #openstack-infra | 01:56 | |
| *** annp has quit IRC | 01:58 | |
| *** longkb has quit IRC | 01:58 | |
| *** longkb has joined #openstack-infra | 01:58 | |
| *** annp has joined #openstack-infra | 01:58 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: [wip] validate-host: retry network tests and include unbound logs https://review.openstack.org/615698 | 02:06 |
| openstackgerrit | 98k proposed openstack-dev/hacking master: Don't quote {posargs} in tox.ini https://review.openstack.org/609192 | 02:09 |
| *** ianychoi has quit IRC | 02:10 | |
| *** ianychoi has joined #openstack-infra | 02:10 | |
| *** hongbin has quit IRC | 02:11 | |
| openstackgerrit | 98k proposed openstack-infra/python-storyboardclient master: Don't quote {posargs} in tox.ini https://review.openstack.org/609176 | 02:14 |
| *** nicolasbock has quit IRC | 02:17 | |
| *** masayukig[m] has quit IRC | 02:17 | |
| *** toabctl has quit IRC | 02:19 | |
| *** toabctl has joined #openstack-infra | 02:23 | |
| *** emerson has quit IRC | 02:23 | |
| *** hongbin has joined #openstack-infra | 02:29 | |
| dmsimard | ianw: can you explain with context ? not picking it up from the backscroll | 02:30 |
| *** diablo_rojo has quit IRC | 02:33 | |
| *** mrsoul has joined #openstack-infra | 02:46 | |
| ianw | dmsimard: oh, just wondering about the best way to get long-form things into the ansible results, particularly for display in ara | 02:54 |
| dmsimard | what kind of long-form things ? | 02:55 |
| ianw | i guess ara_record ... a log file | 02:55 |
| ianw | anyway, i'm just dumping it in the json with with open(f) as f: f.read() and it's ok | 02:55 |
| ianw | but it could probably be better :) | 02:55 |
| dmsimard | ianw: https://review.openstack.org/#/c/580256/ | 02:55 |
| ianw | ahh | 02:56 |
| dmsimard | clarkb: I might help the tripleo guys a little bit | 02:57 |
| dmsimard | ianw: the thing I like about ara_record is the notion of type | 02:57 |
| dmsimard | if I had the time I would add in things like markdown and rst | 02:58 |
| dmsimard | ianw: but wait | 02:59 |
| dmsimard | ianw: why store the file in ara at all ? | 02:59 |
| dmsimard | could you compute the link at which the file would end up and link to the file instead ? | 02:59 |
| ianw | dmsimard: probably it could become a normal artifact, but we only want to collect it on the odd error path | 03:02 |
| *** agopi_ has joined #openstack-infra | 03:03 | |
| *** agopi has quit IRC | 03:06 | |
| *** eernst has joined #openstack-infra | 03:06 | |
| *** lbragstad has quit IRC | 03:16 | |
| *** eernst has quit IRC | 03:23 | |
| *** yamamoto has joined #openstack-infra | 03:30 | |
| *** bhavikdbavishi has joined #openstack-infra | 03:33 | |
| *** bhavikdbavishi has quit IRC | 03:35 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: validate-host: retry network tests and include unbound logs https://review.openstack.org/615698 | 03:38 |
| ianw | clarkb: ^ i think that's ok | 03:38 |
| *** cburgess has quit IRC | 03:41 | |
| *** cburgess has joined #openstack-infra | 03:41 | |
| *** jamesmcarthur has joined #openstack-infra | 03:45 | |
| *** jamesmcarthur has quit IRC | 03:49 | |
| *** ykarel has joined #openstack-infra | 04:20 | |
| *** janki has joined #openstack-infra | 04:28 | |
| *** yamamoto has quit IRC | 04:34 | |
| *** yamamoto has joined #openstack-infra | 04:50 | |
| *** armax has quit IRC | 04:54 | |
| *** udesale has joined #openstack-infra | 04:55 | |
| *** armax has joined #openstack-infra | 04:55 | |
| *** udesale has quit IRC | 04:58 | |
| *** udesale has joined #openstack-infra | 04:58 | |
| *** udesale has quit IRC | 04:58 | |
| ianw | ok, i wonder what's up with all-clouds.yaml, i'm not seeing the new armci in there and it's been awhile | 05:02 |
| *** udesale has joined #openstack-infra | 05:05 | |
| ianw | ahh ... AnsibleUndefinedVariable: 'dict object' has no attribute openstackci_arm64ci_project_name | 05:06 |
| *** hongbin has quit IRC | 05:09 | |
| ianw | ok ... typos in hiera file. should be ok next run | 05:11 |
| *** jtomasek has joined #openstack-infra | 05:14 | |
| *** agopi_ has quit IRC | 05:22 | |
| *** felipemonteiro has joined #openstack-infra | 05:28 | |
| *** ramishra has joined #openstack-infra | 05:29 | |
| *** rascasoft has quit IRC | 05:32 | |
| *** rh-jelabarre has quit IRC | 05:32 | |
| *** rascasoft has joined #openstack-infra | 05:35 | |
| *** udesale has quit IRC | 05:40 | |
| *** carl_cai has joined #openstack-infra | 05:52 | |
| ianw | yay, that got it, watching cloud launcher now to get the control plane image in | 05:53 |
| *** udesale has joined #openstack-infra | 05:53 | |
| openstackgerrit | Merged openstack-infra/zuul master: Merger: automatically add new hosts to the known_hosts file https://review.openstack.org/608453 | 06:03 |
| *** kjackal has joined #openstack-infra | 06:10 | |
| *** jamesmcarthur has joined #openstack-infra | 06:16 | |
| *** jamesmcarthur has quit IRC | 06:20 | |
| *** e0ne has joined #openstack-infra | 06:24 | |
| *** jtomasek has quit IRC | 06:39 | |
| *** felipemonteiro has quit IRC | 06:43 | |
| *** e0ne has quit IRC | 06:44 | |
| *** e0ne has joined #openstack-infra | 06:48 | |
| *** quiquell|off is now known as quiquell | 06:49 | |
| *** longkb has quit IRC | 06:52 | |
| *** longkb has joined #openstack-infra | 06:53 | |
| *** udesale has quit IRC | 06:54 | |
| *** dayou has quit IRC | 07:07 | |
| *** e0ne has quit IRC | 07:07 | |
| *** dayou has joined #openstack-infra | 07:07 | |
| *** maciejjozefczyk has joined #openstack-infra | 07:13 | |
| *** AJaeger_ has quit IRC | 07:15 | |
| *** dpawlik has joined #openstack-infra | 07:17 | |
| *** AJaeger has joined #openstack-infra | 07:26 | |
| *** udesale has joined #openstack-infra | 07:35 | |
| *** pcaruana has joined #openstack-infra | 07:36 | |
| *** adriancz has joined #openstack-infra | 07:42 | |
| *** aojea has joined #openstack-infra | 07:45 | |
| *** quiquell is now known as quiquell|brb | 07:50 | |
| *** florianf has joined #openstack-infra | 08:00 | |
| *** ccamacho has joined #openstack-infra | 08:03 | |
| *** dayou_ has joined #openstack-infra | 08:03 | |
| *** dayou has quit IRC | 08:04 | |
| *** kopecmartin|off is now known as kopecmartin | 08:06 | |
| *** tdasilva has quit IRC | 08:07 | |
| *** ralonsoh has joined #openstack-infra | 08:07 | |
| *** ginopc has quit IRC | 08:11 | |
| *** quiquell|brb is now known as quiquell | 08:14 | |
| *** ginopc has joined #openstack-infra | 08:14 | |
| *** ramishra has quit IRC | 08:15 | |
| *** ykarel_ has joined #openstack-infra | 08:19 | |
| *** ykarel has quit IRC | 08:22 | |
| *** ramishra has joined #openstack-infra | 08:24 | |
| *** dpawlik has quit IRC | 08:28 | |
| *** dpawlik has joined #openstack-infra | 08:29 | |
| *** rtjure has joined #openstack-infra | 08:33 | |
| *** dpawlik has quit IRC | 08:33 | |
| *** dpawlik has joined #openstack-infra | 08:35 | |
| *** priteau has joined #openstack-infra | 08:39 | |
| *** jtomasek has joined #openstack-infra | 08:41 | |
| *** dpawlik has quit IRC | 08:42 | |
| *** dpawlik has joined #openstack-infra | 08:43 | |
| *** jpena|off is now known as jpena | 08:47 | |
| *** jpich has joined #openstack-infra | 08:57 | |
| *** betherly has joined #openstack-infra | 09:01 | |
| *** markmcd has quit IRC | 09:05 | |
| ianw | mordred: cloud-launcher is broken :/ can you take a look @ https://storyboard.openstack.org/#!/story/2004267 ? the first problem, removing the snat router, seems to be fixed by devel version of ansible. now it's having issues with Lon1 in citycloud for openstackzuul | 09:11 |
| *** ykarel__ has joined #openstack-infra | 09:11 | |
| ianw | mordred: i'm out of time, but if we want to pin different versions of ansible on bridge I'd suggest we take my change to do that with variables in https://review.openstack.org/614894 | 09:11 |
| *** ykarel_ has quit IRC | 09:12 | |
| ianw | fungi / gary_perkins : ^ this is why armci cloud does not have a mirror :) | 09:12 |
| *** ykarel__ is now known as ykarel | 09:19 | |
| *** priteau has quit IRC | 09:20 | |
| *** gfidente has joined #openstack-infra | 09:20 | |
| *** Emine has joined #openstack-infra | 09:20 | |
| *** markmcd has joined #openstack-infra | 09:21 | |
| *** maciejjozefczyk has quit IRC | 09:22 | |
| *** rossella_s has quit IRC | 09:23 | |
| *** rossella_s has joined #openstack-infra | 09:23 | |
| *** priteau has joined #openstack-infra | 09:23 | |
| *** annp has quit IRC | 09:24 | |
| *** derekh has joined #openstack-infra | 09:39 | |
| *** sshnaidm|afk is now known as sshnaidm|rover | 09:45 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Improve windows logging https://review.openstack.org/615804 | 09:53 |
| *** lbragstad has joined #openstack-infra | 10:00 | |
| openstackgerrit | Merged openstack-infra/irc-meetings master: use python 3 to build the site https://review.openstack.org/615269 | 10:02 |
| *** arxcruz has quit IRC | 10:03 | |
| *** electrofelix has joined #openstack-infra | 10:10 | |
| *** ykarel is now known as ykarel|lunch | 10:12 | |
| *** rcarrillocruz has joined #openstack-infra | 10:14 | |
| *** jamesmcarthur has joined #openstack-infra | 10:16 | |
| *** jamesmcarthur has quit IRC | 10:21 | |
| *** gfidente has quit IRC | 10:21 | |
| *** yamamoto has quit IRC | 10:24 | |
| openstackgerrit | Ilya Etingof proposed openstack-infra/git-review master: Improve exit code implementation https://review.openstack.org/480267 | 10:25 |
| *** ykarel|lunch is now known as ykarel | 10:33 | |
| *** arxcruz has joined #openstack-infra | 10:36 | |
| *** bhavikdbavishi has joined #openstack-infra | 10:37 | |
| openstackgerrit | Merged openstack-infra/git-review master: Don't set topic when submitting no-topic patches https://review.openstack.org/610988 | 10:40 |
| *** dayou_ has quit IRC | 10:43 | |
| *** yamamoto has joined #openstack-infra | 10:53 | |
| *** lbragstad has quit IRC | 11:02 | |
| *** lbragstad has joined #openstack-infra | 11:04 | |
| *** bhavikdbavishi has quit IRC | 11:05 | |
| *** florianf is now known as florianf|lunch | 11:06 | |
| *** yamamoto has quit IRC | 11:06 | |
| openstackgerrit | Sorin Sbarnea proposed openstack/gertty master: Fix all pyflakes errors https://review.openstack.org/581871 | 11:08 |
| *** shrasool has joined #openstack-infra | 11:10 | |
| *** e0ne has joined #openstack-infra | 11:11 | |
| openstackgerrit | Sorin Sbarnea proposed openstack/gertty master: Adds linting zuul job https://review.openstack.org/581871 | 11:18 |
| *** gfidente has joined #openstack-infra | 11:18 | |
| openstackgerrit | Sorin Sbarnea proposed openstack/gertty master: Adds linting zuul job https://review.openstack.org/581871 | 11:21 |
| openstackgerrit | Merged openstack-infra/zuul master: Fix issue in Github connection with large diffs https://review.openstack.org/612989 | 11:30 |
| *** ramishra has quit IRC | 11:30 | |
| *** longkb has quit IRC | 11:30 | |
| *** carl_cai has quit IRC | 11:31 | |
| *** dtantsur|afk is now known as dtantsur | 11:33 | |
| *** yamamoto has joined #openstack-infra | 11:36 | |
| *** kaiokmo has joined #openstack-infra | 11:39 | |
| *** shrasool has quit IRC | 11:47 | |
| *** janki has quit IRC | 12:00 | |
| *** florianf|lunch is now known as florianf | 12:08 | |
| *** shrasool has joined #openstack-infra | 12:10 | |
| *** shrasool has quit IRC | 12:10 | |
| *** pbourke has quit IRC | 12:11 | |
| *** pbourke has joined #openstack-infra | 12:11 | |
| *** dpawlik has quit IRC | 12:12 | |
| *** carl_cai has joined #openstack-infra | 12:12 | |
| *** dpawlik has joined #openstack-infra | 12:12 | |
| *** dpawlik has quit IRC | 12:16 | |
| *** yamamoto has quit IRC | 12:24 | |
| *** dayou has joined #openstack-infra | 12:25 | |
| openstackgerrit | Sorin Sbarnea proposed openstack-infra/git-review master: Improve exit code implementation https://review.openstack.org/480267 | 12:26 |
| *** gfidente has quit IRC | 12:26 | |
| openstackgerrit | Sorin Sbarnea proposed openstack-infra/git-review master: Fix unused warn function argument. https://review.openstack.org/456929 | 12:27 |
| *** markvoelker has joined #openstack-infra | 12:27 | |
| *** ginopc has quit IRC | 12:27 | |
| *** ginopc has joined #openstack-infra | 12:28 | |
| *** e0ne has quit IRC | 12:29 | |
| *** dave-mccowan has joined #openstack-infra | 12:29 | |
| *** yamamoto has joined #openstack-infra | 12:29 | |
| *** shrasool has joined #openstack-infra | 12:30 | |
| *** jamesmcarthur has joined #openstack-infra | 12:30 | |
| *** yamamoto has quit IRC | 12:34 | |
| *** yamamoto has joined #openstack-infra | 12:34 | |
| *** yamamoto has quit IRC | 12:35 | |
| *** jamesmcarthur has quit IRC | 12:35 | |
| *** yamamoto has joined #openstack-infra | 12:35 | |
| *** jpena is now known as jpena|lunch | 12:38 | |
| *** ansmith has quit IRC | 12:39 | |
| openstackgerrit | Frode Nordahl proposed openstack-infra/project-config master: Add new charm repositories https://review.openstack.org/615855 | 12:43 |
| *** e0ne has joined #openstack-infra | 12:43 | |
| *** gfidente has joined #openstack-infra | 12:45 | |
| *** rtjure has quit IRC | 12:50 | |
| openstackgerrit | Frode Nordahl proposed openstack-infra/project-config master: Add new charm repositories https://review.openstack.org/615855 | 12:55 |
| *** Emine has quit IRC | 12:59 | |
| *** yamamoto has quit IRC | 13:01 | |
| *** rlandy has joined #openstack-infra | 13:02 | |
| *** yamamoto has joined #openstack-infra | 13:02 | |
| *** yamamoto has quit IRC | 13:07 | |
| openstackgerrit | Frode Nordahl proposed openstack-infra/project-config master: Add new charm repositories https://review.openstack.org/615855 | 13:07 |
| *** dpawlik has joined #openstack-infra | 13:07 | |
| *** dpawlik has quit IRC | 13:07 | |
| *** dpawlik has joined #openstack-infra | 13:08 | |
| odyssey4me | infra-root yesterday clarkb held a node running the openstack-ansible-deploy-aio_metal-centos-7 test for https://review.openstack.org/#/c/615258/ for me, I'd like to get the IP and have my ssh key added please to take a look | 13:10 |
| odyssey4me | my pub ssh key is https://github.com/odyssey4me.keys | 13:10 |
| *** ykarel_ has joined #openstack-infra | 13:11 | |
| *** ykarel has quit IRC | 13:14 | |
| fungi | odyssey4me: no prob, gimme a moment | 13:15 |
| *** quiquell is now known as quiquell|lunch | 13:15 | |
| *** Emine has joined #openstack-infra | 13:15 | |
| *** quiquell|lunch is now known as quiquell | 13:17 | |
| fungi | odyssey4me: ssh root@158.69.64.229 | 13:17 |
| *** emine__ has joined #openstack-infra | 13:19 | |
| *** Emine has quit IRC | 13:20 | |
| openstackgerrit | Andrey Kurilin proposed openstack-infra/project-config master: [rally] Remove readthedocs jobs https://review.openstack.org/615868 | 13:21 |
| *** markvoelker has quit IRC | 13:21 | |
| *** ykarel_ is now known as ykarel | 13:21 | |
| *** shrasool has quit IRC | 13:23 | |
| *** kjackal has quit IRC | 13:23 | |
| *** jcoufal has joined #openstack-infra | 13:23 | |
| *** kjackal has joined #openstack-infra | 13:23 | |
| *** quiquell is now known as quiquell|lunch | 13:26 | |
| *** jpena|lunch is now known as jpena | 13:28 | |
| *** shrasool has joined #openstack-infra | 13:30 | |
| *** boden has joined #openstack-infra | 13:31 | |
| *** kgiusti has joined #openstack-infra | 13:37 | |
| *** quiquell|lunch is now known as quiquell | 13:37 | |
| *** tdasilva has joined #openstack-infra | 13:40 | |
| *** ykarel has quit IRC | 13:42 | |
| *** eharney has joined #openstack-infra | 13:48 | |
| fungi | stephenfin: any chance you've got a spare moment to take a look at https://review.openstack.org/583535 for git-review? | 13:50 |
| stephenfin | fungi: Sure | 13:52 |
| *** sthussey has joined #openstack-infra | 13:52 | |
| mordred | AJaeger: re: https://review.openstack.org/#/c/615543/1/tox.ini@10 - what's the correct way to do things here? what we _want_ to do is have python3 be the python for all envs except for py27 | 13:58 |
| Shrews | mordred: drop support for py27???? ;) | 14:00 |
| fungi | mordred: the fix for https://github.com/tox-dev/tox/issues/477 was introduction of an ignore_basepython_conflict option | 14:02 |
| fungi | which you'll need anyway if you want to test py35 and py36 on the same system, since otherwise both will use whatever your default python3 is | 14:03 |
| *** felipemonteiro has joined #openstack-infra | 14:03 | |
| fungi | https://tox.readthedocs.io/en/latest/config.html#conf-ignore_basepython_conflict | 14:03 |
| *** ykarel has joined #openstack-infra | 14:03 | |
| *** kaisers has quit IRC | 14:04 | |
| mordred | fungi: so - like this: https://review.openstack.org/615543 ? | 14:07 |
| openstackgerrit | Merged openstack-infra/git-review master: Improve exit code implementation https://review.openstack.org/480267 | 14:07 |
| *** aluria has joined #openstack-infra | 14:07 | |
| *** ansmith has joined #openstack-infra | 14:07 | |
| fungi | mordred: yes | 14:07 |
| *** agopi_ has joined #openstack-infra | 14:09 | |
| fungi | mordred: the pre-tox-3.1 workaround was to separately define testenv:py27, testenv:py35, testenv:py36 and so on each with its own distinct basepython override | 14:09 |
| fungi | so you'll likely see that in lots of places too | 14:09 |
| fungi | but it's no longer necessary with ignore_basepython_conflict=True | 14:10 |
| mordred | awesome | 14:10 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Add ignore_basepython_conflicts option to tox.ini https://review.openstack.org/615887 | 14:10 |
| mordred | fungi: https://review.openstack.org/615887 Add ignore_basepython_conflicts option to tox.ini | 14:10 |
| *** kaisers has joined #openstack-infra | 14:10 | |
| mordred | fungi: added one to zuul to handle 3.5/3.6 | 14:10 |
| *** mriedem has joined #openstack-infra | 14:11 | |
| fungi | almost, see typo marked inline | 14:11 |
| *** ramishra has joined #openstack-infra | 14:13 | |
| *** emine__ has quit IRC | 14:15 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Add ignore_basepython_conflict option to tox.ini https://review.openstack.org/615887 | 14:17 |
| mordred | fungi: thanks! | 14:18 |
| *** shrasool has quit IRC | 14:18 | |
| *** bobh has joined #openstack-infra | 14:19 | |
| *** carl_cai has quit IRC | 14:21 | |
| *** niska has quit IRC | 14:25 | |
| *** roman_g has joined #openstack-infra | 14:29 | |
| *** ginopc has quit IRC | 14:34 | |
| *** ginopc has joined #openstack-infra | 14:34 | |
| *** niska has joined #openstack-infra | 14:35 | |
| odyssey4me | infra-root the nodepool node 158.69.64.229 can now be released from the hold - thanks for that, we found the bug we needed :) | 14:36 |
| mordred | \o/ | 14:36 |
| Shrews | odyssey4me: scheduled for delete now. thx | 14:42 |
| odyssey4me | tyvm! | 14:42 |
| *** felipemonteiro has quit IRC | 14:43 | |
| openstackgerrit | Merged openstack-infra/zuul master: Don't do live streaming in loops https://review.openstack.org/615657 | 14:47 |
| *** agopi_ is now known as agopi | 14:47 | |
| openstackgerrit | Merged openstack-infra/zuul master: Improve windows logging https://review.openstack.org/615804 | 14:49 |
| openstackgerrit | Sorin Sbarnea proposed openstack/gertty master: Adds linting zuul job https://review.openstack.org/581871 | 14:55 |
| *** munimeha1 has joined #openstack-infra | 14:58 | |
| *** rh-jelabarre has joined #openstack-infra | 14:59 | |
| openstackgerrit | Sorin Sbarnea proposed openstack/gertty master: Adds linting zuul job https://review.openstack.org/581871 | 15:01 |
| openstackgerrit | Frode Nordahl proposed openstack-infra/project-config master: Add new charm repositories https://review.openstack.org/615855 | 15:01 |
| *** armax has joined #openstack-infra | 15:03 | |
| *** e0ne has quit IRC | 15:06 | |
| *** ykarel is now known as ykarel|away | 15:22 | |
| *** ginux has joined #openstack-infra | 15:25 | |
| *** ginux is now known as Guest4815 | 15:25 | |
| AJaeger | diablo_rojo_phon: could you put https://review.openstack.org/#/c/615636/ on your review queue, please? An infra change for elections repo. | 15:27 |
| *** Guest4815 has quit IRC | 15:28 | |
| *** ginopc has quit IRC | 15:28 | |
| *** dpawlik has quit IRC | 15:33 | |
| *** felipemonteiro has joined #openstack-infra | 15:34 | |
| *** ramishra has quit IRC | 15:40 | |
| *** ykarel|away has quit IRC | 15:43 | |
| *** yamamoto has joined #openstack-infra | 15:44 | |
| *** kopecmartin is now known as kopecmartin|off | 15:46 | |
| *** felipemonteiro has quit IRC | 15:46 | |
| *** yamamoto has quit IRC | 15:49 | |
| *** priteau has quit IRC | 15:49 | |
| openstackgerrit | Nguyen Hai Truong proposed openstack-dev/hacking master: Add python 3.6 unit test job https://review.openstack.org/615915 | 15:51 |
| clarkb | mordred have you had a chance to look at ianw's cloud launcher thing? I wonder if it is related to the weird citycloud keystone setup | 15:52 |
| mordred | clarkb: it might be - I just got off a phone call, looking now | 15:53 |
| mordred | clarkb: oh - no - easy fix for the Lon1 issue | 15:56 |
| fungi | still seeing some odd tcp open rate patterns on the cacti graph for etherpad, and the background system cpu activity seems to have returned early yesterday: | 15:56 |
| fungi | http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=441&rra_id=all | 15:56 |
| fungi | http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=440&rra_id=all | 15:56 |
| clarkb | fungi: those links show graphite.o.o for me | 15:57 |
| fungi | oh, funny | 15:57 |
| fungi | i guess some of it is in post variables? | 15:57 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add Lon1 and Sto2 to openstackzuul cloud entry https://review.openstack.org/615918 | 15:57 |
| mordred | clarkb: ^^ | 15:57 |
| fungi | or! i was looking at the wrong graph, yes ;) | 15:58 |
| fungi | yep, totally looking at the wrong graphs | 15:58 |
| clarkb | mordred: you need to add the auth_url fix too | 15:58 |
| mordred | clarkb: ++ one sec | 15:58 |
| mordred | clarkb: oh - wait - actually - we released 0.19.0 of openstacksdk | 15:59 |
| clarkb | mordred: does that have the auth_url template built in? if so then likely just need to check that 0.19.0 is installed on bridge.o.o | 16:00 |
| mordred | clarkb: fix is in that - so we can actually remove the workaround | 16:00 |
| clarkb | and now I need to join a call | 16:00 |
| mordred | clarkb: http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/config/vendors/citycloud.json?h=0.19.0 | 16:00 |
| clarkb | also while I'm noticing it, tripleo seems to have grown a few non voting gate jobs? | 16:01 |
| *** emerson has joined #openstack-infra | 16:01 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remove citycloud auth_url workaround https://review.openstack.org/615919 | 16:01 |
| clarkb | EmilienM: mwhahaha ^ fyi paunch, tripleo heat templates, and tripleo upgrade all haev at least one non voting gate job | 16:01 |
| mwhahaha | i just cleaned them up :/ | 16:02 |
| EmilienM | is it the scenarios? | 16:02 |
| * mwhahaha looks | 16:02 | |
| stephenfin | fungi: Reviewed https://review.openstack.org/#/c/583535/8 One comment left that maybe you could weigh in on | 16:02 |
| clarkb | mordred: openstacksdk==0.17.1 # git sha f28a1b0 is what is installed on bridge.o.o | 16:02 |
| fungi | thanks stephenfin | 16:02 |
| fungi | ! | 16:02 |
| *** cloudnull has quit IRC | 16:02 | |
| *** d34dh0r53 has quit IRC | 16:02 | |
| *** eglute has quit IRC | 16:02 | |
| mwhahaha | EmilienM: it's containered upgrades (for tripleo-ugrades) | 16:03 |
| mordred | clarkb: ok. well, we should certainly upgrade that :) | 16:03 |
| mwhahaha | it's also the stable branches | 16:03 |
| EmilienM | mwhahaha: let me fix that | 16:03 |
| mwhahaha | i need to cleanup the scenarios on the stable branches | 16:03 |
| mwhahaha | EmilienM: also containerized upgrades for paunch | 16:04 |
| EmilienM | k | 16:05 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: Revert "Revert "Cleanup down ports"" https://review.openstack.org/614370 | 16:06 |
| *** panda is now known as panda|ko | 16:06 | |
| *** jamesmcarthur has joined #openstack-infra | 16:07 | |
| *** quiquell is now known as quiquell|off | 16:08 | |
| *** eglute has joined #openstack-infra | 16:08 | |
| *** d34dh0r53 has joined #openstack-infra | 16:08 | |
| *** kjackal has quit IRC | 16:16 | |
| *** imacdonn has quit IRC | 16:17 | |
| *** imacdonn has joined #openstack-infra | 16:17 | |
| *** hamzy_ has quit IRC | 16:19 | |
| *** gyee has joined #openstack-infra | 16:21 | |
| dmsimard | Is ask.openstack.org supposed to be down ? Noticed it when following a google link | 16:21 |
| fungi | not as far as i know. taking a look now | 16:23 |
| dmsimard | Looked quickly at the server and it doesn't seem unhealthy, I don't know how the app works though | 16:23 |
| fungi | [Tue Nov 06 06:53:42.278436 2018] [mpm_event:notice] [pid 24561:tid 140624258938752] AH00493: SIGUSR1 received. Doing graceful restart | 16:24 |
| fungi | [Tue Nov 06 06:53:42.469710 2018] [core:notice] [pid 24561] AH00060: seg fault or similar nasty error detected in the parent process | 16:24 |
| dmsimard | apache segfault ? | 16:24 |
| fungi | during log rotation, looks like | 16:24 |
| fungi | i've started apache again | 16:24 |
| dmsimard | looks back online, yay | 16:24 |
| Tim_ok | Any pointers to a doc/github where the staring point to the zuul triggered third party CI builds setup is explained w/an example? | 16:26 |
| *** ccamacho has quit IRC | 16:27 | |
| fungi | Tim_ok: help on https://specs.openstack.org/openstack-infra/infra-specs/specs/zuulv3-3rd-party-ci.html would be appreciated | 16:28 |
| openstackgerrit | Merged openstack-infra/zuul master: Resume paused job with skipped children https://review.openstack.org/615493 | 16:30 |
| *** ginopc has joined #openstack-infra | 16:31 | |
| *** udesale has quit IRC | 16:32 | |
| *** panda|ko is now known as panda|off | 16:34 | |
| *** andymccr has quit IRC | 16:36 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/openstackid master: Migration to PHP 7.x https://review.openstack.org/611936 | 16:37 |
| *** quiquell|off is now known as quiquell | 16:38 | |
| *** quiquell is now known as quiquell|off | 16:38 | |
| *** auristor has quit IRC | 16:42 | |
| *** hamzy_ has joined #openstack-infra | 16:44 | |
| fungi | corvus: how have we disabled abandoned mailing lists in the past? just sent a final message to the list noting that it's being shut down, and then run rmlist on the server? | 16:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Only set basepython once https://review.openstack.org/615942 | 16:49 |
| fungi | saved a copy of the config_list output? | 16:49 |
| *** auristor has joined #openstack-infra | 16:50 | |
| *** chandankumar is now known as chkumar|pto | 16:59 | |
| clarkb | mordred: I think modern pip doesn't update all your deps when you pip install -U, so our ansible (and whatever else installs) don't pull in latest openstacksdk | 16:59 |
| clarkb | mordred: do we want to encode pulling in latest sdk or just update the citycloud clouds.yaml to work with older sdk too (I kinda like working with older sdk too) | 17:00 |
| mordred | clarkb: yah. I think we have to set state: latest on our pip install of openstacksdk | 17:00 |
| fungi | right, the default upgrade strategy changed from always to as-needed | 17:00 |
| mordred | clarkb: I don't see any benefit to having old sdk installed - but also, it doesn't come in as a depend of ansible- we install it directly | 17:00 |
| clarkb | ah | 17:00 |
| fungi | so if your dependency set claims you need a newer version then pip will upgrade those dependencies, but only those dependencies | 17:01 |
| clarkb | as for supporting older sdk a lot of people use our clouds .yaml and modify as necessary for various clouds | 17:01 |
| clarkb | and it might be more friendly to them if our config didn't expect latest sdk | 17:01 |
| mordred | that doesn't make any sense to me - why would they copy our list of cloud accounts? | 17:01 |
| clarkb | mordred: not the entire list, but if someone wants to use rax they see how we do it | 17:02 |
| clarkb | and so on | 17:02 |
| clarkb | because it turns out that things are still just enough inside baseball to confuse people (largely around keystone | 17:03 |
| mordred | right. but that's literally the entire point of the vendor config files in sdk - if we maintain our clouds.yaml as a thing for people to cargo cult, we're maintaining a second library - and if there are things that are general we should just update the vendor config in sdk, no? | 17:03 |
| clarkb | well what we need to do is make keystone api versioning stop being impossible to figure out | 17:04 |
| clarkb | (our config for packethost is wrong btw we get a bunch of warnings from openstackclient) | 17:04 |
| clarkb | in theory once v2 is properly gone then this is a none issue because you use v3 everywhere | 17:04 |
| mordred | right. that's because openstackclient re-implements half of keystoneauth and we haven't fixed it yet | 17:04 |
| clarkb | 'Ignoring domain related config project_domain_name for openstackci-packethostbecause identity API version is 2.0' etc | 17:04 |
| mordred | yeah | 17:05 |
| mordred | osc and sdk both work around keystone issues in an opposite way | 17:05 |
| mordred | I've got an idea of how to improve part of that in sdk I'm going to try out tomorrow | 17:05 |
| mordred | but from what I can see, there isn't any information related to public clouds in our clouds.yaml file that is salient | 17:05 |
| clarkb | well the cityclodu thing is if using older openstacksdk | 17:06 |
| mordred | (basically trying to figure out what info there would be valuable and how we can not make our specific clouds.yaml file valuable to other people) | 17:06 |
| mordred | sure. but that info is also in newer openstacksdk vendor files ... Im just saying that our specific config shouldn't be an alternate source of information from the sdk repo such that we avoid upgrading things or updating our files because we're maintaining an alternate view of the universe | 17:07 |
| *** ginopc has quit IRC | 17:07 | |
| mordred | our config should be our config and should serve us - and if there are ways in which things need to be clearer to people we should fix the actual docs or the actual source of truth | 17:08 |
| clarkb | yup I agree, but at the same time no one with an older sdk install is going to undersatnd how to use citycloud | 17:08 |
| mordred | right. they should upgrade openstacksdk | 17:08 |
| clarkb | I think sdk/oscc/keystone should (can) do more to address that | 17:08 |
| mordred | as it is designed right now | 17:08 |
| mordred | running with old openstacksdk == being broken | 17:08 |
| clarkb | the proposal to put the clouds.yaml stuff at a well know location is a good step in that direction | 17:08 |
| mordred | that can and should be made better | 17:08 |
| mordred | yeah | 17:08 |
| clarkb | then you don't depend on sdk version (once support is added) and just talk to the cloud for that info | 17:09 |
| mordred | I'm super happy mnaser stuck a vendor file up there -I'm gonna try to get that iplemented RSN | 17:09 |
| mordred | *exactly* | 17:09 |
| mordred | but the thing to put into clouds.yaml is here: http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/config/vendors/citycloud.json - and will totally work for people on older sdk ... so the question is - how can we get people to cargo cult from there instead of from our clouds.yaml file? | 17:10 |
| *** rkukura has joined #openstack-infra | 17:11 | |
| clarkb | mordred: in this case its too late for openstacksdk to say "hey this thing is weird check foo" but maybe citycloud could do that if talkign to the old keystone url via some error/exception | 17:11 |
| clarkb | mordred: that may be more trouble than its worth but basically have the old config error in such a way that it is obvious to the user that they need to go do something to fix it and not wait for the cloud to fix itself | 17:12 |
| clarkb | current error is "that cloud region doesn't exist anymore" | 17:12 |
| clarkb | whcih isn't true. Should be "please go update your config to this in order to see these cloud regions" | 17:13 |
| mordred | yeah. that would actually be much easier if we split the local vendor files into their own repo/library like with os-service-types | 17:13 |
| mordred | cause then we could do a freshness check on them | 17:13 |
| mordred | but I think getting vendors to publish the files themselves is likely more better if we can do it | 17:14 |
| clarkb | yes I agree | 17:14 |
| mnaser | mordred: you're welcome :D | 17:19 |
| *** felipemonteiro has joined #openstack-infra | 17:19 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Add Lon1 and Sto2 to openstackzuul cloud entry https://review.openstack.org/615918 | 17:20 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Remove citycloud auth_url workaround https://review.openstack.org/615919 | 17:20 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Install latest openstacksdk on bridge https://review.openstack.org/615951 | 17:20 |
| mordred | clarkb: there's a set of thigns that should all work - and allow us to decide on removing the auth_url template separately :) | 17:20 |
| coreycb | AJaeger: hi, would you be able to comment on the last question posed here? https://review.openstack.org/#/c/610708/5/goals/stein/python3-first.rst@45 | 17:22 |
| *** ccamacho has joined #openstack-infra | 17:22 | |
| *** e0ne has joined #openstack-infra | 17:32 | |
| *** admcleod has quit IRC | 17:32 | |
| *** ccamacho has quit IRC | 17:33 | |
| *** jpich has quit IRC | 17:34 | |
| clarkb | mordred: stack lgtm. I left my thoughts on the auth_url stuff in the last change but +2'd it | 17:34 |
| clarkb | infra-root can you review https://review.openstack.org/615918 in particular to fix the cloud launcher | 17:38 |
| *** boden has quit IRC | 17:40 | |
| Shrews | clarkb: openstacksdk 0.19.0 is released, so should that auth_url entry be there? | 17:41 |
| ssbarnea | clarkb mordred : can you please review https://review.openstack.org/#/c/613383/ ? -- i really hate to do horizontal scrolling on zuul. is only a CSS change. | 17:41 |
| clarkb | Shrews: the whole stack has the set of changes that you want. (look at the child changes). Basically we still have 0.17.x installled. The following changes install 0.19.0 and then clean up the configs | 17:41 |
| Shrews | clarkb: oh, oops | 17:41 |
| clarkb | Shrews: we need that first one to fix the problem then followups clean things up | 17:42 |
| clarkb | reminder that the infra meeting is an hour earlier for those of you in the northern hemisphere that observe DST | 17:48 |
| clarkb | just over an hour from now | 17:48 |
| *** armax has quit IRC | 17:51 | |
| fungi | but still at 19:00 for those of you who keep your clocks sanely set to utc! | 17:51 |
| persia | (hint for those who don't have "UTC" as a timezone choice: Iceland sanely keeps clocks set to UTC, so it is safe to set the clock to "Iceland time") | 17:52 |
| clarkb | and I think it is an hour later for those of you in the southern hemisphere that observe DST? but timezone math is weird | 17:52 |
| persia | clarkb: Yes, but the time change isn't particularly synchronised by hemisphere. For example, Australia changed time 7th October this year. | 17:52 |
| persia | Whereas Brazil only did it last weekend. | 17:53 |
| clarkb | persia: ya and europe is earlier than USA too | 17:53 |
| *** felipemonteiro has quit IRC | 17:53 | |
| clarkb | and not all US states observe DST (hawaii and arizona don't and maybe others now too?) | 17:53 |
| clarkb | this is why we use UTC for scheduling around here :) | 17:54 |
| persia | Anyway, for those particularly interested, I understand tzdata is currently up-to-date. | 17:54 |
| fungi | and apparently next year the eu votes on whether to finally ditch dst? | 17:54 |
| *** diablo_rojo has joined #openstack-infra | 17:55 | |
| persia | Russia seems to be doing quite well in the aftermath of timezone rationalisation. | 17:55 |
| openstackgerrit | Merged openstack-infra/os-loganalyze master: Assures that wrapping on PRE occurs on any kind of characters https://review.openstack.org/613383 | 17:55 |
| *** sshnaidm|rover is now known as sshnaidm|afk | 17:56 | |
| *** derekh has quit IRC | 18:01 | |
| *** eandersson has joined #openstack-infra | 18:01 | |
| eandersson | How do you get the nicely formated logs for a service, where you can filter based on log level? | 18:01 |
| *** e0ne has quit IRC | 18:03 | |
| *** apetrich has quit IRC | 18:03 | |
| clarkb | eandersson: we run a wsgi application on the log server that filters the logs and renders them that way on the fly. os-loganalyze is the tool | 18:03 |
| eandersson | Is this something I need to configure in devstack, or maybe a project config? | 18:04 |
| *** jpena is now known as jpena|off | 18:05 | |
| clarkb | eandersson: it is configured on the web server serving your log files | 18:05 |
| *** dtantsur is now known as dtantsur|afk | 18:06 | |
| *** e0ne has joined #openstack-infra | 18:06 | |
| mordred | although - I believe the logs-in-swift code switches it around to run os-loganalyze on the logs as a pre-publication filter, yeah? | 18:12 |
| clarkb | mordred: ya we would publish multiple copies of things | 18:13 |
| *** rkukura has quit IRC | 18:13 | |
| *** jtomasek has quit IRC | 18:18 | |
| *** bhavikdbavishi has joined #openstack-infra | 18:19 | |
| *** armax has joined #openstack-infra | 18:20 | |
| *** dpawlik has joined #openstack-infra | 18:21 | |
| *** gfidente is now known as gfidente|afk | 18:21 | |
| corvus | fungi: i don't think we have done that for mailing lists in the past, but i think that's a fine idea. | 18:22 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for one git worker https://review.openstack.org/615966 | 18:22 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for all git workers https://review.openstack.org/615967 | 18:22 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for git load balancer https://review.openstack.org/615968 | 18:22 |
| AJaeger | coreycb: can do... | 18:23 |
| *** rpioso is now known as rpioso|afk | 18:24 | |
| coreycb | AJaeger: ty | 18:24 |
| *** e0ne has quit IRC | 18:24 | |
| *** rfolco|ruck is now known as rfolco|doctor | 18:30 | |
| *** markvoelker has joined #openstack-infra | 18:31 | |
| fungi | corvus: mainly thinking ahead to december 3 when we shut down a bunch of openstack mailing lists in favor of using openstack-discuss | 18:31 |
| AJaeger | coreycb: not the answer you expected - but I left a lengthy explanation | 18:31 |
| *** gyee has quit IRC | 18:34 | |
| corvus | fungi: oh, sorry, i don't think we should run rmlist; i think we should just disable it. | 18:34 |
| coreycb | AJaeger: that's fine. the main question I wanted to get +/-1 on was enabling of 3 sets of py3 unit tests at the same time | 18:35 |
| corvus | fungi: disable subscription, posting, etc. rmlist might affect accessibility of archives. | 18:35 |
| *** armax has quit IRC | 18:35 | |
| coreycb | AJaeger: which, I don't know if that's what you -1'd in particular. any chance you could clear that bit up? | 18:36 |
| eandersson | clarkb, I was hoping this would enable that https://review.openstack.org/#/c/615688/ | 18:40 |
| fungi | corvus: okay, that's why i'm asking. i couldn't figure out how to "disable" a list without using rmlist (which still leaves the archives behind unless you ask it not to) | 18:41 |
| *** Swami has joined #openstack-infra | 18:42 | |
| *** ociuhandu has joined #openstack-infra | 18:42 | |
| corvus | fungi: i imagine it will be removed from the index of lists. if we're okay with that, maybe rmlist is all right. | 18:43 |
| AJaeger | corvus: see my last sentence | 18:43 |
| corvus | fungi: but otherwise, we can almost certainly make it so no one can subscribe or post to the list. | 18:43 |
| fungi | you can set emergency moderation mode on so even subscribers end up in the moderation queue, you can set it so subscription requests require list admin approval, et cetera | 18:44 |
| *** ralonsoh has quit IRC | 18:44 | |
| corvus | AJaeger: was that intended for coreycb? | 18:44 |
| corvus | fungi: probably worth trying it out on a test instance or something to see what rmlist looks like. | 18:45 |
| fungi | but if you want, say, the mta to reject posts rather than having mailman drop them or start participating in spam backscatter, your options seem to be rmlist or adding overrides in the exim config | 18:45 |
| corvus | fungi: if it's still sufficiently discoverable, rmlist is probably preferable. | 18:45 |
| *** dpawlik has quit IRC | 18:45 | |
| AJaeger | corvus: indeed - sorry | 18:45 |
| *** ociuhandu has quit IRC | 18:45 | |
| AJaeger | coreycb: see my last sentence, I added it again on IRC | 18:46 |
| fungi | i've had a request to retire the openstack-mentors ml (it's had one legitimate post in the past year and is basically just a spam magnet) | 18:46 |
| fungi | though i'm still confirming with the active mentors that's okay | 18:47 |
| coreycb | AJaeger: are you on the infra team? | 18:47 |
| clarkb | eandersson: ah so you want the senlin logs that infra hosts to be rendered that way? | 18:47 |
| eandersson | Yea | 18:47 |
| clarkb | eandersson: I misunderstood and thought you were looking for your own hosting | 18:47 |
| corvus | fungi: could be a good test case | 18:48 |
| *** armax has joined #openstack-infra | 18:48 | |
| clarkb | eandersson: https://git.openstack.org/cgit/openstack-infra/os-loganalyze/tree/os_loganalyze/filter.py#n24 needs to be updated to match your log files if it is in a format that is understood. The systemd log format is understood | 18:48 |
| eandersson | Trying to clean up devstack in a few projects, and I thought it would be a neat addition | 18:48 |
| coreycb | AJaeger: I really just want a hard +/-1 from infra so I can work on this or just move on | 18:48 |
| fungi | coreycb: you need to get that from the tc, not from the infra team | 18:49 |
| fungi | goal texts are a governance matter for the tc | 18:50 |
| coreycb | fungi: well yes but I think infra needs to give input on what can be handled by infra | 18:50 |
| AJaeger | coreycb: but infra is either the PTL - and that's clarkb - or a couple of individuals with strong opininons but not all the same one ;) | 18:51 |
| eandersson | So looking at that clarkb we just need to add something like sl to > r'ceil|key|sah|des|tr)' # openstack logs | 18:51 |
| fungi | coreycb: there are lots of ways openstack could choose to reduce resource consumption while testing against three versions of python interpreters. the implementation isn't something the infra team is supporting one way or the other | 18:51 |
| coreycb | AJaeger: that's the problem, too many chiefs and different answers :) | 18:51 |
| clarkb | eandersson: if that is the string that identifies your log files, yes | 18:51 |
| fungi | coreycb: you're choosing to add chiefs. the change you've proposed is a governance change, and you need to get the tc to decide. they may choose (and often do) to take input from a number of other stakeholders in the process of doing so | 18:52 |
| *** hamzy_ is now known as hamzy | 18:53 | |
| clarkb | as I've told others (including the TC) before I really don't think we should worry so much about what airship or kolla or nova etc are doing with their testing as long as tripleo uses the bulk of the resources. tripleo is improving but you won't drastically incrase resource consumption by adding some jobs to the 3% | 18:53 |
| clarkb | the tail is incredibly long our per project and job resource usage | 18:53 |
| coreycb | fungi: i don't mean to offend with that it's just that there are so many suggestions that go in different directions | 18:53 |
| clarkb | adding an extra python job shouldn't have a major impact | 18:54 |
| AJaeger | my -1 was not for the three jobs - but -1 was for piggypacking it to that proposal. I asked for a separate document... | 18:54 |
| * AJaeger would not block adding this to a few projects.... | 18:54 | |
| eandersson | Thanks clarkb | 18:54 |
| fungi | yes, i haven't weighed in with my tc hat on (yet) but similarly think we shouldn't go "moving goalposts" in goals once they've been approved and announced | 18:55 |
| *** bobh has quit IRC | 18:55 | |
| fungi | a new document (t cycle goal, tc resolution, something) would be preferable | 18:56 |
| coreycb | AJaeger: if you could add that comment to the governance proposal that might be useful. i've had an email thread, and a governance proposal update, and some have told me to go back to the email thread is enough for +1 to move on with this, others have suggested another governance update. so not sure where to go. | 18:56 |
| AJaeger | coreycb: I did. | 18:56 |
| coreycb | AJaeger: i know, but it differes from what other say | 18:57 |
| AJaeger | coreycb: "So, I give a -1: Either do it without this goal - or make it a separate goal but don't piggypack on this one." | 18:57 |
| openstackgerrit | Erik Olof Gunnar Andersson proposed openstack-infra/os-loganalyze master: Add support for Senlin https://review.openstack.org/615977 | 18:58 |
| AJaeger | coreycb: and I'm not in the TC - that's what the TC has to decide on. I trust them to take my input and make the right decision for OpenStack - I have no problems to get overruled | 18:58 |
| clarkb | infra meeting over in #openstack-meeting in jsut a moment | 18:59 |
| coreycb | AJaeger: ok thanks | 18:59 |
| *** shardy has quit IRC | 19:00 | |
| *** rkukura has joined #openstack-infra | 19:03 | |
| coreycb | dhellmann: AJaeger doesn't seem completely opposed to adding 3 x py3 unit tests, at least to a subset of projects. clarkb also doesn't seem to be opposed either. do you think we could approve this on my original mail thread rather than a governance proposal? perhaps I could just clear up my plans and get +1's on the ML? | 19:03 |
| clarkb | fwiw I do still think py35 should be deleted along with xenial jobs on master | 19:04 |
| coreycb | clarkb: stop it :) | 19:04 |
| coreycb | clarkb: kidding of course | 19:05 |
| clarkb | thats part of normal support from openstack | 19:05 |
| clarkb | (independent of the python3 first goal) | 19:05 |
| coreycb | clarkb: there were projects opposing that in particular keystone i believe and they pointed at a statement in the python3-first goal that said the min version of py35 wouldn't be changed in stein. | 19:06 |
| openstackgerrit | Merged openstack-infra/system-config master: Add Lon1 and Sto2 to openstackzuul cloud entry https://review.openstack.org/615918 | 19:06 |
| dhellmann | coreycb : yeah, a ML thread makes sense | 19:07 |
| dhellmann | I'd like for us to just take things 1 step at a time | 19:07 |
| clarkb | coreycb: ya I see that as a bug in the python3 goal that should be fixed :) | 19:07 |
| dhellmann | I don't know yet which version of python 3 RHEL 8 will have. I am willing to gamble it is at least 3.5, but not 3.6 yet | 19:07 |
| clarkb | we've never continued to support old python before whether 2.6 or 3.x | 19:07 |
| coreycb | clarkb: fair enough :) | 19:07 |
| coreycb | dhellmann: ok i'll re-raise the ML thread and seek out +1's, thanks. | 19:08 |
| dhellmann | ++ | 19:08 |
| *** agopi has quit IRC | 19:09 | |
| *** diablo_rojo has quit IRC | 19:12 | |
| fungi | infra-root: heads up we just got a ticket from rackspace saying the host ze09 is on got rebooted | 19:12 |
| fungi | i can't ssh into it at the moment | 19:13 |
| *** diablo_rojo has joined #openstack-infra | 19:13 | |
| *** electrofelix has quit IRC | 19:14 | |
| fungi | oh, actually says it alerted them and they're investigating, but not that they've rebooted it (yet) | 19:14 |
| fungi | so i suppose that's expected | 19:14 |
| *** rpioso|afk is now known as rpioso | 19:15 | |
| *** mriedem is now known as mriedem_vote | 19:18 | |
| AJaeger | diablo_rojo: could you put https://review.openstack.org/#/c/615636/ on your review queue, please? An infra change for elections repo. | 19:19 |
| openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Install latest openstacksdk on bridge https://review.openstack.org/615951 | 19:21 |
| ianw | mordred: ^ that adds a skip for ANSIBLE0010 (packages shouldn't install from latest) which I feel like is inappropriate for our environment | 19:22 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/puppet-pip master: [debug] Fix openstack_pip provider for pip 18 https://review.openstack.org/606021 | 19:25 |
| diablo_rojo | AJaeger, yep! Its in the queue :) | 19:25 |
| AJaeger | thanks, diablo_rojo | 19:25 |
| *** rlandy is now known as rlandy|vote | 19:26 | |
| fungi | ianw: normal solution to ANSIBLE0010 seems to be to use a variable for it, then default it to latest | 19:29 |
| fungi | or at least that's what the openstack-ansible crew have been doing | 19:30 |
| ianw | fungi: is that a solution ... ? | 19:30 |
| fungi | gives us a place to optionally override package versions in a role if we, say, want to reuse the same role on two servers with different version numbers (e.g. a -dev server and production?) | 19:31 |
| fungi | of course ignoring ANSIBLE0010 doesn't prevent us from still doing that where we want to | 19:32 |
| ianw | fungi: ok, well that's essentially what i did (for other reasons) with https://review.openstack.org/#/c/614894 ... want to look at that, and may be we can do similar? | 19:33 |
| ianw | we actually need that, or similar, ATM, because the cloud-launcher currently requires bug fixes in a non-released ansible | 19:34 |
| fungi | i guess that could do latest instead of omit if we wanted? | 19:34 |
| ianw | you mean "instead of omit the rule"? yes, i think we can tackle it all together maybe for consistency | 19:36 |
| openstackgerrit | Merged openstack-infra/zuul master: Parallelize prime_installation_map https://review.openstack.org/607876 | 19:43 |
| openstackgerrit | Merged openstack-infra/zuul master: Add ignore_basepython_conflict option to tox.ini https://review.openstack.org/615887 | 19:43 |
| *** yamamoto has joined #openstack-infra | 19:45 | |
| *** david-lyle has joined #openstack-infra | 19:46 | |
| *** kjackal has joined #openstack-infra | 19:48 | |
| *** dklyle has quit IRC | 19:49 | |
| *** eernst has joined #openstack-infra | 19:49 | |
| cmurphy | ianw: did you have a change to dig into the puppet-pip on centos issue we talked about a while ago? | 19:50 |
| *** yamamoto has quit IRC | 19:51 | |
| ianw | cmurphy: ahhh, yeah, sorry i got distracted | 19:51 |
| cmurphy | this is an example of it happening http://logs.openstack.org/21/606021/14/check/puppet-beaker-rspec-centos-7-infra/37ecc04/job-output.txt.gz#_2018-11-05_20_14_20_972971 it clearly downloads pbr from the rax mirror but then later can't find it and tries to get it from pypi which is intentionally blocked in the test | 19:51 |
| openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: bridge.o.o: Use latest openstacksdk https://review.openstack.org/615982 | 19:51 |
| ianw | fungi: ^ how's that look? | 19:51 |
| cmurphy | so i worry this has a bigger impact than just the puppet module since it could mean pip is always bypassing the mirror on centos | 19:52 |
| ianw | cmurphy: hrrm "Running setup.py (path:/tmp/pip-install-6FwWi8/os-service-types/setup.py)" ... is os-service-types involved here? | 19:53 |
| fungi | cmurphy: i think that's the effect of using an extra index url instead of overriding the index url... could that be the cause | 19:54 |
| fungi | ? | 19:54 |
| cmurphy | ianw: os-service-types is involved because it's installing shade | 19:55 |
| cmurphy | fungi: hmm maybe? | 19:55 |
| fungi | ze09 did eventually reboot, up 26 minutes now | 19:55 |
| cmurphy | fungi: hmm no it is overriding extra-index-url http://logs.openstack.org/21/606021/14/check/puppet-beaker-rspec-centos-7-infra/37ecc04/job-output.txt.gz#_2018-11-05_20_05_34_541733 | 19:55 |
| fungi | yeah, we normally override the primary index-url to our pypi cache, and then set an extra-index-url for our wheelhouse | 19:56 |
| fungi | so that looks right | 19:57 |
| *** bhavikdbavishi has quit IRC | 19:57 | |
| fungi | index-url = http://mirror.iad.rax.openstack.org/pypi/simple | 19:57 |
| fungi | extra-index-url = http://mirror.iad.rax.openstack.org/wheel/centos-7.5.1804-x86_64 | 19:57 |
| fungi | it shouldn't try to hit pipy.org at all if that configuration is in use (unless i misunderstand how pip works, or its behavior has changed yet again) | 19:58 |
| *** rlandy|vote is now known as rlandy | 19:58 | |
| fungi | er, pypi.org | 19:58 |
| cmurphy | so far pip 18 and pip 10 exhibit this, latest patchset will see if pip 9 is the same | 19:59 |
| ianw | pip behaviour change seems likely :/ | 20:01 |
| ianw | fungi: nix review on those pinning changes ... i've realised it's not quite right as "version=latest" isn't what we want, it's "state=latest" | 20:01 |
| ianw | i'll have to redo them | 20:01 |
| fungi | ianw: yeah, "state" is what openstackansible is using for them as well | 20:02 |
| ianw | fungi: i wonder though if "latest" means we always re-install, i'm assuming it's not smart enough to check the version before doing anything | 20:03 |
| fungi | i do not know | 20:03 |
| ianw | "present" == "install" , "latest" == "install -U" | 20:05 |
| ianw | so either way, it's fully invoking pip | 20:05 |
| *** shrasool has joined #openstack-infra | 20:10 | |
| *** jamesmcarthur has quit IRC | 20:10 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: WIP - Pagure driver https://review.openstack.org/604404 | 20:11 |
| *** bobh has joined #openstack-infra | 20:11 | |
| AJaeger | config-core, could I get a second +2 on https://review.openstack.org/615637 (clarkb already reviewed) - removes now obsolete publish-static jobs from project-config | 20:13 |
| AJaeger | , please? | 20:13 |
| *** rfolco|doctor is now known as rfolco|off | 20:16 | |
| *** mriedem_vote is now known as mriedem | 20:20 | |
| *** jtomasek has joined #openstack-infra | 20:21 | |
| *** aojea has quit IRC | 20:24 | |
| *** jtomasek has quit IRC | 20:24 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Pin bridge.o.o to ansible 2.7.0, add devel testing job https://review.openstack.org/614894 | 20:26 |
| *** jamesmcarthur has joined #openstack-infra | 20:27 | |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for wiki.openstack.org https://review.openstack.org/615659 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for logstash.openstack.org https://review.openstack.org/615660 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for subunit workers https://review.openstack.org/615661 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for elasticsearch.openstack.org https://review.openstack.org/615665 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for one git worker https://review.openstack.org/615966 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for all git workers https://review.openstack.org/615967 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for git load balancer https://review.openstack.org/615968 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for mirror-update.o.o https://review.openstack.org/615991 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for rax mirror https://review.openstack.org/615992 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for all mirrors https://review.openstack.org/615993 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for files.openstack.org https://review.openstack.org/615994 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for on zookeeper instance https://review.openstack.org/615995 | 20:28 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for all zookeeper instances https://review.openstack.org/615996 | 20:28 |
| cmurphy | same problem with pip 9 http://logs.openstack.org/21/606021/16/check/puppet-beaker-rspec-centos-7-infra/195c5ea/job-output.txt.gz | 20:32 |
| openstackgerrit | Merged openstack-infra/storyboard-webclient master: Handle project names in the new Story modal controller https://review.openstack.org/612778 | 20:33 |
| *** jamesmcarthur has quit IRC | 20:34 | |
| *** mmethot has joined #openstack-infra | 20:40 | |
| ianw | cmurphy: so it's always been a problem? or just with this new matching? | 20:43 |
| *** munimeha1 has quit IRC | 20:46 | |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for one nameserver https://review.openstack.org/615998 | 20:50 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for all nameservers https://review.openstack.org/615999 | 20:50 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for master nameserver https://review.openstack.org/616000 | 20:50 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for openstackid-dev https://review.openstack.org/616001 | 20:50 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for openstackid https://review.openstack.org/616002 | 20:50 |
| cmurphy | ianw: i don't think it's always been a problem, it worked at some point | 20:51 |
| mordred | ianw: ANSIBLE0010 ... sigh | 20:58 |
| mordred | ianw: thanks for the followup on that | 20:59 |
| AJaeger | ianw, mordred, could I get a second +2 on https://review.openstack.org/615637 (clarkb already reviewed) - removes now obsolete publish-static jobs from project-config, please? | 20:59 |
| ianw | mordred: per conversions with fungi, having thoughts on how we can pin it from variables ... working on it now | 21:00 |
| mordred | ianw: nod. although - honestly, I like your comment in the file ... we do frequently run various things CD and I do think, depending on the thing, that that's ok | 21:09 |
| clarkb | this is one of many of the nasible lint rules I disagree with | 21:10 |
| clarkb | the other being you have to use the git module which only does like 10% of what git can do | 21:11 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/nodepool master: Correct heading levels for Kubernetes config docs https://review.openstack.org/616007 | 21:11 |
| fungi | clarkb: ansiblelint should totally be renamed youredoingitwrong | 21:13 |
| openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Pin bridge.o.o to ansible 2.7.0, add devel testing job https://review.openstack.org/614894 | 21:13 |
| openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: bridge.o.o: Use latest openstacksdk https://review.openstack.org/615982 | 21:13 |
| clarkb | for that 10% its great, but it shouldn't be considered a sin to shell/command out to git itself | 21:15 |
| ianw | i don't like that, afaics, there appears to be no contextal information around the reasoning, e.g. https://github.com/ansible/ansible-lint/commit/f5fcee4eb3bea623e625de2ede02a3e2686e5e1e | 21:16 |
| ianw | the entire changelog is "Add two new rules and fix tests appropriately" | 21:16 |
| ianw | i know i go a bit overboard on change comments ... but come on ... | 21:16 |
| clarkb | ianw: ya, fwiw I find that openstack in general is good about writing commit messages. I think we've learned over the years that we are doing this with others and communication is both hard and important | 21:17 |
| fungi | i too had no luck finding any explanation of that rule other than its extremely brief title | 21:20 |
| *** bobh has quit IRC | 21:20 | |
| *** boden has joined #openstack-infra | 21:21 | |
| clarkb | ianw: I always appreciate your overboard comments | 21:21 |
| clarkb | ianw: it looks like lon1 has the floating ip router detach problem now with cloud launcher? | 21:23 |
| clarkb | why are we trying to delete subnets there ? | 21:23 |
| ianw | clarkb: oh, that's an ansible bug | 21:24 |
| clarkb | thats a scary bug if its trying to delete networking | 21:24 |
| ianw | clarkb: comments in https://storyboard.openstack.org/#!/story/2004267, that's why i'm trying to get this ansible pinning thing working because i guess we need a non-release version | 21:24 |
| *** jcoufal has quit IRC | 21:25 | |
| clarkb | gotcha, hrm | 21:25 |
| openstackgerrit | Merged openstack-infra/system-config master: Install latest openstacksdk on bridge https://review.openstack.org/615951 | 21:25 |
| clarkb | so if we use devel ansible it will fix cloud launcher if we also fix openstacksdk/clouds.yaml | 21:26 |
| ianw | clarkb: i think so, at least i did a manual run with ansible devel branch version, and it then got stuck on the clouds.yaml | 21:27 |
| ianw | so now ^^ is in, can probably try again with the manual devel-branch run ... ? | 21:28 |
| clarkb | ianw: ya | 21:28 |
| clarkb | I think the change prior to that should've fixed the clouds.yaml issue too for older sdk | 21:28 |
| clarkb | so you can run it now or wait for sdk to update | 21:28 |
| *** apetrich has joined #openstack-infra | 21:29 | |
| ianw | i've got the window up, so will do manual run now | 21:30 |
| ianw | clarkb: while on it, do you think we can add infra-core to ansible-role-cloud-launcher? | 21:30 |
| ianw | like that change you have to silence the warnings would be nice | 21:30 |
| clarkb | ianw: another workaround may be to set enable_snat to True in the module params for citycloud? | 21:30 |
| clarkb | ianw: uh if it is an infra project sure (I'm not sure if it was added as an infra project /me checks) | 21:31 |
| *** pcaruana has quit IRC | 21:31 | |
| clarkb | ianw: grepping for it in openstack/governance shows no results | 21:31 |
| clarkb | I think its a rcarrillocruz and pabelanger collaboration outside of infra? | 21:32 |
| clarkb | huh how did I get added to core on that | 21:32 |
| ianw | yeah, that's why i was wondering if we can come up with some sort of agreement | 21:32 |
| clarkb | rcarrillocruz: ^ the infra team is still using this tool. Do you mind if we adopt it?' | 21:33 |
| ianw | clarkb: hrm, yeah the explicit snat:true might work around that too, i guess that seems otherwise pretty safe? | 21:33 |
| ianw | hrm, it still bailed on | 21:34 |
| ianw | "msg": "Region Lon1 is not a valid region name for cloud openstackzuul-citycloud. Valid choices are Kna1,La1. Please note that region names are case sensitive." | 21:34 |
| ianw | i'll wait for the sdk update and try again | 21:34 |
| clarkb | ianw: looks like the all-clouds.yaml update hasn't applied yet | 21:34 |
| clarkb | when did that merge | 21:34 |
| clarkb | two and a half hours ago | 21:35 |
| ianw | oh, ha, i just assumed that was done :) might be a typo stopping it? ... | 21:35 |
| clarkb | 2018-11-06 21:30:15,642 p=25732 u=root | ERROR! Completely failed to parse inventory source /opt/system-config/inventory/openstack.yaml | 21:35 |
| rcarrillocruz | clarkb: by all means | 21:36 |
| ianw | yep ... | 21:36 |
| rcarrillocruz | more than happy to | 21:36 |
| clarkb | rcarrillocruz: thanks! | 21:36 |
| rcarrillocruz | yw! | 21:36 |
| clarkb | #status log infra-core added to ansible-role-cloud-launcher-core after getting rcarrillocruz's go ahead | 21:36 |
| openstackstatus | clarkb: finished logging | 21:36 |
| clarkb | ianw: openstack plugin: __init__() got an unexpected keyword argument 'interface' | 21:37 |
| clarkb | openstacksdk hasn't updated yet | 21:39 |
| clarkb | https://github.com/ansible/ansible/blob/v2.7.0/lib/ansible/plugins/inventory/openstack.py doesn't show an 'interface' when grepped | 21:39 |
| clarkb | so something is passing it interface and it is getting mad? | 21:39 |
| clarkb | mordred: ^ that look familiar to you? | 21:40 |
| *** ansmith has quit IRC | 21:42 | |
| mordred | clarkb: not immediately, no. I'd like to know what is passing interface to the plugin - that seems super weird | 21:42 |
| clarkb | ya unfortunately that is the extent of the error message we seem to get in ansible.log | 21:42 |
| clarkb | (I dno't see a traceback) | 21:42 |
| mordred | clarkb: _blerg_ | 21:43 |
| mordred | clarkb: looking | 21:43 |
| clarkb | it comes after stevedore loads all the keystone auth entrypoints | 21:43 |
| clarkb | so could be that interface is happening deeper than the ansible plugin level | 21:43 |
| *** hamzy has quit IRC | 21:45 | |
| mordred | yeah ... but still, interface should be an acceptable parameter | 21:46 |
| *** bobh has joined #openstack-infra | 21:46 | |
| mordred | clarkb: I got tracebacks by doing: | 21:47 |
| mordred | ansible -vvvv bridge.openstack.org -m setup | 21:47 |
| mordred | although it's not making me happier | 21:47 |
| mordred | oh - wait a second - do we have interface in an auth dict ? | 21:48 |
| mordred | yup. there it is | 21:48 |
| mordred | patch coming | 21:48 |
| clarkb | ya armci cloud added it looks like | 21:48 |
| fungi | yeah, thanks again ianw for working on that! | 21:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: interface is not an auth option https://review.openstack.org/616015 | 21:49 |
| mordred | clarkb, fungi: ^^ | 21:49 |
| fungi | oof | 21:50 |
| mordred | clarkb: I edited the clouds.yaml by hand for now to verify | 21:50 |
| clarkb | we may need to manually apply that to /etc/openstack/clouds.yaml on bridge.o.o once it merges as it won't update itself on that host | 21:50 |
| clarkb | cool | 21:50 |
| ianw | hrm, i guess the import we do during unit testing is only checking yaml-ish syntax, not correctness ... probably can't do that without actually running things | 21:51 |
| mordred | clarkb: this once again makes me think I want to update the running of the bridge playbook to not use the openstack inventory | 21:51 |
| mordred | ianw: we can (and should) make a clouds.yaml linter | 21:52 |
| clarkb | mordred: and just do localhost localconnection and call it a day? | 21:52 |
| mordred | clarkb: yeah - or even execute it pointing to a whole other inventory and ansible.cfg so that it doesn't even try to run it or something | 21:52 |
| mordred | clarkb: I havne't written a patch yet because i'm nt 100% what feels right | 21:52 |
| clarkb | mordred: the nice thing about the current setup is that its consistent and clear | 21:53 |
| mordred | but there's too much opportunity to get wedged while applying changes to the system | 21:53 |
| mordred | yah | 21:53 |
| clarkb | yes if it breaks we have to manually unwedge it | 21:53 |
| clarkb | but maybe that is worthwhile? I don't know | 21:53 |
| *** geekinutah has quit IRC | 21:57 | |
| *** geekinutah has joined #openstack-infra | 21:57 | |
| *** bobh has quit IRC | 22:02 | |
| *** kgiusti has left #openstack-infra | 22:09 | |
| *** rh-jelabarre has quit IRC | 22:14 | |
| *** eernst has quit IRC | 22:19 | |
| ianw | mordred: a voluptuous style parser, or you thinking something else? | 22:26 |
| mordred | ianw: I was thinking something provided by the sdk codebase that actually parses semantics ... although maybe that could be done via voluptuous | 22:27 |
| mordred | ianw: of course, for that to reall work, we should probably fully define and document all of the semantics :) | 22:28 |
| ianw | mordred: yeah ... voluptuous errors would be better than weird "this isn't a dict" type errors you can get from deep inside bits of the sdk when you screw it up currently | 22:31 |
| mordred | yeah | 22:32 |
| ianw | clarkb: not sure if you saw it but https://review.openstack.org/#/c/615698/ implements the discussion on the traceroute failures we discussed yesterday | 22:32 |
| ianw | namely a retry, and capturing of unbound logs | 22:32 |
| clarkb | ianw: I hadn't but I shall now take a look | 22:33 |
| *** felipemonteiro has joined #openstack-infra | 22:33 | |
| *** dave-mccowan has quit IRC | 22:35 | |
| openstackgerrit | Merged openstack-infra/system-config master: interface is not an auth option https://review.openstack.org/616015 | 22:37 |
| clarkb | I'll edit the clouds.yaml file on bridge.o.o now so that ^ can apply | 22:38 |
| *** bobh has joined #openstack-infra | 22:42 | |
| *** bobh has quit IRC | 22:45 | |
| *** bobh has joined #openstack-infra | 22:46 | |
| *** bobh has quit IRC | 22:50 | |
| clarkb | ianw: looks like all-clouds.yaml has the citycloud fix now. And next ansible pass should add the interface fix | 22:52 |
| clarkb | if you want to give cloud launcher another run | 22:52 |
| ianw | ok, bash history recall coming | 22:54 |
| ianw | and running | 22:55 |
| ianw | hrm, ansible-role-cloud-launcher doens't seem to be registered on readthedocs, but seems to think it's publishing there | 22:57 |
| *** boden has quit IRC | 22:57 | |
| ianw | keystoneauth1.exceptions.http.Unauthorized: The request you have made requires authentication. (HTTP 401) (Request-ID: req-b085e685-c63d-430c-8d74-db63e4e9f22f) | 23:01 |
| ianw | hrumph, it doesn't tell you the cloud it was working on | 23:02 |
| clarkb | ianw: that might be the interface bug in all-clouds | 23:02 |
| clarkb | since that would cause broken auth aiui | 23:02 |
| ianw | well it's a different error to before | 23:02 |
| *** shrasool has quit IRC | 23:03 | |
| ianw | do we consume ansible-role-cloud-launcher from git or releases? putting in your fix, and adding the cloud name to the running task would help with this a lot | 23:03 |
| clarkb | ianw: system-config/roles.yaml says we consume HEAD | 23:05 |
| *** felipemonteiro has quit IRC | 23:07 | |
| openstackgerrit | Ian Wienand proposed openstack/ansible-role-cloud-launcher master: Add the cloud name to more task details https://review.openstack.org/616027 | 23:09 |
| ianw | clarkb: ^ i think that would let us know what cloud is failing in the logs | 23:09 |
| clarkb | ianw: agreed | 23:11 |
| openstackgerrit | Merged openstack/ansible-role-cloud-launcher master: Don't use deprecated when jinja2 syntax https://review.openstack.org/600551 | 23:18 |
| openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Fix ansible-role-cloud-launcher docs https://review.openstack.org/616029 | 23:21 |
| clarkb | ianw: on the network debugging change I am trying to understand what zuul_debug_info.py s doing now | 23:27 |
| clarkb | what is an image_manifest? | 23:28 |
| clarkb | the other thing is this is in zuul-jobs and is going to change debug info existing users gets | 23:28 |
| ianw | clarkb: it grabs like /etc/dib-builddate and stuff and stuff it into the output | 23:29 |
| clarkb | ah | 23:29 |
| ianw | i don't think the changes are incompatible ... it only adds info | 23:30 |
| clarkb | I think the interface thing is only a minor concern since the role itself is the interface and that doesn't change | 23:30 |
| clarkb | ya would only be an issue if someone uses that python module directly instead of the role | 23:30 |
| ianw | yeah, personally i think we can rule that out, it's under library/ in a role directory | 23:31 |
| clarkb | you've convinced me +2 | 23:31 |
| ianw | clarkb: can we take the more relaxed approach with ansible-role-cloud-launcher and singe approve 616027? i'm really not sure how else to tell which cloud it's getting hung up on without a bunch of effort | 23:32 |
| *** rfolco|off has quit IRC | 23:33 | |
| ianw | i'm not sure what we could do for testing in that project | 23:33 |
| clarkb | ianw: ya I think you can do that since it really only changes the names and not function | 23:33 |
| ianw | i guess it would have to be a devsatck-ish based test | 23:34 |
| clarkb | hrm clouds.yaml broke itself again so ansible isn't running again | 23:34 |
| clarkb | maybe there was a race between me editing it and ansible switching it back again | 23:35 |
| clarkb | I'll edit it again under that assumption and if it breaks again dig in | 23:35 |
| openstackgerrit | Merged openstack/ansible-role-cloud-launcher master: Add the cloud name to more task details https://review.openstack.org/616027 | 23:41 |
| *** yamamoto has joined #openstack-infra | 23:48 | |
| *** yamamoto has quit IRC | 23:53 | |
| *** rpioso is now known as rpioso|afk | 23:54 | |
| ianw | ok, running the launcher and it's grabbed those two changes | 23:56 |
| ianw | output is much nicer | 23:56 |
| clarkb | system-config on bridge updated to include the interface fix so I expect the last manual fix was the last one | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!