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.

Sign up for GitHub

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

Jump to bottom

helm update doesn't update the pods of rc's or daemonset's #1120

Closed
kmala opened this issue Aug 26, 2016 · 4 comments
Closed

helm update doesn't update the pods of rc's or daemonset's #1120

kmala opened this issue Aug 26, 2016 · 4 comments

Comments

@kmala
Copy link
Contributor

kmala commented Aug 26, 2016

when we do a helm upgrade the deployment, rc and daemon set are updated and since deployment take care of the pods to be updated they are fine but rc and daemon set pods are not upgraded.
@technosophos
Copy link
Member

technosophos commented Aug 31, 2016 •
edited
Loading

Does kubectl automatically re-deploy pods on update of RC/DS?

@kmala
Copy link
Contributor Author

kmala commented Aug 31, 2016

no..it doesn't...i think this more sort of an enhancement request than an issue.

@technosophos
Copy link
Member

This was implemented with the --restart-pods flag.

@omerlh
Copy link

omerlh commented Nov 12, 2018

With a newer version of helm, use --recreate-pods flag to achieve the same result.

MichaelMorrisEst pushed a commit to Nordix/helm that referenced this issue Nov 17, 2023
@mumoshu
feat: --detailed-exitcode for helmfile apply (helm#1120)
0186254 
Resolves helm#1113
MichaelMorrisEst pushed a commit to Nordix/helm that referenced this issue Nov 17, 2023
@jouve
resolved deps even when installed=false (helm#1120)
66935c4 
fix helm#1119
revert helm#1054

Signed-off-by: Cyril Jouve <jv.cyril@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@technosophos @omerlh @kmala