| @westphahl:matrix.org | Clark: could you take a look at my comment here https://review.opendev.org/c/zuul/zuul/+/944175/3/zuul/manager/__init__.py | 06:49 |
|---|---|---|
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 944598: Retry moto describe_instance exceptions https://review.opendev.org/c/zuul/zuul/+/944598 | 07:56 | |
| -@gerrit:opendev.org- Dong Zhang proposed wip: [zuul/zuul] 944131: WIP: Refactor KeyStorage to utilize ZKObject https://review.opendev.org/c/zuul/zuul/+/944131 | 13:47 | |
| @clarkb:matrix.org | That is some good feedback. I like your suggestion. I'll try to get that pushed up at some point today | 14:51 |
| -@gerrit:opendev.org- Dong Zhang proposed wip: [zuul/zuul] 944131: WIP: Refactor KeyStorage to utilize ZKObject https://review.opendev.org/c/zuul/zuul/+/944131 | 15:32 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 889431: Fix null dereference in gitlab https://review.opendev.org/c/zuul/zuul/+/889431 | 15:34 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 944175: Fix pause event verification test race https://review.opendev.org/c/zuul/zuul/+/944175 | 15:35 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 940872: Implement keystore functions for OIDC RS256 https://review.opendev.org/c/zuul/zuul/+/940872 | 15:43 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 940872: Implement keystore functions for OIDC RS256 https://review.opendev.org/c/zuul/zuul/+/940872 | 15:56 | |
| @mhuin:matrix.org | hello zuul-maint, this change has 2 +2's https://review.opendev.org/c/zuul/zuul-jobs/+/943586 could we get the final workflow+1 on it please? | 16:00 |
| @fungicide:matrix.org | clarkb: ^ you commented on that a week ago as the second +2 that you might be able to keep an eye out for possible impact later | 16:02 |
| @clarkb:matrix.org | ya I guess my main concern is that it affects a foundational role, but that role has testing so its probably fine? | 16:04 |
| @clarkb:matrix.org | especially with the openstack release efforts going on I didn't want to be the one responsible for landing it unless I was really able to debug, fix, etc | 16:05 |
| @clarkb:matrix.org | today is probably a better spot for me that when I reviewed it. Let me eat breakfast and if there are not objections in half an hour or so I can approve it (or someone else can) | 16:05 |
| @jim:acmegating.com | what's going on with https://review.opendev.org/c/zuul/zuul-jobs/+/927582 ? it seems to have persistent test failures | 16:07 |
| @fungicide:matrix.org | that looks like a change in test environment has started trying to install boto into the global system python context on ubuntu noble | 16:10 |
| @fungicide:matrix.org | within the zuul-jobs-test-upload-logs-s3 job | 16:10 |
| @fungicide:matrix.org | looks like that's been impacting other unrelated changes as well | 16:11 |
| @tristanc_:matrix.org | corvus: perhaps we need https://review.opendev.org/c/zuul/zuul-jobs/+/927600 first ? | 16:13 |
| @fungicide:matrix.org | jim: https://review.opendev.org/c/zuul/zuul-jobs/+/927600 seems to be the fix | 16:13 |
| @fungicide:matrix.org | all changes not depending on 927600 failed zuul-jobs-test-upload-logs-s3 starting somewhere between 2024-12-20 and 2025-01-16 looks like, unless they didn't run that job for (irrelevant-)files filter reasons | 16:15 |
| @fungicide:matrix.org | maybe we switched the job from ubuntu-jammy to ubuntu-noble around that time and didn't spot it? | 16:16 |
| @clarkb:matrix.org | mhu: tristanC: are 927582 and 943586 complimentary changes? | 16:19 |
| @clarkb:matrix.org | I think they are as they address different roles right? | 16:19 |
| @tristanc_:matrix.org | Clark: 943586 let user fixes the issue on their own, and 927582 does it automatically | 16:19 |
| @tristanc_:matrix.org | they are complementary yes | 16:20 |
| @jim:acmegating.com | tristanC: i left a question on 582 | 16:23 |
| @jim:acmegating.com | personally, i'd love to see 582 merged before 586, since it does fix an issue for everyone automatically; 586 is fine as an option, but i'd rather we not force people into using 586 because 582 is stalled. | 16:23 |
| @tristanc_:matrix.org | oops, good point, let me revisit that | 16:24 |
| @jim:acmegating.com | tristanC: cool, rest of 582 lgtm | 16:24 |
| @clarkb:matrix.org | maybe cross check the testing that 586 adds and see if any of that should be ported t o582? | 16:24 |
| @clarkb:matrix.org | I've approved the s3 job test file | 16:25 |
| @clarkb:matrix.org | * I've approved the s3 job test fix | 16:25 |
| @fungicide:matrix.org | thanks! | 16:25 |
| -@gerrit:opendev.org- Tristan Cacqueray https://matrix.to/#/@tristanc_:matrix.org proposed: [zuul/zuul-jobs] 927582: Update the set-zuul-log-path-fact scheme to prevent huge url https://review.opendev.org/c/zuul/zuul-jobs/+/927582 | 16:43 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 944107: Allow creating (sharded) ZKObjects w/o makepath https://review.opendev.org/c/zuul/zuul/+/944107 | 17:10 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: | 17:30 | |
| - [zuul/zuul] 944108: Implement launcher relative priority handling https://review.opendev.org/c/zuul/zuul/+/944108 | ||
| - [zuul/zuul] 944109: Add launcher relative priority tests https://review.opendev.org/c/zuul/zuul/+/944109 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 944175: Fix pause event verification test race https://review.opendev.org/c/zuul/zuul/+/944175 | 19:12 | |
| @clarkb:matrix.org | tristanC: minio is available from quay. We should update 927600 to pull from quay instead of docker hub if this latest recheck fails | 19:57 |
| @clarkb:matrix.org | rather than rechecking again | 19:57 |
| -@gerrit:opendev.org- Zuul merged on behalf of Tristan Cacqueray https://matrix.to/#/@tristanc_:matrix.org: [zuul/zuul-jobs] 927600: Fix the upload-logs-s3 test playbook https://review.opendev.org/c/zuul/zuul-jobs/+/927600 | 20:00 | |
| @clarkb:matrix.org | I'll push a new change for that since it just merged | 20:01 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 944801: Pull minio from quay instead of docker hub https://review.opendev.org/c/zuul/zuul-jobs/+/944801 | 20:02 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul-jobs] 944801: Pull minio from quay instead of docker hub https://review.opendev.org/c/zuul/zuul-jobs/+/944801 | 20:28 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:40 | |
| - [zuul/zuul-jobs] 944812: Add upload-image-swift role https://review.opendev.org/c/zuul/zuul-jobs/+/944812 | ||
| - [zuul/zuul-jobs] 944813: Add upload-image-s3 role https://review.opendev.org/c/zuul/zuul-jobs/+/944813 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 944813: Add upload-image-s3 role https://review.opendev.org/c/zuul/zuul-jobs/+/944813 | 21:57 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 944813: Add upload-image-s3 role https://review.opendev.org/c/zuul/zuul-jobs/+/944813 | 22:12 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 944813: Add upload-image-s3 role https://review.opendev.org/c/zuul/zuul-jobs/+/944813 | 22:49 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!