| opendevreview | Verification of a change to openstack/kayobe master failed: Revert "Use OpenStack 2023.2 release" https://review.opendev.org/c/openstack/kayobe/+/903167 | 08:11 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Use gunicorn for Nova services https://review.opendev.org/c/openstack/kolla-ansible/+/903165 | 08:14 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Pin docker installation to <7 https://review.opendev.org/c/openstack/kolla/+/903363 | 08:20 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Move openstack clients setup to a role https://review.opendev.org/c/openstack/kolla-ansible/+/903436 | 08:34 |
| mnasiadka | frickler: https://review.opendev.org/c/openstack/kolla-ansible/+/903364 - that fixes kolla-ansible when depends on kolla to do image builds | 08:37 |
| mnasiadka | and https://review.opendev.org/c/openstack/kolla/+/903363 should fix kolla | 08:37 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Move openstack clients setup to a role https://review.opendev.org/c/openstack/kolla-ansible/+/903436 | 08:38 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Pin docker installation to <7 https://review.opendev.org/c/openstack/kolla/+/903363 | 08:39 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Use gunicorn for Nova services https://review.opendev.org/c/openstack/kolla-ansible/+/903165 | 08:44 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Move openstack clients setup to a role https://review.opendev.org/c/openstack/kolla-ansible/+/903436 | 08:46 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker_sdk: Add a precheck for virtualenv and Debian https://review.opendev.org/c/openstack/ansible-collection-kolla/+/903437 | 08:59 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.2: Add precheck for RabbitMQ quorum queues https://review.opendev.org/c/openstack/kolla-ansible/+/903462 | 09:42 |
| SvenKieske | mnasiadka: what's exactly the issue with docker 7.0.0? what did they change? | 10:09 |
| SvenKieske | that would be good to have in the change imho, looking at the changelog I can't see what the issue is: https://docker-py.readthedocs.io/en/stable/change-log.html#id1 | 10:14 |
| SvenKieske | apparently we need docker-py 7.0.0 for python 3.12 support | 10:14 |
| opendevreview | Alex Welsh proposed openstack/kolla-ansible master: Update keystone service user passwords https://review.opendev.org/c/openstack/kolla-ansible/+/903178 | 10:22 |
| SvenKieske | bbezak: mhm where is that failure with the port in the registry in our code? I wrote a small reproducer locally but I can't seem to trigger this bug | 10:27 |
| SvenKieske | bbezak, mnasiadka: this works for me locally with docker-py 7 and docker 24.0.7 on fedora inside a venv https://paste.opendev.org/show/bKegaARCUtu68vIs1b6o/ I guess I'm doing something different | 10:42 |
| SvenKieske | also works with an underscore, mhm | 10:46 |
| frickler | afaict the error happened when trying to build an image? I don't see that in your log | 10:47 |
| SvenKieske | I was under the impression this happens when tagging and pushing to a local registry, but might be right that this is not enough | 11:42 |
| mnasiadka | When building | 12:09 |
| mnasiadka | docker-py introduced a check | 12:09 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker_sdk: Add a precheck for virtualenv and Debian https://review.opendev.org/c/openstack/ansible-collection-kolla/+/903437 | 12:30 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Pin docker installation to <7 https://review.opendev.org/c/openstack/kolla/+/903363 | 12:35 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Stop running pip install container_engine https://review.opendev.org/c/openstack/kolla-ansible/+/903364 | 12:36 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Move openstack clients setup to a role https://review.opendev.org/c/openstack/kolla-ansible/+/903436 | 12:41 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Move openstack clients setup to a role https://review.opendev.org/c/openstack/kolla-ansible/+/903436 | 12:44 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Use gunicorn for Nova services https://review.opendev.org/c/openstack/kolla-ansible/+/903165 | 12:46 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Use gunicorn for Nova services https://review.opendev.org/c/openstack/kolla-ansible/+/903165 | 12:46 |
| opendevreview | Merged openstack/kayobe master: Stop NetworkManager from overriding resolv.conf https://review.opendev.org/c/openstack/kayobe/+/902561 | 12:58 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Move openstack clients setup to a role https://review.opendev.org/c/openstack/kolla-ansible/+/903436 | 13:03 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/2023.2: Stop NetworkManager from overriding resolv.conf https://review.opendev.org/c/openstack/kayobe/+/903450 | 13:23 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/2023.1: Stop NetworkManager from overriding resolv.conf https://review.opendev.org/c/openstack/kayobe/+/903451 | 13:23 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/zed: Stop NetworkManager from overriding resolv.conf https://review.opendev.org/c/openstack/kayobe/+/903452 | 13:23 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/yoga: Stop NetworkManager from overriding resolv.conf https://review.opendev.org/c/openstack/kayobe/+/903453 | 13:23 |
| Viii | stupid question :) but I don't know where to find the answer (I searched). On ceph we have VMS and VOLUMES folders. When creating a VM with the NOVA API, the disk file is saved in the VMS folder (the disk ID is not visible in the VM details). However, when creating from Horizon, the VM disk is saved in the VOLUMES folder | 13:28 |
| Viii | why? | 13:28 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Move openstack clients setup to a role https://review.opendev.org/c/openstack/kolla-ansible/+/903436 | 13:31 |
| mnasiadka | Viii: it depends if you boot from image or boot from volume | 13:31 |
| opendevreview | Merged openstack/kayobe master: Revert "Use OpenStack 2023.2 release" https://review.opendev.org/c/openstack/kayobe/+/903167 | 13:33 |
| frickler | iirc that is something you can configure in horizon, too, which default to choose | 13:34 |
| frickler | DEFAULT_BOOT_SOURCE in https://docs.openstack.org/horizon/latest/configuration/settings.html | 13:37 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker_sdk: Add a precheck for virtualenv and Debian https://review.opendev.org/c/openstack/ansible-collection-kolla/+/903437 | 13:42 |
| frickler | actually that will still bfv if create_volume=true | 13:44 |
| Viii | thanks for directing me. Now I understand what it depends on. It does not depend on boot_image/boot volume but on "Create New Volume: Yes/NO". Thanks! | 13:51 |
| Viii | Fickler: thanks. I just saw that you wrote a solution | 13:54 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker_sdk: Add a precheck for virtualenv and Debian https://review.opendev.org/c/openstack/ansible-collection-kolla/+/903437 | 13:59 |
| frickler | mnasiadka: https://review.opendev.org/c/openstack/kolla-ansible/+/903364 is still failing for debian | 14:00 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Stop running pip install container_engine https://review.opendev.org/c/openstack/kolla-ansible/+/903364 | 14:01 |
| mnasiadka | frickler: see that, thanks - updating as per your comments | 14:01 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Stop running pip install container_engine https://review.opendev.org/c/openstack/kolla-ansible/+/903364 | 14:02 |
| SvenKieske | I'm still confused by our tagging issue, do we really create tags that include the registry inside the tag, or is that the parsing error happening in docker-py? | 14:29 |
| SvenKieske | it's not clear to me, reading https://github.com/docker/docker-py/issues/3195 | 14:29 |
| mnasiadka | they implemented a check before build, and it seems the parsing is wrong there - the data is not even being sent to docker engine/api | 14:38 |
| mnasiadka | there's a link in one of the patches | 14:38 |
| SvenKieske | well I matched the regex, seems the regex is correct, ":1234" (e.g. port numbers) where never part of the spec afaik | 14:45 |
| SvenKieske | do we include the whole registry fqdn including port number in the tag? what for? | 14:46 |
| SvenKieske | or does the code match on the registry name itself? that's not clear to me | 14:47 |
| SvenKieske | the latter would be a bug, the former would be unfortunate us that were not OCI spec compliant in using banned characters in tags (the ":" specifically) | 14:48 |
| SvenKieske | mnasiadka: actually, maybe the spec has a bug as well xD | 14:59 |
| SvenKieske | https://github.com/opencontainers/distribution-spec/commit/a73835700327bd1c037e33d0834c46ff98ac1286 talks about aligning this with docker, the commit message quotes the code "separator = `(?:[._]|__|[-]+)`" but that separator, especially the ":" doesn't actually get added to the spec | 15:00 |
| mnasiadka | but come on, if the OCI spec says you can use : for port | 15:00 |
| mnasiadka | then you can use it | 15:00 |
| mnasiadka | they need to fix the code | 15:00 |
| SvenKieske | a port is not a tag ;) | 15:00 |
| SvenKieske | I actually think this is a bug in the spec, maybe | 15:01 |
| SvenKieske | well from the text in the reference regexp it seems you're right mnasiadka, afaict, at least. | 15:09 |
| SvenKieske | the spec is very obtuse and unclearly worded imho, so it's hard to tell imho. | 15:10 |
| SvenKieske | ah nvm, my regex foo left me :D "?:" is a non capturing group | 15:16 |
| SvenKieske | mnasiadka: so i proposed both fixes to the spec and to docker-py, maybe they accept that :) | 16:47 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Test control plane deployment using tenks https://review.opendev.org/c/openstack/kayobe/+/620074 | 16:56 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: Fix Docker health check for sahara_engine https://review.opendev.org/c/openstack/kolla-ansible/+/903528 | 17:52 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: Fix Docker health check for sahara_engine https://review.opendev.org/c/openstack/kolla-ansible/+/903528 | 17:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!