Skip to content

Commit

Permalink
ci: Fix coverage/docs upload redirect path (#36423)
Browse files Browse the repository at this point in the history
Signed-off-by: Ryan Northey <[email protected]>
  • Loading branch information
phlax authored Oct 2, 2024
1 parent 520442a commit d78025f
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/_check_coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ jobs:
shell: bash
env:
GCS_ARTIFACT_BUCKET: ${{ inputs.trusted && 'envoy-postsubmit' || 'envoy-pr' }}
GCS_REDIRECT_PATH: ${{ fromJSON(inputs.request).request.pr || fromJSON(inputs.request).request.target-branch }}
target: ${{ matrix.target }}
timeout-minutes: 180
trusted: ${{ inputs.trusted }}
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/_precheck_publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -79,3 +79,4 @@ jobs:
shell: bash
env:
GCS_ARTIFACT_BUCKET: ${{ inputs.trusted && 'envoy-postsubmit' || 'envoy-pr' }}
GCS_REDIRECT_PATH: ${{ fromJSON(inputs.request).request.pr || fromJSON(inputs.request).request.target-branch }}
1 change: 0 additions & 1 deletion ci/do_ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ FETCH_PROTO_TARGETS=(
@com_github_bufbuild_buf//:bin/buf
//tools/proto_format/...)

GCS_REDIRECT_PATH="${SYSTEM_PULLREQUEST_PULLREQUESTNUMBER:-${BUILD_SOURCEBRANCHNAME}}"

retry () {
local n wait iterations
Expand Down
3 changes: 1 addition & 2 deletions ci/run_envoy_docker.sh
Original file line number Diff line number Diff line change
Expand Up @@ -158,15 +158,14 @@ docker run --rm \
-e ENVOY_REPO \
-e ENVOY_TARBALL_DIR \
-e ENVOY_GEN_COMPDB_OPTIONS \
-e SYSTEM_PULLREQUEST_PULLREQUESTNUMBER \
-e GCS_ARTIFACT_BUCKET \
-e GCS_REDIRECT_PATH \
-e GITHUB_REF_NAME \
-e GITHUB_REF_TYPE \
-e GITHUB_TOKEN \
-e GITHUB_APP_ID \
-e GITHUB_INSTALL_ID \
-e MOBILE_DOCS_CHECKOUT_DIR \
-e BUILD_SOURCEBRANCHNAME \
-e BAZELISK_BASE_URL \
-e ENVOY_BUILD_ARCH \
-e SYSTEM_STAGEDISPLAYNAME \
Expand Down

0 comments on commit d78025f

Please sign in to comment.