Skip to content

Add cifmw_cleanup_architecture var to enable/disable cleanup #3081

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

raukadah
Copy link
Contributor

It will help to reuse the ocp environment when architecture deployment failed in pre step.

It will help to reuse the ocp environment when architecture deployment
failed in pre step.

Signed-off-by: Chandan Kumar (raukadah) <[email protected]>
@raukadah raukadah requested a review from a team as a code owner June 19, 2025 08:18
Copy link
Contributor

openshift-ci bot commented Jun 19, 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.

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

Copy link

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/b50a5fb2d5f044c1a51994e87cd7a008

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 43m 42s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 11m 15s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 28m 18s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 05s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 37s
build-push-container-cifmw-client FAILURE in 19m 50s

@@ -29,6 +29,7 @@
no_log: "{{ cifmw_nolog | default(true) | bool }}"
async: "{{ 7200 + cifmw_test_operator_timeout | default(3600) }}" # 2h should be enough to deploy EDPM and rest for tests.
poll: 20
when: cifmw_cleanup_architecture | default('true') | bool
Copy link
Contributor

Choose a reason for hiding this comment

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

why default('true') not default(true) ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants