未验证 提交 c8c08765 编写于 作者: A Ankit Jain 提交者: GitHub

[perf] Set job name for the performance jobs (#67124)

job.yml uses name instead of displayName to set the job's name. But perf jobs don't set that. Without this, pre-defined variables like System.PhaseName get value Job1.
This will help with differentiating builds in kusto.
上级 a6d2b103
......@@ -49,6 +49,12 @@ jobs:
pgoRunType: ${{ parameters.pgoRunType }}
javascriptEngine: ${{ parameters.javascriptEngine }}
iosLlvmBuild: ${{ parameters.iosLlvmBuild }}
${{ if and(eq(parameters.runtimeType, 'wasm'), eq(parameters.codeGenType, 'aot')) }}:
helixTypeSuffix: '/wasm/aot'
${{ if and(eq(parameters.runtimeType, 'wasm'), ne(parameters.codeGenType, 'aot')) }}:
helixTypeSuffix: '/wasm'
# Test job depends on the corresponding build job
dependsOn:
- ${{ if not(in(parameters.runtimeType, 'AndroidMono', 'iOSMono', 'wasm')) }}:
......
......@@ -23,6 +23,7 @@ parameters:
runKind: '' # required -- test category
logicalMachine: '' # required -- Used to specify a which pool of machines the test should run against
javascriptEngine: 'NoJS'
helixTypeSuffix: '' # optional -- appends to HelixType
jobs:
- template: xplat-pipeline-job.yml
......@@ -41,6 +42,7 @@ jobs:
displayName: '${{ parameters.displayName }}'
${{ if eq(parameters.displayName, '') }}:
displayName: '${{ parameters.jobName }}'
name: ${{ replace(replace(parameters.jobName, ' ', '_'), '-', '_') }}
timeoutInMinutes: ${{ parameters.timeoutInMinutes }}
......@@ -131,7 +133,7 @@ jobs:
- template: /eng/pipelines/coreclr/templates/perf-send-to-helix.yml
parameters:
HelixSource: '$(HelixSourcePrefix)/$(Build.Repository.Name)/$(Build.SourceBranch)' # sources must start with pr/, official/, prodcon/, or agent/
HelixType: 'test/performance/$(Kind)/$(_Framework)/$(Architecture)'
HelixType: 'test/performance/$(Kind)/$(_Framework)/$(Architecture)${{ parameters.helixTypeSuffix }}'
HelixAccessToken: $(HelixApiAccessToken)
HelixTargetQueues: $(Queue)
HelixPreCommands: $(HelixPreCommand)
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册