Skip to content

Commit

Permalink
Update probe settings
Browse files Browse the repository at this point in the history
* Reduce readiness probe startup delay
* Increase liveness polling period
* Reduce shutdown grace period to 10 seconds
  • Loading branch information
Claes Mogren authored and mogren committed Jun 24, 2020
1 parent ad7df34 commit 9fea153
Show file tree
Hide file tree
Showing 5 changed files with 17 additions and 10 deletions.
5 changes: 3 additions & 2 deletions config/master/aws-k8s-cni-cn.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 60
"name": "aws-node"
"ports":
- "containerPort": 61678
Expand All @@ -128,7 +128,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 1
"resources":
"requests":
"cpu": "10m"
Expand Down Expand Up @@ -161,6 +161,7 @@
"name": "cni-bin-dir"
"priorityClassName": "system-node-critical"
"serviceAccountName": "aws-node"
"terminationGracePeriodSeconds": 10
"tolerations":
- "operator": "Exists"
"volumes":
Expand Down
5 changes: 3 additions & 2 deletions config/master/aws-k8s-cni-us-gov-east-1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 60
"name": "aws-node"
"ports":
- "containerPort": 61678
Expand All @@ -128,7 +128,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 1
"resources":
"requests":
"cpu": "10m"
Expand Down Expand Up @@ -161,6 +161,7 @@
"name": "cni-bin-dir"
"priorityClassName": "system-node-critical"
"serviceAccountName": "aws-node"
"terminationGracePeriodSeconds": 10
"tolerations":
- "operator": "Exists"
"volumes":
Expand Down
5 changes: 3 additions & 2 deletions config/master/aws-k8s-cni-us-gov-west-1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 60
"name": "aws-node"
"ports":
- "containerPort": 61678
Expand All @@ -128,7 +128,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 1
"resources":
"requests":
"cpu": "10m"
Expand Down Expand Up @@ -161,6 +161,7 @@
"name": "cni-bin-dir"
"priorityClassName": "system-node-critical"
"serviceAccountName": "aws-node"
"terminationGracePeriodSeconds": 10
"tolerations":
- "operator": "Exists"
"volumes":
Expand Down
5 changes: 3 additions & 2 deletions config/master/aws-k8s-cni.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 60
"name": "aws-node"
"ports":
- "containerPort": 61678
Expand All @@ -128,7 +128,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 1
"resources":
"requests":
"cpu": "10m"
Expand Down Expand Up @@ -161,6 +161,7 @@
"name": "cni-bin-dir"
"priorityClassName": "system-node-critical"
"serviceAccountName": "aws-node"
"terminationGracePeriodSeconds": 10
"tolerations":
- "operator": "Exists"
"volumes":
Expand Down
7 changes: 5 additions & 2 deletions config/master/manifests.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,7 @@ local awsnode = {
},
spec: {
priorityClassName: "system-node-critical",
terminationGracePeriodSeconds: 10,
affinity: {
nodeAffinity: {
requiredDuringSchedulingIgnoredDuringExecution: {
Expand Down Expand Up @@ -143,9 +144,11 @@ local awsnode = {
exec: {
command: ["/app/grpc-health-probe", "-addr=:50051"],
},
initialDelaySeconds: 35,
initialDelaySeconds: 1,
},
livenessProbe: self.readinessProbe + {
initialDelaySeconds: 60,
},
livenessProbe: self.readinessProbe,
env_:: {
AWS_VPC_ENI_MTU: "9001",
AWS_VPC_K8S_CNI_CONFIGURE_RPFILTER: "false",
Expand Down

0 comments on commit 9fea153

Please sign in to comment.