| @jim:acmegating.com | ianw: thanks! i reviewed the early ones, i'll take a closer look at the later ones tomorrow | 00:03 |
|---|---|---|
| @iwienand:matrix.org | no rush, thanks | 00:11 |
| -@gerrit:opendev.org- Ian Wienand proposed: | 00:24 | |
| - [zuul/zuul] 855297: web: console: compress datalist https://review.opendev.org/c/zuul/zuul/+/855297 | ||
| - [zuul/zuul] 855574: web: task summary: make more consistent with other tabs https://review.opendev.org/c/zuul/zuul/+/855574 | ||
| - [zuul/zuul] 855575: web: console: better task modal titles https://review.opendev.org/c/zuul/zuul/+/855575 | ||
| - [zuul/zuul] 855578: web: build/autohold: fix props input https://review.opendev.org/c/zuul/zuul/+/855578 | ||
| @iwienand:matrix.org | that fixes padding around the expandable content with an !important. i'm going to put localhost:3000 away now, I have to stop fiddling with it :) | 00:25 |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853208: zuul-stream : Test against a Python 2.7 container https://review.opendev.org/c/zuul/zuul/+/853208 | 03:58 | |
| @jim:acmegating.com | zuul-maint: next steps for ansible: | 15:56 |
| * deprecate ansible 2, make ansible 5 the default | ||
| * add ansible 6 | ||
| * remove ansible 2 | ||
| * deprecate ansible 5, make ansible 6 the default | ||
| * remove ansible 5 | ||
| @jim:acmegating.com | zuul-maint: which of those should be major releases? | 15:56 |
| @tobias.henkel:matrix.org | All but the second maybe? | 15:58 |
| @jim:acmegating.com | i was thinking maybe just the "remove" steps | 16:01 |
| @jim:acmegating.com | (fwiw, previously we have never done a major version for any kind of ansible version change, even a removal; the closest was that we added ansible 5 with zuul 6, but that was only because we relaxed the executor restrictions) | 16:02 |
| @jim:acmegating.com | (so there's not a lot of precedent to go on here) | 16:02 |
| @jim:acmegating.com | if we do want to do major versions for all but the second, then i can cut 7.0 today/this weekend, then 8.0 next week... maybe we can combine "remove ansible2" and "deprecate ansible5, make ansible 6 the default" into one step. | 16:05 |
| @jim:acmegating.com | https://etherpad.opendev.org/p/qF7VE9HzqPVzsCyZLWxb | 16:09 |
| @clarkb:matrix.org | ya I think switching defaults can probably be done without a major release | 16:10 |
| @clarkb:matrix.org | but removing ansible versions people may be depending on should be signaled with a major release | 16:11 |
| @jim:acmegating.com | okay, i sketched those two ideas out on the etherpad | 16:11 |
| @clarkb:matrix.org | we should also skim for other things that can/should be removed at the same time. The merge-conflict/merge-failure compat can be cleaned up for example (I forget which is canonical now) | 16:11 |
| @jim:acmegating.com | Clark: yeah, that's proposed in 854046 and is ready to go into the next major release | 16:12 |
| @jim:acmegating.com | i think we still need to do the queue removal | 16:12 |
| @clarkb:matrix.org | oh yup. I know openstack hasn't full removed that yet, but I send emails about it and even pushed a few chagnes to some projects. So I think opendev is as ready as we can be (openstack tenant was the only one with that issue) | 16:13 |
| @jim:acmegating.com | i'1ll work on that change now | 16:14 |
| @jim:acmegating.com | zomg i just found a zuulv1 layout in the test fixtures :) | 16:26 |
| @jim:acmegating.com | i stared at it for like 30 seconds not understanding it haha | 16:26 |
| @clarkb:matrix.org | corvus: question on https://review.opendev.org/c/zuul/zuul/+/854046 about a couple other places that might need cleanup | 16:27 |
| @jim:acmegating.com | Clark: replied -- those can stay | 16:33 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 855691: Remove deprecated pipeline queue configuration https://review.opendev.org/c/zuul/zuul/+/855691 | 17:03 | |
| @jim:acmegating.com | tobiash: what do you think of option 2 on the etherpad? is it okay to do major releases just for the removals? | 17:05 |
| @tobias.henkel:matrix.org | Works for me | 17:06 |
| @jim:acmegating.com | zuul-maint: how does this look for a zuul release? commit 64fa9db5799d33838267183f5c6734d4ba5ee7c9 (HEAD -> master, tag: 6.3.0, origin/master, gerrit/master) | 17:09 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 17:14 | |
| - [zuul/zuul] 854009: Fix Ansible version testing https://review.opendev.org/c/zuul/zuul/+/854009 | ||
| - [zuul/zuul] 853548: Deprecate Ansible 2, make Ansible 5 default https://review.opendev.org/c/zuul/zuul/+/853548 | ||
| - [zuul/zuul] 853552: Add Ansible 6 https://review.opendev.org/c/zuul/zuul/+/853552 | ||
| - [zuul/zuul] 854046: Remove merge-failure pipeline configuration https://review.opendev.org/c/zuul/zuul/+/854046 | ||
| - [zuul/zuul] 855691: Remove deprecated pipeline queue configuration https://review.opendev.org/c/zuul/zuul/+/855691 | ||
| @clarkb:matrix.org | corvus: that appears to be about 7 commits ahead of opendev | 17:18 |
| @clarkb:matrix.org | Are you thinking of tagging that next week after the opendev restarts or just going for it now or? | 17:18 |
| @jim:acmegating.com | Clark: yeah, maybe monday for that? | 17:22 |
| @jim:acmegating.com | maybe we can do each release in the etherpad every week | 17:25 |
| @clarkb:matrix.org | wfm | 17:29 |
| @fungicide:matrix.org | i like option #2 in the pad | 18:23 |
| @fungicide:matrix.org | i agree that deprecation, addition, or default changes can be minor releases. removing support for ansible versions seems more in line with a major release | 18:24 |
| @iwienand:matrix.org | corvus: minor but could we get https://review.opendev.org/c/zuul/zuul/+/851942 in to 6.3 ; the doc update to the streaming bits; would be good to keep it together. https://review.opendev.org/c/zuul/zuul/+/853208 is related but not particularly important for any release, to test with a 2.7 container. i've also added a follow-on there to disable writing out streaming spool files with https://review.opendev.org/c/zuul/zuul/+/855309 | 21:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!