Skip to content
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

Add list of glanceAPI support #587

Merged

Conversation

fmount
Copy link
Contributor

@fmount fmount commented Dec 5, 2023

This change aligns the openstack-operator with the work done in the glance-operator to support the deployment of a list of glanceAPI. The logic has been improved to make sure the new CRD is supported.

Demo: https://asciinema.org/a/626515

Depends-On: openstack-k8s-operators/data-plane-adoption#221

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://review.rdoproject.org/zuul/buildset/5e6ff11ce6464b5fb403a07ce3d2acf0

openstack-k8s-operators-content-provider FAILURE in 7m 22s
⚠️ 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

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://review.rdoproject.org/zuul/buildset/76c314a2714e4a919af4a8a52ee9b602

openstack-k8s-operators-content-provider FAILURE in 7m 33s
⚠️ 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@fmount fmount force-pushed the list_of_glanceapi branch from 90ed9d9 to 19d1321 Compare December 5, 2023 16:18
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://review.rdoproject.org/zuul/buildset/9cee41d1dd7b4c41a878096b4ed97ef2

openstack-k8s-operators-content-provider FAILURE in 7m 54s
⚠️ 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@fmount fmount force-pushed the list_of_glanceapi branch from 19d1321 to 5ca75b5 Compare December 5, 2023 21:50
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://review.rdoproject.org/zuul/buildset/110e2b22d3d444de9ec34eaae4079504

openstack-k8s-operators-content-provider FAILURE in 12m 50s
⚠️ 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@fmount fmount force-pushed the list_of_glanceapi branch from 5ca75b5 to 5eb5c99 Compare December 6, 2023 22:51
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://review.rdoproject.org/zuul/buildset/d07b9f8a1e074941a3009804003615cf

openstack-k8s-operators-content-provider FAILURE in 12m 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@fmount fmount force-pushed the list_of_glanceapi branch from 5eb5c99 to cf6fab8 Compare December 7, 2023 21:11
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://review.rdoproject.org/zuul/buildset/856bd9e131c14f30832cdfe3c26e5c1a

openstack-k8s-operators-content-provider FAILURE in 14m 37s
⚠️ 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@fmount fmount force-pushed the list_of_glanceapi branch from cf6fab8 to cd369dc Compare December 8, 2023 21:14
Copy link

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/openstack-operator for 587,cd369dcb44ccefa891f008503955c7913774ac1a

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://review.rdoproject.org/zuul/buildset/34149648dec34e138839109316954455

openstack-k8s-operators-content-provider FAILURE in 12m 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@fmount fmount force-pushed the list_of_glanceapi branch from b7b3c45 to a01e9f7 Compare December 8, 2023 22:19
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://review.rdoproject.org/zuul/buildset/adb45c9302cb41e2a2d08e349883ad9b

openstack-k8s-operators-content-provider FAILURE in 14m 59s
⚠️ 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@fmount
Copy link
Contributor Author

fmount commented Dec 8, 2023

/retest-required

1 similar comment
@fmount
Copy link
Contributor Author

fmount commented Dec 9, 2023

/retest-required

@fmount fmount force-pushed the list_of_glanceapi branch from a01e9f7 to e993bf6 Compare December 9, 2023 09:25
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://review.rdoproject.org/zuul/buildset/0250fad4d13e4fe6be223a2d0d6d0139

openstack-k8s-operators-content-provider FAILURE in 13m 12s
⚠️ 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@fmount fmount force-pushed the list_of_glanceapi branch from e993bf6 to 0a5b349 Compare December 9, 2023 21:51
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://review.rdoproject.org/zuul/buildset/e9d7a9298a744a43b9c57b8da04845bf

openstack-k8s-operators-content-provider FAILURE in 12m 09s
⚠️ 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
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@fmount
Copy link
Contributor Author

fmount commented Dec 13, 2023

recheck

@fmount fmount changed the title WIP - Add list of glanceAPI support Add list of glanceAPI support Dec 13, 2023
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://review.rdoproject.org/zuul/buildset/ff5716edd4bb4258a91788330b27f16e

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 09m 34s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 22m 55s
cifmw-crc-podified-edpm-baremetal RETRY_LIMIT in 4m 05s
cifmw-data-plane-adoption-OSP-17-to-extracted-crc FAILURE in 1h 49m 31s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 27m 14s

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://review.rdoproject.org/zuul/buildset/87f81c936ea94371aa8446e1bb8e54e2

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 46m 04s
podified-multinode-edpm-deployment-crc FAILURE in 1h 24m 51s
cifmw-crc-podified-edpm-baremetal FAILURE in 48m 11s
cifmw-data-plane-adoption-OSP-17-to-extracted-crc NODE_FAILURE Node request 099-0006745051 failed in 0s
openstack-operator-tempest-multinode NODE_FAILURE Node request 099-0006745052 failed in 0s

@fmount
Copy link
Contributor Author

fmount commented Dec 14, 2023

@abays I think that once this patch is rebased on its dependency (openstack-k8s-operators/glance-operator#384), we can try to update the Depends-on for zuul to openstack-k8s-operators/data-plane-adoption#221 and see if the adoption job works as expected. Even though the other jobs are failing, the Glance deployment looks good, so I'll recheck this patch once the Glance one merges.

@fmount
Copy link
Contributor Author

fmount commented Dec 14, 2023

recheck

1 similar comment
@danpawlik
Copy link

recheck

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://review.rdoproject.org/zuul/buildset/344a891610d4499486ab920ce41b8e7b

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 08m 43s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 13m 42s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 17m 14s
cifmw-data-plane-adoption-OSP-17-to-extracted-crc FAILURE in 1h 42m 30s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 26m 42s

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://review.rdoproject.org/zuul/buildset/5971e57780624a58a677e7e3c1c15036

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 52m 56s
podified-multinode-edpm-deployment-crc RETRY_LIMIT in 4s
cifmw-crc-podified-edpm-baremetal RETRY_LIMIT in 4s
cifmw-data-plane-adoption-OSP-17-to-extracted-crc RETRY_LIMIT in 4s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 32m 55s

fmount added a commit to fmount/data-plane-adoption that referenced this pull request Dec 14, 2023
We recently introduced the ability to deploy a list of glanceAPI
in the OpenStackControlPlane. This feature is not required in the
adoption context, but it's critical to adapt the Glance deployment
to the new api.
It might be relevant in the future as long as DCN environments will
be adopted.

Depends-On: openstack-k8s-operators/glance-operator#384
Depends-On: openstack-k8s-operators/openstack-operator#587

Signed-off-by: Francesco Pantano <[email protected]>
@fmount
Copy link
Contributor Author

fmount commented Dec 14, 2023

recheck

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://review.rdoproject.org/zuul/buildset/ac5ef3afdd254c11875cb2ea327e1be6

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 59m 01s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 07m 18s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 16m 39s
cifmw-data-plane-adoption-OSP-17-to-extracted-crc FAILURE in 1h 37m 26s
openstack-operator-tempest-multinode FAILURE in 41m 12s

@fmount
Copy link
Contributor Author

fmount commented Dec 15, 2023

After rechecking zuul we can see that:

  1. Data Plane Adoption failure is expected, because it requires [1];
  2. Tempest multinode fails, but looks like we've lost OCP in that job, even the must-gather is not able to retrieve any log
  3. Both baremetal and crc-net-isolation jobs work as expected, and for example we can see the right CR being created [2]

@abays not sure you have extra comments on this one, but once I have the green light I'll start merging the glance dependency so we can focus to get this one in.

[1] openstack-k8s-operators/data-plane-adoption#221
[2] https://logserver.rdoproject.org/87/587/1a38ed3932845b92e816339a56bfe15a4cba856d/github-check/cifmw-crc-podified-edpm-baremetal/22e8e91/ci-framework-data/logs/quay-io-openstack-k8s-operators-openstack-must-gather-sha256-c5501ac502209eda400116b0511a5cf07f0537f650b706f63006628472a846af/namespaces/openstack/crs/openstackcontrolplanes.core.openstack.org/openstack-galera-network-isolation.yaml

@fmount
Copy link
Contributor Author

fmount commented Dec 15, 2023

recheck

This change aligns the openstack-operator with the work done in the
glance-operator to support the deployment of an arbitrary number of
glanceAPI(s).
The logic has been improved to support the new CRD and the Glance
reconcile logic has been revisited as well.

Signed-off-by: Francesco Pantano <[email protected]>
Copy link
Contributor

@abays abays 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
Contributor

openshift-ci bot commented Dec 15, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, fmount

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

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

@openshift-merge-bot openshift-merge-bot bot merged commit aa63bf3 into openstack-k8s-operators:main Dec 15, 2023
7 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.

4 participants