| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 873834: Expand the scope of zkprofile https://review.opendev.org/c/zuul/zuul/+/873834 | 00:34 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Alex Hornung: [zuul/zuul] 871100: gerrit driver: fix bug around unicode branch names https://review.opendev.org/c/zuul/zuul/+/871100 | 04:04 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 869361: Add scheduler run handler metric https://review.opendev.org/c/zuul/zuul/+/869361 | 08:43 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 869362: Add more log messages for run handler steps https://review.opendev.org/c/zuul/zuul/+/869362 | 08:50 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 873834: Expand the scope of zkprofile https://review.opendev.org/c/zuul/zuul/+/873834 | 08:50 | |
| -@gerrit:opendev.org- Ade Lee proposed: [zuul/zuul-jobs] 873893: Add conditional for UA registration role https://review.opendev.org/c/zuul/zuul-jobs/+/873893 | 10:14 | |
| @tobias-urdin:matrix.org | can I subdir by moving {config-repo}/zuul.d/nodesets.yaml into {config-repo}/zuul.d/nodesets/*.yaml, would that be picked up? | 14:03 |
|---|---|---|
| @jim:acmegating.com | Tobias Urdin: yes; eg https://opendev.org/opendev/system-config/src/branch/master/zuul.d/docker-images | 14:11 |
| @tobias-urdin:matrix.org | great, thanks! | 14:11 |
| -@gerrit:opendev.org- Ade Lee proposed: [zuul/zuul-jobs] 873893: Add conditional for UA registration role https://review.opendev.org/c/zuul/zuul-jobs/+/873893 | 14:26 | |
| @tobias-urdin:matrix.org | corvus: hm, if I remove nodesets.yaml and introduce the same ones in a subdir I get "Nodeset <name here> already defined" for all those. Since it's a trusted repo do I need to merge a patch to remove nodesets.yaml and then (since Zuul would be broken) force merge a change that introduces the separate nodesets files into the trusted repo? | 14:36 |
| @tobias-urdin:matrix.org | zuul 5.0.0 if that helps | 14:43 |
| @jim:acmegating.com | Tobias Urdin: oh i don't know about the behavior in 5.0, it's very old -- i'm not even sure my original answer holds for that. i'd recommend verifying it experimentally first. | 14:48 |
| @tobias-urdin:matrix.org | ack, I will see if I get same error placing them directly in zuul.d, I assume the subdir is read since Zuul gives me a syntax error thought | 14:50 |
| @tobias-urdin:matrix.org | interesting, moving them directly into zuul.d worked so zuul does not complain about syntax. I guess I will hold off moving them to a sub directory in zuul.d until we've upgraded to a newer version | 15:06 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 872485: Simplufy shouldRefresh* checks https://review.opendev.org/c/zuul/zuul/+/872485 | 15:58 | |
| @clarkb:matrix.org | swest: can you check my comment on https://review.opendev.org/c/zuul/zuul/+/872908 I don't know if that was considered and rejected already | 17:41 |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul] 873742: merger: Keep redundant cherry-pick commits https://review.opendev.org/c/zuul/zuul/+/873742 | 17:50 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 873656: Add PBR_VERSION argument to Dockerfile https://review.opendev.org/c/zuul/zuul/+/873656 | 18:06 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 872405: Prevent files cache ltime from going backward https://review.opendev.org/c/zuul/zuul/+/872405 | 19:15 | |
| @arddennis:matrix.org | Hello, I have a question regarding collections installation for ansible. I need to have a fresh openstack.cloud collection. I tried to define it as job.roles for the untrusted project. Tried to use different options: galaxy, zuul. Tried to put the collection near to the playbooks just in the project repo, but still can't make it working. Is there a specific approach to install a collection? | 19:22 |
| @clarkb:matrix.org | > <@arddennis:matrix.org> Hello, I have a question regarding collections installation for ansible. I need to have a fresh openstack.cloud collection. I tried to define it as job.roles for the untrusted project. Tried to use different options: galaxy, zuul. Tried to put the collection near to the playbooks just in the project repo, but still can't make it working. Is there a specific approach to install a collection? | 19:26 |
| Historically you would have needed to include the collection in your zuul's ansible installs or use a nested ansible in the job. I'm not sure if the semi recent changes to allow more actions on the executor have changed this. | ||
| @jim:acmegating.com | no change there, those are still the options. see `zuul-manage-ansible` for the first option. | 19:28 |
| @jim:acmegating.com | the `zuul` roles only work for traditional ansible roles, not collections, so that's probably why that didn't work. | 19:32 |
| @jim:acmegating.com | (but i think you could override a built-in role with a zuul role if it does meet that criteria) | 19:32 |
| @arddennis:matrix.org | Isn't zuul-manage-ansible do the same, just try to install additional package into ansible venv? | 19:40 |
| @clarkb:matrix.org | Yes you would install the collections to the venvs | 20:10 |
| @clarkb:matrix.org | dong.zhang: I've left a comment on https://review.opendev.org/c/zuul/zuul/+/871259 | 20:49 |
| -@gerrit:opendev.org- Zuul merged on behalf of Dmitriy Rabotyagov: [zuul/zuul] 866874: Add project-ssh-key to API docs https://review.opendev.org/c/zuul/zuul/+/866874 | 20:54 | |
| -@gerrit:opendev.org- Christian Mueller proposed wip: [zuul/nodepool] 869015: Adding support to run all unit tests locally in a container (Debian Bullseye), making it easy to execute them without the need to first set-up the environment. https://review.opendev.org/c/zuul/nodepool/+/869015 | 21:00 | |
| -@gerrit:opendev.org- Christian Mueller marked as active: [zuul/nodepool] 869015: Adding support to run all unit tests locally in a container (Debian Bullseye), making it easy to execute them without the need to first set-up the environment. https://review.opendev.org/c/zuul/nodepool/+/869015 | 21:00 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 873694: Fix exception on retry in source base class https://review.opendev.org/c/zuul/zuul/+/873694 | 23:00 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!