Skip to content

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

Open
auniyal61 wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
auniyal61:nova-placement-integration
Open

Update openstack-operator to use placement API from nova-operator#1893
auniyal61 wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
auniyal61:nova-placement-integration

Conversation

@auniyal61
Copy link
Copy Markdown
Contributor

@auniyal61 auniyal61 commented Apr 15, 2026

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 auniyal61/nova-operator fork (temporary)
  • Clean up go.sum files to remove stale placement-operator entries

@openshift-ci openshift-ci bot requested review from abays and stuggi April 15, 2026 15:42
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Apr 15, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: auniyal61
Once this PR has been reviewed and has the lgtm label, please assign rebtoor for approval. For more information see the Code Review Process.

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

@softwarefactory-project-zuul
Copy link
Copy Markdown

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/nova-operator for 1101,b54a701efcaf51fb43e59bca78c890dbf09149aa

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 15, 2026

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 322326 bytes (315KB)
Base branch size 322326 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)

@auniyal61 auniyal61 force-pushed the nova-placement-integration branch from b14a2c0 to c960af3 Compare April 15, 2026 16:12
@softwarefactory-project-zuul
Copy link
Copy Markdown

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/nova-operator for 1101,b54a701efcaf51fb43e59bca78c890dbf09149aa

@auniyal61 auniyal61 force-pushed the nova-placement-integration branch from c960af3 to 3f8b2c4 Compare April 16, 2026 05:18
@softwarefactory-project-zuul
Copy link
Copy Markdown

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

openstack-k8s-operators-content-provider FAILURE in 7m 46s
⚠️ 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

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 auniyal61/nova-operator fork (temporary)
- Clean up go.sum files to remove stale placement-operator entries
@auniyal61 auniyal61 force-pushed the nova-placement-integration branch from 3f8b2c4 to 5ff81e0 Compare April 16, 2026 05:46
@softwarefactory-project-zuul
Copy link
Copy Markdown

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/5349812a78e5482bb23165ea6bae93db

openstack-k8s-operators-content-provider FAILURE in 7m 47s
⚠️ 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

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Apr 16, 2026

@auniyal61: The following tests 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 5ff81e0 link true /test precommit-check
ci/prow/openstack-operator-build-deploy-kuttl-4-18 5ff81e0 link true /test openstack-operator-build-deploy-kuttl-4-18

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.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant