Skip to content

feat: switch valkey dependency to official valkey-io chart#939

Merged
ppawlowski merged 5 commits into
mainfrom
chore-valkey-switch
Jun 24, 2026
Merged

feat: switch valkey dependency to official valkey-io chart#939
ppawlowski merged 5 commits into
mainfrom
chore-valkey-switch

Conversation

@ppawlowski

@ppawlowski ppawlowski commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Description

This pull request replaces the valkey chart dependency from CloudPirates (oci://registry-1.docker.io/cloudpirates, version 0.15.3) with the official valkey-io/valkey-helm chart (https://valkey.io/valkey-helm/, version 0.9.4).

Motivation: CloudPirates have announced the deprecation of their Valkey chart in CloudPirates-io/helm-charts#1083 and recommend the upstream chart as the replacement.

NOTE: the information below about chart behaviour with default and custom values will be added to the README file after the release.

Behaviour for default values

  • forge.localValkey: true with the valkey: block at defaults: transparent upgrade.
  • The workload kind changes StatefulSetDeployment (the official chart uses a Deployment for standalone mode).
  • Service name, port, and DNS hostname are unchanged (<release>-valkey:6379).

Behaviour for custom valkey: overrides

Upstream uses a different values schema. Users who override keys under valkey: need to translate them:

Old (CloudPirates) New (valkey-io)
valkey.persistence.enabled valkey.dataStorage.enabled
valkey.persistence.size valkey.dataStorage.requestedSize
valkey.persistence.storageClass valkey.dataStorage.className
valkey.config.maxMemory line inside valkey.valkeyConfig raw string
valkey.config.maxMemoryPolicy line inside valkey.valkeyConfig raw string
valkey.config.save line inside valkey.valkeyConfig raw string
valkey.replicaCount valkey.replica.replicas (with valkey.replica.enabled: true)
valkey.architecture valkey.replica.enabled (sentinel mode is no longer supported)
valkey.auth.password valkey.auth.aclUsers.default.password (ACL model)

Related Issue(s)

Closes #882

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
  • Link to Changelog Entry PR, or note why one is not needed.

Labels

  • Includes a DB migration? -> add the area:migration label

@ppawlowski
ppawlowski requested a review from hardillb June 9, 2026 11:44
@hardillb

Copy link
Copy Markdown
Contributor

I'll try and test this today

@ppawlowski
ppawlowski merged commit af4641e into main Jun 24, 2026
11 of 12 checks passed
@ppawlowski
ppawlowski deleted the chore-valkey-switch branch June 24, 2026 05:56
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.

Investigate how to migrate the Valkey helm chart

2 participants