sylva-units: add result caching to "unit-def" named template (!7513) · Merge requests · Sylva-projects / sylva-core

Full credits to @priyagoyal for the idea.

This MR introduces caching of the results of the unit-def named template.

This change brings a huge performance improvement 🎉 : approximately 3x speed in my personal benchmark (helm template with the value from my own dev env mgmt cluster), where the render time goes from 9s down to 3s (a few weeks ago, before the other perf improvements, I was closer to 25-30s). 🤩

Implementation details

This MR:

adds caching of the results of the unit-def named template, results of unit definition are stored under .Values._unit_def_cache
.Values._unit_def_cache is cleared before dumping interpreted values in the sylva-units-values Secret.

(The initial code was written by Claude Sonet 4.5 because I was curious to quickly see how much we would win (prompt was: "take the following function, adding a result cache (should be stored under $envAll.Values._processed_units)"; the final code is close to this, with only minor cleanups).

Test

Testing done by diff'ing rendered manifests, "before" vs "after".

Review

💡 This MR is nicer to review one commit at a time:

the first commit has the "add caching part" part
the second commit is only indentation
the third commit is a rename

CI configuration

Below you can choose test deployment variants to run in this MR's CI.

Click to open to CI configuration

Legend:

Icon	Meaning	Available values
☁️	Infra Provider	`capd`, `capo`, `capm3`
🚀	Bootstrap Provider	`kubeadm` (alias `kadm`), `rke2`, `okd`, `ck8s`
🐧	Node OS	`ubuntu`, `suse`, `na`, `leapmicro`
🛠️	Deployment Options	Deployment option list and description
🎬	Pipeline Scenarios	Available scenario list and description
🟢	Enabled units	Any available units name, by default apply to management and workload cluster. Can be prefixed by `mgmt:` or `wkld:` to be applied only to a specific cluster type
🔴	Disabled units	Any available units name, by default apply to management and workload cluster. Can be prefixed by `mgmt:` or `wkld:` to be applied only to a specific cluster type
🏗️	Target platform	Can be used to select specific deployment environment Available platform list and description

Global config for deployment pipelines

autorun pipelines
allow failure on pipelines
record sylvactl events

Notes:

Enabling autorun will make deployment pipelines to be run automatically without human interaction
Disabling allow failure will make deployment pipelines mandatory for pipeline success.
if both autorun and allow failure are disabled, deployment pipelines will need manual triggering but will be blocking the pipeline

Be aware: after configuration change, pipeline is not triggered automatically. Please run it manually (by clicking the run pipeline button in Pipelines tab) or push new code.

Edited Apr 10, 2026 by Thomas Morin

sylva-units: add result caching to "unit-def" named template

Implementation details

Test

Review

CI configuration

Global config for deployment pipelines

Merge request reports