| @clarkb:matrix.org | corvus: ack looking now | 00:00 |
|---|---|---|
| @clarkb:matrix.org | done all three are approved | 00:04 |
| @clarkb:matrix.org | the first one is straightforward enough especially with all three having successful test runs that I felt ok single core approving it. Also a test only change | 00:04 |
| @clarkb:matrix.org | its not clear to me why it thinks that is a stale queue though | 00:05 |
| @jim:acmegating.com | Clark: i agree. thanks! | 00:05 |
| @clarkb:matrix.org | I guess maybe because we do it out of band of the scheduler? | 00:05 |
| @jim:acmegating.com | basically there's a scheduler running in the background of that test | 00:05 |
| @jim:acmegating.com | and it gets the lock, and sees a queue in zk it doesn't know about so it deletes it | 00:06 |
| @jim:acmegating.com | the test method is what created the queue and it did so without holding the lock | 00:06 |
| @clarkb:matrix.org | ya ok so its out of band of the running scheduler | 00:06 |
| @jim:acmegating.com | (even with that fix, it might still end up cleaning up the queue, but at least everything is locked now so there shouldn't be any version mismatches) | 00:06 |
| @jim:acmegating.com | yep | 00:07 |
| @clarkb:matrix.org | corvus: fwiw ze01 has begun its shutdown process for automated updates. I believe this fix only needs schedulers to update though? | 00:08 |
| @clarkb:matrix.org | I'm not too worried about it as a result since the schedulers happen last and we should be hours from that | 00:08 |
| @clarkb:matrix.org | maybe next week we want to try and land the sqla 2.0 stack? Though I'm traveling the week after that so can't help much if something goes wrong. But reverts should be fine | 00:24 |
| @clarkb:matrix.org | or we can restart things earlier than the automated system in opendev to get quicker feedback | 00:24 |
| @jim:acmegating.com | Clark: ack thx | 01:29 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 873437: Fix race in test_queue unit tests https://review.opendev.org/c/zuul/zuul/+/873437 | 06:15 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 872482: Fix race condition in pipeline change list init https://review.opendev.org/c/zuul/zuul/+/872482 | 06:19 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 873410: Remove layout_uuid from PipelineState create call https://review.opendev.org/c/zuul/zuul/+/873410 | 07:08 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 873470: Match events to pipelines based on topic deps https://review.opendev.org/c/zuul/zuul/+/873470 | 18:14 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 873037: Add OpenStack volume quota https://review.opendev.org/c/zuul/nodepool/+/873037 | 19:18 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 20:13 | |
| - [zuul/nodepool] 872722: Add scheduler, volumes, and labels to k8s/openshift https://review.opendev.org/c/zuul/nodepool/+/872722 | ||
| - [zuul/nodepool] 873471: Add support for specifying pod resource limits https://review.opendev.org/c/zuul/nodepool/+/873471 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 873470: Match events to pipelines based on topic deps https://review.opendev.org/c/zuul/zuul/+/873470 | 20:46 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 870821: Update build/job versions in place https://review.opendev.org/c/zuul/zuul/+/870821 | 20:48 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 873470: Match events to pipelines based on topic deps https://review.opendev.org/c/zuul/zuul/+/873470 | 21:52 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!