ci: remove GitLab mirror + add nSpect/scan trigger on release/** (res… - #1806
Conversation
…t of #1795) The remaining changes from #1795, adapted to the release/** RC model: - Remove the GitLab bridge: build_validation.yml (repo mirror + .gitlab-ci.yml trigger) and mirror_repo.sh — superseded by the native ci.yml. - Add trigger-gitlab-nspect to ci.yml: on a push to a release/** branch (a PR merged into release/<x.y.z>), trigger the nixl-ci GitLab pipeline (PIPELINE_TYPE=rc) to run the wheel scan + nSpect registration (NSPECT-WO64-8O3P) against the wheels just uploaded to Artifactory. Runs on the gitlab_ci_runners group (internal reach); WHEEL_VERSION comes from the version job; NIXL_CI_REF selects the nixl-ci branch (defaults to main). (#1795 gated this on v*-rc tags; re-gated for the release/** model.) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThe PR removes the ChangesGitLab CI Infrastructure Replacement
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
#1806) …t of #1795) The remaining changes from #1795, adapted to the release/** RC model: - Remove the GitLab bridge: build_validation.yml (repo mirror + .gitlab-ci.yml trigger) and mirror_repo.sh — superseded by the native ci.yml. - Add trigger-gitlab-nspect to ci.yml: on a push to a release/** branch (a PR merged into release/<x.y.z>), trigger the nixl-ci GitLab pipeline (PIPELINE_TYPE=rc) to run the wheel scan + nSpect registration (NSPECT-WO64-8O3P) against the wheels just uploaded to Artifactory. Runs on the gitlab_ci_runners group (internal reach); WHEEL_VERSION comes from the version job; NIXL_CI_REF selects the nixl-ci branch (defaults to main). (#1795 gated this on v*-rc tags; re-gated for the release/** model.) ## What? _Describe what this PR is doing._ ## Why? _Justification for the PR. If there is an existing issue/bug, please reference it. For bug fixes, the 'Why?' and 'What?' can be merged into a single item._ ## How? _It is optional, but for complex PRs, please provide information about the design, architecture, approach, etc._ <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Removed deprecated validation workflow and repository mirror synchronization script * Enhanced release pipeline with automated nSpect registration and validation for published release builds <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…/** (res… (ai-dynamo#1806)" This reverts commit 6f08b8e.
…t of #1795)
The remaining changes from #1795, adapted to the release/** RC model:
What?
Describe what this PR is doing.
Why?
Justification for the PR. If there is an existing issue/bug, please reference it. For
bug fixes, the 'Why?' and 'What?' can be merged into a single item.
How?
It is optional, but for complex PRs, please provide information about the design,
architecture, approach, etc.
Summary by CodeRabbit