Skip to content

Commit

Permalink
ci/release: Fix secret handling for Dockerhub publishing (#35967)
Browse files Browse the repository at this point in the history
Signed-off-by: Ryan Northey <[email protected]>
  • Loading branch information
phlax authored Sep 4, 2024
1 parent dbcee3e commit 788f266
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/_publish_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,9 @@ jobs:
permissions:
contents: read
packages: read
secrets:
dockerhub-username: ${{ secrets.dockerhub-username }}
dockerhub-password: ${{ secrets.dockerhub-password }}
name: ${{ matrix.name || matrix.target }}
needs:
- binary
Expand All @@ -132,8 +135,6 @@ jobs:
source: |
export NO_BUILD_SETUP=1
export ENVOY_DOCKER_IN_DOCKER=1
export DOCKERHUB_USERNAME=${{ secrets.dockerhub-username }}
export DOCKERHUB_PASSWORD=${{ secrets.dockerhub-password }}
trusted: ${{ inputs.trusted }}
upload-name: docker
upload-path: build_images
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/_run.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ on:
secrets:
app-id:
app-key:
dockerhub-username:
dockerhub-password:
gpg-key:
gpg-key-password:
rbe-key:
Expand Down Expand Up @@ -279,6 +281,8 @@ jobs:
working-directory: ${{ inputs.working-directory }}
env:
GITHUB_TOKEN: ${{ inputs.trusted && steps.appauth.outputs.token || github.token }}
DOCKERHUB_USERNAME: ${{ secrets.dockerhub-username }}
DOCKERHUB_PASSWORD: ${{ secrets.dockerhub-password }}
ENVOY_DOCKER_BUILD_DIR: ${{ runner.temp }}
ENVOY_RBE: ${{ inputs.rbe == true && 1 || '' }}
RBE_KEY: ${{ secrets.rbe-key }}
Expand Down

0 comments on commit 788f266

Please sign in to comment.