fix: remove code that confuses jinja

JasperHG90 · Jan 6, 2024 · 89eedc4 · 89eedc4
1 parent afaf555
commit 89eedc4
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/.github/workflows/pipeline.yaml b/.github/workflows/pipeline.yaml
@@ -64,7 +64,7 @@ jobs:
       #   run: |
       #     VERSION=$(echo $GITHUB_REF | sed -e 's/refs\/tags\///g')
       #     echo "VERSION=$VERSION" >> $GITHUB_ENV
-      - name: Setup nginx
+      - name: Parse values
         uses: cuchi/[email protected]
         with:
           template: deployment/values.yaml.j2

diff --git a/deployment/values.yaml.j2 b/deployment/values.yaml.j2
@@ -126,7 +126,6 @@ deployments:
       enabled: true
       # If `readinessProbe` has no `exec` field, then the following default will be used:
       # exec:
-      #   command: ["dagster", "api", "grpc-health-check", "-p", "{{ $deployment.port }}"]
       periodSeconds: 20
       timeoutSeconds: 10
       successThreshold: 1