Skip to content

Update openstack-operator to use placement API from nova-operator#1900

Open
amartyasinha wants to merge 1 commit into
openstack-k8s-operators:mainfrom
amartyasinha:nova-placement-integration
Open

Update openstack-operator to use placement API from nova-operator#1900
amartyasinha wants to merge 1 commit into
openstack-k8s-operators:mainfrom
amartyasinha:nova-placement-integration

Conversation

@amartyasinha
Copy link
Copy Markdown
Contributor

@amartyasinha amartyasinha commented Apr 24, 2026 •
edited
Loading

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 cleanup logic to remove orphaned placement-operator resources
- Clean up go.sum files to remove stale placement-operator entries
- Add cleanup method to remove obsolete placement resources
- Remove PlaementOperator case

@openshift-ci openshift-ci Bot requested review from abays and dprince April 24, 2026 10:49
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 24, 2026 •
edited
Loading

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 322464 bytes (315KB)
Base branch size 322464 bytes
Change +0.00%
Status yellow — growing
Threshold reference
Color Range Meaning
🟢 green < 300KB Comfortable
🟡 yellow 300–400KB Growing
🟠 orange 400–750KB Concerning
🔴 red > 750KB Approaching 1.5MB etcd limit (cut in half to allow space for update)

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 24, 2026

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/9cdbd134263541e895e2aa3b49940a02

❌ openstack-k8s-operators-content-provider FAILURE in 9m 07s
⚠️ 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
⚠️ adoption-standalone-to-crc-ceph-provider 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
⚠️ openstack-operator-edpm-baremetal-minor-update SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Apr 24, 2026

recheck

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 24, 2026

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/64f6cae427964ff28981e96f4afd8403

❌ openstack-k8s-operators-content-provider FAILURE in 7m 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
⚠️ adoption-standalone-to-crc-ceph-provider 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
⚠️ openstack-operator-edpm-baremetal-minor-update SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Apr 24, 2026

recheck

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 24, 2026

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

❌ 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
⚠️ adoption-standalone-to-crc-ceph-provider 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
⚠️ openstack-operator-edpm-baremetal-minor-update SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@amartyasinha amartyasinha mentioned this pull request Apr 24, 2026
Draft
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Apr 24, 2026

recheck

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 24, 2026

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/4268373fed2342bf987853f8c9d51dd8

❌ openstack-k8s-operators-content-provider FAILURE in 8m 04s
⚠️ 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
⚠️ adoption-standalone-to-crc-ceph-provider 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
⚠️ openstack-operator-edpm-baremetal-minor-update SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Apr 27, 2026

recheck

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 27, 2026

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

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 20m 46s
❌ podified-multinode-edpm-deployment-crc FAILURE in 37m 58s
❌ cifmw-crc-podified-edpm-baremetal FAILURE in 47m 41s
❌ adoption-standalone-to-crc-ceph-provider RETRY_LIMIT in 43m 01s
❌ openstack-operator-tempest-multinode FAILURE in 42m 05s
✔️ openstack-operator-edpm-baremetal-minor-update SUCCESS in 2h 08m 51s

@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Apr 27, 2026

recheck

@amartyasinha amartyasinha reopened this Apr 27, 2026
@amartyasinha amartyasinha force-pushed the nova-placement-integration branch from dc1d814 to f8994e4 Compare April 29, 2026 07:54
@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 29, 2026

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 1900,f8994e4c4b4de70cf3bbf9f2712314586536c92c

@amartyasinha amartyasinha force-pushed the nova-placement-integration branch from f8994e4 to 4e2b552 Compare April 29, 2026 09:58
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented Apr 29, 2026

resolve merge conflict

@amartyasinha amartyasinha force-pushed the nova-placement-integration branch from 4e2b552 to eb476e6 Compare May 8, 2026 08:27
@centosinfra-prod-github-app
Copy link
Copy Markdown

centosinfra-prod-github-app Bot commented May 8, 2026

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 1900,eb476e6508d904cde71d68a1159dcd3db9e7a6d9

@amartyasinha amartyasinha force-pushed the nova-placement-integration branch from eb476e6 to d2613b7 Compare May 8, 2026 08:33
@amartyasinha amartyasinha force-pushed the nova-placement-integration branch 2 times, most recently from 45240e5 to 29cae4c Compare May 8, 2026 10:46
amartyasinha
amartyasinha commented May 8, 2026
View reviewed changes
Comment thread internal/controller/operator/openstack_controller.go
@centosinfra-prod-github-app
Copy link
Copy Markdown

