| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul-jobs] 966020: Fix ensure-rust wrapper script https://review.opendev.org/c/zuul/zuul-jobs/+/966020 | 10:11 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 966033: Don't log request token in log streamer https://review.opendev.org/c/zuul/zuul/+/966033 | 12:31 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 966033: Don't log request token in log streamer https://review.opendev.org/c/zuul/zuul/+/966033 | 12:48 | |
| @fungicide:matrix.org | it shouldn't be funny, but "Increase unit test timeout" (965977) failed in the gate on a TIMED_OUT result for both zuul-nox-py311 and zuul-nox-py312 (as well as zuul-nox-zuul-client) | 19:23 |
|---|---|---|
| @fungicide:matrix.org | the unit test jobs timed out at 3 hours, so presumably there's something else going on there | 19:24 |
| @jim:acmegating.com | that's a real problem that's fixed by https://review.opendev.org/965890 | 21:05 |
| @jim:acmegating.com | that has to land first before anything else can. i'm still hoping to be able to do that, but if it fails again, i'll probably squash it with 965977 | 21:06 |
| @jim:acmegating.com | (unfortunately, the current mix of opendev test nodes is not conducive to getting 890 through without 977) | 21:07 |
| @fajfer:reszka.org | independent researchers also show there's a high possibility of the source code being leaked from our project | 21:11 |
| @fungicide:matrix.org | hah | 21:17 |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 965890: Pin graphql-core in tests to < 3.2.7 https://review.opendev.org/c/zuul/zuul/+/965890 | 21:28 | |
| @mnaser:matrix.org | is nodepool development fully paused at this point? | 21:33 |
| @fungicide:matrix.org | mnaser: feature development yes probably, bug fixes are still fair game | 21:35 |
| @mnaser:matrix.org | I just had https://review.opendev.org/c/zuul/nodepool/+/961330 but wasn't sure if i should move that over to zuul or where-ever that lives now :) | 21:36 |
| @mnaser:matrix.org | has opendev fully moved off nodepool? i wonder if it's worth us moving from it too since i am just about to do a bunch of stuff now.. | 21:38 |
| @fungicide:matrix.org | i think we would want to make sure that any new nodepool functionality also makes it into zuul-launcher, at the very least. so like https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/aws/awsprovider.py maybe in this case | 21:40 |
| @fungicide:matrix.org | opendev is no longer using nodepool at all since several months ago, but the replacement isn't quite considered prime-time yet and therefore isn't recommended for general use. we still merge backward-incompatible fixes and things that require significant config adjustments while we're filing down the rough edges | 21:41 |
| @mnaser:matrix.org | i see, so it's gonna be a little bit | 21:41 |
| @fungicide:matrix.org | best guess, last i heard, is early 2026 | 21:42 |
| @mnaser:matrix.org | gonna have to dig in git history to find the opendev nodepool configs because they have been historically a great example of how to do things :) | 21:42 |
| @fungicide:matrix.org | mnaser: b22bdcf in openstack/project-config was the removal commit, a75a11e is the merge-commit that stitched it into master, so you want ba62a3e which is its trunk parent | 21:45 |
| @fungicide:matrix.org | git checkout ba62a3e ought to get you the final state of our nodepool configs before we ripped them out | 21:45 |
| @mnaser:matrix.org | nice, thats helpful | 21:46 |
| @fungicide:matrix.org | worth noting we ran for months with both nodepool-builder/launcher and zuul-launcher/jobs, so they can coexist to some extent | 21:48 |
| @fungicide:matrix.org | once it's generally recommended, that'll hopefully remain a viable migration path | 21:49 |
| @fungicide:matrix.org | they did end up competing inefficiently for provider quota, but it was still workable even if a bit clunky | 21:51 |
| @fungicide:matrix.org | we moved things over incrementally, label/image at a time | 21:52 |
| @fungicide:matrix.org | and also different tenants at different times | 21:52 |
| @jim:acmegating.com | yes, please don't start using zuul-launcher until it's ready -- we'll make an announcement about it | 21:52 |
| @mnaser:matrix.org | yeah, i can imagine that it we would probably similar, though we don't have much volume, so we can a bit easier just flip things through once the builds are working ok | 21:53 |
| @fungicide:matrix.org | part of why we moved things over so slowly was that we still weren't 100% certain about things like the configuration model, and worked through changes to the overall system as new load exposed further bugs or inefficiencies. once it's stabilized maybe an all-at-once switchover wouldn't be too bad for most environments | 21:55 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 965977: Increase unit test timeout https://review.opendev.org/c/zuul/zuul/+/965977 | 23:05 | |
| @fajfer:reszka.org | interesting, I was wondering why 11 is still the newest version but it makes more sense now | 23:09 |
| @fajfer:reszka.org | btw I have some time again for Zuul, I was wondering if I should push for removing swagger? I've also upgraded React but it needs some more tidying up: | 23:11 |
| 956167: Update React version to 17.0.2 | https://review.opendev.org/c/zuul/zuul/+/956167 | ||
| 958645: Remove Swagger/OpenAPI components | https://review.opendev.org/c/zuul/zuul/+/958645 | ||
| @fajfer:reszka.org | basically it all boils down to the fact that create-react-app hasn't been maintained since 2001 and was considered deprecated this february | 23:11 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 965954: Launcher: honor provider when selecting ready nodes https://review.opendev.org/c/zuul/zuul/+/965954 | 23:38 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!