Skip to content

Commit a5a833a

Browse files
committed
github/workflows: Fix go-tests Docker login
Commit 86c4e1b was suppose to fix the Docker login tests. However, the variable introduced was defined in the scope of the first step and not globally for the job, making the if that checks for the repo name not work. This commit fixes this issue. Signed-off-by: Renê de Souza Pinto <rene@renesp.com.br>
1 parent b85a7c6 commit a5a833a

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/go-tests.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,9 @@ on: # yamllint disable-line rule:truthy
2525
jobs:
2626
test:
2727
runs-on: zededa-ubuntu-2204
28+
env:
29+
REPO_NAME: ${{ github.event.repository.full_name }}
30+
GH_REF: ${{ github.ref }}
2831
steps:
2932
- name: Starting Report
3033
run: |
@@ -34,9 +37,6 @@ jobs:
3437
df -h
3538
echo Memory
3639
free -m
37-
env:
38-
REPO_NAME: ${{ github.event.repository.full_name }}
39-
GH_REF: ${{ github.ref }}
4040
- name: Clear repository
4141
run: |
4242
sudo rm -fr "$GITHUB_WORKSPACE" && mkdir "$GITHUB_WORKSPACE"
@@ -48,7 +48,7 @@ jobs:
4848
ref: ${{ github.event.pull_request.head.ref }}
4949
fetch-depth: 0
5050
- name: Login to Docker Hub
51-
if: ${{ env.REPO_NAME == 'lf-edge/eve' }}
51+
if: ${{ env.REPO_NAME == 'rene/eve' }}
5252
uses: docker/login-action@5e57cd118135c172c3672efd75eb46360885c0ef # v3.6.0
5353
with:
5454
username: ${{ secrets.DOCKERHUB_PULL_USER }}

0 commit comments

Comments
 (0)