Adjust release guides after Helm 1.20.0 release#64083
Open
jscheffl wants to merge 1 commit intoapache:mainfrom
Open
Adjust release guides after Helm 1.20.0 release#64083jscheffl wants to merge 1 commit intoapache:mainfrom
jscheffl wants to merge 1 commit intoapache:mainfrom
Conversation
jscheffl
requested review from
bugraoz93,
jedcunningham,
potiuk,
shahar1 and
vincbeck
as code owners
boring-cyborg
bot
added
area:dev-tools
backport-to-v3-1-test
Member
jedcunningham
left a comment
jedcunningham
left a comment
There was a problem hiding this comment.
Neither date command works on a mac 🤷 (I always do the vote email prep in a linux container).
| ``` | ||
| Thank you everyone. New Helm-Chart is released. | ||
|
|
||
| I invite everyone to help improve providers for the next release, a list of open issues can be found [here](https://github.com/apache/airflow/issues?q=is%3Aissue%20state%3Aopen%20label%3Aarea%3Ahelm-chart). |
Member
There was a problem hiding this comment.
Suggested change
| I invite everyone to help improve providers for the next release, a list of open issues can be found [here](https://github.com/apache/airflow/issues?q=is%3Aissue%20state%3Aopen%20label%3Aarea%3Ahelm-chart). | |
| I invite everyone to help improve the chart for the next release, a list of open issues can be found [here](https://github.com/apache/airflow/issues?q=is%3Aissue%20state%3Aopen%20label%3Aarea%3Ahelm-chart). |
Contributor
There was a problem hiding this comment.
Context switching at release level :D
|
|
||
| ```shell | ||
| curl -s https://airflow.apache.org/index.yaml | yq e '.entries.airflow[0].version' - | ||
| curl -s https://airflow.apache.org/index.yaml | yq -e '.entries.airflow[0].version' - |
Member
There was a problem hiding this comment.
e is for eval, not -e which sets the exit status (not really necessary in this context).
It can also be dropped.
Suggested change
| curl -s https://airflow.apache.org/index.yaml | yq -e '.entries.airflow[0].version' - | |
| curl -s https://airflow.apache.org/index.yaml | yq '.entries.airflow[0].version' - |
|
|
||
| For this to make use the following: | ||
|
|
||
| - Draft a new release in https://github.com/apache/airflow/releases/new |
Member
There was a problem hiding this comment.
Suggested change
| - Draft a new release in https://github.com/apache/airflow/releases/new | |
| - Create a new release in https://github.com/apache/airflow/releases/new |
Let's not use "Draft" since you can actually create a draft release :)
|
|
||
| # Create new folder for the release | ||
| cd airflow-release/helm-chart | ||
| cd asf-dist/release/airflow/helm-chart |
Member
There was a problem hiding this comment.
You missed the peer of this in the "Remove old releases" section.
Comment on lines
+940
to
+941
| For this to make use the following: | ||
|
|
Member
There was a problem hiding this comment.
Suggested change
| For this to make use the following: |
We don't need it.
eladkal
added
changelog:skip
potiuk
reviewed
Mar 23, 2026
| export IS_SHORTEN_VOTE=$([ $VOTE_DURATION_IN_HOURS -ge 72 ] && echo "false" || echo "true") | ||
| export SHORTEN_VOTE_TEXT="This is a shortened ($VOTE_DURATION_IN_HOURS hours vote) as agreed by policy set it https://lists.apache.org/thread/cv194w1fqqykrhswhmm54zy9gnnv6kgm" | ||
| export VOTE_END_TIME=$(LANG=en_US.UTF-8 TZ=UTC date -v+"${VOTE_DURATION_IN_HOURS}"H "+%B %d, %Y %H:%M %p") | ||
| export VOTE_END_TIME=$(date --utc -d "now + $VOTE_DURATION_IN_HOURS hours + 10 minutes" +'%Y-%m-%d %H:%M') |
Contributor
There was a problem hiding this comment.
It doesn't
➜ ~ export VOTE_END_TIME=$(date --utc -d "now + $VOTE_DURATION_IN_HOURS hours + 10 minutes" +'%Y-%m-%d %H:%M')
date: illegal option -- -
usage: date [-jnRu] [-I[date|hours|minutes|seconds|ns]] [-f input_fmt]
[ -z output_zone ] [-r filename|seconds] [-v[+|-]val[y|m|w|d|H|M|S]]
[[[[mm]dd]HH]MM[[cc]yy][.SS] | new_date] [+output_fmt]
Contributor
There was a problem hiding this comment.
Works in linux (deb). Maybe we can utilise uname and execute accordingly
Contributor
There was a problem hiding this comment.
Tried with Claude :) Haven't verified through a Mac yet but it metarilised what I mentioned above
OS=$(uname)
if [ "$OS" = "Darwin" ]; then
# macOS (BSD date)
export VOTE_END_TIME=$(date -u -v "+${VOTE_DURATION_IN_HOURS}H" -v "+10M" +'%Y-%m-%d %H:%M')
elif [ "$OS" = "Linux" ]; then
# Linux (GNU date)
export VOTE_END_TIME=$(date --utc -d "now + $VOTE_DURATION_IN_HOURS hours + 10 minutes" +'%Y-%m-%d %H:%M')
else
echo "Unsupported OS: $OS" >&2
exit 1
fi
potiuk
approved these changes
Mar 23, 2026
Member
potiuk
left a comment
potiuk
left a comment
There was a problem hiding this comment.
Approve with comments from Jed
bugraoz93
approved these changes
Mar 23, 2026
Contributor
bugraoz93
left a comment
bugraoz93
left a comment
There was a problem hiding this comment.
Looks good with comments
| ``` | ||
| Thank you everyone. New Helm-Chart is released. | ||
|
|
||
| I invite everyone to help improve providers for the next release, a list of open issues can be found [here](https://github.com/apache/airflow/issues?q=is%3Aissue%20state%3Aopen%20label%3Aarea%3Ahelm-chart). |
Contributor
There was a problem hiding this comment.
Context switching at release level :D
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Like usual with each release you find some glitches... with this PR some small updates in the docs for Helm chart release for the next release manager.
Note I also stumbled multiple times over the "date" command for voting period which might be working on mac only, not on Linuix... but the one in Helm release works, so replicating this back to providers guide...
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.