-
Notifications
You must be signed in to change notification settings - Fork 101
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
[BUG] Temporarily enable passing of cloud provider credentials in JSON through the directory pointed by ${PROVIDER}_APPLICATION_CREDENTIALS
#729
Comments
This feature was introduced in this PR: #435, at that time backup-restore was also consumed by community via helm-charts(standalone) without etcd-druid.
It should be not removed as community might be consuming it (those who are running backup-restore standalone). We can just harmonize the both type of secrets format in backup-restore. |
/assign @renormalize |
PROVIDER_APPLICATION_CREDENTIALS
PROVIDER_APPLICATION_CREDENTIALS
${PROVIDER}_APPLICATION_CREDENTIALS
It was decided to temporarily enable passing of cloud provider credentials to Though this specific method was never supported by Only Therefore, after discussion, the maintainers have decided to deprecate passing credentials in a JSON format to This will make the both projects aligned on the way credentials are passed. Please include the intent to deprecate JSON credentials in the release notes for v0.29.0. |
Describe the bug:
Example provides 2 options to specify the backup bucket secret. Option 2 (with data as JSON) is not supported today and should be removed.
This only creates confusion for adopters. See https://kubernetes.slack.com/archives/CB57N0BFG/p1713778819927999
The text was updated successfully, but these errors were encountered: