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

Release 3.14 fork prepare #1

Open

StiplinOV wants to merge 5 commits into release-3.14-fork from release-3.14-fork-prepare

Collaborator

StiplinOV commented Jan 15, 2025

тут доработки форкнутого хелм репозитория для использования в сервисе деплоя.

Переменная settings = cli.New() не глобальная (из за этого при параллельной установки чартов могут путаться values)
Добавлена внешняя ручка ExecuteCmd. Она позволяет клиенту вызывать ее также как команду helm install/uninstall/...
Добавлена защита от вызова RegisterFlagCompletionFunc. Она порождала утечки памяти
Добавлено закрытие каналов сигналов interrupt/terminate. Если из не закрывать то они копятся
Все шаги разбиты по комитам

Oleg Stiplin added 5 commits

January 15, 2025 16:27


          extract settings (cli.New()) from global variable

96dbb44


          add ExecuteCmd lever

4b28f19


          don't invoke RegisterFlagCompletionFunc multiple times. It leads to m…

bb7b91d

…emory leaks


          close install & upgrade interrupt & terminate signal for preventing m…

e45e481

…emory leaks


          helm/v3 -> helm

76fe3c6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet