Skip to content

feat(argocd-apps): add rawDirectory flag to opt out of helm rendering#245

Open
yreinhar wants to merge 1 commit into
mainfrom
feat/argocd-apps-raw-dir-source
Open

feat(argocd-apps): add rawDirectory flag to opt out of helm rendering#245
yreinhar wants to merge 1 commit into
mainfrom
feat/argocd-apps-raw-dir-source

Conversation

@yreinhar

Copy link
Copy Markdown

When rawDirectory=true in the project git config, the ArgoCD Application uses a plain directory source instead of Helm. Needed for sources that contain raw Kubernetes YAML (e.g. ApplicationSets) rather than a chart.

When rawDirectory=true in the project git config, the ArgoCD Application
uses a plain directory source instead of Helm. Needed for sources that
contain raw Kubernetes YAML (e.g. ApplicationSets) rather than a chart.
path: {{ $project.git.path | default $projectName }}
repoURL: {{ $project.git.repoUrl }}
targetRevision: {{ $project.git.branch | default "main" }}
{{- if $project.git.rawDirectory }}

@hoeftale hoeftale Jun 24, 2026

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.

If you do not need specific directory settings there is no need to set the directory directive explictily. "recurse: false" is the default.
If you need directory settings, please support setting these via values, e.g. $project.directorySettings

If you do not need for directory settings, it would be enough to omit the helm directive below. In this case please guard the .helm directive via "if $project.tofuValues"

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.

@hoeftale hoeftale 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.

see my comment

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