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

feature: imagePullSecrets from string array. #576

Merged
merged 2 commits into from
Jul 23, 2021

Conversation

benashz
Copy link
Contributor

@benashz benashz commented Jul 23, 2021

  • allow configuring imagePullSecrets from an array of strings in
    addition to the already supported array of maps

@benashz benashz requested review from tomhjp, tvoran and calvn July 23, 2021 13:55
@benashz benashz force-pushed the VAULT-2829/imagePullSecrets-string-values branch from 863f0ef to 500911a Compare July 23, 2021 15:04
* allow configuring imagePullSecrets from an array of strings in
  addition to the already supported array of maps
@benashz benashz force-pushed the VAULT-2829/imagePullSecrets-string-values branch from 500911a to 483332a Compare July 23, 2021 15:07
Copy link
Contributor

@tomhjp tomhjp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@tvoran tvoran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice solution!

@tvoran
Copy link
Member

tvoran commented Jul 23, 2021

We may also want to update the comment in values.yaml to mention that an array of strings is now supported as well.

@benashz benashz merged commit 64b4d88 into master Jul 23, 2021
@benashz benashz deleted the VAULT-2829/imagePullSecrets-string-values branch July 23, 2021 16:08
@tvoran tvoran mentioned this pull request Jul 28, 2021
illegalnumbers pushed a commit to streamnative/vault-helm that referenced this pull request Mar 17, 2022
* allow configuring imagePullSecrets from an array of strings in
  addition to the already supported array of maps
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.

None yet

4 participants