| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 957176: Add AsList to openstack driver https://review.opendev.org/c/zuul/zuul/+/957176 | 15:24 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 958553: Remove model backwards compat https://review.opendev.org/c/zuul/zuul/+/958553 | 15:51 | |
| @jim:acmegating.com | we could merge that before this release, or, if we want to defer the status-url removal until the next release, i can probably separate that out and we can merge the rest of the cleanup now. but the status-url in particular will force a major version rev, so whatever release comes after it will be a major bump. | 15:57 |
|---|---|---|
| @jim:acmegating.com | * we could merge that before this release, or, if we want to defer the status-url removal until the next release, i can probably separate that out and we can merge the rest of the cleanup now. but the status-url in particular will force a major version rev of whatever release comes after it. | 15:57 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 958553: Remove model backwards compat https://review.opendev.org/c/zuul/zuul/+/958553 | 15:59 | |
| @clarkb:matrix.org | corvus: I think I caught a small syntax bug in that change. I don't think I'm in a good position to comment on status-url removal as I'm not sure opendev ever used it | 16:15 |
| @clarkb:matrix.org | from opendev's perspective I think its fine to bundle up status-url removal. It has been deprecated for some time now | 16:16 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 16:16 | |
| - [zuul/zuul] 958470: Shutdown log stream socket instead of close https://review.opendev.org/c/zuul/zuul/+/958470 | ||
| - [zuul/zuul] 958471: Handle retries in zuul_stream https://review.opendev.org/c/zuul/zuul/+/958471 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 958553: Remove model backwards compat https://review.opendev.org/c/zuul/zuul/+/958553 | 16:34 | |
| @jim:acmegating.com | Clark: thanks, fixed and replied | 16:34 |
| @clarkb:matrix.org | +2 thanks | 16:38 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 18:32 | |
| - [zuul/zuul] 941046: Shorten temp test path (1/2) https://review.opendev.org/c/zuul/zuul/+/941046 | ||
| - [zuul/zuul] 958570: Shorten temp test path (2/2) https://review.opendev.org/c/zuul/zuul/+/958570 | ||
| -@gerrit:opendev.org- Damian Fajfer proposed: [zuul/zuul] 958592: Set Server HTTP header to empty https://review.opendev.org/c/zuul/zuul/+/958592 | 22:03 | |
| @clarkb:matrix.org | corvus: should the `configure_mirrors_extra_repos` switch from default True to False announcement go to zuul-discuss or zuul-announce? | 22:38 |
| @jim:acmegating.com | are we breaking backwards compat or requiring user action? | 22:38 |
| @jim:acmegating.com | (sorry i don't remember details) | 22:38 |
| @jim:acmegating.com | "I think the idea was to notify zuul users that zuul-jobs would update to match upstream" you said | 22:39 |
| @clarkb:matrix.org | ya today that config option is set to True which means it will install extra repos on centos and debian. On debian the extra repo is backports. On centos its additional repos for tools. So the user action is tehy will either need to flip back to True or otherwise enable those repos in their jobs if they rely on packages from them | 22:40 |
| @jim:acmegating.com | i think the answer is probably zuul-announce, just to be safe | 22:41 |
| @clarkb:matrix.org | the reason we want to do this is that debian in particular is deleting the backports repo entirely for older releases so it doesn't make sense to have enabled by default as we're breaking in that situation | 22:41 |
| @clarkb:matrix.org | so we'd like to be "working by default with default repos" and give people the option for opting into potentially breaking themselves with extra repos as a choice | 22:41 |
| @jim:acmegating.com | so anyone using the default is currently broken on old repos, and working on new ones | 22:41 |
| @clarkb:matrix.org | ya | 22:41 |
| @jim:acmegating.com | but if we change the default, currently working jobs on new repos *might* break if they rely on this and don't set it | 22:42 |
| @clarkb:matrix.org | correct | 22:42 |
| @jim:acmegating.com | if that's all true, then i think zuul-announce with a 2 week notice | 22:42 |
| @clarkb:matrix.org | sounds good. I'll get the change up first so I can link to it and people can look at it concretely. But then I'll draft something for zuul-announce with a 2 week notice | 22:42 |
| @jim:acmegating.com | ++thx! | 22:42 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 958605: Default configure_mirrors_extra_repos to False in configure-mirrors https://review.opendev.org/c/zuul/zuul-jobs/+/958605 | 22:46 | |
| @clarkb:matrix.org | Thats the change. I'll WIP it now | 22:46 |
| @clarkb:matrix.org | and email is sent. It will probably need to be moderated through | 22:57 |
| @fajfer:reszka.org | I've also sent an email last week but I think it got stuck in moderation | 23:04 |
| @fajfer:reszka.org | regarding https://review.opendev.org/c/zuul/zuul-jobs/+/957219 | 23:04 |
| @clarkb:matrix.org | fajfer: I think that email made it through | 23:06 |
| @clarkb:matrix.org | https://lists.zuul-ci.org/archives/list/zuul-announce@lists.zuul-ci.org/thread/YMOXVUMEBFAQH7C7VD4HSCS723GI5CYK/ | 23:06 |
| @fajfer:reszka.org | huh, it looks like it was delivered so fast I didn't realize | 23:07 |
| @fajfer:reszka.org | sorry then, my bad:) | 23:07 |
| @jim:acmegating.com | no it was stuck in moderation until today, sorry i didn't notice it. feel free to ping me if i ever miss something like that again. | 23:38 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!