starlingx
/
config
Code Issues Proposed changes

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 <joshua.reed@windriver.com>
This commit is contained in:
Joshua Reed 2024-04-24 14:08:52 -06:00
parent 5ca9c6f78f
commit f173b28fbe
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
sysinv/sysinv/sysinv/sysinv/common/utils.py
View File

@ -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