ci: fix workflows lint & test (#1344)

.github/workflows/lint-test.yaml

@@ -24,6 +24,11 @@ jobs:
       - name: Set up chart-testing
         uses: helm/[email protected]
 
+      - name: Add Helm Repositories
+        run: |
+          helm repo add sentry-kubernetes https://sentry-kubernetes.github.io/charts
+          helm repo update
+
       - name: Run chart-testing (list-changed)
         id: list-changed
         run: |

.github/workflows/push.yaml

@@ -28,17 +28,17 @@ jobs:
       - name: Build clickhouse chart
         uses: WyriHaximus/github-action-helm3@v4
         with:
-          exec: helm package -u main/clickhouse --destination gh-pages/charts
+          exec: helm package -u main/charts/clickhouse --destination gh-pages/charts
 
       - name: Build sentry chart
         uses: WyriHaximus/github-action-helm3@v4
         with:
-          exec: helm package -u main/sentry --destination gh-pages/charts
+          exec: helm package -u main/charts/sentry --destination gh-pages/charts
 
       - name: Build sentry-kubernetes chart
         uses: WyriHaximus/github-action-helm3@v4
         with:
-          exec: helm package -u main/sentry-kubernetes --destination gh-pages/charts
+          exec: helm package -u main/charts/sentry-kubernetes --destination gh-pages/charts
 
       - name: Build sentry-kubernetes chart
         uses: WyriHaximus/github-action-helm3@v4

clickhouse/templates/NOTES.txt → charts/clickhouse/templates/NOTES.txt

@@ -1,31 +1,31 @@
-** Please be patient while the chart is being deployed **
-
-1. Get the Clickhouse URL by running:
-
-{{- if .Values.clickhouse.ingress.enabled }}
-
-  export HOSTNAME=$(kubectl get ingress --namespace {{ .Release.Namespace }} {{ template "clickhouse.fullname" . }} -o jsonpath='{.spec.rules[0].host}')
-  echo "Clickhouse URL: http://$HOSTNAME/"
-
-{{- else }}
-
-  echo URL            : http://127.0.0.1:8080/
-  echo Management URL : http://127.0.0.1:8080/manager
-  kubectl port-forward --namespace {{ .Release.Namespace }} svc/{{ template "clickhouse.fullname" . }} 8123:{{ .Values.clickhouse.http_port }}
-  kubectl port-forward --namespace {{ .Release.Namespace }} svc/{{ template "clickhouse.fullname" . }} 9000:{{ .Values.clickhouse.tcp_port }}
-  kubectl port-forward --namespace {{ .Release.Namespace }} svc/{{ template "clickhouse.fullname" . }} 9009:{{ .Values.clickhouse.interserver_http_port }}
-
-{{- end }}
-
-2. Get the Tabix URL by running:
-
-{{- if .Values.tabix.ingress.enabled }}
-
-  export HOSTNAME=$(kubectl get ingress --namespace {{ .Release.Namespace }} {{ template "clickhouse.fullname" . }}-tabix -o jsonpath='{.spec.rules[0].host}')
-  echo "Tabix URL: http://$HOSTNAME/"
-  
-{{- else }}
-
-  kubectl port-forward --namespace {{ .Release.Namespace }} svc/{{ template "clickhouse.fullname" . }}-tabix 80
-  
-{{- end }}
+** Please be patient while the chart is being deployed **
+
+1. Get the Clickhouse URL by running:
+
+{{- if .Values.clickhouse.ingress.enabled }}
+
+  export HOSTNAME=$(kubectl get ingress --namespace {{ .Release.Namespace }} {{ template "clickhouse.fullname" . }} -o jsonpath='{.spec.rules[0].host}')
+  echo "Clickhouse URL: http://$HOSTNAME/"
+
+{{- else }}
+
+  echo URL            : http://127.0.0.1:8080/
+  echo Management URL : http://127.0.0.1:8080/manager
+  kubectl port-forward --namespace {{ .Release.Namespace }} svc/{{ template "clickhouse.fullname" . }} 8123:{{ .Values.clickhouse.http_port }}
+  kubectl port-forward --namespace {{ .Release.Namespace }} svc/{{ template "clickhouse.fullname" . }} 9000:{{ .Values.clickhouse.tcp_port }}
+  kubectl port-forward --namespace {{ .Release.Namespace }} svc/{{ template "clickhouse.fullname" . }} 9009:{{ .Values.clickhouse.interserver_http_port }}
+
+{{- end }}
+
+2. Get the Tabix URL by running:
+
+{{- if .Values.tabix.ingress.enabled }}
+
+  export HOSTNAME=$(kubectl get ingress --namespace {{ .Release.Namespace }} {{ template "clickhouse.fullname" . }}-tabix -o jsonpath='{.spec.rules[0].host}')
+  echo "Tabix URL: http://$HOSTNAME/"
+
+{{- else }}
+
+  kubectl port-forward --namespace {{ .Release.Namespace }} svc/{{ template "clickhouse.fullname" . }}-tabix 80
+
+{{- end }}

sentry-kubernetes/Chart.yaml → charts/sentry-kubernetes/Chart.yaml

@@ -13,3 +13,5 @@ keywords:
 sources:
   - https://github.com/getsentry/sentry-kubernetes
   - https://github.com/sentry-kubernetes/charts
+maintainers:
+  - name: sentry-kubernetes

sentry/values.yaml → charts/sentry/values.yaml

@@ -189,11 +189,11 @@ sentry:
     enableFeedback: false
     enableSpan: false
 
-    #example customFeature to enable Metrics(beta) https://docs.sentry.io/product/metrics/
-  #customFeatures:
-    #- organizations:custom-metric
-    #- organizations:custom-metrics-experimental
-    #- organizations:derive-code-mappings
+  # example customFeature to enable Metrics(beta) https://docs.sentry.io/product/metrics/
+  # customFeatures:
+  # - organizations:custom-metric
+  # - organizations:custom-metrics-experimental
+  # - organizations:derive-code-mappings
 
   worker:
     enabled: true
@@ -1619,13 +1619,13 @@ nginx:
     ports:
       http: 80
   extraLocationSnippet: false
-  #extraLocationSnippet: |
+  # extraLocationSnippet: |
   #  location /admin {
   #    allow 1.2.3.4; # VPN network
   #    deny all;
   #    proxy_pass http://sentry;
   #  }
-  ## Use this to enable an extra service account
+  # Use this to enable an extra service account
   # serviceAccount:
   #   create: false
   #   name: nginx
@@ -1992,7 +1992,7 @@ externalPostgresql:
   username: postgres
   # password: postgres
   # existingSecret: secret-name
-  ## set existingSecretKeys in a secret, if not specified, value from the secret won't be used 
+  ## set existingSecretKeys in a secret, if not specified, value from the secret won't be used
   existingSecretKeys: {}
   #   password: password
   #   username: username

release-please-config.json

@@ -1,15 +1,15 @@
 {
   "$schema": "https://raw.githubusercontent.com/googleapis/release-please/main/schemas/config.json",
   "packages": {
-    "clickhouse": {
+    "charts/clickhouse": {
       "release-type": "helm",
       "changelog-path": "CHANGELOG.md"
     },
-    "sentry": {
+    "charts/sentry": {
       "release-type": "helm",
       "changelog-path": "CHANGELOG.md"
     },
-    "sentry-kubernetes": {
+    "charts/sentry-kubernetes": {
       "release-type": "helm",
       "changelog-path": "CHANGELOG.md"
     }