centosinfra-prod-github-app Bot commented May 8, 2026

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://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/cc714569dc6a4e31abd319f5aeb0298c

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 41m 55s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 33m 33s
❌ cifmw-crc-podified-edpm-baremetal FAILURE in 32m 54s
❌ adoption-standalone-to-crc-ceph-provider POST_FAILURE in 3h 13m 05s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 47m 13s
✔️ openstack-operator-edpm-baremetal-minor-update SUCCESS in 2h 17m 28s

@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented May 11, 2026

recheck

@centosinfra-prod-github-app
Copy link
Copy Markdown

centosinfra-prod-github-app Bot commented May 11, 2026

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://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/b052ddcbc63b4e5881c138f379ad5b8a

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 28m 50s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 25m 06s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 31m 08s
❌ adoption-standalone-to-crc-ceph-provider POST_FAILURE in 3h 14m 14s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 36m 07s
✔️ openstack-operator-edpm-baremetal-minor-update SUCCESS in 2h 03m 11s

@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented May 12, 2026

recheck

@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented May 12, 2026

/retest

@centosinfra-prod-github-app
Copy link
Copy Markdown

centosinfra-prod-github-app Bot commented May 12, 2026

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://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/90e42bbe497846398cdab596f74348c9

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 17m 45s
❌ podified-multinode-edpm-deployment-crc POST_FAILURE in 23m 25s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 30m 12s
❌ adoption-standalone-to-crc-ceph-provider RETRY_LIMIT in 25m 18s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 40m 30s
✔️ openstack-operator-edpm-baremetal-minor-update SUCCESS in 2h 04m 55s

@fserucas
Copy link
Copy Markdown

fserucas commented May 14, 2026

recheck

@fserucas
Copy link
Copy Markdown

fserucas commented May 14, 2026

/retest

abays
abays reviewed May 14, 2026
View reviewed changes
Comment thread internal/controller/operator/openstack_controller.go
@amartyasinha amartyasinha force-pushed the nova-placement-integration branch from 29cae4c to 19ea343 Compare May 14, 2026 11:31
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented May 14, 2026

/retest

1 similar comment
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented May 15, 2026

/retest

gibizer
gibizer reviewed May 15, 2026
View reviewed changes
Comment thread api/go.mod Outdated
Comment thread go.mod Outdated
@amartyasinha amartyasinha force-pushed the nova-placement-integration branch from 19ea343 to 1f14e1f Compare May 15, 2026 09:08
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented May 15, 2026

recheck

@amartyasinha amartyasinha changed the title [DNM] Update openstack-operator to use placement API from nova-operator Update openstack-operator to use placement API from nova-operator May 15, 2026
abays
abays reviewed May 15, 2026
View reviewed changes
Copy link
Copy Markdown
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

@openshift-ci openshift-ci Bot added the lgtm label May 15, 2026
@amartyasinha
Update openstack-operator to use placement API from nova-operator
0046e2c 
    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 cleanup logic to remove orphaned placement-operator resources
    - 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 amartyasinha force-pushed the nova-placement-integration branch from 1f14e1f to 0046e2c Compare May 15, 2026 10:59
@openshift-ci openshift-ci Bot removed the lgtm label May 15, 2026
@amartyasinha
Copy link
Copy Markdown
Contributor Author

amartyasinha commented May 15, 2026

Pointed to the latest RELATED_IMAGE_NOVA_OPERATOR_MANAGER_IMAGE_URL in this force-push.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 15, 2026 •
edited
Loading

@amartyasinha: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/precommit-check 0046e2c link true /test precommit-check

Full PR test history. Your PR dashboard.

Details

Instructions 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.

stuggi
stuggi approved these changes May 15, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@stuggi stuggi left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label May 15, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 15, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: amartyasinha, stuggi

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

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

@gibizer gibizer gibizer left review comments

@abays abays abays left review comments

@stuggi stuggi stuggi approved these changes

@dprince dprince Awaiting requested review from dprince

Assignees

@abays abays

@stuggi stuggi

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@amartyasinha @fserucas @gibizer @abays @stuggi