From a3e9ad1d2c2509608ba6aaf78d0a400587e9b4fe Mon Sep 17 00:00:00 2001
From: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com>
Date: Thu, 26 Jan 2023 16:47:47 -0800
Subject: [PATCH] Sync eng/common directory with azure-sdk-tools for PR 5224
 (#4274)

* feature/upload-proxy-artifact

* update template to ensure that having the file in use on windows doesn't break the build

* incoming PR fixes where test-proxy stores its log. as as result, we also need to update the location of where we read out the test-proxy log

Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
---
 eng/common/testproxy/publish-proxy-logs.yml | 16 ++++++++++++++++
 eng/common/testproxy/test-proxy-tool.yml    |  4 ++--
 eng/pipelines/templates/jobs/ci.tests.yml   |  2 +-
 3 files changed, 19 insertions(+), 3 deletions(-)
 create mode 100644 eng/common/testproxy/publish-proxy-logs.yml

diff --git a/eng/common/testproxy/publish-proxy-logs.yml b/eng/common/testproxy/publish-proxy-logs.yml
new file mode 100644
index 000000000..543527a44
--- /dev/null
+++ b/eng/common/testproxy/publish-proxy-logs.yml
@@ -0,0 +1,16 @@
+parameters:
+  rootFolder: '$(Build.SourcesDirectory)'
+
+steps:
+  - pwsh: |
+      Copy-Item -Path "${{ parameters.rootFolder }}/test-proxy.log" -Destination "${{ parameters.rootFolder }}/proxy.log"
+    displayName: Copy Log File
+
+  - template: ../pipelines/templates/steps/publish-artifact.yml
+    parameters:
+      ArtifactName: "$(System.JobName)-proxy-logs"
+      ArtifactPath: "${{ parameters.rootFolder }}/proxy.log"
+
+  - pwsh: |
+      Remove-Item -Force ${{ parameters.rootFolder }}/proxy.log
+    displayName: Cleanup Copied Log File
diff --git a/eng/common/testproxy/test-proxy-tool.yml b/eng/common/testproxy/test-proxy-tool.yml
index 293848da1..ae9d69051 100644
--- a/eng/common/testproxy/test-proxy-tool.yml
+++ b/eng/common/testproxy/test-proxy-tool.yml
@@ -42,13 +42,13 @@ steps:
     - pwsh: |
         Start-Process $(Build.BinariesDirectory)/test-proxy/test-proxy.exe `
           -ArgumentList "--storage-location ${{ parameters.rootFolder }}" `
-          -NoNewWindow -PassThru -RedirectStandardOutput ${{ parameters.templateRoot }}/test-proxy.log
+          -NoNewWindow -PassThru -RedirectStandardOutput ${{ parameters.rootFolder }}/test-proxy.log
       displayName: 'Run the testproxy - windows'
       condition: and(succeeded(), eq(variables['Agent.OS'],'Windows_NT'), ${{ parameters.condition }})
 
     # nohup does NOT continue beyond the current session if you use it within powershell
     - bash: |
-        nohup $(Build.BinariesDirectory)/test-proxy/test-proxy > ${{ parameters.templateRoot }}/test-proxy.log &
+        nohup $(Build.BinariesDirectory)/test-proxy/test-proxy > ${{ parameters.rootFolder }}/test-proxy.log &
       displayName: "Run the testproxy - linux/mac"
       condition: and(succeeded(), ne(variables['Agent.OS'],'Windows_NT'), ${{ parameters.condition }})
       workingDirectory: "${{ parameters.rootFolder }}"
diff --git a/eng/pipelines/templates/jobs/ci.tests.yml b/eng/pipelines/templates/jobs/ci.tests.yml
index 997ddeea8..d003ab96e 100644
--- a/eng/pipelines/templates/jobs/ci.tests.yml
+++ b/eng/pipelines/templates/jobs/ci.tests.yml
@@ -204,7 +204,7 @@ jobs:
           get-content test-proxy.log
         displayName: TestProxy Log
         condition: and(succeededOrFailed(), contains(variables.CmakeArgs, 'BUILD_TESTING=ON'),ne('${{parameters.ServiceDirectory}}', 'template'))
-        workingDirectory: $(Build.SourcesDirectory)
+        workingDirectory: $(Build.SourcesDirectory)/sdk/${{parameters.ServiceDirectory}}
         continueOnError: true
 
       - task: PublishTestResults@2