From 08d5e45725792faacd63712b2ee79e87415042d3 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 b19bd5c88b..b6a75436fa 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 f663d33cc330d46bc69ca94f5a50b620c64397e7 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 b6a75436fa..b19bd5c88b 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 fe0c38c98b49b5ad965994d5ce792e7b4cf386f7 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 b19bd5c88b..28d9a0bd2a 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 8042292f244f2b7ea65bf6fe7f3d76764f5d6297 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 28d9a0bd2a..671e6f0d3a 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 590ace2f99592c143f1b0551411bb626786dbbf3 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 671e6f0d3a..3d52542ad9 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 b3e7bd4c2fb8e1f303d3179bf66c68b8fc0088b9 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 3d52542ad9..ff2d531625 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 }}"