| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 931043: Fix race in blobstore put https://review.opendev.org/c/zuul/zuul/+/931043 | 09:35 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Clark Boylan: [zuul/zuul-operator] 896142: Convert container to Bookworm + Python3.11 https://review.opendev.org/c/zuul/zuul-operator/+/896142 | 13:47 | |
| -@gerrit:opendev.org- Jan Gutter proposed on behalf of Clark Boylan: [zuul/zuul-operator] 896142: Convert container to Bookworm + Python3.11 https://review.opendev.org/c/zuul/zuul-operator/+/896142 | 13:50 | |
| @jangutter:matrix.org | Hi folks, would anyone be interested in reviewing: | 15:51 |
|---|---|---|
| 924959: Support image parameters in external deps | https://review.opendev.org/c/zuul/zuul-operator/+/924959 | ||
| (also, I'm interested in opening up the broader discussion of what folks think of the future maintenance and development) | ||
| @jangutter:matrix.org | Personally, I have a few hours per week that I might be able to contribute (plus whatever spills over from what we're using). | 16:00 |
| @jangutter:matrix.org | Also, easy +W for folks wanting to bump a version! | 16:41 |
| 896142: Convert container to Bookworm + Python3.11 | https://review.opendev.org/c/zuul/zuul-operator/+/896142 | ||
| @clarkb:matrix.org | > <@mnaser:matrix.org> I guess I missed something with the multiarch stuff, bummer! Almost got it to work but it just used the existing system and the other one sat idle lol | 17:02 |
| Is there a way to tell buildx to prefer hosts with the same arch as the target arch maybe? | ||
| @mnaser:matrix.org | > <@clarkb:matrix.org> Is there a way to tell buildx to prefer hosts with the same arch as the target arch maybe? | 17:17 |
| I was thinking of that, but I couldn't find a way to do that to be honest, at least I dug through the `docker buildx` commands. | ||
| @mnaser:matrix.org | I think the issue is that all the builders are exposed but there isn't a "native arch" flag for builders from what it seems | 17:17 |
| @clarkb:matrix.org | that seems liek a big oversight since in almost very situation it should be a massive speedup | 17:18 |
| @clarkb:matrix.org | the only exception would potentially be i386 vs x86_64 | 17:18 |
| @mnaser:matrix.org | Well, when the buildx builder gets created -- yuo can see all the platforms listed - https://review.opendev.org/c/zuul/zuul-jobs/+/931578 | 17:20 |
| @mnaser:matrix.org | ah oops sorry wrong example | 17:20 |
| @mnaser:matrix.org | https://ci.atmosphere.dev/t/atmosphere/build/dc1b93aec3d0493f9a9548f9aecd394b/console#3/0/60/amd64 | 17:21 |
| @mnaser:matrix.org | This is a more accurate example where you see the platforms listed but nothing hinting at "native" platform | 17:21 |
| @mnaser:matrix.org | https://docs.docker.com/reference/cli/docker/buildx/create/#platform | 17:24 |
| @mnaser:matrix.org | "The node will also automatically detect the platforms it supports, but manual values take priority over the detected ones and can be used when multiple nodes support building for the same platform." | 17:24 |
| @mnaser:matrix.org | i can't tell if that means if you provide --platform manually it will take priority or not | 17:24 |
| @clarkb:matrix.org | my read is that yes it seems like manual settings override. So maybe you could manually set the x86 node to only do x86 and the arm node to only do arm then it will schedule properly | 17:25 |
| @mnaser:matrix.org | i guess, it just seems like not running the multiarch stuff would get you there.. without having to write all that out | 17:27 |
| @clarkb:matrix.org | ya that should work too if it affects the autodetection properly. | 17:27 |
| @mnaser:matrix.org | cause then you'd have to go into host-vars for the platforms of each system, where as if you disable running that multiarch stuff, the builder never becomes "multiarch aware" and will report native only | 17:28 |
| @mnaser:matrix.org | i guess i can toggle that flag for the multiarch multinode job since that is likely where it will be used, and we shouldn't see the any other arch than native | 17:28 |
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-jobs] 931578: Add a method to disable multiarch image https://review.opendev.org/c/zuul/zuul-jobs/+/931578 | 17:29 | |
| @clarkb:matrix.org | mnaser: one small documentation thing in ^ otherwise ya I suspect that would work | 17:31 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 931704: Fix handling large command output with split streams https://review.opendev.org/c/zuul/zuul/+/931704 | 20:00 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 931706: Fix units for ansible max line length https://review.opendev.org/c/zuul/zuul/+/931706 | 20:06 | |
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-jobs] 931578: Add a method to disable multiarch image https://review.opendev.org/c/zuul/zuul-jobs/+/931578 | 20:48 | |
| @mnaser:matrix.org | Thanks for the review, it looks like I am missing yet another thing.. it seems like we're pushing to 127.0.0.1 for the temp registry because of Docker stinkyness about not being able to store multiarch images in it's store.. but it seems the push on the arm node to 127.0.0.1 fails cause its obviously pushing to two different localhosts ;( | 21:21 |
| @clarkb:matrix.org | we might be able to change that to the first inventory node like you did elsewhere? | 21:23 |
| @mnaser:matrix.org | I'm looking at it, the challenge i think here is the fact 127.0.0.1 is hard coded in docker to be ok for insecure registry, and if we change it to push to the ip of the first node i think we're going to have to play the insecure registires dance | 21:50 |
| @mnaser:matrix.org | That change on it's own does solve the whole "picking the first node thing" | 21:51 |
| @mnaser:matrix.org | https://ci.atmosphere.dev/t/atmosphere/build/5ef5a24e14054e70860b68ebf663060b/console so the bootstrap only has native archs with that hcange so thats over with | 21:55 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:10 | |
| - [zuul/zuul] 931704: Fix handling large command output with split streams https://review.opendev.org/c/zuul/zuul/+/931704 | ||
| - [zuul/zuul] 931706: Fix units for ansible max line length https://review.opendev.org/c/zuul/zuul/+/931706 | ||
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-jobs] 931713: Stop using temporary registry https://review.opendev.org/c/zuul/zuul-jobs/+/931713 | 22:29 | |
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-jobs] 931713: Stop using temporary registry https://review.opendev.org/c/zuul/zuul-jobs/+/931713 | 22:36 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 929856: Add include-vars job attribute https://review.opendev.org/c/zuul/zuul/+/929856 | 23:29 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!