Skip to content

Comments

Add new /apis multi-group structure and hide old api/ dir#1073

Draft
auniyal61 wants to merge 4 commits intoopenstack-k8s-operators:mainfrom
auniyal61:merge-placement-operator
Draft

Add new /apis multi-group structure and hide old api/ dir#1073
auniyal61 wants to merge 4 commits intoopenstack-k8s-operators:mainfrom
auniyal61:merge-placement-operator

Conversation

@auniyal61
Copy link
Contributor

  • Create apis/nova/v1beta1/ for Nova API types
  • Create apis/placement/v1beta1/ for Placement API types
  • Copy all Nova API files from api/v1beta1/ to apis/nova/v1beta1/
  • Copy Placement API files from placement-operator to apis/placement/v1beta1/
  • Copy go.mod and go.sum to apis/, and fix the module path
  • Rename api to .api for now, as backup, will be removed later

@openshift-ci openshift-ci bot requested review from dprince and kk7ds February 17, 2026 07:02
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 17, 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

@softwarefactory-project-zuul
Copy link

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/3e7f44c8d9cf4e3ba35a0136756cfffc

openstack-meta-content-provider FAILURE in 6m 34s
⚠️ nova-operator-kuttl SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode-ceph SKIPPED Skipped due to failed job openstack-meta-content-provider

@auniyal61 auniyal61 force-pushed the merge-placement-operator branch from af02279 to f336f75 Compare February 17, 2026 09:41
@softwarefactory-project-zuul
Copy link

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

openstack-meta-content-provider FAILURE in 6m 56s
⚠️ nova-operator-kuttl SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode-ceph SKIPPED Skipped due to failed job openstack-meta-content-provider

- Create apis/nova/v1beta1/ for Nova API types
- Create apis/placement/v1beta1/ for Placement API types
- Copy all Nova API files from api/v1beta1/ to apis/nova/v1beta1/
- Copy Placement API files from placement-operator to apis/placement/v1beta1/
- Copy go.mod and go.sum to apis/, and fix the module path
- Rename api to .api for now, as backup, will be removed later

- Update import paths from api/v1beta1 to apis/nova/v1beta1
	- internal/*
	- test/*
	- cmd/main.go
@auniyal61 auniyal61 force-pushed the merge-placement-operator branch from f336f75 to 7ded143 Compare February 17, 2026 13:17
@softwarefactory-project-zuul
Copy link

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/1584157f66734962b56ebc2e82121698

openstack-meta-content-provider FAILURE in 7m 29s
⚠️ nova-operator-kuttl SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode-ceph SKIPPED Skipped due to failed job openstack-meta-content-provider

@auniyal61 auniyal61 force-pushed the merge-placement-operator branch from 7ded143 to d5619a4 Compare February 18, 2026 10:54
@softwarefactory-project-zuul
Copy link

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

openstack-meta-content-provider FAILURE in 7m 32s
⚠️ nova-operator-kuttl SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode-ceph SKIPPED Skipped due to failed job openstack-meta-content-provider

- go.mod: Replace module reference from /api to /apis add allow-merging comment
- Makefile: Replaced manifests, gowork, operator-lint targets to /apis
- PROJECT: Updated Nova API paths to apis/nova/v1beta1, added PlacementAPI resource
- Generated placement.openstack.org_placementapis.yaml
- Copied all CRD manifests to apis/bases/ Nova (8) + Placement (1)
@auniyal61 auniyal61 force-pushed the merge-placement-operator branch from d5619a4 to ec20cd5 Compare February 18, 2026 11:50
@softwarefactory-project-zuul
Copy link

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/5129cb88fc1545f087a3f6b17434e877

openstack-meta-content-provider FAILURE in 10m 03s
⚠️ nova-operator-kuttl SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode-ceph SKIPPED Skipped due to failed job openstack-meta-content-provider

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant