Skip to content

Commit

Permalink
Merge pull request jupyterhub#576 from yuvipanda/no-multi-hub
Browse files Browse the repository at this point in the history
Revert "Set maxUnavailable appropriately for hub & proxy pods"
  • Loading branch information
minrk authored Mar 15, 2018
2 parents f21eab6 + 26ed088 commit 68725f5
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 19 deletions.
13 changes: 0 additions & 13 deletions jupyterhub/templates/hub/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,19 +4,6 @@ metadata:
name: hub
spec:
replicas: 1
strategy:
rollingUpdate:
{{ if eq .Values.hub.db.type "sqlite-pvc" }}
# If we're using a PVC, set maxUnavailable to 1
# This is required, since the pod attached to PVC must die
# before new pod can run.
maxUnavailable: 1
{{ else }}
# If we aren't using a PVC, it's ok to spawn up a new pod
# before killing the currently running one. This also makes the '--wait'
# flag of helm work properly
maxUnavailable: 0
{{ end }}
template:
metadata:
labels:
Expand Down
6 changes: 0 additions & 6 deletions jupyterhub/templates/proxy/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,6 @@ metadata:
name: proxy
spec:
replicas: 1
strategy:
rollingUpdate:
# There's no PVCs attached here, so is ok to set maxUnavailable to 1
# This means new pod will be spun up and traffic transferred to it
# before old pod is shut down.
maxUnavailable: 1
template:
metadata:
annotations:
Expand Down

0 comments on commit 68725f5

Please sign in to comment.