Skip to content

Helm chart value to set awsAccessSecret not optional#602

Open
Fausto-R91 wants to merge 1 commit into
awslabs:mainfrom
Fausto-R91:main
Open

Helm chart value to set awsAccessSecret not optional#602
Fausto-R91 wants to merge 1 commit into
awslabs:mainfrom
Fausto-R91:main

Conversation

@Fausto-R91
Copy link
Copy Markdown

This PR introduces a new value to the Helm chart, enabling users to configure whether the AWS access secret is optional.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Fausto-R91 Fausto-R91 requested a review from a team as a code owner October 16, 2025 10:25
@muddyfish
Copy link
Copy Markdown
Contributor

Hi @Fausto-R91

We're interested in learning what the usecase for this change is, would you be able to explain more why it's needed?

@Fausto-R91
Copy link
Copy Markdown
Author

Hi,
I'm pulling secrets from an external vault (using External Secrets Operator) and I'm deploying this CSI Driver with a GitOps pipeline (managed by ArgoCD). Therefore, I cannot guarantee that the secret will be ready before the driver. With this PR merged and the value awsAccessSecret.optional set to false, the driver will fail to deploy until the secret has been created. As the cluster is not on EKS, the secret is the easiest way to pass credentials to the driver.

@muddyfish muddyfish requested a review from jet-tong April 27, 2026 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants