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 committed Jun 17, 2020
1 parent 7e224af commit 3d8d14a
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 5 deletions.
4 changes: 3 additions & 1 deletion config/master/aws-k8s-cni-cn.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"periodSeconds": 60
"name": "aws-node"
"ports":
- "containerPort": 61678
Expand All @@ -121,7 +122,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 10
"resources":
"requests":
"cpu": "10m"
Expand Down Expand Up @@ -154,6 +155,7 @@
"name": "cni-bin-dir"
"priorityClassName": "system-node-critical"
"serviceAccountName": "aws-node"
"terminationGracePeriodSeconds": 10
"tolerations":
- "operator": "Exists"
"volumes":
Expand Down
4 changes: 3 additions & 1 deletion config/master/aws-k8s-cni-us-gov-east-1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"periodSeconds": 60
"name": "aws-node"
"ports":
- "containerPort": 61678
Expand All @@ -121,7 +122,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 10
"resources":
"requests":
"cpu": "10m"
Expand Down Expand Up @@ -154,6 +155,7 @@
"name": "cni-bin-dir"
"priorityClassName": "system-node-critical"
"serviceAccountName": "aws-node"
"terminationGracePeriodSeconds": 10
"tolerations":
- "operator": "Exists"
"volumes":
Expand Down
4 changes: 3 additions & 1 deletion config/master/aws-k8s-cni-us-gov-west-1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"periodSeconds": 60
"name": "aws-node"
"ports":
- "containerPort": 61678
Expand All @@ -121,7 +122,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 10
"resources":
"requests":
"cpu": "10m"
Expand Down Expand Up @@ -154,6 +155,7 @@
"name": "cni-bin-dir"
"priorityClassName": "system-node-critical"
"serviceAccountName": "aws-node"
"terminationGracePeriodSeconds": 10
"tolerations":
- "operator": "Exists"
"volumes":
Expand Down
4 changes: 3 additions & 1 deletion config/master/aws-k8s-cni.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"periodSeconds": 60
"name": "aws-node"
"ports":
- "containerPort": 61678
Expand All @@ -121,7 +122,7 @@
"command":
- "/app/grpc-health-probe"
- "-addr=:50051"
"initialDelaySeconds": 35
"initialDelaySeconds": 10
"resources":
"requests":
"cpu": "10m"
Expand Down Expand Up @@ -154,6 +155,7 @@
"name": "cni-bin-dir"
"priorityClassName": "system-node-critical"
"serviceAccountName": "aws-node"
"terminationGracePeriodSeconds": 10
"tolerations":
- "operator": "Exists"
"volumes":
Expand Down
9 changes: 8 additions & 1 deletion config/master/manifests.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ local awsnode = {
},
spec: {
priorityClassName: "system-node-critical",
terminationGracePeriodSeconds: 10,
affinity: {
nodeAffinity: {
requiredDuringSchedulingIgnoredDuringExecution: {
Expand Down Expand Up @@ -135,12 +136,18 @@ local awsnode = {
}],
name: "aws-node",
readinessProbe: {
exec: {
command: ["/app/grpc-health-probe", "-addr=:50051"],
},
initialDelaySeconds: 10,
},
livenessProbe: {
exec: {
command: ["/app/grpc-health-probe", "-addr=:50051"],
},
initialDelaySeconds: 35,
periodSeconds: 60,
},
livenessProbe: self.readinessProbe,
env_:: {
AWS_VPC_ENI_MTU: "9001",
AWS_VPC_K8S_CNI_CONFIGURE_RPFILTER: "false",
Expand Down

0 comments on commit 3d8d14a

Please sign in to comment.