From 4b7fc464b6ada57f4194d3be90edd4aec29eff9c Mon Sep 17 00:00:00 2001 From: Jakob Botsch Nielsen Date: Thu, 29 Aug 2024 14:21:10 +0200 Subject: [PATCH] SPMI: Always enable venv for coreclr_tests.run collection Otherwise if the Helix run failed (because of a failing test) we won't run this step, which will cause the next steps to fail. This is currently happening for collections internally. The other places where we enable Python venv already have this `condition: always()`. --- eng/pipelines/common/templates/runtimes/run-test-job.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/eng/pipelines/common/templates/runtimes/run-test-job.yml b/eng/pipelines/common/templates/runtimes/run-test-job.yml index d33fa76d9504f..0201dae40d4d1 100644 --- a/eng/pipelines/common/templates/runtimes/run-test-job.yml +++ b/eng/pipelines/common/templates/runtimes/run-test-job.yml @@ -593,6 +593,7 @@ jobs: - script: $(PythonSetupScript) displayName: Enable python venv + condition: always() - script: $(PythonScript) $(Build.SourcesDirectory)/src/coreclr/scripts/superpmi.py merge-mch -log_level DEBUG -pattern $(MchFilesLocation)$(CollectionName).$(CollectionType)*.mch -output_mch_path $(MergedMchFileLocation)$(CollectionName).$(CollectionType).$(MchFileTag).mch displayName: 'Merge $(CollectionName)-$(CollectionType) SuperPMI collections'