Skip to content

Commit 2a36f4d

Browse files
committed
fix: get correct cluster for perview
Signed-off-by: YuTang Song <2313186065@qq.com>
1 parent 18297ed commit 2a36f4d

3 files changed

Lines changed: 13 additions & 9 deletions

File tree

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

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -387,10 +387,10 @@ func FetchCurrentAppliedYaml(option *GeneSvcYamlOption) (string, int, error) {
387387
}
388388

389389
// return structured cluster name
390-
// cluster, err := GetCluster(productInfo.ClusterID)
391-
// if err != nil {
392-
// return "", 0, errors.Wrapf(err, "failed to get cluster name by cluster id %s", productInfo.ClusterID)
393-
// }
390+
cluster, err := GetCluster(productInfo.ClusterID)
391+
if err != nil {
392+
return "", 0, errors.Wrapf(err, "failed to get cluster name by cluster id %s", productInfo.ClusterID)
393+
}
394394

395395
curProductSvc := productInfo.GetServiceMap()[option.ServiceName]
396396

@@ -419,7 +419,7 @@ func FetchCurrentAppliedYaml(option *GeneSvcYamlOption) (string, int, error) {
419419
if err != nil {
420420
return "", 0, err
421421
}
422-
fullRenderedYaml = ParseSysKeys(productInfo.Namespace, productInfo.EnvName, option.ProductName, option.ServiceName, option.ClusterName, fullRenderedYaml)
422+
fullRenderedYaml = ParseSysKeys(productInfo.Namespace, productInfo.EnvName, option.ProductName, option.ServiceName, cluster.Name, fullRenderedYaml)
423423
mergedContainers := mergeContainers(prodSvcTemplate.Containers, curProductSvc.Containers)
424424
fullRenderedYaml, _, err = ReplaceWorkloadImages(fullRenderedYaml, mergedContainers)
425425
if err != nil {
@@ -807,6 +807,12 @@ func GenerateRenderedYaml(option *GeneSvcYamlOption) (string, int, []*WorkloadRe
807807
return "", 0, nil, errors.Wrapf(err, "failed to find latest service template %s", option.ServiceName)
808808
}
809809

810+
// get cluster name by id
811+
cluster, err := GetCluster(productInfo.ClusterID)
812+
if err != nil {
813+
return "", 0, nil, errors.Wrapf(err, "failed to get cluster %s", productInfo.ClusterID)
814+
}
815+
810816
var svcContainersInProduct []*models.Container
811817
if curProductSvc != nil {
812818
svcContainersInProduct = curProductSvc.Containers
@@ -848,7 +854,7 @@ func GenerateRenderedYaml(option *GeneSvcYamlOption) (string, int, []*WorkloadRe
848854
if renderErr != nil {
849855
return "", 0, nil, fmt.Errorf("failed to render current service yaml: %v", renderErr)
850856
}
851-
currentRenderedYaml = ParseSysKeys(productInfo.Namespace, productInfo.EnvName, option.ProductName, option.ServiceName, option.ClusterName, currentRenderedYaml)
857+
currentRenderedYaml = ParseSysKeys(productInfo.Namespace, productInfo.EnvName, option.ProductName, option.ServiceName, cluster.Name, currentRenderedYaml)
852858
currentBaseReplicaMap, err = ExtractWorkloadReplicas(currentRenderedYaml)
853859
if err != nil {
854860
return "", 0, nil, fmt.Errorf("failed to extract workload replicas: %v", err)
@@ -877,7 +883,7 @@ func GenerateRenderedYaml(option *GeneSvcYamlOption) (string, int, []*WorkloadRe
877883
if err != nil {
878884
return "", 0, nil, fmt.Errorf("failed to render service yaml: %v", err)
879885
}
880-
fullRenderedYaml = ParseSysKeys(productInfo.Namespace, productInfo.EnvName, option.ProductName, option.ServiceName, option.ClusterName, fullRenderedYaml)
886+
fullRenderedYaml = ParseSysKeys(productInfo.Namespace, productInfo.EnvName, option.ProductName, option.ServiceName, cluster.Name, fullRenderedYaml)
881887

882888
// service may not be deployed in environment, we need to extract containers again, since image related variables may be changed
883889
latestSvcTemplate.KubeYamls = util.SplitYaml(fullRenderedYaml)

pkg/microservice/aslan/core/common/service/version.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,6 @@ func buildEnvServiceVersionYamlOption(env *commonmodels.Product, envSvcVersion *
136136
ProductName: env.ProductName,
137137
EnvName: envName,
138138
ServiceName: serviceName,
139-
ClusterName: clusterName,
140139
UpdateServiceRevision: false,
141140
VariableYaml: envSvcVersion.Service.GetServiceRender().GetOverrideYaml(),
142141
VariableKVs: envSvcVersion.Service.GetServiceRender().OverrideYaml.RenderVariableKVs,

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -433,7 +433,6 @@ func PreviewService(args *PreviewServiceArgs, _ *zap.SugaredLogger) (*SvcDiffRes
433433
ProductName: args.ProductName,
434434
EnvName: args.EnvName,
435435
ServiceName: args.ServiceName,
436-
ClusterName: args.ClusterName,
437436
UpdateServiceRevision: args.UpdateServiceRevision,
438437
VariableYaml: newVariableYaml,
439438
VariableKVs: args.VariableKVs,

0 commit comments

Comments
 (0)