-
Notifications
You must be signed in to change notification settings - Fork 522
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
Error when listing Azure Registries #336
Comments
I investigated this issue and it has to do with container registries who have admin account enabled and yet do not permit you to view the username and password for your specific account. The rest of the registries don't load because this throws a promise rejection which causes the overall set of requests to fail. I am working on a large fix to eliminate the admin enabled problem, and this particular problem, should have a pull request open by later today. |
Thanks for stepping into this @estebanreyl ! We really appreciate your help. |
Thanks for the quick turnaround! Really appreciate it. Looking forward to the PR. |
@estebanreyl Can you also show us how to repro/test this? I assume this is a question of RBAC rights set up in Azure? |
Happy to help, as a heads up, I am almost ready to PR. Also, sorry for the delay replying. To answer @StephenWeatherford replicating the issue is a bit difficult and requires at least 2 accounts, but it goes as follows:
@StephenWeatherford Noting how annoying this is to do I just created a new registry called rbacextensiontest.azurecr.io and added you as a reader ideally just running the extension should make it work, |
Thanks! I can repro with your account. |
@khilscher Just wanted to let you know we've released 0.2.0. Please let us know if this does not fix your issue. Thanks! |
@StephenWeatherford yes that resolved it. Many thanks. |
Glad to hear it! Should out to @estebanreyl |
When I click on "Registries" -> "Azure" and succesfully log in, the following error pops up, and I cannot view any of my Azure container registries.
The text was updated successfully, but these errors were encountered: