From f173b28fbeb05f535a233ec6eb2e8bedd1386af1 Mon Sep 17 00:00:00 2001 From: Joshua Reed Date: Wed, 24 Apr 2024 14:08:52 -0600 Subject: [PATCH] Change is_chart_enabled log output to warnings. In both exception cases, the function return False and do not thrown an error. Therefore the previous logging as an exception gives a misleading error. Instead, log as a warning. Story: 2010929 Task: 49968 Change-Id: Idfde4a18375ed1746ef139e2d4ae0f4c0342a1bd Signed-off-by: Joshua Reed --- sysinv/sysinv/sysinv/sysinv/common/utils.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sysinv/sysinv/sysinv/sysinv/common/utils.py b/sysinv/sysinv/sysinv/sysinv/common/utils.py index d353dfa32c..b55a66dd38 100644 --- a/sysinv/sysinv/sysinv/sysinv/common/utils.py +++ b/sysinv/sysinv/sysinv/sysinv/common/utils.py @@ -2508,10 +2508,10 @@ def is_chart_enabled(dbapi, app_name, chart_name, namespace): db_app = find_kube_app(dbapi, app_name) db_chart = dbapi.helm_override_get(db_app.id, chart_name, namespace) except exception.KubeAppNotFound: - LOG.exception("is_chart_enabled: %s application unknown" % (app_name)) + LOG.warning("is_chart_enabled: %s application unknown" % (app_name)) return False except exception.HelmOverrideNotFound: - LOG.exception("is_chart_enabled: %s/%s/%s overrides missing" % ( + LOG.warning("is_chart_enabled: %s/%s/%s overrides missing" % ( app_name, chart_name, namespace)) return False