Skip to content

Set fail-fast to true in upstream zuul project#3280

Merged
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:fail-fast
Sep 10, 2025
Merged

Set fail-fast to true in upstream zuul project#3280
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:fail-fast

Conversation

@amartyasinha
Copy link
Copy Markdown
Contributor

fail-fast allows us to fail the entire pipeline the moment any job from the pipeline fails. Currently, we have to wait for all jobs to return their status for the pipeline to get complete. It wastes compute resoures as well as time.

@amartyasinha amartyasinha requested a review from a team as a code owner September 10, 2025 08:41
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Sep 10, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@softwarefactory-project-zuul
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/182390de39f54ed4aaea7f7e84b43d61

⚠️ openstack-k8s-operators-content-provider CANCELED
⚠️ podified-multinode-edpm-deployment-crc CANCELED
⚠️ cifmw-crc-podified-edpm-baremetal CANCELED
cifmw-pod-zuul-files FAILURE in 5m 02s
✔️ noop SUCCESS in 0s
⚠️ cifmw-pod-ansible-test CANCELED
⚠️ cifmw-pod-pre-commit CANCELED
⚠️ build-push-container-cifmw-client CANCELED

fail-fast allows us to fail the entire pipeline the moment any job from the pipeline fails. Currently, we have to wait for all jobs to return their status for the pipeline to get complete. It wastes compute resoures as well as time.
Copy link
Copy Markdown
Contributor

@dasm dasm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FTR: https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.%3Cpipeline%3E.fail-fast

@amartyasinha thank you for the PR. I approved it initially, because that looks solid. Now I need to remove my approval.
I'd like to hear from CI-DevOps team to see how we're moving forward with CI-FMW promotions.

@dasm dasm self-requested a review September 10, 2025 14:33
@amartyasinha amartyasinha merged commit 5215b4f into openstack-k8s-operators:main Sep 10, 2025
4 checks passed
@danpawlik
Copy link
Copy Markdown
Contributor

I'm just concerned, that If that is enabled, post job would not be executed in other job. It means, that infra cleaner needs to be executed more often.

@amartyasinha
Copy link
Copy Markdown
Contributor Author

I had enabled auto-merge, and seems like it got merged. Let's see if it is impacting promotions or infra cleaning. If it happens, we can revert this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants