| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 940872: Implement keystore functions for OIDC RS256 https://review.opendev.org/c/zuul/zuul/+/940872 | 11:11 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 940872: Implement keystore functions for OIDC RS256 https://review.opendev.org/c/zuul/zuul/+/940872 | 11:24 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 940872: Implement keystore functions for OIDC RS256 https://review.opendev.org/c/zuul/zuul/+/940872 | 11:26 | |
| @martinpeck:matrix.org | Hello. I'm new to this room, so thought say a quick "hello". I'm keen to get a better understanding of the compute provider model for Zuul, and to understand how it can be extended. If anyone has some good pointers for getting into this topic let me know...otherwise I'll carry on reviewing the docs/source code. | 12:05 |
|---|---|---|
| @fungicide:matrix.org | martinpeck: it's in the midst of a major overhaul at this moment. traditionally, zuul's nodepool component has been responsible for compute resource lifecycle management, but this spec details the revised model that's in the process of being implemented now: https://zuul-ci.org/docs/zuul/latest/developer/specs/nodepool-in-zuul.html | 13:06 |
| @fungicide:matrix.org | most of the changes for it are tagged "niz" for ease of discoverability: https://review.opendev.org/q/hashtag:%22niz%22 | 13:11 |
| @mnaser:matrix.org | is there plans of a Zuul release soon? There’s quite a few fixes we’re waiting for and I’m wondering if I just go ahead and patch locally vs wait | 13:16 |
| @martinpeck:matrix.org | Interesting. Thanks for this fungi . It's useful to know that this is in flux right now. | 13:39 |
| @jim:acmegating.com | mnaser: i don't think now is a great time for a release due to what's in progress and hasn't seen live testing yet. | 14:47 |
| @mnaser:matrix.org | ah yeah, i figured, i fell into the void of figuring out authorization being broken for whitelabled tenants till i found the commit that you fixed it :) | 14:48 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 940824: Use niz nodes for unit tests https://review.opendev.org/c/zuul/zuul/+/940824 | 15:08 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 940824: Use niz nodes for unit tests https://review.opendev.org/c/zuul/zuul/+/940824 | 15:15 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 940824: Use niz nodes for unit tests https://review.opendev.org/c/zuul/zuul/+/940824 | 15:19 | |
| @fungicide:matrix.org | it's queued! | 15:20 |
| @jim:acmegating.com | indeed -- as i watch it run here... i think we're at the point of maybe removing some debug log lines from the launcher. :) | 15:25 |
| @fungicide:matrix.org | linters succeeded and the others are running | 15:25 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 940888: Remove some launcher debug log messages https://review.opendev.org/c/zuul/zuul/+/940888 | 15:27 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 940891: Use annotated logger with nodescan requests https://review.opendev.org/c/zuul/zuul/+/940891 | 15:40 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 940893: Fix node listing web API https://review.opendev.org/c/zuul/zuul/+/940893 | 16:09 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 940895: Launcher: log selected provider https://review.opendev.org/c/zuul/zuul/+/940895 | 16:32 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 940898: Launcher: fix image decompression https://review.opendev.org/c/zuul/zuul/+/940898 | 17:10 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 18:51 | |
| - [zuul/zuul] 940910: Launcher: improve nodescan testing https://review.opendev.org/c/zuul/zuul/+/940910 | ||
| - [zuul/zuul] 940911: Launcher: move key-name in test fixtures https://review.opendev.org/c/zuul/zuul/+/940911 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 19:05 | |
| - [zuul/zuul] 940910: Launcher: improve nodescan testing https://review.opendev.org/c/zuul/zuul/+/940910 | ||
| - [zuul/zuul] 940911: Launcher: move key-name in test fixtures https://review.opendev.org/c/zuul/zuul/+/940911 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 19:07 | |
| - [zuul/zuul] 940910: Launcher: improve nodescan testing https://review.opendev.org/c/zuul/zuul/+/940910 | ||
| - [zuul/zuul] 940911: Launcher: move key-name in test fixtures https://review.opendev.org/c/zuul/zuul/+/940911 | ||
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834045: [upload-logs-base] add ipa extension to known mime types https://review.opendev.org/c/zuul/zuul-jobs/+/834045 | 19:20 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 20:03 | |
| - [zuul/zuul] 940910: Launcher: improve nodescan testing https://review.opendev.org/c/zuul/zuul/+/940910 | ||
| - [zuul/zuul] 940911: Launcher: move key-name in test fixtures https://review.opendev.org/c/zuul/zuul/+/940911 | ||
| - [zuul/zuul] 940929: Fix nested provider validation with hyphens https://review.opendev.org/c/zuul/zuul/+/940929 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 940933: Pin googleapis-common-protos https://review.opendev.org/c/zuul/zuul/+/940933 | 21:42 | |
| @jim:acmegating.com | Clark: fungi ^ the assemble/install-from-bindep process did not handle that well ^ | 21:43 |
| @jim:acmegating.com | example failure: https://zuul.opendev.org/t/zuul/build/a790a6a6fda74adc8a4c0c70c5dc48be | 21:43 |
| @jim:acmegating.com | that's mostly an fyi. the patch resolves the issue; but i mention it in case anyone feels like digging more. | 21:44 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:47 | |
| - [zuul/zuul] 940888: Remove some launcher debug log messages https://review.opendev.org/c/zuul/zuul/+/940888 | ||
| - [zuul/zuul] 940891: Use annotated logger with nodescan requests https://review.opendev.org/c/zuul/zuul/+/940891 | ||
| - [zuul/zuul] 940893: Fix node listing web API https://review.opendev.org/c/zuul/zuul/+/940893 | ||
| - [zuul/zuul] 940895: Launcher: log selected provider https://review.opendev.org/c/zuul/zuul/+/940895 | ||
| - [zuul/zuul] 940898: Launcher: fix image decompression https://review.opendev.org/c/zuul/zuul/+/940898 | ||
| - [zuul/zuul] 940929: Fix nested provider validation with hyphens https://review.opendev.org/c/zuul/zuul/+/940929 | ||
| - [zuul/zuul] 940910: Launcher: improve nodescan testing https://review.opendev.org/c/zuul/zuul/+/940910 | ||
| - [zuul/zuul] 940911: Launcher: move key-name in test fixtures https://review.opendev.org/c/zuul/zuul/+/940911 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:52 | |
| - [zuul/zuul] 940898: Launcher: fix image decompression https://review.opendev.org/c/zuul/zuul/+/940898 | ||
| - [zuul/zuul] 940929: Fix nested provider validation with hyphens https://review.opendev.org/c/zuul/zuul/+/940929 | ||
| - [zuul/zuul] 940910: Launcher: improve nodescan testing https://review.opendev.org/c/zuul/zuul/+/940910 | ||
| - [zuul/zuul] 940911: Launcher: move key-name in test fixtures https://review.opendev.org/c/zuul/zuul/+/940911 | ||
| @fungicide:matrix.org | that dep solver issue is really odd, pip install normally shouldn't consider pre-release versions of packages unless it's either been passed the --pre option or a pre-release version was used in the version specifier for the package (though perhaps that was done within a transitive dependency?) | 22:07 |
| @jim:acmegating.com | fungi: certainly not directly. possibly transitive. it downloaded both; the rc1 later after the non-rc. | 22:11 |
| @fungicide:matrix.org | man, that monorepo they use for opentelemetry is massive | 22:11 |
| @fungicide:matrix.org | it does mention .dev versions of some deps in the pyproject.toml, so maybe that's sufficient for it to look for prereleases of the others too: https://github.com/open-telemetry/opentelemetry-python/blob/main/exporter/opentelemetry-exporter-otlp-proto-grpc/pyproject.toml#L35-L37 | 22:14 |
| @jim:acmegating.com | i heard you like dev versions | 22:14 |
| @jim:acmegating.com | have some more | 22:14 |
| @fungicide:matrix.org | basically | 22:15 |
| @mordred:waterwanders.com | yeah. I think the logic is "if you see a pre-release version mentioned, then set --pre to true" | 22:17 |
| @mordred:waterwanders.com | or something like that | 22:17 |
| @jim:acmegating.com | interesting that the rc1 download happens so late (i mean, like, a couple pages of text scrolling by later)... seems like it leaves open the question of when exactly that bit was flipped | 22:18 |
| @jim:acmegating.com | so then the other weird thing: pip wheel only built the rc1 wheel; but then later pip install didn't want to install it, it wanted to install the non-rc. | 22:19 |
| @mordred:waterwanders.com | yeah - that's the part that bothers me more - that makes me think we have a logic bug in assemble somewhere | 22:20 |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834046: [upload-logs-base] add android mime-type https://review.opendev.org/c/zuul/zuul-jobs/+/834046 | 22:39 | |
| @aureliojargas:matrix.org | Hey, anyone in for an easy review? Adding `ensure-uv` role https://review.opendev.org/c/zuul/zuul-jobs/+/940271 (same as current `ensure-poetry` role, just changing all `poetry` mentions to `uv`) | 22:43 |
| @clarkb:matrix.org | I wonder if we should make a meta role for these ensure thinsg that pip can install roles | 23:16 |
| @clarkb:matrix.org | I don't think we need to solve that before ensure-uv lands, but an idea since poetry and twine and others are all basically the same system | 23:16 |
| @aureliojargas:matrix.org | yeah, I thought the same | 23:17 |
| @aureliojargas:matrix.org | the current code repetition seems "wrong" :) | 23:17 |
| @clarkb:matrix.org | then ensure-uv, ensure-twine, ensure-poetry can just pass in a packge name, version, and path | 23:18 |
| @clarkb:matrix.org | the change looks good to me other than this idea | 23:18 |
| @aureliojargas:matrix.org | I can give it a try on the meta role if no one else is interested on doing it | 23:20 |
| @jim:acmegating.com | sgtm; i'd review that | 23:27 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 23:37 | |
| - [zuul/zuul] 940910: Launcher: improve nodescan testing https://review.opendev.org/c/zuul/zuul/+/940910 | ||
| - [zuul/zuul] 940911: Launcher: move key-name in test fixtures https://review.opendev.org/c/zuul/zuul/+/940911 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Aurelio Jargas: [zuul/zuul-jobs] 940271: Add ensure-uv role https://review.opendev.org/c/zuul/zuul-jobs/+/940271 | 23:43 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!