-
Notifications
You must be signed in to change notification settings - Fork 78
enable new cpu pool #1435
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
enable new cpu pool #1435
Conversation
Signed-off-by: chensuyue <suyue.chen@intel.com>
There was a problem hiding this 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_CMDinside thedocker execcommand (and log its value). - Change default
USER_ID/GROUP_IDbuild args in the dev Dockerfile from1000to1001.
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>
There was a problem hiding this 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>
Signed-off-by: chensuyue <suyue.chen@intel.com> (cherry picked from commit 0c7f4af)
Description
enable new cpu pool
Type of Change
Related Issues
Fixes or relates to #
Checklist Before Submitting