| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 855890: Don't try to report build w/o buildset to DB https://review.opendev.org/c/zuul/zuul/+/855890 | 13:38 | |
| @jim:acmegating.com | zuul-maint: how does this revised tag for zuul look? commit df8874a560631d38f988f27c2c4aad731ad96178 (HEAD -> master, tag: 6.3.0, origin/master) | 15:26 |
|---|---|---|
| @clarkb:matrix.org | OpenDev has been running that for a few hours and I haven't seen anyone scream about it. 855890 doesn't seem critical enough to hold up a release? Lgtm | 15:29 |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 854505: Add Ansible version to a job's Zuul vars https://review.opendev.org/c/zuul/zuul/+/854505 | 15:48 | |
| @jim:acmegating.com | swest: i did +2 that change because it makes sense for that to be available in the job (since it's also an input to the job). but fyi, for the use cases cited in the commit message (selecting appropriate linter, etc), it would probably be most technically correct to use {{ansible_version}}. there is a 1:1 relationship between ansible and the major+minor version of ansible-core. we rely on that in the test suite (the test suite has to do the things you mentioned in the commit message too). | 15:59 |
| @westphahl:matrix.org | corvus: yeah, I just think for users the community version might be easier to understand as that's the one configured in the job. So from that perspective I think it's more consistent to do feature switch based on that | 16:19 |
| @jim:acmegating.com | yeah, it's the difference between "correct" and "technically correct" :) | 16:25 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 855966: Use json view for complex artifact metadata https://review.opendev.org/c/zuul/zuul/+/855966 | 16:26 | |
| @jim:acmegating.com | i'll go ahead and push zuul 6.3.0 now | 16:51 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 855801: Add nodeset alternatives https://review.opendev.org/c/zuul/zuul/+/855801 | 19:54 | |
| @jim:acmegating.com | release failed: https://zuul.opendev.org/t/zuul/build/602f515a0cf34688bf37560c8197ad9c | 19:55 |
| @jim:acmegating.com | it's an opendev problem; fix in https://review.opendev.org/855980 | 20:03 |
| @clarkb:matrix.org | I've approved the fix | 20:33 |
| @jim:acmegating.com | i re-enqueued the tag | 21:12 |
| @jim:acmegating.com | released and email sent | 21:42 |
| @iwienand:matrix.org | thanks, sorry about that | 22:00 |
| @jim:acmegating.com | ianw: np, was easy fix. :) | 23:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!