Skip to content
This repository has been archived by the owner on Apr 30, 2021. It is now read-only.

Commit

Permalink
Merge pull request #634 from alphagov/system-gsp-critical-priorityclass
Browse files Browse the repository at this point in the history
Create gsp-critical PriorityClass
  • Loading branch information
philandstuff authored Oct 9, 2019
2 parents 5f2ec5f + 61d17c7 commit 7c3cca3
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 2 deletions.
15 changes: 15 additions & 0 deletions charts/gsp-cluster/templates/02-gsp-system/gsp-priorityclass.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
apiVersion: scheduling.k8s.io/v1
kind: PriorityClass
metadata:
name: gsp-critical
# This PriorityClass exists because the admission controller won't let
# you use `system-cluster-critical` in non-`kube-system` namespaces,
# but we still have pods we want to run in gsp-system at high priority
# level. We create a priority class with the highest-allowed
# userspace priority value (which is still lower than
# system-cluster-critical) as a workaround.
# See also https://github.com/kubernetes/kubernetes/issues/60596
description: |
Used for GSP critical pods that must run in the cluster.
value: 1000000000

3 changes: 1 addition & 2 deletions charts/gsp-cluster/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,7 @@ cluster-autoscaler:
tag: v1.14.5 # upgrade this when upgrading kubernetes
rbac:
create: true
# we can only set this if cluster-autoscaler is in the kube-system namespace D:
# priorityClassName: system-cluster-critical
priorityClassName: gsp-critical
serviceMonitor:
enabled: true
namespace: gsp-system
Expand Down

0 comments on commit 7c3cca3

Please sign in to comment.