| *** jph6 is now known as jph | 00:21 | |
| opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 08:27 |
|---|---|---|
| jovial | SvenKieske, Just trying to get https://review.opendev.org/c/openstack/kolla-ansible/+/909910 over the line. For the TODO linking to a bug report. Is that a new bug I should file under the k-a project? | 10:00 |
| jovial | Or should I link to https://bugs.launchpad.net/kolla/+bug/2054685? | 10:01 |
| jovial | I was thinking I could put the details you post in the code review in https://bugs.launchpad.net/kolla/+bug/2054685 | 10:02 |
| SvenKieske | any bug that has the actual content on what to do is fine imho :) no need to do too much work :) | 10:03 |
| jovial | Cool, thanks. I've posted your comments from code review here: https://bugs.launchpad.net/kolla/+bug/2054685/comments/3 as that contained some handy links :) | 10:06 |
| SvenKieske | thank you. | 10:07 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Adds feature flag for ironic-inspector in bifrost https://review.opendev.org/c/openstack/kolla-ansible/+/909910 | 10:18 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Skip buggy release of docker PyPI package https://review.opendev.org/c/openstack/kayobe/+/909913 | 10:27 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Skip buggy release of docker PyPI package https://review.opendev.org/c/openstack/kayobe/+/909913 | 10:28 |
| opendevreview | Mark Goddard proposed openstack/ansible-collection-kolla master: Add stats callback plugin https://review.opendev.org/c/openstack/ansible-collection-kolla/+/910347 | 10:36 |
| SvenKieske | mgoddard: is https://review.opendev.org/c/openstack/ansible-collection-kolla/+/910347 still WIP or are reviews appreciated? :) | 10:41 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 11:21 |
| jovial | I see: "dnsmasq: process is missing required capability NET_RAW" in the kayobe podman CI jobs. Is it OK to add NET_RAW to ironic dnsmasq? What I don't quite understand is how I don't hit this when running locally. Any one got any ideas? | 11:34 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 11:51 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Add NET_RAW to dnsmasq container https://review.opendev.org/c/openstack/kolla-ansible/+/910501 | 11:52 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 11:55 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 11:59 |
| SvenKieske | jovial: are you maybe adding some capabilities by default when running locally? it's an often seen advice - wrongly - to start local containers in privileged mode etc. | 12:00 |
| mnasiadka | jovial: is it maybe Ubuntu? I saw some discussion around some buggy dnsmasq version affecting Ironic | 12:08 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 12:14 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 12:15 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: prometheus: Add friendly instance labels for ironic and alertmanager https://review.opendev.org/c/openstack/kolla-ansible/+/899614 | 12:19 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: prometheus: Use api_interface_address for scrape targets https://review.opendev.org/c/openstack/kolla-ansible/+/899615 | 12:19 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Support reducing scope of delegated fact gathering https://review.opendev.org/c/openstack/kolla-ansible/+/910503 | 12:19 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 12:25 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Add support for rpm to repos.yaml https://review.opendev.org/c/openstack/kolla/+/909879 | 12:47 |
| mgoddard | SvenKieske: reviews appreciated | 13:05 |
| SvenKieske | mgoddard: what's the benefit of this compared to ara which we already have? | 13:11 |
| mgoddard | SvenKieske: it's easily machine-readable, so can be used by higher layers to make decisions about how to handle failures | 13:14 |
| mgoddard | and unless things have changed recently ARA isn't provided out of the box by Kolla | 13:14 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Support configuring persistent systemd journal https://review.opendev.org/c/openstack/kayobe/+/909685 | 13:23 |
| opendevreview | Will Szumski proposed openstack/kayobe master: WIP: Add podman support https://review.opendev.org/c/openstack/kayobe/+/909686 | 13:23 |
| mgoddard | SvenKieske: to be a bit more clear, there's a kayobe patch coming that will make use of this. Having the plugin in a collection makes it much simpler to use by both K-A and Kayobe | 13:24 |
| mgoddard | It could also be used outside of Kayobe | 13:24 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 13:29 |
| jovial | SvenKieske, mnasiadka The CAP_NET_RAW thing was on Rocky 9. I've dumped the podman inspect here: https://paste.openstack.org/show/bmOo7IqJmU9prpNOXRoc/. Perhaps I need to extract that from the CI job to compare. | 13:35 |
| jovial | I was wondering if it could the BoundingCaps that differed. Is that affected by some OS setting? | 13:36 |
| jovial | Do we deploy ironic in the kolla-ansible CI job? | 13:37 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 13:42 |
| mnasiadka | mgoddard mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu - meeting in 10 | 13:50 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Allow to continue when some hosts are unreachable https://review.opendev.org/c/openstack/kayobe/+/910509 | 13:58 |
| opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 13:59 |
| mnasiadka | #startmeeting kolla | 14:00 |
| opendevmeet | Meeting started Wed Feb 28 14:00:32 2024 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 |
| jangutter | o/ | 14:00 |
| jovial | \o | 14:00 |
| mhiner | o/ | 14:00 |
| SvenKieske | o/ | 14:00 |
| bbezak | o/ | 14:01 |
| matusjenca | o/ | 14:02 |
| mnasiadka | #topic Agenda | 14:02 |
| mnasiadka | * CI status | 14:02 |
| mnasiadka | * Release tasks | 14:02 |
| mnasiadka | * Current cycle planning | 14:02 |
| mnasiadka | * Additional agenda (from whiteboard) | 14:02 |
| mnasiadka | * Open discussion | 14:02 |
| mnasiadka | #topic CI status | 14:02 |
| mnasiadka | We had faulty toolbox image in 2023.1 - but that's fixed now | 14:03 |
| mnasiadka | any other CI breakages? | 14:03 |
| mnasiadka | Guess not | 14:03 |
| mnasiadka | #topic Release tasks | 14:04 |
| mnasiadka | It's R-5 week - nothing planned for Kolla | 14:05 |
| mnasiadka | #topic Current cycle planning | 14:05 |
| jovial | Just on the subject of CI breakage. If anyone could review: https://review.opendev.org/c/openstack/kolla-ansible/+/909910 so that I can fix kayobe CI, that would be appreciated. | 14:05 |
| mnasiadka | Now is the time to paste URLs to your patches, discuss them, or discuss ideas, etc | 14:05 |
| mnasiadka | jovial: reviewed | 14:07 |
| mnasiadka | mhiner: Podman migration? how is it going? | 14:08 |
| mhiner | I have recently fixed a problem with variables, now I'm working on fixing cleanup | 14:08 |
| mnasiadka | good | 14:09 |
| mnasiadka | I started some work on getting rpm repos defined in repos.yaml - so people having local mirrors have an easier life - https://review.opendev.org/c/openstack/kolla/+/909879 | 14:10 |
| mnasiadka | Ok, let's move on | 14:12 |
| mnasiadka | #topic Additional agenda (from whiteboard) | 14:12 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: DNM: Test --continue-on-unreachable https://review.opendev.org/c/openstack/kayobe/+/910511 | 14:13 |
| mnasiadka | (SvenKieske): Implement (TLS) Redis Caching Backend needs Reviews: https://review.opendev.org/q/topic:%22redis-cache%22 | 14:14 |
| frickler | yeah, I have that on my list | 14:16 |
| opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Refactor of docker worker https://review.opendev.org/c/openstack/kolla-ansible/+/908295 | 14:16 |
| matusjenca | I made the Redis patches, so if anyone has any questions or I am doing something wrong please ask me | 14:17 |
| mnasiadka | Ok, I'll have a look this week | 14:24 |
| mnasiadka | #topic Open discussion | 14:24 |
| mnasiadka | Anyone with any open topic? ;-) | 14:24 |
| SvenKieske | doesn't look like it | 14:28 |
| mnasiadka | Then it's a wrap | 14:28 |
| mnasiadka | Thank you all for coming - see you next week! | 14:29 |
| mnasiadka | #endmeeting | 14:29 |
| opendevmeet | Meeting ended Wed Feb 28 14:29:03 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:29 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-02-28-14.00.html | 14:29 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-02-28-14.00.txt | 14:29 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-02-28-14.00.log.html | 14:29 |
| SvenKieske | thanks mnasiadka | 14:29 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Bump up Ansible supported versions to 8.x/9.x https://review.opendev.org/c/openstack/kayobe/+/910513 | 14:31 |
| opendevreview | Uwe Jäger proposed openstack/kolla-ansible master: Skyline configure Prometheus https://review.opendev.org/c/openstack/kolla-ansible/+/910514 | 14:32 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Update barbican.conf for oslo.db 15.0.0 https://review.opendev.org/c/openstack/kolla-ansible/+/910517 | 14:44 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Adds feature flag for ironic-inspector in bifrost https://review.opendev.org/c/openstack/kolla-ansible/+/909910 | 14:59 |
| spatel | anybody using kolla-ansible with cinder NFS backend? | 15:11 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic https://review.opendev.org/c/openstack/kayobe/+/909671 | 15:15 |
| opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 15:26 |
| SvenKieske | question: should you really vote +2 as a core maintainer for a backport you did yourself of a change you also did yourself? e.g. https://review.opendev.org/c/openstack/kolla-ansible/+/909542 seems weird to me. I get why it's done (lack of cores) but I'm even hesitant to +1 my own changes, because I think the point of a review is, that different people look at my code, not myself. | 15:55 |
| SvenKieske | so I think this kind of defeats the purpose, no? if we only follow policy to follow policy and there is no real observable benefit, maybe we should question the policy or do some other changes so the policy makes sense again? | 15:56 |
| SvenKieske | but maybe I'm misunderstanding the review system :) | 15:59 |
| opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 16:22 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Allow to continue when some hosts are unreachable https://review.opendev.org/c/openstack/kayobe/+/910509 | 17:06 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: DNM: Test --continue-on-unreachable https://review.opendev.org/c/openstack/kayobe/+/910511 | 17:06 |
| SvenKieske | mgoddard: fwiw I guess you broke the prometheus copy config task somehow, still trying to wrap my head around why exactly though: https://review.opendev.org/c/openstack/kolla-ansible/+/899615 | 17:16 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: DNM: Test --continue-on-unreachable https://review.opendev.org/c/openstack/kayobe/+/910511 | 19:50 |
| mgoddard | SvenKieske: re backports, IMHO the policy applies to the initial review. A backport review is different. It's asking 1. is the change appropriate to backport, 2. has the backport been performed correctly. Hopefully we can trust people to judge 1 for their own changes, but 2x +2 requirement should catch most issues here. For 2 this is a bit more blurry, but if the cherry pick | 19:57 |
| mgoddard | was clean or required a minimal fix up, I don't see why it needs to be a different proposer. Otherwise we're into the realm of asking someone else to cherry pick a change in order to +2 it... | 19:57 |
| mgoddard | SvenKieske: re prometheus, thanks, I'll take a look | 19:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!