[DNM] Update openstack-operator to use placement API from nova-operator#1900
[DNM] Update openstack-operator to use placement API from nova-operator#1900amartyasinha wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
Conversation
OpenStackControlPlane CRD Size Report
Threshold reference
|
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: amartyasinha The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/9cdbd134263541e895e2aa3b49940a02 ❌ openstack-k8s-operators-content-provider FAILURE in 9m 07s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/64f6cae427964ff28981e96f4afd8403 ❌ openstack-k8s-operators-content-provider FAILURE in 7m 59s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ca8ee6cf284c46f7b4009a213ffe9a29 ❌ openstack-k8s-operators-content-provider FAILURE in 7m 22s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/4268373fed2342bf987853f8c9d51dd8 ❌ openstack-k8s-operators-content-provider FAILURE in 8m 04s |
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/cf648887b27c47748448a240a767bce4 ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 20m 46s |
|
recheck |
amartyasinha
force-pushed
the
nova-placement-integration
branch
from
dc1d814 to
f8994e4
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. |
amartyasinha
force-pushed
the
nova-placement-integration
branch
from
f8994e4 to
4e2b552
Compare
|
resolve merge conflict |
amartyasinha
force-pushed
the
nova-placement-integration
branch
from
4e2b552 to
eb476e6
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. |
amartyasinha
force-pushed
the
nova-placement-integration
branch
from
eb476e6 to
d2613b7
Compare
amartyasinha
force-pushed
the
nova-placement-integration
branch
from
d2613b7 to
45240e5
Compare
Replace placement-operator dependency with nova-operator's placement API
now that placement code has been integrated into nova-operator
- Update all imports from placement-operator/api to nova-operator/api/placement
- Remove placement-operator from go.mod and api/go.mod dependencies
- Update nova-operator dependency to placement-integration-v5
- Add replace directives pointing to amartyasinha/nova-operator fork (temporary)
- Clean up go.sum files to remove stale placement-operator entries
- Add cleanup method to remove obsolete placement resources
- Remove PlaementOperator case
Signed-off-by: Amartya Sinha <amsinha@redhat.com>
amartyasinha
force-pushed
the
nova-placement-integration
branch
from
45240e5 to
29cae4c
Compare
|
|
||
| return nil | ||
| } | ||
|
|
There was a problem hiding this comment.
@lmiccini Hey! may I ask for your initial review for this cleanup method? Also, can you recommend how to test this upgrade?
|
@amartyasinha: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. I understand the commands that are listed here. |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 41m 55s |
1 participant
Replace placement-operator dependency with nova-operator's placement API now that placement code has been integrated into nova-operator
Creating this PR to test changes by creating nova-operator bundle image in my own quay.io repo and use that in sync-bindata
Depends-On: #1901