Skip to content

PLT-1362 feat(deploy): add subpath support for git-based helm charts#149

Merged
uptickmetachu merged 2 commits intodevelopfrom
plt-1362/helm-chart-subpath
Mar 28, 2026
Merged

PLT-1362 feat(deploy): add subpath support for git-based helm charts#149
uptickmetachu merged 2 commits intodevelopfrom
plt-1362/helm-chart-subpath

Conversation

@uptickmetachu
Copy link
Copy Markdown
Collaborator

@uptickmetachu uptickmetachu commented Mar 28, 2026

Motivation

Git-based helm charts currently assume the chart is at the root of the repository. Some repos contain multiple helm charts in subdirectories, requiring a subpath to be specified.

PLT-1362

Changes

  • Added path attribute default for string chart definitions in Chart.__init__
  • Modified deploy logic to append chart.path to the cloned repo directory for both helm dependency build and helm secrets upgrade commands
  • Added test for git chart deployment with subpath

Usage in deployment.yml:

chart:
  type: git
  git_repo_url: https://github.com/org/repo
  git_sha: main
  path: charts/myapp  # optional subpath within the repo

Test plan

  • Existing deploy tests pass (no regression)
  • New test_deployer_git_with_subpath verifies correct path in helm commands
  • Manual test with a real git repo containing a chart in a subdirectory

🤖 Generated with Claude Code

uptickmetachu and others added 2 commits March 13, 2026 20:58
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Allow specifying a `path` in git chart definitions to deploy charts
located in subdirectories of a repository instead of only the root.

PLT-1362

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

Docker Images

Commit: 4016ee03424860ba72069b8be26309090c698dc9

Tag
610829907584.dkr.ecr.ap-southeast-2.amazonaws.com/gitops:test-4016ee0

@uptickmetachu uptickmetachu merged commit 09f8211 into develop Mar 28, 2026
3 checks passed
@uptickmetachu uptickmetachu deleted the plt-1362/helm-chart-subpath branch March 28, 2026 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant