local_cluster_name in bootstrap.yaml.tpl needs check TopologyInjectorDisabled? #6830
Description
Description:
Describe the issue.
The serviceCluster init checked by TopologyInjectorDisabled
{{- if not .TopologyInjectorDisabled }}
- connect_timeout: 10s
eds_cluster_config:
eds_config:
ads: {}
resource_api_version: 'V3'
service_name: {{ .ServiceClusterName }}
load_balancing_policy:
policies:
- typed_extension_config:
name: 'envoy.load_balancing_policies.least_request'
typed_config:
'@type': 'type.googleapis.com/envoy.extensions.load_balancing_policies.least_request.v3.LeastRequest'
locality_lb_config:
zone_aware_lb_config:
min_cluster_size: '1'
name: {{ .ServiceClusterName }}
type: EDS
{{- end }}Does the local_cluster_name need the same logic? like follows
{{- if not .TopologyInjectorDisabled }}
cluster_manager:
local_cluster_name: {{ .ServiceClusterName }}
{{- end }}[optional Relevant Links:]
Any extra documentation required to understand the issue.