Wednesday, 2024-08-21

r-krcekHi! 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-krcekI 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
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Fix upgrade jobs failing on adding new certificates  https://review.opendev.org/c/openstack/kolla-ansible/+/92628406:58
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: CI: Fix upgrade jobs failing on adding new certificates  https://review.opendev.org/c/openstack/kolla-ansible/+/92628407:00
opendevreviewMagnus Lööf proposed openstack/kolla-ansible master: Enable TLS backend for designate  https://review.opendev.org/c/openstack/kolla-ansible/+/86652407:19
mnasiadkakevko: 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
SvenKiesker-krcek: I think a separate command is needed in any case you want to specifically manually start a recovery, no?07:58
mnasiadkaAnd as mentioned in the TC meeting - it's a PTG discussion.07:58
kevkomnasiadka: that's not about build and publish ... it's about to land the code how to build ...it's up to user ..08:01
kevkomnasiadka: 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
mnasiadkaNo thanks, Ceph is the same situation - we support External Ceph - why not support External Consul?08:02
kevkoit's easier 08:05
kevkofoundation legal counsel has said that openstack software interfacing with bsl software doesn't present license incompatibilities   .... so why ? 08:07
mnasiadkaAs mentioned before - it's a PTG discussion - because we can then discuss it in proper forum.08:07
kevkototally agree08:07
kevkoi am not trying to push some code fast ... i will be patient and will follow the discussion 08:08
kevkobut it's proper question according to latest various companies just half closed their codes ...08:08
kevkoelastic, redis, vault, consul ...what will be next ? rabbit ? :D 08:09
mnasiadkawe jumped ship with elastic, we never packaged vault and consul, and yes - we need to move to valkey08:09
opendevreviewMichal 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/+/92663208:11
opendevreviewMichal 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/+/92663208:11
opendevreviewMichal 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/+/92663208:17
r-krcekSvenKieske: 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
SvenKieskesure! I just wanted to say we still need to be able to trigger this manually, especially for staying backward compatible08:25
mnasiadkaand for not running whole deploy just to get mariadb behave properly08:26
mnasiadkaand still mariadb_recovery could fail08:26
r-krcekI see, yeah, backwards compatibility should definitely stay there.08:28
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Remove "distutils" library  https://review.opendev.org/c/openstack/kolla-ansible/+/85700308:52
opendevreviewMark Goddard proposed openstack/kolla master: Pin OpenSearch Dashboards to 2.15  https://review.opendev.org/c/openstack/kolla/+/92652808:53
SvenKieskecan 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/+/92261808:56
opendevreviewMichal 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/+/92663208:57
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Move actions to kolla_container_facts  https://review.opendev.org/c/openstack/kolla-ansible/+/91150509:02
mnasiadkaSvenKieske: https://review.opendev.org/c/openstack/kolla-ansible/+/922617 - this needs to go in first09:07
Viiwill 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-vpn09:12
mnasiadkaVii: there is unresolved conversation on first of them09:14
Viimnasiadka: 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
mnasiadkaVii: you're more interested than me, so feel free to ping the owner :)09:16
Vii:)09:16
opendevreviewMichal 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/+/92663209:28
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Implement TLS for Redis  https://review.opendev.org/c/openstack/kolla-ansible/+/90918809:46
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Add Redis as caching backend for Ceilometer  https://review.opendev.org/c/openstack/kolla-ansible/+/92672109:46
opendevreviewMerged openstack/kolla-ansible master: Add support for customising Neutron physical network names  https://review.opendev.org/c/openstack/kolla-ansible/+/92232009:54
opendevreviewBartosz 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/+/92602810:25
opendevreviewBartosz 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/+/92602810:26
opendevreviewBartosz 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/+/92602910:27
opendevreviewBartosz 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/+/92602910:28
opendevreviewBartosz 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/+/92603010:29
opendevreviewBartosz 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/+/92603010:30
opendevreviewMerged openstack/kolla-ansible master: reorder requirements.txt alphabetically  https://review.opendev.org/c/openstack/kolla-ansible/+/92261710:31
opendevreviewMichal 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/+/92663210:31
opendevreviewMichal 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/+/92663211:09
opendevreviewMagnus Lööf proposed openstack/kolla-ansible master: Enable TLS backend for designate  https://review.opendev.org/c/openstack/kolla-ansible/+/86652411:09
opendevreviewMerged openstack/kolla-ansible master: fact gathering: Remove gathering of package facts  https://review.opendev.org/c/openstack/kolla-ansible/+/90105611:22
opendevreviewMerged openstack/kolla-ansible master: fact gathering: Group hosts before gathering facts  https://review.opendev.org/c/openstack/kolla-ansible/+/89959211:22
opendevreviewMerged openstack/kayobe master: Bump Ansible version installed in Kolla Ansible virtualenv  https://review.opendev.org/c/openstack/kayobe/+/92355611:26
opendevreviewWill Szumski proposed openstack/kayobe master: Aligns Kayobe Ansible version with Kolla Ansible  https://review.opendev.org/c/openstack/kayobe/+/92663911:31
opendevreviewWill Szumski proposed openstack/kayobe master: Remove unnecessary dependency  https://review.opendev.org/c/openstack/kayobe/+/92666411:54
opendevreviewWill Szumski proposed openstack/kayobe master: Remove unnecessary dependency  https://review.opendev.org/c/openstack/kayobe/+/92666411:55
opendevreviewMichal Arbet proposed openstack/kolla master: Remove KOLLA_DISTRO_PYTHON_VERSION  https://review.opendev.org/c/openstack/kolla/+/92674412:08
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Remove distro_python_version  https://review.opendev.org/c/openstack/kolla-ansible/+/92674512:18
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2024.1: fact gathering: Remove gathering of package facts  https://review.opendev.org/c/openstack/kolla-ansible/+/92674612:24
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2023.2: fact gathering: Remove gathering of package facts  https://review.opendev.org/c/openstack/kolla-ansible/+/92674912:33
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2023.1: fact gathering: Remove gathering of package facts  https://review.opendev.org/c/openstack/kolla-ansible/+/92675012:34
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2024.1: fact gathering: Group hosts before gathering facts  https://review.opendev.org/c/openstack/kolla-ansible/+/92675112:35
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2023.2: fact gathering: Group hosts before gathering facts  https://review.opendev.org/c/openstack/kolla-ansible/+/92675212:35
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2023.1: fact gathering: Group hosts before gathering facts  https://review.opendev.org/c/openstack/kolla-ansible/+/92675312:36
opendevreviewJakub Darmach proposed openstack/kolla master: Add exclude for permissions in kolla config.json file  https://review.opendev.org/c/openstack/kolla/+/65562012:43
opendevreviewMerged openstack/kolla-ansible master: Apply public firewalld rules immediately  https://review.opendev.org/c/openstack/kolla-ansible/+/88436512:47
opendevreviewMerged openstack/kolla-ansible stable/2023.2: Support CloudKitty deployment with internal TLS  https://review.opendev.org/c/openstack/kolla-ansible/+/92464612:48
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Support CloudKitty deployment with internal TLS  https://review.opendev.org/c/openstack/kolla-ansible/+/92464712:48
mnasiadkamgoddard mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu darmach - meeting in 6 minutes12:54
opendevreviewMichal Arbet proposed openstack/kolla master: Remove KOLLA_DISTRO_PYTHON_VERSION  https://review.opendev.org/c/openstack/kolla/+/92674412:55
mnasiadka#startmeeting kolla13:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:00
opendevmeetThe meeting name has been set to 'kolla'13:00
mnasiadka#topic rollcall13:00
mnasiadkao/13:00
SvenKieskeo/13:00
mmalchuko/13:00
mattcreeso/13:00
bbezak_alto/13:00
IvanVnuko[m]o/13:00
mhinero/13:00
frickler\o13:00
r-krceko/13:01
mnasiadka#topic agenda13:02
mnasiadka* Announcements13:02
mnasiadka* CI status13:02
mnasiadka* Current cycle planning13:02
mnasiadka* Additional agenda (from whiteboard)13:02
mnasiadka* Open discussion13:02
mnasiadka#topic Announcements13:02
jovialo/13:02
darmach6o/13:02
mnasiadkaI'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/+/92628513:02
SvenKieske\o/13:02
mnasiadka#topic CI status13:03
mnasiadkaI haven't seen any obvious breakages, so I guess nothing to report here ;-)13:03
mnasiadka#topic Current cycle planning13:03
jovialKayobe CI back to green as of this morning13:03
mnasiadkaansible bump?13:03
jovialIndeed13:03
mnasiadkaOk, let's have a look on planned ''features'' this cycle13:04
mnasiadkaUbuntu Noble - that's obvious, nearly getting there13:04
mnasiadkaOVN BGP agent - I haven't done anything - bbezak_alt - do you have some spare cycles?13:04
bbezak_altyeah, will revive that13:05
mnasiadkagreat13:05
mnasiadkaOpeninfra mirrors for some problematic packages - SvenKieske how's it going?13:05
SvenKieskemhm 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
opendevreviewMerged openstack/kolla-ansible stable/2024.1: CI: drop RMQ reconfigure step in queue migrations  https://review.opendev.org/c/openstack/kolla-ansible/+/92612713:06
mnasiadkayeah, I think we wanted to discuss with openinfra - but would be good to have a list what we want to mirror and some capacity calculations13:07
mnasiadkaso if you could update that on whiteboard - would be nice13:07
mnasiadkaMultiple RMQ versions per cycle  - mattcrees how's it going?13:07
SvenKieskefuture 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 back13:07
SvenKieskeokay, I can certainly compile a list of currently external repositories :)13:07
mattcreesWe'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
mnasiadkaOk, I'll have a look into those later13:09
mattcreesCheers 13:09
mnasiadkaif other cores can do the same - it would be nice13:09
mnasiadkaok, I added that topic to whiteboard13:09
mnasiadkaI don't have anything more in the whitebaord, so let's move on13:10
mnasiadka#topic Additional agenda (from whiteboard)13:10
mnasiadkar-krcek [21th August] my wishlist :)13:10
mnasiadkaplease review https://review.opendev.org/c/openstack/kolla-ansible/+/925714 and https://review.opendev.org/c/openstack/kolla/+/92571213:10
mnasiadkaThis got us stumped with Sven, anyone with bandit knowledge? https://review.opendev.org/c/openstack/kolla-ansible/+/92311013:10
mnasiadkaAny feedback would be greatly appreciated https://review.opendev.org/c/openstack/kolla-ansible/+/92465113:10
SvenKieskeah, 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
kevkobash to python ? it will be fun :D 13:12
SvenKieskeok I'm wrong, it's called twice.. maybe just use subprocess directly without a wrapper function? I mean it's two calls13:12
SvenKieskenot really worth the wrapper imho, would get rid of the bandit warning I guess13:13
mnasiadkaok, the first two look heavy, second one also is not a light one - I added my review on the last one13:13
mnasiadkabbezak_alt: willing to have a look in the last one as well?13:13
mnasiadkaSvenKieske: let's discuss in the patch, shall we?13:14
SvenKieskefine :)13:14
bbezak_alt924651 that one?13:14
r-krcekThank you guys for the feedback :) 13:14
bbezak_altwill check13:14
mnasiadkabbezak_alt: yup13:14
mnasiadkaok, next one13:14
mnasiadkaivnucko [21th August]13:14
mnasiadkaadding TLS for other than internal/external and backend (proxy to service API)13:14
mnasiadkalike galera replication - https://review.opendev.org/c/openstack/kolla-ansible/+/925317 or 13:14
mnasiadkarabbitmq internode - https://review.opendev.org/c/openstack/kolla-ansible/+/92138113: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
mnasiadkais 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
mnasiadkaI don't think it's worth any complication - I would handle them as part of backend13:15
mnasiadkaif anybody needs anything more sophisticated in future - he can add that functionality in a separate patch?13:16
mnasiadkaAny opinions?13:16
IvanVnuko[m]It's mainly about galera, rabbitmq needs distinct certificates for it uses names13: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
SvenKieskeagreed13:17
IvanVnuko[m]and a bit of promotion for the TLS encryption change requests we have put up:)13:17
SvenKieskeI would use the same certs for a start where possible.13:17
IvanVnuko[m]thank you13:17
SvenKieskewe 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
mnasiadkaok, then agreed - keep it simple as possible13:19
mnasiadkanext one13:19
mnasiadkajovial [21st] Should we lower minimum ansible as we should technically support python 3.9 in 2024.213:19
mnasiadkahttps://governance.openstack.org/tc/reference/runtimes/2024.2.html13:19
mnasiadka2.15.x is last version support on 3.913:19
mnasiadkaAlso affects kayobe13:19
jovialAm I right in thinking we should be supporting 3.9 still?13:20
mnasiadkawe are supporting 3.913:20
fricklercan't you use 3.11 on rocky?13:20
mnasiadkaon the destination hosts13:20
mnasiadkayou can use 3.11 or 3.1213:20
mnasiadkamy problem with relaxing the requirement is that we would need to have that running in CI13:20
mnasiadkaso that's a bit more jobs that we currently have13:20
mnasiadkaand we never supported more than two versions per release13:21
jovialJust wondering if I should follow suit in Kayobe and drop 3.9 really13:22
SvenKieskemhm, 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
mnasiadkawell, as I mentioned - Ansible requires 3.10+ only on the host where you're running Ansible13:22
frickleryes, I don't think the PTI applies in this case13:23
mnasiadkaPython 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
mnasiadkaagainst Python versions above is to ensure language compatibility, having more extensive testing is allowed.13:23
mnasiadkanot reminding that the PTI claims that you need to have passing unit tests on Py3.9 ;-)13:23
mnasiadkaand nothing more13:24
fricklerIMO kolla isn't a python project in the sense of that whole document13:24
mnasiadkawell, kolla-ansible more likely13:24
fricklerkolla* even13:24
mnasiadkaI don't think we should add additional work of supporting older Ansible release13:25
frickler+113:25
SvenKieske+113:25
kevkohmm, are u sure that https://review.opendev.org/c/openstack/kolla-ansible/+/921743 didn't break anything ? 13:25
mnasiadkaAnd for those sticking to SLURP - they will still need to upgrade to Rocky 10 before jumping from 2024.1 to 2025.113:25
mnasiadkakevko: I'll fix the rocky upgrade - missed that, but thanks for reminding13:26
SvenKieskeshortly afk..13:26
mnasiadkawell, actually it's not that patch that broke it13:27
jovialOK, 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
mnasiadkajovial: 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 way13:28
kevkomnasiadka: it's actually this one https://review.opendev.org/c/openstack/kolla/+/92424513:28
kevkorevert ? or what ? 13:28
mnasiadkawhat did it break?13:29
kevkobuild rocky 13:29
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Fix post-config of OVS for manila-share servers  https://review.opendev.org/c/openstack/kolla-ansible/+/92603013:29
kevkokolla-toolbox 13:29
mnasiadkawhere?13:29
mnasiadkawhat's the error?13:29
kevkomnasiadka: validating ..give me 5 minutes 13:30
mnasiadkakevko[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
mnasiadka5 minutes is too much13:30
mnasiadkaI'll leave it here - since kevko did not add any comments13:30
kevko:D13:31
mnasiadka#topic Open discussion13:31
mnasiadkaanybody anything?13:31
kevkothank you very much 13:31
SvenKieskethe usual begging for reviews ;)13:31
mmalchukorphaned backports13:32
mmalchukhttps://review.opendev.org/c/openstack/kolla-ansible/+/92618613:32
mmalchukhttps://review.opendev.org/c/openstack/kolla-ansible/+/92457313:32
mmalchukand also please review:13:32
SvenKieskeone question though, regarding my WIP prometheus config validation: https://review.opendev.org/c/openstack/kolla-ansible/+/92620913:32
mmalchukhttps://review.opendev.org/c/openstack/kolla/+/92608413:32
mmalchukhttps://review.opendev.org/c/openstack/kolla-ansible/+/92619013:32
SvenKieskewould 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
mnasiadkait's unmaintained, it should be orphaned13:33
mnasiadkaSvenKieske: I would say we don't care about exporters - that should be validated in the CI if we're able to scrape from them13:34
mmalchukmnasiadka not all13:34
SvenKieskealright, so that will simplify the validation a lot, I'll just check the main prometheus server configs then :)13:34
mmalchukhttps://review.opendev.org/c/openstack/kolla-ansible/+/92457313:34
mmalchukis the A13:34
mmalchukhttps://review.opendev.org/q/Ic8e2dd42b95a8f5c2141a820c55642a3ed7beabd13:35
mmalchukmonth ago merged all but A lost13:35
mnasiadkabbezak_alt: can you have a look ^^?13:36
bbezak_altlooking13:37
bbezak_altsome already done13:37
mmalchukthanks a lot13:37
mmalchukstill lack of Kayobe reviews :(13:38
mmalchukhttps://review.opendev.org/c/openstack/kayobe/+/92162813:38
mmalchuk2 months(13:38
mnasiadkajovial: ^^13:38
mmalchukwhat about new core reviewers for Kolla/Kayobe ? outside the StackHPC ?13:38
mnasiadkaI'll leave that to jovial since he's more responsible for Kayobe13:40
mnasiadkaOk, unfortunately I need to run - see you all next week!13:40
mnasiadka#endmeeting13:40
opendevmeetMeeting ended Wed Aug 21 13:40:35 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:40
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-08-21-13.00.html13:40
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-08-21-13.00.txt13:40
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-08-21-13.00.log.html13:40
SvenKieskety13:40
mmalchukthx mnasiadka 13:41
mmalchukI will repeat the question next week again)13:41
opendevreviewMichal 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/+/92663213:43
opendevreviewMichal 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/+/92663213:43
jovialmmalchuk, 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
mmalchukjovial thanks13:48
opendevreviewMichal Arbet proposed openstack/kolla master: Remove KOLLA_DISTRO_PYTHON_VERSION  https://review.opendev.org/c/openstack/kolla/+/92674413:57
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620913:58
kevkobtw, 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
opendevreviewMerged openstack/kolla-ansible stable/2023.2: Fix post-config of OVS for manila-share servers  https://review.opendev.org/c/openstack/kolla-ansible/+/92602914:08
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620914:24
opendevreviewPierre Riteau proposed openstack/kayobe master: Remove unnecessary dependency  https://review.opendev.org/c/openstack/kayobe/+/92666414:38
opendevreviewPierre Riteau proposed openstack/kayobe master: Remove unnecessary dependency  https://review.opendev.org/c/openstack/kayobe/+/92666414:39
opendevreviewMerged openstack/kolla-ansible master: Fix venus when proxysql used  https://review.opendev.org/c/openstack/kolla-ansible/+/92029715:02
opendevreviewMerged openstack/kolla-ansible stable/2024.1: Fix post-config of OVS for manila-share servers  https://review.opendev.org/c/openstack/kolla-ansible/+/92602815:03
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Fix the docker container dimensions comparison for short notation  https://review.opendev.org/c/openstack/kolla-ansible/+/92457315:03
opendevreviewMerged openstack/kolla master: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92608415:10
opendevreviewMaksim Malchuk proposed openstack/kolla stable/2024.1: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679515:13
opendevreviewMaksim Malchuk proposed openstack/kolla stable/2023.2: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679615:13
opendevreviewMaksim Malchuk proposed openstack/kolla stable/2023.1: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679715:13
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Fix issue with Swift Recon middleware  https://review.opendev.org/c/openstack/kolla-ansible/+/92619015:23
opendevreviewWill Szumski proposed openstack/kayobe master: Aligns Kayobe Ansible version with Kolla Ansible  https://review.opendev.org/c/openstack/kayobe/+/92663916:02
opendevreviewMerged openstack/kayobe master: Add missing "cloud-init" as "cloud-init-datasource" don't have dependency on it.  https://review.opendev.org/c/openstack/kayobe/+/92279516:06
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620916:25
mnasiadkakevko: 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
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Remove support for really old Docker versions  https://review.opendev.org/c/openstack/kolla-ansible/+/92682717:16
opendevreviewMerged openstack/kolla master: Add exclude for permissions in kolla config.json file  https://review.opendev.org/c/openstack/kolla/+/65562017:18
opendevreviewVerification of a change to openstack/kolla-ansible stable/2024.1 failed: Fix prometheus.yml templating  https://review.opendev.org/c/openstack/kolla-ansible/+/92661817:21
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Add handling for bytes literals in docker  https://review.opendev.org/c/openstack/kolla-ansible/+/92683117:31
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Remove support for really old Docker versions  https://review.opendev.org/c/openstack/kolla-ansible/+/92682717:33
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Add handling for bytes literals in docker  https://review.opendev.org/c/openstack/kolla-ansible/+/92683117:33
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Add initial support for systemd-networkd link configuration  https://review.opendev.org/c/openstack/kayobe/+/92162817:38
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Remove support for really old Docker versions  https://review.opendev.org/c/openstack/kolla-ansible/+/92682717:40
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Add handling for bytes literals in docker  https://review.opendev.org/c/openstack/kolla-ansible/+/92683117:40
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Add initial support for systemd-networkd link configuration  https://review.opendev.org/c/openstack/kayobe/+/92162818:29
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Add initial support for systemd-networkd link configuration  https://review.opendev.org/c/openstack/kayobe/+/92162819:18
samcat116Is 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
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Unify of cron and crond commands  https://review.opendev.org/c/openstack/kolla-ansible/+/92684920:23
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: [DNM] Unify of cron and crond commands  https://review.opendev.org/c/openstack/kolla-ansible/+/92684920:29
opendevreviewVerification of a change to openstack/kolla stable/2024.1 failed: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679521:39
opendevreviewMerged openstack/kolla stable/2023.2: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679621:44
opendevreviewMerged openstack/kolla-ansible master: Fix issue with Swift Recon middleware  https://review.opendev.org/c/openstack/kolla-ansible/+/92619021:55
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible stable/2024.1: Fix issue with Swift Recon middleware  https://review.opendev.org/c/openstack/kolla-ansible/+/92685422:05
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible stable/2023.2: Fix issue with Swift Recon middleware  https://review.opendev.org/c/openstack/kolla-ansible/+/92685622:05
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible stable/2023.1: Fix issue with Swift Recon middleware  https://review.opendev.org/c/openstack/kolla-ansible/+/92685722:06

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!