Skip to content

Commit 2b56a28

Browse files
committed
ci: Only push to quay if github.actor is not dependabot
We skipped logging in to quay.io when github.actor is dependabot in [#96] but did not check if any pushes were attempted. This change will avoid that failure. This was missed in the subsequent dependabot PRs [#89] and [#95] because there was no branch protection setup and I added the ready-to-merge label so mergify merged it :(. I've added branch protection to require the actions to pass. [#89]: #89 [#95]: #95 [#96]: #96 Signed-off-by: Manuel Mendez <mmendez@equinix.com>
1 parent 07b0c29 commit 2b56a28

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/push.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,15 +30,15 @@ jobs:
3030
with:
3131
context: ./bootkit/
3232
platforms: linux/amd64,linux/arm64
33-
push: true
33+
push: ${{ github.actor != 'dependabot[bot]' }}
3434
tags: quay.io/tinkerbell/hook-bootkit:0.0
3535

3636
- name: Build and push tink-docker
3737
uses: docker/build-push-action@v2.4.0
3838
with:
3939
context: ./tink-docker/
4040
platforms: linux/amd64,linux/arm64
41-
push: true
41+
push: ${{ github.actor != 'dependabot[bot]' }}
4242
tags: quay.io/tinkerbell/hook-docker:0.0
4343

4444
- uses: cachix/install-nix-action@v16

0 commit comments

Comments
 (0)