Merge pull request #3 from bytedance/fix/update-docker-registory

fix: update dockerhub in workflow
bytedance · Jun 2, 2020 · 0b3ebb2 · 0b3ebb2
2 parents d70ac95 + 18a2c58
commit 0b3ebb2
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/.github/workflows/build_push.yaml b/.github/workflows/build_push.yaml
@@ -24,10 +24,11 @@ jobs:
 
     - name: Docker Login
       run: |
-        docker login https://docker.pkg.github.com -u ${{ github.actor }} --password ${{ secrets.GITHUB_TOKEN }}
+        docker login -u fedlearner --password ${{ secrets.DOCKER_REGISTRY }}
 
     - name: Build Docker image
       run: |
         make build
         REGISTRY=docker.pkg.github.com/${{ github.repository }} TAG=${{ steps.vars.outputs.sha_short }} make container
-        docker push docker.pkg.github.com/${{ github.repository }}/nginx-ingress-controller-amd64:${{ steps.vars.outputs.sha_short }}
+        docker tag docker.pkg.github.com/${{ github.repository }}/nginx-ingress-controller-amd64:${{ steps.vars.outputs.sha_short }} fedlearner/nginx-ingress-controller:${{ steps.vars.outputs.sha_short }}
+        docker push fedlearner/nginx-ingress-controller:${{ steps.vars.outputs.sha_short }}