Skip to content
This repository has been archived by the owner on Jan 22, 2021. It is now read-only.

allow concurrency #53

Closed
Morriz opened this issue Jun 13, 2019 · 8 comments
Closed

allow concurrency #53

Morriz opened this issue Jun 13, 2019 · 8 comments

Comments

@Morriz
Copy link

Morriz commented Jun 13, 2019

It is currently not possible to run concurrent helm tiller run -- helm ... commands. Any idea if that will be possible in the future? Right now I am using helmfile which can't deploy multiple charts simultaneously, nor delete them concurrently.

@rimusz
Copy link
Owner

rimusz commented Jun 13, 2019

I would recommend to run concurrent helm tiller run -- helm ... commands in docker containers, as making it in the plugin is out of the scope.

@Morriz
Copy link
Author

Morriz commented Jun 26, 2019

Does that address my reported issue about concurrency? Are you suggesting multiple spun up docker containers executing commands concurrently is possible without conflicts? Then I can suggest the author of helmfile @mumoshu to do just that.

@rimusz
Copy link
Owner

rimusz commented Jun 26, 2019

if you don't do multiple releases from at the same time for the same helm release, everything will be fine

@Morriz
Copy link
Author

Morriz commented Jun 26, 2019

great...that means concurrency is possible...why not from the plugin? why is that out of scope?

@rimusz
Copy link
Owner

rimusz commented Jun 26, 2019

as you need to run helm + pluggin per container

@rustycl0ck
Copy link

Isn't it because helm tiller run -- helm ... always uses a constant port for running tiller? If each helm tiller run would run tiller on a random port, then this problem would be solved I think. I can't even run helm tiller start-ci && helmfile apply in one window and helm tiller run -- helm status my-app in another window because the helm tiller run kills the start-ci server as well at the time of exit.

@rimusz
Copy link
Owner

rimusz commented Sep 25, 2019

as I mentioned before to get to concurrency run helm+tiller in the container.

Helm v3 is under way (very well back compatible with v2) and beta is already available that will make this plugin obsolete soon.

@rimusz
Copy link
Owner

rimusz commented Oct 24, 2019

closing this one, as Helm v3 does not have such issues

@rimusz rimusz closed this as completed Oct 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants