NationTech/harmony
3
0
Fork 2
Code Issues 38 Pull Requests 29 Actions Packages Projects Releases 1 Wiki Activity

feat(helm): roll-forward upgrades for pinned releases #305

Merged
johnride merged 1 commits from feat/helm-rollforward-min into master 2026-05-29 00:21:25 +00:00
Conversation 0 Commits 1 Files Changed 1 +7 -7
1 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
harmony/src/modules/helm/chart.rs
View File

@@ -223,18 +223,18 @@ impl<T: Topology + HelmCommand> Interpret<T> for HelmChartInterpret {
self.score.release_name self.score.release_name
))); )));
} }
// Pinned-version safety net: if the score pins a *different* // install_only=false is explicit upgrade mode; a pinned
// version than what's installed, refuse to silently // version change is an intentional roll-forward (ADR-012-2).
// upgrade/downgrade — that's a manual decision. // Apply it — helm fails loudly if convergence fails; no
// auto-rollback.
if let Some(expected) = self.expected_chart_field() if let Some(expected) = self.expected_chart_field()
&& Self::normalize_chart_field(&expected) && Self::normalize_chart_field(&expected)
!= Self::normalize_chart_field(&installed_chart) != Self::normalize_chart_field(&installed_chart)
{ {
return Err(InterpretError::new(format!( warn!(
"Helm release '{}' already installed as '{}', but score requests '{}'. \ "Helm release '{}' installed as '{}'; rolling forward to '{}'.",
Refusing to upgrade/downgrade; resolve manually.",
self.score.release_name, installed_chart, expected self.score.release_name, installed_chart, expected
))); );
} }
// Otherwise (no pin, or pinned and matching) fall through to // Otherwise (no pin, or pinned and matching) fall through to
// `helm upgrade --install`. Helm is the source of truth on // `helm upgrade --install`. Helm is the source of truth on