unified logs, added check if serial connection is open to exit-handler

src/entrypoint.sh

@@ -2,7 +2,7 @@
 
 # Function to handle SIGTERM
 cleanup() {
-    echo "Container stopped, cleaning up..."
+    echo "INFO:SeplosBMS:Container stopped, cleaning up and exiting..."
     # Send SIGTERM to the Python script
     kill -s SIGTERM $!
     wait $!

src/fetch_bms_data.py

@@ -30,13 +30,12 @@ def graceful_exit(signum=None, frame=None):
             mqtt_client.loop_stop()
 
         # close serial connections if open
-        if SERIAL_MASTER_INSTANCE is not None:
+        if SERIAL_MASTER_INSTANCE is not None and SERIAL_MASTER_INSTANCE.isOpen():
             logger.info("Closing serial connection to master")
             SERIAL_MASTER_INSTANCE.close()
-        if SERIAL_SLAVES_INSTANCE is not None:
+        if SERIAL_SLAVES_INSTANCE is not None and SERIAL_SLAVES_INSTANCE.isOpen():
             logger.info("Closing serial connection to slaves")
             SERIAL_SLAVES_INSTANCE.close()
-        logger.info("Exiting the program.")
 
         if signum is not None:
             sys.exit(0)
@@ -1158,11 +1157,11 @@ def read_serial_data(self):
 
 # catch exceptions related to the initial connection to the serial port
 except serial.SerialException as e:
-    logger.error("Serial port disconnected, exiting...")
+    logger.error("Serial port disconnected, cleaning up and exiting...")
 
 # handle keyboard-interruption
 except KeyboardInterrupt:
-    logger.info("Interrupt received! Cleaning up...")
+    logger.info("Interrupt received, cleaning up and exiting...")
 
 finally:
     graceful_exit()