From 2004b81253742233c166ce5c8393bb2740f0cd02 Mon Sep 17 00:00:00 2001 From: Dror Levy Date: Tue, 3 Dec 2024 09:26:44 +0200 Subject: [PATCH] fix comments --- .../log_analyzers/events_log_analyzer.py | 2 +- .../ibdiagnet2_port_counters_analyzer.py | 14 ++++++++------ .../log_analyzers/ibdiagnet_log_analyzer.py | 4 ++-- 3 files changed, 11 insertions(+), 9 deletions(-) diff --git a/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/events_log_analyzer.py b/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/events_log_analyzer.py index ad288bb9..e4a4262a 100644 --- a/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/events_log_analyzer.py +++ b/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/events_log_analyzer.py @@ -28,7 +28,7 @@ def __init__(self, logs_csvs: List[str], hours: int, dest_image_path): self.plot_critical_events_per_aggregation_time, self.plot_link_up_down_count_per_aggregation_time, self.plot_top_n_critical_events_over_time, - self.get_critical_event_bursts + self.get_critical_event_bursts, } # Function to split "object_id" into "device" and "description" diff --git a/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/ibdiagnet2_port_counters_analyzer.py b/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/ibdiagnet2_port_counters_analyzer.py index 7f26d82c..68f4131e 100644 --- a/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/ibdiagnet2_port_counters_analyzer.py +++ b/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/ibdiagnet2_port_counters_analyzer.py @@ -34,11 +34,13 @@ def __init__( self._log_data_sorted[col] = pd.to_numeric( self._log_data_sorted[col], errors="coerce" ).astype("Int64") - self._funcs_for_analysis = {self.plot_iteration_time_over_time, - self.save_last_iterations_time_stats, - self.save_first_last_iteration_timestamp, - self.save_number_of_switches_and_ports, - self.save_number_of_core_dumps} + self._funcs_for_analysis = { + self.plot_iteration_time_over_time, + self.save_last_iterations_time_stats, + self.save_first_last_iteration_timestamp, + self.save_number_of_switches_and_ports, + self.save_number_of_core_dumps, + } # Based on the log path, decided if this is primary or secondary if "ufm_logs" in logs_csvs[0]: self.telemetry_type = "primary" @@ -221,4 +223,4 @@ def save_number_of_core_dumps(self): ["Amount"], ) ) - + diff --git a/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/ibdiagnet_log_analyzer.py b/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/ibdiagnet_log_analyzer.py index 6a2bb071..1b65e196 100644 --- a/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/ibdiagnet_log_analyzer.py +++ b/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/ibdiagnet_log_analyzer.py @@ -28,7 +28,7 @@ def print_fabric_size(self): def get_fabric_size(self): return self._log_data_sorted - + def save_fabric_size(self): fabric_info = self.get_fabric_size() self._dataframes_for_pdf.append(("Fabric info", fabric_info)) @@ -36,4 +36,4 @@ def save_fabric_size(self): def full_analysis(self): super().full_analysis() self.print_fabric_size() - +