Skip to content

feat(seedbox): add Valkey persistence wiring (chart 0.4.0) - #77

Merged
michaelbeutler merged 2 commits into
mainfrom
feat/seedbox-valkey-persistence
Aug 18, 2026
Merged

feat(seedbox): add Valkey persistence wiring (chart 0.4.0)#77
michaelbeutler merged 2 commits into
mainfrom
feat/seedbox-valkey-persistence

Conversation

@niko-kriznik-globtim

@niko-kriznik-globtim niko-kriznik-globtim commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Bumps truvami-seedbox chart to 0.4.0
  • Adds seedbox.valkey config for route/battery checkpoint persistence
  • Injects VALKEY_PASSWORD from a Kubernetes secret when enabled
  • Allows Valkey egress on port 6379 when NetworkPolicy is enabled

Related

Test plan

  • helm template with demo values renders Valkey env + config
  • Demo rollout restores route/battery checkpoints after pod restart

Summary by CodeRabbit

  • New Features

    • Added optional Valkey support for persisting route and battery checkpoints.
    • Added configurable Valkey connection, authentication, and secret settings.
    • Added a configurable maximum battery voltage change per report.
    • Updated network access and container configuration when Valkey is enabled.
  • Documentation

    • Updated chart version information and documented the new configuration options.

Expose seedbox.valkey settings, inject VALKEY_PASSWORD from a secret, and
allow Valkey egress when NetworkPolicy is enabled.
@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 26d170e1-643a-4521-b6b3-034d2709d29d

📥 Commits

Reviewing files that changed from the base of the PR and between 1cbd56f and 9d49d2b.

📒 Files selected for processing (5)
  • charts/truvami-seedbox/Chart.yaml
  • charts/truvami-seedbox/README.md
  • charts/truvami-seedbox/templates/deployment.yaml
  • charts/truvami-seedbox/templates/networkpolicy.yaml
  • charts/truvami-seedbox/values.yaml

📝 Walkthrough

Walkthrough

The Helm chart version increases to 0.4.0. New values configure Valkey checkpoint persistence and battery voltage changes. Deployment templates inject the Valkey password from a Secret. Network policies allow Valkey traffic when enabled.

Changes

Seedbox chart configuration

Layer / File(s) Summary
Configuration and documentation
charts/truvami-seedbox/Chart.yaml, charts/truvami-seedbox/values.yaml, charts/truvami-seedbox/README.md
The chart version changes to 0.4.0. Values and documentation add Valkey checkpoint settings, secret configuration, and maxVoltageStep.
Secret and network wiring
charts/truvami-seedbox/templates/deployment.yaml, charts/truvami-seedbox/templates/networkpolicy.yaml
The deployment injects VALKEY_PASSWORD from a required Secret. The network policy allows TCP egress to Valkey on port 6379 when enabled.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: michaelbeutler

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/seedbox-valkey-persistence

Warning

Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption.


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

@niko-kriznik-globtim
niko-kriznik-globtim marked this pull request as ready for review August 18, 2026 13:26
@michaelbeutler
michaelbeutler merged commit 7c70cd7 into main Aug 18, 2026
1 check was pending
@michaelbeutler
michaelbeutler deleted the feat/seedbox-valkey-persistence branch August 18, 2026 13:27
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.

2 participants