Skip to content

Commit

Permalink
fix(dka): Change default healthCheckPath to fix k8s probes (#2584)
Browse files Browse the repository at this point in the history
Previously the paths used by probes returned large responses when the
number of clusters was large (>60ish). This seems to cause the kubelet
to restart DKA pod. Changing the healthCheckPath to a static path
ensures smaller responses and is safe to do because the http server is
only started after full initialization of managed cluster endpoints.
  • Loading branch information
jimmidyson authored Aug 27, 2024
1 parent dcd636d commit 67478bd
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
kind: ConfigMap
metadata:
name: dex-k8s-authenticator-1.3.4-d2iq-defaults
name: dex-k8s-authenticator-1.3.5-d2iq-defaults
namespace: ${releaseNamespace}
data:
values.yaml: |-
Expand Down Expand Up @@ -41,3 +41,7 @@ data:
memory: 128Mi
rbac:
create: true
# Use a static file for the health check path. Using the default health check path
# returns a large response which seemingly kubelet does not like, causing pods to
# be killed unnecessarily.
healthCheckPath: /token/static/clippy.svg
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ spec:
releaseName: dex-k8s-authenticator
valuesFrom:
- kind: ConfigMap
name: dex-k8s-authenticator-1.3.4-d2iq-defaults
name: dex-k8s-authenticator-1.3.5-d2iq-defaults
- kind: ConfigMap
name: dex-k8s-authenticator-overrides
optional: true
Expand Down

0 comments on commit 67478bd

Please sign in to comment.