| opendevreview | Michael Still proposed openstack/kolla master: Transition to the new rabbitmq Debian package archives. https://review.opendev.org/c/openstack/kolla/+/967475 | 05:02 |
|---|---|---|
| opendevreview | Michael Still proposed openstack/kolla master: Transition to the new rabbitmq Debian package archives. https://review.opendev.org/c/openstack/kolla/+/967475 | 08:51 |
| mikal | mnasiadka: are there other things you want as well? I am not super excited by going round and round in little steps. | 09:13 |
| mnasiadka | mikal: no, we just need a bug report linked | 09:14 |
| mnasiadka | mikal: I’m also keen to getting it merged ASAP and backport to all stable branches | 09:14 |
| BertrandLanson[m] | Hello guys ! I was going over some old patches of mine, and this one https://review.opendev.org/c/openstack/kolla-ansible/+/934429 only needs a final approval I think to get merged ? If anyone has time :) | 09:18 |
| mikal | mnasiadka: I think I would have really appreciated more than two words in your initial review to be honest, especially given I am not a Kolla contributor, this was announced a long time ago and ignored, and its taken a fair chunk of my day to fix. | 09:23 |
| mnasiadka | mikal: All of the Kolla contributors are come and go, there’s only 2-3 people trying to maintain the project (because nobody else wants to do that) - unfortunately we’re not going to copy paste the contributor guide in Gerrit to everyone that is coming to merge his little patch. And no - we have not ignored it, nobody pays us for maintaining that project and the scope is so huge - that we can’t really pay close attention to | 09:26 |
| mnasiadka | every service or container the project delivers. | 09:26 |
| mikal | mnasiadka: and yet the TC has repeatedly tried to improve the contribution experience... | 09:27 |
| mnasiadka | mikal: I don’t understand, all of the contributor guidelines are documented (https://docs.openstack.org/kolla-ansible/latest/contributor/index.html) and still you are trying to offend the maintainers of the project you use that we ignored something | 09:28 |
| mikal | mnasiadka: I am not trying to offend. I am trying to help. I am unsure what I have done to offend you apart from fixing a bug? | 09:29 |
| mnasiadka | mikal: well, you wrote that we ignored something and due to that ignorance it has taken a fair chunk of your day to fix it | 09:30 |
| mikal | mnasiadka: and you wrote a two word -1 review comment. I think perhaps you're taking all this a little personally. | 09:30 |
| mnasiadka | Me? Nah - you complained that you wanted more than two words - I assumed you have read the contributor guide, especially the section about that we require release notes and what section should you use. | 09:31 |
| mnasiadka | That’s why somebody has spent significant amount of time to document that, so we don’t need to write it in every Gerrit review. | 09:32 |
| mnasiadka | So please - next time please at least think that somebody else time is of the same value as your time | 09:32 |
| mikal | Go back to my original question here... All I was asking was if there was anything else you wanted. | 09:33 |
| opendevreview | Michael Still proposed openstack/kolla master: Transition to the new rabbitmq Debian package archives. https://review.opendev.org/c/openstack/kolla/+/967475 | 09:35 |
| mnasiadka | And I replied that we only need a bug report linked | 09:35 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Transition to the new rabbitmq Debian package archives. https://review.opendev.org/c/openstack/kolla/+/967475 | 09:36 |
| opendevreview | OpenStack Release Bot proposed openstack/kayobe stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/kayobe/+/967491 | 09:46 |
| opendevreview | OpenStack Release Bot proposed openstack/kayobe stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/kayobe/+/967492 | 09:46 |
| opendevreview | OpenStack Release Bot proposed openstack/kayobe master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/kayobe/+/967493 | 09:46 |
| opendevreview | OpenStack Release Bot proposed openstack/kayobe-config stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/kayobe-config/+/967494 | 09:46 |
| opendevreview | OpenStack Release Bot proposed openstack/kayobe-config stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/kayobe-config/+/967495 | 09:46 |
| opendevreview | OpenStack Release Bot proposed openstack/kayobe-config-dev stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/kayobe-config-dev/+/967496 | 09:47 |
| opendevreview | OpenStack Release Bot proposed openstack/kayobe-config-dev stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/kayobe-config-dev/+/967497 | 09:47 |
| mikal | mnasiadka: Look, I am actually not trying to be a dick here, I am trying to help. That said, it is nearly 9pm after an 11 hour work day. Hopefully this bug fix helps. | 09:48 |
| mnasiadka | mikal: No one is trying, I understand your frustration with trying to build RabbitMQ, but we just need more people active in the community to be proactive. The patch itself looks good - when it will pass CI we’ll merge it (and start backporting to stable branches) | 09:50 |
| mnasiadka | BertrandLanson[m]: once 2025.2 final is tagged - we’ll merge it - so just wait a day or two ;) | 09:51 |
| BertrandLanson[m] | understood ! thanks | 09:51 |
| mikal | mnasiadka: I have a whole stack of changes for Kolla and Kolla-Ansible stacked up to support Nova's new spice-direct console type. It is unclear to me from the contributors guide whether blueprints are still actively used or not. The ones tagged to releases are for quite old releases. Do you want a blueprint before changes start appearing? Or do | 09:54 |
| mikal | you want to see changes before a blueprint? | 09:54 |
| mnasiadka | We don’t really use blueprints, I think we had that disabled on Launchpad | 09:54 |
| mnasiadka | You can raise a ,,bug’’ and prepend the topic with [RFE] if you want to add some description on what you want to do (and then use Closes-Bug in the patch implementing it) | 09:55 |
| mikal | mnasiadka: https://blueprints.launchpad.net/kolla-ansible definitely still has content, and https://docs.openstack.org/kolla-ansible/latest/contributor/contributing.html says to create one. If that is incorrect I'll fire off a quick change to update the guide to whatever the new process is. | 09:55 |
| mnasiadka | Just disabled blueprints for kolla-ansible - I’ll fire off the patch to change contribution guideliness | 09:56 |
| mnasiadka | Or you can if you want to | 09:56 |
| mikal | mnasiadka: this is more like five or so patches across the two projects at the moment because I've tried to keep them as small as possible. | 09:56 |
| mikal | mnasiadka: I am happy to make the change, but also largely ambivalent about who does it. | 09:57 |
| mnasiadka | Basically we’d like something like this: https://bugs.launchpad.net/kolla-ansible/+bug/2048392 | 09:57 |
| mnasiadka | [RFE] prefix in the topic and ideally an ‘rfe’ label so we can filter on it | 09:57 |
| mikal | Ok, I will make a change which refers to that then. | 09:57 |
| mnasiadka | Thanks! | 09:58 |
| mikal | What does "RFE" stand for? "Request For Enhancement"? | 10:00 |
| opendevreview | Michael Still proposed openstack/kolla-ansible master: Drop requirement for blueprints. https://review.opendev.org/c/openstack/kolla-ansible/+/967501 | 10:03 |
| mikal | mnasiadka: Ok, https://bugs.launchpad.net/kolla/+bug/2131793 is my first swing at a tracking bug then. I will tag reviews as `Partially-Implements: #2131793 (spice-direct consoles)` when I upload them. | 10:09 |
| mnasiadka | mikal: great, thanks | 10:12 |
| opendevreview | Michael Still proposed openstack/kolla-ansible master: Drop requirement for blueprints. https://review.opendev.org/c/openstack/kolla-ansible/+/967501 | 10:14 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 11:21 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 11:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 11:24 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 11:29 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 11:53 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 11:55 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 11:57 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 11:57 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 12:00 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Transition to the new rabbitmq Debian package archives. https://review.opendev.org/c/openstack/kolla/+/967475 | 12:05 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Revert "CI: Pin docker to <29" https://review.opendev.org/c/openstack/kolla-ansible/+/966998 | 12:07 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 12:16 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 12:26 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 12:37 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 13:46 |
| opendevreview | Pierre Riteau proposed openstack/kolla master: Remove special handling for python3.12 https://review.opendev.org/c/openstack/kolla/+/967536 | 13:56 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Move to Rocky Linux 10 https://review.opendev.org/c/openstack/kolla/+/967507 | 14:04 |
| *** mikal4 is now known as mikal | 14:49 | |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Allow enabling both novnc and spice consoles simultaneously https://review.opendev.org/c/openstack/kolla-ansible/+/966111 | 16:46 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Transition to the new rabbitmq Debian package archives. https://review.opendev.org/c/openstack/kolla/+/967475 | 18:21 |
| mikal | mnasiadka: I will test Ubuntu today, I suspect the URL is subtly wrong. My CI is all Debian, so I was relying on the gate to test the other distros. | 18:27 |
| mnasiadka | mikal: thanks, although majority of the CI failures today are accredited to Cloudflare | 18:50 |
| mikal | mnasiadka: I only just realised there was a cloudflare outage yesterday as well as I am pre-coffee. So yeah, that might be it too. I will test it in my private CI more explicitly regardless. | 18:58 |
| mnasiadka | mikal: great, it’s 8pm here - so I’ll have a look in the morning :) | 19:04 |
| mikal | mnasiadka: I'll also try and give 2023.1 person a hand once the gerrit review merges. | 19:07 |
| mnasiadka | 2023.1 is heading EOL, so we might not be able to fix it - but let’s see how it goes | 19:08 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!