Merge pull request #792 from rstudio/kegs-fix-connect-ci-check

Fix test script to use launcher.enabled and strict bash flags

Author: dbkegley

.github/workflows/chart-test.yaml

@@ -240,6 +240,9 @@ jobs:
       - name: Set up Helm
         uses: azure/setup-helm@v4
 
+      - name: Add rstudio helm repo
+        run: helm repo add rstudio https://helm.rstudio.com
+
       - uses: extractions/setup-just@v2
 
       - name: Run executable verification for default interpreters

Justfile

@@ -88,31 +88,35 @@ test chart='all':
 
 test-connect-interpreter-versions:
   #!/usr/bin/env bash
-  set -xe
+  set -euo pipefail
   cd ./charts/rstudio-connect && helm dependency build && cd -
 
   # find the default image
   image=$(
     helm template ./charts/rstudio-connect \
+    --set launcher.enabled=false \
     --show-only templates/deployment.yaml | \
     grep "image\:.*rstudio-connect.*" | \
     awk -F": " '{print $2}' | \
     xargs)
 
   for lang in "Python" "Quarto" "R"
   do
+    echo "Testing $lang"
+    
     # print the default connect config file for local execution in ini format
     # print the section and grep for the Executables to find each interpreter
     executables=$(
       helm template ./charts/rstudio-connect \
-      --set config.Launcher.Enabled=false \
+      --set launcher.enabled=false \
       --show-only templates/configmap.yaml | \
       sed -n -e "/\[$lang\]/,/\[*\]/ p" | \
       grep Executable | awk -F= '{print $2}' | \
-      xargs)
+      xargs || echo "")
 
     for ex in $executables
     do
+      echo "Checking $ex"
       docker run --rm $image /bin/bash -c "command -v $ex"
     done
   done

charts/rstudio-connect/Chart.yaml

@@ -1,6 +1,6 @@
 name: rstudio-connect
 description: Official Helm chart for Posit Connect
-version: 0.8.29
+version: 0.8.30
 apiVersion: v2
 appVersion: 2026.02.0
 icon: https://raw.githubusercontent.com/rstudio/helm/main/images/posit-icon-fullcolor.svg

charts/rstudio-connect/NEWS.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.8.30
+
+- Fix connect-version-check in CI 
+
 ## 0.8.29
 
 - Re-enable TensorFlow serving and remove executable path from Connect configuration

charts/rstudio-connect/README.md

@@ -1,6 +1,6 @@
 # Posit Connect
 
-![Version: 0.8.29](https://img.shields.io/badge/Version-0.8.29-informational?style=flat-square) ![AppVersion: 2026.02.0](https://img.shields.io/badge/AppVersion-2026.02.0-informational?style=flat-square)
+![Version: 0.8.30](https://img.shields.io/badge/Version-0.8.30-informational?style=flat-square) ![AppVersion: 2026.02.0](https://img.shields.io/badge/AppVersion-2026.02.0-informational?style=flat-square)
 
 #### _Official Helm chart for Posit Connect_
 
@@ -30,11 +30,11 @@ To ensure reproducibility in your environment and insulate yourself from future
 
 ## Installing the chart
 
-To install the chart with the release name `my-release` at version 0.8.29:
+To install the chart with the release name `my-release` at version 0.8.30:
 
 ```{.bash}
 helm repo add rstudio https://helm.rstudio.com
-helm upgrade --install my-release rstudio/rstudio-connect --version=0.8.29
+helm upgrade --install my-release rstudio/rstudio-connect --version=0.8.30
 ```
 
 To explore other chart versions, look at:

charts/rstudio-connect/values.yaml

@@ -438,13 +438,13 @@ config:
     # https://docs.posit.co/connect/admin/python/
     Executable:
       - /opt/python/3.12.11/bin/python
-      - /opt/python/3.11.13/bin/python
+      - /opt/python/3.13.9/bin/python
   Quarto:
     Enabled: true
     # Note: The `Executable` listed below is only used for Local Execution.
     # For Off-Host Execution, Quarto versions are defined by the set of Execution Environments
     # https://docs.posit.co/connect/admin/quarto/
-    Executable: "/opt/quarto/1.4.557/bin/quarto"
+    Executable: "/opt/quarto/1.8.25/bin/quarto"
   TensorFlow:
     Enabled: true
     # For off-host Execution, TensorFlow versions are defined by the set of Execution Environments https://docs.posit.co/connect/admin/tensorflow/