diff --git a/.github/actions/build-ci-images/action.yml b/.github/actions/build-ci-images/action.yml index c437835a4d89b..820c858366910 100644 --- a/.github/actions/build-ci-images/action.yml +++ b/.github/actions/build-ci-images/action.yml @@ -23,9 +23,7 @@ runs: steps: - name: "Install Breeze" uses: ./.github/actions/breeze - - name: "Regenerate dependencies" - # This is done in case some someone updated provider dependencies and did not generate - # dependencies - in which case build image might fail because of lack of new dependencies + - name: "Check if dependencies are properly regenerated" shell: bash run: | pip install rich>=12.4.4 pyyaml diff --git a/generated/provider_dependencies.json b/generated/provider_dependencies.json index 780009c5365e0..b755f6c839c3c 100644 --- a/generated/provider_dependencies.json +++ b/generated/provider_dependencies.json @@ -418,7 +418,7 @@ "asgiref>=3.5.2", "gcloud-aio-auth>=4.0.0,<5.0.0", "gcloud-aio-bigquery>=6.1.2", - "gcloud-aio-storage", + "gcloud-aio-storage>=9.0.0", "gcsfs>=2023.10.0", "google-ads>=22.1.0", "google-api-core>=2.11.0", diff --git a/scripts/ci/pre_commit/pre_commit_update_providers_dependencies.py b/scripts/ci/pre_commit/pre_commit_update_providers_dependencies.py index ada9f884be81a..a3056df3a14b7 100755 --- a/scripts/ci/pre_commit/pre_commit_update_providers_dependencies.py +++ b/scripts/ci/pre_commit/pre_commit_update_providers_dependencies.py @@ -213,11 +213,12 @@ def check_if_different_provider_used(file_path: Path) -> None: DEPENDENCIES_JSON_FILE_PATH.write_text(json.dumps(unique_sorted_dependencies, indent=2) + "\n") if os.environ.get("CI"): console.print() - console.print(f"[info]Written {DEPENDENCIES_JSON_FILE_PATH}") + console.print(f"[info]There is a need to regenerate {DEPENDENCIES_JSON_FILE_PATH}") console.print( - f"[yellow]You will need to run breeze locally and commit " - f"{DEPENDENCIES_JSON_FILE_PATH.relative_to(AIRFLOW_SOURCES_ROOT)}!\n" + f"[red]You need to run the following command locally and commit generated " + f"{DEPENDENCIES_JSON_FILE_PATH.relative_to(AIRFLOW_SOURCES_ROOT)} file:\n" ) + console.print("breeze static-checks --type update-providers-dependencies --all-files") console.print() else: console.print() @@ -227,6 +228,7 @@ def check_if_different_provider_used(file_path: Path) -> None: ) console.print(f"[info]Written {DEPENDENCIES_JSON_FILE_PATH}") console.print() + sys.exit(1) else: console.print( "[green]No need to regenerate dependencies!\n[/]"