mongo 2.17.4 → 2.21.0

data/spec/spec_tests/data/sdam/rs/use_setversion_without_electionid.yml

@@ -4,7 +4,7 @@ uri: "mongodb://a/?replicaSet=rs"
 
 phases: [
 
-    # Primary A has setVersion and electionId, tells us about B.
+    # Primary A has electionId and setVersion, tells us about B.
     {
         responses: [
             ["a:27017", {
@@ -16,7 +16,7 @@ phases: [
                 setVersion: 1,
                 electionId: {"$oid": "000000000000000000000001"},
                 minWireVersion: 0,
-                maxWireVersion: 6
+                maxWireVersion: 17
             }]
         ],
 
@@ -42,7 +42,7 @@ phases: [
         }
     },
 
-    # Reconfig the set and elect B, it has a new setVersion but no electionId.
+    # Reconfig, B reports as primary, B is missing the electionId but reports setVersion
     {
         responses: [
             ["b:27017", {
@@ -53,33 +53,34 @@ phases: [
                 setName: "rs",
                 setVersion: 2,
                 minWireVersion: 0,
-                maxWireVersion: 6
+                maxWireVersion: 17
             }]
         ],
 
         outcome: {
             servers: {
-                "a:27017": {
+               "a:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId: { "$oid": "000000000000000000000001" }
+                },
+                "b:27017": {
                     type: "Unknown",
                     setName: ,
                     electionId:
-                },
-                "b:27017": {
-                    type: "RSPrimary",
-                    setName: "rs",
-                    setVersion: 2
                 }
             },
             topologyType: "ReplicaSetWithPrimary",
             logicalSessionTimeoutMinutes: null,
             setName: "rs",
-            maxSetVersion: 2,
+            maxSetVersion: 1,
             maxElectionId: {"$oid": "000000000000000000000001"},
         }
     },
 
-    # Delayed response from A, reporting its reelection. Its setVersion shows
-    # the election preceded B's so we ignore it.
+    # A reports as primary, A has been reelection (electionId greater than our recorded maxElectionId).
+    # A's setVersion is less than our maxSetVersion, but electionId takes precedence so B's primary claim is ignored
     {
         responses: [
             ["a:27017", {
@@ -91,27 +92,28 @@ phases: [
                 setVersion: 1,
                 electionId: {"$oid": "000000000000000000000002"},
                 minWireVersion: 0,
-                maxWireVersion: 6
+                maxWireVersion: 17
             }]
         ],
         outcome: {
             servers: {
                 "a:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId: {"$oid": "000000000000000000000002"}
+                },
+                "b:27017":{
                     type: "Unknown",
                     setName: ,
                     electionId:
-                },
-                "b:27017": {
-                    type: "RSPrimary",
-                    setName: "rs",
-                    setVersion: 2
                 }
             },
             topologyType: "ReplicaSetWithPrimary",
             logicalSessionTimeoutMinutes: null,
             setName: "rs",
-            maxSetVersion: 2,
-            maxElectionId: {"$oid": "000000000000000000000001"},
+            maxSetVersion: 1,
+            maxElectionId: {"$oid": "000000000000000000000002"},
         }
     }
 ]

data/spec/spec_tests/data/sdam_monitoring/discovered_standalone.yml

@@ -5,7 +5,7 @@ phases:
     responses:
       -
         - "a:27017"
-        - { ok: 1, helloOk: true, isWritablePrimary: true, minWireVersion: 0, maxWireVersion: 4 }
+        - { ok: 1, helloOk: true, isWritablePrimary: true, minWireVersion: 0, maxWireVersion: 6 }
 
     outcome:
       events:

data/spec/spec_tests/data/sdam_monitoring/replica_set_with_no_primary.yml

@@ -1,11 +1,11 @@
 description: "Monitoring a topology that is a replica set with no primary connected"
 uri: "mongodb://a,b"
-phases: 
-  - 
-    responses: 
-      - 
+phases:
+  -
+    responses:
+      -
         - "a:27017"
-        - 
+        -
           ok: 1
           helloOk: true
           isWritablePrimary: false
@@ -13,101 +13,100 @@ phases:
           setName: "rs"
           setVersion: 1
           primary: "b:27017"
-          hosts: 
+          hosts:
             - "a:27017"
             - "b:27017"
           minWireVersion: 0
-          maxWireVersion: 4
-    outcome: 
-      events: 
-        - 
-          topology_opening_event: 
+          maxWireVersion: 6
+    outcome:
+      events:
+        -
+          topology_opening_event:
             topologyId: "42"
-        - 
-          topology_description_changed_event: 
+        -
+          topology_description_changed_event:
             topologyId: "42"
-            previousDescription: 
+            previousDescription:
               topologyType: "Unknown"
               servers: []
-            newDescription: 
+            newDescription:
               topologyType: "Unknown"
-              servers: 
-                - 
+              servers:
+                -
                   address: "a:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "Unknown"
-                - 
+                -
                   address: "b:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "Unknown"
-        - 
-          server_opening_event: 
+        -
+          server_opening_event:
             topologyId: "42"
             address: "a:27017"
-        - 
-          server_opening_event: 
+        -
+          server_opening_event:
             topologyId: "42"
             address: "b:27017"
-        - 
-          server_description_changed_event: 
+        -
+          server_description_changed_event:
             topologyId: "42"
             address: "a:27017"
-            previousDescription: 
+            previousDescription:
               address: "a:27017"
               arbiters: []
               hosts: []
               passives: []
               type: "Unknown"
-            newDescription: 
+            newDescription:
               address: "a:27017"
               arbiters: []
-              hosts: 
+              hosts:
                 - "a:27017"
                 - "b:27017"
               passives: []
               primary: "b:27017"
               setName: "rs"
               type: "RSSecondary"
-        - 
-          topology_description_changed_event: 
+        -
+          topology_description_changed_event:
             topologyId: "42"
-            previousDescription: 
+            previousDescription:
               topologyType: "Unknown"
-              servers: 
-                - 
+              servers:
+                -
                   address: "a:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "Unknown"
-                - 
+                -
                   address: "b:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "Unknown"
-            newDescription: 
+            newDescription:
               topologyType: "ReplicaSetNoPrimary"
               setName: "rs"
-              servers: 
-                - 
+              servers:
+                -
                   address: "a:27017"
                   arbiters: []
-                  hosts: 
+                  hosts:
                     - "a:27017"
                     - "b:27017"
                   passives: []
                   primary: "b:27017"
                   setName: "rs"
                   type: "RSSecondary"
-                - 
+                -
                   address: "b:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "PossiblePrimary"
-

data/spec/spec_tests/data/sdam_monitoring/replica_set_with_primary.yml

@@ -1,112 +1,111 @@
 description: "Monitoring a topology that is a replica set with a primary connected"
 uri: "mongodb://a,b"
-phases: 
-  - 
-    responses: 
-      - 
+phases:
+  -
+    responses:
+      -
         - "a:27017"
-        - 
+        -
           ok: 1
           helloOk: true
           isWritablePrimary: true
           setName: "rs"
           setVersion: 1
           primary: "a:27017"
-          hosts: 
+          hosts:
             - "a:27017"
             - "b:27017"
           minWireVersion: 0
-          maxWireVersion: 4
-    outcome: 
-      events: 
-        - 
-          topology_opening_event: 
+          maxWireVersion: 6
+    outcome:
+      events:
+        -
+          topology_opening_event:
             topologyId: "42"
-        - 
-          topology_description_changed_event: 
+        -
+          topology_description_changed_event:
             topologyId: "42"
-            previousDescription: 
+            previousDescription:
               topologyType: "Unknown"
               servers: []
-            newDescription: 
+            newDescription:
               topologyType: "Unknown"
-              servers: 
-                - 
+              servers:
+                -
                   address: "a:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "Unknown"
-                - 
+                -
                   address: "b:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "Unknown"
-        - 
-          server_opening_event: 
+        -
+          server_opening_event:
             topologyId: "42"
             address: "a:27017"
-        - 
-          server_opening_event: 
+        -
+          server_opening_event:
             topologyId: "42"
             address: "b:27017"
-        - 
-          server_description_changed_event: 
+        -
+          server_description_changed_event:
             topologyId: "42"
             address: "a:27017"
-            previousDescription: 
+            previousDescription:
               address: "a:27017"
               arbiters: []
               hosts: []
               passives: []
               type: "Unknown"
-            newDescription: 
+            newDescription:
               address: "a:27017"
               arbiters: []
-              hosts: 
+              hosts:
                 - "a:27017"
                 - "b:27017"
               passives: []
               primary: "a:27017"
               setName: "rs"
               type: "RSPrimary"
-        - 
-          topology_description_changed_event: 
+        -
+          topology_description_changed_event:
             topologyId: "42"
-            previousDescription: 
+            previousDescription:
               topologyType: "Unknown"
-              servers: 
-                - 
+              servers:
+                -
                   address: "a:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "Unknown"
-                - 
+                -
                   address: "b:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "Unknown"
-            newDescription: 
+            newDescription:
               topologyType: "ReplicaSetWithPrimary"
               setName: "rs"
-              servers: 
-                - 
+              servers:
+                -
                   address: "a:27017"
                   arbiters: []
-                  hosts: 
+                  hosts:
                     - "a:27017"
                     - "b:27017"
                   passives: []
                   primary: "a:27017"
                   setName: "rs"
                   type: "RSPrimary"
-                - 
+                -
                   address: "b:27017"
                   arbiters: []
                   hosts: []
                   passives: []
                   type: "Unknown"
-

data/spec/spec_tests/data/sdam_monitoring/replica_set_with_removal.yml

@@ -50,7 +50,7 @@ phases:
             primary: "a:27017",
             hosts: [ "a:27017" ],
             minWireVersion: 0,
-            maxWireVersion: 4
+            maxWireVersion: 6
           }
       -
         - "b:27017"

data/spec/spec_tests/data/sdam_monitoring/required_replica_set.yml

@@ -14,7 +14,7 @@ phases:
             primary: "a:27017",
             hosts: [ "a:27017", "b:27017" ],
             minWireVersion: 0,
-            maxWireVersion: 4
+            maxWireVersion: 6
           }
     outcome:
       events:

