opendevreview | Michal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 06:39 |
---|---|---|
opendevreview | Michal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 06:56 |
mikal | zigo: is anyone else reporting errors from osbpo? I see this when building Kolla images -- | 07:38 |
mikal | Aug 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 Unavailable | 07:38 |
mikal | I 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 |
zigo | mikal: Let me have a look. | 07:53 |
zigo | Works for me ... | 07:54 |
mikal | Yeah, 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 |
mikal | As 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 |
zigo | zigo@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.deb | 07:55 |
zigo | % Total % Received % Xferd Average Speed Time Time Time Current | 07:55 |
zigo | Dload Upload Total Spent Left Speed | 07:55 |
zigo | 100 778k 100 778k 0 0 60.1M 0 --:--:-- --:--:-- --:--:-- 63.3M | 07:55 |
zigo | mikal: You mean like this one? http://mirror.infomaniak.com/osbpo/ | 07:55 |
zigo | :) | 07:55 |
mikal | That's not an rsync module though? | 07:56 |
zigo | Also, that repo is mirrored by wikimedia foundation somewhere. | 07:56 |
mikal | root@test:~/debian# rsync rsync://mirror.infomaniak.ch | 07:56 |
mikal | debian The Debian Archive (https://www.debian.org/mirror/size) | 07:56 |
mikal | debian-cd The Debian Archive (https://www.debian.org/mirror/size) | 07:56 |
mikal | debian-archive The Debian Archive (https://www.debian.org/mirror/size) | 07:56 |
mikal | debian-securityThe Debian Security Archive | 07:56 |
mikal | ubuntu The Debian Archive (https://www.debian.org/mirror/size) | 07:56 |
zigo | osbpo itself does have an rsync service. | 07:57 |
zigo | Under osbpo as name. | 07:57 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 07:57 |
mikal | zigo: ahhh cool. I didn't realise. I'll just use that then. | 07:57 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 07:59 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 07:59 |
zigo | Let me know if anything ... | 08:01 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 08:11 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 08:11 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 08:30 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: debian: Use {{ openstack_release }} for extrepo https://review.opendev.org/c/openstack/kolla/+/956634 | 08:36 |
opendevreview | Amir Hossein Ahmadi proposed openstack/kolla master: swift: Reintroduce support for Swift deployment https://review.opendev.org/c/openstack/kolla/+/950534 | 08:36 |
opendevreview | Amir Hossein Ahmadi proposed openstack/kolla-ansible master: swift: reintroduce Swift role and restore deployment https://review.opendev.org/c/openstack/kolla-ansible/+/953576 | 08:41 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: debian: Use Epoxy extrepo https://review.opendev.org/c/openstack/kolla/+/956641 | 08:58 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources" https://review.opendev.org/c/openstack/kolla/+/949755 | 09:01 |
opendevreview | Bartosz Bezak proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 09:11 |
opendevreview | Bartosz Bezak proposed openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 09:50 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: ironic: Stop copying rootwrap since Ironic dropped it https://review.opendev.org/c/openstack/kolla/+/956648 | 09:57 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: ironic: Stop copying rootwrap since Ironic dropped it https://review.opendev.org/c/openstack/kolla/+/956648 | 10:02 |
ba | Good afternoon everyone | 10:21 |
ba | I 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 |
ba | Am I doing something wrong? or is it a bug? | 10:21 |
ba | To 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-inspector | 10:23 |
frickler | ba: feel free to submit a bug report or even a fix | 11:19 |
ba | I was making sure that it was not a misconfig from my side before opening a bug report :)) | 11:20 |
ba | I will try to open the bug report and submit a fix asap | 11:20 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: public-openrc: avoid duplicate OS_CACERT https://review.opendev.org/c/openstack/kayobe/+/954593 | 11:44 |
bbezak | meeting in 8 mnasiadka bbezak frickler kevko mmalchuk gkoper jovial mattcrees dougszu darmach pabloclsn ravlew amir58118 r-krcek | 12:52 |
bbezak | #startmeeting kolla | 13:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
opendevmeet | The meeting name has been set to 'kolla' | 13:00 |
bbezak | #topic rollcall | 13:00 |
jovial | o/ | 13:00 |
amir58118 | o/ | 13:00 |
bbezak | #topic agenda | 13: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 discussion | 13:02 |
bbezak | #topic CI status | 13:02 |
bbezak | Kolla and kolla-ansible with depends on to kolla are red | 13:03 |
bbezak | waiting for https://review.opendev.org/c/openstack/kolla/+/956456 | 13:03 |
mmalchuk | o/ | 13:03 |
bbezak | hopefully we will merge that today | 13:04 |
bbezak | #topic Regular stable releases (first meeting in a month) | 13:04 |
bbezak | I'll raise new point releases when we will finish this CI drama | 13:04 |
bbezak | thx for everybody who helped with merging/fixing CI issues in recent weeks | 13:05 |
amir58118 | +1 | 13:05 |
bbezak | #topic Current cycle planning | 13:05 |
bbezak | I probably will look into bumping mariadb soon | 13:06 |
bbezak | as it is long overdue | 13:06 |
bbezak | looking at Valkey support - I will chase gkoper | 13:07 |
bbezak | mnasiadka: surely continues his uwsgi crusade shortly | 13:07 |
bbezak | recent 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 well | 13:09 |
bbezak | but that is different story | 13:09 |
bbezak | #topic Open discussion | 13:10 |
amir58118 | The 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%22 | 13:10 |
mmalchuk | bbezak 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 |
seunghunlee | It would be nice we can review the process of https://bugs.launchpad.net/kolla-ansible/+bug/2080402 | 13:11 |
frickler | late o/ | 13:12 |
mmalchuk | amir58118 did we run integration tests during deploy in your change in k-a ? | 13:13 |
bbezak | excuse me a bit as frickler just joined - we would need review for https://review.opendev.org/c/openstack/kolla/+/956456 | 13:13 |
bbezak | and interesting point mmalchuk - I was testing that and that indeed works - https://review.opendev.org/c/openstack/kolla-ansible/+/956364 | 13:14 |
bbezak | however that will not solve other people deployments | 13:14 |
bbezak | as not all people are using ansible-collection-kolla bits to deploy hosts | 13:14 |
bbezak | at least I don't think so | 13:14 |
frickler | yes, we have our own docker pins downstream, so I'd rather try to avoid that | 13:15 |
mmalchuk | bbezak sure! we pin python libs, but use latest os packages - that's not good) | 13:15 |
frickler | I'll look at the other fix today | 13:15 |
bbezak | well, kolla-ansible is not about host OS | 13:15 |
bbezak | kayobe surely | 13:15 |
mmalchuk | yep. but we can mention version pin recomendations in docs and use them on CI | 13:16 |
amir58118 | mmalchuk: I'm not exactly sure, I need to check. | 13:16 |
bbezak | that is another bit to maintain | 13:16 |
mmalchuk | amir58118 ok. we can continue to discuss in the review | 13:17 |
bbezak | docker-py should fix it, but I didn't see traction there | 13:17 |
bbezak | so we're fixing it in kolla for the time being | 13:17 |
mmalchuk | ok. but it'll work until the next version with a new issue))) | 13:18 |
bbezak | that is also true :) | 13:19 |
mnasiadka | sorry for being late | 13:19 |
fungi | oh, thought you were still on vacation | 13:19 |
bbezak | kolla assumes stable docker engine libraries. I think that is a good assumption | 13:19 |
bbezak | (podman too) | 13:19 |
mnasiadka | fungi: got back yesterday :) | 13:19 |
fungi | i 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 log | 13:19 |
mmalchuk | sometime ago mnasiadka said something like 'We're putting out fires' on my comment like this) | 13:20 |
mnasiadka | yeah, that's the life of a deployment project mainly ;-) | 13:20 |
mmalchuk | sad | 13:20 |
bbezak | seunghunlee: could you please elaborate your point on https://bugs.launchpad.net/kolla-ansible/+bug/2080402 ? | 13:22 |
seunghunlee | I 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/2119344 | 13:24 |
frickler | iiuc https://review.opendev.org/c/openstack/kolla-ansible/+/956409 needs reviews | 13:24 |
seunghunlee | The 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 |
bbezak | I agree. I think this is general problem of lack of meaningful reviews | 13:27 |
bbezak | as there are not many active reviewers | 13:27 |
mnasiadka | I think this can be said to hundreds of other patches | 13:27 |
mmalchuk | seunghunlee it was merged after +2 from mnasiadka | 13:27 |
mnasiadka | But it is what it is | 13:27 |
bbezak | there is last comment mentioning that it was a wrong way of doing things | 13:28 |
frickler | we can't test everything in CI, so I'd say we need to live with occasional regressions | 13:28 |
mnasiadka | we don't have an OIDC scenario, so yes - we need to live with that | 13:28 |
frickler | the question is whether a second reviewer would have had a better chance of spotting the regression, which may not be likely, too | 13:28 |
bbezak | true. but I also understand frustration about such cases | 13:29 |
mnasiadka | But I'm a bit worried about people writing what needs to be fixed in bug report, but not submitting the patch in Gerrit | 13:29 |
frickler | iiuc the fix is submitted now, so we can just review it and move on? | 13:29 |
bbezak | yeap | 13:30 |
seunghunlee | I agree not everything can be tested through CI and also we're lacking reviews but this particular case looked bad, so I wanted to raise | 13:30 |
mmalchuk | there is now good or wrong way to fix things. we should fix. lets merge one new fix) | 13:30 |
seunghunlee | Yeah I proposed a fix, so let's move on | 13:30 |
mmalchuk | there is no* | 13:30 |
mmalchuk | seunghunlee good! thanks | 13:30 |
bbezak | fungi: it looks like we need more time to analyze your points | 13:31 |
mnasiadka | seunghunlee: basically people historically have been putting a list of patches they need reviews on in the Kolla Whiteboard | 13:31 |
mnasiadka | And I liked that approach | 13:31 |
frickler | yes, but even that has been difficult to get to recentish | 13:32 |
mnasiadka | seunghunlee: https://etherpad.opendev.org/p/KollaWhiteBoard#L66 - here | 13:33 |
seunghunlee | mnasiadka: Thanks | 13:33 |
bbezak | and as usual if anybody have some change to review, and it feels stale. whiteboard is a good place, but IRC as well | 13:33 |
mnasiadka | frickler: 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 past | 13:33 |
frickler | mnasiadka: yes, I try to look at the important things, but more often than not do not get to it in time | 13:34 |
mnasiadka | We don't really have candidates for more core reviewers, so it is what it is. | 13:35 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible master: Re-add missing permission attribute https://review.opendev.org/c/openstack/kolla-ansible/+/956667 | 13:35 |
seunghunlee | That's unfortunate | 13:36 |
bbezak | ok, good discussion today | 13:36 |
bbezak | any other points to discuss? | 13:37 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Remove ironic-inspector deployment https://review.opendev.org/c/openstack/kolla-ansible/+/951864 | 13:38 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Remove ironic-inspector deployment https://review.opendev.org/c/openstack/kolla-ansible/+/951864 | 13:38 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible stable/2025.1: Re-add missing permission attribute https://review.opendev.org/c/openstack/kolla-ansible/+/956668 | 13:39 |
bbezak | thank you all for lively chat today! | 13:39 |
bbezak | #endmeeting | 13:39 |
opendevmeet | Meeting ended Wed Aug 6 13:39:40 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:39 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-08-06-13.00.html | 13:39 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-08-06-13.00.txt | 13:39 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-08-06-13.00.log.html | 13:39 |
seunghunlee | Thank you | 13:40 |
frickler | thx bbezak | 13:40 |
amir58118 | thanks | 13:40 |
mmalchuk | bbezak thanks | 13:42 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Remove ironic-inspector deployment https://review.opendev.org/c/openstack/kolla-ansible/+/951864 | 13:43 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: ironic: Stop copying rootwrap since Ironic dropped it https://review.opendev.org/c/openstack/kolla/+/956648 | 13:55 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources" https://review.opendev.org/c/openstack/kolla/+/949755 | 13:56 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: keystone: Drop copying of wsgi files https://review.opendev.org/c/openstack/kolla/+/956670 | 13:58 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: keystone: Drop copying of wsgi files https://review.opendev.org/c/openstack/kolla/+/956670 | 14:00 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: manila: Drop copying of wsgi files https://review.opendev.org/c/openstack/kolla/+/956671 | 14:01 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: manila: Drop copying of wsgi files https://review.opendev.org/c/openstack/kolla/+/956671 | 14:02 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible master: Fix etcd config.json template https://review.opendev.org/c/openstack/kolla-ansible/+/956672 | 14:02 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources" https://review.opendev.org/c/openstack/kolla/+/949755 | 14:02 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: keystone: Drop copying of wsgi files https://review.opendev.org/c/openstack/kolla/+/956670 | 14:03 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: manila: Drop copying of wsgi files https://review.opendev.org/c/openstack/kolla/+/956671 | 14:03 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible stable/2025.1: Fix etcd config.json template https://review.opendev.org/c/openstack/kolla-ansible/+/956673 | 14:03 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources" https://review.opendev.org/c/openstack/kolla/+/949755 | 14:03 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Revert "Switch to 2025.1 sources" https://review.opendev.org/c/openstack/kolla/+/949755 | 14:03 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible stable/2025.1: Fix etcd config.json template https://review.opendev.org/c/openstack/kolla-ansible/+/956673 | 14:04 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible stable/2025.1: Re-add missing permission attribute https://review.opendev.org/c/openstack/kolla-ansible/+/956668 | 14:06 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible master: Only add OIDCXForwardedHeaders when variable is set https://review.opendev.org/c/openstack/kolla-ansible/+/956409 | 14:11 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: keystone: Add local wsgi file https://review.opendev.org/c/openstack/kolla/+/956670 | 14:21 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: keystone: Add local wsgi file https://review.opendev.org/c/openstack/kolla/+/956670 | 14:22 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: keystone: Add local wsgi file https://review.opendev.org/c/openstack/kolla/+/956670 | 14:24 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible stable/2025.1: Re-add missing permission attribute https://review.opendev.org/c/openstack/kolla-ansible/+/956668 | 14:26 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible master: Fix etcd config.json template https://review.opendev.org/c/openstack/kolla-ansible/+/956672 | 14:27 |
opendevreview | Seunghun Lee proposed openstack/kolla-ansible stable/2025.1: Fix etcd config.json template https://review.opendev.org/c/openstack/kolla-ansible/+/956673 | 14:30 |
opendevreview | Merged openstack/kolla master: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956456 | 15:13 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956691 | 15:14 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/2024.2: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956692 | 15:14 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/2024.1: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956693 | 15:15 |
opendevreview | Merged openstack/kayobe master: public-openrc: avoid duplicate OS_CACERT https://review.opendev.org/c/openstack/kayobe/+/954593 | 15:42 |
opendevreview | Merged openstack/kayobe master: use current debian version supported by kolla https://review.opendev.org/c/openstack/kayobe/+/956484 | 15:42 |
opendevreview | Pierre Riteau proposed openstack/kayobe stable/2025.1: public-openrc: avoid duplicate OS_CACERT https://review.opendev.org/c/openstack/kayobe/+/956702 | 15:51 |
opendevreview | Pierre Riteau proposed openstack/kayobe stable/2024.2: public-openrc: avoid duplicate OS_CACERT https://review.opendev.org/c/openstack/kayobe/+/956703 | 15:52 |
opendevreview | Pierre Riteau proposed openstack/kayobe stable/2024.1: public-openrc: avoid duplicate OS_CACERT https://review.opendev.org/c/openstack/kayobe/+/956704 | 15:53 |
kevko | mnasiadka: i will be active ...but I have long vacation now :) ... for now I am in bulgaria :) | 15:57 |
mnasiadka | kevko: have fun :) | 16:07 |
opendevreview | Merged openstack/kolla-ansible master: Only add OIDCXForwardedHeaders when variable is set https://review.opendev.org/c/openstack/kolla-ansible/+/956409 | 17:50 |
opendevreview | Merged openstack/kolla stable/2024.2: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956692 | 18:30 |
opendevreview | Merged openstack/kolla stable/2024.1: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956693 | 18:30 |
opendevreview | Merged openstack/kolla stable/2025.1: send empty X-Registry-Auth for anonymous pushes https://review.opendev.org/c/openstack/kolla/+/956691 | 19:22 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!