From 1151e5b1fce912e5ea6000a57e6444781d1abf32 Mon Sep 17 00:00:00 2001 From: Mike Harder Date: Fri, 6 Jan 2023 16:43:36 -0800 Subject: [PATCH 1/6] Remove language from profile zip --- eng/common/pipelines/templates/jobs/perf.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/common/pipelines/templates/jobs/perf.yml b/eng/common/pipelines/templates/jobs/perf.yml index b19bd5c88..b6a75436f 100644 --- a/eng/common/pipelines/templates/jobs/perf.yml +++ b/eng/common/pipelines/templates/jobs/perf.yml @@ -173,7 +173,7 @@ jobs: - task: PublishPipelineArtifact@1 inputs: - targetPath: $(System.DefaultWorkingDirectory)/${{ parameters.Language }}-profile.zip + targetPath: $(System.DefaultWorkingDirectory)/profile.zip artifactName: ${{ parameters.Language }}-profile.zip condition: ${{ parameters.Profile }} From 3522b841b1e9f7420246d9f19e71a86519fb2699 Mon Sep 17 00:00:00 2001 From: Mike Harder Date: Fri, 6 Jan 2023 17:00:02 -0800 Subject: [PATCH 2/6] Revert "Remove language from profile zip" This reverts commit 2f670435b25f89955338170a7bdfbe4432cbe23f. --- eng/common/pipelines/templates/jobs/perf.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/common/pipelines/templates/jobs/perf.yml b/eng/common/pipelines/templates/jobs/perf.yml index b6a75436f..b19bd5c88 100644 --- a/eng/common/pipelines/templates/jobs/perf.yml +++ b/eng/common/pipelines/templates/jobs/perf.yml @@ -173,7 +173,7 @@ jobs: - task: PublishPipelineArtifact@1 inputs: - targetPath: $(System.DefaultWorkingDirectory)/profile.zip + targetPath: $(System.DefaultWorkingDirectory)/${{ parameters.Language }}-profile.zip artifactName: ${{ parameters.Language }}-profile.zip condition: ${{ parameters.Profile }} From ad99bbb8cd15ef97a9f171af15400b7dcd0c70d2 Mon Sep 17 00:00:00 2001 From: Mike Harder Date: Mon, 23 Jan 2023 23:00:20 -0800 Subject: [PATCH 3/6] Remove Services parameter --- eng/common/pipelines/templates/jobs/perf.yml | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/eng/common/pipelines/templates/jobs/perf.yml b/eng/common/pipelines/templates/jobs/perf.yml index b19bd5c88..28d9a0bd2 100644 --- a/eng/common/pipelines/templates/jobs/perf.yml +++ b/eng/common/pipelines/templates/jobs/perf.yml @@ -38,9 +38,6 @@ parameters: - name: ServiceDirectory type: string default: '' -- name: Services - type: string - default: '' - name: PackageVersions type: string default: '.*' @@ -118,17 +115,12 @@ jobs: Location: westus ResourceType: perf - - pwsh: | - set-content -path config.yml -value "WorkingDirectories:" - add-content -path config.yml -value " ${{ parameters.Language }}: $(Agent.BuildDirectory)/s" - workingDirectory: azure-sdk-tools/tools/perf-automation/Azure.Sdk.Tools.PerfAutomation - displayName: Create config.yml - - script: >- dotnet run -- run --no-sync - --languages ${{ parameters.Language }} - --services "${{ parameters.Services }}" + --language ${{ parameters.Language }} + --repo-root $(System.DefaultWorkingDirectory) + --tests-file $(System.DefaultWorkingDirectory)/${{ parameters.ServiceDirectory }}/perf-tests.yml --package-versions "${{ parameters.PackageVersions }}" --tests "${{ parameters.Tests }}" --arguments "${{ parameters.Arguments }}" From 0643eedb03624d34d4c9bf2d681cf061c6106a81 Mon Sep 17 00:00:00 2001 From: Mike Harder Date: Mon, 23 Jan 2023 23:23:30 -0800 Subject: [PATCH 4/6] Fix path to tests-file --- eng/common/pipelines/templates/jobs/perf.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/common/pipelines/templates/jobs/perf.yml b/eng/common/pipelines/templates/jobs/perf.yml index 28d9a0bd2..671e6f0d3 100644 --- a/eng/common/pipelines/templates/jobs/perf.yml +++ b/eng/common/pipelines/templates/jobs/perf.yml @@ -120,7 +120,7 @@ jobs: --no-sync --language ${{ parameters.Language }} --repo-root $(System.DefaultWorkingDirectory) - --tests-file $(System.DefaultWorkingDirectory)/${{ parameters.ServiceDirectory }}/perf-tests.yml + --tests-file $(System.DefaultWorkingDirectory)/sdk/${{ parameters.ServiceDirectory }}/perf-tests.yml --package-versions "${{ parameters.PackageVersions }}" --tests "${{ parameters.Tests }}" --arguments "${{ parameters.Arguments }}" From f096c5b3fa781fb9ae4434ffa9f89f30cc9ff927 Mon Sep 17 00:00:00 2001 From: Mike Harder Date: Thu, 26 Jan 2023 22:19:28 -0800 Subject: [PATCH 5/6] Run both sync and async tests by default --- eng/common/pipelines/templates/jobs/perf.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/eng/common/pipelines/templates/jobs/perf.yml b/eng/common/pipelines/templates/jobs/perf.yml index 671e6f0d3..3d52542ad 100644 --- a/eng/common/pipelines/templates/jobs/perf.yml +++ b/eng/common/pipelines/templates/jobs/perf.yml @@ -117,7 +117,6 @@ jobs: - script: >- dotnet run -- run - --no-sync --language ${{ parameters.Language }} --repo-root $(System.DefaultWorkingDirectory) --tests-file $(System.DefaultWorkingDirectory)/sdk/${{ parameters.ServiceDirectory }}/perf-tests.yml From 8df0ea0cdb03955cea33f13973363887afc43274 Mon Sep 17 00:00:00 2001 From: Mike Harder Date: Tue, 31 Jan 2023 14:24:19 -0800 Subject: [PATCH 6/6] Add LanguageVersion to common perf.yml --- eng/common/pipelines/templates/jobs/perf.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/eng/common/pipelines/templates/jobs/perf.yml b/eng/common/pipelines/templates/jobs/perf.yml index 3d52542ad..ff2d53162 100644 --- a/eng/common/pipelines/templates/jobs/perf.yml +++ b/eng/common/pipelines/templates/jobs/perf.yml @@ -17,6 +17,9 @@ parameters: - name: Language type: string default: '' +- name: LanguageVersion + type: string + default: '' - name: LanguageRepoName type: string default: $(Build.Repository.Name) @@ -118,6 +121,7 @@ jobs: - script: >- dotnet run -- run --language ${{ parameters.Language }} + --language-version ${{ parameters.LanguageVersion }} --repo-root $(System.DefaultWorkingDirectory) --tests-file $(System.DefaultWorkingDirectory)/sdk/${{ parameters.ServiceDirectory }}/perf-tests.yml --package-versions "${{ parameters.PackageVersions }}"