From 8b247add4764d5e33ca03e66a213901fd9fb5cb3 Mon Sep 17 00:00:00 2001
From: Jose Ignacio Lorenzo <74208929+joselo85@users.noreply.github.com>
Date: Wed, 14 Sep 2022 12:13:25 -0300
Subject: [PATCH] [CONSUL-429] Replace Docker run with Docker exec (#57)

---
 .../Dockerfile-consul-local-windows                 |  2 +-
 test/integration/connect/envoy/helpers.windows.bash | 13 ++++++++++++-
 test/integration/connect/envoy/run-tests.windows.sh | 10 +++-------
 3 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/build-support-windows/Dockerfile-consul-local-windows b/build-support-windows/Dockerfile-consul-local-windows
index 346e22c493..9d75e7fcb7 100644
--- a/build-support-windows/Dockerfile-consul-local-windows
+++ b/build-support-windows/Dockerfile-consul-local-windows
@@ -49,4 +49,4 @@ EXPOSE 19000 19001 19002 19003 19004
 EXPOSE 21000 21001 21002 21003 21004
 EXPOSE 5000 1234 2345
 
-RUN SETX /M path "%PATH%;C:\consul;C:\envoy;C:\fortio;C:\jaeger;C:\Program Files\Git\bin;C:\Program Files\OpenSSL-Win64\bin;C:\bats\bin\bats;C:\ProgramData\chocolatey\lib\jq\tools;C:\socat;"
+RUN SETX /M path "%PATH%;C:\consul;C:\envoy;C:\fortio;C:\jaeger;C:\Program Files\Git\bin;C:\Program Files\Git\usr\bin;C:\Program Files\OpenSSL-Win64\bin;C:\bats\bin\;C:\ProgramData\chocolatey\lib\jq\tools;C:\socat;"
diff --git a/test/integration/connect/envoy/helpers.windows.bash b/test/integration/connect/envoy/helpers.windows.bash
index 3d78c2bbcf..1c8c9a261f 100644
--- a/test/integration/connect/envoy/helpers.windows.bash
+++ b/test/integration/connect/envoy/helpers.windows.bash
@@ -615,10 +615,21 @@ function kill_envoy {
   tskill $PID
 }
 
+# This function is needed since socats SYSTEM isn't working. 
+function split_lines_in_stats {
+  local MATCH=$1
+  local FILE=$2
+
+  sed -i "s/$MATCH/\n$MATCH/g" $FILE
+}
+
 function must_match_in_statsd_logs {
   local DC=${2:-primary}
+  local FILE="/c/workdir/${DC}/statsd/statsd.log"
+
+  split_lines_in_stats envoy $FILE 
 
-  run cat /workdir/${DC}/statsd/statsd.log
+  run cat $FILE
   echo "$output"
   COUNT=$( echo "$output" | grep -Ec $1 )
 
diff --git a/test/integration/connect/envoy/run-tests.windows.sh b/test/integration/connect/envoy/run-tests.windows.sh
index 45fd7086bb..02dc68979c 100644
--- a/test/integration/connect/envoy/run-tests.windows.sh
+++ b/test/integration/connect/envoy/run-tests.windows.sh
@@ -803,15 +803,11 @@ function run_container_terminating-gateway-primary {
 }
 
 function run_container_fake-statsd {
+  local CONTAINER_NAME="$SINGLE_CONTAINER_BASE_NAME"-"primary"_1
   # This magic SYSTEM incantation is needed since Envoy doesn't add newlines and so
   # we need each packet to be passed to echo to add a new line before
-  # appending.
-  docker.exe run -d --name $(container_name) \
-    $WORKDIR_SNIPPET \
-    $(network_snippet primary) \
-    "${HASHICORP_DOCKER_PROXY}/windows/socat" \
-    -u UDP-RECVFROM:8125,fork,reuseaddr \
-    SYSTEM:'xargs -0 echo >> /workdir/primary/statsd/statsd.log'
+  # appending. But it does not work on Windows.   
+  docker.exe exec -d $CONTAINER_NAME bash -c "socat -u UDP-RECVFROM:8125,fork,reuseaddr OPEN:/workdir/primary/statsd/statsd.log,create,append"                                            
 }
 
 function run_container_zipkin {