| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 962449: Record missing info in job-output.json https://review.opendev.org/c/zuul/zuul/+/962449 | 01:15 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 04:10 | |
| - [zuul/zuul] 960927: Launcher: add max-age https://review.opendev.org/c/zuul/zuul/+/960927 | ||
| - [zuul/zuul] 961292: Launcher: handle reused node failure https://review.opendev.org/c/zuul/zuul/+/961292 | ||
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 962760: Use global S3 endpoint to get bucket location https://review.opendev.org/c/zuul/zuul/+/962760 | 08:31 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 961557: Assign unassigned building nodes to requests https://review.opendev.org/c/zuul/zuul/+/961557 | 09:26 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 962145: Use a subnode for request assignment https://review.opendev.org/c/zuul/zuul/+/962145 | 11:27 | |
| -@gerrit:opendev.org- Jan Gutter proposed: [zuul/zuul-jobs] 962794: Update ensure-helm role to add more functionality https://review.opendev.org/c/zuul/zuul-jobs/+/962794 | 13:44 | |
| -@gerrit:opendev.org- Jan Gutter proposed: [zuul/zuul-jobs] 962794: Update ensure-helm role to add more functionality https://review.opendev.org/c/zuul/zuul-jobs/+/962794 | 13:46 | |
| -@gerrit:opendev.org- Jan Gutter proposed: [zuul/zuul-jobs] 962794: Update ensure-helm role to add more functionality https://review.opendev.org/c/zuul/zuul-jobs/+/962794 | 14:21 | |
| @jangutter:matrix.org | Any zuul-jobs users out there using the ensure-helm role? I've updated it a bit and may have gone overboard. | 14:40 |
|---|---|---|
| @fungicide:matrix.org | jangutter: codesearch.opendev.org turns up several users there... https://opendev.org/openstack/openstack-helm/src/branch/master/playbooks/build-chart.yaml#L18 https://opendev.org/openstack/openstack-helm/src/branch/master/playbooks/lint.yaml#L20 https://opendev.org/starlingx/app-gen-tool/src/branch/master/playbooks/app-gen-tool-tox-coverage/pre.yaml#L3 https://opendev.org/vexxhost/openstack-operator/src/branch/master/playbooks/functional/pre.yaml#L21 | 14:43 |
| @fungicide:matrix.org | looks like mnaser might be a good one to provide input on behalf of vexxhost/openstack-operator | 14:44 |
| @jangutter:matrix.org | Yeah, looking at the version of helm pinned in the defaults (oof), I think folks using this role likely rely on pinned behavior. | 14:46 |
| @jim:acmegating.com | i'll just add that these days we try to avoid pinning versions in defaults, and instead expect users to specify versions (if necessary) when they use roles/jobs | 14:49 |
| @jim:acmegating.com | and i think typically any migration away from versions in defaults would be welcome (but needs to be announced and handled with care) | 14:49 |
| @jangutter:matrix.org | Yeah - I think a good argument can be made that removing a version from the default here is warranted because the bundled one is properly ancient. I'm deliberately not doing it in this patch because I know the pain that can be caused with accidentally unconstrained linters. | 14:54 |
| @jangutter:matrix.org | Looks like openstack-helm is the most active, (using a pinned helm version), so I'll check there to see if things work as expected. | 14:55 |
| @jim:acmegating.com | if you remove it, making that a standalone change (so that people can deal with that individually) is a good idea | 15:08 |
| -@gerrit:opendev.org- Szymon Datko proposed: [zuul/zuul] 962831: [minor] Correct GitLab driver doc https://review.opendev.org/c/zuul/zuul/+/962831 | 15:13 | |
| @jangutter:matrix.org | Good news is that the failures in openstack/openstack-helm seems to be unrelated. It's exercising the role as intended. | 16:47 |
| @fungicide:matrix.org | that's refreshing news | 16:48 |
| @jangutter:matrix.org | mnaser: if you have time, would https://review.opendev.org/c/zuul/zuul-jobs/+/962794 make sense to you? | 17:12 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 961551: Launcher: clean tempdir at start https://review.opendev.org/c/zuul/zuul/+/961551 | 17:16 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Szymon Datko: [zuul/zuul] 962831: [minor] Correct GitLab driver doc https://review.opendev.org/c/zuul/zuul/+/962831 | 18:21 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 962498: Update launcher schema print tool https://review.opendev.org/c/zuul/zuul/+/962498 | 18:32 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 962864: Update gerrit change_url format https://review.opendev.org/c/zuul/zuul/+/962864 | 18:44 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!