refactor: refine helm package

1. Rename helm.Manager to helm.Loader;

2. Add 'pkg/helm/values*" to make values operation more flexiable;

3. Update unit test cases;

Signed-off-by: zyy17 <zyylsxm@gmail.com>

pkg/deployer/k8s/deployer.go

@@ -30,11 +30,11 @@ import (
 )
 
 type deployer struct {
-	helmManager *helm.Manager
-	client      *kube.Client
-	timeout     time.Duration
-	logger      logger.Logger
-	dryRun      bool
+	helmLoader *helm.Loader
+	client     *kube.Client
+	timeout    time.Duration
+	logger     logger.Logger
+	dryRun     bool
 }
 
 const (
@@ -46,14 +46,14 @@ var _ Interface = &deployer{}
 type Option func(*deployer)
 
 func NewDeployer(l logger.Logger, opts ...Option) (Interface, error) {
-	hm, err := helm.NewManager(l)
+	hc, err := helm.NewLoader(l)
 	if err != nil {
 		return nil, err
 	}
 
 	d := &deployer{
-		helmManager: hm,
-		logger:      l,
+		helmLoader: hc,
+		logger:     l,
 	}
 
 	for _, opt := range opts {
@@ -127,7 +127,16 @@ func (d *deployer) CreateGreptimeDBCluster(ctx context.Context, name string, opt
 		options.ConfigValues += fmt.Sprintf("image.registry=%s,initializer.registry=%s,", AliCloudRegistry, AliCloudRegistry)
 	}
 
-	manifests, err := d.helmManager.LoadAndRenderChart(ctx, resourceName, resourceNamespace, artifacts.GreptimeDBChartName, options.GreptimeDBChartVersion, options.UseGreptimeCNArtifacts, *options)
+	opts := &helm.LoadOptions{
+		ReleaseName:   resourceName,
+		Namespace:     resourceNamespace,
+		ChartName:     artifacts.GreptimeDBChartName,
+		ChartVersion:  options.GreptimeDBChartVersion,
+		FromCNRegion:  options.UseGreptimeCNArtifacts,
+		ValuesOptions: *options,
+		EnableCache:   true,
+	}
+	manifests, err := d.helmLoader.LoadAndRenderChart(ctx, opts)
 	if err != nil {
 		return err
 	}
@@ -186,7 +195,16 @@ func (d *deployer) CreateEtcdCluster(ctx context.Context, name string, options *
 		options.ConfigValues += fmt.Sprintf("image.registry=%s,", AliCloudRegistry)
 	}
 
-	manifests, err := d.helmManager.LoadAndRenderChart(ctx, resourceName, resourceNamespace, artifacts.EtcdChartName, artifacts.DefaultEtcdChartVersion, options.UseGreptimeCNArtifacts, *options)
+	opts := &helm.LoadOptions{
+		ReleaseName:   resourceName,
+		Namespace:     resourceNamespace,
+		ChartName:     artifacts.EtcdChartName,
+		ChartVersion:  artifacts.DefaultEtcdChartVersion,
+		FromCNRegion:  options.UseGreptimeCNArtifacts,
+		ValuesOptions: *options,
+		EnableCache:   true,
+	}
+	manifests, err := d.helmLoader.LoadAndRenderChart(ctx, opts)
 	if err != nil {
 		return fmt.Errorf("error while loading helm chart: %v", err)
 	}
@@ -222,7 +240,16 @@ func (d *deployer) CreateGreptimeDBOperator(ctx context.Context, name string, op
 		options.ConfigValues += fmt.Sprintf("image.registry=%s,", AliCloudRegistry)
 	}
 
-	manifests, err := d.helmManager.LoadAndRenderChart(ctx, resourceName, resourceNamespace, artifacts.GreptimeDBOperatorChartName, options.GreptimeDBOperatorChartVersion, options.UseGreptimeCNArtifacts, *options)
+	opts := &helm.LoadOptions{
+		ReleaseName:   resourceName,
+		Namespace:     resourceNamespace,
+		ChartName:     artifacts.GreptimeDBOperatorChartName,
+		ChartVersion:  options.GreptimeDBOperatorChartVersion,
+		FromCNRegion:  options.UseGreptimeCNArtifacts,
+		ValuesOptions: *options,
+		EnableCache:   true,
+	}
+	manifests, err := d.helmLoader.LoadAndRenderChart(ctx, opts)
 	if err != nil {
 		return err
 	}