Skip to content

Commit 95cc64c

Browse files
authored
Load balancer doesn't obtain the endpoint when too many nodes are added on cluster-up (#2338)
1 parent 0ba1d96 commit 95cc64c

File tree

2 files changed

+9
-13
lines changed

2 files changed

+9
-13
lines changed

manager/manifests/istio.yaml.j2

-12
Original file line numberDiff line numberDiff line change
@@ -55,18 +55,12 @@ spec:
5555
app: operator-istio-gateway
5656
istio: ingressgateway-operator
5757
ports:
58-
- name: status-port # should be first in the list, see https://github.com/istio/istio/issues/12503
59-
port: 15021
60-
targetPort: 15021
6158
- name: http2
6259
port: 80
6360
targetPort: 80
6461
- name: https
6562
port: 443
6663
targetPort: 443
67-
- name: tls # used for SNI
68-
port: 15443
69-
targetPort: 15443
7064
resources:
7165
requests:
7266
cpu: 100m
@@ -114,18 +108,12 @@ spec:
114108
app: apis-istio-gateway
115109
istio: ingressgateway-apis
116110
ports:
117-
- name: status-port # should be first in the list, see https://github.com/istio/istio/issues/12503
118-
port: 15021
119-
targetPort: 15021
120111
- name: http2
121112
port: 80
122113
targetPort: 80
123114
- name: https
124115
port: 443
125116
targetPort: 443
126-
- name: tls # used for SNI
127-
port: 15443
128-
targetPort: 15443
129117
resources:
130118
requests:
131119
cpu: 400m

pkg/types/clusterconfig/cluster_config.go

+9-1
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,16 @@ import (
4848
const (
4949
// MaxNodeGroups represents the max number of node groups in a cluster
5050
MaxNodeGroups = 100
51+
5152
// MaxNodesToAddOnClusterUp represents the max number of nodes to add on cluster up
52-
MaxNodesToAddOnClusterUp = 250
53+
// Limited to 200 nodes (rounded down from 248 nodes) for two reasons:
54+
//
55+
// * To prevent overloading the API servers when the nodes are being added.
56+
//
57+
// * To prevent hitting the 500 targets per LB (when the cross-load balancing is enabled) limit (quota code L-B211E961);
58+
// 500 divided by 2 target listeners - 1 operator node - 1 prometheus node => 248
59+
MaxNodesToAddOnClusterUp = 200
60+
5361
// MaxNodesToAddOnClusterConfigure represents the max number of nodes to add on cluster up/configure
5462
MaxNodesToAddOnClusterConfigure = 100
5563
// ClusterNameTag is the tag used for storing a cluster's name in AWS resources

0 commit comments

Comments
 (0)