| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.2: Update repo GPG key for influxdata. https://review.opendev.org/c/openstack/kolla/+/972990 | 05:52 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Update repo GPG key for influxdata. https://review.opendev.org/c/openstack/kolla/+/972991 | 05:52 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2025.1: Update repo GPG key for influxdata. https://review.opendev.org/c/openstack/kolla/+/972991 | 08:08 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: Rework validate-all-file into custom rules https://review.opendev.org/c/openstack/kolla-ansible/+/972443 | 09:50 |
| opendevreview | Merged openstack/kolla stable/2025.2: Update repo GPG key for influxdata. https://review.opendev.org/c/openstack/kolla/+/972990 | 09:50 |
| frickler | mnasiadka: bbezak: do we want to drop publishing to dockerhub for the stable branches on this occasion, or would you rather want to create new credentials for that, too? I'd be fine with the first option | 09:51 |
| frickler | (https://review.opendev.org/c/openstack/kolla/+/972911 needs to be amended in either direction) | 09:52 |
| mnasiadka | +1 to drop | 09:55 |
| mnasiadka | We could even delete the account there | 09:55 |
| mikal | Sorry, maybe I misunderstand. Wouldn't stopping publishing images to the Docker Hub require some period of advertising first so that deployers know to move over to quay.io? The docker hub is definitely still mentioned in the Kolla README and kolla command line defaults for example. | 11:18 |
| mikal | Also, https://review.opendev.org/c/openstack/kolla/+/972441 is probably important because SPICE support (including the Horizon consoles) are currently broken in master with debian trixie. | 11:20 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: cinder/keystone: implement serial restart and healthcheck validation https://review.opendev.org/c/openstack/kolla-ansible/+/972678 | 11:37 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: cinder/keystone: implement serial restart and healthcheck validation https://review.opendev.org/c/openstack/kolla-ansible/+/972678 | 11:38 |
| frickler | mikal: deployers are to create their own images. hint or patches to make that more clear are welcome | 11:41 |
| Vii | First, you need to file a bug for the SPICE error, adding logs. Also, include which versions the problem affects. | 11:43 |
| Vii | https://bugs.launchpad.net/kolla-ansible | 11:44 |
| Vii | in 2025.1 from what I remember SPICE works without any problems (I think I tested it for a while) | 11:48 |
| mikal | frickler: I don't recall any doc which says that and every deployer I've met using Kolla-Ansible is just fetching them from the docker hub. Could you point me at a doc which recommends local builds please? | 12:00 |
| mikal | Vii: it's a problem tied entirely to the move from bookwork to trixie in master, it's just Kolla doesn't have tests for spice so didn't notice. | 12:01 |
| Vii | I don't use Debian. Submit a report, and one of the core reviewers who uses Debian will test/verify and probably accept your fix. | 12:05 |
| Vii | temporarily you can use an image from another distribution for this one container by changing its tag, e.g. from ubuntu->debian | 12:06 |
| Vii | if you have a problem and you want it solved quickly (locally) | 12:07 |
| frickler | mikal: as I said, docs may need updates. do you know a doc that says "use images from dockerhub" though? | 12:08 |
| frickler | we'll likely need to amend our mission statement, I'll work on a patch for that | 12:09 |
| frickler | oh, it is a bit hidden in https://opendev.org/openstack/kolla/src/branch/master/README.rst but that definitely needs rewording, yes | 12:29 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla master: Update README https://review.opendev.org/c/openstack/kolla/+/972998 | 12:41 |
| frickler | ^^ WIP for now, likely we'll need to talk it over during the next meeting | 12:41 |
| frickler | I guess there should also be a reno restating this | 12:42 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: cinder/keystone: implement serial restart and healthcheck validation https://review.opendev.org/c/openstack/kolla-ansible/+/972678 | 13:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/2024.2: Update repo GPG key for influxdata. https://review.opendev.org/c/openstack/kolla/+/972999 | 14:12 |
| frickler | mikal: fyi we haven't been publishing to dockerhub for a couple of months, see https://review.opendev.org/c/openstack/kolla/+/965345 | 15:10 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla stable/2025.1: CI: rotate zuul secrets https://review.opendev.org/c/openstack/kolla/+/972911 | 15:12 |
| frickler | yay, debian mirror is too old. last synced 2 months ago :( | 15:50 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: cinder/keystone: implement serial restart and healthcheck validation https://review.opendev.org/c/openstack/kolla-ansible/+/972678 | 15:57 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: cinder/keystone: implement serial restart and healthcheck validation https://review.opendev.org/c/openstack/kolla-ansible/+/972678 | 16:04 |
| *** jhorstmann is now known as Guest35573 | 16:27 | |
| opendevreview | Verification of a change to openstack/kolla stable/2025.1 failed: Update repo GPG key for influxdata. https://review.opendev.org/c/openstack/kolla/+/972991 | 17:27 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: cinder/keystone: implement serial restart and healthcheck validation https://review.opendev.org/c/openstack/kolla-ansible/+/972678 | 18:33 |
| mikal | frickler: Yeah, I am not saying its not documented, I am saying the message seems to have not gotten across given I've met a few Kolla-Ansible deployers and none of them local images. I guess given Docker Hub has been published to for more than a decade it has a lot of momentum. I'm _not_ saying you should stop. I _am_ saying that it might need | 19:36 |
| mikal | some amount of warning / announcement. Stopping for an existing stable release seems a bit scary too. | 19:36 |
| mikal | Vii: I am very confused. Its broken on master. I fixed it. I don't need another local fix. I was suggesting that master might want to take my fix. | 19:37 |
| Vii | mikal: I'm not a reviewer. :) I contribute just as much as you do ;) | 19:39 |
| mikal | Vii: how did I say you don't contribute? | 19:40 |
| mikal | Vii: This all started with a link to a gerrit review for a fix for SPICE on Debian Trixie for master. | 19:40 |
| Vii | mikal: I have a few patches that haven't been accepted either. But I'm downloading them locally and adapting them to the new code/version. | 19:41 |
| Vii | as I said, create a BUG with logs and maybe on Monday one of the contributors will look at it | 19:42 |
| Vii | They will definitely require a BUG. Just like your second patch, where RENO and the BUG were missing https://review.opendev.org/c/openstack/kolla/+/972990 | 19:45 |
| Vii | * https://review.opendev.org/c/openstack/kolla/+/972437 | 19:45 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: cinder/keystone: implement serial restart and healthcheck validation https://review.opendev.org/c/openstack/kolla-ansible/+/972678 | 20:35 |
| mikal | Hey, just letting people know it looks like the mirroring from git.opendev.org to github.com has been broken for about three weeks for at least the kolla repo. I haven't checked other repos though. | 23:38 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!