Skip to content

Commit

Permalink
SPMI: Always enable venv for coreclr_tests.run collection (#107133)
Browse files Browse the repository at this point in the history
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()`.
  • Loading branch information
jakobbotsch authored Aug 30, 2024
1 parent 35c793a commit ee5a4bb
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions eng/pipelines/common/templates/runtimes/run-test-job.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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'
Expand Down

0 comments on commit ee5a4bb

Please sign in to comment.