From a2260687d4e3c8b22bf9bd42f4833440c27711bc Mon Sep 17 00:00:00 2001
From: narrieta <narrieta>
Date: Wed, 14 Jun 2023 15:48:32 -0700
Subject: [PATCH 1/3] Eliminate duplicate list of test suites to run

---
 tests_e2e/pipeline/pipeline.yml             | 11 +++++++----
 tests_e2e/pipeline/scripts/execute_tests.sh |  9 +++++++--
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/tests_e2e/pipeline/pipeline.yml b/tests_e2e/pipeline/pipeline.yml
index d5d3eaf6d1..fdc086a0af 100644
--- a/tests_e2e/pipeline/pipeline.yml
+++ b/tests_e2e/pipeline/pipeline.yml
@@ -5,17 +5,20 @@
   #
 
 parameters:
+    #
     # See the test wiki for a description of the parameters
-  - name: test_suites
-    displayName: Test Suites
-    type: string
-    default: agent_bvt, no_outbound_connections, extensions_disabled, agent_not_provisioned, agent_update, fips
+    #
     # NOTES:
     #         * 'image', 'location' and 'vm_size' override any values in the test suites/images definition
     #            files. Those parameters are useful for 1-off tests, like testing a VHD or checking if
     #            an image is supported in a particular location.
     #         * Azure Pipelines do not allow empty string for the parameter value, using "-" instead.
     #
+  - name: test_suites
+    displayName: Test Suites (comma-separated list of test suites to run)
+    type: string
+    default: "-"
+
   - name: image
     displayName: Image (image/image set name, URN, or VHD)
     type: string
diff --git a/tests_e2e/pipeline/scripts/execute_tests.sh b/tests_e2e/pipeline/scripts/execute_tests.sh
index 15c9f0b5f6..2a76244920 100755
--- a/tests_e2e/pipeline/scripts/execute_tests.sh
+++ b/tests_e2e/pipeline/scripts/execute_tests.sh
@@ -39,6 +39,11 @@ docker pull waagenttests.azurecr.io/waagenttests:latest
 
 # Azure Pipelines does not allow an empty string as the value for a pipeline parameter; instead we use "-" to indicate
 # an empty value. Change "-" to "" for the variables that capture the parameter values.
+if [[ $TEST_SUITES == "-" ]]; then
+    TEST_SUITES=""  # Don't set the test_suites variable
+else
+    TEST_SUITES="-v test_suites:\"$TEST_SUITES\""
+fi
 if [[ $IMAGE == "-" ]]; then
     IMAGE=""
 fi
@@ -69,13 +74,13 @@ docker run --rm \
           -v cloud:$CLOUD \
           -v subscription_id:$SUBSCRIPTION_ID \
           -v identity_file:\$HOME/.ssh/id_rsa \
-          -v test_suites:\"$TEST_SUITES\" \
           -v log_path:\$HOME/logs \
           -v collect_logs:\"$COLLECT_LOGS\" \
           -v keep_environment:\"$KEEP_ENVIRONMENT\" \
           -v image:\"$IMAGE\" \
           -v location:\"$LOCATION\" \
-          -v vm_size:\"$VM_SIZE\"" \
+          -v vm_size:\"$VM_SIZE\" \
+          $TEST_SUITES" \
 || echo "exit $?" > /tmp/exit.sh
 
 #

From 991de76fba3e6c4772f391c85aadf78583dbaa1f Mon Sep 17 00:00:00 2001
From: narrieta <narrieta>
Date: Wed, 14 Jun 2023 15:54:13 -0700
Subject: [PATCH 2/3] fix paths

---
 tests_e2e/test_suites/no_outbound_connections.yml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests_e2e/test_suites/no_outbound_connections.yml b/tests_e2e/test_suites/no_outbound_connections.yml
index eaf6268860..2e2e2d1a84 100644
--- a/tests_e2e/test_suites/no_outbound_connections.yml
+++ b/tests_e2e/test_suites/no_outbound_connections.yml
@@ -11,9 +11,9 @@ name: "NoOutboundConnections"
 tests:
   - source: "no_outbound_connections/check_no_outbound_connections.py"
     blocks_suite: true  # If the NSG is not setup correctly, there is no point in executing the rest of the tests.
-  - "bvts/extension_operations.py"
-  - "bvts/run_command.py"
-  - "bvts/vm_access.py"
+  - "agent_bvt/extension_operations.py"
+  - "agent_bvt/run_command.py"
+  - "agent_bvt/vm_access.py"
   - "no_outbound_connections/check_fallback_to_hgap.py"
 images: "random(endorsed)"
 template: "no_outbound_connections/template.py"

From 7a09fd7a87ec64d2a225ac28af6f4a0572cdb03d Mon Sep 17 00:00:00 2001
From: narrieta <narrieta>
Date: Wed, 14 Jun 2023 16:01:26 -0700
Subject: [PATCH 3/3] add agent update

---
 tests_e2e/orchestrator/runbook.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests_e2e/orchestrator/runbook.yml b/tests_e2e/orchestrator/runbook.yml
index 661472e8b6..b19148ceec 100644
--- a/tests_e2e/orchestrator/runbook.yml
+++ b/tests_e2e/orchestrator/runbook.yml
@@ -49,7 +49,7 @@ variable:
   #
   # The test suites to execute
   - name: test_suites
-    value: "agent_bvt, no_outbound_connections, extensions_disabled, agent_not_provisioned, fips"
+    value: "agent_bvt, no_outbound_connections, extensions_disabled, agent_not_provisioned, agent_update, fips"
   - name: cloud
     value: "AzureCloud"
     is_case_visible: true