Fixing a small bug in rest api logs + removing echo from ci

Mellanox · Sep 2, 2024 · dd118cc · dd118cc
1 parent dd77672
commit dd118cc
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/rest_api_log_analyzer.py b/plugins/ufm_log_analyzer_plugin/src/loganalyze/log_analyzers/rest_api_log_analyzer.py
@@ -22,11 +22,12 @@ def __init__(self, logs_csvs: List[str], hours: int, dest_image_path: str, sort_
         self._log_data_sorted = self._log_data_sorted.loc\
             [self._log_data_sorted['user'] != 'ufmsystem']
         #Splitting the URL for better analysis
-        self._log_data_sorted[['uri', 'query_params']] = self._log_data_sorted['url']\
-            .apply(self.split_url_to_uri_and_query_params).apply(pd.Series)
-        self._have_duration = self._have_data_in_column('duration')
-        self._have_user = self._have_data_in_column('user')
-        self._funcs_for_analysis = {self.analyze_endpoints_freq}
+        if not self._log_data_sorted.empty:
+            self._log_data_sorted[['uri', 'query_params']] = self._log_data_sorted['url']\
+                .apply(self.split_url_to_uri_and_query_params).apply(pd.Series)
+            self._have_duration = self._have_data_in_column('duration')
+            self._have_user = self._have_data_in_column('user')
+            self._funcs_for_analysis = {self.analyze_endpoints_freq}
 
     @staticmethod
     def split_url_to_uri_and_query_params(url):