Wednesday, 2021-05-05

*** tkajinam has joined #puppet-openstack01:13
openstackgerritTakashi Kajinami proposed openstack/puppet-ironic master: Add support for audit middleware options  https://review.opendev.org/c/openstack/puppet-ironic/+/78967601:15
openstackgerritTakashi Kajinami proposed openstack/puppet-nova master: Clean up deprecated parameters of nova::compute::libvirt  https://review.opendev.org/c/openstack/puppet-nova/+/78955901:33
openstackgerritTakashi Kajinami proposed openstack/puppet-cinder master: Add support for MultiStrOpt  https://review.opendev.org/c/openstack/puppet-cinder/+/78967701:45
openstackgerritTakashi Kajinami proposed openstack/puppet-heat master: Add support for MultiStrOpt  https://review.opendev.org/c/openstack/puppet-heat/+/78967801:49
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add support for independent heat-dashboard configurations  https://review.opendev.org/c/openstack/puppet-horizon/+/77532202:01
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add support for independent heat-dashboard configurations  https://review.opendev.org/c/openstack/puppet-horizon/+/77532202:55
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add support for independent octavia-dashboard configurations  https://review.opendev.org/c/openstack/puppet-horizon/+/78970102:55
*** rcernin has quit IRC03:05
*** rcernin has joined #puppet-openstack03:16
*** ykarel has joined #puppet-openstack04:20
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add support for independent heat-dashboard configurations  https://review.opendev.org/c/openstack/puppet-horizon/+/77532206:12
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add support for independent octavia-dashboard configurations  https://review.opendev.org/c/openstack/puppet-horizon/+/78970106:12
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add support for independent heat-dashboard configurations  https://review.opendev.org/c/openstack/puppet-horizon/+/77532206:28
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add support for independent octavia-dashboard configurations  https://review.opendev.org/c/openstack/puppet-horizon/+/78970106:28
openstackgerritTakashi Kajinami proposed openstack/puppet-ironic master: Add support for audit middleware options  https://review.opendev.org/c/openstack/puppet-ironic/+/78967606:43
*** rpittau|afk is now known as rpittau07:12
*** rcernin has quit IRC07:21
*** rcernin has joined #puppet-openstack07:25
*** tosky has joined #puppet-openstack07:30
*** jpena|off is now known as jpena07:57
*** rcernin has quit IRC07:59
*** derekh has joined #puppet-openstack08:00
*** amoralej|off is now known as amoralej08:14
*** rcernin has joined #puppet-openstack08:23
*** ysandeep is now known as ysandeep|lunch08:28
*** rcernin has quit IRC08:29
*** rcernin has joined #puppet-openstack08:38
*** rcernin has quit IRC08:42
*** rcernin has joined #puppet-openstack08:46
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add support for independent heat-dashboard configurations  https://review.opendev.org/c/openstack/puppet-horizon/+/77532209:03
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add support for independent octavia-dashboard configurations  https://review.opendev.org/c/openstack/puppet-horizon/+/78970109:03
*** derekh has quit IRC09:05
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/train: DNM: Test beaker job  https://review.opendev.org/c/openstack/puppet-nova/+/78976009:12
*** dtantsur|afk is now known as dtantsur09:21
*** ykarel has quit IRC09:22
*** ykarel has joined #puppet-openstack09:22
*** rcernin has quit IRC09:30
*** derekh has joined #puppet-openstack09:53
*** rcernin has joined #puppet-openstack09:54
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration stable/train: Add selinux-policy workaround to low dep in openstack-selinux  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78977310:03
tobias-urdintkajinam: ^ maybe?10:03
*** rcernin has quit IRC10:05
*** ysandeep|lunch is now known as ysandeep10:10
openstackgerritTobias Urdin proposed openstack/puppet-keystone stable/train: Check length of unique array in roles::admin  https://review.opendev.org/c/openstack/puppet-keystone/+/78950810:14
*** tosky has quit IRC10:38
*** tosky has joined #puppet-openstack10:38
*** tosky_ has joined #puppet-openstack10:45
*** tosky has quit IRC10:47
*** tosky_ is now known as tosky10:49
*** rcernin has joined #puppet-openstack10:52
tkajinamtobias-urdin, yeah I was thinking of the same10:58
*** rcernin has quit IRC10:58
*** rcernin has joined #puppet-openstack11:02
*** rcernin has quit IRC11:06
*** rcernin has joined #puppet-openstack11:24
*** jpena is now known as jpena|lunch11:26
*** rcernin has quit IRC11:28
openstackgerritTakashi Kajinami proposed openstack/puppet-openstack-integration master: Disable selinux in CentOS7 beaker job  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78979511:44
openstackgerritTakashi Kajinami proposed openstack/puppet-openstack-integration stable/train: Disable selinux in CentOS7 beaker job  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78981611:45
*** derekh has quit IRC11:45
*** rcernin has joined #puppet-openstack11:45
openstackgerritTakashi Kajinami proposed openstack/puppet-openstack-integration master: Remove leftover about removed beaker job  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78981711:47
tkajinamtobias-urdin, it seems the same error is still caused even with that patch11:48
tkajinamtobias-urdin, ^^^ I have submitted a patch to disable selinux on centos7 beaker job. This is dirty but I believe it is a better workaround now11:49
tkajinamtobias-urdin, fyi, selinux-policy-3.13.1-268.el7_9.2 seems to be the latest one currently available... so updating selinux-policy has no effect, I'm afraid11:50
*** rcernin has quit IRC11:55
ykareltkajinam, just in case you not aware, we hit similar in packstack some time back https://bugs.launchpad.net/packstack/+bug/1923005, bug have some more context11:58
openstackLaunchpad bug 1923005 in Packstack "[Train][CentOS7] Packstack deployment fails while starting httpd with SELINUX enabled" [High,Triaged]11:58
ykarellikely same is hit in poi too11:58
tkajinamykarel, ahh12:07
tkajinamykarel, thanks for the info. yeah that looks same12:07
tkajinamykarel, so IIUC recent change in openstack-selinux causes an issue in CentOS7 because CentOS7 doesn't have some rules for podman ?12:10
tkajinamif my understanding is correct then we'd need to branch openstack-selinux for stable releases. especially ones running on centos712:11
ykareltkajinam, yes container-linux version is old in centos712:11
ykareltrain is both c7 and c812:11
ykarelare u hitting before train too?12:11
ykarelbefore train i think it's pinned, but can recheck12:11
tkajinamI've not yet checked that. let me quickly try it12:11
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/stein: DNM: Test beaker job  https://review.opendev.org/c/openstack/puppet-nova/+/78982012:12
tkajinam^^^ let's see how this goes...12:12
tkajinambut if we have the same version of openstack-selinux in older branches then I guess we are likely to hit the same12:13
tkajinamas long as I see the issue is not reproduced in ussuri on CentOS8 but that would be because tests are executed on CentOS8 which has newer containers-selinux12:13
*** amoralej is now known as amoralej|lunch12:14
*** rcernin has joined #puppet-openstack12:18
tkajinamykarel, I think you are correct about the pin. I see openstack-selinux-0.8.23-0.20200821105100.f05f4b2.el7.noarch.rpm in centos7-stein12:22
tkajinamwhich is likely to be the version without latest 3 changes12:22
tkajinam(and the latest 2 are causing the problem, IIUC12:23
*** rcernin has quit IRC12:23
tobias-urdintkajinam: not sure beaker job even checks out the p-o-i change12:24
tobias-urdinbut maybe you've pinned down the problem already? :)12:25
ykareltkajinam, mainly this is the culprit https://github.com/redhat-openstack/openstack-selinux/commit/1f3ab78f0d9b5e1d76ca420873889e9c6f54faf012:26
*** jpena|lunch is now known as jpena12:30
tobias-urdinykarel: is there a newer container-selinux available for centos7 we are just not updating it or it's just not there at all?12:32
ykareltobias-urdin, container-selinux good version which is compatible with latest openstack-selinux not available in centos 7 base repos12:33
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration stable/train: Add selinux-policy workaround to low dep in openstack-selinux  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78977312:33
openstackgerritTobias Urdin proposed openstack/puppet-keystone stable/train: Check length of unique array in roles::admin  https://review.opendev.org/c/openstack/puppet-keystone/+/78950812:34
tobias-urdinykarel: ack12:35
tkajinamanother option would be to pin openstack-selinux to an older version12:36
openstackgerritTobias Urdin proposed openstack/puppet-keystone stable/train: Check length of unique array in roles::admin  https://review.opendev.org/c/openstack/puppet-keystone/+/78950812:36
tkajinam... ignore it. it seems that only the latest version is available12:36
tobias-urdintkajinam: ack - i added dependency on your selinux disable patch to above12:37
tkajinamtobias-urdin, yeah I think that would be better until we have the current issue fixed in rdo side12:37
tkajinamtobias-urdin, thanks for updating the patch12:38
tobias-urdintkajinam: ykarel thank you guys for checking everything :)12:39
*** amoralej|lunch is now known as amoralej13:09
*** derekh has joined #puppet-openstack13:10
*** morazi has quit IRC13:20
*** morazi has joined #puppet-openstack13:20
*** dpawlik has quit IRC13:22
openstackgerritTobias Urdin proposed openstack/puppet-neutron master: Escape hyphen in neutron_agent_sriov_numvfs regexp  https://review.opendev.org/c/openstack/puppet-neutron/+/78983413:45
*** dpawlik5 has joined #puppet-openstack14:27
*** dpawlik5 is now known as dpawlik14:32
openstackgerritRonelle Landy proposed openstack/puppet-openstack-integration master: Add fully quaified path to openstack/tempest  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78985214:49
openstackgerritMerged openstack/puppet-openstack-integration stable/train: Disable selinux in CentOS7 beaker job  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78981615:03
*** ysandeep is now known as ysandeep|away15:27
*** ykarel has quit IRC15:28
*** rpittau is now known as rpittau|afk15:52
openstackgerritTakashi Kajinami proposed openstack/puppet-openstack-integration stable/train: Fix ansible error to make selinux permissive  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78992316:14
openstackgerritTakashi Kajinami proposed openstack/puppet-openstack-integration stable/train: Fix ansible error when making selinux permissive  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78992316:17
tkajinam^^ tobias-urdin mwhahaha, sorry about this :-(16:18
mwhahahaoops :D16:18
*** dtantsur is now known as dtantsur|afk16:38
*** jpena is now known as jpena|off16:46
*** gfidente is now known as gfidente|afk16:51
*** derekh has quit IRC17:00
*** amoralej is now known as amoralej|off17:43
*** amoralej|off is now known as amoralej18:32
*** amoralej is now known as amoralej|off18:33
openstackgerritMerged openstack/puppet-openstack-integration stable/train: Fix ansible error when making selinux permissive  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78992318:58
*** tosky has quit IRC23:03
*** rcernin has joined #puppet-openstack23:04
openstackgerritTakashi Kajinami proposed openstack/puppet-cinder master: Fix handling of the backend_host parameter  https://review.opendev.org/c/openstack/puppet-cinder/+/78908723:17
openstackgerritTakashi Kajinami proposed openstack/puppet-cinder master: Deprecate the redundant cinder::backend_host parameter  https://review.opendev.org/c/openstack/puppet-cinder/+/78908823:23
openstackgerritTakashi Kajinami proposed openstack/puppet-openstack-integration stable/train: Update selinux mode by root  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/78996023:33
* tkajinam sighs23:36

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