| mnasiadka | frickler: thanks for progressing the ansible-core bug | 07:14 |
|---|---|---|
| mnasiadka | frickler: I only meant that it seems cberendt has and it would be nice if that would land before we cut B release | 07:19 |
| SvenKieske | o/ | 07:19 |
| frickler | mnasiadka: 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 workaround | 07:26 |
| mnasiadka | frickler: it does help, let me raise a patch | 07:40 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Rename per role filters files https://review.opendev.org/c/openstack/kolla-ansible/+/898799 | 07:45 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Revert "CI: Pin ansible-core due to breakage" https://review.opendev.org/c/openstack/kolla-ansible/+/898767 | 07:46 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Revert "CI: Pin ansible-core due to breakage" https://review.opendev.org/c/openstack/kolla-ansible/+/898767 | 07:46 |
| frickler | oh, nice, I assumed there would be more filters | 08:01 |
| SvenKieske | mnasiadka: 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 |
| SvenKieske | maybe we should prefix all filter names with "kolla_"? :D | 08:01 |
| frickler | SvenKieske: well the patch is much easier than I expected, so I'm fine with just solving this issue now | 08:02 |
| SvenKieske | sure, it's just renaming.. funny that my initial hunch was correct that the changed caching strategy broke this. caching is hard. | 08:03 |
| SvenKieske | that being said, renaming is also hard :D | 08:03 |
| mnasiadka | well, if we're persistent on kolla prefix, e.g. kolla_common_filters.py - sure | 08:04 |
| SvenKieske | I don't really care much, might be nice :shrug: | 08:05 |
| frickler | I agree with the kolla_ prefix. do we even need the *filters* part in the filename though? we have /filter_plugins/ in the path already | 08:06 |
| frickler | meh, that isn't meant to be bold, just a search pattern, dumb irc client ;-P | 08:07 |
| SvenKieske | my client doesn't print anything there bold :D | 08:10 |
| SvenKieske | just plain text over here with good old "Konversation" | 08:10 |
| mnasiadka | ok, updated | 08:11 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Rename per role filters files https://review.opendev.org/c/openstack/kolla-ansible/+/898799 | 08:11 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Revert "CI: Pin ansible-core due to breakage" https://review.opendev.org/c/openstack/kolla-ansible/+/898767 | 08:20 |
| frickler | mnasiadka: 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 |
| mnasiadka | sure, let me check | 08:23 |
| mnasiadka | frickler: remember we start in non ptg timetable time :) | 08:23 |
| frickler | oh, I sure would have misplanned that indeed :-D | 08:24 |
| mnasiadka | done | 08:24 |
| frickler | thx | 08:28 |
| mhiner | SvenKieske: 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 |
| mhiner | I think it would be easier to go back to workaround solution for now, what do you think? | 09:05 |
| SvenKieske | ah in that case: sure | 09:08 |
| mhiner | alright, thanks | 09:09 |
| opendevreview | Martin Hiner proposed openstack/ansible-collection-kolla master: Add Podman support https://review.opendev.org/c/openstack/ansible-collection-kolla/+/852240 | 09:45 |
| opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Add support of podman deployment https://review.opendev.org/c/openstack/kolla-ansible/+/799229 | 09:51 |
| opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Add support of podman deployment https://review.opendev.org/c/openstack/kolla-ansible/+/799229 | 11:05 |
| SvenKieske | mhm, 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 |
| frickler | yes | 11:32 |
| opendevreview | Merged openstack/kolla-ansible master: Rename per role filters files https://review.opendev.org/c/openstack/kolla-ansible/+/898799 | 13:06 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2023.1: Rename per role filters files https://review.opendev.org/c/openstack/kolla-ansible/+/898771 | 13:07 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server https://review.opendev.org/c/openstack/kolla-ansible/+/893031 | 13:07 |
| kevko | also going to check podman when you are discussing it :P | 14:34 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server https://review.opendev.org/c/openstack/kolla-ansible/+/893031 | 14:57 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server https://review.opendev.org/c/openstack/kolla-ansible/+/893031 | 15:00 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server https://review.opendev.org/c/openstack/kolla-ansible/+/893031 | 15:05 |
| opendevreview | Michal Arbet proposed openstack/ansible-collection-kolla master: Add Podman support https://review.opendev.org/c/openstack/ansible-collection-kolla/+/852240 | 15:48 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Rename per role filters files https://review.opendev.org/c/openstack/kolla-ansible/+/898771 | 16:28 |
| opendevreview | Michal Arbet proposed openstack/ansible-collection-kolla master: Add Podman support https://review.opendev.org/c/openstack/ansible-collection-kolla/+/852240 | 17:24 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add support of podman deployment https://review.opendev.org/c/openstack/kolla-ansible/+/799229 | 18:59 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Just test something https://review.opendev.org/c/openstack/kolla-ansible/+/893609 | 19:03 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add support of podman deployment https://review.opendev.org/c/openstack/kolla-ansible/+/799229 | 19:08 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add migration scenario from Docker to Podman (WIP) https://review.opendev.org/c/openstack/kolla-ansible/+/836941 | 19:09 |
| kevko | frickler: 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/!