Wednesday, 2025-08-06

opendevreviewMichal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645606:39
opendevreviewMichal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645606:56
mikalzigo: is anyone else reporting errors from osbpo? I see this when building Kolla images -- 07:38
mikalAug 06 06:41:31 2025.1 ERROR:kolla.common.utils.kolla-toolbox:E: Failed to fetch http://osbpo.debian.net/debian/pool/bookworm-dalmatian-backports-nochange/main/s/setuptools/python3-setuptools-whl_68.1.2-2%7ebpo12%2b1_all.deb  503  Service Unavailable07:38
mikalI am also unsure if its a bug that an epoxy Kolla image build is using the dalmatian backports, but that's a tangent.07:39
zigomikal: Let me have a look.07:53
zigoWorks for me ...07:54
mikalYeah, its intermittent for me. It is however regular enough that its caused every attempt to build the epoxy container images to fail for me today.07:54
mikalAs an aside, would you consider adding an osbpo module to rsync on mirror.infomaniak.ch? I'd then just run a local mirror for my CI jobs.07:55
zigozigo@k:zbuz>_ /tmp/blah $ wcurl http://osbpo.debian.net/debian/pool/bookworm-dalmatian-backports-nochange/main/s/setuptools/python3-setuptools-whl_68.1.2-2~bpo12+1_all.deb07:55
zigo  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current07:55
zigo                                 Dload  Upload   Total   Spent    Left  Speed07:55
zigo100  778k  100  778k    0     0  60.1M      0 --:--:-- --:--:-- --:--:-- 63.3M07:55
zigomikal: You mean like this one? http://mirror.infomaniak.com/osbpo/07:55
zigo:)07:55
mikalThat's not an rsync module though?07:56
zigoAlso, that repo is mirrored by wikimedia foundation somewhere.07:56
mikalroot@test:~/debian# rsync rsync://mirror.infomaniak.ch07:56
mikaldebian         The Debian Archive (https://www.debian.org/mirror/size)07:56
mikaldebian-cd      The Debian Archive (https://www.debian.org/mirror/size)07:56
mikaldebian-archive The Debian Archive (https://www.debian.org/mirror/size)07:56
mikaldebian-securityThe Debian Security Archive07:56
mikalubuntu         The Debian Archive (https://www.debian.org/mirror/size)07:56
zigoosbpo itself does have an rsync service.07:57
zigoUnder osbpo as name.07:57
opendevreviewMichal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645607:57
mikalzigo: ahhh cool. I didn't realise. I'll just use that then.07:57
opendevreviewMichal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645607:59
opendevreviewMichal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645607:59
zigoLet me know if anything ...08:01
opendevreviewMichal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645608:11
opendevreviewMichal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645608:11
opendevreviewMichal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645608:30
opendevreviewMichal Nasiadka proposed openstack/kolla master: debian: Use {{ openstack_release }} for extrepo  https://review.opendev.org/c/openstack/kolla/+/95663408:36
opendevreviewAmir Hossein Ahmadi proposed openstack/kolla master: swift: Reintroduce support for Swift deployment  https://review.opendev.org/c/openstack/kolla/+/95053408:36
opendevreviewAmir Hossein Ahmadi proposed openstack/kolla-ansible master: swift: reintroduce Swift role and restore deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/95357608:41
opendevreviewMichal Nasiadka proposed openstack/kolla stable/2025.1: debian: Use Epoxy extrepo  https://review.opendev.org/c/openstack/kolla/+/95664108:58
opendevreviewMichal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources"  https://review.opendev.org/c/openstack/kolla/+/94975509:01
opendevreviewBartosz Bezak proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645609:11
opendevreviewBartosz Bezak proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645609:50
opendevreviewMichal Nasiadka proposed openstack/kolla master: ironic: Stop copying rootwrap since Ironic dropped it  https://review.opendev.org/c/openstack/kolla/+/95664809:57
opendevreviewMichal Nasiadka proposed openstack/kolla master: ironic: Stop copying rootwrap since Ironic dropped it  https://review.opendev.org/c/openstack/kolla/+/95664810:02
baGood afternoon everyone10:21
baI have a problem when deploying Ironic with ansible kolla, as it does not deploy ironic-inspector (due to deprecation), and at the same time does not configure In-Band Inspection correctly.10:21
baAm I doing something wrong? or is it a bug?10:21
baTo make it work. I needed to enable In-Band Inspection, and change PXE boot params to point to it. The default PXE boot params points to a non-existing API endpoint (port 5050) which is used by ironic-inspector10:23
fricklerba: feel free to submit a bug report or even a fix11:19
baI was making sure that it was not a misconfig from my side before opening a bug report :))11:20
baI will try to open the bug report and submit a fix asap11:20
opendevreviewPierre Riteau proposed openstack/kayobe master: public-openrc: avoid duplicate OS_CACERT  https://review.opendev.org/c/openstack/kayobe/+/95459311:44
bbezakmeeting in 8 mnasiadka bbezak frickler kevko mmalchuk gkoper jovial mattcrees dougszu darmach pabloclsn ravlew amir58118 r-krcek12:52
bbezak#startmeeting kolla13:00
opendevmeetMeeting started Wed Aug  6 13:00:33 2025 UTC and is due to finish in 60 minutes.  The chair is bbezak. 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
bbezak#topic rollcall13:00
jovialo/13:00
amir58118o/13:00
bbezak#topic agenda13:02
bbezak_2* Roll-call * Agenda * Announcements * Review action items from the last meeting * CI status * Release tasks * Regular stable releases (first meeting in a month) * Current cycle planning * Additional agenda (from whiteboard) * Open discussion13:02
bbezak#topic CI status13:02
bbezakKolla and kolla-ansible with depends on to kolla are red13:03
bbezakwaiting for https://review.opendev.org/c/openstack/kolla/+/95645613:03
mmalchuk o/13:03
bbezakhopefully we will merge that today13:04
bbezak#topic Regular stable releases (first meeting in a month)13:04
bbezakI'll raise new point releases when we will finish this CI drama13:04
bbezakthx for everybody who helped with merging/fixing CI issues in recent weeks13:05
amir58118+113:05
bbezak#topic Current cycle planning13:05
bbezakI probably will look into bumping mariadb soon13:06
bbezakas it is long overdue13:06
bbezaklooking at Valkey support - I will chase gkoper13:07
bbezakmnasiadka: surely continues his uwsgi crusade shortly13:07
bbezakrecent docker/kolla issues is showing that we should pursue other options - I saw some patches already. and it will ease doing a multiarch containers as well13:09
bbezakbut that is different story13:09
bbezak#topic Open discussion13:10
amir58118The patches to restore Swift support in Kolla and Kolla-ansible are ready for review after about 100 hours of work. The core implementation is complete and available for feedback and structural review. https://review.opendev.org/q/topic:%22dev-swift-modernization%2213:10
mmalchukbbezak shouldn't we pin docker version for the deploy like we pin docker-py in https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L537 ?13:11
seunghunleeIt would be nice we can review the process of https://bugs.launchpad.net/kolla-ansible/+bug/208040213:11
fricklerlate o/13:12
mmalchukamir58118 did we run integration tests during deploy in your change in k-a ?13:13
bbezakexcuse me a bit as frickler just joined - we would need review for https://review.opendev.org/c/openstack/kolla/+/95645613:13
bbezakand interesting point mmalchuk - I was testing that and that indeed works - https://review.opendev.org/c/openstack/kolla-ansible/+/95636413:14
bbezakhowever that will not solve other people deployments13:14
bbezakas not all people are using ansible-collection-kolla bits to deploy hosts13:14
bbezakat least I don't think so13:14
frickleryes, we have our own docker pins downstream, so I'd rather try to avoid that13:15
mmalchukbbezak sure! we pin python libs, but use latest os packages - that's not good)13:15
fricklerI'll look at the other fix today13:15
bbezakwell, kolla-ansible is not about host OS13:15
bbezakkayobe surely13:15
mmalchukyep. but we can mention version pin recomendations in docs and use them on CI13:16
amir58118mmalchuk: I'm not exactly sure, I need to check.13:16
bbezakthat is another bit to maintain13:16
mmalchukamir58118 ok. we can continue to discuss in the review13:17
bbezakdocker-py should fix it, but I didn't see traction there13:17
bbezakso we're fixing it in kolla for the time being13:17
mmalchukok. but it'll work until the next version with a new issue)))13:18
bbezakthat is also true :)13:19
mnasiadkasorry for being late13:19
fungioh, thought you were still on vacation13:19
bbezakkolla assumes stable docker engine libraries. I think that is a good assumption13:19
bbezak(podman too)13:19
mnasiadkafungi: got back yesterday :)13:19
fungii considered asking whether there's been any more thoughts about the contributor/maintainer survey results and metrics discussion from last week, but wanted to give you time to catch up on the meeting log13:19
mmalchuksometime ago mnasiadka said something like 'We're putting out fires' on my comment like this)13:20
mnasiadkayeah, that's the life of a deployment project mainly ;-)13:20
mmalchuksad13:20
bbezakseunghunlee: could you please elaborate your point on https://bugs.launchpad.net/kolla-ansible/+bug/2080402 ?13:22
seunghunleeI don't think we should introduce new feature like this. The bug report is not explaining much and the review of the feature was not don thoroughly in my opinion. So,the feature ended up introducing a new bug https://bugs.launchpad.net/kolla-ansible/+bug/211934413:24
frickleriiuc https://review.opendev.org/c/openstack/kolla-ansible/+/956409 needs reviews13:24
seunghunleeThe feature doesn't provide default value or disable this new option by default. It was merged with the author's own +2. As a user, this looks rushed.13:25
bbezakI agree. I think this is general problem of lack of meaningful reviews13:27
bbezakas there are not many active reviewers13:27
mnasiadkaI think this can be said to hundreds of other patches13:27
mmalchukseunghunlee it was merged after +2 from mnasiadka 13:27
mnasiadkaBut it is what it is13:27
bbezakthere is last comment mentioning that it was a wrong way of doing things13:28
fricklerwe can't test everything in CI, so I'd say we need to live with occasional regressions13:28
mnasiadkawe don't have an OIDC scenario, so yes - we need to live with that13:28
fricklerthe question is whether a second reviewer would have had a better chance of spotting the regression, which may not be likely, too13:28
bbezaktrue. but I also understand frustration about such cases13:29
mnasiadkaBut I'm a bit worried about people writing what needs to be fixed in bug report, but not submitting the patch in Gerrit13:29
frickleriiuc the fix is submitted now, so we can just review it and move on?13:29
bbezakyeap13:30
seunghunleeI agree not everything can be tested through CI and also we're lacking reviews but this particular case looked bad, so I wanted to raise13:30
mmalchukthere is now good or wrong way to fix things. we should fix. lets merge one new fix)13:30
seunghunleeYeah I proposed a fix, so let's move on13:30
mmalchukthere is no*13:30
mmalchukseunghunlee good! thanks13:30
bbezakfungi: it looks like we need more time to analyze your points13:31
mnasiadkaseunghunlee: basically people historically have been putting a list of patches they need reviews on in the Kolla Whiteboard13:31
mnasiadkaAnd I liked that approach13:31
frickleryes, but even that has been difficult to get to recentish13:32
mnasiadkaseunghunlee: https://etherpad.opendev.org/p/KollaWhiteBoard#L66 - here13:33
seunghunleemnasiadka: Thanks13:33
bbezakand as usual if anybody have some change to review, and it feels stale. whiteboard is a good place, but IRC as well13:33
mnasiadkafrickler: well, I'd say it's mainly me and bbezak doing reviews, you probably have $other priorities and kevko seems to not be that active as in the past13:33
fricklermnasiadka: yes, I try to look at the important things, but more often than not do not get to it in time13:34
mnasiadkaWe don't really have candidates for more core reviewers, so it is what it is.13:35
opendevreviewSeunghun Lee proposed openstack/kolla-ansible master: Re-add missing permission attribute  https://review.opendev.org/c/openstack/kolla-ansible/+/95666713:35
seunghunleeThat's unfortunate13:36
bbezakok, good discussion today13:36
bbezakany other points to discuss?13:37
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Remove ironic-inspector deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/95186413:38
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Remove ironic-inspector deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/95186413:38
opendevreviewSeunghun Lee proposed openstack/kolla-ansible stable/2025.1: Re-add missing permission attribute  https://review.opendev.org/c/openstack/kolla-ansible/+/95666813:39
bbezakthank you all for lively chat today!13:39
bbezak#endmeeting13:39
opendevmeetMeeting ended Wed Aug  6 13:39:40 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:39
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-08-06-13.00.html13:39
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-08-06-13.00.txt13:39
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-08-06-13.00.log.html13:39
seunghunleeThank you13:40
fricklerthx bbezak 13:40
amir58118thanks13:40
mmalchukbbezak thanks13:42
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Remove ironic-inspector deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/95186413:43
opendevreviewMichal Nasiadka proposed openstack/kolla master: ironic: Stop copying rootwrap since Ironic dropped it  https://review.opendev.org/c/openstack/kolla/+/95664813:55
opendevreviewMichal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources"  https://review.opendev.org/c/openstack/kolla/+/94975513:56
opendevreviewMichal Nasiadka proposed openstack/kolla master: keystone: Drop copying of wsgi files  https://review.opendev.org/c/openstack/kolla/+/95667013:58
opendevreviewMichal Nasiadka proposed openstack/kolla master: keystone: Drop copying of wsgi files  https://review.opendev.org/c/openstack/kolla/+/95667014:00
opendevreviewMichal Nasiadka proposed openstack/kolla master: manila: Drop copying of wsgi files  https://review.opendev.org/c/openstack/kolla/+/95667114:01
opendevreviewMichal Nasiadka proposed openstack/kolla master: manila: Drop copying of wsgi files  https://review.opendev.org/c/openstack/kolla/+/95667114:02
opendevreviewSeunghun Lee proposed openstack/kolla-ansible master: Fix etcd config.json template  https://review.opendev.org/c/openstack/kolla-ansible/+/95667214:02
opendevreviewMichal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources"  https://review.opendev.org/c/openstack/kolla/+/94975514:02
opendevreviewMichal Nasiadka proposed openstack/kolla master: keystone: Drop copying of wsgi files  https://review.opendev.org/c/openstack/kolla/+/95667014:03
opendevreviewMichal Nasiadka proposed openstack/kolla master: manila: Drop copying of wsgi files  https://review.opendev.org/c/openstack/kolla/+/95667114:03
opendevreviewSeunghun Lee proposed openstack/kolla-ansible stable/2025.1: Fix etcd config.json template  https://review.opendev.org/c/openstack/kolla-ansible/+/95667314:03
opendevreviewMichal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources"  https://review.opendev.org/c/openstack/kolla/+/94975514:03
opendevreviewMichal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources"  https://review.opendev.org/c/openstack/kolla/+/94975514:03
opendevreviewSeunghun Lee proposed openstack/kolla-ansible stable/2025.1: Fix etcd config.json template  https://review.opendev.org/c/openstack/kolla-ansible/+/95667314:04
opendevreviewSeunghun Lee proposed openstack/kolla-ansible stable/2025.1: Re-add missing permission attribute  https://review.opendev.org/c/openstack/kolla-ansible/+/95666814:06
opendevreviewSeunghun Lee proposed openstack/kolla-ansible master: Only add OIDCXForwardedHeaders when variable is set  https://review.opendev.org/c/openstack/kolla-ansible/+/95640914:11
opendevreviewMichal Nasiadka proposed openstack/kolla master: keystone: Add local wsgi file  https://review.opendev.org/c/openstack/kolla/+/95667014:21
opendevreviewMichal Nasiadka proposed openstack/kolla master: keystone: Add local wsgi file  https://review.opendev.org/c/openstack/kolla/+/95667014:22
opendevreviewMichal Nasiadka proposed openstack/kolla master: keystone: Add local wsgi file  https://review.opendev.org/c/openstack/kolla/+/95667014:24
opendevreviewSeunghun Lee proposed openstack/kolla-ansible stable/2025.1: Re-add missing permission attribute  https://review.opendev.org/c/openstack/kolla-ansible/+/95666814:26
opendevreviewSeunghun Lee proposed openstack/kolla-ansible master: Fix etcd config.json template  https://review.opendev.org/c/openstack/kolla-ansible/+/95667214:27
opendevreviewSeunghun Lee proposed openstack/kolla-ansible stable/2025.1: Fix etcd config.json template  https://review.opendev.org/c/openstack/kolla-ansible/+/95667314:30
opendevreviewMerged openstack/kolla master: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95645615:13
opendevreviewMichal Nasiadka proposed openstack/kolla stable/2025.1: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95669115:14
opendevreviewMichal Nasiadka proposed openstack/kolla stable/2024.2: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95669215:14
opendevreviewMichal Nasiadka proposed openstack/kolla stable/2024.1: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95669315:15
opendevreviewMerged openstack/kayobe master: public-openrc: avoid duplicate OS_CACERT  https://review.opendev.org/c/openstack/kayobe/+/95459315:42
opendevreviewMerged openstack/kayobe master: use current debian version supported by kolla  https://review.opendev.org/c/openstack/kayobe/+/95648415:42
opendevreviewPierre Riteau proposed openstack/kayobe stable/2025.1: public-openrc: avoid duplicate OS_CACERT  https://review.opendev.org/c/openstack/kayobe/+/95670215:51
opendevreviewPierre Riteau proposed openstack/kayobe stable/2024.2: public-openrc: avoid duplicate OS_CACERT  https://review.opendev.org/c/openstack/kayobe/+/95670315:52
opendevreviewPierre Riteau proposed openstack/kayobe stable/2024.1: public-openrc: avoid duplicate OS_CACERT  https://review.opendev.org/c/openstack/kayobe/+/95670415:53
kevkomnasiadka: i will be active ...but I have long vacation now :) ... for now I am in bulgaria :) 15:57
mnasiadkakevko: have fun :)16:07
opendevreviewMerged openstack/kolla-ansible master: Only add OIDCXForwardedHeaders when variable is set  https://review.opendev.org/c/openstack/kolla-ansible/+/95640917:50
opendevreviewMerged openstack/kolla stable/2024.2: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95669218:30
opendevreviewMerged openstack/kolla stable/2024.1: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95669318:30
opendevreviewMerged openstack/kolla stable/2025.1: send empty X-Registry-Auth for anonymous pushes  https://review.opendev.org/c/openstack/kolla/+/95669119:22

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