Skip to content

Commit

Permalink
update proper default values 1 sec is too low for liveness (#267)
Browse files Browse the repository at this point in the history
  • Loading branch information
harshavardhana committed Aug 26, 2020
1 parent 4612151 commit cbd1648
Show file tree
Hide file tree
Showing 10 changed files with 23 additions and 23 deletions.
8 changes: 4 additions & 4 deletions examples/tenant-console.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ spec:
prometheus.io/port: "9000"
prometheus.io/scrape: "true"
## Registry location and Tag to download MinIO Server image
image: minio/minio:RELEASE.2020-08-18T19-41-00Z
image: minio/minio:RELEASE.2020-08-26T00-00-49Z
imagePullPolicy: IfNotPresent
## Secret with credentials to be used by MinIO instance.
credsSecret:
Expand Down Expand Up @@ -121,9 +121,9 @@ spec:
## is not working properly and needs restart. Kubernetes automatically
## restarts the pods if liveness checks fail.
liveness:
initialDelaySeconds: 10
periodSeconds: 1
timeoutSeconds: 1
initialDelaySeconds: 30
periodSeconds: 10
timeoutSeconds: 10

## nodeSelector parameters for MinIO Pods. It specifies a map of key-value pairs. For the pod to be
## eligible to run on a node, the node must have each of the
Expand Down
8 changes: 4 additions & 4 deletions examples/tenant-kes.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ spec:
prometheus.io/port: "9000"
prometheus.io/scrape: "true"
## Registry location and Tag to download MinIO Server image
image: minio/minio:RELEASE.2020-08-18T19-41-00Z
image: minio/minio:RELEASE.2020-08-26T00-00-49Z
imagePullPolicy: IfNotPresent
## Secret with credentials to be used by MinIO instance.
credsSecret:
Expand Down Expand Up @@ -136,9 +136,9 @@ spec:
## is not working properly and needs restart. Kubernetes automatically
## restarts the pods if liveness checks fail.
liveness:
initialDelaySeconds: 10
periodSeconds: 1
timeoutSeconds: 1
initialDelaySeconds: 30
periodSeconds: 10
timeoutSeconds: 10

## nodeSelector parameters for MinIO Pods. It specifies a map of key-value pairs. For the pod to be
## eligible to run on a node, the node must have each of the
Expand Down
6 changes: 3 additions & 3 deletions examples/tenant-pod-security-policy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ spec:
prometheus.io/port: "9000"
prometheus.io/scrape: "true"
## Registry location and Tag to download MinIO Server image
image: minio/minio:RELEASE.2020-08-18T19-41-00Z
image: minio/minio:RELEASE.2020-08-26T00-00-49Z
## Service account to be used for all the MinIO Pods
serviceAccountName: minio-pods
zones:
Expand Down Expand Up @@ -144,8 +144,8 @@ spec:
## is not working properly and needs restart. Kubernetes automatically
## restarts the pods if liveness checks fail.
liveness:
initialDelaySeconds: 10
periodSeconds: 1
initialDelaySeconds: 30
periodSeconds: 10
## nodeSelector parameters for MinIO Pods. It specifies a map of key-value pairs. For the pod to be
## eligible to run on a node, the node must have each of the
## indicated key-value pairs as labels.
Expand Down
8 changes: 4 additions & 4 deletions examples/tenant.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ spec:
prometheus.io/scrape: "true"

## Registry location and Tag to download MinIO Server image
image: minio/minio:RELEASE.2020-08-18T19-41-00Z
image: minio/minio:RELEASE.2020-08-26T00-00-49Z
imagePullPolicy: IfNotPresent
zones:
- servers: 4
Expand Down Expand Up @@ -101,9 +101,9 @@ spec:
## is not working properly and needs restart. Kubernetes automatically
## restarts the pods if liveness checks fail.
liveness:
initialDelaySeconds: 10
periodSeconds: 1
timeoutSeconds: 1
initialDelaySeconds: 30
periodSeconds: 10
timeoutSeconds: 10

## nodeSelector parameters for MinIO Pods. It specifies a map of key-value pairs. For the pod to be
## eligible to run on a node, the node must have each of the
Expand Down
2 changes: 1 addition & 1 deletion kubectl-minio/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ example: `kubectl minio tenant create --name tenant1 --secret cred-secret --serv
Options:

- `--namespace=minio`
- `--image=minio/minio:RELEASE.2020-08-18T19-41-00Z`
- `--image=minio/minio:RELEASE.2020-08-26T00-00-49Z`
- `--storageClass=local`
- `--kms-secret=secret-name`
- `--console-secret=secret-name`
Expand Down
2 changes: 1 addition & 1 deletion kubectl-minio/cmd/helpers/constants.go
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ const (
DefaultOperatorImage = "minio/k8s-operator:v3.0.13"

// DefaultTenantImage is the default MinIO image used while creating tenant
DefaultTenantImage = "minio/minio:RELEASE.2020-08-18T19-41-00Z"
DefaultTenantImage = "minio/minio:RELEASE.2020-08-26T00-00-49Z"

// DefaultKESImage is the default KES image used while creating tenant
DefaultKESImage = "minio/kes:v0.11.0"
Expand Down
2 changes: 1 addition & 1 deletion kubectl-minio/cmd/tenant-upgrade.go
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ import (
const (
upgradeDesc = `
'upgrade' command upgrades a MinIO tenant to the specified MinIO version`
upgradeExample = ` kubectl minio tenant upgrade --name tenant1 --image minio/minio:RELEASE.2020-08-18T19-41-00Z --namespace tenant1-ns`
upgradeExample = ` kubectl minio tenant upgrade --name tenant1 --image minio/minio:RELEASE.2020-08-26T00-00-49Z --namespace tenant1-ns`
)

type upgradeCmd struct {
Expand Down
6 changes: 3 additions & 3 deletions pkg/apis/minio.min.io/v1/constants.go
Original file line number Diff line number Diff line change
Expand Up @@ -96,13 +96,13 @@ const DefaultZoneName = "zone-0"
const LivenessPath = "/minio/health/live"

// LivenessInitialDelay specifies the initial delay in calling the liveness endpoint
const LivenessInitialDelay = 10
const LivenessInitialDelay = 30 // initial delay of upto 30 seconds

// LivenessPeriod specifies the interval in calling the liveness endpoint
const LivenessPeriod = 1
const LivenessPeriod = 10 // once in 10 seconds

// LivenessTimeout specifies the timeout for the liveness probe to expect a response
const LivenessTimeout = 1
const LivenessTimeout = 10 // once in 10 seconds

// LivenessFailureThreshold specifies the number of times the liveness probe can fail before
// the probe to be considered failed.
Expand Down
2 changes: 1 addition & 1 deletion testing/patching/upgrade-one-zone-step-2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ kind: Tenant
metadata:
name: upgrade-one-zone
spec:
image: minio/minio:RELEASE.2020-08-18T19-41-00Z
image: minio/minio:RELEASE.2020-08-26T00-00-49Z
zones:
- servers: 1
volumesPerServer: 4
Expand Down
2 changes: 1 addition & 1 deletion testing/patching/upgrade-two-zones-step-2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ kind: Tenant
metadata:
name: upgrade-two-zones
spec:
image: minio/minio:RELEASE.2020-08-18T19-41-00Z
image: minio/minio:RELEASE.2020-08-26T00-00-49Z
zones:
- servers: 4
volumesPerServer: 4
Expand Down

0 comments on commit cbd1648

Please sign in to comment.