| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/nodepool] 870683: driver/openstack: order flavor results https://review.opendev.org/c/zuul/nodepool/+/870683 | 00:58 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 869376: wip: try out orjson https://review.opendev.org/c/zuul/zuul/+/869376 | 14:01 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/nodepool] 870683: driver/openstack: order flavor results https://review.opendev.org/c/zuul/nodepool/+/870683 | 14:40 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 870821: Update build/job versions in place https://review.opendev.org/c/zuul/zuul/+/870821 | 14:45 | |
| @morucci:matrix.org | Hi corvus thanks for your feedback for the Gitlab driver "diff_refs" issue, I've updated according to your comment and added some more testing: https://review.opendev.org/c/zuul/zuul/+/867804 | 14:56 |
|---|---|---|
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Fabien Boucher: [zuul/zuul] 867804: Handle missing diff_refs attribute https://review.opendev.org/c/zuul/zuul/+/867804 | 15:03 | |
| @jim:acmegating.com | fbo: thanks -- there was an unecessary rebase in there that made it hard to review the inter-patchset diff, so i have rebased it on the same parent as ps2 | 15:04 |
| @vonschultz:matrix.org | We've been running with 867333 (Fix AWS quota limits for vCPUs | https://review.opendev.org/c/zuul/nodepool/+/867333) in production since late last year, and Nodepool is much better at respecting AWS quota limits this way. Any chance I could get a review? Should I add someone specific as a reviewer in Gerrit? | 15:08 |
| @jim:acmegating.com | Christian von Schultz: that's on my list; expect something from me later this week. thanks for your patience! :) | 15:11 |
| @morucci:matrix.org | corvus: ok 👍️ | 15:12 |
| @jim:acmegating.com | fbo: and lgtm | 15:13 |
| @morucci:matrix.org | \o/ | 15:13 |
| @morucci:matrix.org | thanks for the review | 15:15 |
| @g_gobi:matrix.org | Hi, | 16:07 |
| I have a doubt about this https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.%3Cpipeline%3E.fail-fast | ||
| I have a pipeline with 3 jobs. | ||
| **Expected:** | ||
| If job 2 fails, Both job 1 & 3 will stop the execution. Mark the buildset as a failure. | ||
| **Happened:** | ||
| Job 2 failed but Job 1 and 3 executed entirely. | ||
| Pleas let me know If I'm reading it wrong. | ||
| @clarkb:matrix.org | > <@g_gobi:matrix.org> Hi, | 16:27 |
| > I have a doubt about this https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.%3Cpipeline%3E.fail-fast | ||
| > I have a pipeline with 3 jobs. | ||
| > **Expected:** | ||
| > If job 2 fails, Both job 1 & 3 will stop the execution. Mark the buildset as a failure. | ||
| > **Happened:** | ||
| > Job 2 failed but Job 1 and 3 executed entirely. | ||
| > Pleas let me know If I'm reading it wrong. | ||
| Can you share your config that is setting the value so that we can double check it is configured properly? | ||
| @g_gobi:matrix.org | ``` | 16:32 |
| - project: | ||
| name: zuul/testing1 | ||
| default-branch: main | ||
| merge-mode: squash-merge | ||
| gate-stage: | ||
| jobs: | ||
| - job1 | ||
| - job2 | ||
| - job3 | ||
| check-stage: | ||
| fail-fast: true | ||
| jobs: | ||
| - job1 | ||
| - job2 | ||
| - job3 | ||
| ``` | ||
| @clarkb:matrix.org | I would expect that to work. Its basically identical to the config in the test case for this functionality | 16:41 |
| @clarkb:matrix.org | Looks like the feature has been in place since Zuul 3.8.1. Are you running a new enough zuul? If so then you may need to dig into logs to see what is happening since this is tested by the test suite it should be working | 16:43 |
| @fajfer:matrix.org | tdlaw: I think it just finished almost at the same time thus it works like this. I have a fail-fast on many of my pipelines and they work as intended | 16:46 |
| @fajfer:matrix.org | > <@clarkb:matrix.org> Looks like the feature has been in place since Zuul 3.8.1. Are you running a new enough zuul? If so then you may need to dig into logs to see what is happening since this is tested by the test suite it should be working | 16:46 |
| This is a very good advice also so we won't be pondering the orb | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 870845: Honor independent pipeline requirements for non-live changes https://review.opendev.org/c/zuul/zuul/+/870845 | 16:59 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 870845: Honor independent pipeline requirements for non-live changes https://review.opendev.org/c/zuul/zuul/+/870845 | 16:59 | |
| @g_gobi:matrix.org | > <@fajfer:matrix.org> tdlaw: I think it just finished almost at the same time thus it works like this. I have a fail-fast on many of my pipelines and they work as intended | 17:05 |
| The job2 was failed in 30 mins. But job1 and 3 runs 1.5 hours. | ||
| @g_gobi:matrix.org | > <@clarkb:matrix.org> Looks like the feature has been in place since Zuul 3.8.1. Are you running a new enough zuul? If so then you may need to dig into logs to see what is happening since this is tested by the test suite it should be working | 17:05 |
| I'm using Zuul version: 4.8.2.dev14 25faf184 | ||
| @g_gobi:matrix.org | One info I missed here. Job 2 was non-voting. Is it related to voting? | 17:07 |
| @clarkb:matrix.org | no, voting shouldn't affect that if I read the code correctly. It just checks if fail fast is set then cancels the other builds https://opendev.org/zuul/zuul/src/tag/3.9.0/zuul/manager/__init__.py#L904-L910 | 17:10 |
| @clarkb:matrix.org | but looking for that debug log message is what I would do next and work back from that | 17:10 |
| @g_gobi:matrix.org | **build.job.voting** mentioned there in ln 904 | 17:11 |
| @clarkb:matrix.org | oh heh (I linked the wrong tag version too, but that hasn't changed) | 17:12 |
| @clarkb:matrix.org | sorry I originally parsed if the to be canceled jobs are non voting, But it does check if the failure is voting or not | 17:13 |
| @g_gobi:matrix.org | Thanks, Clark for letting me identify the reason | 17:14 |
| @g_gobi:matrix.org | > <@clarkb:matrix.org> but looking for that debug log message is what I would do next and work back from that | 17:16 |
| https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.%3Cpipeline%3E.debug Is this the only way to enable debug logs? Can we able to do it via command or override in job? | ||
| @jim:acmegating.com | tdlaw: `-d` on the cli in recent versions (your version is quite old, i'm not sure if it's present) | 17:16 |
| @jim:acmegating.com | * tdlaw: `-d` on the zuul-scheduler cli in recent versions (your version is quite old, i'm not sure if it's present) | 17:17 |
| @g_gobi:matrix.org | Wow, nice. Thanks, corvus . Is it in the enqueue command? | 17:18 |
| @jim:acmegating.com | when the zuul-scheduler process is started (sorry i edited my comment above) | 17:18 |
| @g_gobi:matrix.org | Got it. Thank you. | 17:20 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 870845: Honor independent pipeline requirements for non-live changes https://review.opendev.org/c/zuul/zuul/+/870845 | 17:37 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 870845: Honor independent pipeline requirements for non-live changes https://review.opendev.org/c/zuul/zuul/+/870845 | 17:37 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 870845: Honor independent pipeline requirements for non-live changes https://review.opendev.org/c/zuul/zuul/+/870845 | 19:16 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 869609: Re-elect James Blair as project lead https://review.opendev.org/c/zuul/zuul/+/869609 | 19:27 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!