From f7932b4ffa9621cf6eef6c6512f96b34a2b2d104 Mon Sep 17 00:00:00 2001
From: "R.B. Boyer" <4903+rboyer@users.noreply.github.com>
Date: Wed, 20 Oct 2021 15:50:45 -0500
Subject: [PATCH] test: remove some envoy integ test warnings (#11369)

We launch one container as part of the test with --pid=host but
apparently within that container it launches a copy of "tini" as a
process supervisor that prefers to be PID 1.

Because it's not PID 1 it logs a warning message about this to the envoy
integration test logs that can lead to thinking somehow that a test
failure is related when in fact it's completely unrelated.

Adding this environment variable avoids the warning.
---
 test/integration/connect/envoy/run-tests.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/test/integration/connect/envoy/run-tests.sh b/test/integration/connect/envoy/run-tests.sh
index 2197d4102..736b3d9ff 100755
--- a/test/integration/connect/envoy/run-tests.sh
+++ b/test/integration/connect/envoy/run-tests.sh
@@ -212,7 +212,10 @@ function verify {
 
   echo "Running ${DC} verification step for ${CASE_DIR}..."
 
+  # need to tell the PID 1 inside of the container that it won't be actual PID
+  # 1 because we're using --pid=host so we use TINI_SUBREAPER
   if docker run --name envoy_verify-${DC}_1 -t \
+    -e TINI_SUBREAPER=1 \
     -e ENVOY_VERSION \
     $WORKDIR_SNIPPET \
     --pid=host \