Skip to content

Reuse OCP cluster#2919

Merged
eshulman2 merged 1 commit intoopenstack-k8s-operators:mainfrom
eshulman2:deploy_reuse
May 13, 2025
Merged

Reuse OCP cluster#2919
eshulman2 merged 1 commit intoopenstack-k8s-operators:mainfrom
eshulman2:deploy_reuse

Conversation

@eshulman2
Copy link
Copy Markdown
Contributor

@eshulman2 eshulman2 commented Apr 23, 2025 •
edited
Loading

This patch is made to allow re-deploying openstack without having to re-deploy OCP every time. This patch will allow optimizing CI jobs run time for NFV jobs.

@eshulman2 eshulman2 requested a review from a team as a code owner April 23, 2025 10:50
@eshulman2 eshulman2 force-pushed the deploy_reuse branch 2 times, most recently from d11603a to fa655db Compare April 24, 2025 13:31
@eshulman2 eshulman2 force-pushed the deploy_reuse branch 4 times, most recently from 0cf7e96 to ca36950 Compare April 27, 2025 08:07
@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 27, 2025

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/3ba427c477894cc794487dd417e17d32

❌ openstack-k8s-operators-content-provider FAILURE in 13m 36s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 18s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 49s
✔️ build-push-container-cifmw-client SUCCESS in 17m 11s
✔️ cifmw-molecule-reproducer SUCCESS in 14m 35s

@eshulman2 eshulman2 force-pushed the deploy_reuse branch 4 times, most recently from f68c3d1 to d9aa0d8 Compare April 27, 2025 12:14
@eshulman2 eshulman2 force-pushed the deploy_reuse branch 3 times, most recently from d97869d to 2b1acee Compare May 7, 2025 11:27
@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented May 7, 2025

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/0fe3e4056e514618a780ed92dcb8cec6

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 57m 25s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 09m 47s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 43m 35s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 37s
❌ cifmw-pod-pre-commit FAILURE in 9m 01s
✔️ cifmw-architecture-validate-hci SUCCESS in 5m 32s
✔️ build-push-container-cifmw-client SUCCESS in 22m 14s
✔️ cifmw-molecule-kustomize_deploy SUCCESS in 4m 40s
✔️ cifmw-molecule-reproducer SUCCESS in 14m 49s

@dasm dasm requested review from a team May 8, 2025 21:29
dasm
dasm previously approved these changes May 8, 2025
View reviewed changes
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.

@eshulman2 I've added CI-FMW and CI-Ops teams to the review.
I like where you're going with this change, even though it's not fully in-sync with other initiatives happening at the time. However, we can reconvene and change it, if it's needed.

@eshulman2 eshulman2 requested a review from dasm May 12, 2025 09:17
@eshulman2
Copy link
Copy Markdown
Contributor Author

eshulman2 commented May 12, 2025

@dasm I changed something in the commit message mind re-adding your approval?

@eshulman2
Reuse OCP cluster
80577e3 
This patch creates playbooks and roles to allow re-using OCP cluster in
zuul CI jobs
@dasm dasm requested review from a team May 12, 2025 16:29
sdatko
sdatko approved these changes May 13, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@sdatko sdatko left a comment

Choose a reason for hiding this comment

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

I saw it in action on uni03gamma, it is fine for now, though some further work needs to be added for other uni jobs (as a separate changes preferably to not make this one even bigger)

@arxcruz
Copy link
Copy Markdown
Contributor

arxcruz commented May 13, 2025

/approve
/lgtm

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 13, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: arxcruz

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

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

Reviewers

@dasm dasm dasm approved these changes

@sdatko sdatko sdatko approved these changes

@arxcruz arxcruz Awaiting requested review from arxcruz

Assignees

@arxcruz arxcruz

Labels

lgtm Ready For Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@eshulman2 @arxcruz @dasm @sdatko