Skip to content

docs(agents): document stale Helm subchart cleanup#1957

Merged
elezar merged 1 commit into
mainfrom
skill-helm-lint-stale-deps-pr-1947
Jun 22, 2026
Merged

docs(agents): document stale Helm subchart cleanup#1957
elezar merged 1 commit into
mainfrom
skill-helm-lint-stale-deps-pr-1947

Conversation

@elezar

@elezar elezar commented Jun 18, 2026

Copy link
Copy Markdown
Member

Summary

Document how agents should handle stale Helm subchart artifacts when helm lint reports undeclared dependencies, using the postgresql cleanup path from PR #1947.

Related Issue

Related: #1947

Changes

  • Added a Helm chart checks section to the Helm dev environment skill.
  • Included targeted cleanup commands for stale deploy/helm/openshell/charts/postgresql artifacts.
  • Clarified that charts/ is ignored and regenerated by helm dependency build for declared dependencies.

Testing

  • mise run pre-commit passes - fails in this environment on unrelated openshell-supervisor-process current user/group lookup tests.
  • Unit tests added/updated - not applicable; docs-only skill update.
  • E2E tests added/updated (if applicable) - not applicable.
  • mise run helm:lint passes.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable) - not applicable; agent skill documentation only.

Refs: #1947
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Jun 18, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@elezar elezar enabled auto-merge (squash) June 22, 2026 11:51
@elezar

elezar commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

/ok-to-test 9bf63e1

@elezar elezar merged commit 8d02733 into main Jun 22, 2026
30 checks passed
@elezar elezar deleted the skill-helm-lint-stale-deps-pr-1947 branch June 22, 2026 12:00
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