From a0a97eb0781964642b9fe0e2fa7e96e3786f6b93 Mon Sep 17 00:00:00 2001 From: Wyatt Walter Date: Mon, 9 Mar 2026 14:59:29 -0500 Subject: [PATCH] chore: move dp cluster to traefik (#41600) ## 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="" ### :mag: Cypress test results > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [x] No ## Summary by CodeRabbit * **Chores** * Enhanced deployment reliability by implementing graceful service shutdown before cleanup operations in the recreation flow. * Updated ingress controller configuration and removed SSL certificate annotation from load balancer settings. --- scripts/deploy_preview.sh | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/scripts/deploy_preview.sh b/scripts/deploy_preview.sh index 436c8cbca839..6547d8462a45 100755 --- a/scripts/deploy_preview.sh +++ b/scripts/deploy_preview.sh @@ -55,7 +55,7 @@ if [[ -n "${RECREATE-}" ]]; then # execute this db.dropDatabase() from the k8s cluster because there's network restrictions on Atlas cluster. # The \$ is used to escape the $ character in the APPSMITH_DB_URL environment variable so it's interpolated inside the kubectl exec command. kubectl exec "$pod_name" -n "$NAMESPACE" -- bash -c "mongosh \$APPSMITH_DB_URL --eval 'db.dropDatabase()'" - kubectl exec "$pod_name" -n "$NAMESPACE" -- bash -c "rm -rf /appsmith-stacks/*" + kubectl exec "$pod_name" -n "$NAMESPACE" -- bash -c "supervisorctl stop all && rm -rf /appsmith-stacks/*" kubectl delete ns "$NAMESPACE" || true kubectl patch pv "$NAMESPACE-appsmith" -p '{"metadata":{"finalizers":null}}' || true kubectl delete pv "$NAMESPACE-appsmith" --grace-period=0 --force || true @@ -103,10 +103,9 @@ helm upgrade -i "$CHARTNAME" "appsmith-ee/$HELMCHART" -n "$NAMESPACE" --create-n --set postgresql.enabled=false \ --set mongodb.enabled=false \ --set ingress.enabled=true \ - --set "ingress.annotations.service\.beta\.kubernetes\.io/aws-load-balancer-ssl-cert=$AWS_RELEASE_CERT" \ --set "ingress.hosts[0].host=$DOMAINNAME, ingress.hosts[0].paths[0].path=/, ingress.hosts[0].paths[0].pathType=Prefix" \ --set autoupdate.enabled=false \ - --set ingress.className="nginx" \ + --set ingress.className="traefik" \ --set persistence.efs.enabled=true \ --set persistence.efs.driver="efs.csi.aws.com" \ --set persistence.storageClass="efs-sc" \