r-krcek | Hi! I was looking into this bug report https://bugs.launchpad.net/kolla-ansible/+bug/1831120 and examining MariaDB code in kolla-ansible. And I see that k-a has already implemented basically everything that the guide advises for safe restart of the cluster. Is there any specific reason why mariadb_recovery is a separate command and the recovery is not ran automatically when restarting k-a deployment? | 06:58 |
---|---|---|
r-krcek | I think it might be because you don't want to "re-bootstrap" the cluster on each deploy? But that could avoided if we detect if the cluster is in sync or not. | 06:58 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Fix upgrade jobs failing on adding new certificates https://review.opendev.org/c/openstack/kolla-ansible/+/926284 | 06:58 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: CI: Fix upgrade jobs failing on adding new certificates https://review.opendev.org/c/openstack/kolla-ansible/+/926284 | 07:00 |
opendevreview | Magnus Lööf proposed openstack/kolla-ansible master: Enable TLS backend for designate https://review.opendev.org/c/openstack/kolla-ansible/+/866524 | 07:19 |
mnasiadka | kevko: supporting connection to external hashicorp vault vs publishing images with vault is a separate thing - we could discuss using Hashicorp provided container images in kolla-ansible as an option - but I don't think we should build, publish and distribute container images with Hashicorp's BSL code. | 07:58 |
SvenKieske | r-krcek: I think a separate command is needed in any case you want to specifically manually start a recovery, no? | 07:58 |
mnasiadka | And as mentioned in the TC meeting - it's a PTG discussion. | 07:58 |
kevko | mnasiadka: that's not about build and publish ... it's about to land the code how to build ...it's up to user .. | 08:01 |
kevko | mnasiadka: from another view ... published images from kolla team are not supposed to be the production one ...so it's testing ...so it's also not against the licensing as fungi mentioned in TC meeting ... | 08:02 |
mnasiadka | No thanks, Ceph is the same situation - we support External Ceph - why not support External Consul? | 08:02 |
kevko | it's easier | 08:05 |
kevko | foundation legal counsel has said that openstack software interfacing with bsl software doesn't present license incompatibilities .... so why ? | 08:07 |
mnasiadka | As mentioned before - it's a PTG discussion - because we can then discuss it in proper forum. | 08:07 |
kevko | totally agree | 08:07 |
kevko | i am not trying to push some code fast ... i will be patient and will follow the discussion | 08:08 |
kevko | but it's proper question according to latest various companies just half closed their codes ... | 08:08 |
kevko | elastic, redis, vault, consul ...what will be next ? rabbit ? :D | 08:09 |
mnasiadka | we jumped ship with elastic, we never packaged vault and consul, and yes - we need to move to valkey | 08:09 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: service-python-path: Introduce a role to get python_path of an image https://review.opendev.org/c/openstack/kolla-ansible/+/926632 | 08:11 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: service-python-path: Introduce a role to get python_path of an image https://review.opendev.org/c/openstack/kolla-ansible/+/926632 | 08:11 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: service-python-path: Introduce a role to get python_path of an image https://review.opendev.org/c/openstack/kolla-ansible/+/926632 | 08:17 |
r-krcek | SvenKieske: But wouldn't it be more convenient to have it done automatically? I'm looking for any reasons how/if it might cause problems just in case I missed anything (I cant see any), because I think that could be a nice addition. :) | 08:23 |
SvenKieske | sure! I just wanted to say we still need to be able to trigger this manually, especially for staying backward compatible | 08:25 |
mnasiadka | and for not running whole deploy just to get mariadb behave properly | 08:26 |
mnasiadka | and still mariadb_recovery could fail | 08:26 |
r-krcek | I see, yeah, backwards compatibility should definitely stay there. | 08:28 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Remove "distutils" library https://review.opendev.org/c/openstack/kolla-ansible/+/857003 | 08:52 |
opendevreview | Mark Goddard proposed openstack/kolla master: Pin OpenSearch Dashboards to 2.15 https://review.opendev.org/c/openstack/kolla/+/926528 | 08:53 |
SvenKieske | can someone push this over the line? is this stuck because the parent commit changed or is this CI infra related? https://review.opendev.org/c/openstack/kolla-ansible/+/922618 | 08:56 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: service-python-path: Introduce a role to get python_path of an image https://review.opendev.org/c/openstack/kolla-ansible/+/926632 | 08:57 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Move actions to kolla_container_facts https://review.opendev.org/c/openstack/kolla-ansible/+/911505 | 09:02 |
mnasiadka | SvenKieske: https://review.opendev.org/c/openstack/kolla-ansible/+/922617 - this needs to go in first | 09:07 |
Vii | will you merge until there are too many conflicts? https://review.opendev.org/c/openstack/kolla/+/924302 and https://review.opendev.org/c/openstack/kolla-ansible/+/924575 this solution works very well - neutron-ovn-vpn | 09:12 |
mnasiadka | Vii: there is unresolved conversation on first of them | 09:14 |
Vii | mnasiadka: maybe ping the owner :) , it would be a shame to lose a working solution. I think a lot of people are waiting for it :) | 09:16 |
mnasiadka | Vii: you're more interested than me, so feel free to ping the owner :) | 09:16 |
Vii | :) | 09:16 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: service-python-path: Introduce a role to get python_path of an image https://review.opendev.org/c/openstack/kolla-ansible/+/926632 | 09:28 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 09:46 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Add Redis as caching backend for Ceilometer https://review.opendev.org/c/openstack/kolla-ansible/+/926721 | 09:46 |
opendevreview | Merged openstack/kolla-ansible master: Add support for customising Neutron physical network names https://review.opendev.org/c/openstack/kolla-ansible/+/922320 | 09:54 |
opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2024.1: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926028 | 10:25 |
opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2024.1: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926028 | 10:26 |
opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2023.2: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926029 | 10:27 |
opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2023.2: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926029 | 10:28 |
opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2023.1: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926030 | 10:29 |
opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2023.1: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926030 | 10:30 |
opendevreview | Merged openstack/kolla-ansible master: reorder requirements.txt alphabetically https://review.opendev.org/c/openstack/kolla-ansible/+/922617 | 10:31 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: service-python-path: Introduce a role to get python_path of an image https://review.opendev.org/c/openstack/kolla-ansible/+/926632 | 10:31 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: service-python-path: Introduce a role to get python_path of an image https://review.opendev.org/c/openstack/kolla-ansible/+/926632 | 11:09 |
opendevreview | Magnus Lööf proposed openstack/kolla-ansible master: Enable TLS backend for designate https://review.opendev.org/c/openstack/kolla-ansible/+/866524 | 11:09 |
opendevreview | Merged openstack/kolla-ansible master: fact gathering: Remove gathering of package facts https://review.opendev.org/c/openstack/kolla-ansible/+/901056 | 11:22 |
opendevreview | Merged openstack/kolla-ansible master: fact gathering: Group hosts before gathering facts https://review.opendev.org/c/openstack/kolla-ansible/+/899592 | 11:22 |
opendevreview | Merged openstack/kayobe master: Bump Ansible version installed in Kolla Ansible virtualenv https://review.opendev.org/c/openstack/kayobe/+/923556 | 11:26 |
opendevreview | Will Szumski proposed openstack/kayobe master: Aligns Kayobe Ansible version with Kolla Ansible https://review.opendev.org/c/openstack/kayobe/+/926639 | 11:31 |
opendevreview | Will Szumski proposed openstack/kayobe master: Remove unnecessary dependency https://review.opendev.org/c/openstack/kayobe/+/926664 | 11:54 |
opendevreview | Will Szumski proposed openstack/kayobe master: Remove unnecessary dependency https://review.opendev.org/c/openstack/kayobe/+/926664 | 11:55 |
opendevreview | Michal Arbet proposed openstack/kolla master: Remove KOLLA_DISTRO_PYTHON_VERSION https://review.opendev.org/c/openstack/kolla/+/926744 | 12:08 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Remove distro_python_version https://review.opendev.org/c/openstack/kolla-ansible/+/926745 | 12:18 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/2024.1: fact gathering: Remove gathering of package facts https://review.opendev.org/c/openstack/kolla-ansible/+/926746 | 12:24 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/2023.2: fact gathering: Remove gathering of package facts https://review.opendev.org/c/openstack/kolla-ansible/+/926749 | 12:33 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/2023.1: fact gathering: Remove gathering of package facts https://review.opendev.org/c/openstack/kolla-ansible/+/926750 | 12:34 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/2024.1: fact gathering: Group hosts before gathering facts https://review.opendev.org/c/openstack/kolla-ansible/+/926751 | 12:35 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/2023.2: fact gathering: Group hosts before gathering facts https://review.opendev.org/c/openstack/kolla-ansible/+/926752 | 12:35 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/2023.1: fact gathering: Group hosts before gathering facts https://review.opendev.org/c/openstack/kolla-ansible/+/926753 | 12:36 |
opendevreview | Jakub Darmach proposed openstack/kolla master: Add exclude for permissions in kolla config.json file https://review.opendev.org/c/openstack/kolla/+/655620 | 12:43 |
opendevreview | Merged openstack/kolla-ansible master: Apply public firewalld rules immediately https://review.opendev.org/c/openstack/kolla-ansible/+/884365 | 12:47 |
opendevreview | Merged openstack/kolla-ansible stable/2023.2: Support CloudKitty deployment with internal TLS https://review.opendev.org/c/openstack/kolla-ansible/+/924646 | 12:48 |
opendevreview | Merged openstack/kolla-ansible stable/2023.1: Support CloudKitty deployment with internal TLS https://review.opendev.org/c/openstack/kolla-ansible/+/924647 | 12:48 |
mnasiadka | mgoddard mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu darmach - meeting in 6 minutes | 12:54 |
opendevreview | Michal Arbet proposed openstack/kolla master: Remove KOLLA_DISTRO_PYTHON_VERSION https://review.opendev.org/c/openstack/kolla/+/926744 | 12:55 |
mnasiadka | #startmeeting kolla | 13:00 |
opendevmeet | Meeting started Wed Aug 21 13:00:16 2024 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 |
SvenKieske | o/ | 13:00 |
mmalchuk | o/ | 13:00 |
mattcrees | o/ | 13:00 |
bbezak_alt | o/ | 13:00 |
IvanVnuko[m] | o/ | 13:00 |
mhiner | o/ | 13:00 |
frickler | \o | 13:00 |
r-krcek | o/ | 13:01 |
mnasiadka | #topic agenda | 13:02 |
mnasiadka | * Announcements | 13:02 |
mnasiadka | * CI status | 13:02 |
mnasiadka | * Current cycle planning | 13:02 |
mnasiadka | * Additional agenda (from whiteboard) | 13:02 |
mnasiadka | * Open discussion | 13:02 |
mnasiadka | #topic Announcements | 13:02 |
jovial | o/ | 13:02 |
darmach6 | o/ | 13:02 |
mnasiadka | I've decided to fill in for the PTL role for another cycle, the patch has been merged already - https://review.opendev.org/c/openstack/election/+/926285 | 13:02 |
SvenKieske | \o/ | 13:02 |
mnasiadka | #topic CI status | 13:03 |
mnasiadka | I haven't seen any obvious breakages, so I guess nothing to report here ;-) | 13:03 |
mnasiadka | #topic Current cycle planning | 13:03 |
jovial | Kayobe CI back to green as of this morning | 13:03 |
mnasiadka | ansible bump? | 13:03 |
jovial | Indeed | 13:03 |
mnasiadka | Ok, let's have a look on planned ''features'' this cycle | 13:04 |
mnasiadka | Ubuntu Noble - that's obvious, nearly getting there | 13:04 |
mnasiadka | OVN BGP agent - I haven't done anything - bbezak_alt - do you have some spare cycles? | 13:04 |
bbezak_alt | yeah, will revive that | 13:05 |
mnasiadka | great | 13:05 |
mnasiadka | Openinfra mirrors for some problematic packages - SvenKieske how's it going? | 13:05 |
SvenKieske | mhm nothing to report, as the downstream funding is running out I doubt I'll get servers/infra capacity from there. But I guess I can still bother opendev infra team if they can cough up some gigabytes? :) | 13:06 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: CI: drop RMQ reconfigure step in queue migrations https://review.opendev.org/c/openstack/kolla-ansible/+/926127 | 13:06 |
mnasiadka | yeah, I think we wanted to discuss with openinfra - but would be good to have a list what we want to mirror and some capacity calculations | 13:07 |
mnasiadka | so if you could update that on whiteboard - would be nice | 13:07 |
mnasiadka | Multiple RMQ versions per cycle - mattcrees how's it going? | 13:07 |
SvenKieske | future funding of the downstream is a little unclear to me currently, so if we're lucky maybe there will still be capacity, but I guess I can only wait for end of year to report back | 13:07 |
SvenKieske | okay, I can certainly compile a list of currently external repositories :) | 13:07 |
mattcrees | We've got a topic set up now: https://review.opendev.org/q/topic:%22slurp-rabbitmq%22, been getting helpful reviews so far :). I'm making sure to respond asap so we can get these merged soon | 13:08 |
mnasiadka | Ok, I'll have a look into those later | 13:09 |
mattcrees | Cheers | 13:09 |
mnasiadka | if other cores can do the same - it would be nice | 13:09 |
mnasiadka | ok, I added that topic to whiteboard | 13:09 |
mnasiadka | I don't have anything more in the whitebaord, so let's move on | 13:10 |
mnasiadka | #topic Additional agenda (from whiteboard) | 13:10 |
mnasiadka | r-krcek [21th August] my wishlist :) | 13:10 |
mnasiadka | please review https://review.opendev.org/c/openstack/kolla-ansible/+/925714 and https://review.opendev.org/c/openstack/kolla/+/925712 | 13:10 |
mnasiadka | This got us stumped with Sven, anyone with bandit knowledge? https://review.opendev.org/c/openstack/kolla-ansible/+/923110 | 13:10 |
mnasiadka | Any feedback would be greatly appreciated https://review.opendev.org/c/openstack/kolla-ansible/+/924651 | 13:10 |
SvenKieske | ah, thx for the reminder, need to check the bandit stuff.. but question is, if this function really needs to call arbitrary commands, afaik it's only used exactly once to call a single command iirc? | 13:11 |
kevko | bash to python ? it will be fun :D | 13:12 |
SvenKieske | ok I'm wrong, it's called twice.. maybe just use subprocess directly without a wrapper function? I mean it's two calls | 13:12 |
SvenKieske | not really worth the wrapper imho, would get rid of the bandit warning I guess | 13:13 |
mnasiadka | ok, the first two look heavy, second one also is not a light one - I added my review on the last one | 13:13 |
mnasiadka | bbezak_alt: willing to have a look in the last one as well? | 13:13 |
mnasiadka | SvenKieske: let's discuss in the patch, shall we? | 13:14 |
SvenKieske | fine :) | 13:14 |
bbezak_alt | 924651 that one? | 13:14 |
r-krcek | Thank you guys for the feedback :) | 13:14 |
bbezak_alt | will check | 13:14 |
mnasiadka | bbezak_alt: yup | 13:14 |
mnasiadka | ok, next one | 13:14 |
mnasiadka | ivnucko [21th August] | 13:14 |
mnasiadka | adding TLS for other than internal/external and backend (proxy to service API) | 13:14 |
mnasiadka | like galera replication - https://review.opendev.org/c/openstack/kolla-ansible/+/925317 or | 13:14 |
mnasiadka | rabbitmq internode - https://review.opendev.org/c/openstack/kolla-ansible/+/921381 | 13:14 |
mnasiadka | ...handle them as part of backend - enable them with kolla_enable_tls_backend and use the same cert from certificate role? | 13:14 |
mnasiadka | ...or a separate type, create a new play in certificates role and enable to have distinct certificate for them? | 13:14 |
mnasiadka | is there a use case for separate certificates for MariaDB to ProxySQL and for inter-node replication, is it worth the complication (cannot use service-cert-copy)? | 13:14 |
mnasiadka | I don't think it's worth any complication - I would handle them as part of backend | 13:15 |
mnasiadka | if anybody needs anything more sophisticated in future - he can add that functionality in a separate patch? | 13:16 |
mnasiadka | Any opinions? | 13:16 |
IvanVnuko[m] | It's mainly about galera, rabbitmq needs distinct certificates for it uses names | 13:16 |
IvanVnuko[m] | but thax, I did it as a part of backend, but maybe galera replication is another beast, so I'm asking for advice... | 13:17 |
SvenKieske | agreed | 13:17 |
IvanVnuko[m] | and a bit of promotion for the TLS encryption change requests we have put up:) | 13:17 |
SvenKieske | I would use the same certs for a start where possible. | 13:17 |
IvanVnuko[m] | thank you | 13:17 |
SvenKieske | we can make it complicated later ;) | 13:17 |
IvanVnuko[m] | yeah, was my initial idea also, just that there are some rumors about a bit complicated restart of galera cluster when new certificates are set up, so... | 13:19 |
mnasiadka | ok, then agreed - keep it simple as possible | 13:19 |
mnasiadka | next one | 13:19 |
mnasiadka | jovial [21st] Should we lower minimum ansible as we should technically support python 3.9 in 2024.2 | 13:19 |
mnasiadka | https://governance.openstack.org/tc/reference/runtimes/2024.2.html | 13:19 |
mnasiadka | 2.15.x is last version support on 3.9 | 13:19 |
mnasiadka | Also affects kayobe | 13:19 |
jovial | Am I right in thinking we should be supporting 3.9 still? | 13:20 |
mnasiadka | we are supporting 3.9 | 13:20 |
frickler | can't you use 3.11 on rocky? | 13:20 |
mnasiadka | on the destination hosts | 13:20 |
mnasiadka | you can use 3.11 or 3.12 | 13:20 |
mnasiadka | my problem with relaxing the requirement is that we would need to have that running in CI | 13:20 |
mnasiadka | so that's a bit more jobs that we currently have | 13:20 |
mnasiadka | and we never supported more than two versions per release | 13:21 |
jovial | Just wondering if I should follow suit in Kayobe and drop 3.9 really | 13:22 |
SvenKieske | mhm, maybe ask TC (frickler? ;) ) if it's a breakage of the requirements? I mean technically you can run openstack with python 3.9, it's just not what we test or deploy? use another deployment tool? ;) | 13:22 |
mnasiadka | well, as I mentioned - Ansible requires 3.10+ only on the host where you're running Ansible | 13:22 |
frickler | yes, I don't think the PTI applies in this case | 13:23 |
mnasiadka | Python 3.9 (available as default in Debian 11 and Centos 9 Stream) Python 3.9 is the the minimum supported/required version for 2024.2. This does not imply that Debian 11 is the minimum supported distribution but it may be used for tox testing. supporting Python 3.9 does not require full tempest testing, but py39 unit tests are expected as a minimum requirement for all Python projects. The minimal requirement for testing jobs | 13:23 |
mnasiadka | against Python versions above is to ensure language compatibility, having more extensive testing is allowed. | 13:23 |
mnasiadka | not reminding that the PTI claims that you need to have passing unit tests on Py3.9 ;-) | 13:23 |
mnasiadka | and nothing more | 13:24 |
frickler | IMO kolla isn't a python project in the sense of that whole document | 13:24 |
mnasiadka | well, kolla-ansible more likely | 13:24 |
frickler | kolla* even | 13:24 |
mnasiadka | I don't think we should add additional work of supporting older Ansible release | 13:25 |
frickler | +1 | 13:25 |
SvenKieske | +1 | 13:25 |
kevko | hmm, are u sure that https://review.opendev.org/c/openstack/kolla-ansible/+/921743 didn't break anything ? | 13:25 |
mnasiadka | And for those sticking to SLURP - they will still need to upgrade to Rocky 10 before jumping from 2024.1 to 2025.1 | 13:25 |
mnasiadka | kevko: I'll fix the rocky upgrade - missed that, but thanks for reminding | 13:26 |
SvenKieske | shortly afk.. | 13:26 |
mnasiadka | well, actually it's not that patch that broke it | 13:27 |
jovial | OK, I will following suit in Kayobe then to keep the test matrix smaller (and it makes sense if we are aligned to kolla). Was just checking that it wasn't an oversight. | 13:27 |
mnasiadka | jovial: no, I was thinking about that in the past, but it just doesn't make any sense if you can install python3.11/3.12 on Rocky 9 and get it working that way | 13:28 |
kevko | mnasiadka: it's actually this one https://review.opendev.org/c/openstack/kolla/+/924245 | 13:28 |
kevko | revert ? or what ? | 13:28 |
mnasiadka | what did it break? | 13:29 |
kevko | build rocky | 13:29 |
opendevreview | Merged openstack/kolla-ansible stable/2023.1: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926030 | 13:29 |
kevko | kolla-toolbox | 13:29 |
mnasiadka | where? | 13:29 |
mnasiadka | what's the error? | 13:29 |
kevko | mnasiadka: validating ..give me 5 minutes | 13:30 |
mnasiadka | kevko[21.st aug] https://review.opendev.org/q/owner:michal.arbet@ultimum.io+-status:abandoned+-status:merged+branch:master+(repo:openstack/kolla+OR+repo:openstack/kolla-ansible) | 13:30 |
mnasiadka | 5 minutes is too much | 13:30 |
mnasiadka | I'll leave it here - since kevko did not add any comments | 13:30 |
kevko | :D | 13:31 |
mnasiadka | #topic Open discussion | 13:31 |
mnasiadka | anybody anything? | 13:31 |
kevko | thank you very much | 13:31 |
SvenKieske | the usual begging for reviews ;) | 13:31 |
mmalchuk | orphaned backports | 13:32 |
mmalchuk | https://review.opendev.org/c/openstack/kolla-ansible/+/926186 | 13:32 |
mmalchuk | https://review.opendev.org/c/openstack/kolla-ansible/+/924573 | 13:32 |
mmalchuk | and also please review: | 13:32 |
SvenKieske | one question though, regarding my WIP prometheus config validation: https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 13:32 |
mmalchuk | https://review.opendev.org/c/openstack/kolla/+/926084 | 13:32 |
mmalchuk | https://review.opendev.org/c/openstack/kolla-ansible/+/926190 | 13:32 |
SvenKieske | would it be ok for a start to only validate the prometheus.yml inside the main prometheus-server container? because we have dozens of exporters and constructing the loop for that makes my head hurt :) | 13:33 |
mnasiadka | it's unmaintained, it should be orphaned | 13:33 |
mnasiadka | SvenKieske: I would say we don't care about exporters - that should be validated in the CI if we're able to scrape from them | 13:34 |
mmalchuk | mnasiadka not all | 13:34 |
SvenKieske | alright, so that will simplify the validation a lot, I'll just check the main prometheus server configs then :) | 13:34 |
mmalchuk | https://review.opendev.org/c/openstack/kolla-ansible/+/924573 | 13:34 |
mmalchuk | is the A | 13:34 |
mmalchuk | https://review.opendev.org/q/Ic8e2dd42b95a8f5c2141a820c55642a3ed7beabd | 13:35 |
mmalchuk | month ago merged all but A lost | 13:35 |
mnasiadka | bbezak_alt: can you have a look ^^? | 13:36 |
bbezak_alt | looking | 13:37 |
bbezak_alt | some already done | 13:37 |
mmalchuk | thanks a lot | 13:37 |
mmalchuk | still lack of Kayobe reviews :( | 13:38 |
mmalchuk | https://review.opendev.org/c/openstack/kayobe/+/921628 | 13:38 |
mmalchuk | 2 months( | 13:38 |
mnasiadka | jovial: ^^ | 13:38 |
mmalchuk | what about new core reviewers for Kolla/Kayobe ? outside the StackHPC ? | 13:38 |
mnasiadka | I'll leave that to jovial since he's more responsible for Kayobe | 13:40 |
mnasiadka | Ok, unfortunately I need to run - see you all next week! | 13:40 |
mnasiadka | #endmeeting | 13:40 |
opendevmeet | Meeting ended Wed Aug 21 13:40:35 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:40 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-08-21-13.00.html | 13:40 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-08-21-13.00.txt | 13:40 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-08-21-13.00.log.html | 13:40 |
SvenKieske | ty | 13:40 |
mmalchuk | thx mnasiadka | 13:41 |
mmalchuk | I will repeat the question next week again) | 13:41 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: service-python-path: Introduce a role to get python_path of an image https://review.opendev.org/c/openstack/kolla-ansible/+/926632 | 13:43 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: service-python-path: Introduce a role to get python_path of an image https://review.opendev.org/c/openstack/kolla-ansible/+/926632 | 13:43 |
jovial | mmalchuk, We did have an internal discussion amongst the current cores. We didn't think any candidates were quite ready to be promoted yet. We'd like to see +1's given out with a bit more discretion, more willingness to listen to review comments, and more in the way of code contributions (not just reviews). This will be reviewed at the next PTG. | 13:47 |
mmalchuk | jovial thanks | 13:48 |
opendevreview | Michal Arbet proposed openstack/kolla master: Remove KOLLA_DISTRO_PYTHON_VERSION https://review.opendev.org/c/openstack/kolla/+/926744 | 13:57 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 13:58 |
kevko | btw, it's funny that we have in kolla KOLLA_DISTRO_PYTHON_VERSION .. but if it is not working as it is defined in a code ... vuala ..just someone create if conditional and install newer python :D :D :D :D | 14:02 |
opendevreview | Merged openstack/kolla-ansible stable/2023.2: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926029 | 14:08 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 14:24 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: Remove unnecessary dependency https://review.opendev.org/c/openstack/kayobe/+/926664 | 14:38 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: Remove unnecessary dependency https://review.opendev.org/c/openstack/kayobe/+/926664 | 14:39 |
opendevreview | Merged openstack/kolla-ansible master: Fix venus when proxysql used https://review.opendev.org/c/openstack/kolla-ansible/+/920297 | 15:02 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926028 | 15:03 |
opendevreview | Merged openstack/kolla-ansible stable/2023.1: Fix the docker container dimensions comparison for short notation https://review.opendev.org/c/openstack/kolla-ansible/+/924573 | 15:03 |
opendevreview | Merged openstack/kolla master: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926084 | 15:10 |
opendevreview | Maksim Malchuk proposed openstack/kolla stable/2024.1: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926795 | 15:13 |
opendevreview | Maksim Malchuk proposed openstack/kolla stable/2023.2: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926796 | 15:13 |
opendevreview | Maksim Malchuk proposed openstack/kolla stable/2023.1: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926797 | 15:13 |
opendevreview | Verification of a change to openstack/kolla-ansible master failed: Fix issue with Swift Recon middleware https://review.opendev.org/c/openstack/kolla-ansible/+/926190 | 15:23 |
opendevreview | Will Szumski proposed openstack/kayobe master: Aligns Kayobe Ansible version with Kolla Ansible https://review.opendev.org/c/openstack/kayobe/+/926639 | 16:02 |
opendevreview | Merged openstack/kayobe master: Add missing "cloud-init" as "cloud-init-datasource" don't have dependency on it. https://review.opendev.org/c/openstack/kayobe/+/922795 | 16:06 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 16:25 |
mnasiadka | kevko: distro python version in kolla is totally different thing, and it's not making it complicated to introduce new distro versions - I'm more focused on the kolla-ansible side. | 16:43 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Remove support for really old Docker versions https://review.opendev.org/c/openstack/kolla-ansible/+/926827 | 17:16 |
opendevreview | Merged openstack/kolla master: Add exclude for permissions in kolla config.json file https://review.opendev.org/c/openstack/kolla/+/655620 | 17:18 |
opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: Fix prometheus.yml templating https://review.opendev.org/c/openstack/kolla-ansible/+/926618 | 17:21 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Add handling for bytes literals in docker https://review.opendev.org/c/openstack/kolla-ansible/+/926831 | 17:31 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Remove support for really old Docker versions https://review.opendev.org/c/openstack/kolla-ansible/+/926827 | 17:33 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Add handling for bytes literals in docker https://review.opendev.org/c/openstack/kolla-ansible/+/926831 | 17:33 |
opendevreview | Maksim Malchuk proposed openstack/kayobe master: Add initial support for systemd-networkd link configuration https://review.opendev.org/c/openstack/kayobe/+/921628 | 17:38 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Remove support for really old Docker versions https://review.opendev.org/c/openstack/kolla-ansible/+/926827 | 17:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Add handling for bytes literals in docker https://review.opendev.org/c/openstack/kolla-ansible/+/926831 | 17:40 |
opendevreview | Maksim Malchuk proposed openstack/kayobe master: Add initial support for systemd-networkd link configuration https://review.opendev.org/c/openstack/kayobe/+/921628 | 18:29 |
opendevreview | Maksim Malchuk proposed openstack/kayobe master: Add initial support for systemd-networkd link configuration https://review.opendev.org/c/openstack/kayobe/+/921628 | 19:18 |
samcat116 | Is there any documentation about how tags vs stable branches pull in containers? I have a deployment script where I pinned k-a to 17.0.0, however now doing a deploy with that pulls in images based off of master. whereas if I pin stable/2023.2 it keeps the images in that series. | 19:41 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Unify of cron and crond commands https://review.opendev.org/c/openstack/kolla-ansible/+/926849 | 20:23 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Unify of cron and crond commands https://review.opendev.org/c/openstack/kolla-ansible/+/926849 | 20:29 |
opendevreview | Verification of a change to openstack/kolla stable/2024.1 failed: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926795 | 21:39 |
opendevreview | Merged openstack/kolla stable/2023.2: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926796 | 21:44 |
opendevreview | Merged openstack/kolla-ansible master: Fix issue with Swift Recon middleware https://review.opendev.org/c/openstack/kolla-ansible/+/926190 | 21:55 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/2024.1: Fix issue with Swift Recon middleware https://review.opendev.org/c/openstack/kolla-ansible/+/926854 | 22:05 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/2023.2: Fix issue with Swift Recon middleware https://review.opendev.org/c/openstack/kolla-ansible/+/926856 | 22:05 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/2023.1: Fix issue with Swift Recon middleware https://review.opendev.org/c/openstack/kolla-ansible/+/926857 | 22:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!