| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 00:00 | |
| - [zuul/zuul] 966650: Revert "Make Gerrit Connection Replication Aware" https://review.opendev.org/c/zuul/zuul/+/966650 | ||
| - [zuul/zuul] 966651: Add replication_delay to Gerrit driver https://review.opendev.org/c/zuul/zuul/+/966651 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Mohammed Naser: [zuul/zuul] 966643: Switch to using assertEqual() in tests https://review.opendev.org/c/zuul/zuul/+/966643 | 16:36 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 966741: Launcher; use waitForSync as a fallback for older ZK https://review.opendev.org/c/zuul/zuul/+/966741 | 17:36 | |
| @mnaser:matrix.org | [@jim:acmegating.com](https://matrix.to/#/@jim:acmegating.com) sorry continuing in the right place. Ok no worries, if you have some digestible small bits I can pick up here and there. I saw PF has a log viewer component too which might be interesting.. | 18:04 |
|---|---|---|
| @jim:acmegating.com | mnaser: ack, will do :) | 18:05 |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: | 19:21 | |
| - [zuul/zuul] 966528: Enable IPv6 endpoint when IMDS support is enabled https://review.opendev.org/c/zuul/zuul/+/966528 | ||
| - [zuul/nodepool] 966526: Enable IPv6 endpoint when IMDS support is enabled https://review.opendev.org/c/zuul/nodepool/+/966526 | ||
| @clarkb:matrix.org | corvus: https://review.opendev.org/c/zuul/zuul/+/966501 appears to have a bunch of test case failures that look like they could be related to the change. I think in CI we are already using 3.9 so I don't think we need the fallback change earlier in the stack. Anyway I wanted to make sure that was known and hold off on reviewing if the second half of the stack is undergoing active updates to get it working (the first two changes lgtm. I approved the first and rechecked the second) | 19:51 |
| @clarkb:matrix.org | corvus: looks like timeouts waiting for nodes to be recycled. Maybe the self.wake_event.set() call isn't triggering the expected cache updates leading to timeouts? | 20:03 |
| @jim:acmegating.com | Clark: ack, thanks. i'll take another look in a bit. i think it's no rush on those. getting 501 in soon is good. the rest of the stack can/should all go in together when we're ready (so we land the fallback) | 21:02 |
| @clarkb:matrix.org | corvus: is it safe to land 501 before opendev upgrades zookeeper? I Think yes because the -1 value is the default in that case and we don't have the version check exception until the next change | 21:07 |
| @clarkb:matrix.org | just want to double check before I approve it too | 21:07 |
| @jim:acmegating.com | Clark: yes -- but only if we also merge 966741 -- so we want to get approvals on all those then +w them all | 21:08 |
| @jim:acmegating.com | * Clark: we want to merge 966501 through 966741 as a unit -- so we want to get approvals on all those then +w them all | 21:09 |
| @clarkb:matrix.org | got it. I won't approve 501 then | 21:10 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:39 | |
| - [zuul/zuul] 966502: Add k8s driver https://review.opendev.org/c/zuul/zuul/+/966502 | ||
| - [zuul/zuul] 966764: Add openshift support to k8s driver https://review.opendev.org/c/zuul/zuul/+/966764 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 966500: Delete unlocked used nodes https://review.opendev.org/c/zuul/zuul/+/966500 | 21:55 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:57 | |
| - [zuul/zuul] 966501: Ensure requests referred to in assignments are up to date https://review.opendev.org/c/zuul/zuul/+/966501 | ||
| - [zuul/zuul] 966637: Add min_request_version to assignUnassignedNode method https://review.opendev.org/c/zuul/zuul/+/966637 | ||
| - [zuul/zuul] 966638: Make ZuulTreeCache.waitForSync more robust https://review.opendev.org/c/zuul/zuul/+/966638 | ||
| - [zuul/zuul] 966741: Launcher; use waitForSync as a fallback for older ZK https://review.opendev.org/c/zuul/zuul/+/966741 | ||
| @jim:acmegating.com | Clark: was a test problem | 21:58 |
| @clarkb:matrix.org | corvus: got it. I'm popping out for a bit but will review when I return | 22:00 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!