Skip to content

Conversation

@chensuyue
Copy link
Contributor

@chensuyue chensuyue commented Feb 11, 2026

Description

enable new cpu pool

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Performance improvement
  • Code refactoring
  • Other (please specify):

Related Issues

Fixes or relates to #

Checklist Before Submitting

  • My code has been tested locally.
  • Documentation has been updated as needed.
  • New or updated tests are included where applicable.

Signed-off-by: chensuyue <suyue.chen@intel.com>
Copilot AI review requested due to automatic review settings February 11, 2026 06:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates Azure Pipelines configuration to run unit tests on a new CPU pool and adds optional NUMA-aware execution for UT runs, plus tweaks the dev Docker image’s default UID/GID.

Changes:

  • Switch Azure Pipelines unit-test pool to EMR-28C.
  • Prefix UT execution with an optional $NUMA_CMD inside the docker exec command (and log its value).
  • Change default USER_ID/GROUP_ID build args in the dev Dockerfile from 1000 to 1001.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
.azure-pipelines/unit-test.yml Moves unit tests to the EMR-28C agent pool.
.azure-pipelines/template/ut-template.yml Adds NUMA command prefixing for UT execution in the container.
.azure-pipelines/docker/Dockerfile.devel Adjusts default user/group IDs used to create hostuser.

Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
Signed-off-by: chensuyue <suyue.chen@intel.com>
@chensuyue chensuyue changed the title test for new cpu pool enable new cpu pool Feb 12, 2026
@chensuyue chensuyue requested a review from Copilot February 12, 2026 02:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Signed-off-by: chensuyue <suyue.chen@intel.com>
@chensuyue chensuyue merged commit 0c7f4af into main Feb 12, 2026
9 of 14 checks passed
@chensuyue chensuyue deleted the suyue/ci branch February 12, 2026 03:57
chensuyue added a commit that referenced this pull request Feb 12, 2026
Signed-off-by: chensuyue <suyue.chen@intel.com>
(cherry picked from commit 0c7f4af)
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