-
Notifications
You must be signed in to change notification settings - Fork 198
[robotlb] fix chart name for installing #1237
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,3 @@ | ||
| apiVersion: v2 | ||
| name: hetzner-robotlb | ||
| name: cozy-hetzner-robotlb | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainCheck downstream references before renaming the chart 🏁 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 mdLength 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 • packages/system/hetzner-robotlb/values.yaml
• packages/core/platform/bundles/paas-full.yaml
• packages/core/platform/bundles/paas-hosted.yaml
• packages/core/platform/bundles/distro-full.yaml
• packages/core/platform/bundles/distro-hosted.yaml
After making these updates, rerun any CI workflows or tests that consume these bundle manifests to confirm nothing is broken. 🤖 Prompt for AI Agents |
||
| version: 0.1.3 # Placeholder, the actual version will be automatically set during the build process | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The chart name has been updated, but the
Makefilestill uses the old name. This inconsistency can cause issues during deployment. Please update theMakefileto reflect the new chart name.