Skip to content

Cleanup Helm Chart documentation#62544

Merged
jscheffl merged 9 commits intoapache:mainfrom
Miretpl:helm-chart-cleanup-docs
Mar 4, 2026
Merged

Cleanup Helm Chart documentation#62544
jscheffl merged 9 commits intoapache:mainfrom
Miretpl:helm-chart-cleanup-docs

Conversation

@Miretpl
Copy link
Contributor

@Miretpl Miretpl commented Feb 26, 2026

During searching of possible things missed in #61018, I found a couple of more things in the Helm Chart docs and decided to fix some things. The changes consist of:

  1. Rewriting some parts to be easier to follow
  2. Add warnings regarding security concerns in some places
  3. Make headers unified in terms of hierarchy
  4. Update some links
  5. In most cases, change webserver to apiServer
  6. Remove some duplicated information between different sections
  7. Remove information that KEDA is experimental (it is so long in the chart that I thought that maybe it is about time)
  8. Finally..., remove some missed parts in the mentioned PR 😄

Edit: Additional things done:

  1. Fix indent to match official Sphinx doc (3 spaces)
  2. Change kubernetes to Kubernetes
  3. Add a caption for almost all code blocks
  4. Add missing language for code blocks

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

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

Very cool rework! Wow.

@Miretpl Miretpl force-pushed the helm-chart-cleanup-docs branch 3 times, most recently from f6c60d9 to 58a79b2 Compare March 2, 2026 19:58
@Miretpl Miretpl force-pushed the helm-chart-cleanup-docs branch from 4fc6da7 to ae7aaff Compare March 2, 2026 22:42
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

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

Cool!

@jscheffl jscheffl merged commit 28c2a2a into apache:main Mar 4, 2026
129 checks passed
@Miretpl Miretpl deleted the helm-chart-cleanup-docs branch March 4, 2026 18:52
1Ninad pushed a commit to 1Ninad/airflow that referenced this pull request Mar 6, 2026
* Cleanup Helm Chart docs

* Change kubernetes to Kubernetes

* Fix indends & other formatting

* Specify missing language in code blocks

* Add caption instead of comments to code blocks

* Misc

* Fix keda doc

* Add pgbouncer to spelling wordlist

* Nits
dominikhei pushed a commit to dominikhei/airflow that referenced this pull request Mar 11, 2026
* Cleanup Helm Chart docs

* Change kubernetes to Kubernetes

* Fix indends & other formatting

* Specify missing language in code blocks

* Add caption instead of comments to code blocks

* Misc

* Fix keda doc

* Add pgbouncer to spelling wordlist

* Nits
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants