chore: move warning logs to error logs (#27728)

* chore: move warning logs to error logs

* chore: fix python linting

Author: TeddyCr

ingestion/src/metadata/clients/domo_client.py

@@ -130,7 +130,7 @@ def get_chart_details(self, page_id) -> Optional[DomoChartDetails]:
                 )
 
         except Exception as exc:
-            logger.warning(f"Error while getting details for Card {page_id} - {exc}")
+            logger.error(f"Error while getting details for Card {page_id} - {exc}")
             logger.debug(traceback.format_exc())
 
         return None
@@ -140,7 +140,7 @@ def get_pipelines(self):
             response = self.client.get(path=WORKFLOW_URL, headers=HEADERS)
             return response
         except Exception as exc:
-            logger.warning(f"Error while getting pipelines - {exc}")
+            logger.error(f"Error while getting pipelines - {exc}")
             logger.debug(traceback.format_exc())
         return []
 
@@ -164,7 +164,7 @@ def test_list_cards(self) -> None:
             self.client.get(path="content/v1/cards", headers=HEADERS)
         except Exception as exc:
             logger.debug(traceback.format_exc())
-            logger.warning(f"Error listing cards due to [{exc}]")
+            logger.error(f"Error listing cards due to [{exc}]")
             raise exc
 
 

ingestion/src/metadata/data_quality/validations/column/base/columnValueLengthsToBeBetween.py

@@ -75,7 +75,7 @@ def _run_validation(self) -> TestCaseResult:
         except (ValueError, RuntimeError) as exc:
             msg = f"Error computing {self.test_case.fullyQualifiedName}: {exc}"  # type: ignore
             logger.debug(traceback.format_exc())
-            logger.warning(msg)
+            logger.error(msg)
             return self.get_test_case_result_object(
                 self.execution_date,
                 TestCaseStatus.Aborted,

ingestion/src/metadata/data_quality/validations/column/base/columnValueMaxToBeBetween.py

@@ -68,7 +68,7 @@ def _run_validation(self) -> TestCaseResult:
         except (ValueError, RuntimeError) as exc:
             msg = f"Error computing {self.test_case.fullyQualifiedName}: {exc}"  # type: ignore
             logger.debug(traceback.format_exc())
-            logger.warning(msg)
+            logger.error(msg)
             return self.get_test_case_result_object(
                 self.execution_date,
                 TestCaseStatus.Aborted,

ingestion/src/metadata/data_quality/validations/column/base/columnValueMeanToBeBetween.py

@@ -68,7 +68,7 @@ def _run_validation(self) -> TestCaseResult:
         except (ValueError, RuntimeError) as exc:
             msg = f"Error computing {self.test_case.fullyQualifiedName}: {exc}"  # type: ignore
             logger.debug(traceback.format_exc())
-            logger.warning(msg)
+            logger.error(msg)
             return self.get_test_case_result_object(
                 self.execution_date,
                 TestCaseStatus.Aborted,

ingestion/src/metadata/data_quality/validations/column/base/columnValueMedianToBeBetween.py

@@ -68,7 +68,7 @@ def _run_validation(self) -> TestCaseResult:
         except (ValueError, RuntimeError) as exc:
             msg = f"Error computing {self.test_case.fullyQualifiedName}: {exc}"  # type: ignore
             logger.debug(traceback.format_exc())
-            logger.warning(msg)
+            logger.error(msg)
             return self.get_test_case_result_object(
                 self.execution_date,
                 TestCaseStatus.Aborted,

ingestion/src/metadata/data_quality/validations/column/base/columnValueMinToBeBetween.py

@@ -65,7 +65,7 @@ def _run_validation(self) -> TestCaseResult:
         except (ValueError, RuntimeError) as exc:
             msg = f"Error computing {self.test_case.fullyQualifiedName}: {exc}"  # type: ignore
             logger.debug(traceback.format_exc())
-            logger.warning(msg)
+            logger.error(msg)
             return self.get_test_case_result_object(
                 self.execution_date,
                 TestCaseStatus.Aborted,

ingestion/src/metadata/data_quality/validations/column/base/columnValueStdDevToBeBetween.py

@@ -70,7 +70,7 @@ def _run_validation(self) -> TestCaseResult:
         except (ValueError, RuntimeError) as exc:
             msg = f"Error computing {self.test_case.fullyQualifiedName}: {exc}"  # type: ignore
             logger.debug(traceback.format_exc())
-            logger.warning(msg)
+            logger.error(msg)
             return self.get_test_case_result_object(
                 self.execution_date,
                 TestCaseStatus.Aborted,

ingestion/src/metadata/data_quality/validations/column/base/columnValuesMissingCount.py

@@ -84,7 +84,7 @@ def _run_validation(self) -> TestCaseResult:
         except (ValueError, RuntimeError) as exc:
             msg = f"Error computing {self.test_case.fullyQualifiedName}: {exc}"  # type: ignore
             logger.debug(traceback.format_exc())
-            logger.warning(msg)
+            logger.error(msg)
             return self.get_test_case_result_object(
                 self.execution_date,
                 TestCaseStatus.Aborted,

ingestion/src/metadata/data_quality/validations/column/base/columnValuesSumToBeBetween.py

@@ -68,7 +68,7 @@ def _run_validation(self) -> TestCaseResult:
         except (ValueError, RuntimeError) as exc:
             msg = f"Error computing {self.test_case.fullyQualifiedName}: {exc}"  # type: ignore
             logger.debug(traceback.format_exc())
-            logger.warning(msg)
+            logger.error(msg)
             return self.get_test_case_result_object(
                 self.execution_date,
                 TestCaseStatus.Aborted,

ingestion/src/metadata/data_quality/validations/column/base/columnValuesToBeAtExpectedLocation.py

@@ -230,7 +230,7 @@ def _run_validation(self) -> TestCaseResult:
         except (ValueError, RuntimeError) as exc:
             msg = f"Error computing {self.test_case.fullyQualifiedName}: {exc}"  # type: ignore
             logger.debug(traceback.format_exc())
-            logger.warning(msg)
+            logger.error(msg)
             return self.get_test_case_result_object(
                 self.execution_date,
                 TestCaseStatus.Aborted,