Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid clientid or client secret #46

Open
lsmith77 opened this issue Nov 17, 2022 · 3 comments
Open

Invalid clientid or client secret #46

lsmith77 opened this issue Nov 17, 2022 · 3 comments
Labels
need-to-triage Requires investigation

Comments

@lsmith77
Copy link

I suddenly get errors when pushing new docker containers to azure container registry with:

unauthorized: Invalid clientid or client secret.
Error: Process completed with exit code 1.

It was working fine for exactly a year but started breaking today, so I guess something expired.

My Github Action can be seen below. It fails on the build nlp api step but it seems to pass the login step:
image

What since it passes the login step, I don't get what has expired to cause the issue?

on: [push]
name: Build a Docker image and Push it to ACR
jobs:
  build:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3

      - uses: azure/docker-login@v1
        with:
          login-server: ${{ secrets.REGISTRY_LOGIN_SERVER }}
          username: ${{ secrets.REGISTRY_USERNAME }}
          password: ${{ secrets.REGISTRY_PASSWORD }}

      - name: Set up Python ${{ matrix.python-version }}
        uses: actions/setup-python@v4
        with:
          python-version: "3.9"

      - name: Install pipenv
        run: |
          python -m pip install --upgrade pipenv wheel

      - name: Update requirements.txt for Docker image
        run: |
          pipenv requirements > requirements.txt

      # build nlp api
      - run: |
          docker build . -t ${{ secrets.REGISTRY_LOGIN_SERVER }}/nlpapi:${{ github.sha }}
          docker push ${{ secrets.REGISTRY_LOGIN_SERVER }}/nlpapi:${{ github.sha }}
          # release version
          ref=${{ github.ref_name }}
          tag=${ref////-}
          echo $tag
          docker tag ${{ secrets.REGISTRY_LOGIN_SERVER }}/nlpapi:${{ github.sha }} ${{ secrets.REGISTRY_LOGIN_SERVER }}/nlpapi:$tag
          docker push ${{ secrets.REGISTRY_LOGIN_SERVER }}/nlpapi:$tag
@lsmith77 lsmith77 added the need-to-triage Requires investigation label Nov 17, 2022
@dalefrancis88
Copy link

@lsmith77 did you happen to figure out what expired? we're in the same boat and all things considered it should be all up to date credentials but it's still failing

@lsmith77
Copy link
Author

no. I even get help from multiple experts at Microsoft. in the end we just dropped it all together.

@medhabhatt
Copy link

Facing the same issue. @dalefrancis88 did you happen to resolve this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need-to-triage Requires investigation
Projects
None yet
Development

No branches or pull requests

3 participants