data/spec/spec_tests/data/sdam_monitoring/standalone.yml

@@ -5,7 +5,7 @@ phases:
     responses:
       -
         - "a:27017"
-        - { ok: 1, helloOk: true, isWritablePrimary: true, minWireVersion: 0, maxWireVersion: 4 }
+        - { ok: 1, helloOk: true, isWritablePrimary: true, minWireVersion: 0, maxWireVersion: 6 }
 
     outcome:
       events:

data/spec/spec_tests/data/sdam_monitoring/standalone_suppress_equal_description_changes.yml

@@ -5,10 +5,10 @@ phases:
     responses:
       -
         - "a:27017"
-        - { ok: 1, helloOk: true, isWritablePrimary: true, minWireVersion: 0, maxWireVersion: 4 }
+        - { ok: 1, helloOk: true, isWritablePrimary: true, minWireVersion: 0, maxWireVersion: 6 }
       -
         - "a:27017"
-        - { ok: 1, helloOk: true, isWritablePrimary: true, minWireVersion: 0, maxWireVersion: 4 }
+        - { ok: 1, helloOk: true, isWritablePrimary: true, minWireVersion: 0, maxWireVersion: 6 }
 
     outcome:
       events:

data/spec/spec_tests/data/sdam_unified/auth-error.yml

