Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/system/hetzner-robotlb/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
apiVersion: v2
name: hetzner-robotlb
name: cozy-hetzner-robotlb

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

The chart name has been updated, but the Makefile still uses the old name. This inconsistency can cause issues during deployment. Please update the Makefile to reflect the new chart name.

export NAME=cozy-hetzner-robotlb
export NAMESPACE=$(NAME)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Verification agent

🧩 Analysis chain

Check downstream references before renaming the chart
Renaming alters the value exposed by .Chart.Name. If any template, script, or documentation still has the literal string hetzner-robotlb, label ⇄ selector symmetry or CI logic could break. Please grep the repo (including vendored sub-charts) and update or explicitly confirm no stale references remain.


🏁 Script executed:

#!/bin/bash
# Locate hard-coded uses of the old chart name
rg -n "hetzner-robotlb" --type yml --type yaml --type tpl --type sh --type md

Length of output: 105


🏁 Script executed:

#!/bin/bash
# Re-run search without relying on ripgrep file-type aliases to catch all occurrences
rg -n "hetzner-robotlb" --glob '*.{yaml,yml,tpl,sh,md}'

Length of output: 1200


Update all downstream references to the renamed chart
Renaming .Chart.Name to cozy-hetzner-robotlb has left multiple hard-coded references to the old name. Please update the following locations (and any others you may find) so they match the new chart name and namespace:

• packages/system/hetzner-robotlb/values.yaml

  • Line 3: existingSecrets: replace "hetzner-robotlb-credentials""cozy-hetzner-robotlb-credentials"

• packages/core/platform/bundles/paas-full.yaml

  • Line 419: - name: hetzner-robotlb- name: cozy-hetzner-robotlb
  • Line 422: chart: cozy-hetzner-robotlb (already correct)
  • Line 423: namespace: cozy-hetzner-robotlb (verify intended)

• packages/core/platform/bundles/paas-hosted.yaml

  • Line 242: - name: hetzner-robotlb- name: cozy-hetzner-robotlb
  • Lines 245–246: verify chart: and namespace: lines match the new name

• packages/core/platform/bundles/distro-full.yaml

  • Line 262: - name: hetzner-robotlb- name: cozy-hetzner-robotlb
  • Lines 265–266: verify chart: and namespace:

• packages/core/platform/bundles/distro-hosted.yaml

  • Line 175: - name: hetzner-robotlb- name: cozy-hetzner-robotlb
  • Lines 178–179: verify chart: and namespace:

After making these updates, rerun any CI workflows or tests that consume these bundle manifests to confirm nothing is broken.

🤖 Prompt for AI Agents
In packages/system/hetzner-robotlb/Chart.yaml at line 2, the chart name was
changed to cozy-hetzner-robotlb, but related references in other files still use
the old name. Update all downstream references accordingly: in
packages/system/hetzner-robotlb/values.yaml line 3, change existingSecrets from
"hetzner-robotlb-credentials" to "cozy-hetzner-robotlb-credentials"; in
packages/core/platform/bundles/paas-full.yaml line 419, change name from
hetzner-robotlb to cozy-hetzner-robotlb and verify lines 422 and 423 for chart
and namespace; similarly update lines in paas-hosted.yaml (line 242 and lines
245-246), distro-full.yaml (line 262 and lines 265-266), and distro-hosted.yaml
(line 175 and lines 178-179) to use cozy-hetzner-robotlb consistently for name,
chart, and namespace. After these changes, rerun CI and tests to ensure
correctness.

version: 0.1.3 # Placeholder, the actual version will be automatically set during the build process
Loading