| -@gerrit:opendev.org- Tobias Henkel proposed: [zuul/nodepool] 842538: Add more logs to cleanup workers https://review.opendev.org/c/zuul/nodepool/+/842538 | 13:18 | |
| -@gerrit:opendev.org- Simon Westphahl proposed on behalf of Tobias Henkel: [zuul/nodepool] 842538: Add more logs to cleanup workers https://review.opendev.org/c/zuul/nodepool/+/842538 | 13:21 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 837852: Clarify zuul admin CLI scope https://review.opendev.org/c/zuul/zuul/+/837852 | 13:35 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 842290: Remove zuul CLI, REST ops https://review.opendev.org/c/zuul/zuul/+/842290 | 14:20 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Tobias Henkel: [zuul/nodepool] 842538: Add more logs to cleanup workers https://review.opendev.org/c/zuul/nodepool/+/842538 | 15:29 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Joshua Watt: [zuul/nodepool] 839226: Do not reset quota cache timestamp when invalid https://review.opendev.org/c/zuul/nodepool/+/839226 | 15:47 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 842109: Don't fail config loading on missing min. ltime https://review.opendev.org/c/zuul/zuul/+/842109 | 16:06 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/nodepool] 839681: AWS tests: cleanup image deletion checking https://review.opendev.org/c/zuul/nodepool/+/839681 | 16:26 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 16:34 | |
| - [zuul/zuul-jobs] 842572: Re-sync test-mirror-workspace-git-repos https://review.opendev.org/c/zuul/zuul-jobs/+/842572 | ||
| - [zuul/zuul-jobs] 842573: Correct git config item name in mirror-workspace-git-repos https://review.opendev.org/c/zuul/zuul-jobs/+/842573 | ||
| @jim:acmegating.com | zuul-maint: ^ those are easy approvals since they're only to a test role; that will allow us to test a fix that appears to be necessary for ansible 5 in a base-test cycle | 16:35 |
|---|---|---|
| @jim:acmegating.com | https://zuul.opendev.org/t/zuul/build/a5243fd91bf4440ba03bd0dd694d429b/console is an example failure | 16:36 |
| @clarkb:matrix.org | corvus: I'll take a look once I've sent this deprecated queues syntax email to the openstack mailing list | 16:39 |
| @jim:acmegating.com | Clark: thx | 16:40 |
| @clarkb:matrix.org | corvus: for the deprecated syntax it will break CI for the project using it or for all projects in the tenant? | 16:47 |
| @fungicide:matrix.org | presumably it has to be corrected in stable branches too? or does only teh default branch decide the queue? | 16:48 |
| @fungicide:matrix.org | * presumably it has to be corrected in all branches too? or does only the default branch decide the queue? | 16:48 |
| @clarkb:matrix.org | it has to be on all branches | 16:48 |
| @clarkb:matrix.org | because what will break is the parsing of the config | 16:48 |
| @fungicide:matrix.org | ahh, good point | 16:49 |
| @jim:acmegating.com | yeah... there's nuance about which project stanzas zuul encounters first determining which queue the project joins; but in this case i think that takes a back seat to the "breaking the config" part which i think reduces the problem to the point where it just needs to be fixed universally | 16:56 |
| @clarkb:matrix.org | corvus: meaning the entire tenant may be broken? | 16:57 |
| @jim:acmegating.com | Clark: to your question -- it will cause that project stanza to be ignored; so if you leave it on a stable branch, then that stable branch won't run any jobs until fixed | 16:57 |
| @jim:acmegating.com | shouldn't break other branches on the project or the tenant in general | 16:57 |
| @clarkb:matrix.org | got it so branch specific | 16:57 |
| @clarkb:matrix.org | mostly want to be able to tell people what the impact will be if left as is | 16:57 |
| @jim:acmegating.com | it will also be many, many config errors in the ui, masking any real problems | 16:58 |
| @clarkb:matrix.org | would it be any change to any project on that stable branch or just the projcet with the broken stable config? | 16:58 |
| @jim:acmegating.com | just the project | 16:59 |
| @clarkb:matrix.org | another reason I'm asking is projects like windmill have the deprecated syntax and it isn't clear to me what the future of that project is | 16:59 |
| @jim:acmegating.com | basically zuul is going to drop the whole "project" stanza. | 16:59 |
| @clarkb:matrix.org | got it | 16:59 |
| @fungicide:matrix.org | so jobs added in a central config project will still run potentially, but jobs added in the project's git repository will be ignored | 17:01 |
| @jim:acmegating.com | fungi: yup | 17:01 |
| @jim:acmegating.com | and this applies to periodic, etc, jobs which are added in-repo (but not centrally) | 17:01 |
| @fungicide:matrix.org | and changes may merge based on the results of the jobs which still run? leading to changes landing in a minimally-tested state | 17:02 |
| @jim:acmegating.com | yup | 17:02 |
| @fungicide:matrix.org | that seems like the biggest risk, so good to call out on the opendev side when we're bringing it up with projects | 17:02 |
| -@gerrit:opendev.org- Zuul merged on behalf of Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org: [zuul/nodepool] 841908: Clarify reasons for host-key-checking setting https://review.opendev.org/c/zuul/nodepool/+/841908 | 17:17 | |
| @jpew:matrix.org | I can't quite figure out why https://review.opendev.org/c/zuul/zuul/+/838924 timed out, would anyone be able to help me sort it our? Near as I can tell, the tests just run longer than the timeout and they aren't "stuck" per-se | 17:25 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 842572: Re-sync test-mirror-workspace-git-repos https://review.opendev.org/c/zuul/zuul-jobs/+/842572 | 18:01 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 842573: Correct git config item name in mirror-workspace-git-repos https://review.opendev.org/c/zuul/zuul-jobs/+/842573 | 18:03 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 842598: Make test-prepare-workspace-git role https://review.opendev.org/c/zuul/zuul-jobs/+/842598 | 18:11 | |
| @jim:acmegating.com | zuul-maint: ^ oops, looks like we need that for a base-job test cycle too... i think that's probably new since the last time we did one for these roles. | 18:12 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul-client] 771853: Add show running-jobs subcommand https://review.opendev.org/c/zuul/zuul-client/+/771853 | 18:40 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 842598: Make test-prepare-workspace-git role https://review.opendev.org/c/zuul/zuul-jobs/+/842598 | 18:40 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 842361: zuul-web: harmonize behavior for boolean parameters https://review.opendev.org/c/zuul/zuul/+/842361 | 18:44 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul-client] 752909: Add buildsets, buildset-info to subcommands https://review.opendev.org/c/zuul/zuul-client/+/752909 | 19:03 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul-client] 751238: Add console-stream subcommand https://review.opendev.org/c/zuul/zuul-client/+/751238 | 19:09 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 758985: zuul-client: Test buildsets, buildset-info subcommands https://review.opendev.org/c/zuul/zuul/+/758985 | 19:12 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 804177: Add include- and exclude-branches tenant config options https://review.opendev.org/c/zuul/zuul/+/804177 | 19:20 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 771962: zuul-client: test "show running-jobs" command https://review.opendev.org/c/zuul/zuul/+/771962 | 19:24 | |
| @fungicide:matrix.org | > <@jpew:matrix.org> I can't quite figure out why https://review.opendev.org/c/zuul/zuul/+/838924 timed out, would anyone be able to help me sort it our? Near as I can tell, the tests just run longer than the timeout and they aren't "stuck" per-se | 19:25 |
| that same job passed for the change a week ago. looking at the log, it spent roughly 15 minutes getting everything set up and then churned through unit tests steadily without any huge gaps in time, finally killed after the remaining 75 (out of its initial 90) minutes ran out while it was still slogging through the unit tests | ||
| @fungicide:matrix.org | the successful run for it a week ago took "1 hr 11 mins 52 secs" according to the result page | 19:26 |
| @fungicide:matrix.org | the job nodes in opendev are wildly inconsistent performance-wise, so we may simply need to increase the timeout or prune some of the testing | 19:27 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 842598: Make test-prepare-workspace-git role https://review.opendev.org/c/zuul/zuul-jobs/+/842598 | 19:29 | |
| @fungicide:matrix.org | jpew: that previous run was within 19% of hitting the timeout, and our test nodes vary in performance by a lot more than that | 19:30 |
| @fungicide:matrix.org | i would recheck it for an updated result | 19:30 |
| @fungicide:matrix.org | i went ahead and did that | 19:32 |
| @jpew:matrix.org | fungi: Thanks! | 19:33 |
| @jpew:matrix.org | Well, it's regate actually | 19:33 |
| @fungicide:matrix.org | they have the same effect in our gate pipeline config | 19:34 |
| @fungicide:matrix.org | comparing the successful run, it was 20% faster setting up to start the unit tests, though the setup might be impacted differently by performance variance than the unit tests themselves | 19:36 |
| @jpew:matrix.org | Also, I just had Zuul spin up 450 openstack nodes and it seems to be keeping itself together pretty well..... This is _so_ much better that what we had before! | 19:40 |
| -@gerrit:opendev.org- Zuul merged on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/zuul] 837852: Clarify zuul admin CLI scope https://review.opendev.org/c/zuul/zuul/+/837852 | 20:42 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Joshua Watt: [zuul/zuul] 838924: merger: Handle merges with cherry-pick merge-mode https://review.opendev.org/c/zuul/zuul/+/838924 | 20:45 | |
| @vlotorev:matrix.org | Hi, could zuul maint have a look at https://review.opendev.org/#/c/833515, it already has one CR+2. | 20:59 |
| @vlotorev:matrix.org | * Hi, could zuul maint have a look at https://review.opendev.org/c/zuul/nodepool/+/833515, it already has one CR+2. | 21:00 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 23:51 | |
| - [zuul/zuul-jobs] 842647: Test base-test with Ansible 2.8 https://review.opendev.org/c/zuul/zuul-jobs/+/842647 | ||
| - [zuul/zuul-jobs] 842648: Test base-test with Ansible 2.9 https://review.opendev.org/c/zuul/zuul-jobs/+/842648 | ||
| - [zuul/zuul-jobs] 842649: Test base-test with Ansible 5 https://review.opendev.org/c/zuul/zuul-jobs/+/842649 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 842243: DNM: Try jobs with Ansible 5 https://review.opendev.org/c/zuul/zuul/+/842243 | 23:51 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!