fix: Make MOTD logs valid

* Card ID: CCT-573 `logging` does not work with field names in strings. This patch removes those to use %-formatting, which also makes it consistent throughout the method.
RedHatInsights · Jun 20, 2024 · df4329d · df4329d
1 parent cd63055
commit df4329d
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/insights_client/__init__.py b/src/insights_client/__init__.py
@@ -376,18 +376,16 @@ def update_motd_message():
         # .registered or .unregistered exist, MOTD should not be displayed
         if os.path.lexists(MOTD_FILE):
             logger.debug(
-                ".registered or .unregistered exist; removing the MOTD file '{path}'",
+                ".registered or .unregistered exist; removing the MOTD file '%s'",
                 MOTD_FILE,
             )
             try:
                 os.remove(MOTD_FILE)
             except OSError as exc:
-                logger.debug(
-                    "could not remove the MOTD file '{path}': {exc}", MOTD_FILE, exc
-                )
+                logger.debug("could not remove the MOTD file '%s': %s", MOTD_FILE, exc)
         else:
             logger.debug(
-                ".registered or .unregistered exist; file '{motd}' correctly does not exist",
+                ".registered or .unregistered exist; file '%s' correctly does not exist",
                 MOTD_FILE,
             )