Skip to content

Commit b650221

Browse files
committed
ci(satp-hermes): fix docker login
Signed-off-by: Rafael Belchior <[email protected]>
1 parent 22ccebe commit b650221

File tree

1 file changed

+6
-9
lines changed

1 file changed

+6
-9
lines changed

.github/workflows/satp-hermes-docker.yaml

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,8 @@ jobs:
9090
# Standardized image names for both registries
9191
BASE_IMAGE_NAME="hyperledger/cacti-satp-hermes-gateway"
9292
DOCKERHUB_IMAGE="${BASE_IMAGE_NAME}"
93-
GHCR_IMAGE="${BASE_IMAGE_NAME}"
93+
# GHCR requires the full path with owner
94+
GHCR_IMAGE="hyperledger-cacti/cacti-satp-hermes-gateway"
9495
9596
{
9697
echo "package_version=${PACKAGE_VERSION}"
@@ -279,14 +280,10 @@ jobs:
279280
uses: docker/setup-buildx-action@v3
280281

281282
- name: Login to Docker Hub
282-
run: |
283-
echo "🔍 Attempting Docker Hub login..."
284-
echo "Username: ${DOCKERHUB_USERNAME:0:3}***"
285-
echo "PAT configured: $([ -n "$DOCKERHUB_PAT" ] && echo 'yes' || echo 'no')"
286-
docker login -u "$DOCKERHUB_USERNAME" -p "$DOCKERHUB_PAT"
287-
env:
288-
DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
289-
DOCKERHUB_PAT: ${{ secrets.DOCKERHUB_PAT }}
283+
uses: docker/login-action@v3
284+
with:
285+
username: ${{ secrets.DOCKERHUB_USERNAME }}
286+
password: ${{ secrets.DOCKERHUB_PAT }}
290287

291288
- name: Build and push to Docker Hub (Release)
292289
if: needs.set-docker-tags.outputs.is_release == 'true'

0 commit comments

Comments
 (0)