Skip to content

ci: split release-container tests into separate job#2434

Open
poroh wants to merge 1 commit into
NVIDIA:mainfrom
poroh:separate-tests-ci-job
Open

ci: split release-container tests into separate job#2434
poroh wants to merge 1 commit into
NVIDIA:mainfrom
poroh:separate-tests-ci-job

Conversation

@poroh

@poroh poroh commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Description

Motivation: CI takes too long and big part of it is waiting while we build release and then run tests.

Move Rust test execution out of the release-container Docker image builds. The release Dockerfiles now run the build/package-only cargo-make task, while a new x86 CI job runs the release-container service test flow inside the private build container.

Keep the existing combined cargo-make task for compatibility, add separate build and test tasks, and wire the new test job into build summary, notifications, and core-ci-pass gating.

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Related Issues (Optional)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

@poroh poroh requested review from a team as code owners June 10, 2026 22:35
@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 4948d066-d5aa-41a7-8522-f5fb06e8b544

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

Move Rust test execution out of the release-container Docker image
builds.  The release Dockerfiles now run the build/package-only
cargo-make task, while a new x86 CI job runs the release-container
service test flow inside the private build container.

Keep the existing combined cargo-make task for compatibility, add
separate build and test tasks, and wire the new test job into build
summary, notifications, and core-ci-pass gating.

Signed-off-by: Dmitry Porokh <dporokh@nvidia.com>
@poroh poroh force-pushed the separate-tests-ci-job branch from 6169e53 to eeb0ff6 Compare June 10, 2026 22:42
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