Skip to content

[helm] Rewrite README to point to website docs#2846

Merged
swuferhong merged 2 commits into
apache:mainfrom
affo:la-helm-readme
Mar 17, 2026
Merged

[helm] Rewrite README to point to website docs#2846
swuferhong merged 2 commits into
apache:mainfrom
affo:la-helm-readme

Conversation

@affo

@affo affo commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #2840

Refactor README to point to official docs to avoid double-edits on changes.

Brief change log

Rewrite the README to point to website docs.

I checked all sections, and the information in the README was either redundant or obsolete compared to the docs.

I would say this is the way to go.

Furthermore I added a section on how to run the tests we recently introduced and contribution guidelines (add tests :) )

Tests

API and Format

Documentation

Comment thread helm/README.md Outdated
Comment thread helm/README.md Outdated
Comment thread helm/README.md Outdated
@affo

affo commented Mar 12, 2026

Copy link
Copy Markdown
Contributor Author

@morazow thanks for the review!

I opted for a compromise:

In Development I point to the latest built doc website.

In Contributing I added a paragraph that points to the website doc file in the repo 🤝

@morazow morazow left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good 🚀

@swuferhong swuferhong left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@swuferhong swuferhong merged commit aa5d166 into apache:main Mar 17, 2026
1 check passed
wxplovecc pushed a commit to tongcheng-elong/fluss that referenced this pull request Apr 17, 2026
wxplovecc pushed a commit to tongcheng-elong/fluss that referenced this pull request Apr 20, 2026
polyzos pushed a commit that referenced this pull request Apr 23, 2026
* [helm] Enable pulling from private Docker registry (#2692)

* [helm] Enable pulling from private Docker registry

Added instructions for using a private Docker registry and included image values reference.

---------

Co-authored-by: xx789 <348448708@qq.com>
(cherry picked from commit 43f76a5)

* [helm] Fix wrong resource name in coordinator sts (#2834)

(cherry picked from commit 2c49fbc)

* [helm] Add CI workflow to run Helm tests (#2777)

* [helm] Add CI workflow to run Helm tests

(cherry picked from commit dd181eb)

* [Helm] Chart Component Configuration Isolation (#2472)

* Helm Chart Component Configuration Isolation

* Retrigger CI tests

* Retriggering CI/CD build pipeline

(cherry picked from commit d956e75)

* [Helm] Revert Chart Component Configuration Isolation (#2472) (#2863)

This reverts commit d956e75.

(cherry picked from commit 8cd9a6f)

* [helm] Enable SASL authentication configurations (#2506)

(cherry picked from commit f7e4498)

* [helm] Rewrite README to point to website docs (#2846)

(cherry picked from commit aa5d166)

* [helm][hotfix] Go template whitespace trimming caused exceptions (#2893)

* [helm][hotfix] Go template whitespace trimming caused exceptions

* Update and use without trimming

(cherry picked from commit 22ece48)

* [helm] Enable metrics reporting in helm charts (#2711)

---

Co-authored-by: Lorenzo Affetti <lorenzo.affetti@gmail.com>
(cherry picked from commit 2b207a4)

* [helm] Fix .helmignore to not package tests (#2847)

(cherry picked from commit d4748f2)

* [helm] Enable SASL authenticated connection to Zookeeper nodes (#2700)

---

Co-authored-by: Lorenzo Affetti <lorenzo.affetti@gmail.com>
(cherry picked from commit bdbbbce)

* [helm] Fix Zookeeper client config path (#3015)

(cherry picked from commit e9bfd72)

* [helm] Fix wrong resource name in coordinator sts (#3044)

(cherry picked from commit 8df3873)

* [helm] Add extraVolumes, extraVolumeMounts, initContainers (#3034)

(cherry picked from commit 17f5400)

---------

Co-authored-by: Lorenzo Affetti <lorenzo.affetti@gmail.com>
Co-authored-by: xx789 <348448708@qq.com>
Co-authored-by: Hemanth Savasere <hemanth.savasere@gmail.com>
Ugbot pushed a commit to Ugbot/fluss that referenced this pull request Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove redundant configuration specification in Helm README

3 participants