Skip to content

Ignore earlier failed/completed deployments for deployment status#1607

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
rabi:OSPRH-15709
Sep 25, 2025
Merged

Ignore earlier failed/completed deployments for deployment status#1607
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
rabi:OSPRH-15709

Conversation

@rabi

@rabi rabi commented Sep 18, 2025

Copy link
Copy Markdown
Contributor

Only consider running deployments and the latest deployment either failed/completed containing the nodeset for overall deployment status.

Jira: https://issues.redhat.com/browse/OSPRH-15709

Signed-off-by: rabi ramishra@redhat.com
Assisted-By: Claude noreply@anthropic.com

@openshift-ci openshift-ci Bot requested review from rebtoor and stuggi September 18, 2025 05:56
@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/cc77e9dc05924a8387e6f6a0acc9d44d

✔️ openstack-k8s-operators-content-provider SUCCESS in 4h 37m 49s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 19m 58s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 32m 54s
adoption-standalone-to-crc-ceph-provider RETRY_LIMIT in 4h 08m 45s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 34m 30s

@rabi

rabi commented Sep 18, 2025

Copy link
Copy Markdown
Contributor Author

recheck RETRY_LIMIT

@openshift-ci

openshift-ci Bot commented Sep 23, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rabi, slagle

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

The pull request process is described 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

Only consideer running deployments and the last deployment
containing the nodeset either failed/completed.

Jira: https://issues.redhat.com/browse/OSPRH-15709
Signed-off-by: rabi <ramishra@redhat.com>
@openshift-ci

openshift-ci Bot commented Sep 23, 2025

Copy link
Copy Markdown
Contributor

New changes are detected. LGTM label has been removed.

@slagle

slagle commented Sep 23, 2025

Copy link
Copy Markdown
Contributor

I rebased this one since there was a conflict, hope you don't mind.

@rabi

rabi commented Sep 24, 2025

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-kuttl

I rebased this one since there was a conflict, hope you don't mind.

Thanks! It's the golang bump which forced all PRs to be rebased.

@rabi

rabi commented Sep 24, 2025

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-kuttl

2 similar comments
@rabi

rabi commented Sep 24, 2025

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-kuttl

@rabi

rabi commented Sep 24, 2025

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-kuttl

@slagle

slagle commented Sep 24, 2025

Copy link
Copy Markdown
Contributor

/retest

@rabi

rabi commented Sep 25, 2025

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-kuttl

@rabi

rabi commented Sep 25, 2025

Copy link
Copy Markdown
Contributor Author

Adding lgtm as it was only rebased after approval.

@rabi rabi added the lgtm label Sep 25, 2025
@openshift-merge-bot openshift-merge-bot Bot merged commit 8013333 into openstack-k8s-operators:main Sep 25, 2025
8 checks passed
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