Conversation
|
Need to decide if we are happy with no auth to start with here? |
If we are not going to store fragile data in the cache, I'm ok with no auth + network policy that will allow connections to the cache from the Forge deployment only. |
Co-authored-by: PPawlowski <ppawlowski@users.noreply.github.com>
6717f22 to
4326bd0
Compare
|
@ppawlowski i rebased this on main to resolve some merge conflicts, can you double check the |
|
@ppawlowski I'm going to merge this, we should be aware this will deploy Valkey (even with a single Forge app instance) when the CloudProject helm version releases are updated. |
Go for it. |
|
I've added a valkey values template guard to ensure that proper valkey configuration is always present, depending on the provided values. Feel free to accept or reject it - it does not affect the overall feature implementation. |
fixes #699
Description
Adds a Valkey instance if more than 2 Forge app instances are configured.
Related Issue(s)
#699
FlowFuse/flowfuse#3642
Checklist
flowforge.yml?FlowFuse/helmto update ConfigMap TemplateFlowFuse/CloudProjectto update values for Staging/ProductionLabels
area:migrationlabel