@@ -0,0 +1,130 @@
+description: auth-error
+
+schemaVersion: "1.10"
+
+runOnRequirements:
+    # failCommand appName requirements
+  - minServerVersion: "4.4"
+    auth: true
+    serverless: forbid
+    topologies: [ single, replicaset, sharded ]
+
+createEntities:
+  - client:
+      id: &setupClient setupClient
+      useMultipleMongoses: false
+
+initialData: &initialData
+  - collectionName: &collectionName auth-error
+    databaseName: &databaseName sdam-tests
+    documents:
+      - _id: 1
+      - _id: 2
+tests:
+  - description: Reset server and pool after AuthenticationFailure error
+    operations:
+      - name: failPoint
+        object: testRunner
+        arguments:
+          client: *setupClient
+          failPoint:
+            configureFailPoint: failCommand
+            mode:
+              times: 1
+            data:
+              failCommands:
+                - saslContinue
+              appName: authErrorTest
+              errorCode: 18
+      - name: createEntities
+        object: testRunner
+        arguments:
+          entities:
+            - client:
+                id: &client client
+                useMultipleMongoses: false
+                observeEvents:
+                  - commandStartedEvent
+                  - serverDescriptionChangedEvent
+                  - poolClearedEvent
+                uriOptions:
+                  retryWrites: false
+                  appname: authErrorTest
+            - database:
+                id: &database database
+                client: *client
+                databaseName: *databaseName
+            - collection:
+                id: &collection collection
+                database: *database
+                collectionName: *collectionName
+      - name: insertMany
+        object: *collection
+        arguments:
+          documents:
+            - _id: 3
+            - _id: 4
+        expectError:
+          isError: true
+      - name: waitForEvent
+        object: testRunner
+        arguments:
+          client: *client
+          event:
+            serverDescriptionChangedEvent:
+              newDescription:
+                type: Unknown
+          count: 1
+      - name: waitForEvent
+        object: testRunner
+        arguments:
+          client: *client
+          event:
+            poolClearedEvent: {}
+          count: 1
+      # Perform another operation to ensure the node is rediscovered.
+      - name: insertMany
+        object: *collection
+        arguments:
+          documents:
+            - _id: 5
+            - _id: 6
+      # Assert the server was marked Unknown and pool was cleared exactly once.
+      - name: assertEventCount
+        object: testRunner
+        arguments:
+          client: *client
+          event:
+            serverDescriptionChangedEvent:
+              newDescription:
+                type: Unknown
+          count: 1
+      - name: assertEventCount
+        object: testRunner
+        arguments:
+          client: *client
+          event:
+            poolClearedEvent: {}
+          count: 1
+    expectEvents:
+      # Note: The first insert command is never attempted because connection
+      # checkout fails.
+      - client: *client
+        eventType: command
+        events:
+          - commandStartedEvent:
+              command:
+                insert: auth-error
+                documents:
+                  - _id: 5
+                  - _id: 6
+              commandName: insert
+              databaseName: *databaseName
+    outcome:
+      - collectionName: *collectionName
+        databaseName: *databaseName
+        documents:
+          - _id: 1
+          - _id: 2
+          - _id: 5
+          - _id: 6