Merge pull request #545 from powerapi-ng/refactor/logging-format-string

refactor: Format logging messages the same way across the codebase

Author: gfieni

.ruff.toml

@@ -19,8 +19,6 @@ ignore = [
     "B006",   # mutable-argument-default
     "B008",   # function-call-in-default-argument
     "F401",   # unused-import
-    "G003",   # logging-string-concat
-    "G004",   # logging-f-string
     "RUF005", # collection-literal-concatenation
     "RUF013", # implicit-optional
     "RUF015", # unnecessary-iterable-allocation-for-first-element

src/powerapi/handler/handler.py

@@ -84,9 +84,9 @@ def delegate_message_handling(self, msg: Message):
             handler = self.state.get_corresponding_handler(msg)
             handler.handle_message(msg)
         except UnknownMessageTypeException:
-            self.state.actor.logger.warning("UnknowMessageTypeException: " + str(msg))
+            self.state.actor.logger.warning("UnknownMessageTypeException: %s", msg)
         except HandlerException:
-            self.state.actor.logger.warning("HandlerException")
+            self.state.actor.logger.warning("HandlerException: %s", msg)
 
 
 class InitHandler(Handler):

src/powerapi/pusher/handlers.py

@@ -91,7 +91,7 @@ def handle(self, msg):
 
             try:
                 self.state.database.save_many(self.state.buffer)
-                self.state.actor.logger.debug('save ' + str(len(self.state.buffer)) + ' reports in database')
+                self.state.actor.logger.debug('Saved %d reports in the database', len(self.state.buffer))
                 self.state.buffer = []
             except BadInputData as ex:
-                self.state.actor.logger.warning(f"The report cannot be saved: {ex.msg}")
+                self.state.actor.logger.warning('The report cannot be saved: %s', ex.msg)