Skip to content

Conversation

@jamepark4
Copy link
Contributor

@jamepark4 jamepark4 commented Feb 18, 2025

@openshift-ci
Copy link

openshift-ci bot commented Feb 18, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci
Copy link

openshift-ci bot commented Feb 18, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jamepark4
Once this PR has been reviewed and has the lgtm label, please assign cjeanner for approval. For more information see the Code Review Process.

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
Contributor

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/5d3ae482a8fb47e5b3b4fa681080a6d3

✔️ noop SUCCESS in 0s
rhoso-architecture-validate-uni05epsilon FAILURE in 5m 48s

@jamepark4 jamepark4 force-pushed the add_3cells_uni-epsilon-fr1 branch from 714b1ac to 787fe94 Compare February 19, 2025 15:21
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ noop SUCCESS in 0s
rhoso-architecture-validate-uni05epsilon FAILURE in 5m 57s

@jamepark4
Copy link
Contributor Author

recheck

@jamepark4 jamepark4 force-pushed the add_3cells_uni-epsilon-fr1 branch from 787fe94 to f6f495d Compare February 26, 2025 16:12
@softwarefactory-project-zuul
Copy link
Contributor

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#2742 is needed.

@jamepark4 jamepark4 marked this pull request as ready for review February 26, 2025 16:14
@jamepark4
Copy link
Contributor Author

recheck

@abays
Copy link
Contributor

abays commented Mar 3, 2025

@jamepark4 Has this been tested?

@abays abays added the needs-testing Testing proof is required prior to merge label Mar 3, 2025
@abays
Copy link
Contributor

abays commented Mar 17, 2025

@jamepark4 Has this been tested?

@jamepark4 Any updates on this one?

@jamepark4
Copy link
Contributor Author

Hi @abays, sorry for the delay. Yes it was tested but talking about it downstream with FR2 content landing we decided this backport was not necessary for FR1.

@jamepark4 jamepark4 closed this Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-testing Testing proof is required prior to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants