Skip to content

refactor: use ${releaseName}-${appVersion} prefix in resources#4247

Merged
takirala merged 20 commits intomainfrom
tga/use-appname-var
Nov 10, 2025
Merged

refactor: use ${releaseName}-${appVersion} prefix in resources#4247
takirala merged 20 commits intomainfrom
tga/use-appname-var

Conversation

@takirala
Copy link
Copy Markdown
Contributor

@takirala takirala commented Nov 7, 2025

What problem does this PR solve?:

I bumped the version of apps only if they haven't been bumped since 2.16.0
No functional changes were made to underlying applications.

Which issue(s) does this PR fix?:

Special notes for your reviewer:

Summary

This PR contains two major refactoring efforts:

  1. OCIRepository naming standardization - Standardized all OCIRepository resource names to use template variables
  2. Version path bumping - Prepared 12 applications for patch version updates

1. OCIRepository Naming Standardization

Standardized OCIRepository and ConfigMap resource names and corresponding HelmRelease chartRef.name and values references to use template variables (${releaseName}-${appVersion}-*) instead of hardcoded names.

Applications Refactored (11 total)

Single OCIRepository Applications

  • ai-navigator-cluster-info-agent (2 OCIRepositories)
    • Main: ${releaseName}-${appVersion}-chart
    • API: ${releaseName}-${appVersion}-api

Multi-OCIRepository Applications

  1. harbor (5 OCIRepositories)

    • Main: ${releaseName}-${appVersion}-chart
    • Components: valkey, database, manual-storage, cosi-bucket-kit
  2. cert-manager (2 OCIRepositories)

    • Main: ${releaseName}-${appVersion}-chart
    • CRDs: ${releaseName}-${appVersion}-crds
  3. cosi-driver-nutanix (2 OCIRepositories)

    • Main: ${releaseName}-${appVersion}-chart
    • COSI: ${releaseName}-${appVersion}-cosi-bucket-kit
  4. gatekeeper (2 OCIRepositories)

    • Main: ${releaseName}-${appVersion}-chart
    • Proxy: ${releaseName}-${appVersion}-proxy-mutations
  5. knative (2 OCIRepositories)

    • Main: ${releaseName}-${appVersion}-chart
    • Operator: ${releaseName}-${appVersion}-operator
  6. logging-operator (2 OCIRepositories)

    • Main: ${releaseName}-${appVersion}-chart
    • Logging: ${releaseName}-${appVersion}-logging
  7. project-grafana-loki (2 OCIRepositories)

    • Main: ${releaseName}-${appVersion}-chart
    • OBC: ${releaseName}-${appVersion}-object-bucket-claim
  8. traefik (2 OCIRepositories)

    • Main: ${releaseName}-${appVersion}-chart
    • CRDs: ${releaseName}-${appVersion}-crds
  9. rook-ceph-cluster (3 OCIRepositories)

    • Main: ${releaseName}-${appVersion}-chart
    • OBC: ${releaseName}-${appVersion}-object-bucket-claim
    • COSI: ${releaseName}-${appVersion}-cosi-bucket-kit
  10. istio-helm (5 OCIRepositories)

    • Components: base, cni, istiod, ztunnel, gateway

Naming Pattern

  • Single OCIRepository apps: Use ${releaseName}-${appVersion}-chart
  • Multi-OCIRepository apps: Use descriptive suffixes (e.g., -crds, -api, -operator, -logging)

2. Version Path Bumping

Bumped patch versions for 12 applications by updating path references and renaming version directories. Chart tags remain unchanged as these versions don't exist upstream yet.

Applications Updated

Application Old Version New Version
ai-navigator-app 0.5.1 0.5.2
ai-navigator-cluster-info-agent 0.4.1 0.4.2
ai-navigator-rag 0.7.3 0.7.4
grafana-loki 0.80.4 0.80.5
istio 1.23.4 1.23.5
jaeger 2.57.2 2.57.3
kubefed 0.11.2 0.11.3
kubernetes-dashboard 7.13.0 7.13.1
rook-ceph 1.17.6 1.17.7
rook-ceph-cluster 1.17.6 1.17.7
thanos 15.8.1 15.8.2
vgpu-token-operator 1.0.7 1.0.8

Changes Made

  • ✅ Updated all path: references in helmrelease.yaml, kustomization.yaml, and related files
  • ✅ Renamed version directories using git mv to preserve history (106 files moved)
  • ✅ Updated dependency references (e.g., prereq job names for rook-ceph-cluster)
  • Chart tags remain unchanged - All OCIRepository chart tags stay at current versions

Impact

  • 11 applications standardized for OCIRepository naming consistency
  • 12 applications were bumped
  • Better alignment with catalog v2 layout

Testing

  • Verified all path references point to correct new version directories
  • Confirmed chart tags remain unchanged
  • Validated directory renames preserve git history
  • Verified OCIRepository naming follows consistent pattern

Signed-off-by: Tarun Gupta Akirala <tarun.akirala@nutanix.com>
…riables

- Update main harbor OCIRepository to ${appName}-${releaseName}-source
- Update valkey OCIRepository to ${appName}-${releaseName}-valkey
- Update database OCIRepository to ${appName}-${releaseName}-database
- Update manual-storage OCIRepository to ${appName}-${releaseName}-manual-storage
- Update all corresponding HelmRelease chartRef.name references

This standardizes the naming pattern across all harbor components to use
template variables for better consistency and maintainability.
…naming to use template variables

- Update main agent OCIRepository to ${appName}-${releaseName}-source
- Update API OCIRepository to ${appName}-${releaseName}-api
- Update all corresponding HelmRelease chartRef.name references
…ate variables

- Update main cert-manager OCIRepository to ${appName}-${releaseName}-source
- Update cert-manager-crds OCIRepository to ${appName}-${releaseName}-crds
- Update all corresponding HelmRelease chartRef.name references
…e template variables

- Update main cosi-driver-nutanix OCIRepository to ${appName}-${releaseName}-source
- Update cosi-bucket-kit OCIRepository to ${appName}-${releaseName}-cosi-bucket-kit
- Update all corresponding HelmRelease chartRef.name references
…e variables

- Update main gatekeeper OCIRepository to ${appName}-${releaseName}-source
- Update gatekeeper-proxy-mutations OCIRepository to ${appName}-${releaseName}-proxy-mutations
- Update all corresponding HelmRelease chartRef.name references
…ariables

- Update main knative OCIRepository to ${appName}-${releaseName}-source
- Update knative-operator OCIRepository to ${appName}-${releaseName}-operator
- Update all corresponding HelmRelease chartRef.name references
…emplate variables

- Update main logging-operator OCIRepository to ${appName}-${releaseName}-source
- Update logging-operator-logging OCIRepository to ${appName}-${releaseName}-logging
- Update all corresponding HelmRelease chartRef.name references
…se template variables

- Update main loki-distributed OCIRepository to ${appName}-${releaseName}-source
- Update object-bucket-claim OCIRepository to ${appName}-${releaseName}-object-bucket-claim
- Update all corresponding HelmRelease chartRef.name references
…ariables

- Update main traefik OCIRepository to ${appName}-${releaseName}-source
- Update traefik-crds OCIRepository to ${appName}-${releaseName}-crds
- Update all corresponding HelmRelease chartRef.name references
…template variables

- Update main rook-ceph-cluster OCIRepository to ${appName}-${releaseName}-source
- Update object-bucket-claim OCIRepository to ${appName}-${releaseName}-object-bucket-claim
- Update cosi-bucket-kit OCIRepository to ${appName}-${releaseName}-cosi-bucket-kit
- Update all corresponding HelmRelease chartRef.name references
…e variables

- Update istio-helm-base OCIRepository to ${appName}-${releaseName}-base
- Update istio-helm-cni OCIRepository to ${appName}-${releaseName}-cni
- Update istio-helm-istiod OCIRepository to ${appName}-${releaseName}-istiod
- Update istio-helm-ztunnel OCIRepository to ${appName}-${releaseName}-ztunnel
- Update istio-helm-gateway OCIRepository to ${appName}-${releaseName}-gateway
- Update all corresponding HelmRelease chartRef.name references
Update path references to point to new patch versions:
- ai-navigator-app: 0.5.1 -> 0.5.2
- ai-navigator-cluster-info-agent: 0.4.1 -> 0.4.2
- ai-navigator-rag: 0.7.3 -> 0.7.4
- grafana-loki: 0.80.4 -> 0.80.5
- istio: 1.23.4 -> 1.23.5
- jaeger: 2.57.2 -> 2.57.3
- kubefed: 0.11.2 -> 0.11.3
- kubernetes-dashboard: 7.13.0 -> 7.13.1
- rook-ceph: 1.17.6 -> 1.17.7
- rook-ceph-cluster: 1.17.6 -> 1.17.7 (includes prereq job name updates)
- thanos: 15.8.1 -> 15.8.2
- vgpu-token-operator: 1.0.7 -> 1.0.8

Note: Chart tags remain unchanged as these versions don't exist upstream yet.
Signed-off-by: Tarun Gupta Akirala <tarun.akirala@nutanix.com>
Signed-off-by: Tarun Gupta Akirala <tarun.akirala@nutanix.com>
Signed-off-by: Tarun Gupta Akirala <tarun.akirala@nutanix.com>
Signed-off-by: Tarun Gupta Akirala <tarun.akirala@nutanix.com>
Signed-off-by: Tarun Gupta Akirala <tarun.akirala@nutanix.com>
Signed-off-by: Tarun Gupta Akirala <tarun.akirala@nutanix.com>
@takirala takirala force-pushed the tga/use-appname-var branch from aa2dc82 to 235f9e5 Compare November 10, 2025 00:28
@takirala takirala changed the title refactor: use appName & releaseName in oci repo name refactor: use ${releaseName}-${appVersion} prefix in resources Nov 10, 2025
@johnliu25 johnliu25 self-requested a review November 10, 2025 18:39
@takirala takirala merged commit 8b84f2a into main Nov 10, 2025
235 of 241 checks passed
@takirala
Copy link
Copy Markdown
Contributor Author

Will follow up if there are more changes. This is not tested end to end yet.

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

Labels

applications/ai-navigator-app applications/ai-navigator-cluster-info-agent applications/ai-navigator-rag applications/centralized-grafana applications/centralized-opencost applications/cert-manager applications/chartmuseum applications/cilium-hubble-relay-traefik applications/cloudnative-pg applications/cosi-driver-nutanix applications/dex applications/dex-k8s-authenticator applications/external-dns applications/external-secrets applications/fluent-bit applications/gatekeeper applications/gateway-api-crds applications/grafana-logging applications/grafana-loki applications/harbor applications/istio applications/istio-helm applications/jaeger applications/karma applications/karma-traefik applications/kiali applications/knative applications/kommander applications/kommander-appmanagement applications/kommander-flux applications/kommander-ui applications/kube-oidc-proxy applications/kube-prometheus-stack applications/kubefed applications/kubernetes-dashboard applications/kubetunnel applications/logging-operator applications/nkp-insights applications/nkp-insights-management applications/nkp-pulse-management applications/nkp-pulse-workspace applications/nvidia-gpu-operator applications/opencost applications/project-grafana-logging applications/project-grafana-loki applications/prometheus-adapter applications/prometheus-thanos-traefik applications/reloader applications/rook-ceph applications/rook-ceph-cluster applications/thanos applications/traefik applications/traefik-forward-auth applications/traefik-forward-auth-mgmt applications/velero applications/vgpu-token-operator auto-merge do-not-merge/testing Do not merge because there is still on-going testing ok-to-test Signals mergebot that CI checks are ready to be kicked off open-kommander-pr Automatically triggers the creation of a PR in Kommander repo size/M Denotes a PR that changes 30-99 lines, ignoring generated files. update-licenses signals mergebot to update licenses.d2iq.yaml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants