946d1e6ac0
It looks like the use of chartutil.ProcessDependencies in the HelmChart Controller was not correct, this method seems to be used in Helm only during install/upgrade. The intention was to load the dependencies but this seems to not be needed as it's already done through the loaders (loader.Load). The use of this method caused a regression where Chart.yaml files would be overwritten and registered subcharts that had aliases would be renamed using the alias name. While this is an expected behaviour of chartutil.ProcessDependencies it is not what the controller should do to the chart during (re)packaging. Signed-off-by: Aurel Canciu <aurelcanciu@gmail.com> |
||
|---|---|---|
| .. | ||
| testdata | ||
| bucket_controller.go | ||
| dependency_manager.go | ||
| dependency_manager_test.go | ||
| gitrepository_controller.go | ||
| gitrepository_controller_test.go | ||
| helmchart_controller.go | ||
| helmchart_controller_test.go | ||
| helmrepository_controller.go | ||
| helmrepository_controller_test.go | ||
| storage.go | ||
| storage_test.go | ||
| suite_test.go | ||