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
Add support for file based authorizer for Azure #10876
Conversation
Commit ea92dfe00560795823cc26f4614aa6913643b82d does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
Please set the appropriate release note label. |
Please set the appropriate release note label. |
Signed-off-by: Ashray Jain <ashrayj@palantir.com>
ea92dfe
to
19f2127
Compare
Commit ea92dfe00560795823cc26f4614aa6913643b82d does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
test-me-please |
// failure does not seem related https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/18562/testReport/Suite-k8s-1/18/K8sFQDNTest_Restart_Cilium_validate_that_FQDN_is_still_working/ test-me-please |
restart-ginkgo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks!
This adds support for file based credentials when using Azure.
This PR prefers file based first and then falls back to environment based if that fails, but we could also reverse that if that's preferable.
Upstream Azure docs for file based auth live here: https://docs.microsoft.com/en-us/azure/developer/go/azure-sdk-authorization#use-file-based-authentication