| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 00:04 | |
| - [zuul/zuul] 860988: Deprecate Ansible 5, make Ansible 6 the default https://review.opendev.org/c/zuul/zuul/+/860988 | ||
| - [zuul/zuul] 860989: Remove Ansible 5 https://review.opendev.org/c/zuul/zuul/+/860989 | ||
| @fungicide:matrix.org | following discussion in the #gerritcodereview channel, it seems there's some zuul confusion. i started to look into it, but it's unclear to me why it doesn't have links to build results in its comments, only -1 votes with no additional detail that i can find: https://gerrit-review.googlesource.com/c/gitiles/+/344594 | 11:57 |
|---|---|---|
| @fungicide:matrix.org | looks like the change added a new module, but instead of adjusting the job to include it in required projects they just deleted the -1 from zuul and merged the change | 12:01 |
| @fungicide:matrix.org | oh, i see, they're set up with the checks plugin which i'm just not familiar with, hence my confusion about where to look for links to the build results | 12:03 |
| @fungicide:matrix.org | oh, also their configs have hard-coded ansible 2.8 but they're running a newer zuul which doesn't support that any longer | 12:07 |
| @fungicide:matrix.org | so seems there are multiple problems to unravel | 12:07 |
| @clarkb:matrix.org | fungi: you should tell them what you find via the matrix discord channel for Gerrit. Unfortunately, I think my Gerrit CLA is still not working so I can't help push fixed right now | 13:02 |
| @fungicide:matrix.org | it's unclear to me just yet what they're looking for, but also i'm in the middle of rebuilding my local python interpreters for yesterday's releases so i can't re-test my cla membership yet either | 13:05 |
| @jim:acmegating.com | fungi: should we chat here or in #gerritcodereview:matrix.org | 13:35 |
| @fungicide:matrix.org | in gerritcodereview i think, i'm just not all that familiar with their configuration (yet anyway) so was mainly trying to call attention to it | 13:43 |
| @jim:acmegating.com | oops, sorry my chat was out of sync but it's better now. :) i followed up in #gerritcodereview:matrix.org | 14:02 |
| @fungicide:matrix.org | thanks, i too was in the middle of system upgrades and browser restarts | 14:40 |
| @fungicide:matrix.org | and also my google cla problem still hasn't been sorted out from what i can tell | 14:43 |
| @clarkb:matrix.org | We should ask them about that today. Maybe it is a syncing issue? | 14:44 |
| @jim:acmegating.com | zuul-maint: https://review.opendev.org/860988 is the next step in our release odyssey (deprecate ansible 5) | 16:38 |
| @vlotorev:matrix.org | > following discussion in the #gerritcodereview channel, it seems there's some zuul confusion. i started to look into it, but it's unclear to me why it doesn't have links to build results in its comments, only -1 votes with no additional detail that i can find: https://gerrit-review.googlesource.com/c/gitiles/+/344594 | 17:37 |
| @fungi: Here is a ticket for restoring comments/messages from CI in Gerrit Changelog panel https://bugs.chromium.org/p/gerrit/issues/detail?id=15620. It seems only related when Checks API is used. | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 18:48 | |
| - [zuul/zuul] 860984: DNM: run lots of unit tests https://review.opendev.org/c/zuul/zuul/+/860984 | ||
| - [zuul/zuul] 861102: Add timer driver debugging https://review.opendev.org/c/zuul/zuul/+/861102 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 19:09 | |
| - [zuul/zuul] 860984: DNM: run lots of unit tests https://review.opendev.org/c/zuul/zuul/+/860984 | ||
| - [zuul/zuul] 861104: Fix shutdown race in timer driver https://review.opendev.org/c/zuul/zuul/+/861104 | ||
| @jim:acmegating.com | i think i'm making progress on the current batch of test races; i think 861104 may fix the failure that manifests as "half the tests throw the 'More than one thread is running' exception" (i think that is related to some of the job timeouts too) | 19:11 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 20:05 | |
| - [zuul/zuul] 861104: Fix shutdown race in timer driver https://review.opendev.org/c/zuul/zuul/+/861104 | ||
| - [zuul/zuul] 860984: DNM: run lots of unit tests https://review.opendev.org/c/zuul/zuul/+/860984 | ||
| @clarkb:matrix.org | corvus: are you thinking the debugging changes are worth landing permanently? or should I wait for the stack to settle a bit before reviewing? | 20:15 |
| @jim:acmegating.com | Clark: i think everything but the DNM is okay to merge (i don't think anything is too chatty), but i'm open to suggestions if folks want to reduce that. | 20:24 |
| @jim:acmegating.com | also i have confirmed 861104 looks good in a local test run, so i think it's ready for review | 20:25 |
| @clarkb:matrix.org | fungi: maybe you have time for https://review.opendev.org/c/zuul/zuul/+/860988 to keep the zuul schedule on track? | 20:34 |
| @clarkb:matrix.org | I think this is highly dependent on node selection but the py310 jobs often run in about 40 minutes | 20:38 |
| @clarkb:matrix.org | I think that is a good 20% quicker than py38 at its best | 20:39 |
| @jim:acmegating.com | Clark: i agree with the observation | 20:40 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 860987: Change merge mode default based on driver https://review.opendev.org/c/zuul/zuul/+/860987 | 21:56 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 860987: Change merge mode default based on driver https://review.opendev.org/c/zuul/zuul/+/860987 | 21:57 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 860988: Deprecate Ansible 5, make Ansible 6 the default https://review.opendev.org/c/zuul/zuul/+/860988 | 23:21 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!