search

eclipse-symphony / symphony

Symphony project
MIT License
34 stars 26 forks source link

Helm upgrade error was swallowed since helm install will be invoked directly after helm upgrade failed #424

Open msftcoderdjw opened 3 weeks ago

msftcoderdjw commented 3 weeks ago

https://github.com/eclipse-symphony/symphony/blob/cafed21af784e4c9e7af5b0c07ef32f54cdd377e/api/pkg/apis/v1alpha1/providers/target/helm/helm.go#L517-L518

msftcoderdjw commented 1 week ago

upgradeClient should do helm upgrade --install. Sometimes this will fail but install would work.

msftcoderdjw commented 2 days ago

Assign to @yanjiaxin534