This is an automated email from the ASF dual-hosted git repository.
potiuk 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 3edb3ae98b0 Improve relase process after releasing rc2 candidates
(#58415)
3edb3ae98b0 is described below
commit 3edb3ae98b0501a3771ed6a9891a2a293297f006
Author: Jarek Potiuk <[email protected]>
AuthorDate: Tue Nov 18 23:24:12 2025 +0100
Improve relase process after releasing rc2 candidates (#58415)
Two fixes in the release process after RC2 preparation:
* svn add commoand adds the newly added ${RELEASE_DATE} folder
* skipped packages were reported as successful, now they are reported
skipped as already released
---
dev/README_RELEASE_PROVIDERS.md | 2 +-
...e-management_prepare-provider-documentation.txt | 2 +-
.../commands/release_management_commands.py | 26 +++++++++++++++++-----
3 files changed, 22 insertions(+), 8 deletions(-)
diff --git a/dev/README_RELEASE_PROVIDERS.md b/dev/README_RELEASE_PROVIDERS.md
index 38e733489ac..a9449a5e2ab 100644
--- a/dev/README_RELEASE_PROVIDERS.md
+++ b/dev/README_RELEASE_PROVIDERS.md
@@ -384,7 +384,7 @@ mkdir -p ${RELEASE_DATE}
mv ${AIRFLOW_REPO_ROOT}/dist/* "${RELEASE_DATE}"
# Add and commit
-svn add */*
+svn add ${RELEASE_DATA}
svn commit -m "Add artifacts for Airflow Providers ${RELEASE_DATE}"
cd ${AIRFLOW_REPO_ROOT}
diff --git
a/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.txt
b/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.txt
index f54c900ffea..a2d60b00293 100644
---
a/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.txt
+++
b/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.txt
@@ -1 +1 @@
-7ae99d9488b65f5872b32f07f8fa6d3b
+5a7356a41f6f0e7c0a5862d8ae1a2ba9
diff --git
a/dev/breeze/src/airflow_breeze/commands/release_management_commands.py
b/dev/breeze/src/airflow_breeze/commands/release_management_commands.py
index 6c69ea2be60..bd16fa799d6 100644
--- a/dev/breeze/src/airflow_breeze/commands/release_management_commands.py
+++ b/dev/breeze/src/airflow_breeze/commands/release_management_commands.py
@@ -783,7 +783,6 @@ def provider_action_summary(description: str, message_type:
MessageType, package
help="Base branch to use as diff for documentation generation (used for
releasing from old branch)",
)
@option_github_repository
-@argument_provider_distributions
@option_include_not_ready_providers
@option_include_removed_providers
@click.option(
@@ -827,6 +826,7 @@ def provider_action_summary(description: str, message_type:
MessageType, package
help="Planned release date for the providers release in format "
"YYYY-MM-DD[_NN] (e.g., 2025-11-16 or 2025-11-16_01).",
)
+@argument_provider_distributions
@option_verbose
@option_answer
@option_dry_run
@@ -1012,11 +1012,21 @@ def _build_provider_distributions(
distribution_format: str,
skip_tag_check: bool,
skip_deleting_generated_files: bool,
-):
+) -> bool:
+ """
+ Builds provider distribution.
+
+ :param provider_id: id of the provider package
+ :param package_version_suffix: suffix to append to the package version
+ :param distribution_format: format of the distribution to build (wheel or
sdist)
+ :param skip_tag_check: whether to skip tag check
+ :param skip_deleting_generated_files: whether to skip deleting generated
files
+ :return: True if package was built, False if it was skipped.
+ """
if not skip_tag_check:
should_skip, package_version_suffix =
should_skip_the_package(provider_id, package_version_suffix)
if should_skip:
- return
+ return False
get_console().print()
with ci_group(f"Preparing provider package [special]{provider_id}"):
get_console().print()
@@ -1039,6 +1049,7 @@ def _build_provider_distributions(
skip_cleanup=skip_deleting_generated_files,
delete_only_build_and_dist_folders=True,
)
+ return True
@release_management.command(
@@ -1137,7 +1148,7 @@ def prepare_provider_distributions(
for provider_id in packages_list:
try:
basic_provider_checks(provider_id)
- _build_provider_distributions(
+ created = _build_provider_distributions(
provider_id,
version_suffix,
distribution_format,
@@ -1154,12 +1165,15 @@ def prepare_provider_distributions(
suspended_packages.append(provider_id)
else:
get_console().print(f"\n[success]Generated package
[special]{provider_id}")
- success_packages.append(provider_id)
+ if created:
+ success_packages.append(provider_id)
+ else:
+ skipped_as_already_released_packages.append(provider_id)
get_console().print()
get_console().print("\n[info]Summary of prepared packages:\n")
provider_action_summary("Success", MessageType.SUCCESS, success_packages)
provider_action_summary(
- "Skipped as already released", MessageType.SUCCESS,
skipped_as_already_released_packages
+ "Skipped as already released", MessageType.INFO,
skipped_as_already_released_packages
)
provider_action_summary("Suspended", MessageType.WARNING,
suspended_packages)
provider_action_summary("Wrong setup generated", MessageType.ERROR,
wrong_setup_packages)
