Skip to content

Commit c921a3c

Browse files
chore: fix syntax error in_docker variable (#41041)
## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!WARNING] > Tests have not run on the HEAD 48aa447 yet > <hr>Wed, 25 Jun 2025 13:11:28 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Bug Fixes** - Corrected formatting and quoting issues in the deployment script to ensure proper environment variable expansion and configuration output. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 7d75f3c commit c921a3c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

scripts/deploy_preview.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ export cluster_name=uatx-cluster
1515

1616
echo "[default]
1717
region = $region
18-
output = json
18+
output = json"
1919

2020
echo "Region: $region"
2121
echo "Cluster name: $cluster_name"
@@ -105,6 +105,6 @@ helm upgrade -i "$CHARTNAME" "appsmith-ee/$HELMCHART" -n "$NAMESPACE" --create-n
105105
--set applicationConfig.APPSMITH_CARBON_API_KEY="$APPSMITH_CARBON_API_KEY" \
106106
--set applicationConfig.APPSMITH_CARBON_API_BASE_PATH="$APPSMITH_CARBON_API_BASE_PATH" \
107107
--set applicationConfig.APPSMITH_AI_SERVER_MANAGED_HOSTING="$APPSMITH_AI_SERVER_MANAGED_HOSTING" \
108-
--set applicationConfig.IN_DOCKER='$IN_DOCKER' \
108+
--set applicationConfig.IN_DOCKER="$IN_DOCKER" \
109109
--set applicationConfig.APPSMITH_CUSTOMER_PORTAL_URL="https://release-customer.appsmith.com" \
110110
-f dp-node-affinity-values.yaml

0 commit comments

Comments
 (0)