From 3e0ccb61a8a2608ec0c53d0e1c5e879dcbb69905 Mon Sep 17 00:00:00 2001 From: Andrei Sekretenko Date: Tue, 31 Oct 2023 14:37:08 +0100 Subject: [PATCH] fix: make dkp-insights use "uninstall" upgrade remediation startegy (#1702) fix: use "uninstall" upgrade remediation startegy for dkp-insights --- services/dkp-insights/1.0.0/helmrelease/helmrelease.yaml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/services/dkp-insights/1.0.0/helmrelease/helmrelease.yaml b/services/dkp-insights/1.0.0/helmrelease/helmrelease.yaml index 999716209..0fdda97eb 100644 --- a/services/dkp-insights/1.0.0/helmrelease/helmrelease.yaml +++ b/services/dkp-insights/1.0.0/helmrelease/helmrelease.yaml @@ -21,6 +21,14 @@ spec: upgrade: remediation: retries: 30 + # After the database migration has happened, the previous version of + # Insights will no longer work. Hence, the default `rollback` strategy + # can be worse than no remediation at all. + # On the other hand, we go to significant lengths to ensure + # that Insights work after being uninstalled and reinstalled again + # using previously persisted data (note that uninstall does not delete + # corresponding persistent volumes). + strategy: uninstall valuesFrom: - kind: ConfigMap name: dkp-insights-1.0.0-dev.3-d2iq-defaults