Skip to content

Commit

Permalink
Move a nodejs test to a different machine pool (#17970)
Browse files Browse the repository at this point in the history
### Description
This is a temp fix for the failing "Zip-Nuget-Java-Nodejs Packaging
Pipeline". The pipeline is failing because I removed NodeJS from the
build machine pool's image, to reduce the number of dependencies we need
to maintain in VMs.
So this PR will temporarily move the test to a different machine pool to
get the test passed. Then I will move the test to docker. Docker images
are relatively easier to update and maintain. Now we almost run all
Linux test in docker, except for this one. Moving it to docker is needed
for enabling GPU support in nodejs, because all our Linux VMs do not
have CUDA.


### Motivation and Context
  • Loading branch information
snnn committed Nov 1, 2023
1 parent 30feae9 commit cb365f4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
parameters:
AgentPool: 'onnxruntime-Ubuntu2004-AMD-CPU'
AgentPool: 'Azure-Pipelines-EO-Ubuntu-2004-aiinfra'
StageSuffix: ''
stages:
- stage: Nodejs_Test_${{ parameters.StageSuffix }}
Expand All @@ -18,4 +18,4 @@ stages:
value: '$(Build.BinariesDirectory)'
steps:
- template: test.yml


Original file line number Diff line number Diff line change
Expand Up @@ -804,7 +804,7 @@ stages:

- template: ../nodejs/templates/test_linux.yml
parameters:
AgentPool : 'onnxruntime-Ubuntu2004-AMD-CPU'
AgentPool : 'Azure-Pipelines-EO-Ubuntu-2004-aiinfra'
StageSuffix : 'Linux_CPU_x64'

- template: ../nodejs/templates/test_macos.yml
Expand Down

0 comments on commit cb365f4

Please sign in to comment.