Skip to content

ci: run helm dependency update before packaging charts#109

Merged
kupratyu-splunk merged 2 commits into
mainfrom
make_customers_exp_better
Jun 15, 2026
Merged

ci: run helm dependency update before packaging charts#109
kupratyu-splunk merged 2 commits into
mainfrom
make_customers_exp_better

Conversation

@kupratyu-splunk

Copy link
Copy Markdown
Collaborator

helm package fails when Chart.yaml declares dependencies but charts/ directory is missing the downloaded tarballs. Adding helm dependency update for each chart before packaging resolves the missing dependencies error.

Description

Related Issues

  • Related to #

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test improvement
  • CI/CD improvement
  • Chore (dependency updates, etc.)

Changes Made

Testing Performed

  • Unit tests pass (make test)
  • Linting passes (make lint)
  • Integration tests pass (if applicable)
  • E2E tests pass (if applicable)
  • Manual testing performed

Test Environment

  • Kubernetes Version:
  • Cloud Provider:
  • Deployment Method:

Test Steps

Documentation

  • Updated inline code comments
  • Updated README.md (if adding features)
  • Updated API documentation
  • Updated deployment guides
  • Updated CHANGELOG.md
  • No documentation needed

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published
  • I have updated the Helm chart version (if applicable)
  • I have updated CRD schemas (if applicable)

Breaking Changes

Impact:

Migration Path:

Screenshots/Recordings

Additional Notes

Reviewer Notes

Please pay special attention to:


Commit Message Convention: This PR follows Conventional Commits

helm package fails when Chart.yaml declares dependencies but charts/ directory
is missing the downloaded tarballs. Adding helm dependency update for each
chart before packaging resolves the missing dependencies error.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copilot AI 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.

Pull request overview

This PR updates the Helm chart release workflow to ensure chart dependencies are downloaded into each chart’s charts/ directory before running helm package, preventing packaging failures when Chart.yaml declares dependencies but the dependency tarballs are not present.

Changes:

  • Run helm dependency update for helm-chart/splunk-ai-operator prior to packaging.
  • Conditionally run helm dependency update for helm-chart/splunk-ai-platform (if the chart exists) prior to packaging.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/release-package-helm.yml
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@kupratyu-splunk kupratyu-splunk merged commit e204914 into main Jun 15, 2026
10 checks passed
@kupratyu-splunk kupratyu-splunk deleted the make_customers_exp_better branch June 15, 2026 15:43
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.

3 participants