From c9879fae998db0f10c18150c9143ed4f096394db Mon Sep 17 00:00:00 2001 From: mengw15 <125719918+mengw15@users.noreply.github.com> Date: Sun, 12 Jul 2026 16:39:17 -0700 Subject: [PATCH] ci: add a computing-unit-master boot smoke test in amber-integration amber-integration builds and unzips the amber dist to boot-smoke-test texera-web (#6319), but computing-unit-master -- the other server from the same WorkflowExecutionService/dist (bin/computing-unit-master, port 8085) -- is never started anywhere in CI, so a runtime classpath/linkage regression (the #6204 class of bug) in it would pass build and unit tests and only surface at deploy. Add one more boot step, reusing the dist already built and unzipped for the texera-web boot. Like texera-web it boots on postgres only: main starts a local Pekko actor master (clusterMode defaults to false, so no external seed and the cluster-only checkip.amazonaws.com lookup is skipped) and run() calls SqlServer.initConnection; the boot-time result-store cleanup is gated on ApplicationConfig.cleanupAllExecutionResults, which defaults to false, so no iceberg/S3 access. ubuntu-only, mirroring the texera-web step. Closes #6344. --- .github/workflows/build.yml | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 67943b907e6..fa3a51ff26b 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -639,6 +639,19 @@ jobs: # amber/src/main/resources/web-config.yml (port 8080). if: matrix.os == 'ubuntu-latest' run: .github/scripts/smoke-boot.sh "/tmp/dists/amber-*/bin/texera-web-application" 8080 + - name: Smoke-test computing-unit-master boots + # computing-unit-master boots on postgres only, like texera-web: main + # starts a LOCAL Pekko actor master (clusterMode defaults to false with + # no --cluster arg, so no external seed/discovery and the cluster-only + # checkip.amazonaws.com lookup is never reached), then run() calls + # SqlServer.initConnection against the postgres provisioned above. The + # boot-time result-store cleanup is gated on + # ApplicationConfig.cleanupAllExecutionResults, which defaults to false, + # so no iceberg / S3 access. Config resolves via Utils.amberHomePath to + # amber/src/main/resources/computing-unit-master-config.yml (port 8085). + # Reuses the amber dist built + unzipped above for the texera-web boot. + if: matrix.os == 'ubuntu-latest' + run: .github/scripts/smoke-boot.sh "/tmp/dists/amber-*/bin/computing-unit-master" 8085 - name: Run Python integration tests # --junit-xml feeds the Test Analytics upload below. run: |