WIP: Rename nova-operator to workloads-operator - #1192
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: amartyasinha The full list of commands accepted by this bot can be found here. The pull request process is described 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 |
|
/retest |
|
Build failed (check pipeline). Post ❌ openstack-meta-content-provider FAILURE in 12m 10s |
|
recheck |
c44520f to
4943eba
Compare
|
recheck |
Rename the operator from nova-operator to workloads-operator to reflect its expanded scope managing multiple OpenStack workload services (Nova, Placement, and Cyborg). Changes include: - Updated projectName to workloads-operator in PROJECT file - Renamed CSV file and updated displayName to "Workloads Operator" - Updated all Kubernetes resources (deployments, services, RBAC, webhooks) - Added description noting "Formerly known as Nova Operator" - Added keywords: Placement, Cyborg, Compute, Workloads Container image URLs in Makefile, Go module paths, and GitHub repository URLs intentionally kept as nova-operator for CI compatibility. These will be updated in follow-up commits after the repository is renamed. Signed-off-by: Amartya Sinha <amsinha@redhat.com> Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
4943eba to
d9f69c7
Compare
|
Build failed (check pipeline). Post ✔️ openstack-meta-content-provider SUCCESS in 1h 37m 03s |
|
recheck |
|
Build failed (check pipeline). Post ✔️ openstack-meta-content-provider SUCCESS in 1h 39m 42s |
|
recheck |
|
Build failed (check pipeline). Post Warning: |
- Rename all Zuul job names from nova-operator-* to workloads-operator-* - Rename Zuul variables: nova_repo → workloads_repo, nova_hook → workloads_hook - Update all CI directory references in .zuul.yaml - Rename CI directories: • ci/nova-operator-* → ci/workloads-operator-* - Update CI config files with new directory paths - Rename GitHub workflow files - Update workflow job names and image references Signed-off-by: Amartya Sinha <amartya@redhat.com> Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Update README.md title, descriptions, and examples - Update AGENTS.md with workloads-operator references and Cyborg mention - Update all doc/*.md files with new operator name - Update CI playbook paths and variable names in documentation Signed-off-by: Amartya Sinha <amartya@redhat.com> Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Update Dockerfile with workloads-operator image metadata - Update Makefile with workloads-operator image URLs and targets - Update ci/olm.sh with workloads-operator CatalogSource and Subscription Signed-off-by: Amartya Sinha <amartya@redhat.com> Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Update API type comments to "generated by the workloads-operator" - Update CRD descriptions in all auto-generated files - Update cmd/main.go package comment - Update internal code comments (reconciler, controller) - Update hack/run_with_local_webhook.sh CSV selector - Update templates and test assertions - Update deployment name references in tests Signed-off-by: Amartya Sinha <amartya@redhat.com> Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com>
bc74868 to
8ca07be
Compare
|
Build failed (check pipeline). Post ✔️ openstack-meta-content-provider SUCCESS in 2h 21m 46s |
Rename the operator from nova-operator to workloads-operator to reflect
its expanded scope managing multiple OpenStack workload services (Nova,
Placement, and Cyborg).
Changes include:
Go module paths intentionally kept as nova-operator to maintain compatibility
with existing consumers (e.g., openstack-operator) during the transition.
Module paths will be updated after the GitHub repository is renamed.
Assisted-By: Claude Sonnet 4.5 noreply@anthropic.com