| opendevreview | Merged openstack/kayobe stable/2024.1: Fix Depends-On https://review.opendev.org/c/openstack/kayobe/+/940230 | 01:00 |
|---|---|---|
| opendevreview | Amith Kotu proposed openstack/kolla-ansible master: fix(docs): Correct typo in variable name from "neutron_physical_network" to "neutron_physical_networks" https://review.opendev.org/c/openstack/kolla-ansible/+/942163 | 03:14 |
| frickler | mnasiadka: can you please hold your W+1 for ^^? | 05:59 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: fix(docs): Correct typo in variable name from "neutron_physical_network" to "neutron_physical_networks" https://review.opendev.org/c/openstack/kolla-ansible/+/942163 | 06:01 |
| mnasiadka | frickler: done - I haven't looked at the commit message in depth - duh | 06:01 |
| mnasiadka | Is there an option pep8 would do those checks for us? | 06:01 |
| mnasiadka | as in line length :) | 06:02 |
| frickler | mnasiadka: thx, np, normally I'd have agreed with you in simply approving, but I see this is a training opportunity | 06:02 |
| frickler | not sure about pep8, I can try to find something | 06:02 |
| frickler | but likely I'd also prefer not to make it a hard rule, just try and teach people some kind of common style ;) | 06:03 |
| mnasiadka | Well, line length probably would not be that bad, we sort of do that with the code and docs | 06:18 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: Add gitlint to linters and make this commit message extra weird to show it working https://review.opendev.org/c/openstack/kolla-ansible/+/942168 | 06:33 |
| opendevreview | Matt Anson proposed openstack/kolla-ansible master: DNM: Test overriding Prometheus scrape target https://review.opendev.org/c/openstack/kolla-ansible/+/941581 | 08:02 |
| opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: Fix for haproxy not copying external-frontend-map https://review.opendev.org/c/openstack/kolla-ansible/+/942121 | 08:33 |
| Vii | hello, if you are already fixing 'haproxy', you might see this right away https://review.opendev.org/c/openstack/kolla-ansible/+/940810 | 08:40 |
| r-krcek | Hello, as I finished some of my work I am looking for some new work in the k-a project. I went through some of the PTG notes and blueprints and found quite a bit of stuff that you are guys planning. (mad respect) But I am unsure on the state of some of those tasks and I don't want to step on anyone's toes. As a half-time junior dev I can't offer to rewrite k-a to assembly, but I was looking at replacing CI bash scripts with tempest. Would that | 08:45 |
| opendevreview | Merged openstack/kayobe master: CI: create ansible.cfg for pre-upgrade deployment https://review.opendev.org/c/openstack/kayobe/+/940441 | 08:48 |
| mnasiadka | r-krcek: kevko tried to do that in the past, but I don't think it had enough reviewers :) | 09:25 |
| r-krcek | Ah, the age old problem :) can you think of anything non-critical that is in the back of the drawer for month and nobody has time for that? | 09:34 |
| mnasiadka | Migration from Redis to Valkey? | 09:34 |
| mnasiadka | I started with a Kolla side patch | 09:34 |
| r-krcek | That sounds like a nice idea. I'll take a look at your patch and think of how to go from there. | 09:43 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 09:44 |
| mnasiadka | r-krcek: and we always would value your reviews on other people patches - ideally those that have RP+1 :) | 09:45 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 09:56 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 10:05 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 10:13 |
| frickler | oh, great, looks like gitlint has very strict ideas about which version of click to use, and those ideas don't align with what u-c says. it is worth setting up running it in a dedicated, un-u-c-ed venv? or rather look for a more sociable tool? | 10:25 |
| mnasiadka | I guess no need to raise our nodepool vms count :) | 10:34 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Revert "docker/podman_sdk: Fix dbus-python installation in virtualenv" https://review.opendev.org/c/openstack/ansible-collection-kolla/+/942177 | 11:30 |
| opendevreview | hamid lotfi proposed openstack/kolla-ansible stable/2024.1: Change restart_policy from oneshot to no https://review.opendev.org/c/openstack/kolla-ansible/+/942178 | 11:41 |
| yuval | mnasiadka hey - kind reminder regarding this patch: https://review.opendev.org/c/openstack/kolla-ansible/+/938649 | 11:45 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Fix dbus deps https://review.opendev.org/c/openstack/ansible-collection-kolla/+/942179 | 11:50 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 11:50 |
| opendevreview | hamid lotfi proposed openstack/kolla-ansible stable/2024.1: Change the host from "database_address" to "api_interface_address" https://review.opendev.org/c/openstack/kolla-ansible/+/942178 | 11:52 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Fix dbus deps https://review.opendev.org/c/openstack/ansible-collection-kolla/+/942179 | 11:58 |
| opendevreview | hamid lotfi proposed openstack/kolla-ansible stable/2024.1: Change the host from "database_address" to "api_interface_address" https://review.opendev.org/c/openstack/kolla-ansible/+/941021 | 12:05 |
| opendevreview | hamid lotfi proposed openstack/kolla-ansible stable/2024.1: Change the host from "database_address" to "api_interface_address" https://review.opendev.org/c/openstack/kolla-ansible/+/941021 | 12:09 |
| mnasiadka | kolla-core: FYI - it seems the podman jobs are broken due to a new release of podman pip package - but they broke submodules visibility - fix here (but no new release yet) - https://github.com/containers/podman-py/commit/0b9dcf3fbf6d0c15b4693a38129cea683c217b63 | 13:46 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Fix dbus deps https://review.opendev.org/c/openstack/ansible-collection-kolla/+/942179 | 13:52 |
| mnasiadka | mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu darmach pabloclsn - meeting in ~3 minutes | 13:57 |
| frickler | mnasiadka: well the fix was merged just 1h ago and the release is in the works they say, I guess we can wait a couple of hours https://github.com/containers/podman-py/pull/515#issuecomment-2668473893 | 13:58 |
| mnasiadka | frickler: that's what I thought as well, happy to pin tomorrow if it still persists | 13:58 |
| frickler | ack | 13:58 |
| mnasiadka | #startmeeting kolla | 14:00 |
| opendevmeet | Meeting started Wed Feb 19 14:00:07 2025 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'kolla' | 14:00 |
| mnasiadka | #topic rollcall | 14:00 |
| mnasiadka | o/ | 14:00 |
| MattCrees[m] | o/ | 14:00 |
| bbezak | o/ | 14:00 |
| mhiner | o/ | 14:00 |
| mmalchuk | o/ | 14:00 |
| r-krcek | o/ | 14:00 |
| koperg[m] | o/ | 14:01 |
| frickler | \o | 14:01 |
| mnasiadka | #topic agenda | 14:01 |
| yuval_ | 0/ | 14:01 |
| mnasiadka | * Announcements | 14:01 |
| mnasiadka | * CI status | 14:01 |
| mnasiadka | * Release tasks | 14:01 |
| mnasiadka | * Current cycle planning | 14:01 |
| mnasiadka | * Additional agenda (from whiteboard) | 14:01 |
| mnasiadka | * Open discussion | 14:01 |
| mnasiadka | #topic Announcements | 14:02 |
| mnasiadka | Since me and bbezak will be going for OpenInfra Days US Pasadena - the weekly meeting on 5th March will be cancelled. | 14:02 |
| mnasiadka | (unless anybody has an urge to host the meeting, but I assume not) | 14:02 |
| mnasiadka | #topic CI status | 14:02 |
| mnasiadka | podman-py has released a new version and has broken our CI - since they already fixed that in the main branch code - we'll wait until tomorrow for a new release instead of pinning it now | 14:03 |
| mnasiadka | Overall the CI looks ok, but I have a feeling we have a lot of failures related to proxysql running in a resource constrained env | 14:04 |
| mnasiadka | frickler, bbezak for some stability I would probably think about disabling proxysql in our CI and work out the issues in a dedicated patch | 14:04 |
| mnasiadka | instead of wasting OpenDev resources on rechecks... | 14:04 |
| mnasiadka | Did you notice the same - or is it only me? | 14:05 |
| mnasiadka | Ok, let's move forward, we can discuss that after the meeting. | 14:06 |
| frickler | I didn't look at failures a lot recently, but I'm fine with your proposal unless kevko wants to pick up a fight with zuul once again ;) | 14:06 |
| mnasiadka | Well, I did some things, but they haven't fixed the issues | 14:06 |
| mnasiadka | I'm thinking about disabling mariadb healthchecks in proxysql, that should ease some of the load | 14:06 |
| mnasiadka | But I'd prefer we regain clean CI jobs for the sake of couple of last months of the release cycle | 14:07 |
| frickler | disable or just reduce the frequency? | 14:07 |
| mnasiadka | I'll try that out | 14:07 |
| mnasiadka | reducing the frequency makes the proxysql watchdog behave in a weird way | 14:07 |
| mnasiadka | we could just disable the watchdog, maybe that helps | 14:07 |
| mnasiadka | I'll experiment | 14:08 |
| mnasiadka | #topic Release tasks | 14:08 |
| mnasiadka | It's R-6 | 14:08 |
| mnasiadka | R-8: Switch images to current release¶ | 14:08 |
| mnasiadka | Any volunteer for carrying out https://docs.openstack.org/kolla/latest/contributor/release-management.html#r-8-switch-images-to-current-release ? | 14:09 |
| frickler | didn't we talk about that last week and someone already volunteered? | 14:09 |
| frickler | bbezak ? | 14:10 |
| mnasiadka | Ah, I didn't check the logs | 14:10 |
| frickler | but I haven't seen a patch yet, so maybe it got lost | 14:10 |
| bbezak | I did indeed. But just did the releases. I look into that next. | 14:11 |
| opendevreview | Piotr proposed openstack/kolla-ansible master: Added vault support to barbican as back-end secret https://review.opendev.org/c/openstack/kolla-ansible/+/935704 | 14:11 |
| mnasiadka | bbezak: maybe koperg or somebody else from SHPC could have a look? it's a simple enough job :) | 14:11 |
| koperg[m] | no problem | 14:11 |
| bbezak | Makes sense | 14:11 |
| mnasiadka | goodie | 14:11 |
| mnasiadka | #action koperg to carry out R-8: switch images to current release | 14:12 |
| mnasiadka | #topic Current cycle planning | 14:13 |
| mnasiadka | Anybody wants to raise any feature patches they have been working on? | 14:13 |
| jovial | One I was hoping to get in: https://review.opendev.org/c/openstack/kolla-ansible/+/920294 | 14:13 |
| jovial | So we can remove the need for sudo on the control host with Kayobe | 14:14 |
| mnasiadka | Do you want to backport this? | 14:14 |
| yuval_ | https://review.opendev.org/c/openstack/kolla-ansible/+/938649 | 14:15 |
| jovial | If upstream are happy to backport, then I'd be all for it, but otherwise I can cherry pick back to our fork | 14:16 |
| jovial | Hoping to use it with Caracal myself | 14:16 |
| frickler | oh, it's almost a year that I looked at that, can check again | 14:16 |
| mnasiadka | jovial: There's a valid comment by Sven - if you want to get it backported - raise a bug :) | 14:16 |
| mnasiadka | if not, we can merge it as is I think | 14:17 |
| jovial | OK, I'll make a bug report if backporting is acceptable | 14:17 |
| jovial | Will try and get that done after this meeting | 14:17 |
| mnasiadka | frickler: I know that we agreed to rework all Cinder backends support to something more generic (+ Ceph because that's the most used one) - but I have a feeling that it won't happen this cycle - and don't want to block yuval_ - what do you think? | 14:17 |
| mnasiadka | jovial: thanks | 14:18 |
| opendevreview | Piotr proposed openstack/kolla-ansible master: Added vault support to barbican as back-end secret https://review.opendev.org/c/openstack/kolla-ansible/+/935704 | 14:19 |
| frickler | mnasiadka: hmm, difficult question, if we just keep delaying it we'll need to add ever more deprecations. but I also don't wan't to block ppl | 14:19 |
| mmalchuk | mnasiadka: another one brick (backend) on the wall) | 14:19 |
| yuval_ | I am new here, just want to say I really love kolla-ansible. | 14:21 |
| mnasiadka | Well, how do we think it's going to happen? we have a high number of backends - people use this, do we expect to have a generic method of copying in additional files plus an additional config override fully written by people? (I mean the cinder.conf backend section) | 14:21 |
| yuval_ | I would glad to help with making generic solution, but I need some guidelines | 14:21 |
| mnasiadka | For now there's simplicity, if we drop that all in one cycle - we might make a lot of people angry - and also now we have sort of control over ''the recommended kolla-ansible way to configure those backends'' | 14:22 |
| mnasiadka | I have a feeling we need to rethink it in details on the next PTG | 14:22 |
| jovial | Also its a lot harder to keep out of tree config up to date | 14:22 |
| mnasiadka | I'll try to draft out something for the next meeting and we can discuss what are the options to make it more generic, but still keep the simplicity | 14:23 |
| mnasiadka | For now let's review yuval_ 's patch and try to get it in Epoxy | 14:23 |
| mnasiadka | Ok, there's one more I see from mhiner | 14:23 |
| mnasiadka | [2025-02-12] (mhiner): Move to high level client in DockerWorker | https://review.opendev.org/c/openstack/kolla-ansible/+/908295... (full message at <https://matrix.org/oftc/media/v1/media/download/AWL9HsVOlOuCRXM643x6ythC_7IRTDuK5i4VrfYQV_Xf7WFIEBGibpYEfWeGPdUHrq9Vb7cxbIVQh3NrhzutFLhCeVZrD3gAAG1hdHJpeC5vcmcvZ2p4Vk9oVXZ1S21RdUVIRFh3cHRldmtx>) | 14:23 |
| mnasiadka | mhiner: I think easiest would be to try to get Canonical to update the python3-docker package - forcing people to use venvs is a bit of a major move | 14:24 |
| mnasiadka | so let's try the easier way for now | 14:24 |
| mhiner | I have trouble forcing even our CI to use it, so I agree :) | 14:25 |
| * mmalchuk removed -1 from yuval_ review | 14:25 | |
| mnasiadka | They have a lot newer version in oracular, so it should be possible | 14:25 |
| mnasiadka | mhiner: so please try and let us know :) | 14:25 |
| mhiner | Alright | 14:26 |
| mnasiadka | #topic Additional agenda (from whiteboard) | 14:26 |
| mnasiadka | [2025-02-19] (r-krcek): | 14:26 |
| mnasiadka | please review https://review.opendev.org/c/openstack/kolla-ansible/+/933073 | 14:26 |
| mnasiadka | (podman migration) please review the cahin starting at https://review.opendev.org/c/openstack/kolla-ansible/+/912460 | 14:26 |
| mnasiadka | 933073 is not dependent on this ubuntu python3-docker problem, right? | 14:26 |
| r-krcek | Not, this one is not | 14:26 |
| mnasiadka | frickler: can you re-review? I've seen r-krcek has updated the patch after your comments | 14:27 |
| mnasiadka | jovial: Willing to have a look in podman migration patch? | 14:27 |
| mnasiadka | (I know you did some Kayobe related work with Podman, so maybe this would be useful) | 14:27 |
| frickler | ack, added to my list once again | 14:28 |
| jovial | Yeah, it did look super cool. I can certainly take a look. | 14:28 |
| mnasiadka | thank you | 14:28 |
| mnasiadka | (both) | 14:28 |
| mnasiadka | #topic Open discussion | 14:28 |
| r-krcek | Thank you guys :) | 14:28 |
| mnasiadka | Anything else anybody? | 14:28 |
| mhiner | Can someone please close this: https://review.opendev.org/c/openstack/kolla-ansible/+/905837 | 14:29 |
| mnasiadka | As in abandon? | 14:29 |
| mhiner | Original author is not available anymore to do it | 14:29 |
| mhiner | Yes | 14:29 |
| mnasiadka | done | 14:29 |
| mhiner | Thanks | 14:29 |
| mnasiadka | Ok then - I'm sorry but I need to run | 14:29 |
| mnasiadka | Thank you all for coming :) | 14:30 |
| yuval_ | Thank you | 14:30 |
| mnasiadka | If there's anything - please ask after the meeting and I'll reply later | 14:30 |
| mnasiadka | #endmeeting | 14:30 |
| opendevmeet | Meeting ended Wed Feb 19 14:30:15 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:30 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-02-19-14.00.html | 14:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-02-19-14.00.txt | 14:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-02-19-14.00.log.html | 14:30 |
| r-krcek | Thanks | 14:30 |
| mmalchuk | mnasiadka: thanks | 14:30 |
| opendevreview | Verification of a change to openstack/kayobe stable/2023.2 failed: Fix Depends-On https://review.opendev.org/c/openstack/kayobe/+/940231 | 14:33 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 15:53 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 16:01 |
| opendevreview | Piotr proposed openstack/kolla-ansible master: Added vault support to barbican as back-end secret https://review.opendev.org/c/openstack/kolla-ansible/+/935704 | 16:13 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 16:21 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 16:45 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 17:00 |
| opendevreview | Piotr proposed openstack/kolla-ansible master: Added vault support to barbican as back-end secret https://review.opendev.org/c/openstack/kolla-ansible/+/935704 | 17:06 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 17:10 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 17:11 |
| opendevreview | Piotr proposed openstack/kolla-ansible master: Added vault support to barbican as back-end secret https://review.opendev.org/c/openstack/kolla-ansible/+/935704 | 17:26 |
| mnasiadka | Seems we have new podman version that is fixed | 17:29 |
| mnasiadka | frickler: Maybe we should finally add podman to u-c? | 17:29 |
| mnasiadka | (but I assume it's too late for this cycle) | 17:29 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 18:06 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Allow to run post-deploy without privilege escalation https://review.opendev.org/c/openstack/kolla-ansible/+/920294 | 18:55 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Allow to run post-deploy without privilege escalation https://review.opendev.org/c/openstack/kolla-ansible/+/920294 | 18:56 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Support running without root privileges https://review.opendev.org/c/openstack/kayobe/+/923430 | 19:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Use ansible-lint for CI roles and playbooks https://review.opendev.org/c/openstack/kolla/+/937614 | 20:06 |
| opendevreview | Amith Kotu proposed openstack/kolla-ansible master: docs: Correct typo in variable name https://review.opendev.org/c/openstack/kolla-ansible/+/942163 | 20:19 |
| opendevreview | Amith Kotu proposed openstack/kolla-ansible master: docs: Correct typo in variable name https://review.opendev.org/c/openstack/kolla-ansible/+/942163 | 20:54 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Remove Git remote origin after cloning sources https://review.opendev.org/c/openstack/kolla/+/942299 | 22:25 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Remove Git remote origin after cloning sources https://review.opendev.org/c/openstack/kolla/+/942299 | 22:30 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Remove Git remote origin after cloning sources https://review.opendev.org/c/openstack/kolla/+/942299 | 22:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!