Skip to content

Add option to use local Helm values for apps - #775

Open
wtripp180901 wants to merge 1 commit into
silogen:mainfrom
stackhpc:feat/local-helm-values
Open

Add option to use local Helm values for apps#775
wtripp180901 wants to merge 1 commit into
silogen:mainfrom
stackhpc:feat/local-helm-values

Conversation

@wtripp180901

@wtripp180901 wtripp180901 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Currently the only way override Helm values in a way that the changes won't be reverted by ArgoCD is to use externalValues and reference an external repository or the local Gitea server containing an overrides files. For local development or when consuming cluster-forge dependencies as part of a manual Helm installation of AIWB/AIRM where a Gitea server is not a hard dependency, it is useful to be able to iterate on the configuration purely through upgrading the Helm release rather than pushing commits to a remote repository.

Adds localHelmValues.enabled option which disables the cluster-forge wrapper Application. This allows the user to deploy cluster-forge with local overrides e.g with helm install cluster-forge ./root --values values_<size>.yaml --values custom_overrides.yaml while ArgoCD still consumes the remote OCI artifacts and defaults for each of the constituent applications for a given revision of cluster-forge.

Needs docs but the docs/configuration-reference.md files referenced in CONTRIBUTING.md seems to be missing so unsure where the best place for these would be.

@wtripp180901
wtripp180901 requested a review from a team as a code owner July 10, 2026 14:46

@brownzebra brownzebra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@brownzebra
brownzebra enabled auto-merge July 23, 2026 12:54
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