fix helm chart preivew (#4608)

Signed-off-by: Patrick Zhao <zhaoyu@koderover.com>

Author: PetrusZ

pkg/microservice/aslan/core/common/service/kube/helm.go

@@ -688,6 +688,7 @@ func DownloadInstantiateChart(projectName, chartRepo, chartName, chartVersion, r
 	if err != nil {
 		return err
 	}
+
 	err = hClient.DownloadChart(commonutil.GeneHelmRepo(chartRepoEntry), chartRef, chartVersion, localPath, true)
 	if err != nil {
 		return fmt.Errorf("failed to download chart, chartName: %s, chartRepo: %+v, err: %s", chartName, chartRepo, err)
@@ -948,9 +949,9 @@ func GetHelmChartManifest(env *commonmodels.Product, service *commonmodels.Servi
 	releaseName := util.GeneReleaseName(service.GetReleaseNaming(), env.ProductName, env.Namespace, env.EnvName, service.ServiceName)
 	chartSpec := &helmclient.ChartSpec{
 		ReleaseName: releaseName,
-		ChartName:    chartPath,
-		Version:      chartVersion,
-		ValuesYaml:   valuesYaml,
+		ChartName:   chartPath,
+		Version:     chartVersion,
+		ValuesYaml:  valuesYaml,
 	}
 
 	var templateOptions *helmclient.HelmTemplateOptions

pkg/microservice/aslan/core/common/util/helm.go

@@ -363,6 +363,8 @@ func NewHelmClient(chartRepo *commonmodels.HelmRepo) (*helmtool.HelmClient, erro
 			return nil, fmt.Errorf("enabled proxy for helm client, but no proxy found")
 		}
 
+		log.Debugf("add proxy to helm client, proxy url: %s", proxy[0].GetProxyURL())
+
 		transport, err := util.NewTransport(chartRepo.URL, "", "", "", false, proxy[0].GetProxyURL())
 		if err != nil {
 			return nil, fmt.Errorf("failed to new transport, err: %s", err)

pkg/microservice/aslan/core/environment/service/environment.go

@@ -536,10 +536,10 @@ func generateMobileCustomWorkflow(projectName, workflowName string, focalBasicIm
 }
 
 type UpdateServiceArg struct {
-	ServiceName     string                          `json:"service_name"`
-	DeployStrategy  string                          `json:"deploy_strategy"`
-	VariableKVs     []*commontypes.RenderVariableKV `json:"variable_kvs"`
-	OverrideResource bool                           `json:"override_resource"`
+	ServiceName      string                          `json:"service_name"`
+	DeployStrategy   string                          `json:"deploy_strategy"`
+	VariableKVs      []*commontypes.RenderVariableKV `json:"variable_kvs"`
+	OverrideResource bool                            `json:"override_resource"`
 }
 
 type UpdateEnv struct {
@@ -1630,7 +1630,7 @@ func GenEstimatedValues(projectName, envName, namespace, serviceOrReleaseName st
 			if err != nil {
 				return nil, fmt.Errorf("failed to query chart-repo info, repoName: %s", render.ChartRepo)
 			}
-			client, err := helmtool.NewClientFromNamespace(prod.ClusterID, prod.Namespace)
+			client, err := commonutil.NewHelmClient(chartRepo)
 			if err != nil {
 				return nil, fmt.Errorf("failed to new helm client, err %s", err)
 			}
@@ -1704,7 +1704,8 @@ func GenEstimatedValues(projectName, envName, namespace, serviceOrReleaseName st
 		if err != nil {
 			return nil, fmt.Errorf("failed to query chart-repo info, repoName: %s", arg.ChartRepo)
 		}
-		client, err := helmtool.NewClientFromNamespace(prod.ClusterID, prod.Namespace)
+
+		client, err := commonutil.NewHelmClient(chartRepo)
 		if err != nil {
 			return nil, fmt.Errorf("failed to new helm client, err %s", err)
 		}