PHPC-2395: Fetch Atlas connectivity URIs from AWS Secrets Manager

Author: jmikola

.evergreen/config/functions.yml

@@ -210,7 +210,6 @@ functions:
       params:
         include_expansions_in_env:
           - API_VERSION
-          - ATLAS_CONNECTIVITY_URIS
           - CRYPT_SHARED_LIB_PATH
           - MONGODB_URI
           - APPEND_URI
@@ -223,6 +222,18 @@ functions:
           ${PREPARE_SHELL}
           bash ${PROJECT_DIRECTORY}/.evergreen/run-tests.sh
 
+  "run atlas connectivity tests":
+    - command: shell.exec
+      type: test
+      params:
+        working_dir: "src"
+        shell: bash
+        script: |
+          ${PREPARE_SHELL}
+          . ${DRIVERS_TOOLS}/.evergreen/secrets_handling/setup-secrets.sh drivers/atlas_connect
+          TESTS="tests/atlas.phpt" \
+          bash ${PROJECT_DIRECTORY}/.evergreen/run-tests.sh
+
   "cleanup":
     - command: shell.exec
       params:

.evergreen/config/test-tasks.yml

@@ -2,9 +2,7 @@ tasks:
   - name: "test-atlas-connectivity"
     tags: ["atlas", "nodb"]
     commands:
-      - func: "run tests"
-        vars:
-          TESTS: "tests/atlas.phpt"
+      - func: "run atlas connectivity tests"
 
   # Test different storage engines
   - name: "test-mongodb-4.0-standalone-inmemory"

.gitignore

@@ -67,3 +67,6 @@ mongodb-*tgz
 
 # Coverage files
 coverage*
+
+# drivers-evergreen-tools secrets handling
+secrets-export.sh

tests/atlas.phpt

@@ -2,23 +2,40 @@
 Atlas Connectivity Tests
 --SKIPIF--
 <?php require __DIR__ . "/utils/basic-skipif.inc"; ?>
-<?php skip_if_no_atlas_connectivity_urls(); ?>
 <?php if (getenv('TESTS') !== 'tests/atlas.phpt') { die('skip Atlas tests not wanted'); } ?>
 --FILE--
 <?php
 require_once __DIR__ . "/utils/basic.inc";
 
+$envs = [
+	'ATLAS_SERVERLESS',
+	'ATLAS_SRV_SERVERLESS',
+	'ATLAS_FREE',
+	'ATLAS_SRV_FREE',
+	'ATLAS_REPL',
+	'ATLAS_SRV_REPL',
+	'ATLAS_SHRD',
+	'ATLAS_SRV_SHRD',
+	'ATLAS_TLS11',
+	'ATLAS_SRV_TLS11',
+	'ATLAS_TLS12',
+	'ATLAS_SRV_TLS12',
+];
+
 $command = new \MongoDB\Driver\Command(['ping' => 1]);
 $query = new \MongoDB\Driver\Query([]);
 
-foreach (getAtlasConnectivityUrls() as $url) {
-	if (strpos($url, '#') === 0) {
-		echo trim(substr($url, 1)), "\n";
+foreach ($envs as $env) {
+	echo $env, ': ';
+	$uri = getenv($env);
+
+	if (! is_string($uri)) {
+		echo "FAIL: env var is undefined\n";
 		continue;
 	}
 
 	try {
-		$m = new \MongoDB\Driver\Manager($url);
+		$m = new \MongoDB\Driver\Manager($uri);
 		$m->executeCommand('admin', $command);
 		iterator_to_array($m->executeQuery('test.test', $query));
 		echo "PASS\n";
@@ -30,25 +47,16 @@ foreach (getAtlasConnectivityUrls() as $url) {
 ===DONE===
 <?php exit(0); ?>
 --EXPECTF--
-Atlas Serverless
-PASS
-PASS
-Atlas replica set (4.0)
-PASS
-PASS
-Atlas sharded cluster (4.0)
-PASS
-PASS
-Atlas free tier replica set
-PASS
-PASS
-Atlas with only TLSv1.1 enabled (4.0)
-PASS
-PASS
-Atlas with only TLSv1.2 enabled (4.0)
-PASS
-PASS
-Atlas with only TLSv1.2 enabled (4.0) and bad credentials
-FAIL: %s
-FAIL: %s
+ATLAS_SERVERLESS: PASS
+ATLAS_SRV_SERVERLESS: PASS
+ATLAS_FREE: PASS
+ATLAS_SRV_FREE: PASS
+ATLAS_REPL: PASS
+ATLAS_SRV_REPL: PASS
+ATLAS_SHRD: PASS
+ATLAS_SRV_SHRD: PASS
+ATLAS_TLS11: PASS
+ATLAS_SRV_TLS11: PASS
+ATLAS_TLS12: PASS
+ATLAS_SRV_TLS12: PASS
 ===DONE===

tests/utils/skipif.php

@@ -494,10 +494,3 @@ function skip_if_no_crypt_shared()
         exit('skip crypt_shared is not available');
     }
 }
-
-function skip_if_no_atlas_connectivity_urls()
-{
-    if (getAtlasConnectivityUrls() === []) {
-        exit('skip No Atlas URIs found');
-    }
-}

tests/utils/tools.php

@@ -847,26 +847,3 @@ function failGetMore(Manager $manager)
 
     throw new Exception("Trying to configure a getMore fail point for a server version ($version) that doesn't support it");
 }
-
-function getAtlasConnectivityUrls(): array
-{
-    $atlasUriString = getenv('ATLAS_CONNECTIVITY_URIS') ?: '';
-    if (!$atlasUriString) {
-        return [];
-    }
-
-    $rawUrls = explode("\n", $atlasUriString);
-
-    $urls = [];
-    foreach ($rawUrls as $url) {
-        $url = trim($url);
-
-        if ($url == '') {
-            continue;
-        }
-
-        $urls[] = $url;
-    }
-
-    return $urls;
-}