| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 862522: WIP: openstack statemachine https://review.opendev.org/c/zuul/nodepool/+/862522 | 00:32 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 860506: Include skipped builds in database and web ui https://review.opendev.org/c/zuul/zuul/+/860506 | 04:10 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 859939: Implement Gitea driver https://review.opendev.org/c/zuul/zuul/+/859939 | 08:59 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 859940: Implement tests of the gitea driver https://review.opendev.org/c/zuul/zuul/+/859940 | 08:59 | |
| @clarkb:matrix.org | corvus: have time to quickly rereview https://review.opendev.org/c/zuul/nodepool/+/862500/ ? The child change is where I ended up eventually in trying to fix the test issue too if you have time for a slightly more involved change | 15:13 |
|---|---|---|
| @clarkb:matrix.org | swest: if you have time for https://review.opendev.org/c/zuul/nodepool/+/862526 too that should address a nodepool test race that has been hitting us frequently | 15:25 |
| @clarkb:matrix.org | (and should actually fix the static driver) | 15:25 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/nodepool] 862414: Fix gce test pausing at quote https://review.opendev.org/c/zuul/nodepool/+/862414 | 15:27 | |
| @clarkb:matrix.org | I rebased ^ onto 862526 because it hit the issue 862526 ran into in the gate | 15:27 |
| @clarkb:matrix.org | * I rebased ^ onto 862526 because it hit the issue 862526 fixes in the gate | 15:27 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 862622: Add Python 3.11 testing https://review.opendev.org/c/zuul/zuul/+/862622 | 15:58 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/nodepool] 862500: Add debug logs to the static driver https://review.opendev.org/c/zuul/nodepool/+/862500 | 16:06 | |
| @clarkb:matrix.org | Python 3.11 is not slower at least. Hard to say if it is quicker yet. The job there ran in just under 36 minutes on inmotion | 16:42 |
| @clarkb:matrix.org | Which if you think about python 3.8 taking more than an hour is pretty amazing | 16:42 |
| @clarkb:matrix.org | corvus: the persistent nodepool image build failures on https://review.opendev.org/c/zuul/nodepool/+/862414 have me wondering if we've cached bad data somewhere for some of the layers. The three failures today ran on two different clouds which likely rules out our apache proxy caches. Could it be the intermediate registry or is docker hub serving us incomplete data? | 16:48 |
| @clarkb:matrix.org | I guess I can do a docker pull of the image and see if it breaks from my desktop | 16:48 |
| @clarkb:matrix.org | neat my local client talking to docker hub is doing retries for some layers | 16:52 |
| @clarkb:matrix.org | it ended up succeeding but it definitely retried a number of times | 16:52 |
| @jim:acmegating.com | maybe docker buildx is more sensitive to that failure? | 16:53 |
| @clarkb:matrix.org | ya, could also be that my interactive session is going to retry more? | 16:54 |
| @clarkb:matrix.org | oh also zuul's test suite appears to be python3.11 compatible. No changes were required to make it pass | 16:56 |
| @jim:acmegating.com | Clark: pushing to dockerhub is returning http 530 for some pushes now | 17:06 |
| @jim:acmegating.com | everything is a-ok at https://status.docker.com/ though! | 17:08 |
| -@gerrit:opendev.org- Clark Boylan proposed: | 17:09 | |
| - [zuul/zuul-jobs] 862628: Pin py38 jobs to focal https://review.opendev.org/c/zuul/zuul-jobs/+/862628 | ||
| - [zuul/zuul-jobs] 862629: Add tox-py311 job https://review.opendev.org/c/zuul/zuul-jobs/+/862629 | ||
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/nodepool] 862630: Add Python 3.11 testing https://review.opendev.org/c/zuul/nodepool/+/862630 | 17:11 | |
| @clarkb:matrix.org | zuulians both of those zuul-jobs changes should be good to go. | 17:11 |
| @clarkb:matrix.org | No changes to make nodepool python3.11 tests pass either. All of this is good canary for major problems, but as mentioned in the commit messages I'm not sure we want to run the tests under these versions just yet. I think at a high level the plan should be to update opendev's base python images to include 3.11 (in progress via removal of 3.8 first) then ensure that both unittests and the more functional tests that rely on the docker images are happy then we can switch | 17:32 |
| @westphahl:matrix.org | Clark: +3 on 862526, interesting find. I guess we never ran into this issue because we don't update our config in-place. | 18:47 |
| @clarkb:matrix.org | I think it is very timing dependent too as the unittests have gotten flakier recently | 18:49 |
| @jim:acmegating.com | i think it's extremely unlikely to happen in a production setting, and it self-corrects | 18:56 |
| @jim:acmegating.com | * i think it's extremely unlikely to happen in a production setting due to timing, and it self-corrects | 18:56 |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/nodepool] 862526: Add idle state to driver providers https://review.opendev.org/c/zuul/nodepool/+/862526 | 19:39 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/nodepool] 862414: Fix gce test pausing at quote https://review.opendev.org/c/zuul/nodepool/+/862414 | 20:56 | |
| @iwienand:matrix.org | corvus: it looks like gating.dev doesn't trigger the infra-prod dns deployment job on commit -> https://opendev.org/opendev/zone-gating.dev/src/branch/master/.zuul.yaml ... is this by design or just something that hasn't been done? | 21:16 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 21:18 | |
| - [zuul/zuul] 860605: Include skip reason in build error_detail https://review.opendev.org/c/zuul/zuul/+/860605 | ||
| - [zuul/zuul] 860606: Include some skipped jobs in the code-review report https://review.opendev.org/c/zuul/zuul/+/860606 | ||
| @jim:acmegating.com | ianw: i think not done; possibly due to cross-tenant issues making that complex. | 21:59 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 23:31 | |
| - [zuul/zuul] 858987: Rename admin-rule to authorization-rule https://review.opendev.org/c/zuul/zuul/+/858987 | ||
| - [zuul/zuul] 859353: Remove unused /api/user/authorizations REST endpoint https://review.opendev.org/c/zuul/zuul/+/859353 | ||
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!