cleanup

Author: anandsyncs

controllers/searchcontroller/mongodbsearch_reconcile_helper_test.go

@@ -152,58 +152,3 @@ func TestGetMongodConfigParameters_TransportAndPorts(t *testing.T) {
 		})
 	}
 }
-
-func TestGetMongodConfigParameters_TransportAndPorts(t *testing.T) {
-	cases := []struct {
-		name            string
-		withWireproto   bool
-		expectedUseGrpc bool
-	}{
-		{
-			name:            "grpc only (default)",
-			withWireproto:   false,
-			expectedUseGrpc: true,
-		},
-		{
-			name:            "grpc + wireproto via annotation",
-			withWireproto:   true,
-			expectedUseGrpc: false,
-		},
-	}
-
-	for _, tc := range cases {
-		t.Run(tc.name, func(t *testing.T) {
-			search := &searchv1.MongoDBSearch{
-				ObjectMeta: metav1.ObjectMeta{
-					Name:      "test-mongodb-search",
-					Namespace: "test",
-				},
-			}
-			if tc.withWireproto {
-				search.Annotations = map[string]string{searchv1.ForceWireprotoAnnotation: "true"}
-			}
-
-			clusterDomain := "cluster.local"
-			params := GetMongodConfigParameters(search, clusterDomain)
-
-			setParams := params["setParameter"].(map[string]any)
-
-			useGrpc := setParams["useGrpcForSearch"].(bool)
-			assert.Equal(t, tc.expectedUseGrpc, useGrpc)
-
-			expectedPort := search.GetMongotGrpcPort()
-			if tc.withWireproto {
-				expectedPort = search.GetMongotWireprotoPort()
-			}
-			expectedPrefix := fmt.Sprintf("%s.%s.svc.%s", search.Name+"-search-svc", search.Namespace, clusterDomain)
-			expectedSuffix := fmt.Sprintf(":%d", expectedPort)
-
-			for _, key := range []string{"mongotHost", "searchIndexManagementHostAndPort"} {
-				value := setParams[key].(string)
-				if !strings.HasPrefix(value, expectedPrefix) || !strings.HasSuffix(value, expectedSuffix) {
-					t.Fatalf("%s mismatch: expected prefix %q and suffix %q, got %q", key, expectedPrefix, expectedSuffix, value)
-				}
-			}
-		})
-	}
-}

docs/search/01-search-community-deploy/code_snippets/01_0040_validate_env.sh

@@ -14,23 +14,11 @@ required=(
   OPERATOR_HELM_CHART
 )
 
-optional=(
-  OPERATOR_ADDITIONAL_HELM_VALUES
-  PRERELEASE_IMAGE_PULLSECRET
-)
-
 missing_req=()
 for v in "${required[@]}"; do [[ -n "${!v:-}" ]] || missing_req+=("$v"); done
 if (( ${#missing_req[@]} )); then
   echo "ERROR: Missing required environment variables:" >&2
   for m in "${missing_req[@]}"; do echo "  - $m" >&2; done
-  exit 1
 fi
 
-missing_opt=()
-for v in "${optional[@]}"; do [[ -n "${!v:-}" ]] || missing_opt+=("$v"); done
-
 echo "All required environment variables present."
-if (( ${#missing_opt[@]} )); then
-  echo "Optional variables missing (will continue): ${missing_opt[*]}"
-fi

docs/search/01-search-community-deploy/code_snippets/01_0307_prepare_cert_manager_issuer.sh

@@ -50,7 +50,6 @@ EOF_MANIFEST
 kubectl --context "${K8S_CTX}" wait --for=condition=Ready clusterissuer "${MDB_TLS_CA_ISSUER}"
 
 TMP_CA_CERT="$(mktemp)"
-trap 'rm -f "${TMP_CA_CERT}"' EXIT
 
 kubectl --context "${K8S_CTX}" \
   get secret "${MDB_TLS_CA_SECRET_NAME}" -n "${CERT_MANAGER_NAMESPACE}" \

docs/search/01-search-community-deploy/env_variables.sh

@@ -34,6 +34,6 @@ export OPERATOR_HELM_CHART="mongodb/mongodb-kubernetes"
 export OPERATOR_ADDITIONAL_HELM_VALUES=""
 
 # TLS is mandatory; connection string must include tls=true
-export MDB_CONNECTION_STRING="mongodb://mdb-user:${MDB_USER_PASSWORD}@${MDB_RESOURCE_NAME}-0.${MDB_RESOURCE_NAME}-svc.${MDB_NS}.svc.cluster.local:27017/?replicaSet=${MDB_RESOURCE_NAME}&tls=true"
+export MDB_CONNECTION_STRING="mongodb://mdb-user:${MDB_USER_PASSWORD}@${MDB_RESOURCE_NAME}-0.${MDB_RESOURCE_NAME}-svc.${MDB_NS}.svc.cluster.local:27017/?replicaSet=${MDB_RESOURCE_NAME}&tls=true&tlsCAFile=/tls/ca.crt"
 
 export CERT_MANAGER_NAMESPACE="cert-manager"

scripts/code_snippets/tests/outputs/test_kind_search_community_snippets/03_0445_list_vector_search_indexes.out

@@ -4,16 +4,16 @@
     ns: 'sample_mflix.embedded_movies',
     firstBatch: [
       {
-        id: '6900e503c2c18a60ebc367dd',
+        id: '68c974a12bc7c076f12cd34e',
         name: 'vector_index',
         type: 'vectorSearch',
         latestDefinition: {
           type: 'vectorSearch',
-          indexID: ObjectId('6900e503c2c18a60ebc367dd'),
+          indexID: ObjectId('68c974a12bc7c076f12cd34e'),
           name: 'vector_index',
           database: 'sample_mflix',
           lastObservedCollectionName: 'embedded_movies',
-          collectionUUID: UUID('a0633632-cf14-4a80-8e35-2fb57bb896c6'),
+          collectionUUID: UUID('ac18b1d2-d1a2-49a6-9a02-de2bdbf1ecb0'),
           numPartitions: 1,
           fields: [
             {
@@ -30,11 +30,11 @@
   },
   ok: 1,
   '$clusterTime': {
-    clusterTime: Timestamp({ t: 1761666370, i: 1 }),
+    clusterTime: Timestamp({ t: 1758033116, i: 1 }),
     signature: {
-      hash: Binary.createFromBase64('aT9ULTNqVJgvq0QazViadCNQQb0=', 0),
-      keyId: Long('7566297611662000134')
+      hash: Binary.createFromBase64('Wehb6njq4tsFr6uYMz9n7vyRLAw=', 0),
+      keyId: Long('7550693115007336453')
     }
   },
-  operationTime: Timestamp({ t: 1761666370, i: 1 })
+  operationTime: Timestamp({ t: 1758033116, i: 1 })
 }