fix(ci): refresh staging QA source for libssh2 - #8959
Conversation
|
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)
📝 WalkthroughWalkthroughThe staging QA workflow now references the ChangesStaging QA source update
Estimated code review effort: 1 (Trivial) | ~5 minutes Mergeability Score: ⚪ Minimal · up to This localized change refreshes the staging QA source and matching integration assertion without changing public behavior or supported workflows; no actionable merge-blocking risk remains after normal checks. Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Code Coverage OverviewLanguages: TypeScript TypeScript / code-coverage/pluginThe overall coverage in commit 7445324 in the TypeScript / code-coverage/cliThe overall coverage in commit 7445324 in the Show a code coverage summary of the most impacted files.
Updated |
PR Review Advisor — InformationalAdvisor assessment: Informational / low confidence Model lanes
Second-opinion terminology and E2E selections are advisory. Live E2E does not run automatically for pull requests. E2E guidanceAdvisory only. A maintainer can dispatch the default E2E suite for the commit under review. Recommended E2E: None Manual-only E2E: This automated review informs maintainers. Warnings and suggestions do not require a response. A maintainer decides whether to merge. |
cv
left a comment
There was a problem hiding this comment.
The immutable staging source now carries the libssh2 nemoclaw2 inventory, the workflow assertion pins the same reviewed commit, and managed-image staging QA passed on 7445324. The completed advisor reported no findings. No security, correctness, or documentation issue remains.
prekshivyas
left a comment
There was a problem hiding this comment.
Approved based on code-diff review. Required GitHub checks must pass before merge.
Summary
Refresh the Deep Agents Code staging QA source from
af2a73f…to PR #8941's merge commit,ce96811…. The rebuilt staging base now contains thelibssh2nemoclaw2inventory required by current candidate images, so unrelated PRs such as #8903 no longer inherit this mismatch frommain.Changes
STAGING_QA_SOURCE_SHAtoce96811ddb418ad01c040521a1fe912b5bcb405e.nemoclaw2contract.libssh2contract, while the staging QA source remained on the earliernemoclaw1inventory.Type of Change
Quality Gates
7445324bfacross all nine security categories. The full commit SHA remains immutable, checkout credentials remain disabled, permissions remaincontents: read, and identity failures still stop the job.Documentation Writer Review
no-docs-neededDGX Station Hardware Evidence
scripts/prepare-dgx-station-host.shis unchanged.Verification
Signed-off-by:line and every commit appears asVerifiedin GitHubpre-commit,commit-msg, andpre-pushhooks passed, ornpm run validate:prpassed after refreshingorigin/mainwhen hooks were skipped or unavailablenpx vitest run --project integration test/managed-image-publication-workflow.test.tspassed 19 tests.npm testfor broad runtime/test-harness changes;npm run checkfor repo-wide validation/coverage changes — not applicable to this focused workflow pin update.npm run docsbuilds without warnings (doc changes only)Signed-off-by: San Dang sdang@nvidia.com
Summary by CodeRabbit
Chores
Tests