| opendevreview | Merged openstack/kolla-ansible master: CI: Test ovn without vpnaas https://review.opendev.org/c/openstack/kolla-ansible/+/995198 | 00:40 |
|---|---|---|
| kevko | blanson[m], mnasiadka: Guys, can we merge also this ? https://review.opendev.org/c/openstack/kolla-ansible/+/989850 | 06:25 |
| mnasiadka | Oh geez, can we stop making such huge patches? | 06:28 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Add kolla-ansible ovn-migration subcommand https://review.opendev.org/c/openstack/kolla-ansible/+/996096 | 06:36 |
| kevko | mnasiadka: why not ? its't not end of the cycle, it's tested, all comments addressed, part of the bigger chain and on the end it's same change but for every single config.json ... what's the problem ? | 06:38 |
| opendevreview | Mahmoud proposed openstack/kolla-ansible master: ceilometer: Copy custom meter definitions into notification agent https://review.opendev.org/c/openstack/kolla-ansible/+/996396 | 06:41 |
| opendevreview | Mahmoud proposed openstack/kolla-ansible master: prometheus: Fix Skyline blackbox endpoint checks https://review.opendev.org/c/openstack/kolla-ansible/+/996397 | 06:41 |
| opendevreview | Mahmoud proposed openstack/kolla-ansible master: Fix oslo.messaging qmanager collision between uWSGI APIs https://review.opendev.org/c/openstack/kolla-ansible/+/996398 | 06:42 |
| opendevreview | Mahmoud proposed openstack/kolla-ansible master: mariadb: Use CA certificate in recovery readiness check https://review.opendev.org/c/openstack/kolla-ansible/+/996399 | 06:42 |
| opendevreview | Mahmoud proposed openstack/kolla-ansible master: ceph-rgw: Move ceph_rgw_swift_compatibility to group_vars https://review.opendev.org/c/openstack/kolla-ansible/+/996400 | 06:42 |
| opendevreview | Mahmoud proposed openstack/kolla-ansible master: docs: Clarify config override merge vs replace semantics https://review.opendev.org/c/openstack/kolla-ansible/+/996401 | 06:42 |
| opendevreview | Mahmoud proposed openstack/kolla-ansible master: docs: Warn about matching RadosGW options with external Ceph https://review.opendev.org/c/openstack/kolla-ansible/+/996402 | 06:42 |
| kevko | +zuul passing ... | 06:49 |
| mnasiadka | kevko: last cycle we’ve done such things in a per service manner, so it’s easy to review and see impact, the fact that Zuul is passing doesn’t mean you’re not breaking anything we’re not testing or is tested in disabled or always failing scenarios | 07:13 |
| mnasiadka | Reviewing a patch that changes 2500+ lines in hundred files is not what I personally like to do | 07:15 |
| mikal | mnasiadka: speaking of which, I can't think of a way to split up https://review.opendev.org/c/openstack/kolla-ansible/+/976889 into smaller patches given its all one role, but I am open to suggestions it anyone has any. | 07:22 |
| mnasiadka | mikal: well, that is a new role, and just needs thorough review, I’m rather worried about changing small bits in every role we have that requires extra scrutiny about typos etc | 07:23 |
| mikal | mnasiadka: I think that's fair. I guess I'm more trying to say I don't feel "above the rules", but can't think of a way to not be a bit big here. | 07:23 |
| kevko | mnasiadka: Firstly I raised change in kolla to fix shared logdir and subdirectories owned by services in patch https://review.opendev.org/c/openstack/kolla/+/985300?tab=comments ... you have been the one who wrote me this >>> "I personally would prefer we don't introduce another script, especially with a wildcard in sudoers - a cleaner approach | 07:32 |
| kevko | would be to add support for doing this to kolla_set_configs and manage these permissions in kolla-ansible config.json (or future config.yaml)" in that patch | 07:32 |
| kevko | mnasiadka: So you're saying I should split a single cohesive change with one clear purpose into 48 separate patches, one per service? Am I understanding that correctly? And you won't then tell me I'm wasting Zuul CPU time? No problem ...but don't think it is a right way | 07:34 |
| kevko | mnasiadka: gerrit has button to "mark reviewed" | 07:35 |
| opendevreview | Michael Still proposed openstack/kolla-ansible stable/2025.1: Replace periodic-stable-jobs with local job list. https://review.opendev.org/c/openstack/kolla-ansible/+/996412 | 07:45 |
| mikal | ^--- this change attempts to address this problem: https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-py39&project=openstack%2Fkolla-ansible&branch=stable%2F2025.1&pipeline=periodic-stable | 07:46 |
| opendevreview | Michael Still proposed openstack/kolla-ansible stable/2025.2: Replace periodic-stable-jobs with local job list. https://review.opendev.org/c/openstack/kolla-ansible/+/996414 | 07:49 |
| mikal | ^--- and this one is similar but for stable/2025.2: https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-py310&project=openstack%2Fkolla-ansible&branch=stable%2F2025.2&pipeline=periodic-stable | 07:50 |
| opendevreview | Alma MAILLET-CONTOZ proposed openstack/kolla-ansible stable/2026.1: Fix typo in neutron group vars https://review.opendev.org/c/openstack/kolla-ansible/+/996284 | 07:51 |
| opendevreview | Michael Still proposed openstack/kolla-ansible stable/2026.1: Replace periodic-stable-jobs with local job list. https://review.opendev.org/c/openstack/kolla-ansible/+/996415 | 07:52 |
| mikal | ^--- and then finally 2026.1: https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-py310&project=openstack%2Fkolla-ansible&branch=stable%2F2026.1&pipeline=periodic-stable | 07:52 |
| opendevreview | Alma MAILLET-CONTOZ proposed openstack/kolla-ansible stable/2025.2: Fix typo in neutron group vars https://review.opendev.org/c/openstack/kolla-ansible/+/996285 | 07:53 |
| mnasiadka | kevko: Thanks, I’ll let the other cores review it. | 07:53 |
| kevko | mnasiadka: okay, thank you also | 07:56 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Add kolla-ansible ovn-migration subcommand https://review.opendev.org/c/openstack/kolla-ansible/+/996096 | 07:59 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Explicitly load VPNaaS config in neutron services https://review.opendev.org/c/openstack/kolla-ansible/+/996216 | 08:06 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Add Selenium readiness check and session retry https://review.opendev.org/c/openstack/kolla-ansible/+/996419 | 08:12 |
| opendevreview | Merged openstack/kolla-ansible master: mariadb: Increase healthcheck start_period to 120s https://review.opendev.org/c/openstack/kolla-ansible/+/996326 | 08:38 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Add kolla-ansible ovn-migration subcommand https://review.opendev.org/c/openstack/kolla-ansible/+/996096 | 08:53 |
| kevko | bugfix > https://review.opendev.org/c/openstack/kolla-ansible/+/996216 < | 08:55 |
| kevko | mnasiadka: thanks | 08:59 |
| kevko | fwaas has same problem ...will send in hours ..maybe tomorrow | 09:00 |
| mnasiadka | Well actually, API was running without ml2_conf.ini and it was fine, are you sure we need it in API as well? | 09:00 |
| *** jhorstmann is now known as Guest12905 | 09:04 | |
| kevko | mnasiadka: I am hundred percent sure | 09:17 |
| mnasiadka | Ok, so let’s not use pyargv then and I’m fine :) | 09:17 |
| kevko | what ? | 09:18 |
| kevko | mnasiadka: if there is not defined and neutron is runned as neutron.wsgi.api:application ...from code it's visible that neutron.conf and /etc/neutron/plugins/ml2/ml2_conf.ini is read -> https://github.com/openstack/neutron/blob/04f6410f49bf85f7b63a156683eb0e69dd8a90f6/neutron/server/__init__.py#L30-L32 <- here | 09:20 |
| mnasiadka | kevko: I commented in the patch, don’t use pyargv, use the OS_NEUTRON_CONFIG_FILES env var | 09:21 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Add Selenium readiness check and session retry https://review.opendev.org/c/openstack/kolla-ansible/+/996419 | 09:23 |
| kevko | mnasiadka: what's wrong with pyargv ? I know OS_NEUTRON_CONFIG_FILES is an option ... but why it's better ? | 09:23 |
| mnasiadka | Because pyargv is not an option in other wsgi servers like gunicorn or granian which we might add support for in near future | 09:24 |
| kevko | mnasiadka: okay, good point ... | 09:26 |
| kevko | mnasiadka: It's just that I've seen a few cases in our kolla compare scripts where env wasn't compared correctly....or things broke because of nested kolla-toolbox module and similar issues.... I'm just not comfortable relying on it ....file file on disk is just a file on disk ..... but, as I said, I can't really argue with your point .... it's a | 09:28 |
| kevko | good one. | 09:28 |
| kevko | mnasiadka: aaaa, okay ..env can be defined in ini file :) ...coool | 09:29 |
| kevko | mnasiadka: okay, should I change a patch for pyargv option and replace by env ? or send a new ? with env ...so we will have both ? as a option | 09:33 |
| kevko | i mean this one | 09:33 |
| kevko | https://review.opendev.org/c/openstack/kolla-ansible/+/996214 | 09:33 |
| mnasiadka | I think if we don’t want to use pyargv, then just rework it to use env? | 09:33 |
| kevko | ok | 09:35 |
| blanson[m] | mnasiadka: https://review.opendev.org/c/openstack/kolla/+/996091 this is fine by me, should we +w it ? | 09:39 |
| mnasiadka | blanson[m]: fire in the hole | 09:45 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add env option service-uwsgi-config role https://review.opendev.org/c/openstack/kolla-ansible/+/996214 | 09:46 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Explicitly load VPNaaS config in neutron services https://review.opendev.org/c/openstack/kolla-ansible/+/996216 | 09:46 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add env option service-uwsgi-config role https://review.opendev.org/c/openstack/kolla-ansible/+/996214 | 09:48 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Explicitly load VPNaaS config in neutron services https://review.opendev.org/c/openstack/kolla-ansible/+/996216 | 09:48 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Explicitly load VPNaaS config in neutron services https://review.opendev.org/c/openstack/kolla-ansible/+/996216 | 09:59 |
| opendevreview | Sean Butler-Lee proposed openstack/kolla-ansible master: Add support for configuring Skyline to use Keystone public URL, c.f. existing support in Horizon https://review.opendev.org/c/openstack/kolla-ansible/+/996434 | 10:02 |
| opendevreview | Piotr Milewski proposed openstack/kolla master: rabbitmq: Update to 4.3 https://review.opendev.org/c/openstack/kolla/+/988425 | 10:09 |
| opendevreview | Sean Butler-Lee proposed openstack/kolla-ansible master: Add support for configuring Skyline to use Keystone public URL https://review.opendev.org/c/openstack/kolla-ansible/+/996434 | 10:11 |
| opendevreview | Verification of a change to openstack/kolla master failed: Migrate to pyproject.toml https://review.opendev.org/c/openstack/kolla/+/996091 | 11:05 |
| frickler | hmm, is my perception skewed or is it really rocky causing most of the CI failures currently? | 11:19 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Add kolla-ansible ovn-migration subcommand https://review.opendev.org/c/openstack/kolla-ansible/+/996096 | 11:20 |
| kevko | frickler: i had same feeling :P | 11:24 |
| mnasiadka | I would say podman is the problem | 11:25 |
| salmankh | I have a couple trivial commits, can they be reviewed. Contributing first time to this repo, not sure who to ask or the process. 1. https://review.opendev.org/c/openstack/kolla-ansible/+/996139 2. https://review.opendev.org/c/openstack/kolla-ansible/+/996170 | 11:32 |
| mnasiadka | salmankh: reviewed both | 11:39 |
| mnasiadka | If any other core wants to have a look at 996170 then it can be merged | 11:39 |
| mnasiadka | salmankh: the ,,process’’ is sort of defined in https://docs.openstack.org/kolla-ansible/latest/contributor/index.html - if there’s anything missing anybody can add something there | 11:40 |
| salmankh | thanks for the reviews, will have a look at the guide. | 11:42 |
| blanson[m] | +2 +w 996170, that's a nice fiux thank you | 11:43 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: podman: Fix parse_volumes ignoring rslave propagation in compound modes https://review.opendev.org/c/openstack/kolla-ansible/+/996442 | 11:44 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Switch neutron_plugin_agent default to ovn https://review.opendev.org/c/openstack/kolla-ansible/+/970114 | 11:44 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Switch neutron_plugin_agent default to ovn https://review.opendev.org/c/openstack/kolla-ansible/+/970114 | 11:45 |
| blanson[m] | I love how the podman worker is slowly turning into an alternative api for podman because everything behaves weirdly for it | 11:47 |
| blanson[m] | pass a list of volumes ? nah let's parse every flag because otherwise the podman api will be mad at us | 11:48 |
| mnasiadka | Don’t get me started | 11:48 |
| mnasiadka | :D | 11:48 |
| mnasiadka | And I love that they change things from version to version, because why not :) | 11:48 |
| blanson[m] | it truly is a special child | 11:49 |
| blanson[m] | welp, ast least the rpc worker fix seems to have somewhat resolved our ci issue | 11:51 |
| blanson[m] | https://review.opendev.org/c/openstack/kolla-ansible/+/996280 | 11:52 |
| blanson[m] | I've had 0 failures in like 20 runs of the podman gate job | 11:52 |
| blanson[m] | I'll keep spamming it for a little bit | 11:52 |
| blanson[m] | but I think that's the fix | 11:52 |
| mnasiadka | Wonder if bumping to the same value as the openstack_service_workers would be enough | 11:53 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Drop DHCP agent support in ML2/OVN deployments https://review.opendev.org/c/openstack/kolla-ansible/+/992881 | 11:53 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Drop DHCP agent support in ML2/OVN deployments https://review.opendev.org/c/openstack/kolla-ansible/+/992881 | 11:55 |
| mnasiadka | For the people attending the meeting in an hour - I would like to gather feedback on https://review.opendev.org/c/openstack/kolla/+/995025 | 11:57 |
| opendevreview | Owen Jones proposed openstack/kolla-ansible master: Fix nova-compute startup after reprovisioning https://review.opendev.org/c/openstack/kolla-ansible/+/936785 | 12:12 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Add kolla-ansible ovn-migration subcommand https://review.opendev.org/c/openstack/kolla-ansible/+/996096 | 12:39 |
| mnasiadka | frickler kevko mmalchuk gkoper jovial mattcrees dougszu darmach pabloclsn ravlew salmankh amir58118 r-krcek blanson[m] - meeting in 8 minutes (reminder) | 12:52 |
| mnasiadka | #startmeeting kolla | 13:00 |
| opendevmeet | Meeting started Wed Jul 8 13:00:12 2026 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
| opendevmeet | The meeting name has been set to 'kolla' | 13:00 |
| mnasiadka | #topic rollcall | 13:00 |
| mnasiadka | o/ | 13:00 |
| salmankh | o/ | 13:00 |
| owenjones | o/ | 13:00 |
| ravlew | o/ | 13:01 |
| blanson[m] | hello | 13:01 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: neutron: Remove legacy upgrade logic https://review.opendev.org/c/openstack/kolla-ansible/+/992880 | 13:03 |
| AlmaMC[m] | o/ | 13:04 |
| mnasiadka | #topic agenda | 13:05 |
| mnasiadka | * Announcements | 13:05 |
| mnasiadka | * CI status | 13:05 |
| mnasiadka | * Release tasks | 13:05 |
| mnasiadka | * Current cycle planning | 13:05 |
| mnasiadka | * Additional agenda (from whiteboard) | 13:05 |
| mnasiadka | * Open discussion | 13:05 |
| mnasiadka | #topic Announcements | 13:05 |
| mnasiadka | I signed up Kolla for one session on the upcoming PTG - that is Operator Hour (or however we want to call that) | 13:06 |
| mnasiadka | The next elections are coming up at around end of August - beginning of Sep - if anyone wants to relive me from my duty ;-) | 13:07 |
| mnasiadka | #topic CI status | 13:07 |
| mnasiadka | Failures here and there, today it seems that either OpenDev mirrors or pypi has some problems | 13:07 |
| mnasiadka | I would rather bet for the latter, because that’s in multiple clouds | 13:07 |
| * frickler lost track of time, sorry. \o | 13:07 | |
| mnasiadka | (As in Zuul provider clouds) | 13:07 |
| mnasiadka | We merged some mariadb improvements that should make jobs better, but Kolla CI needs love 24 hours a day | 13:08 |
| mnasiadka | #topic Release tasks | 13:09 |
| mnasiadka | It’s R-12 | 13:10 |
| mnasiadka | Nothing on the Kolla release calendar | 13:10 |
| mnasiadka | But I think we should start wrapping up to bump Ansible and other infrastructure services like RMQ and maybe MariaDB | 13:10 |
| mnasiadka | Anybody wants to volunteer to create a list of things that need working on? Maybe on the KollaWhiteboard? | 13:11 |
| blanson[m] | as in bump versions of ? | 13:11 |
| Vii | rabbitmq has a bug in the upgrade to the new version | 13:12 |
| tafkamax | Vii: noooo :((( | 13:12 |
| Vii | link: https://github.com/rabbitmq/rabbitmq-server/discussions/16823 | 13:12 |
| Vii | Fixes #16824 addresses this exception and will ship in 4.3.3. | 13:13 |
| mnasiadka | blanson[m]: mariadb, rabbitmq, ansible version supported, and anything else that needs bumping that is not an OpenStack service | 13:13 |
| blanson[m] | I can work on a list | 13:14 |
| Vii | We need to wait for the new version. | 13:14 |
| mnasiadka | blanson[m]: great, thanks | 13:14 |
| mnasiadka | #topic Current cycle planning | 13:14 |
| blanson[m] | Vii: there is enough work without even thinking asbout rabbitmq for the next few weeks, we should be fine | 13:15 |
| mnasiadka | So, as mentioned before the meeting - I’d like to understand if everyone is happy with merging frickler’s AI policy patch | 13:15 |
| mnasiadka | #link https://review.opendev.org/c/openstack/kolla/+/995025 | 13:15 |
| AlmaMC[m] | As I said, I think that this is a good think to have | 13:16 |
| blanson[m] | this seems reasonable to me. This is pretty standard AI policy to avoid getting swamped by LLM patches right ? as in: we're not asking anything extraordinary ? | 13:16 |
| mnasiadka | I’m thinking to do a followup to this with CLAUDE.md and AGENTS.md that would feed the same to people using LLMs | 13:19 |
| mnasiadka | blanson[m]: I think we’re not asking anything extraordinary ;-) | 13:20 |
| mnasiadka | frickler: fine with merging that as is? | 13:20 |
| frickler | fine with me | 13:20 |
| salmankh | just worried by discouraging use of AI, will it not slow down some development or fixes. | 13:20 |
| salmankh | some smart projects using AI extensively may take over. | 13:21 |
| frickler | quite the opposite IMHO | 13:21 |
| AlmaMC[m] | it explains that you need to fully understand what you're doing. So I don't think so. | 13:21 |
| mnasiadka | Ok then, let me merge that. | 13:23 |
| salmankh | ofcourse there would be humans reviewing the code and checking before merge, but discouraging it I may not like. But in general agree to limit it somehow. | 13:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Add CLAUDE.md and AGENTS.md fulfilling established AI policy https://review.opendev.org/c/openstack/kolla/+/996448 | 13:24 |
| frickler | salmankh: that's exactly the issue. noone want to review code where one cannot assume that it has been written with good intentions and some basic understanding of what's going on | 13:25 |
| frickler | mnasiadka: do we need to backport the change? I would copy it for all other repos I guess? | 13:26 |
| mnasiadka | frickler: that would make sense I guess | 13:27 |
| mnasiadka | Ok then, let’s move on | 13:28 |
| opendevreview | Owen Jones proposed openstack/kolla-ansible master: Fix nova-compute startup after reprovisioning https://review.opendev.org/c/openstack/kolla-ansible/+/936785 | 13:28 |
| mnasiadka | Anyone have any current cycle topic? | 13:28 |
| Vii | #link https://etherpad.opendev.org/p/KollaWhiteBoard#L68 | 13:30 |
| dougszu | Improved baremetal testing: Jed has Sushy working in Tenks here: https://review.opendev.org/c/openstack/kolla-ansible/+/995952 | 13:30 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Add AI/LLM assistant guide documents https://review.opendev.org/c/openstack/kolla/+/996448 | 13:30 |
| dougszu | Any ideas for extension in future patches welcome | 13:30 |
| Vii | Maybe we can move some of these forward with merging/review | 13:30 |
| mnasiadka | Ok then, I’ll have a look into the list from Vii later this week | 13:32 |
| Vii | ok ty | 13:32 |
| mnasiadka | And good work Jed and dougszu | 13:32 |
| blanson[m] | also | 13:33 |
| blanson[m] | #link https://review.opendev.org/c/openstack/kolla-ansible/+/988882 | 13:34 |
| blanson[m] | :) | 13:34 |
| dougszu | thanks for updating, I will take another look | 13:34 |
| mnasiadka | Ok then, seems we have covered the additional agenda from Vii | 13:35 |
| mnasiadka | Let’s move into open discussion | 13:35 |
| mnasiadka | #topic Open discussion | 13:35 |
| mnasiadka | Anybody has any additional topics? | 13:35 |
| mnasiadka | Seems not | 13:40 |
| mnasiadka | Reviews welcome on the AI/LLM assistant guides | 13:40 |
| mnasiadka | #link https://review.opendev.org/c/openstack/kolla/+/996448 | 13:40 |
| mnasiadka | And see you next week :) | 13:40 |
| mnasiadka | #endmeeting | 13:40 |
| opendevmeet | Meeting ended Wed Jul 8 13:40:34 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:40 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2026/kolla.2026-07-08-13.00.html | 13:40 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2026/kolla.2026-07-08-13.00.txt | 13:40 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2026/kolla.2026-07-08-13.00.log.html | 13:40 |
| opendevreview | Merged openstack/kolla-ansible stable/2025.2: Fix typo in neutron group vars https://review.opendev.org/c/openstack/kolla-ansible/+/996285 | 13:49 |
| opendevreview | Owen Jones proposed openstack/kolla-ansible master: Fix nova-compute startup after reprovisioning https://review.opendev.org/c/openstack/kolla-ansible/+/936785 | 13:50 |
| opendevreview | Owen Jones proposed openstack/kolla-ansible master: Fix nova-compute startup after reprovisioning https://review.opendev.org/c/openstack/kolla-ansible/+/936785 | 13:52 |
| opendevreview | Merged openstack/kolla master: Adopt an AI policy https://review.opendev.org/c/openstack/kolla/+/995025 | 13:55 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Fix RabbitMQ restart in check mode https://review.opendev.org/c/openstack/kolla-ansible/+/996170 | 14:06 |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/2026.1 failed: Fix typo in neutron group vars https://review.opendev.org/c/openstack/kolla-ansible/+/996284 | 14:08 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Add kolla-ansible ovn-migration subcommand https://review.opendev.org/c/openstack/kolla-ansible/+/996096 | 14:31 |
| opendevreview | Verification of a change to openstack/kolla master failed: Migrate to pyproject.toml https://review.opendev.org/c/openstack/kolla/+/996091 | 14:39 |
| kevko | dougszu: here ? | 14:54 |
| opendevreview | Owen Jones proposed openstack/kolla-ansible master: Fix nova-compute startup after reprovisioning https://review.opendev.org/c/openstack/kolla-ansible/+/936785 | 15:03 |
| dougszu | kevko, i'm here | 15:07 |
| opendevreview | Salman proposed openstack/kolla-ansible master: Fix RabbitMQ restart in check mode https://review.opendev.org/c/openstack/kolla-ansible/+/996170 | 15:09 |
| dougszu | ok, I see your kolla patch now, I hadn't found: https://review.opendev.org/c/openstack/kolla/+/989849 (it's not a dependency) | 15:09 |
| dougszu | Apologies, I hadn't seen all the discussion. | 15:13 |
| kevko | dougszu: so do you understand now ? | 15:27 |
| kevko | dougszu: it's cross dependency problem ..so after k-a piece will be merged ..kolla patch will start pass | 15:27 |
| kevko | dougszu: Right now they simply overlap ... k-a is applied after the image code, so even if something is wrong in the image,k-a ultimately corrects it. Once the Kolla part is removed, only the k-a mplementation remains. | 15:29 |
| opendevreview | Owen Jones proposed openstack/kolla-ansible master: Fix nova-compute startup after reprovisioning https://review.opendev.org/c/openstack/kolla-ansible/+/936785 | 16:18 |
| dougszu | kevko: yeah, I do, I was mostly wondering where the Kolla patch was to modify the scripts, since I didn't want to +2 the KA patch without having it ready. | 16:21 |
| dougszu | the KA patch can still depend on the Kolla patch though right? It would be nice to see them tested together | 16:23 |
| dougszu | You can temporarily disable the KA CI in the Kolla patch | 16:23 |
| dougszu | and then revert immediately once it merges | 16:24 |
| opendevreview | Owen Jones proposed openstack/kolla-ansible master: Fix nova-compute startup after reprovisioning https://review.opendev.org/c/openstack/kolla-ansible/+/936785 | 16:24 |
| opendevreview | Merged openstack/kolla master: Migrate to pyproject.toml https://review.opendev.org/c/openstack/kolla/+/996091 | 16:33 |
| kevko | dougszu: you mean add to kolla patch dependency to kolla-ansible patch ? | 16:36 |
| kevko | dougszu: or vice versa ? | 16:36 |
| dougszu | I was thinking the KA patch since there are more relevant tests | 16:37 |
| kevko | dougszu: okay, maybe you will get it ... let me try | 16:38 |
| dougszu | thanks, I will find the strength to read the last 15 service files | 16:38 |
| dougszu | (tomorrow now) | 16:39 |
| kevko | dougszu: last time please ... | 16:39 |
| kevko | dougszu: what depend to where ? | 16:39 |
| opendevreview | Doug Szumski proposed openstack/kolla-ansible master: Centralize service log directory management https://review.opendev.org/c/openstack/kolla-ansible/+/989850 | 16:40 |
| dougszu | I've added it (others are merged) | 16:40 |
| dougszu | Do you want to fix merge conflicts on the Kolla patch? | 16:40 |
| kevko | dougszu: yeah, i can | 16:41 |
| opendevreview | Doug Szumski proposed openstack/kolla master: Remove creating logdirs in docker images https://review.opendev.org/c/openstack/kolla/+/989849 | 16:42 |
| dougszu | thanks, I will look at them both tomorrow | 16:42 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Remove creating logdirs in docker images https://review.opendev.org/c/openstack/kolla/+/989849 | 16:43 |
| kevko | dougszu: But you're aware that the Kolla patch will simply fail, right? Because the Kolla-Ansible part hasn't been merged yet. During CI/testing it just checks out the current master of Kolla-Ansible and runs with that. Since the image changes have already removed those bits everywhere (kolla-patch), but the Kolla-Ansible changes aren't merged yet, | 16:48 |
| kevko | everything just falls over. Having a Needed-By relationship doesn't save you here it's just metadata keyword not actually working ... 🙂 Did you get what I mean? | 16:48 |
| dougszu | kevko, yeah, added just for the link. I would like to see the KA CI pass using the Kolla patch. | 16:55 |
| dougszu | I don't mind what has to happen to merge both after that | 16:55 |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/2026.1 failed: Fix typo in neutron group vars https://review.opendev.org/c/openstack/kolla-ansible/+/996284 | 19:08 |
| mikal | I am not sure why this review has two +2's but hasn't been merged? https://review.opendev.org/c/openstack/kolla-ansible/+/989612 | 19:18 |
| mnasiadka | mikal: do you think that should be backported? | 19:31 |
| mnasiadka | Ah ok, has BC +1 | 19:31 |
| mnasiadka | Small enough that probably doesn’t need bug or reno | 19:31 |
| mikal | mnasiadka: I think it would be nice, but not absolutely necessary. That is, it would be a quality of life improvement for deployers, but it isn't a critical feature or a security risk. | 19:32 |
| mikal | So I think whether or not we bothered would depend on workload -- what would we be deferring in order to make time to backport it? | 19:32 |
| mikal | Also, thanks for the +W. | 19:33 |
| mnasiadka | mikal: usually we require a bug linked or a release note for backporting, but this is so minimal that we can live without that | 19:33 |
| mikal | Ok. I am happy to give doing the backports a go, but noting a made a mess of it last time because its not something I've done much. Is there a cheat sheet or something I should follow? | 19:33 |
| mnasiadka | mikal: I usually just use Gerrit UI for backports, that patch shouldn’t give us any merge conflicts. The tricky part is if the k-a patch needs a new kolla image version, then we need to add some safeguards (like you did when asked last time) | 19:38 |
| mikal | mnasiadka: Huh, I didn't even know you could use the UI. Where is it in that? | 19:39 |
| mnasiadka | If you do that from CLI - just remember doing -x (so there’s master commit id trace jn commit message) | 19:39 |
| mnasiadka | mikal: three dots on the top right corner and click cherry pick | 19:39 |
| opendevreview | Michael Still proposed openstack/kolla-ansible stable/2026.1: Make the MariaDB WSREP sync wait defensive. https://review.opendev.org/c/openstack/kolla-ansible/+/996524 | 19:40 |
| mikal | I did that, but I think maybe I am meant to add the "(cherry picked from ...)" manually myself? | 19:41 |
| kevko | mnasiadka: linters failing for https://review.opendev.org/c/openstack/kolla-ansible/+/989612 now :D | 20:03 |
| kevko | mnasiadka: check zuul filrer | 20:03 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Make the MariaDB WSREP sync wait defensive. https://review.opendev.org/c/openstack/kolla-ansible/+/989612 | 20:04 |
| mnasiadka | kevko: it just needs a rebase | 20:06 |
| opendevreview | Michael Still proposed openstack/kolla-ansible master: Make the MariaDB WSREP sync wait defensive. https://review.opendev.org/c/openstack/kolla-ansible/+/989612 | 20:06 |
| mnasiadka | mikal: if you cherry pick a merged change then it gets added automatically | 20:06 |
| mikal | Oh, ok. I will abandon that one and do it again when the merge has happened. | 20:07 |
| mnasiadka | mikal: leave it open and just redo the chery pick when it’s merged - it will get updated automatically | 20:09 |
| mnasiadka | And if you abandon - it will complain it’s closed and you will need to restore it first | 20:10 |
| mikal | Right. An unabandon it is then! | 20:10 |
| mikal | Sorry to be a n00b. | 20:11 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Add kolla-ansible ovn-migration subcommand https://review.opendev.org/c/openstack/kolla-ansible/+/996096 | 20:14 |
| opendevreview | Kurt Bendl proposed openstack/kolla-ansible master: DNM: selinux security_opt and policy generation https://review.opendev.org/c/openstack/kolla-ansible/+/986399 | 21:11 |
| opendevreview | Kurt Bendl proposed openstack/kolla-ansible master: DNM: selinux security_opt and policy generation https://review.opendev.org/c/openstack/kolla-ansible/+/986399 | 21:25 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!