Skip to content

Add instructions for modifying storage class names - #104

Merged
brucetony merged 1 commit into
masterfrom
storage-class-names
Jul 20, 2026
Merged

Add instructions for modifying storage class names#104
brucetony merged 1 commit into
masterfrom
storage-class-names

Conversation

@brucetony

@brucetony brucetony commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

For the flame-node helm chart

Summary by CodeRabbit

  • Documentation
    • Added installation guidance for configuring storage classes across supported Helm components.
    • Included a minimal configuration example for setting storage class names.
    • Updated the optional Node Data Store example to use SeaweedFS instead of MinIO, including revised credentials configuration.

@brucetony brucetony linked an issue Jul 20, 2026 that may be closed by this pull request
@brucetony
brucetony merged commit 94b9c06 into master Jul 20, 2026
1 check passed
@brucetony
brucetony deleted the storage-class-names branch July 20, 2026 11:25
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a8e4d324-2c80-4a66-8d35-34ff145c2aab

📥 Commits

Reviewing files that changed from the base of the PR and between 8f4899a and 74c9cd6.

📒 Files selected for processing (1)
  • src/guide/deployment/node-installation.md

📝 Walkthrough

Walkthrough

The deployment installation guide adds shared storage class configuration guidance for persistent services and changes the node data store example from MinIO to SeaweedFS with updated credential fields.

Changes

Deployment guidance

Layer / File(s) Summary
Storage class configuration
src/guide/deployment/node-installation.md
Adds a my-values.yaml example defining storage classes for PostgreSQL, Kong PostgreSQL, Keycloak PostgreSQL, MongoDB, SeaweedFS, optional Victorialogs, and optional Blaze FHIR storage.
SeaweedFS data store example
src/guide/deployment/node-installation.md
Updates the node data store example to use SeaweedFS S3 settings and adminUser/adminPassword credentials instead of MinIO fields.

Estimated code review effort: 1 (Trivial) | ~3 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch storage-class-names

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

Add page on creating storage class

1 participant