apache-airflow-providers-google 15.0.1rc1__py3-none-any.whl → 15.1.0__py3-none-any.whl

airflow/providers/google/__init__.py

@@ -29,7 +29,7 @@ from airflow import __version__ as airflow_version
 
 __all__ = ["__version__"]
 
-__version__ = "15.0.1"
+__version__ = "15.1.0"
 
 if packaging.version.parse(packaging.version.parse(airflow_version).base_version) < packaging.version.parse(
     "2.9.0"

airflow/providers/google/cloud/hooks/bigquery.py

@@ -123,7 +123,7 @@ class BigQueryHook(GoogleBaseHook, DbApiHook):
         from airflow.providers.google.cloud.utils.validators import ValidJson
 
         connection_form_widgets = super().get_connection_form_widgets()
-        connection_form_widgets["use_legacy_sql"] = BooleanField(lazy_gettext("Use Legacy SQL"), default=True)
+        connection_form_widgets["use_legacy_sql"] = BooleanField(lazy_gettext("Use Legacy SQL"))
         connection_form_widgets["location"] = StringField(
             lazy_gettext("Location"), widget=BS3TextFieldWidget()
         )
@@ -1376,8 +1376,7 @@ class BigQueryHook(GoogleBaseHook, DbApiHook):
         def var_print(var_name):
             if var_name is None:
                 return ""
-            else:
-                return f"Format exception for {var_name}: "
+            return f"Format exception for {var_name}: "
 
         if table_input.count(".") + table_input.count(":") > 3:
             raise ValueError(f"{var_print(var_name)}Use either : or . to specify project got {table_input}")
@@ -1955,8 +1954,7 @@ def split_tablename(
     def var_print(var_name):
         if var_name is None:
             return ""
-        else:
-            return f"Format exception for {var_name}: "
+        return f"Format exception for {var_name}: "
 
     if table_input.count(".") + table_input.count(":") > 3:
         raise ValueError(f"{var_print(var_name)}Use either : or . to specify project got {table_input}")

airflow/providers/google/cloud/hooks/cloud_batch.py

@@ -147,19 +147,18 @@ class CloudBatchHook(GoogleBaseHook):
                 status: JobStatus.State = job.status.state
                 if status == JobStatus.State.SUCCEEDED:
                     return job
-                elif status == JobStatus.State.FAILED:
+                if status == JobStatus.State.FAILED:
                     message = (
                         "Unexpected error in the operation: "
                         "Batch job with name {job_name} has failed its execution."
                     )
                     raise AirflowException(message)
-                elif status == JobStatus.State.DELETION_IN_PROGRESS:
+                if status == JobStatus.State.DELETION_IN_PROGRESS:
                     message = (
                         "Unexpected error in the operation: Batch job with name {job_name} is being deleted."
                     )
                     raise AirflowException(message)
-                else:
-                    time.sleep(polling_period_seconds)
+                time.sleep(polling_period_seconds)
             except Exception as e:
                 self.log.exception("Exception occurred while checking for job completion.")
                 raise e

airflow/providers/google/cloud/hooks/cloud_sql.py

@@ -634,35 +634,32 @@ class CloudSqlProxyRunner(LoggingMixin):
         self._download_sql_proxy_if_needed()
         if self.sql_proxy_process:
             raise AirflowException(f"The sql proxy is already running: {self.sql_proxy_process}")
-        else:
-            command_to_run = [self.sql_proxy_path]
-            command_to_run.extend(self.command_line_parameters)
-            self.log.info("Creating directory %s", self.cloud_sql_proxy_socket_directory)
-            Path(self.cloud_sql_proxy_socket_directory).mkdir(parents=True, exist_ok=True)
-            command_to_run.extend(self._get_credential_parameters())
-            self.log.info("Running the command: `%s`", " ".join(command_to_run))
-
-            self.sql_proxy_process = Popen(command_to_run, stdin=PIPE, stdout=PIPE, stderr=PIPE)
-            self.log.info("The pid of cloud_sql_proxy: %s", self.sql_proxy_process.pid)
-            while True:
-                line = (
-                    self.sql_proxy_process.stderr.readline().decode("utf-8")
-                    if self.sql_proxy_process.stderr
-                    else ""
-                )
-                return_code = self.sql_proxy_process.poll()
-                if line == "" and return_code is not None:
-                    self.sql_proxy_process = None
-                    raise AirflowException(
-                        f"The cloud_sql_proxy finished early with return code {return_code}!"
-                    )
-                if line != "":
-                    self.log.info(line)
-                if "googleapi: Error" in line or "invalid instance name:" in line:
-                    self.stop_proxy()
-                    raise AirflowException(f"Error when starting the cloud_sql_proxy {line}!")
-                if "Ready for new connections" in line:
-                    return
+        command_to_run = [self.sql_proxy_path]
+        command_to_run.extend(self.command_line_parameters)
+        self.log.info("Creating directory %s", self.cloud_sql_proxy_socket_directory)
+        Path(self.cloud_sql_proxy_socket_directory).mkdir(parents=True, exist_ok=True)
+        command_to_run.extend(self._get_credential_parameters())
+        self.log.info("Running the command: `%s`", " ".join(command_to_run))
+
+        self.sql_proxy_process = Popen(command_to_run, stdin=PIPE, stdout=PIPE, stderr=PIPE)
+        self.log.info("The pid of cloud_sql_proxy: %s", self.sql_proxy_process.pid)
+        while True:
+            line = (
+                self.sql_proxy_process.stderr.readline().decode("utf-8")
+                if self.sql_proxy_process.stderr
+                else ""
+            )
+            return_code = self.sql_proxy_process.poll()
+            if line == "" and return_code is not None:
+                self.sql_proxy_process = None
+                raise AirflowException(f"The cloud_sql_proxy finished early with return code {return_code}!")
+            if line != "":
+                self.log.info(line)
+            if "googleapi: Error" in line or "invalid instance name:" in line:
+                self.stop_proxy()
+                raise AirflowException(f"Error when starting the cloud_sql_proxy {line}!")
+            if "Ready for new connections" in line:
+                return
 
     def stop_proxy(self) -> None:
         """
@@ -672,10 +669,9 @@ class CloudSqlProxyRunner(LoggingMixin):
         """
         if not self.sql_proxy_process:
             raise AirflowException("The sql proxy is not started yet")
-        else:
-            self.log.info("Stopping the cloud_sql_proxy pid: %s", self.sql_proxy_process.pid)
-            self.sql_proxy_process.kill()
-            self.sql_proxy_process = None
+        self.log.info("Stopping the cloud_sql_proxy pid: %s", self.sql_proxy_process.pid)
+        self.sql_proxy_process.kill()
+        self.sql_proxy_process = None
         # Cleanup!
         self.log.info("Removing the socket directory: %s", self.cloud_sql_proxy_socket_directory)
         shutil.rmtree(self.cloud_sql_proxy_socket_directory, ignore_errors=True)
@@ -704,8 +700,7 @@ class CloudSqlProxyRunner(LoggingMixin):
         matched = re.search("[Vv]ersion (.*?);", result)
         if matched:
             return matched.group(1)
-        else:
-            return None
+        return None
 
     def get_socket_path(self) -> str:
         """
@@ -908,10 +903,9 @@ class CloudSQLDatabaseHook(BaseHook):
         secret_data = json.loads(base64.b64decode(secret.payload.data))
         if cert_name in secret_data:
             return secret_data[cert_name]
-        else:
-            raise AirflowException(
-                "Invalid secret format. Expected dictionary with keys: `sslcert`, `sslkey`, `sslrootcert`"
-            )
+        raise AirflowException(
+            "Invalid secret format. Expected dictionary with keys: `sslcert`, `sslkey`, `sslrootcert`"
+        )
 
     def _set_temporary_ssl_file(
         self, cert_name: str, cert_path: str | None = None, cert_value: str | None = None
@@ -1205,8 +1199,7 @@ class CloudSQLDatabaseHook(BaseHook):
 
         if self.database_type == "postgres":
             return self.cloudsql_connection.login.split(".gserviceaccount.com")[0]
-        else:
-            return self.cloudsql_connection.login.split("@")[0]
+        return self.cloudsql_connection.login.split("@")[0]
 
     def _generate_login_token(self, service_account) -> str:
         """Generate an IAM login token for Cloud SQL and return the token."""

airflow/providers/google/cloud/hooks/cloud_storage_transfer_service.py

@@ -218,7 +218,7 @@ class CloudDataTransferServiceHook(GoogleBaseHook):
                     return (
                         self.get_conn().transferJobs().create(body=body).execute(num_retries=self.num_retries)
                     )
-                elif transfer_job.get(STATUS) == GcpTransferJobsStatus.DISABLED:
+                if transfer_job.get(STATUS) == GcpTransferJobsStatus.DISABLED:
                     return self.enable_transfer_job(job_name=job_name, project_id=body.get(PROJECT_ID))
             else:
                 raise e

airflow/providers/google/cloud/hooks/compute_ssh.py

@@ -148,8 +148,7 @@ class ComputeEngineSSHHook(SSHHook):
             return ComputeEngineHook(
                 gcp_conn_id=self.gcp_conn_id, impersonation_chain=self.impersonation_chain
             )
-        else:
-            return ComputeEngineHook(gcp_conn_id=self.gcp_conn_id)
+        return ComputeEngineHook(gcp_conn_id=self.gcp_conn_id)
 
     def _load_connection_config(self):
         def _boolify(value):
@@ -158,7 +157,7 @@ class ComputeEngineSSHHook(SSHHook):
             if isinstance(value, str):
                 if value.lower() == "false":
                     return False
-                elif value.lower() == "true":
+                if value.lower() == "true":
                     return True
             return False
 

airflow/providers/google/cloud/hooks/dataflow.py

@@ -51,6 +51,7 @@ from googleapiclient.discovery import Resource, build
 
 from airflow.exceptions import AirflowException, AirflowProviderDeprecationWarning
 from airflow.providers.apache.beam.hooks.beam import BeamHook, BeamRunnerType, beam_options_to_args
+from airflow.providers.google.common.deprecated import deprecated
 from airflow.providers.google.common.hooks.base_google import (
     PROVIDE_PROJECT_ID,
     GoogleBaseAsyncHook,
@@ -261,15 +262,14 @@ class _DataflowJobsController(LoggingMixin):
         """
         if not self._multiple_jobs and self._job_id:
             return [self.fetch_job_by_id(self._job_id)]
-        elif self._jobs:
+        if self._jobs:
             return [self.fetch_job_by_id(job["id"]) for job in self._jobs]
-        elif self._job_name:
+        if self._job_name:
             jobs = self._fetch_jobs_by_prefix_name(self._job_name.lower())
             if len(jobs) == 1:
                 self._job_id = jobs[0]["id"]
             return jobs
-        else:
-            raise ValueError("Missing both dataflow job ID and name.")
+        raise ValueError("Missing both dataflow job ID and name.")
 
     def fetch_job_by_id(self, job_id: str) -> dict[str, str]:
         """
@@ -434,12 +434,12 @@ class _DataflowJobsController(LoggingMixin):
                 f"'{current_expected_state}' is invalid."
                 f" The value should be any of the following: {terminal_states}"
             )
-        elif is_streaming and current_expected_state == DataflowJobStatus.JOB_STATE_DONE:
+        if is_streaming and current_expected_state == DataflowJobStatus.JOB_STATE_DONE:
             raise AirflowException(
                 "Google Cloud Dataflow job's expected terminal state cannot be "
                 "JOB_STATE_DONE while it is a streaming job"
             )
-        elif not is_streaming and current_expected_state == DataflowJobStatus.JOB_STATE_DRAINED:
+        if not is_streaming and current_expected_state == DataflowJobStatus.JOB_STATE_DRAINED:
             raise AirflowException(
                 "Google Cloud Dataflow job's expected terminal state cannot be "
                 "JOB_STATE_DRAINED while it is a batch job"
@@ -1063,6 +1063,11 @@ class DataflowHook(GoogleBaseHook):
         )
         jobs_controller.cancel()
 
+    @deprecated(
+        planned_removal_date="July 01, 2025",
+        use_instead="airflow.providers.google.cloud.hooks.dataflow.DataflowHook.launch_beam_yaml_job",
+        category=AirflowProviderDeprecationWarning,
+    )
     @GoogleBaseHook.fallback_to_default_project_id
     def start_sql_job(
         self,

airflow/providers/google/cloud/hooks/datafusion.py

@@ -170,9 +170,9 @@ class DataFusionHook(GoogleBaseHook):
     def _check_response_status_and_data(response, message: str) -> None:
         if response.status == 404:
             raise AirflowNotFoundException(message)
-        elif response.status == 409:
+        if response.status == 409:
             raise ConflictException("Conflict: Resource is still in use.")
-        elif response.status != 200:
+        if response.status != 200:
             raise AirflowException(message)
         if response.data is None:
             raise AirflowException(
@@ -572,8 +572,7 @@ class DataFusionAsyncHook(GoogleBaseAsyncHook):
                         raise
         if pipeline:
             return pipeline
-        else:
-            raise AirflowException("Could not retrieve pipeline. Aborting.")
+        raise AirflowException("Could not retrieve pipeline. Aborting.")
 
     async def get_pipeline(
         self,

airflow/providers/google/cloud/hooks/dataprep.py

@@ -31,7 +31,7 @@ from tenacity import retry, stop_after_attempt, wait_exponential
 from airflow.hooks.base import BaseHook
 
 
-def _get_field(extras: dict, field_name: str):
+def _get_field(extras: dict, field_name: str) -> str | None:
     """Get field from extra, first checking short name, then for backcompat we check for prefixed name."""
     backcompat_prefix = "extra__dataprep__"
     if field_name.startswith("extra__"):

airflow/providers/google/cloud/hooks/dlp.py

@@ -268,7 +268,7 @@ class CloudDLPHook(GoogleBaseHook):
 
             if job.state == DlpJob.JobState.DONE:
                 return job
-            elif job.state in [
+            if job.state in [
                 DlpJob.JobState.PENDING,
                 DlpJob.JobState.RUNNING,
                 DlpJob.JobState.JOB_STATE_UNSPECIFIED,

airflow/providers/google/cloud/hooks/gcs.py

@@ -358,11 +358,10 @@ class GCSHook(GoogleBaseHook):
                     )
                     self.log.info("File downloaded to %s", filename)
                     return filename
-                else:
-                    get_hook_lineage_collector().add_input_asset(
-                        context=self, scheme="gs", asset_kwargs={"bucket": bucket.name, "key": blob.name}
-                    )
-                    return blob.download_as_bytes()
+                get_hook_lineage_collector().add_input_asset(
+                    context=self, scheme="gs", asset_kwargs={"bucket": bucket.name, "key": blob.name}
+                )
+                return blob.download_as_bytes()
 
             except GoogleCloudError:
                 if attempt == num_max_attempts - 1:
@@ -556,7 +555,7 @@ class GCSHook(GoogleBaseHook):
                 "specify a single parameter, either 'filename' for "
                 "local file uploads or 'data' for file content uploads."
             )
-        elif filename:
+        if filename:
             if not mime_type:
                 mime_type = "application/octet-stream"
             if gzip:

airflow/providers/google/cloud/hooks/kubernetes_engine.py

@@ -352,8 +352,7 @@ class GKEHook(GoogleBaseHook):
             or node_pools_autoscaled
         ):
             return True
-        else:
-            return False
+        return False
 
 
 class GKEAsyncHook(GoogleBaseAsyncHook):

airflow/providers/google/cloud/hooks/managed_kafka.py

@@ -120,7 +120,7 @@ class ManagedKafkaHook(GoogleBaseHook):
             error = operation.exception(timeout=timeout)
             raise AirflowException(error)
 
-    def get_confluent_token(self):
+    def get_confluent_token(self, config_str: str):
         """Get the authentication token for confluent client."""
         token_provider = ManagedKafkaTokenProvider(credentials=self.get_credentials())
         token = token_provider.confluent_token()

airflow/providers/google/cloud/hooks/mlengine.py

@@ -78,8 +78,7 @@ def _poll_with_exponential_delay(
             if e.resp.status != 429:
                 log.info("Something went wrong. Not retrying: %s", format(e))
                 raise
-            else:
-                time.sleep((2**i) + random.random())
+            time.sleep((2**i) + random.random())
 
     raise ValueError(f"Connection could not be established after {max_n} retries.")
 
@@ -219,12 +218,11 @@ class MLEngineHook(GoogleBaseHook):
             if e.resp.status == 404:
                 self.log.error("Job with job_id %s does not exist. ", job_id)
                 raise
-            elif e.resp.status == 400:
+            if e.resp.status == 400:
                 self.log.info("Job with job_id %s is already complete, cancellation aborted.", job_id)
                 return {}
-            else:
-                self.log.error("Failed to cancel MLEngine job: %s", e)
-                raise
+            self.log.error("Failed to cancel MLEngine job: %s", e)
+            raise
 
     def get_job(self, project_id: str, job_id: str) -> dict:
         """

airflow/providers/google/cloud/hooks/stackdriver.py

@@ -121,10 +121,9 @@ class StackdriverHook(GoogleBaseHook):
         )
         if format_ == "dict":
             return [AlertPolicy.to_dict(policy) for policy in policies_]
-        elif format_ == "json":
+        if format_ == "json":
             return [AlertPolicy.to_jsoon(policy) for policy in policies_]
-        else:
-            return policies_
+        return policies_
 
     @GoogleBaseHook.fallback_to_default_project_id
     def _toggle_policy_status(
@@ -395,10 +394,9 @@ class StackdriverHook(GoogleBaseHook):
         )
         if format_ == "dict":
             return [NotificationChannel.to_dict(channel) for channel in channels]
-        elif format_ == "json":
+        if format_ == "json":
             return [NotificationChannel.to_json(channel) for channel in channels]
-        else:
-            return channels
+        return channels
 
     @GoogleBaseHook.fallback_to_default_project_id
     def _toggle_channel_status(

airflow/providers/google/cloud/hooks/vision.py

@@ -107,8 +107,7 @@ class NameDeterminer:
         # Not enough parameters to construct the name. Trying to use the name from Product / ProductSet.
         if explicit_name:
             return entity
-        else:
-            raise AirflowException(ERR_UNABLE_TO_CREATE.format(label=self.label, id_label=self.id_label))
+        raise AirflowException(ERR_UNABLE_TO_CREATE.format(label=self.label, id_label=self.id_label))
 
 
 class CloudVisionHook(GoogleBaseHook):

airflow/providers/google/cloud/openlineage/mixins.py

@@ -207,15 +207,14 @@ class _BigQueryInsertJobOperatorOpenLineageMixin:
                 name=dataset_name,
                 facets=dataset_facets,
             )
-        elif dataset_type == "output":
+        if dataset_type == "output":
             # Logic specific to creating OutputDataset (if needed)
             return OutputDataset(
                 namespace=BIGQUERY_NAMESPACE,
                 name=dataset_name,
                 facets=dataset_facets,
             )
-        else:
-            raise ValueError("Invalid dataset_type. Must be 'input' or 'output'")
+        raise ValueError("Invalid dataset_type. Must be 'input' or 'output'")
 
     def _get_table_facets_safely(self, table_name: str) -> dict[str, DatasetFacet]:
         try:

airflow/providers/google/cloud/operators/alloy_db.py

@@ -145,8 +145,7 @@ class AlloyDBWriteBaseOperator(AlloyDBBaseOperator):
         if self.validate_request:
             # Validation requests are only validated and aren't executed, thus no operation result is expected
             return None
-        else:
-            return self.hook.wait_for_operation(timeout=self.timeout, operation=operation)
+        return self.hook.wait_for_operation(timeout=self.timeout, operation=operation)
 
 
 class AlloyDBCreateClusterOperator(AlloyDBWriteBaseOperator):

airflow/providers/google/cloud/operators/automl.py

@@ -259,11 +259,11 @@ class AutoMLPredictOperator(GoogleCloudBaseOperator):
                 gcp_conn_id=self.gcp_conn_id,
                 impersonation_chain=self.impersonation_chain,
             )
-        else:  # endpoint_id defined
-            return PredictionServiceHook(
-                gcp_conn_id=self.gcp_conn_id,
-                impersonation_chain=self.impersonation_chain,
-            )
+        # endpoint_id defined
+        return PredictionServiceHook(
+            gcp_conn_id=self.gcp_conn_id,
+            impersonation_chain=self.impersonation_chain,
+        )
 
     @cached_property
     def model(self) -> Model | None:

airflow/providers/google/cloud/operators/bigquery.py

@@ -287,7 +287,7 @@ class BigQueryCheckOperator(
     def _validate_records(self, records) -> None:
         if not records:
             raise AirflowException(f"The following query returned zero rows: {self.sql}")
-        elif not all(records):
+        if not all(records):
             self._raise_exception(  # type: ignore[attr-defined]
                 f"Test failed.\nQuery:\n{self.sql}\nResults:\n{records!s}"
             )
@@ -2976,14 +2976,13 @@ class BigQueryInsertJobOperator(GoogleCloudBaseOperator, _BigQueryInsertJobOpera
                     f"Or, if you want to reattach in this scenario add {job.state} to `reattach_states`"
                 )
 
-            else:
-                # Job already reached state DONE
-                if job.state == "DONE":
-                    raise AirflowException("Job is already in state DONE. Can not reattach to this job.")
+            # Job already reached state DONE
+            if job.state == "DONE":
+                raise AirflowException("Job is already in state DONE. Can not reattach to this job.")
 
-                # We are reattaching to a job
-                self.log.info("Reattaching to existing Job in state %s", job.state)
-                self._handle_job_error(job)
+            # We are reattaching to a job
+            self.log.info("Reattaching to existing Job in state %s", job.state)
+            self._handle_job_error(job)
 
         job_types = {
             LoadJob._JOB_TYPE: ["sourceTable", "destinationTable"],
@@ -3036,24 +3035,23 @@ class BigQueryInsertJobOperator(GoogleCloudBaseOperator, _BigQueryInsertJobOpera
             self._handle_job_error(job)
 
             return self.job_id
-        else:
-            if job.running():
-                self.defer(
-                    timeout=self.execution_timeout,
-                    trigger=BigQueryInsertJobTrigger(
-                        conn_id=self.gcp_conn_id,
-                        job_id=self.job_id,
-                        project_id=self.project_id,
-                        location=self.location or hook.location,
-                        poll_interval=self.poll_interval,
-                        impersonation_chain=self.impersonation_chain,
-                        cancel_on_kill=self.cancel_on_kill,
-                    ),
-                    method_name="execute_complete",
-                )
-            self.log.info("Current state of job %s is %s", job.job_id, job.state)
-            self._handle_job_error(job)
-            return self.job_id
+        if job.running():
+            self.defer(
+                timeout=self.execution_timeout,
+                trigger=BigQueryInsertJobTrigger(
+                    conn_id=self.gcp_conn_id,
+                    job_id=self.job_id,
+                    project_id=self.project_id,
+                    location=self.location or hook.location,
+                    poll_interval=self.poll_interval,
+                    impersonation_chain=self.impersonation_chain,
+                    cancel_on_kill=self.cancel_on_kill,
+                ),
+                method_name="execute_complete",
+            )
+        self.log.info("Current state of job %s is %s", job.job_id, job.state)
+        self._handle_job_error(job)
+        return self.job_id
 
     def execute_complete(self, context: Context, event: dict[str, Any]) -> str | None:
         """

airflow/providers/google/cloud/operators/cloud_batch.py

@@ -100,19 +100,18 @@ class CloudBatchSubmitJobOperator(GoogleCloudBaseOperator):
 
             return Job.to_dict(completed_job)
 
-        else:
-            self.defer(
-                trigger=CloudBatchJobFinishedTrigger(
-                    job_name=job.name,
-                    project_id=self.project_id,
-                    gcp_conn_id=self.gcp_conn_id,
-                    impersonation_chain=self.impersonation_chain,
-                    location=self.region,
-                    polling_period_seconds=self.polling_period_seconds,
-                    timeout=self.timeout_seconds,
-                ),
-                method_name="execute_complete",
-            )
+        self.defer(
+            trigger=CloudBatchJobFinishedTrigger(
+                job_name=job.name,
+                project_id=self.project_id,
+                gcp_conn_id=self.gcp_conn_id,
+                impersonation_chain=self.impersonation_chain,
+                location=self.region,
+                polling_period_seconds=self.polling_period_seconds,
+                timeout=self.timeout_seconds,
+            ),
+            method_name="execute_complete",
+        )
 
     def execute_complete(self, context: Context, event: dict):
         job_status = event["status"]
@@ -120,8 +119,7 @@ class CloudBatchSubmitJobOperator(GoogleCloudBaseOperator):
             hook: CloudBatchHook = CloudBatchHook(self.gcp_conn_id, self.impersonation_chain)
             job = hook.get_job(job_name=event["job_name"])
             return Job.to_dict(job)
-        else:
-            raise AirflowException(f"Unexpected error in the operation: {event['message']}")
+        raise AirflowException(f"Unexpected error in the operation: {event['message']}")
 
 
 class CloudBatchDeleteJobOperator(GoogleCloudBaseOperator):

airflow/providers/google/cloud/operators/cloud_build.py

@@ -275,8 +275,7 @@ class CloudBuildCreateBuildOperator(GoogleCloudBaseOperator):
                     build_id=event["id_"],
                 )
             return event["instance"]
-        else:
-            raise AirflowException(f"Unexpected error in the operation: {event['message']}")
+        raise AirflowException(f"Unexpected error in the operation: {event['message']}")
 
 
 class CloudBuildCreateBuildTriggerOperator(GoogleCloudBaseOperator):