-
Notifications
You must be signed in to change notification settings - Fork 7k
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 a flag to execute chart hooks in parallel #7763
Comments
I'd like to take a shot at implementing this if there are no immediate objections. |
Go for it @akhilles |
Same thing as #7241 |
This issue has been marked as stale because it has been open for 90 days with no activity. This thread will be automatically closed in 30 days if no further activity occurs. |
Enables parallel execution of chart hooks. --hook-parallelism defaults to 1 to make this an opt-in feature. Batching is used to comply with "helm.sh/hook-weight" annotations. A semaphore is used to limit parallelism to the user-provided value. Some additional synchronization was necessary to run cfg.recordRelease() in a thread-safe manner. Based off of helm#7792 by akhilles. Closes helm#7763. Signed-off-by: Andrew Baehre <abaehre@morningconsult.com>
Just in case, the discussion of this issue is moved to the community proposals and here is my last comment that suggest a simple way to solve this issue. |
Currently, all chart hooks are executed serially. For some use cases (such as running
helm test
in CI environments), this is just too slow. In Helm v2, thehelm test
command supports the--parallel
and--max
flags which enable and limit parallel execution of tests, respectively. This functionality wasn't ported to Helm v3 due to test infrastructure changes.Adding a
--parallelism
flag to Helm v3 that allows for hooks to be executed in parallel (while still respecting"helm.sh/hook-weight"
annotations) would be really useful.The text was updated successfully, but these errors were encountered: