CI: add basic Helm testing (apache#352)

* CI: add basic Helm testing

* Make test required

---------

Co-authored-by: Robert Stupp <[email protected]>

.asf.yaml

@@ -51,6 +51,7 @@ github:
           - build
           - regtest
           - site
+          - helm-tests
 
   features:
     wiki: false

.github/workflows/helm.yml

@@ -0,0 +1,91 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+
+# This workflow uses actions that are not certified by GitHub.
+# They are provided by a third-party and are governed by
+# separate terms of service, privacy policy, and support
+# documentation.
+
+name: Helm tests
+
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    branches: [ "main" ]
+
+jobs:
+
+  helm-tests:
+    name: Helm tests
+    runs-on: ubuntu-latest
+    timeout-minutes: 60
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Set up Helm
+        uses: azure/[email protected]
+        with:
+          version: 'v3.16.0'
+
+      - 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 }} --chart-dirs helm)
+          if [[ -n "$changed" ]]; then
+            echo "changed=true" >> "$GITHUB_OUTPUT"
+          fi
+
+      - name: Run 'helm template' validation
+        if: steps.list-changed.outputs.changed == 'true'
+        run: |
+          cd helm/polaris
+          for f in values.yaml ci/*.yaml; do
+            echo "::group::helm template $f"
+            helm template --debug --namespace polaris-ns --values $f .
+            echo "::endgroup::"
+          done
+
+      - name: Run chart-testing (lint)
+        if: steps.list-changed.outputs.changed == 'true'
+        run: ct lint --target-branch ${{ github.event.repository.default_branch }} --debug --charts ./helm/polaris
+
+      - name: Set up Minikube
+        if: steps.list-changed.outputs.changed == 'true'
+        uses: medyagh/[email protected]
+
+      - name: Docker build
+        if: steps.list-changed.outputs.changed == 'true'
+        run: |
+          eval $(minikube docker-env)
+          docker build -f ./Dockerfile -t polaris:latest .
+
+      - name: Run chart-testing (install)
+        if: steps.list-changed.outputs.changed == 'true'
+        run: |
+          ct install --target-branch ${{ github.event.repository.default_branch }} \
+            --helm-extra-set-args "--set=image.repository=polaris --set=image.tag=latest" \
+            --debug --charts ./helm/polaris

helm/polaris/Chart.yaml

@@ -29,3 +29,6 @@ sources:
 keywords:
   - polaris
   - iceberg
+maintainers:
+  - name: MonkeyCanCode
+  - name: adutra

helm/polaris/ci/simple-values.yaml

@@ -0,0 +1,23 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+
+image:
+  repository: polaris
+  tag: latest
+  pullPolicy: Never

helm/polaris/values.yaml

@@ -257,7 +257,7 @@ polarisServerConfig:
 
     # The hostname of the interface to which the HTTP server socket wil be found. If omitted, the
     # socket will listen on all interfaces.
-    #bindHost: localhost
+    # bindHost: localhost
 
     #  ssl:
     #    keyStore: ./example.keystore