Add missing default for version suffix (#35842)

The missing default was causing the suffix to be None when not specified and crashing package building.
apache · Nov 26, 2023 · db69770 · db69770
1 parent 4fd6166
commit db69770
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -830,6 +830,10 @@ jobs:
         if: needs.build-info.outputs.affected-providers-list-as-string != ''
         env:
           SKIP_CONSTRAINTS: "${{ needs.build-info.outputs.upgrade-to-newer-dependencies }}"
+      - name: "Prepare airflow package: wheel without suffix and skipping the tag check"
+        run: >
+          breeze release-management prepare-provider-packages --skip-tag-check
+          --package-format wheel ${{ needs.build-info.outputs.affected-providers-list-as-string }}
 
   provider-airflow-compatibility-check:
     timeout-minutes: 80

diff --git a/dev/breeze/src/airflow_breeze/utils/common_options.py b/dev/breeze/src/airflow_breeze/utils/common_options.py
@@ -414,6 +414,7 @@ def _set_default_from_parent(ctx: click.core.Context, option: click.core.Option,
     "--version-suffix-for-pypi",
     help="Version suffix used for PyPI packages (alpha, beta, rc1, etc.).",
     envvar="VERSION_SUFFIX_FOR_PYPI",
+    default="",
 )
 option_version_suffix_for_pypi_ci = click.option(
     "--version-suffix-for-pypi",

diff --git a/images/breeze/output_prod-image_build.txt b/images/breeze/output_prod-image_build.txt
@@ -1 +1 @@
-5ae644dee2b69e91dbcd593cdeebd3e7
+efdd516ea49fd7f8f9c770fa466d879b
diff --git a/images/breeze/output_release-management_prepare-airflow-package.txt b/images/breeze/output_release-management_prepare-airflow-package.txt
@@ -1 +1 @@
-91b0782d87b4e0ed7ce6e8cb231adafd
+5077bcfc4272c969f8b80ea4438a438a
diff --git a/images/breeze/output_release-management_prepare-provider-packages.txt b/images/breeze/output_release-management_prepare-provider-packages.txt
@@ -1 +1 @@
-2bb361f519c8165e884439722e76e945
+1b7d25cb5f379cee89c5345c60be1992
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		5ae644dee2b69e91dbcd593cdeebd3e7
		efdd516ea49fd7f8f9c770fa466d879b
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		91b0782d87b4e0ed7ce6e8cb231adafd
		5077bcfc4272c969f8b80ea4438a438a
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		2bb361f519c8165e884439722e76e945
		1b7d25cb5f379cee89c5345c60be1992