-
Notifications
You must be signed in to change notification settings - Fork 45
feat(auth): add support for DefaultAzureCredential #665
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
base: main
Are you sure you want to change the base?
feat(auth): add support for DefaultAzureCredential #665
Conversation
Signed-off-by: Gabriele Fedi <gabriele.fedi@enterprisedb.com>
02915f4 to
1c3dd85
Compare
Signed-off-by: Gabriele Fedi <gabriele.fedi@enterprisedb.com>
Signed-off-by: Gabriele Fedi <gabriele.fedi@enterprisedb.com>
Signed-off-by: Gabriele Fedi <gabriele.fedi@enterprisedb.com>
Signed-off-by: Gabriele Fedi <gabriele.fedi@enterprisedb.com>
Signed-off-by: Gabriele Fedi <gabriele.fedi@enterprisedb.com>
a4fbd84 to
2cb76ef
Compare
|
We should not go ahead with this patch. Barman has added an explicit |
Do you know if that work is planned? Cause all I can find mentioned is this cloudnative-pg/barman-cloud#115 but that’s much older than this ticket. |
Actually reviewing cloudnative-pg/barman-cloud#59 and https://github.com/cloudnative-pg/barman-cloud/pull/64/files Does this not already chain to your linked commit and pass default in? Or can we not set the context flag without https://github.com/cloudnative-pg/barman-cloud/pull/115/files? |
Add support for authenticating to Azure via
DefaultAzureCredential.If the annotation
barmancloud.cnpg.io/useDefaultAzureCredentials="true"is detected on the ObjectStore, in addition of setting the value.spec.configuration.azureCredentials.inheritFromAzureAD: true, the barman-cloud commands will be executed to useDefaultAzureCredential.Fixes #662