Skip to content

Commit

Permalink
examples: Update AWS ELB config
Browse files Browse the repository at this point in the history
- Remove incorrect `aws-load-balancer-internal`.
- Sets `aws-load-balancer-scheme: internet-facing`.
  This is match what our examples expect.
- Change the loadbalancer for Grafana from "CLB" (Classic) to "NLB"
  (Network)

Related:
- pingcap#3941
  • Loading branch information
dveeden committed May 14, 2021
1 parent e405d4d commit 77a54dc
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
4 changes: 2 additions & 2 deletions examples/aws/tidb-cluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,8 @@ spec:
service:
annotations:
service.beta.kubernetes.io/aws-load-balancer-cross-zone-load-balancing-enabled: 'true'
service.beta.kubernetes.io/aws-load-balancer-internal: '0.0.0.0/0'
service.beta.kubernetes.io/aws-load-balancer-type: nlb
service.beta.kubernetes.io/aws-load-balancer-type: nlb
service.beta.kubernetes.io/aws-load-balancer-scheme: internet-facing
exposeStatus: true
externalTrafficPolicy: Local
type: LoadBalancer
Expand Down
7 changes: 6 additions & 1 deletion examples/aws/tidb-monitor.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,12 @@ spec:
logLevel: info
password: admin
service:
portName: http-grafana
annotations:
service.beta.kubernetes.io/aws-load-balancer-cross-zone-load-balancing-enabled: 'true'
service.beta.kubernetes.io/aws-load-balancer-type: nlb
service.beta.kubernetes.io/aws-load-balancer-scheme: internet-facing
exposeStatus: true
externalTrafficPolicy: Local
type: LoadBalancer
username: admin
version: 6.1.6
Expand Down

0 comments on commit 77a54dc

Please sign in to comment.