Tuesday, 2021-04-06

*** dpawlik5 has quit IRC00:10
*** dpawlik7 has joined #openstack-ansible00:11
*** spatel_ has joined #openstack-ansible00:34
*** spatel_ is now known as spatel00:34
*** spatel has quit IRC00:36
*** dpawlik7 has quit IRC00:57
*** dpawlik5 has joined #openstack-ansible00:57
*** dpawlik2 has joined #openstack-ansible01:23
*** dpawlik5 has quit IRC01:23
*** jamesdenton has quit IRC01:27
*** jamesdenton has joined #openstack-ansible01:27
*** dpawlik2 has quit IRC01:33
*** dpawlik3 has joined #openstack-ansible01:36
*** spatel_ has joined #openstack-ansible02:28
*** spatel_ is now known as spatel02:28
*** evrardjp has quit IRC02:33
*** evrardjp has joined #openstack-ansible02:33
*** dpawlik9 has joined #openstack-ansible02:35
*** dpawlik3 has quit IRC02:36
*** rohit02 has joined #openstack-ansible02:42
openstackgerritSatish Patel proposed openstack/openstack-ansible-os_neutron master: Add OVN Distributed L3 routing config documentation  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/78487303:14
openstackgerritSatish Patel proposed openstack/openstack-ansible-os_neutron master: Add OVN Distributed L3 routing config documentation  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/78487303:16
*** rohit02 has quit IRC03:23
*** spatel has quit IRC03:35
*** rh-jelabarre has quit IRC03:43
*** rohit02 has joined #openstack-ansible03:48
*** dpawlik9 has quit IRC04:35
*** dpawlik6 has joined #openstack-ansible04:35
*** jamesdenton has quit IRC04:45
*** jamesdenton has joined #openstack-ansible04:47
*** miloa has joined #openstack-ansible05:31
*** dpawlik7 has joined #openstack-ansible05:34
*** dpawlik6 has quit IRC05:35
*** yasemind has joined #openstack-ansible05:35
*** miloa has quit IRC05:44
*** pcaruana has joined #openstack-ansible06:13
*** SiavashSardari has joined #openstack-ansible06:33
*** dpawlik5 has joined #openstack-ansible06:42
*** dpawlik7 has quit IRC06:42
*** dpawlik5 has quit IRC06:52
*** dpawlik07 has joined #openstack-ansible06:55
*** shyamb has joined #openstack-ansible06:57
*** dpawlik07 has quit IRC06:57
*** dpawlik5 has joined #openstack-ansible06:58
*** sshnaidm is now known as sshnaidm|afk06:58
*** dpawlik5 has quit IRC07:12
*** dpawlik5 has joined #openstack-ansible07:13
*** dpawlik5 is now known as dpawlik07:14
*** luksky has joined #openstack-ansible07:16
*** andrewbonney has joined #openstack-ansible07:19
*** dpawlik has quit IRC07:28
*** dpawlik6 has joined #openstack-ansible07:28
*** macz_ has joined #openstack-ansible07:29
*** macz_ has quit IRC07:34
*** tosky has joined #openstack-ansible07:38
*** dpawlik6 has quit IRC07:43
*** pcaruana has quit IRC07:43
*** fanfi has quit IRC07:45
*** shyamb has quit IRC08:00
openstackgerritJonathan Herlin proposed openstack/openstack-ansible-os_tempest master: Use include-list instead of deprecated option  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/78488508:17
*** pcaruana has joined #openstack-ansible08:29
*** rohit02 has quit IRC09:03
*** lkoranda has joined #openstack-ansible09:12
*** rohit02 has joined #openstack-ansible09:19
*** shyamb has joined #openstack-ansible09:49
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Add OVN Distributed L3 routing config documentation  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/78487309:51
*** rohit02 has quit IRC09:56
*** rohit02 has joined #openstack-ansible09:58
*** sshnaidm|afk has quit IRC10:09
*** rohit02 has quit IRC10:18
*** sshnaidm has joined #openstack-ansible10:18
*** rohit02 has joined #openstack-ansible10:19
jonhernoonedeadpunk i don't see tempest_test_whitelist_file_path being overridden by any other repo/playbook. For the second comment about the integrated repo, as long as "tempest_test_whitelist" is not renamed there should not need to be any change in any other repo. Or am I missing something?10:31
*** shyamb has quit IRC10:32
noonedeadpunkjonher: well, I mean that these rename inside the tempest is caused by diversity aim I guess (or dunno how tolerantly say that). So we also should rename variables inside the OSA to match ones in tempest10:33
*** luksky has quit IRC10:34
noonedeadpunkoh, right tempest_test_blacklist_file_path is a local thing10:34
*** luksky has joined #openstack-ansible10:34
jonheryeah sure, but changing that in all sub-repos is a lot larger of a change10:35
openstackgerritJonathan Herlin proposed openstack/openstack-ansible-os_tempest master: Use include-list instead of deprecated option  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/78488510:36
noonedeadpunkyeah, agree. so let's just rename that in temepst role but set old value as a default one ?10:36
noonedeadpunkand then we could iterate over and change in other repos10:36
jonherif we set the old value as default the lists would not merge if passed both tempest_test_whitelist and tempest_test_includelist right?10:40
noonedeadpunkyeah, just posted comment what I mean10:41
*** yasemind has quit IRC10:56
*** SiavashSardari has quit IRC11:10
openstackgerritGeorgina Shippey proposed openstack/openstack-ansible-os_barbican master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/78085911:10
openstackgerritGeorgina Shippey proposed openstack/openstack-ansible-os_keystone master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/78153711:13
*** jamesdenton has quit IRC11:34
*** jamesdenton has joined #openstack-ansible11:35
*** fanfi has joined #openstack-ansible11:38
*** rh-jelabarre has joined #openstack-ansible11:56
*** shyamb has joined #openstack-ansible11:59
*** shyamb has quit IRC12:05
openstackgerritMerged openstack/openstack-ansible-os_tacker master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/78231312:17
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova master: Re-add nova_dhcp_domain variable  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/78493912:20
openstackgerritGeorgina Shippey proposed openstack/openstack-ansible-os_panko master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_panko/+/78228512:22
openstackgerritMerged openstack/openstack-ansible-os_senlin master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/78230912:32
openstackgerritMerged openstack/openstack-ansible-os_designate master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/78089612:33
*** yasemind has joined #openstack-ansible12:34
openstackgerritJonathan Herlin proposed openstack/openstack-ansible-os_tempest master: Rename whitelist and blacklist in role  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/78488512:56
openstackgerritGeorgina Shippey proposed openstack/openstack-ansible-os_magnum master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/78153812:56
jonhertook a shot on the releasenotes as well, not sure what level of detail you want it on, your comments should be adressed12:57
openstackgerritMerged openstack/openstack-ansible-os_nova master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/78228412:59
*** spatel_ has joined #openstack-ansible12:59
*** spatel_ is now known as spatel12:59
openstackgerritGeorgina Shippey proposed openstack/openstack-ansible-os_sahara master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/78231013:00
openstackgerritJonathan Herlin proposed openstack/openstack-ansible-os_tempest master: Rename whitelist and blacklist in role  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/78488513:02
noonedeadpunkjonher: reno looks good13:02
jonherjust as i pushed an update because i read the release notes section where upgrades should not mention this type of change (from how i interpreted it)13:02
spatelnoonedeadpunk why this codes are not colorful? - https://docs.openstack.org/openstack-ansible-os_neutron/latest/app-ovn.html#optional-dvr-or-distributed-l3-routing13:03
noonedeadpunkjonher: whatever:)13:04
noonedeadpunkspatel: because it should be `.. code-block:: yaml` here https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/784873/2/doc/source/app-ovn.rst13:04
noonedeadpunkand we missed that during review13:05
jonheri believe the reno note is short and specific enough in the last patchset anyways, should be fine13:05
noonedeadpunklgtm13:07
spatelnoonedeadpunk all other code are console so i just copy paste from there13:09
noonedeadpunkspatel: below yes, upper - not13:09
spateldamn it13:09
spatelhow to fix that, do you want me to cut new patch ?13:10
spatelthis is already merged13:10
noonedeadpunkif we want to fix it - yes, new patch only13:10
spatellets fix it, it looks good on screen with colors :)13:10
* noonedeadpunk reads sources :p13:10
spatelhaha13:10
noonedeadpunkjust push a patch13:11
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova master: Re-add nova_dhcp_domain variable  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/78493913:12
spateldoing it13:12
openstackgerritSatish Patel proposed openstack/openstack-ansible-os_neutron master: Fix documentation formating  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/78495413:17
openstackgerritMerged openstack/openstack-ansible-openstack_hosts stable/victoria: Ability to add extra package manager configuration  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/78458013:25
spotznoonedeadpunk: You want me too quick edit or make comments on your doc patch?13:37
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible master: Return PyMySQL installation for distro installs  https://review.opendev.org/c/openstack/openstack-ansible/+/78495713:39
noonedeadpunkspotz: how you feel more comfortable. You may feel free to quick edit it :)13:41
spotznoonedeadpunk: Ok:)13:41
openstackgerritMerged openstack/openstack-ansible-os_masakari master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/78154113:44
spotznoonedeadpunk: Ok I think the edits are there, I did them in gettit as they were minor but still getting used to the new version13:46
spotzha new button!13:47
noonedeadpunkyep, it's a bit different... I still haven't found how to stop editing without publish to see the diff...13:47
openstackgerritAmy Marrich (spotz) proposed openstack/openstack-ansible-os_trove master: [doc] Document how to use separate RabbitMQ cluster  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78478113:47
spotznoonedeadpunk: I can't go back noow but I think I did it under files it said Base -> Edit and click on edit either that or the patch link right next to it13:50
*** jamesdenton has quit IRC13:51
*** jamesdenton has joined #openstack-ansible13:52
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Fix documentation formating  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/78495413:52
openstackgerritMerged openstack/openstack-ansible-os_trove master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78231414:01
*** spatel has quit IRC14:12
*** yasemind has quit IRC14:13
openstackgerritMerged openstack/openstack-ansible-os_ironic master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/78151914:15
*** spatel_ has joined #openstack-ansible14:18
*** spatel_ is now known as spatel14:18
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible master: Map dbaas and lbaas with role defaults  https://review.opendev.org/c/openstack/openstack-ansible/+/78411314:19
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Use uWSGI for trove-api  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78420214:29
noonedeadpunkspatel: now https://docs.openstack.org/openstack-ansible-os_neutron/latest/app-ovn.html#optional-dvr-or-distributed-l3-routing is colorful :)14:29
spatelnoonedeadpunk :) love it14:30
*** rohit02 has quit IRC14:30
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Update trove configuration  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78457114:36
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: [doc] Document how to use separate RabbitMQ cluster  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78478114:37
*** macz_ has joined #openstack-ansible14:42
*** fanfi has quit IRC14:45
*** luksky has quit IRC14:46
*** luksky has joined #openstack-ansible14:46
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Update trove configuration  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78457114:46
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Update trove configuration  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78457115:14
*** gshippey has joined #openstack-ansible15:21
*** d34dh0r53 has quit IRC15:47
noonedeadpunkfolks, can we skip todays meeting? can't make it because of the summer time :(15:52
-openstackstatus- NOTICE: POST_FAILURE results between 14:00 and 15:50 UTC can be safely rechecked, and were due to authentication problems in one of our storage donor regions15:53
jrossernoonedeadpunk: sure, we can move it earlier by an hour if you like because it's difficult for me too now15:54
noonedeadpunkI think I will make a poll then for new time :)15:54
jrosserso long as its totally clear what TZ the poll talks about :)15:55
noonedeadpunkI'd stick to UTC anyway:)15:56
noonedeadpunkor you want to play with summer time?:)15:56
* noonedeadpunk hopes that summer time will be cancelled here next year15:58
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Update trove configuration  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78457116:02
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible master: Map dbaas and lbaas with role defaults  https://review.opendev.org/c/openstack/openstack-ansible/+/78411316:05
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: [doc] Document how to use separate RabbitMQ cluster  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78478116:08
*** d34dh0r53 has joined #openstack-ansible16:08
openstackgerritDmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: [doc] Document how to use separate RabbitMQ cluster  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/78478116:08
openstackgerritMerged openstack/openstack-ansible-os_cinder master: [goal] Deprecate the JSON formatted policy file  https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/78086916:44
*** macz_ has quit IRC16:59
openstackgerritMerged openstack/openstack-ansible-os_senlin master: [reno] Stop publishing release notes  https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/77204617:02
*** andrewbonney has quit IRC17:16
openstackgerritMerged openstack/openstack-ansible-os_octavia stable/ussuri: Omit amp_ssh_access_allowed and remove amp_image_id options.  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/76749417:19
*** macz_ has joined #openstack-ansible17:22
openstackgerritMerged openstack/openstack-ansible master: Adjust magnum CI image  https://review.opendev.org/c/openstack/openstack-ansible/+/77424317:47
*** rh-jelabarre has quit IRC18:31
*** jamesdenton has quit IRC18:54
*** jamesden_ has joined #openstack-ansible18:54
openstackgerritMerged openstack/openstack-ansible-os_trove master: [reno] Stop publishing release notes  https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/77205219:36
*** fanfi has joined #openstack-ansible19:37
*** rh-jelabarre has joined #openstack-ansible20:32
*** jbadiapa has quit IRC20:43
openstackgerritMerged openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in group_vars  https://review.opendev.org/c/openstack/openstack-ansible/+/77799721:01
*** jamesden_ has quit IRC21:11
*** jamesdenton has joined #openstack-ansible21:12
*** lkoranda has quit IRC21:18
*** pcaruana has quit IRC21:28
jonhernow that 778352 has merged, can https://review.opendev.org/c/openstack/openstack-ansible-os_adjutant/+/777607 be changed to only require mariadb-devel in the container where adjutant will run? or is there still a conflict that will affect rhel metal deployments?21:55
*** spatel has quit IRC21:58
jrosserjonher: i think the issue is that the tests that are being done there are all 'metal' types, so whilst the jobs all pass, it won't work for an LXC deployment22:11
jrosseras it stands now 777607 only installs mariadb-devel wherever adjutant will run, be that on the metal host or in its own container22:12
jonherthe requirement is dropped from requirements.txt in the adjutant so it should not be needed on wheel build anyore right? just during execution, would that still not be possible with the rhel situation? 2 mariadb-devel packages are mentioned with different capitalisation22:14
jrosserwheel build is generally making python bindings into some shared library .so file22:15
jrosserso if the .so / -devel package in the repo server is not the same one as is found on the target where the venv is installed then unexpected bad things can happen22:15
jrosserhence the issue with metal deploys which will have the proper mariadb repo and packages, rather than the one from the distro22:16
jonheri see22:17
jrosseri am really not sure what the right answer is22:19
jrossereverything is much simpler with the native python mysql client libraries22:19
jrosserjonher: did adjutant remove the need for this?22:20
jrosseroh well not really https://github.com/openstack/adjutant/commit/ad19ed13b65b583e65b5a19e04a0f0403c366b0922:20
jonherthey didn't i'm afraid, pymysql is not supported with the framework used22:20
jonherpymysql can register and from my local testing it seemed to work, but the dev was going to have a look at it22:21
*** jamesdenton has quit IRC22:43
*** jamesdenton has joined #openstack-ansible22:44
*** macz_ has quit IRC23:53

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!