| @y2kenny:matrix.org | Hi, what is the expected behavior when a Timer trigger is used with job.branches filter? I understand from the doc that Timer enqueue all branches of the project so would job.branches limit the job execution to only the listed branches? | 12:38 |
|---|---|---|
| @y2kenny:matrix.org | right now I seems to get no trigger at all after adding job.branches to the job. I seems to have some vague recollection that this is expected but I just want to double check. | 12:48 |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul-jobs] 986645: FIX(roles/ensure-javascript-packages): Install version exact https://review.opendev.org/c/zuul/zuul-jobs/+/986645 | 13:09 | |
| @fungicide:matrix.org | Kenny Ho: is the job defined in a single-branch repository, or in a multi-branch repository? | 13:17 |
| @fungicide:matrix.org | oh, though you're putting it on the trigger not the job definition | 13:18 |
| @fungicide:matrix.org | Kenny Ho: have you tried instead putting the filter on the job variant where it's added to that pipeline, e.g. https://opendev.org/openstack/project-config/src/branch/master/zuul.d/projects.yaml#L3652 | 13:21 |
| @y2kenny:matrix.org | the job is defined in a single-branch repository externally (in `zuul-jobs`). The trigger is defined in another trusted repository (zuul-projects) for the repo in question | 13:24 |
| @fungicide:matrix.org | alternatively, looks like you could set it on a project stanza: https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.branches | 13:25 |
| @y2kenny:matrix.org | basically we wanted to have a periodic trigger acting on an upstream repo that does not use Zuul themselves | 13:25 |
| @y2kenny:matrix.org | so we define the job in an untrusted project so we can test the job and then put the trigger in a trusted project | 13:25 |
| @y2kenny:matrix.org | without the jobs.branches, the job was triggered on all the upstream branches as expected. | 13:26 |
| @fungicide:matrix.org | i don't see branches listed as an option for `project.pipeline` sections though: https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.%3Cpipeline%3E | 13:26 |
| @y2kenny:matrix.org | the branches was added to the job definition | 13:26 |
| @fungicide:matrix.org | you can do `branches` in a job variant (my first example) where it's added to the `project.pipeline` | 13:27 |
| @y2kenny:matrix.org | um... ok. I am not sure I understand fully but I will take a look. | 13:30 |
| @y2kenny:matrix.org | we haven't been using template but may be we should be | 13:31 |
| @y2kenny:matrix.org | I did not realize there's a project.branches... I will try that as well | 13:34 |
| @fungicide:matrix.org | doesn't need to be in a template, can just be wherever you're adding the pipeline to the project | 13:40 |
| -@gerrit:opendev.org- Kaloyan Kotlarski proposed: [zuul/zuul-operator] 986657: Allow the operator to configure Zuul authentication https://review.opendev.org/c/zuul/zuul-operator/+/986657 | 13:51 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 986659: Several fixes https://review.opendev.org/c/zuul/zuul/+/986659 | 14:02 | |
| -@gerrit:opendev.org- Ruisi Jian proposed: [zuul/zuul] 986663: mqtt-reporter: include change info on buildset start https://review.opendev.org/c/zuul/zuul/+/986663 | 14:20 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 986659: Several fixes https://review.opendev.org/c/zuul/zuul/+/986659 | 14:25 | |
| -@gerrit:opendev.org- Kaloyan Kotlarski proposed: [zuul/zuul-operator] 986657: Allow the operator to configure Zuul authentication https://review.opendev.org/c/zuul/zuul-operator/+/986657 | 14:46 | |
| -@gerrit:opendev.org- Kaloyan Kotlarski proposed: [zuul/zuul-operator] 986657: Allow the operator to configure Zuul authentication https://review.opendev.org/c/zuul/zuul-operator/+/986657 | 14:53 | |
| -@gerrit:opendev.org- Kaloyan Kotlarski proposed: [zuul/zuul-operator] 986657: Allow the operator to configure Zuul authentication https://review.opendev.org/c/zuul/zuul-operator/+/986657 | 15:03 | |
| -@gerrit:opendev.org- Kaloyan Kotlarski proposed: [zuul/zuul-operator] 986657: Allow the operator to configure Zuul authentication https://review.opendev.org/c/zuul/zuul-operator/+/986657 | 15:06 | |
| -@gerrit:opendev.org- Kaloyan Kotlarski proposed: [zuul/zuul-operator] 986657: Allow the operator to configure Zuul authentication https://review.opendev.org/c/zuul/zuul-operator/+/986657 | 15:09 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 986659: Several fixes https://review.opendev.org/c/zuul/zuul/+/986659 | 16:10 | |
| -@gerrit:opendev.org- Ruisi Jian proposed: [zuul/zuul] 986663: mqtt-reporter: include change info on buildset start https://review.opendev.org/c/zuul/zuul/+/986663 | 16:10 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 986695: Upload-logs: skip symlinks https://review.opendev.org/c/zuul/zuul-jobs/+/986695 | 16:39 | |
| @jim:acmegating.com | We just made a Zuul release with some important security fixes: https://lists.zuul-ci.org/archives/list/zuul-announce@lists.zuul-ci.org/thread/AANNLH37KT7JNG5QPBEWSLT7YZ4CKEEV/ | 17:22 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 986699: Re-enable python 3.12 tests https://review.opendev.org/c/zuul/zuul/+/986699 | 17:52 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 986699: Re-enable python 3.12 tests https://review.opendev.org/c/zuul/zuul/+/986699 | 22:05 | |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!