Skip to content

Add helm to the smoke-test matrix#15554

Open
casey-robertson-paypal wants to merge 1 commit into
dependabot:mainfrom
casey-robertson-paypal:helm-smoke-matrix
Open

Add helm to the smoke-test matrix#15554
casey-robertson-paypal wants to merge 1 commit into
dependabot:mainfrom
casey-robertson-paypal:helm-smoke-matrix

Conversation

@casey-robertson-paypal

@casey-robertson-paypal casey-robertson-paypal commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #15218 (helm versioning-strategy support). .github/smoke-filters.yml already routes helm/** changes to the helm suite, but .github/smoke-matrix.json had no helm entry, so no smoke test actually ran for helm changes. This adds it.

Pairs with dependabot/smoke-tests#547, which adds tests/smoke-helm.yaml — a recording that exercises an operator-preserving update (^3.0.0^3.13.1, not exact-pinned) and replays at 100% cache locally. The dependabot-updater-helm image is published, so the suite has everything it needs once the test file merges.

Note: the failing Check for spelling errors (codespell) check is pre-existing on main and unrelated to this diff — it flags independ in a comment in python/lib/dependabot/python/dependency_grapher/requirements_layers.rb:47 (introduced by #15521). This PR only touches .github/smoke-matrix.json. It'll clear on rebase once that's fixed upstream.

Wires the helm ecosystem into the smoke suite now that versioning-strategy
support landed (dependabot#15218). Pairs with the smoke-helm.yaml test in
dependabot/smoke-tests (dependabot/smoke-tests#547).
@casey-robertson-paypal casey-robertson-paypal requested a review from a team as a code owner July 10, 2026 17:41
Copilot AI review requested due to automatic review settings July 10, 2026 17:41

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

Adds the Helm ecosystem to the Dependabot Core smoke-test suite matrix so that changes under helm/** actually trigger an E2E smoke run via the existing .github/smoke-filters.yml routing.

Changes:

  • Add a helm entry to .github/smoke-matrix.json with core, test, and ecosystem all set to "helm".

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