[chores] Log non-zero exit code warnings

openwisp · Jun 24, 2022 · 4bf1fd9 · 4bf1fd9
1 parent 07aa783
commit 4bf1fd9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/openwisp_monitoring/check/classes/iperf.py b/openwisp_monitoring/check/classes/iperf.py
@@ -27,7 +27,6 @@ def check(self, store=True):
                 f'DeviceConnection is not properly set for "{device}", iperf check skipped!'
             )
             return
-        device_connection.connect()
         # The DeviceConnection could fail if the management tunnel is down.
         if not device_connection.connect():
             logger.warning(
@@ -40,6 +39,9 @@ def check(self, store=True):
             command, raise_unexpected_exit=False
         )
         if store and exit_code != 0:
+            logger.warning(
+                f'Iperf check failed for "{device}", {json.loads(res)["error"]}'
+            )
             self.store_result_fail()
             device_connection.disconnect()
             return
@@ -51,6 +53,9 @@ def check(self, store=True):
                 command, raise_unexpected_exit=False
             )
             if store and exit_code != 0:
+                logger.warning(
+                    f'Iperf check failed for "{device}", {json.loads(res)["error"]}'
+                )
                 self.store_result_fail()
                 device_connection.disconnect()
                 return

diff --git a/openwisp_monitoring/monitoring/configuration.py b/openwisp_monitoring/monitoring/configuration.py
@@ -586,7 +586,7 @@ def _get_access_tech():
                 'summary_labels': [
                     _('Received bytes'),
                     _('Sent bytes'),
-                    _('Total upload traffic'),
+                    _('Total bytes'),
                 ],
                 'unit': _(' GB'),
                 'order': 290,