Skip to content

Commit 60eb9d8

Browse files
committed
add note of caution on the config-item that enables kube-janitor in production
1 parent c009bbe commit 60eb9d8

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

cluster/config-defaults.yaml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1173,6 +1173,17 @@ sysctl_settings: ""
11731173
# kube-janitor configuration
11741174
{{if eq .Cluster.Environment "production"}}
11751175
# This makes kube-janitor opt-in for production clusters
1176+
1177+
# IMPORTANT:
1178+
# Please note that before enabling kube-janitor for a production cluster, you
1179+
# must ensure that no existing resources should be annotated with a TTL.
1180+
# This can happen in the case where a test deployment is deployed to production
1181+
# as is. Currently, it's a no-op since kube-janitor doesn't run in production.
1182+
#
1183+
# This is needed until we can implement namespace prefix matching to reduce
1184+
# the scope of kube-janitor to a set of namespace names that aren't known
1185+
# at the time of enaling kube-janitor. Once the feature is in place, it would
1186+
# be easier to limit the scope.
11761187
kube_janitor_enabled: "false"
11771188
{{else}}
11781189
kube_janitor_enabled: "true"

0 commit comments

Comments
 (0)