| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Move globals to zuul definitions https://review.opendev.org/c/openstack/kolla-ansible/+/971257 | 05:57 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Move globals to zuul definitions https://review.opendev.org/c/openstack/kolla-ansible/+/971257 | 07:03 |
| *** mrunge_ is now known as mrunge | 07:14 | |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Move globals to zuul definitions https://review.opendev.org/c/openstack/kolla-ansible/+/971257 | 07:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Move globals to zuul definitions https://review.opendev.org/c/openstack/kolla-ansible/+/971257 | 07:30 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: tempest: Rework regex to a list https://review.opendev.org/c/openstack/kolla-ansible/+/971256 | 07:35 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: ansible-lint: Fix fqcn[actions-core] for barbican https://review.opendev.org/c/openstack/kolla-ansible/+/971269 | 08:06 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: ansible-lint: Fix fqcn[actions-core] for bifrost https://review.opendev.org/c/openstack/kolla-ansible/+/971270 | 08:09 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Move globals to zuul definitions https://review.opendev.org/c/openstack/kolla-ansible/+/971257 | 08:10 |
| frickler | mnasiadka: I tend to disagree with the actions-core changes, these are too verbose IMO | 08:12 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: ansible-lint: Fix fqcn[actions-core] for blazar https://review.opendev.org/c/openstack/kolla-ansible/+/971271 | 08:13 |
| mnasiadka | frickler: well, that’s the best practice, but I’m fine with leaving fqcn’s in whitelist | 08:15 |
| bbezak | blanson[m]: no, sorry. Too much customer’s related on my plate recently. I’ll try to review it this week. Thx for reminder | 08:17 |
| blanson[m] | bbezak: yh I'm sorry if I'm to insistant about this one but it seems very important for podman users :/ | 08:20 |
| frickler | mnasiadka: ok, I retract my objection, just found that we're doing the same downstream. will review later | 08:38 |
| mnasiadka | frickler: I abandoned these for now, but will restore after reworking certificates role to Ansible for Trixie Kolla-Ansible patch | 08:41 |
| frickler | ah, ok, sorry for creation too much noise on a bleak morning | 08:42 |
| mnasiadka | Nah, no problem :) | 08:48 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework generate-root to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 09:05 |
| *** jhorstmann is now known as Guest33781 | 09:12 | |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework generate-root to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 12:14 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Remove br_netfilter loading https://review.opendev.org/c/openstack/kolla-ansible/+/971317 | 12:33 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework generate-root to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 12:43 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: [DNM] Test removal of module load https://review.opendev.org/c/openstack/kayobe/+/971318 | 12:44 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework generate-root to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 13:17 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework generate-root to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 13:28 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework generate-root to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 13:47 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework generate-root to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 13:48 |
| mnasiadka | bbezak frickler kevko mmalchuk gkoper jovial mattcrees dougszu darmach pabloclsn ravlew amir58118 r-krcek blanson[m] - courtesy reminder that we have a meeting here in 9 minutes | 13:51 |
| mnasiadka | #startmeeting kolla | 14:01 |
| opendevmeet | Meeting started Wed Dec 17 14:01:25 2025 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'kolla' | 14:01 |
| mnasiadka | #topic rollcall | 14:01 |
| mnasiadka | \o | 14:01 |
| blanson[m] | Hello | 14:01 |
| mnasiadka | Two person meeting | 14:07 |
| mnasiadka | I think we’ll pass :) | 14:07 |
| blanson[m] | yh... | 14:07 |
| mnasiadka | blanson[m]: I reviewed all your podman patches, also the keystone credentials one | 14:08 |
| blanson[m] | oooooh | 14:08 |
| mnasiadka | Basically I went through everything that is in whiteboard | 14:08 |
| blanson[m] | what do you think ? | 14:08 |
| mnasiadka | So you need a second core for the podman stuff | 14:08 |
| mnasiadka | Looks reasonable, I don’t use podman | 14:08 |
| mnasiadka | But whatever makes it better ;) | 14:08 |
| blanson[m] | podman stuff I'm fairly confident I spent like 3 weeks tracking the weird stuff in here | 14:08 |
| mnasiadka | So let’s skip the meeting today, and actually we have the next one 8th of Jan :) | 14:09 |
| mnasiadka | #endmeeting | 14:09 |
| opendevmeet | Meeting ended Wed Dec 17 14:09:14 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:09 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-12-17-14.01.html | 14:09 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-12-17-14.01.txt | 14:09 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-12-17-14.01.log.html | 14:09 |
| blanson[m] | the keystone credential encryption one is newer but doesn't seem like rocket sience to me | 14:09 |
| blanson[m] | the only thing I'm wondering is about rotation for the credential keys | 14:10 |
| blanson[m] | cause the docs doesn't really give a lot of info about it, so idk. I have a draft for rotating credentials (automatically if needs be), but idk if that's really necessary | 14:11 |
| mnasiadka | Well, I’d say the fernet rotation script isn’t my favourite | 14:12 |
| mnasiadka | So having another one in that form isn’t my favourite as well | 14:12 |
| blanson[m] | should I make it a single python script for both ? | 14:13 |
| blanson[m] | with like an argument --credential or --fernet os smthg ? | 14:13 |
| mnasiadka | That would probably be better, and if we could just template a config file instead of generating the whole script in K-A would probably make it more readable | 14:14 |
| mnasiadka | The other thing is how it’s scheduled today using cron, but I guess that’s another thing | 14:14 |
| blanson[m] | moving rotation to python is gonna be another patch entirely I think tho | 14:15 |
| blanson[m] | doesn't really fit with the "simple" addition of credential-keys | 14:16 |
| mnasiadka | Yeah, I think we could go with that, so it’s backportable | 14:16 |
| mnasiadka | And then do a refactor in a followup | 14:16 |
| blanson[m] | cool, so I'll leave the credential one as-is | 14:17 |
| blanson[m] | then make a subsequent patch for the whole rotation situation | 14:17 |
| blanson[m] | I've also flooded you with ansible-lint patches | 14:17 |
| blanson[m] | (sorry) | 14:17 |
| mnasiadka | no problem, will have a look tomorrow | 14:18 |
| fprzewozn | mnasiadka: thanks for the reviews, what about workload votes? | 14:19 |
| mnasiadka | fprzewozn: If you’re asking about workflow votes - then a second core reviewer is needed, you need to nicely ask bbezak or frickler to have a look :) | 14:22 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework generate-root to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 14:38 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Revert "CI: Use centos:stream10 image for molecule" https://review.opendev.org/c/openstack/kayobe/+/971329 | 15:00 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 15:16 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 15:17 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 15:20 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add support for Debian Trixie (13) https://review.opendev.org/c/openstack/kolla-ansible/+/969940 | 15:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: tempest: Rework regex to a list https://review.opendev.org/c/openstack/kolla-ansible/+/971256 | 15:26 |
| opendevreview | Merged openstack/kolla stable/2025.1: [2025.1 only] Fix Gnocchi memory leak caused by numpy==2.2.3 https://review.opendev.org/c/openstack/kolla/+/960371 | 15:59 |
| blanson[m] | mnasiadka: regarding your review in the keystone credential patch, are you asking why we're doing a separate task and script rather than using the current fernet one to do everything at once ? | 17:40 |
| mnasiadka | blanson[m]: first of all I lack some background in the commit message :-) | 17:57 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 18:09 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 19:17 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 19:22 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 19:45 |
| blanson[m] | mnasiadka: oh yes, that's a friday evening commit I'll add some context lol | 20:04 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 20:34 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 20:43 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 21:02 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: certificates: rework to use Ansible modules https://review.opendev.org/c/openstack/kolla-ansible/+/970099 | 21:14 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!