Skip to content

chore(helm): document storageclass parameter#2909

Merged
M0NsTeRRR merged 2 commits intoseerr-team:developfrom
danjuv:fix/add-storageclass-helm-values
Apr 19, 2026
Merged

chore(helm): document storageclass parameter#2909
M0NsTeRRR merged 2 commits intoseerr-team:developfrom
danjuv:fix/add-storageclass-helm-values

Conversation

@danjuv
Copy link
Copy Markdown
Contributor

@danjuv danjuv commented Apr 19, 2026

Description

The storageClass parameter isn't listed in the values file, but does seem to be supported: https://github.com/seerr-team/seerr/blob/develop/charts/seerr-chart/templates/persistentvolumeclaim.yaml#L20.

How Has This Been Tested?

Run this on a k3s cluster

Screenshots / Logs (if applicable)

Checklist:

  • I have read and followed the contribution guidelines.
  • Disclosed any use of AI (see our policy)
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • Successful build pnpm build
  • Translation keys pnpm i18n:extract
  • Database migration (if required)

Summary by CodeRabbit

  • New Features

    • Added storage class configuration option for persistent volume claims, enabling selection of custom storage classes and granular control over dynamic provisioning behavior.
  • Documentation

    • Updated documentation to detail new storage class configuration options, providing guidance on custom storage class selection and dynamic provisioning control.
    • Helm chart version updated to 3.5.1.

@danjuv danjuv requested a review from a team as a code owner April 19, 2026 02:22
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 19, 2026

📝 Walkthrough

Walkthrough

Updated the seerr Helm chart from version 3.5.0 to 3.5.1. Added a new config.persistence.storageClass configuration field to enable users to specify custom storage classes for persistent volumes, with support for disabling dynamic provisioning. Updated README documentation accordingly.

Changes

Cohort / File(s) Summary
Helm Chart Configuration
charts/seerr-chart/README.md, charts/seerr-chart/values.yaml
Added new config.persistence.storageClass configuration option with documentation, including support for disabling dynamic provisioning via "-". Bumped chart version to 3.5.1 and corrected source code link path from charts/seerr to charts/seerr-chart.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested reviewers

  • M0NsTeRRR
  • fallenbagel

Poem

🐰 A storage class blooms in the values today,
With flexibility granted in each Helm display,
Dynamic or static, the choice now is thine,
Our persistent volumes will work out just fine! ✨

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore(helm): document storageclass parameter' accurately describes the main change—documenting the storageClass parameter in the Helm chart values.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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 and usage tips.

@danjuv danjuv changed the title Fix/add storageclass helm values fix: add storageclass parameter to chart values file Apr 19, 2026
@M0NsTeRRR M0NsTeRRR changed the title fix: add storageclass parameter to chart values file chore(helm): document storageclass parameter Apr 19, 2026
@seerr-automation-bot seerr-automation-bot added this to the v3.3.0 milestone Apr 19, 2026
@M0NsTeRRR M0NsTeRRR merged commit 22d3b0f into seerr-team:develop Apr 19, 2026
19 of 21 checks passed
lucianchauvin pushed a commit to lucianchauvin/jellyseerr that referenced this pull request Apr 20, 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.

4 participants