diff --git a/.github/workflows/helm-tests.yml b/.github/workflows/helm-tests.yml index 65b866785c704..e889e25160cea 100644 --- a/.github/workflows/helm-tests.yml +++ b/.github/workflows/helm-tests.yml @@ -130,11 +130,10 @@ jobs: SIGN_WITH: dev@airflow.apache.org - name: "Fetch Git Tags" run: git fetch --tags - - name: "Test helm chart issue generation automatically" - # Adding same tags for now, will address in a follow-up + - name: "Test helm chart issue generation" run: > - breeze release-management generate-issue-content-helm-chart --limit-pr-count 10 - --latest --verbose + breeze release-management generate-issue-content-helm-chart --limit-pr-count 2 + --previous-release helm-chart/1.15.0 --current-release helm-chart/1.16.0 --verbose - name: "Upload Helm artifacts" uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2 with: diff --git a/dev/README_RELEASE_HELM_CHART.md b/dev/README_RELEASE_HELM_CHART.md index ff58a5957b428..0364f947af5e5 100644 --- a/dev/README_RELEASE_HELM_CHART.md +++ b/dev/README_RELEASE_HELM_CHART.md @@ -331,7 +331,7 @@ EOF Content is generated with: ```shell -breeze release-management generate-issue-content-helm-chart +breeze release-management generate-issue-content-helm-chart \ --previous-release helm-chart/ --current-release helm-chart/${VERSION}${VERSION_SUFFIX} ``` diff --git a/dev/breeze/doc/images/output_release-management_generate-issue-content-helm-chart.svg b/dev/breeze/doc/images/output_release-management_generate-issue-content-helm-chart.svg index 648f163976ecf..e9feecddd6ca0 100644 --- a/dev/breeze/doc/images/output_release-management_generate-issue-content-helm-chart.svg +++ b/dev/breeze/doc/images/output_release-management_generate-issue-content-helm-chart.svg @@ -38,7 +38,9 @@ .breeze-release-management-generate-issue-content-helm-chart-r4 { fill: #68a0b3;font-weight: bold } .breeze-release-management-generate-issue-content-helm-chart-r5 { fill: #868887 } .breeze-release-management-generate-issue-content-helm-chart-r6 { fill: #8d7b39 } -.breeze-release-management-generate-issue-content-helm-chart-r7 { fill: #98a84b;font-weight: bold } +.breeze-release-management-generate-issue-content-helm-chart-r7 { fill: #cc555a } +.breeze-release-management-generate-issue-content-helm-chart-r8 { fill: #8a4346 } +.breeze-release-management-generate-issue-content-helm-chart-r9 { fill: #98a84b;font-weight: bold } @@ -120,19 +122,19 @@ Generates content for issue to test the helm chart release. ╭─ Generate issue flags ───────────────────────────────────────────────────────────────────────────────────────────────╮ -│--github-tokenGitHub token used to authenticate. You can set omit it if you have GITHUB_TOKEN env variable  │ -│set. Can be generated with:                                                                   │ -│https://github.com/settings/tokens/new?description=Read%20sssues&scopes=repo:status           │ -│(TEXT)                                                                                        │ -│--previous-releasecommit reference (for example hash or tag) of the previous release.(TEXT)│ -│--current-releasecommit reference (for example hash or tag) of the current release.(TEXT)│ -│--excluded-pr-listComa-separated list of PRs to exclude from the issue.(TEXT)│ -│--limit-pr-countLimit PR count processes (useful for testing small subset of PRs).(INTEGER)│ -│--latestRun the command against latest released version of airflow helm charts│ +│--github-tokenGitHub token used to authenticate. You can set omit it if you have GITHUB_TOKEN env        │ +│variable set. Can be generated with:                                                       │ +│https://github.com/settings/tokens/new?description=Read%20sssues&scopes=repo:status        │ +│(TEXT)                                                                                     │ +│*--previous-releasecommit reference (for example hash or tag) of the previous release.(TEXT)[required]│ +│*--current-releasecommit reference (for example hash or tag) of the current release.(TEXT)[required]│ +│--excluded-pr-listComa-separated list of PRs to exclude from the issue.(TEXT)│ +│--limit-pr-countLimit PR count processes (useful for testing small subset of PRs).(INTEGER)│ +│--latestRun the command against latest released version of airflow helm charts│ ╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯ ╭─ Common options ─────────────────────────────────────────────────────────────────────────────────────────────────────╮ -│--verbose-vPrint verbose information about performed steps.│ -│--help-hShow this message and exit.│ +│--verbose-vPrint verbose information about performed steps.│ +│--help-hShow this message and exit.│ ╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯ diff --git a/dev/breeze/doc/images/output_release-management_generate-issue-content-helm-chart.txt b/dev/breeze/doc/images/output_release-management_generate-issue-content-helm-chart.txt index 68f8a48a8b621..135b4c86b52fc 100644 --- a/dev/breeze/doc/images/output_release-management_generate-issue-content-helm-chart.txt +++ b/dev/breeze/doc/images/output_release-management_generate-issue-content-helm-chart.txt @@ -1 +1 @@ -798f6631dad1b650e8a48fa5d51d73e6 +b18a6246850723983de6bdd98273b095 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 f9cf24bae709c..8ebbef8194d1c 100644 --- a/dev/breeze/src/airflow_breeze/commands/release_management_commands.py +++ b/dev/breeze/src/airflow_breeze/commands/release_management_commands.py @@ -2676,11 +2676,13 @@ def print_issue_content( "--previous-release", type=str, help="commit reference (for example hash or tag) of the previous release.", + required=True, ) @click.option( "--current-release", type=str, help="commit reference (for example hash or tag) of the current release.", + required=True, ) @click.option("--excluded-pr-list", type=str, help="Coma-separated list of PRs to exclude from the issue.") @click.option(