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

Add imagePullSecrets to helm chart #430

Merged
merged 5 commits into from
Nov 12, 2018
Merged

Conversation

Dean-Coakley
Copy link
Contributor

@Dean-Coakley Dean-Coakley commented Nov 8, 2018

Fixes: #415

Proposed changes

  • Add imagePullSecrets to serviceaccount template of Helm chart
  • Update Helm chart docs

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto master
  • I will ensure my PR is targeting the master branch and pulling from my branch from my own fork

@Dean-Coakley Dean-Coakley added the enhancement Pull requests for new features/feature enhancements label Nov 8, 2018
@Dean-Coakley Dean-Coakley self-assigned this Nov 8, 2018
deployments/helm-chart/README.md Outdated Show resolved Hide resolved
@pleshakov pleshakov added the change Pull requests that introduce a change label Nov 9, 2018
@pleshakov
Copy link
Contributor

Added the label "change" since the parameter controller.serviceAccountName had been changed to controller.serviceAccount.name

deployments/helm-chart/README.md Outdated Show resolved Hide resolved
pleshakov and others added 2 commits November 12, 2018 17:01
Co-Authored-By: Dean-Coakley <dean.s.coakley@gmail.com>
Co-Authored-By: Dean-Coakley <dean.s.coakley@gmail.com>
@Dean-Coakley Dean-Coakley merged commit 9da9506 into master Nov 12, 2018
@Dean-Coakley Dean-Coakley deleted the add-imagepullsecrets-helm branch November 12, 2018 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
change Pull requests that introduce a change enhancement Pull requests for new features/feature enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants