From 7fff4086523678f3c6533da4c500f86f6fb47efa Mon Sep 17 00:00:00 2001
From: Matt Clay <matt@mystile.com>
Date: Thu, 13 Jan 2022 09:01:31 -0800
Subject: [PATCH] ansible-test - Remove azure support for Sherlock.

---
 .../fragments/ansible-test-azure-sherlock.yml |  2 +
 .../commands/integration/cloud/azure.py       | 48 +++----------------
 .../config/cloud-config-azure.ini.template    |  4 +-
 3 files changed, 9 insertions(+), 45 deletions(-)
 create mode 100644 changelogs/fragments/ansible-test-azure-sherlock.yml

diff --git a/changelogs/fragments/ansible-test-azure-sherlock.yml b/changelogs/fragments/ansible-test-azure-sherlock.yml
new file mode 100644
index 00000000000..6d0bf950a35
--- /dev/null
+++ b/changelogs/fragments/ansible-test-azure-sherlock.yml
@@ -0,0 +1,2 @@
+minor_changes:
+  - ansible-test - Removed support for Sherlock from the Azure provisioning plugin.
diff --git a/test/lib/ansible_test/_internal/commands/integration/cloud/azure.py b/test/lib/ansible_test/_internal/commands/integration/cloud/azure.py
index 002fa581dba..6ad80b0fe55 100644
--- a/test/lib/ansible_test/_internal/commands/integration/cloud/azure.py
+++ b/test/lib/ansible_test/_internal/commands/integration/cloud/azure.py
@@ -2,14 +2,8 @@
 from __future__ import annotations
 
 import configparser
-import os
-import urllib.parse
 import typing as t
 
-from ....io import (
-    read_text_file,
-)
-
 from ....util import (
     ApplicationError,
     display,
@@ -23,10 +17,6 @@ from ....target import (
     IntegrationTarget,
 )
 
-from ....http import (
-    HttpClient,
-)
-
 from ....core_ci import (
     AnsibleCoreCI,
 )
@@ -40,8 +30,6 @@ from . import (
 
 class AzureCloudProvider(CloudProvider):
     """Azure cloud provider plugin. Sets up cloud resources before delegation."""
-    SHERLOCK_CONFIG_PATH = os.path.expanduser('~/.ansible-sherlock-ci.cfg')
-
     def __init__(self, args):  # type: (IntegrationConfig) -> None
         super().__init__(args)
 
@@ -56,9 +44,6 @@ class AzureCloudProvider(CloudProvider):
         if aci.available:
             return
 
-        if os.path.isfile(self.SHERLOCK_CONFIG_PATH):
-            return
-
         super().filter(targets, exclude)
 
     def setup(self):  # type: () -> None
@@ -78,40 +63,19 @@ class AzureCloudProvider(CloudProvider):
         super().cleanup()
 
     def _setup_dynamic(self):  # type: () -> None
-        """Request Azure credentials through Sherlock."""
+        """Request Azure credentials through ansible-core-ci."""
         display.info('Provisioning %s cloud environment.' % self.platform, verbosity=1)
 
         config = self._read_config_template()
         response = {}
 
-        if os.path.isfile(self.SHERLOCK_CONFIG_PATH):
-            sherlock_uri = read_text_file(self.SHERLOCK_CONFIG_PATH).splitlines()[0].strip() + '&rgcount=2'
-
-            parts = urllib.parse.urlparse(sherlock_uri)
-            query_string = urllib.parse.parse_qs(parts.query)
-            base_uri = urllib.parse.urlunparse(parts[:4] + ('', ''))
-
-            if 'code' not in query_string:
-                example_uri = 'https://example.azurewebsites.net/api/sandbox-provisioning'
-                raise ApplicationError('The Sherlock URI must include the API key in the query string. Example: %s?code=xxx' % example_uri)
-
-            display.info('Initializing azure/sherlock from: %s' % base_uri, verbosity=1)
-
-            http = HttpClient(self.args)
-            result = http.get(sherlock_uri)
-
-            display.info('Started azure/sherlock from: %s' % base_uri, verbosity=1)
-
-            if not self.args.explain:
-                response = result.json()
-        else:
-            aci = self._create_ansible_core_ci()
+        aci = self._create_ansible_core_ci()
 
-            aci_result = aci.start()
+        aci_result = aci.start()
 
-            if not self.args.explain:
-                response = aci_result['azure']
-                self.aci = aci
+        if not self.args.explain:
+            response = aci_result['azure']
+            self.aci = aci
 
         if not self.args.explain:
             values = dict(
diff --git a/test/lib/ansible_test/config/cloud-config-azure.ini.template b/test/lib/ansible_test/config/cloud-config-azure.ini.template
index ac5266badb4..766553d12ba 100644
--- a/test/lib/ansible_test/config/cloud-config-azure.ini.template
+++ b/test/lib/ansible_test/config/cloud-config-azure.ini.template
@@ -9,9 +9,7 @@
 # fill in the values below and save this file without the .template extension.
 # This will cause ansible-test to use the given configuration instead of temporary credentials.
 #
-# NOTE: Automatic provisioning of Azure credentials requires one of:
-#       1) ansible-core-ci API key in ~/.ansible-core-ci.key
-#       2) Sherlock URL (including API key) in ~/.ansible-sherlock-ci.cfg
+# NOTE: Automatic provisioning of Azure credentials requires an ansible-core-ci API key in ~/.ansible-core-ci.key
 
 [default]
 # Provide either Service Principal or Active Directory credentials below.