[BUG] 🐛 fix invalid ci for chart

Author: Eneman Donatien

.github/workflows/ci-chart.yaml

@@ -5,37 +5,54 @@ on:
     branches:
       - "**"
     paths-ignore:
-      - 'docs/**'
+      - "docs/**"
     tags:
       - "*"
   pull_request:
-    paths-ignore:
-      - 'docs/**'
+    paths:
+      - "deploy/charts/**"
 
 jobs:
-
-  helm-lint:
+  helm-lint-test:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v3
+        with:
+          fetch-depth: 0
 
-      - name: Install Helm on Runner
+      - name: Set up Helm
         uses: azure/[email protected]
+        with:
+          version: v3.17.0
 
-      - name: Lint Chart
-        run: helm lint .
-  
-  helm-unitest:
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
+      - uses: actions/[email protected]
+        with:
+          python-version: "3.x"
+          check-latest: true
 
-      - name: Install Helm on Runner
-        uses: azure/[email protected]
+      - name: Set up chart-testing
+        uses: helm/[email protected]
+
+      - name: Run chart-testing (list-changed)
+        id: list-changed
+        run: |
+          changed=$(ct list-changed --target-branch ${{ github.event.repository.default_branch }})
+          if [[ -n "$changed" ]]; then
+            echo "changed=true" >> "$GITHUB_OUTPUT"
+          fi
+
+      - name: Run chart-testing (lint)
+        if: steps.list-changed.outputs.changed == 'true'
+        run: ct lint --target-branch ${{ github.event.repository.default_branch }}
+
+      - name: Create kind cluster
+        if: steps.list-changed.outputs.changed == 'true'
+        uses: helm/[email protected]
 
-      - name: Lint Chart
-        run: echo "TODO"
+      - name: Run chart-testing (install)
+        if: steps.list-changed.outputs.changed == 'true'
+        run: ct install --target-branch ${{ github.event.repository.default_branch }}
 
   release_helm:
     runs-on: ubuntu-latest
@@ -57,4 +74,4 @@ jobs:
           charts_dir: deploy/charts
           skip_existing: true
         env:
-          CR_TOKEN: "${{ secrets.CR_TOKEN }}"
+          CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"

deploy/charts/s3-operator/Chart.yaml

@@ -13,9 +13,9 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.6.0
+version: 0.7.0
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to
 # follow Semantic Versioning. They should reflect the version the application is using.
 # It is recommended to use it with quotes.
-appVersion: "1.0.0"
+appVersion: "v0.12.0"

deploy/charts/s3-operator/values.yaml

@@ -45,7 +45,7 @@ s3:
     # allowedNamespaces: ""
     # Should the operator try to delete the resource from the S3 backend upon CR deletion ?
     deletion:
-      bucket: false
+      bucket: true
       path: false
       policy: false
       s3user: false