Migrate nova-operator from single to multi-group #990
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ratailor The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
ratailor
requested review from
gibizer
and removed request for
bogdando and
lewisdenny
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/5925f9411a0b46c98ccba35ad9c94b59 ❌ openstack-meta-content-provider FAILURE in 55m 08s |
|
lgtm as this transition looks technically correct, albeit the upgrade path should be covered as well:
|
ratailor
force-pushed
the
migrate-nova-to-multigroup
branch
from
189c2dc to
563e49d
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a31134e8933347c9bc95f9e72170c2f1 ❌ openstack-meta-content-provider FAILURE in 45m 38s |
ratailor
force-pushed
the
migrate-nova-to-multigroup
branch
from
563e49d to
4ad26aa
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/4f082ed24864447680a12accb9919212 ❌ openstack-meta-content-provider FAILURE in 50m 46s |
ratailor
force-pushed
the
migrate-nova-to-multigroup
branch
from
4ad26aa to
d80fef9
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/8182f70bb2bb4c3ab6fbb4a2bbe300eb ❌ openstack-meta-content-provider FAILURE in 15m 31s |
ratailor
force-pushed
the
migrate-nova-to-multigroup
branch
from
d80fef9 to
9957268
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. |
ratailor
force-pushed
the
migrate-nova-to-multigroup
branch
from
9957268 to
a81e01b
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/0d6483d1aa7140ce80b7ab0d9faef213 ❌ openstack-meta-content-provider FAILURE in 10m 35s |
ratailor
force-pushed
the
migrate-nova-to-multigroup
branch
from
a81e01b to
532331a
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/21ea5c6188264dcd8ea5687dfefce601 ❌ openstack-meta-content-provider FAILURE in 9m 24s |
This commit restructures the nova-operator codebase to support multi-group operators by: - Moving API definitions from api/ to apis/nova/v1beta1/ - Reorganizing controllers into controllers/nova/ directory - Relocating test files to test/functional/nova/ and test/kuttl/nova/ - Updating all import paths and references throughout the codebase - Modifying build and configuration files to work with new structure This restructuring prepares the operator to support multiple API groups, which is required to merge placement-operator codebase into nova-operator Commit message assisted by: claude-4-sonnet Related: OSPRH-14957
ratailor
force-pushed
the
migrate-nova-to-multigroup
branch
from
532331a to
b00359f
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/7b8fbe91fbc2411595bb3d6ed437c144 ❌ openstack-meta-content-provider FAILURE in 9m 46s |
paramite
left a comment
paramite
left a comment
There was a problem hiding this comment.
Just out of curiosity, do we plan to have more that one KUTTL test suite for a service? I wonder why the kuttl tests moved to service subfolder while there could be default suite which deploys and verifies both nova and placement and then suite just for nova resources tests and suite just for placement resources tests?
4 participants
This change migrate nova-operator from single group operator to support multi-group operators, which is required to merge placement-operator code base from placement-operator into nova-operator.
Related: OSPRH-14957