From 97b34d7a9c2f77192eb033fd33aaa68887722c5f Mon Sep 17 00:00:00 2001
From: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com>
Date: Thu, 14 Mar 2024 15:52:56 -0400
Subject: [PATCH] Sync eng/common directory with azure-sdk-tools for PR 7877
 (#34774)

* create necessary eng/common changes to allow proxy to be shut down

---------

Co-authored-by: Scott Beddall (from Dev Box) <scbedd@microsoft.com>
---
 eng/common/testproxy/test-proxy-tool-shutdown.yml | 10 ++++++++++
 eng/common/testproxy/test-proxy-tool.yml          |  8 +++++++-
 2 files changed, 17 insertions(+), 1 deletion(-)
 create mode 100644 eng/common/testproxy/test-proxy-tool-shutdown.yml

diff --git a/eng/common/testproxy/test-proxy-tool-shutdown.yml b/eng/common/testproxy/test-proxy-tool-shutdown.yml
new file mode 100644
index 000000000000..20e24e70a0aa
--- /dev/null
+++ b/eng/common/testproxy/test-proxy-tool-shutdown.yml
@@ -0,0 +1,10 @@
+steps:
+  - pwsh: |
+      Stop-Process -Id $(PROXY_PID)
+    displayName: 'Shut down the testproxy - windows'
+    condition: and(succeeded(), eq(variables['Agent.OS'],'Windows_NT'))
+
+  - bash: |
+      kill -9 $(PROXY_PID)
+    displayName: "Shut down the testproxy - linux/mac"
+    condition: and(succeeded(), ne(variables['Agent.OS'],'Windows_NT'))
diff --git a/eng/common/testproxy/test-proxy-tool.yml b/eng/common/testproxy/test-proxy-tool.yml
index 7aea55d472d2..d9a166841926 100644
--- a/eng/common/testproxy/test-proxy-tool.yml
+++ b/eng/common/testproxy/test-proxy-tool.yml
@@ -1,3 +1,4 @@
+# This template sets variable PROXY_PID to be used for shutdown later.
 parameters:
   rootFolder: '$(Build.SourcesDirectory)'
   runProxy: true
@@ -42,15 +43,20 @@ steps:
       condition: and(succeeded(), ${{ parameters.condition }})
 
     - pwsh: |
-        Start-Process $(Build.BinariesDirectory)/test-proxy/test-proxy.exe `
+        $Process = Start-Process $(Build.BinariesDirectory)/test-proxy/test-proxy.exe `
           -ArgumentList "start --storage-location ${{ parameters.rootFolder }} -U" `
           -NoNewWindow -PassThru -RedirectStandardOutput ${{ parameters.rootFolder }}/test-proxy.log
+
+        Write-Host "##vso[task.setvariable variable=PROXY_PID]$($Process.Id)"
       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 &>$(Build.SourcesDirectory)/test-proxy.log &
+        
+        echo $! > $(Build.SourcesDirectory)/test-proxy.pid
+        echo "##vso[task.setvariable variable=PROXY_PID]$(cat $(Build.SourcesDirectory)/test-proxy.pid)"
       displayName: "Run the testproxy - linux/mac"
       condition: and(succeeded(), ne(variables['Agent.OS'],'Windows_NT'), ${{ parameters.condition }})
       workingDirectory: "${{ parameters.rootFolder }}"