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

Can we make a variable for imagePullSecrets for flagger helm chart? #265

Closed
cmoonExpedia opened this issue Aug 1, 2019 · 4 comments · Fixed by #266
Closed

Can we make a variable for imagePullSecrets for flagger helm chart? #265

cmoonExpedia opened this issue Aug 1, 2019 · 4 comments · Fixed by #266

Comments

@cmoonExpedia
Copy link
Contributor

Hi,

Can we parameterize imagePullSecrets for https://github.com/weaveworks/flagger/blob/master/charts/flagger/templates/deployment.yaml#L40 something like below so I can use your helm chart to deploy flagger from a private repository?

https://github.com/weaveworks/flagger/blob/master/charts/flagger/values.yaml

image:
  repository: weaveworks/flagger
  tag: 0.18.1
  pullPolicy: IfNotPresent
  pullSecret:

https://github.com/weaveworks/flagger/blob/master/charts/flagger/templates/deployment.yaml

...
{{- if .Values.image.pullSecret }}
imagePullSecrets:
        - name: {{ .Values.image.pullSecret }}
{{- end }}
...

Thanks.

@stefanprodan
Copy link
Member

Sure, do you want to make a PR for this?

Thanks

@cmoonExpedia
Copy link
Contributor Author

Sure, I think I don't have a write permission to this repo. Could you give it to me?

Thanks.

@stefanprodan
Copy link
Member

stefanprodan commented Aug 1, 2019

You don't need write permission to make a PR. Fork the Flagger repo, create a branch in your clone and submit your changes as a pull request.

@stefanprodan
Copy link
Member

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 a pull request may close this issue.

2 participants