-
Notifications
You must be signed in to change notification settings - Fork 109
[RFC-0010] Add azure auth library #909
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
Conversation
4790c7f to
d30ee7b
Compare
a41f5ed to
c8d661e
Compare
8685bc4 to
4d32efc
Compare
|
Wanted to register that I tried the |
dipti-pai
left a comment
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.
Would be nice to have some tests (where possible) for the new functionality.
cd86f6e to
849f54a
Compare
IMO testing only errors does not add much value. Integration tests would be valuable, but we don't have credits for Azure, only AWS and GCP |
eb537f2 to
21e0113
Compare
d3aa23e to
2a25d16
Compare
Signed-off-by: Matheus Pimenta <[email protected]>
stefanprodan
left a comment
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.
LGTM
This PR introduces the
azurelibrary from RFC-0010.Part of: fluxcd/flux2#5022
PR Stack:
azure<---gcpawscore