WIP: Rename nova-operator to workloads-operator and add cleanup logic - #2044
Conversation
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: amartyasinha The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Important Review skippedIgnore keyword(s) in the title. ⛔ Ignored keywords (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
OpenStackControlPlane CRD Size Report
Threshold reference
|
1bee1e4 to
cba592e
Compare
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider FAILURE in 6m 47s |
|
recheck |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider FAILURE in 6m 55s |
4ef2d16 to
b5c1213
Compare
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider FAILURE in 7m 20s |
|
recheck |
1 similar comment
|
recheck |
|
This change depends on a change that failed to merge. Change openstack-k8s-operators/nova-operator#1192 is needed. |
b5c1213 to
32de3df
Compare
|
Build failed (check pipeline). Post Warning: |
|
recheck |
|
Build failed (check pipeline). Post Warning: |
|
recheck |
1 similar comment
|
recheck |
32de3df to
b06c2a2
Compare
|
recheck |
1 similar comment
|
recheck |
b06c2a2 to
5297a9c
Compare
Update openstack-operator to deploy and manage workloads-operator instead of nova-operator. The nova-operator has been renamed to workloads-operator to reflect its expanded scope managing multiple OpenStack workload services (Nova, Placement, and Cyborg). Changes include: - Added cleanupNovaOperator() function to remove old nova-operator resources during upgrade (deployment, service account, RBAC, certificates) - Renamed bindata/services from nova-operator-services.yaml to workloads-operator-services.yaml - Updated all K8s resource names from nova-operator to workloads-operator (deployments, services, roles, rolebindings, certificates, issuers) - Renamed RELATED_IMAGE_NOVA_OPERATOR_MANAGER_IMAGE_URL to RELATED_IMAGE_WORKLOADS_OPERATOR_MANAGER_IMAGE_URL - Updated CRD documentation comments to reference workloads-operator - Updated internal comments to reference workloads-operator The cleanup logic follows the same pattern used for placement-operator migration, removing only namespaced resources while leaving cluster-scoped RBAC intact to avoid breaking user-installed operators in other namespaces. Signed-off-by: Amartya Sinha <amsinha@redhat.com> Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
5297a9c to
e409b2e
Compare
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 54m 43s |
1 participant
Update openstack-operator to deploy and manage workloads-operator instead of nova-operator. The nova-operator has been renamed to workloads-operator to reflect its expanded scope managing multiple OpenStack workload services (Nova, Placement, and Cyborg).
Changes include:
Go module imports intentionally kept as nova-operator/api to maintain compatibility during the transition. Module paths will be updated in a follow-up commit after the nova-operator repository is renamed.
The cleanup logic follows the same pattern used for placement-operator migration, removing only namespaced resources while leaving cluster-scoped RBAC intact to avoid breaking user-installed operators in other namespaces.
Assisted-By: Claude Sonnet 4.5 noreply@anthropic.com
Depends-On: #1901
Depends-On: openstack-k8s-operators/nova-operator#1192