Thursday, 2023-10-19

mnasiadkafrickler: thanks for progressing the ansible-core bug07:14
mnasiadkafrickler: I only meant that it seems cberendt has and it would be nice if that would land before we cut B release07:19
SvenKieskeo/07:19
fricklermnasiadka: seems we've at least found the cause for the ansible issue now, not sure about the fix though. I'll try now whether using different filenames for the plugins in k-a could serve as a workaround07:26
mnasiadkafrickler: it does help, let me raise a patch07:40
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Rename per role filters files  https://review.opendev.org/c/openstack/kolla-ansible/+/89879907:45
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Revert "CI: Pin ansible-core due to breakage"  https://review.opendev.org/c/openstack/kolla-ansible/+/89876707:46
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Revert "CI: Pin ansible-core due to breakage"  https://review.opendev.org/c/openstack/kolla-ansible/+/89876707:46
frickleroh, nice, I assumed there would be more filters08:01
SvenKieskemnasiadka: frickler: is it really worth investing in a workaround when an upstream patch will reintroduce the old behaviour soon(TM)? it's an open honest question, because the patch might still take some time..08:01
SvenKieskemaybe we should prefix all filter names with "kolla_"?  :D08:01
fricklerSvenKieske: well the patch is much easier than I expected, so I'm fine with just solving this issue now08:02
SvenKieskesure, it's just renaming.. funny that my initial hunch was correct that the changed caching strategy broke this. caching is hard.08:03
SvenKieskethat being said, renaming is also hard :D08:03
mnasiadkawell, if we're persistent on kolla prefix, e.g. kolla_common_filters.py - sure08:04
SvenKieskeI don't really care much, might be nice :shrug:08:05
fricklerI agree with the kolla_ prefix. do we even need the *filters* part in the filename though? we have /filter_plugins/ in the path already08:06
fricklermeh, that isn't meant to be bold, just a search pattern, dumb irc client ;-P08:07
SvenKieskemy client doesn't print anything there bold :D08:10
SvenKieskejust plain text over here with good old "Konversation"08:10
mnasiadkaok, updated08:11
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Rename per role filters files  https://review.opendev.org/c/openstack/kolla-ansible/+/89879908:11
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Revert "CI: Pin ansible-core due to breakage"  https://review.opendev.org/c/openstack/kolla-ansible/+/89876708:20
fricklermnasiadka: when putting together my ptg timetable I noticed that the kolla sessions still have the default zoom link, could you please change that to meetpad?08:22
mnasiadkasure, let me check08:23
mnasiadkafrickler: remember we start in non ptg timetable time :)08:23
frickleroh, I sure would have misplanned that indeed :-D08:24
mnasiadkadone08:24
fricklerthx08:28
mhinerSvenKieske: regarding that python podman bug: limiting package version is tricky - Debian's osbpo seems to only have latest 4.7.0 version and Rocky9 has 4.4.1 (at least in the container form)09:05
mhinerI think it would be easier to go back to workaround solution for now, what do you think?09:05
SvenKieskeah in that case: sure09:08
mhineralright, thanks09:09
opendevreviewMartin Hiner proposed openstack/ansible-collection-kolla master: Add Podman support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/85224009:45
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Add support of podman deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/79922909:51
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Add support of podman deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/79922911:05
SvenKieskemhm, seems including tests in this changeset would introduce a dependency cycle: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/852240/comments/283f76e1_80442af7 so, should we do that later?11:30
frickleryes11:32
opendevreviewMerged openstack/kolla-ansible master: Rename per role filters files  https://review.opendev.org/c/openstack/kolla-ansible/+/89879913:06
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.1: Rename per role filters files  https://review.opendev.org/c/openstack/kolla-ansible/+/89877113:07
opendevreviewJake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server  https://review.opendev.org/c/openstack/kolla-ansible/+/89303113:07
kevkoalso going to check podman when you are discussing it :P 14:34
opendevreviewJake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server  https://review.opendev.org/c/openstack/kolla-ansible/+/89303114:57
opendevreviewJake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server  https://review.opendev.org/c/openstack/kolla-ansible/+/89303115:00
opendevreviewJake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server  https://review.opendev.org/c/openstack/kolla-ansible/+/89303115:05
opendevreviewMichal Arbet proposed openstack/ansible-collection-kolla master: Add Podman support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/85224015:48
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Rename per role filters files  https://review.opendev.org/c/openstack/kolla-ansible/+/89877116:28
opendevreviewMichal Arbet proposed openstack/ansible-collection-kolla master: Add Podman support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/85224017:24
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Add support of podman deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/79922918:59
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: [DNM] Just test something  https://review.opendev.org/c/openstack/kolla-ansible/+/89360919:03
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Add support of podman deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/79922919:08
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Add migration scenario from Docker to Podman (WIP)  https://review.opendev.org/c/openstack/kolla-ansible/+/83694119:09
kevkofrickler: can u approve podman ansible-collections patch ? 19:10

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