fix lint

timescale · Sep 14, 2023 · f5791cc · f5791cc
1 parent 94a6f05
commit f5791cc
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 38 deletions.
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -40,7 +40,7 @@ on:
 jobs:
   release:
     name: Build Docker
-    runs-on: "non-prod"
+    runs-on: non-prod
     steps:
     - name: Setup | Checkout
       uses: actions/checkout@v3

diff --git a/.github/workflows/deploy.yaml b/.github/workflows/deploy.yaml
@@ -133,22 +133,22 @@ jobs:
 
     - name: Helm Upgrade
       run: |
-          region_values=''
+        region_values=''
           if [[ -e "./${{ inputs.chart_path }}/values/${{ inputs.env }}-${{ inputs.region }}.yaml" ]]; then
-            region_values=--values=./${{ inputs.chart_path }}/values/${{ inputs.env }}-${{ inputs.region }}.yaml
+            region_values=--values="./${{ inputs.chart_path }}/values/${{ inputs.env }}-${{ inputs.region }}.yaml"
           fi
           standard_values=''
           if [[ "${{inputs.helm_inject_standard_deploy_values}}" == "true" ]]; then
             standard_values="--set-string imageName=${{ inputs.registry }} --set-string imageTag=${{ inputs.tag }} --set-string deployment.region=${{ inputs.region }}"
           fi
-          helm upgrade ${{ inputs.chart_name }} ${{ inputs.chart_path }} --install --wait --atomic --cleanup-on-fail \
-            --kubeconfig=./kubeconfig_${{ inputs.env }}_${{ inputs.region }} \
-            --namespace=${{ inputs.chart_namespace }} \
-            --values=./${{ inputs.chart_path }}/values/${{ inputs.env }}.yaml \
-            --timeout ${{ inputs.helm_timeout }} \
-            ${region_values} \
-            ${standard_values} \
-            ${{ inputs.helm_ext_args }}
+          helm upgrade "${{ inputs.chart_name }}" "${{ inputs.chart_path }}" --install --wait --atomic --cleanup-on-fail \
+            --kubeconfig="./kubeconfig_${{ inputs.env }}_${{ inputs.region }}" \
+            --namespace="${{ inputs.chart_namespace }}" \
+            --values="./${{ inputs.chart_path }}/values/${{ inputs.env }}.yaml" \
+            --timeout "${{ inputs.helm_timeout }}" \
+            "${region_values}" \
+            "${standard_values}" \
+            "${{ inputs.helm_ext_args }}"
 
   # Prod - Deploy against all the regions specified in strategy.matrix.region
   deploy_all_regions_prod:
@@ -188,20 +188,20 @@ jobs:
           echo "####### Helm upgrade for ${{ inputs.env }}_${{ matrix.region }} ####### "
           region_values=''
           if [[ -e "./${{ inputs.chart_path }}/values/${{ inputs.env }}-${{ matrix.region }}.yaml" ]]; then
-            region_values=--values=./${{ inputs.chart_path }}/values/${{ inputs.env }}-${{ matrix.region }}.yaml
+            region_values=--values="./${{ inputs.chart_path }}/values/${{ inputs.env }}-${{ matrix.region }}.yaml"
           fi
           standard_values=''
           if [[ "${{inputs.helm_inject_standard_deploy_values}}" == "true" ]]; then
             standard_values="--set-string imageName=${{ inputs.registry }} --set-string imageTag=${{ inputs.tag }} --set-string deployment.region=${{ matrix.region }}"
           fi
-          helm upgrade ${{ inputs.chart_name }} ${{ inputs.chart_path }} --install --wait --atomic --cleanup-on-fail \
-            --kubeconfig=./kubeconfig_${{ inputs.env }}_${{ matrix.region }} \
-            --namespace=${{ inputs.chart_namespace }} \
-            --values=./${{ inputs.chart_path }}/values/${{ inputs.env }}.yaml \
-            --timeout ${{ inputs.helm_timeout }} \
-            ${region_values} \
-            ${standard_values} \
-            ${{ inputs.helm_ext_args }}
+          helm upgrade "${{ inputs.chart_name }}" "${{ inputs.chart_path }}" --install --wait --atomic --cleanup-on-fail \
+            --kubeconfig="./kubeconfig_${{ inputs.env }}_${{ matrix.region }}" \
+            --namespace="${{ inputs.chart_namespace }}" \
+            --values="./${{ inputs.chart_path }}/values/${{ inputs.env }}.yaml" \
+            --timeout "${{ inputs.helm_timeout }}" \
+            "${region_values}" \
+            "${standard_values}" \
+            "${{ inputs.helm_ext_args }}"
 
 # Dev - Deploy against all the regions specified in strategy.matrix.region
   deploy_all_regions_dev:
@@ -237,17 +237,17 @@ jobs:
           echo "####### Helm upgrade for ${{ inputs.env }}_${{ matrix.region }} ####### "
           region_values=''
           if [[ -e "./${{ inputs.chart_path }}/values/${{ inputs.env }}-${{ matrix.region }}.yaml" ]]; then
-            region_values=--values=./${{ inputs.chart_path }}/values/${{ inputs.env }}-${{ matrix.region }}.yaml
+            region_values=--values="./${{ inputs.chart_path }}/values/${{ inputs.env }}-${{ matrix.region }}.yaml"
           fi
           standard_values=''
           if [[ "${{inputs.helm_inject_standard_deploy_values}}" == "true" ]]; then
             standard_values="--set-string imageName=${{ inputs.registry }} --set-string imageTag=${{ inputs.tag }} --set-string deployment.region=${{ matrix.region }}"
           fi
-          helm upgrade ${{ inputs.chart_name }} ${{ inputs.chart_path }} --install --wait --atomic --cleanup-on-fail \
-            --kubeconfig=./kubeconfig_${{ inputs.env }}_${{ matrix.region }} \
-            --namespace=${{ inputs.chart_namespace }} \
-            --values=./${{ inputs.chart_path }}/values/${{ inputs.env }}.yaml \
-            --timeout ${{ inputs.helm_timeout }} \
-            ${region_values} \
-            ${standard_values} \
-            ${{ inputs.helm_ext_args }}
+          helm upgrade "${{ inputs.chart_name }}" "${{ inputs.chart_path }}" --install --wait --atomic --cleanup-on-fail \
+            --kubeconfig="./kubeconfig_${{ inputs.env }}_${{ matrix.region }}" \
+            --namespace="${{ inputs.chart_namespace }}" \
+            --values="./${{ inputs.chart_path }}/values/${{ inputs.env }}.yaml" \
+            --timeout "${{ inputs.helm_timeout }}" \
+            "${region_values}" \
+            "${standard_values}" \
+            "${{ inputs.helm_ext_args }}"
diff --git a/.github/workflows/linter.yaml b/.github/workflows/linter.yaml
@@ -20,15 +20,7 @@ jobs:
 
     - uses: reviewdog/action-actionlint@v1
       with: 
-        actionlint_flags: -ignore SC2086 -config-file .github/actionlint.yaml
+        actionlint_flags: -config-file .github/actionlint.yaml
         fail_on_error: true
         level: error
-
-    #- name: Download action linter
-    #  run: bash <(curl https://raw.githubusercontent.com/rhysd/actionlint/main/scripts/download-actionlint.bash)
-
-    - name: Run action linter
-      run: |
-        ./actionlint \
-          ${{ steps.files.outputs.all }}
 
diff --git a/actionlint.yaml b/actionlint.yaml
@@ -0,0 +1,3 @@
+self-hosted-runner:
+  labels:
+    - non-prod