| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/nodepool] 836263: Docker image: use unstable debootstrap https://review.opendev.org/c/zuul/nodepool/+/836263 | 01:06 | |
| @iwienand:matrix.org | frickler: ^ we can definitely update the ppa version, but probably worth testing if this is a better way forward | 01:07 |
|---|---|---|
| @iwienand:matrix.org | 022-04-04 01:18:42.682946 | ubuntu-focal | Get:13 http://deb.debian.org/debian unstable/main amd64 debootstrap all 1.0.126+nmu1 [76.9 kB] | 01:38 |
| @iwienand:matrix.org | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_334/836263/1/check/nodepool-build-image-siblings/3345be1/job-output.txt | 01:38 |
| @iwienand:matrix.org | doesn't really seem to pull in anything else crazy | 01:38 |
| -@gerrit:opendev.org- Zuul merged on behalf of Mohammed Naser: [zuul/zuul-jobs] 835162: ensure-kubernetes: fix missing 02-crio.conf https://review.opendev.org/c/zuul/zuul-jobs/+/835162 | 06:24 | |
| -@gerrit:opendev.org- Dr. Jens Harbott proposed on behalf of Ian Wienand: [zuul/nodepool] 836263: Docker image: use unstable debootstrap https://review.opendev.org/c/zuul/nodepool/+/836263 | 06:28 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/zuul-jobs] 831136: ensure-pip: fix typo in ensure_pip_virtualenv_command documentation https://review.opendev.org/c/zuul/zuul-jobs/+/831136 | 07:26 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 836313: Skip reconfig when all project config is excluded https://review.opendev.org/c/zuul/zuul/+/836313 | 11:18 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 836316: Fix deserialization of new management events https://review.opendev.org/c/zuul/zuul/+/836316 | 11:30 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 836322: Only try to revise unaccepted node requests https://review.opendev.org/c/zuul/zuul/+/836322 | 12:35 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Sorin Sbârnea: [zuul/zuul-jobs] 821247: Add tox-py310 job https://review.opendev.org/c/zuul/zuul-jobs/+/821247 | 13:51 | |
| @clarkb:matrix.org | corvus: tobiash have we had a chance to think about https://review.opendev.org/c/zuul/zuul/+/836108 a bit more and whether or not we should just drop the collisions check entirely? I suspect that this change is safe if that further update is safe. We might do this as an intermediate step if we want? Though users would have to deal with behavior changes multiple times in that case which isn't ideal | 15:48 |
| @tobias.henkel:matrix.org | Actually I think the way of least surprise would be to just remove the collision check. Actually since the regex match was never targeted to match a single project but multiple projects I have no clue why I added that in the first place. | 16:29 |
| @tobias.henkel:matrix.org | * I think the way of least surprise would be to just remove the collision check. Actually since the regex match was never targeted to match a single project but multiple projects I have no clue why I added that in the first place. | 16:30 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 835452: Test zuul-client dequeue-all https://review.opendev.org/c/zuul/zuul/+/835452 | 16:30 | |
| @tobias.henkel:matrix.org | To be more precise I think the best behvior would be to merge matches of short and canonical project name | 16:30 |
| @clarkb:matrix.org | tobiash: got it. I can take a look at updating the change to do that when I've got some quiet time | 16:36 |
| @tobias.henkel:matrix.org | Clark: corvus I commented my thoughts on that on the change | 16:36 |
| @mnaser:matrix.org | Sorry for nagging on this, but it's blocked our image builds for a while: | 17:54 |
| - https://review.opendev.org/c/zuul/zuul-registry/+/835772 (changed that introduces a failing test with a short-circuit to demonstrate "its broken") | ||
| - https://review.opendev.org/c/zuul/zuul-registry/+/835773 (actual fix) | ||
| tl;dr: there is a bug in the docker client that since we don't present a specific header, it assumes the remote registry doesn't support manifests. | ||
| @clarkb:matrix.org | mnaser: any idea why our image builds don't seem to be affected? I think we have manifest builds due to multiarch nodepool builds and https://review.opendev.org/c/zuul/nodepool/+/836263 built | 17:59 |
| @mnaser:matrix.org | > <@clarkb:matrix.org> mnaser: any idea why our image builds don't seem to be affected? I think we have manifest builds due to multiarch nodepool builds and https://review.opendev.org/c/zuul/nodepool/+/836263 built | 18:02 |
| I believe these image builds seem to push to use `docker buildx build .. --push` which somehow doesn't seem to hit this, vs the behaviour where `docker manifest create` that we're using here to build images in a bit on a non-conventional way: | ||
| https://review.opendev.org/c/vexxhost/ansible-collection-atmosphere/+/835767 | ||
| we're essentially building a whole image from scatch using `multistrap` which can simply install the right debs with the right arch, then pip installing the arch-specific wheels into the same folder and building an image into that, so we're not using docker buildx at all | ||
| @clarkb:matrix.org | I see relies on specific client steps | 18:02 |
| @mnaser:matrix.org | Yes, the Docker bug is indeed not affected by `docker pull` and `docker push` either, specifically the `docker manifest` sub commands | 18:03 |
| @clarkb:matrix.org | mnaser: left a question on the test parent change | 18:04 |
| -@gerrit:opendev.org- Zuul merged on behalf of Mohammed Naser: [zuul/zuul-jobs] 835156: run-buildset-registry: Drop extra install packages task https://review.opendev.org/c/zuul/zuul-jobs/+/835156 | 18:08 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!