starlingx/portieris-armada-app
Code Issues Proposed changes

Adjust liveness/readiness probe configurations

Browse Source 
Adding support for configuring liveness and readiness probe times, and
changing the values from initial settings to lower the CPU usage. The
new values are configured in the static override file.

Test Plan:
PASS    Validate new values for probes
PASS    Application sanity test

Story: 2011086
Task: 50946

Change-Id: I2e65237f2ed8e6703a11155aa60f5c24a8d14222
Signed-off-by: Tae Park <tae.park@windriver.com>
This commit is contained in:
Tae Park 2024-08-30 15:51:40 -04:00
parent 476872b1b7
commit e15a0dad94
3 changed files with 73 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

64
helm-charts/upstream/portieris-helm/debian/deb_folder/patches/0003-Adjust-liveness-readiness-probe-configurations.patch Normal file
View File

@ -0,0 +1,64 @@
From 3fc69db274f3d0b5480fc9cc409ddeca21ad0b73 Mon Sep 17 00:00:00 2001
From: Tae Park <tae.park@windriver.com>
Date: Fri, 30 Aug 2024 15:07:37 -0400
Subject: [PATCH] Adjust liveness/readiness probe configurations
Adding support for configuring liveness and readiness probe times, and
changing the values from initial settings to lower the CPU usage. The
new values are configured in the static override file.
Signed-off-by: Tae Park <tae.park@windriver.com>
---
helm/portieris/templates/deployment.yaml | 10 ++++++----
helm/portieris/values.yaml | 12 ++++++++++++
2 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/helm/portieris/templates/deployment.yaml b/helm/portieris/templates/deployment.yaml
index 826fdba..5c38dd9 100644
--- a/helm/portieris/templates/deployment.yaml
+++ b/helm/portieris/templates/deployment.yaml
@@ -56,15 +56,17 @@ spec:
port: 8000
path: "/health/liveness"
scheme: HTTPS
- initialDelaySeconds: 10
- timeoutSeconds: 10
+ initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds }}
+ timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds }}
+ periodSeconds: {{ .Values.livenessProbe.periodSeconds }}
readinessProbe:
httpGet:
port: 8000
path: "/health/readiness"
scheme: HTTPS
- initialDelaySeconds: 10
- timeoutSeconds: 10
+ initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds }}
+ timeoutSeconds: {{ .Values.readinessProbe.timeoutSeconds }}
+ periodSeconds: {{ .Values.readinessProbe.periodSeconds }}
env:
resources:
{{ toYaml .Values.resources | indent 12 }}
diff --git a/helm/portieris/values.yaml b/helm/portieris/values.yaml
index b4be5dd..3e1fb20 100644
--- a/helm/portieris/values.yaml
+++ b/helm/portieris/values.yaml
@@ -135,3 +135,15 @@ podDisruptionBudget:
# Service selector includes release in addition to app
selectByRelease: true
+
+# Default values for liveness probe
+livenessProbe:
+ initialDelaySeconds: 17
+ periodSeconds: 19
+ timeoutSeconds: 11
+
+# Default values for readiness probe
+readinessProbe:
+ initialDelaySeconds: 17
+ periodSeconds: 19
+ timeoutSeconds: 11
--
2.25.1

1
helm-charts/upstream/portieris-helm/debian/deb_folder/patches/series
View File

@ -1,2 +1,3 @@
0001-Add-permissive-cluster-policy-by-default.patch
0002-Hardcode-app-label-selector.patch
0003-Adjust-liveness-readiness-probe-configurations.patch

8
stx-portieris-helm/stx-portieris-helm/fluxcd-manifests/portieris/portieris-static-overrides.yaml
View File

@ -21,5 +21,13 @@ tolerations:
- key: "node-role.kubernetes.io/control-plane"
operator: "Exists"
effect: "NoSchedule"
livenessProbe:
initialDelaySeconds: 17
periodSeconds: 19
timeoutSeconds: 11
readinessProbe:
initialDelaySeconds: 17
periodSeconds: 19
timeoutSeconds: 11
SkipSecretCreation: true
UseCertManager: true