Skip to content

[docs] add minimum requirements for cluster configuration to helm guide#5802

Merged
anandbraman merged 1 commit intomainfrom
anandraman/fde-23-document-best-practices-for-persistent-storage
Mar 12, 2026
Merged

[docs] add minimum requirements for cluster configuration to helm guide#5802
anandbraman merged 1 commit intomainfrom
anandraman/fde-23-document-best-practices-for-persistent-storage

Conversation

@anandbraman
Copy link
Copy Markdown
Contributor

Describe Manual Test Plan

Checklist

  • Unit tests added/updated
  • Integration tests added/updated
  • Documentation updated
  • Changelog updated

Breaking Changes?

Mark if you think the answer is yes for any of these components:

Describe Incompatible Changes

@anandbraman anandbraman requested a review from gz March 11, 2026 20:00
@anandbraman anandbraman force-pushed the anandraman/fde-23-document-best-practices-for-persistent-storage branch from 4e1f15d to 8f50310 Compare March 11, 2026 20:01
@anandbraman anandbraman requested a review from ryzhyk March 11, 2026 20:03
@anandbraman anandbraman changed the title [docs] add minimum requirements for cluster configuration to helm guid [docs] add minimum requirements for cluster configuration to helm guide Mar 11, 2026
@anandbraman anandbraman force-pushed the anandraman/fde-23-document-best-practices-for-persistent-storage branch from 8f50310 to 3d62e59 Compare March 11, 2026 20:48
Copy link
Copy Markdown

@mythical-fred mythical-fred left a comment

Choose a reason for hiding this comment

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

LGTM. One nit: the link (operations/guide/#expand-existing-pipeline-storage) is a relative path while the memory link above uses an absolute path (/operations/memory). Worth making it /operations/guide/#expand-existing-pipeline-storage for consistency and to ensure it resolves correctly from any page.

### Kubernetes Cluster Configuration

#### Node Sizing
Nodes with 32-48 CPUs and at least 32 GiB of RAM are the recommended default node size. RAM requirements differ based on usage. For additional guidance on memory, refer to the [memory usage](/operations/memory) documentation.
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.

Clarify that these resources will be used across both the control plane and the pipelines.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Seems like a good opportunity to point out that you could use different nodes among the pipelines and the control plane with node selectors.

@anandbraman anandbraman force-pushed the anandraman/fde-23-document-best-practices-for-persistent-storage branch from 3d62e59 to 64d5ddf Compare March 11, 2026 22:29
@anandbraman anandbraman force-pushed the anandraman/fde-23-document-best-practices-for-persistent-storage branch from 64d5ddf to d804ba3 Compare March 12, 2026 00:02
@anandbraman anandbraman enabled auto-merge March 12, 2026 00:02
@anandbraman anandbraman added this pull request to the merge queue Mar 12, 2026
Merged via the queue into main with commit ae0b18e Mar 12, 2026
1 check passed
@anandbraman anandbraman deleted the anandraman/fde-23-document-best-practices-for-persistent-storage branch March 12, 2026 01:25
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.

3 participants