ci(invocation): register Helm chart release - #1386
Conversation
Register the invocation chart as a release subproject so chart changes produce path-scoped tags and feed the self-managed stack pin workflow. Refs #1385 Signed-off-by: Stephanie Baum <sbaum@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 12 included reviews per hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe release catalog now includes the HTTP invocation Helm chart. A regression test verifies its path, published name, deployment mapping, and ChangesHTTP invocation release lineage
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This change registers the HTTP invocation Helm chart for path-scoped releases and adds coverage for the catalog entry; no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Linked Issues checkExplanation The changes satisfy the coding objectives in issue Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 1 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Comment |
TL;DR
Register the HTTP invocation Helm chart as a release subproject so chart changes publish path-scoped releases and can advance the self-managed stack pin.
Additional Details
The release catalog registered the
http-invocationservice but omitted its chart underdeploy/helm/http-invocation. Chart changes after the existingdeploy/helm/http-invocation/v1.5.6release therefore had no chart release lane, leaving the self-managed stack on1.5.6.This change adds the chart with its published package name, maps it to the
http-invocationservice, and adds a regression test for the path, package name, deployment mapping, and existing tag lineage.Customer release notes: Not customer visible.
Plan summary: Not applicable.
Usage: Not applicable.
Notes: After merge, release automation should publish the pending invocation chart release. The chart release should then refresh #1381 with the invocation pin before that pull request is merged.
Related pull requests: #1371, #1381.
Dependencies: None. No license review or NOTICE update is required.
For the Reviewer
Please verify the
http-invocation-helmidentity and itsdeploysrelationship intools/ci/github-release-subprojects.json.For QA
QA is not needed beyond CI release validation.
Tests run:
python3 -m json.tool tools/ci/github-release-subprojects.jsonpython3 tools/ci/test-github-release.py(47 tests passed)git diff --checkIssues
Closes #1385
Checklist
Summary by CodeRabbit
New Features
Tests