Skip to content

Fix Swift migration PV check#1330

Merged
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
cschwede:fix-ospcix-1305
Apr 9, 2026
Merged

Fix Swift migration PV check#1330
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
cschwede:fix-ospcix-1305

Conversation

@cschwede
Copy link
Copy Markdown
Contributor

Makes the check to ensure that the newly created PVs are showing up in the rings more stable. Timeouts are being kept as they are.

Related: https://issues.redhat.com/browse/OSPCIX-1305

@cschwede cschwede force-pushed the fix-ospcix-1305 branch 2 times, most recently from c3971cc to 4d44375 Compare March 27, 2026 14:22
Copy link
Copy Markdown
Contributor

@jistr jistr left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link
Copy Markdown
Contributor

@holser holser left a comment

Choose a reason for hiding this comment

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

Overall /LGTM though I have 2 small nitpicks

@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/fb252ae8d3064e22ba129a731aab2321

✔️ noop SUCCESS in 0s
adoption-standalone-to-crc-ceph RETRY_LIMIT Host unreachable in 2h 38m 47s
adoption-standalone-to-crc-no-ceph RETRY_LIMIT Host unreachable in 2h 44m 24s

@openshift-ci openshift-ci bot removed the lgtm label Apr 1, 2026
Makes the check to ensure that the newly created PVs are showing up in
the rings more stable. Timeouts are being kept as they are.

Related: https://issues.redhat.com/browse/OSPCIX-1305

Signed-off-by: Christian Schwede <cschwede@redhat.com>
@cschwede cschwede closed this Apr 7, 2026
@cschwede cschwede deleted the fix-ospcix-1305 branch April 7, 2026 15:47
@cschwede cschwede restored the fix-ospcix-1305 branch April 8, 2026 10:08
@cschwede cschwede reopened this Apr 8, 2026
Copy link
Copy Markdown
Contributor

@jistr jistr left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Apr 8, 2026
@ciecierski ciecierski requested a review from karelyatin April 8, 2026 14:08
@ciecierski
Copy link
Copy Markdown
Contributor

/approve

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Apr 8, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ciecierski

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

@openshift-ci openshift-ci bot added the approved label Apr 8, 2026
Copy link
Copy Markdown
Contributor

@karelyatin karelyatin left a comment

Choose a reason for hiding this comment

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

works now

@openshift-merge-bot openshift-merge-bot bot merged commit 624a414 into openstack-k8s-operators:main Apr 9, 2026
6 checks passed
@cschwede cschwede deleted the fix-ospcix-1305 branch April 9, 2026 14:35
@cschwede
Copy link
Copy Markdown
Contributor Author

cschwede commented Apr 9, 2026

/cherry-pick 18.0-fr5

@openshift-cherrypick-robot
Copy link
Copy Markdown

@cschwede: new pull request created: #1359

Details

In response to this:

/cherry-pick 18.0-fr5

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

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.

6 participants