-
Notifications
You must be signed in to change notification settings - Fork 763
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
Publish latest operator build to quay workflow is failing. #498
Comments
We need to fix this on priority. Its been a quite a good time we are seeing this issue. |
@iam-veeramalla Agreed. Happy to help out where/if I can. Feel free to reach out as I'm the one who implemented the workflow I believe. Just don't have any visibility to what those values are on the backend (my suspicion is that the creds are probably fine and it's more around |
Thanks @tylerauerbeck , I will take a look at it asap. |
@iam-veeramalla No problem. For reference, the PR that merged this in is here: #365 If you look at the notes it expects REGISTRY_URL to be quay.io/argocd-operator. So if the secret isn't set to that, then we can just make a small change to the workflow to utilize some other envvar's that are available in GHA to piece together the appropriate URL. |
Describe the bug
It appears that the workflow that publishes the latest operator build to quay has been failing. From looking at the latest push to
master
, the logs suggest that it's because it isn't using the appropriate credentials to talk to quay.Earlier in the workflow it should authenticate against the docker repository with the following secrets:
This is likely because of one of the following:
To Reproduce
Verify that the last push to
master
had a failed workflowOn any subsequent pushes to
master
, this workflow will continue to failExpected behavior
Each time a push to master occurs, the latest version of the operator should be pushed to the
latest
tag on quay.Screenshots
https://github.com/argoproj-labs/argocd-operator/runs/4216376173?check_suite_focus=true
Additional context
N/A
The text was updated successfully, but these errors were encountered: