This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new a747aecea2 fix: try002 for provider alibaba (#38787)
a747aecea2 is described below
commit a747aecea2582499e56e24292f033bae7b04e5d0
Author: Sebastian Daum <[email protected]>
AuthorDate: Sat Apr 6 16:18:49 2024 +0200
fix: try002 for provider alibaba (#38787)
---
airflow/providers/alibaba/cloud/hooks/oss.py | 14 +++++++-------
pyproject.toml | 2 --
2 files changed, 7 insertions(+), 9 deletions(-)
diff --git a/airflow/providers/alibaba/cloud/hooks/oss.py
b/airflow/providers/alibaba/cloud/hooks/oss.py
index cf0e20127d..59b68bd037 100644
--- a/airflow/providers/alibaba/cloud/hooks/oss.py
+++ b/airflow/providers/alibaba/cloud/hooks/oss.py
@@ -338,17 +338,17 @@ class OSSHook(BaseHook):
extra_config = self.oss_conn.extra_dejson
auth_type = extra_config.get("auth_type", None)
if not auth_type:
- raise Exception("No auth_type specified in extra_config. ")
+ raise ValueError("No auth_type specified in extra_config. ")
if auth_type != "AK":
- raise Exception(f"Unsupported auth_type: {auth_type}")
+ raise ValueError(f"Unsupported auth_type: {auth_type}")
oss_access_key_id = extra_config.get("access_key_id", None)
oss_access_key_secret = extra_config.get("access_key_secret", None)
if not oss_access_key_id:
- raise Exception(f"No access_key_id is specified for connection:
{self.oss_conn_id}")
+ raise ValueError(f"No access_key_id is specified for connection:
{self.oss_conn_id}")
if not oss_access_key_secret:
- raise Exception(f"No access_key_secret is specified for
connection: {self.oss_conn_id}")
+ raise ValueError(f"No access_key_secret is specified for
connection: {self.oss_conn_id}")
return oss2.Auth(oss_access_key_id, oss_access_key_secret)
@@ -356,12 +356,12 @@ class OSSHook(BaseHook):
extra_config = self.oss_conn.extra_dejson
auth_type = extra_config.get("auth_type", None)
if not auth_type:
- raise Exception("No auth_type specified in extra_config. ")
+ raise ValueError("No auth_type specified in extra_config. ")
if auth_type != "AK":
- raise Exception(f"Unsupported auth_type: {auth_type}")
+ raise ValueError(f"Unsupported auth_type: {auth_type}")
default_region = extra_config.get("region", None)
if not default_region:
- raise Exception(f"No region is specified for connection:
{self.oss_conn_id}")
+ raise ValueError(f"No region is specified for connection:
{self.oss_conn_id}")
return default_region
diff --git a/pyproject.toml b/pyproject.toml
index f8c3b5a415..d5fe430b34 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -383,8 +383,6 @@ combine-as-imports = true
"airflow/providers/airbyte/hooks/airbyte.py" = ["TRY002"]
"airflow/providers/airbyte/operators/airbyte.py" = ["TRY002"]
"airflow/providers/airbyte/sensors/airbyte.py" = ["TRY002"]
-# alibaba
-"airflow/providers/alibaba/cloud/hooks/oss.py" = ["TRY002"]
# amazon
"airflow/providers/amazon/aws/operators/athena.py" = ["TRY002"]
"airflow/providers/amazon/aws/operators/emr.py" = ["TRY002"]
