Skip to content

Commit ac92fd3

Browse files
committed
Move Helm chart to charts/ and update refs
Rename Helm chart directory from deploy/helm/chart to charts and update all references accordingly. Adjusted devcontainer post-create script, Makefile, skaffold config, GitHub Actions release workflow (packaging, tagging), Dependabot directory, CODEOWNERS, and README examples. Removed values.production.yaml from the repository and added a README note indicating production values are managed externally. Small path fixes applied to chart values/local files and helm dependency commands to point at the new charts location.
1 parent 7971b71 commit ac92fd3

File tree

20 files changed

+20
-45
lines changed

20 files changed

+20
-45
lines changed

.devcontainer/post-create.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ helm repo add open-telemetry https://open-telemetry.github.io/opentelemetry-helm
4444
helm repo update
4545

4646
echo "→ Updating Helm chart dependencies..."
47-
helm dependency update deploy/helm/chart
47+
helm dependency update charts
4848

4949
echo ""
5050
echo "✅ Dev container ready!"

.github/CODEOWNERS

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33

44
* @your-org/platform-team
55

6-
/deploy/helm/ @your-org/platform-team
6+
/charts/ @your-org/platform-team
77
/.github/ @your-org/platform-team
88
/src/Infrastructure/ @your-org/platform-team

.github/dependabot.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ updates:
5151

5252
# ── Helm chart dependencies ───────────────────────────────────────────────────
5353
- package-ecosystem: helm
54-
directory: /deploy/helm/chart
54+
directory: /charts
5555
schedule:
5656
interval: weekly
5757
day: monday

.github/workflows/release.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,10 +115,10 @@ jobs:
115115

116116
- name: Update chart image tag
117117
run: |
118-
sed -i "s/tag: .*/tag: \"${{ steps.version.outputs.VERSION }}\"/" deploy/helm/chart/values.yaml
118+
sed -i "s/tag: .*/tag: \"${{ steps.version.outputs.VERSION }}\"/" charts/values.yaml
119119
120120
- name: Package Helm chart
121-
run: helm package deploy/helm/chart --version ${{ steps.version.outputs.VERSION }} --app-version ${{ steps.version.outputs.VERSION }}
121+
run: helm package charts --version ${{ steps.version.outputs.VERSION }} --app-version ${{ steps.version.outputs.VERSION }}
122122

123123
- name: Push Helm chart
124124
run: helm push *.tgz oci://${{ env.REGISTRY }}/${{ github.repository_owner }}/helm-charts

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ VERSION ?= $(shell git describe --tags --always --dirty 2>/dev/null || ec
1010
CLUSTER_NAME := service-template
1111
REGISTRY := localhost:5001
1212
IMAGE := $(REGISTRY)/service-template
13-
HELM_CHART := deploy/helm/chart
13+
HELM_CHART := charts
1414
HELM_RELEASE := service-template
1515
NAMESPACE := default
1616

README.md

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -213,19 +213,18 @@ make helm-template
213213
# Package
214214
make helm-package
215215

216-
# Deploy to a cluster (staging)
217-
helm upgrade --install my-service deploy/helm/chart \
218-
-f deploy/helm/chart/values.staging.yaml \
216+
# Deploy to staging
217+
helm upgrade --install my-service charts \
218+
-f charts/values.yaml \
219+
-f charts/values.staging.yaml \
219220
--set image.tag=1.2.3 \
220221
--namespace my-namespace --create-namespace
221-
222-
# Deploy to production
223-
helm upgrade --install my-service deploy/helm/chart \
224-
-f deploy/helm/chart/values.production.yaml \
225-
--set image.tag=1.2.3 \
226-
--namespace production
227222
```
228223

224+
> **Production values are not stored in this repo.**
225+
> Production deployments are managed via a dedicated GitOps repository that references the
226+
> Helm chart and container image released from here. Supply your production values there.
227+
229228
**The chart includes:**
230229
- Deployment with rolling updates and `maxUnavailable: 0`
231230
- Horizontal Pod Autoscaler (CPU + memory)
File renamed without changes.
File renamed without changes.

0 commit comments

Comments
 (0)