| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 917782: Speed up merger git resets https://review.opendev.org/c/zuul/zuul/+/917782 | 09:56 | |
| -@gerrit:opendev.org- Andrei Dmitriev proposed: [zuul/nodepool] 916732: Add ssh-private-key to OpenStack CloudImage https://review.opendev.org/c/zuul/nodepool/+/916732 | 11:33 | |
| -@gerrit:opendev.org- Andrei Dmitriev proposed: [zuul/nodepool] 916732: Add ssh-private-key to OpenStack CloudImage https://review.opendev.org/c/zuul/nodepool/+/916732 | 11:37 | |
| -@gerrit:opendev.org- Andrei Dmitriev proposed: [zuul/nodepool] 916732: Add ssh-private-key to OpenStack CloudImage https://review.opendev.org/c/zuul/nodepool/+/916732 | 11:39 | |
| -@gerrit:opendev.org- Andrei Dmitriev proposed: [zuul/nodepool] 916732: Add ssh-private-key to OpenStack CloudImage https://review.opendev.org/c/zuul/nodepool/+/916732 | 12:04 | |
| -@gerrit:opendev.org- Andrei Dmitriev proposed: [zuul/nodepool] 916732: Add ssh-private-key to OpenStack CloudImage https://review.opendev.org/c/zuul/nodepool/+/916732 | 12:16 | |
| -@gerrit:opendev.org- Andrei Dmitriev proposed: [zuul/nodepool] 916732: Add ssh-private-key to OpenStack CloudImage https://review.opendev.org/c/zuul/nodepool/+/916732 | 12:31 | |
| -@gerrit:opendev.org- Andrei Dmitriev proposed: [zuul/nodepool] 916732: Add ssh-private-key to OpenStack CloudImage https://review.opendev.org/c/zuul/nodepool/+/916732 | 13:24 | |
| -@gerrit:opendev.org- Andrei Dmitriev proposed: [zuul/nodepool] 916732: Add ssh-private-key to OpenStack CloudImage https://review.opendev.org/c/zuul/nodepool/+/916732 | 13:40 | |
| @jpew:matrix.org | `tests.unit.test_v3.TestEarlyFailure.test_pre_run_failure_retry` seems to failing somewhat consistently on Python 3.12? | 18:10 |
|---|---|---|
| @clarkb:matrix.org | > <@jpew:matrix.org> `tests.unit.test_v3.TestEarlyFailure.test_pre_run_failure_retry` seems to failing somewhat consistently on Python 3.12? | 18:24 |
| Looking at the log on the chagne I suspect you saw this on it seems that the test job we're iterating waiting for took 34 seconds to complete but the test suite is only waiting 30 seconds before the wait and raise if timeout happens | ||
| @clarkb:matrix.org | its possible that python3.12 is making things generally slower or differences in runtime are exposing some sort of race condition that causes things to take longer | 18:24 |
| @jpew:matrix.org | mmm, makes sense. Needs a longer timeout perhaps? | 18:25 |
| @clarkb:matrix.org | that might be a good quick workaround | 18:25 |
| @clarkb:matrix.org | sorry it was 36 seconds | 18:25 |
| @clarkb:matrix.org | looks like the jobs are doing an ansible wait_for on a file being written to. maybe we need to have that poll the file more often too | 18:26 |
| @clarkb:matrix.org | looks like ansible waits 1 second between checks by default | 18:27 |
| @clarkb:matrix.org | reading a bit more I'm wondering if it is never writing the flag file to let that job continue and that is why it is hitting the timeout. | 18:31 |
| @clarkb:matrix.org | ok I think that job that took 36 seconds is the 4th "wait" job build and we're really waiting on three pre-failure jobs to finish | 18:36 |
| @clarkb:matrix.org | ya my new read on this is that the third job finished up near that 30 seconds limit and we got "lucky". We can probably make the looping and/or ansible a little faster but bumping the timeout is probably reasonable | 18:38 |
| @clarkb:matrix.org | another thought would be to configure zuul to only allow 2 retries instead of three | 18:40 |
| @clarkb:matrix.org | that should cover the same code paths in a quicker amount of time | 18:41 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 918338: Check pre run failure cases with only 2 retry attempts https://review.opendev.org/c/zuul/zuul/+/918338 | 18:48 | |
| @clarkb:matrix.org | something like that maybe | 18:48 |
| @jim:acmegating.com | curious that it's suddenly so persistent with 3.12. there was an ansible release on apr 24, but that doesn't quite line up. | 18:54 |
| -@gerrit:opendev.org- Zuul merged on behalf of Joshua Watt: [zuul/zuul] 914818: gerrit: Add `approval-change` trigger https://review.opendev.org/c/zuul/zuul/+/914818 | 19:33 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 913314: Limit bytes when reading Ansible output https://review.opendev.org/c/zuul/zuul/+/913314 | 20:15 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!