Skip to content

Commit 86de105

Browse files
eamonnmoloneyclaude
andcommitted
ci: include run-hash prefix in ingress host for non-PR triggers
The hostname previously omitted the GITHUB_WORKFLOW_JOB_ID prefix for merge_group and scheduled runs (the gate required pull_request.number), while the namespace always included it. Back-to-back runs sharing a scenario+chart-version slug then collided on the shared host while their namespaces didn't, and Playwright hit a prior run's pods. Surfaced as esa0 auth0-smoke client_id mismatches where Expected was this run's freshly-minted Auth0 IDs and Received was an earlier run's IDs. Drop the is_pr gate so the prefix is applied whenever deployment-ttl is empty, matching the namespace logic above. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 206f997 commit 86de105

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/actions/workflow-vars/action.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ runs:
181181
export INGRESS_HOSTNAME_BASE="${INFRA_INGRESS_HOSTNAME_BASE}"
182182
183183
TEST_INGRESS_HOST="${TEST_IDENTIFIER}.${INGRESS_HOSTNAME_BASE}"
184-
if [[ "${{ inputs.deployment-ttl }}" == "" ]] && is_pr; then
184+
if [[ "${{ inputs.deployment-ttl }}" == "" ]]; then
185185
TEST_INGRESS_HOST="${GITHUB_WORKFLOW_JOB_ID}-${TEST_INGRESS_HOST}"
186186
fi
187187
# The var is needed in some non-shell steps.

0 commit comments

Comments
 (0)