Skip to content

Placement integration#1092

Open
auniyal61 wants to merge 16 commits into
openstack-k8s-operators:mainfrom
auniyal61:placement-integration
Open

Placement integration#1092
auniyal61 wants to merge 16 commits into
openstack-k8s-operators:mainfrom
auniyal61:placement-integration

Conversation

@auniyal61
Copy link
Copy Markdown
Contributor

@auniyal61 auniyal61 commented Apr 6, 2026

Closes: OSPRH-26916

Import complete placement-operator codebase into placement-import
directly using git subtree merge to preserve  full commit history

this merge connects plaement git history to nova-operator, enabling git
blame and git log to show original authors and commit history of all
placement code
Move types from placement-import/api/v1beta1/ to api/placement/v1beta1
following nova-operator multigroup structure
Move controller from placement-import/internal/controller/ to
internal/controller/placement
Move placement business logic from placement-import/internal/placement
to internal/placement/ to keep alongside nova business logic
Move webhook from placement-import/internal/webhook/v1beta1/ to
internal/webhook/placement/v1beta1/
Move placement templates from placement-import/templates/placementapi/
to templates/placement/api/
Move placement tests from placement-import/test/ to
test/functional/placement/
Move placement CRD to config/crd/bases/ and remove all remaining
placement-import/ files
Update all import paths in placement code from
github.com/openstack-k8s-operators/placement-operator/api/v1beta1 to
github.com/openstack-k8s-operators/nova-operator/api/placement/v1beta1
and
github.com/openstack-k8s-operators/placement-operator/internal/placement
github.com/openstack-k8s-operators/nova-operator/internal/placement

package name update functional_test to placement_test
@openshift-ci openshift-ci Bot requested review from bogdando and dprince April 6, 2026 08:35
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 6, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: auniyal61

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

@openshift-ci openshift-ci Bot added the approved label Apr 6, 2026
  - Register PlacementAPI controller in cmd/main.go
  - Register PlacementAPI webhook in cmd/main.go
  - Add placement scheme and defaults initialization
  - Fix placement test paths after moving to test/functional/placement/
  - Add placement API to test coverage in Makefile
@auniyal61 auniyal61 force-pushed the placement-integration branch from 370c3b8 to d0cc289 Compare April 6, 2026 09:54
@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/420d0163be234e7db60b815d59719b07

✔️ openstack-meta-content-provider SUCCESS in 3h 01m 45s
nova-operator-kuttl FAILURE in 1h 04m 05s
✔️ nova-operator-tempest-multinode SUCCESS in 2h 14m 41s
✔️ nova-operator-tempest-multinode-ceph SUCCESS in 2h 45m 41s

Updated Makefile to include placement webhook paths in controller-gen
Adds getTemplateInstanceType in placement/common.go
@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/8aadee82920a42098d1f007ecfb8aa45

✔️ openstack-meta-content-provider SUCCESS in 3h 06m 50s
nova-operator-kuttl FAILURE in 57m 21s
✔️ nova-operator-tempest-multinode SUCCESS in 2h 29m 17s
✔️ nova-operator-tempest-multinode-ceph SUCCESS in 2h 48m 27s

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant