-
Notifications
You must be signed in to change notification settings - Fork 79
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
Add list of glanceAPI support #587
Conversation
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/5e6ff11ce6464b5fb403a07ce3d2acf0 ❌ openstack-k8s-operators-content-provider FAILURE in 7m 22s |
f90343e
to
90ed9d9
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/76c314a2714e4a919af4a8a52ee9b602 ❌ openstack-k8s-operators-content-provider FAILURE in 7m 33s |
90ed9d9
to
19d1321
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/9cee41d1dd7b4c41a878096b4ed97ef2 ❌ openstack-k8s-operators-content-provider FAILURE in 7m 54s |
19d1321
to
5ca75b5
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/110e2b22d3d444de9ec34eaae4079504 ❌ openstack-k8s-operators-content-provider FAILURE in 12m 50s |
5ca75b5
to
5eb5c99
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/d07b9f8a1e074941a3009804003615cf ❌ openstack-k8s-operators-content-provider FAILURE in 12m 36s |
5eb5c99
to
cf6fab8
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/856bd9e131c14f30832cdfe3c26e5c1a ❌ openstack-k8s-operators-content-provider FAILURE in 14m 37s |
cf6fab8
to
cd369dc
Compare
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. |
cd369dc
to
b7b3c45
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/34149648dec34e138839109316954455 ❌ openstack-k8s-operators-content-provider FAILURE in 12m 36s |
b7b3c45
to
a01e9f7
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/adb45c9302cb41e2a2d08e349883ad9b ❌ openstack-k8s-operators-content-provider FAILURE in 14m 59s |
/retest-required |
1 similar comment
/retest-required |
a01e9f7
to
e993bf6
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/0250fad4d13e4fe6be223a2d0d6d0139 ❌ openstack-k8s-operators-content-provider FAILURE in 13m 12s |
e993bf6
to
0a5b349
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/e9d7a9298a744a43b9c57b8da04845bf ❌ openstack-k8s-operators-content-provider FAILURE in 12m 09s |
recheck |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/ff5716edd4bb4258a91788330b27f16e ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 09m 34s |
e711412
to
cea42b3
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/87f81c936ea94371aa8446e1bb8e54e2 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 46m 04s |
@abays I think that once this patch is rebased on its dependency (openstack-k8s-operators/glance-operator#384), we can try to update the |
recheck |
1 similar comment
recheck |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/344a891610d4499486ab920ce41b8e7b ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 08m 43s |
config/samples/core_v1beta1_openstackcontrolplane_galera_network_isolation.yaml
Outdated
Show resolved
Hide resolved
config/samples/core_v1beta1_openstackcontrolplane_network_isolation.yaml
Outdated
Show resolved
Hide resolved
config/samples/core_v1beta1_openstackcontrolplane_network_isolation_ceph.yaml
Outdated
Show resolved
Hide resolved
cea42b3
to
1a38ed3
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/5971e57780624a58a677e7e3c1c15036 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 52m 56s |
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]>
recheck |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/ac5ef3afdd254c11875cb2ea327e1be6 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 59m 01s |
After rechecking zuul we can see that:
@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 |
recheck |
1a38ed3
to
5818366
Compare
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]>
5818366
to
f36c6c4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[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 |
aa63bf3
into
openstack-k8s-operators:main
This change aligns the
openstack-operator
with the work done in the glance-operator to support the deployment of a list ofglanceAPI
. The logic has been improved to make sure the newCRD
is supported.Demo: https://asciinema.org/a/626515
Depends-On: openstack-k8s-operators/data-plane-adoption#221