Skip to content
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

support setting of imagePullSecrets for all controllers #53 #57

Merged
merged 1 commit into from
Dec 21, 2021

Conversation

smurfralf
Copy link

Example helm cli usage to provide an imagePullSecret

helm ... --set "imagePullSecrets[0].name=secret1"

Signed-off-by: Robert R Allen [email protected]

What this PR does / why we need it:

The flux CLI supports customizing the imagePullSecrets. This brings equivalent behavior to the helm chart.

Which issue this PR fixes

Special notes for your reviewer:

Checklist

  • DCO signed
  • Chart Version bumped
  • helm-docs are updated
  • Helm chart is tested

…ity#53

Example helm cli usage to provide an imagePullSecret

```
helm ... --set "imagePullSecrets[0].name=secret1"
```

Signed-off-by: Robert R Allen <[email protected]>
@smurfralf
Copy link
Author

reopen after rebase

@smurfralf smurfralf reopened this Dec 20, 2021
@smurfralf
Copy link
Author

smurfralf commented Dec 20, 2021

@haarchri It looks like you need to approve again since I cleared content of MR to rebase

@haarchri haarchri closed this Dec 21, 2021
@haarchri haarchri reopened this Dec 21, 2021
@dwerder
Copy link
Collaborator

dwerder commented Dec 21, 2021

LGTM. Thanks for the effort.

@dwerder dwerder merged commit e42ed0b into fluxcd-community:main Dec 21, 2021
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.

3 participants