Skip to content

Conversation

@LipuFei
Copy link
Contributor

@LipuFei LipuFei commented Jul 19, 2025

The current k8s pod template in the Helm chart only uses webserver configs. It doesn't take into account the new api-server in 3.0.

This PR adds if-else checks. It uses the api-server configs in case of Airflow 3.0+, otherwise it falls back to webserver.

@boring-cyborg boring-cyborg bot added the area:helm-chart Airflow Helm Chart label Jul 19, 2025
@LipuFei LipuFei force-pushed the fix/chart-job-apiserver branch 4 times, most recently from 400abda to a276eeb Compare July 19, 2025 14:46
@eladkal eladkal added this to the Airflow Helm Chart 1.19.0 milestone Jul 21, 2025
@eladkal eladkal requested a review from romsharon98 July 21, 2025 07:14
@github-actions
Copy link

github-actions bot commented Sep 5, 2025

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Sep 5, 2025
@LipuFei LipuFei force-pushed the fix/chart-job-apiserver branch from a276eeb to 149737d Compare September 5, 2025 07:17
@LipuFei LipuFei force-pushed the fix/chart-job-apiserver branch from 149737d to 8671305 Compare September 5, 2025 07:19
@LipuFei
Copy link
Contributor Author

LipuFei commented Sep 5, 2025

Can someone review this? Thanks.

@github-actions github-actions bot removed the stale Stale PRs per the .github/workflows/stale.yml policy file label Sep 6, 2025
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@potiuk potiuk merged commit 09771cb into apache:main Sep 6, 2025
74 checks passed
mangal-vairalkar pushed a commit to mangal-vairalkar/airflow that referenced this pull request Sep 7, 2025
@LipuFei LipuFei deleted the fix/chart-job-apiserver branch September 8, 2025 13:24
RoyLee1224 pushed a commit to RoyLee1224/airflow that referenced this pull request Sep 8, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Sep 30, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 1, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 2, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 3, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 4, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 7, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 8, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 9, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 10, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 11, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 12, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 14, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 15, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 17, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:helm-chart Airflow Helm Chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants