mongo 2.18.3 → 2.19.0

data/spec/spec_tests/data/retryable_writes/unified/insertOne-noWritesPerformedError.yml

@@ -0,0 +1,54 @@
+description: "retryable-writes insertOne noWritesPerformedErrors"
+
+schemaVersion: "1.0"
+
+runOnRequirements:
+  - minServerVersion: "6.0"
+    topologies: [ replicaset ]
+
+createEntities:
+  - client:
+      id: &client0 client0
+      useMultipleMongoses: false
+      observeEvents: [ commandFailedEvent ]
+  - database:
+      id: &database0 database0
+      client: *client0
+      databaseName: &databaseName retryable-writes-tests
+  - collection:
+      id: &collection0 collection0
+      database: *database0
+      collectionName: &collectionName no-writes-performed-collection
+
+tests:
+  - description: "InsertOne fails after NoWritesPerformed error"
+    operations:
+      - name: failPoint
+        object: testRunner
+        arguments:
+          client: *client0
+          failPoint:
+            configureFailPoint: failCommand
+            mode:
+              times: 2
+            data:
+              failCommands:
+                - insert
+              errorCode: 64
+              errorLabels:
+                - NoWritesPerformed
+                - RetryableWriteError
+      - name: insertOne
+        object: *collection0
+        arguments:
+          document:
+            x: 1
+        expectError:
+          errorCode: 64
+          errorLabelsContain:
+            - NoWritesPerformed
+            - RetryableWriteError
+    outcome:
+      - collectionName: *collectionName
+        databaseName: *databaseName
+        documents: []

data/spec/spec_tests/data/retryable_writes/unified/insertOne-serverErrors.yml

@@ -33,7 +33,7 @@ tests:
       - minServerVersion: "4.0"
         topologies: [ replicaset ]
       - minServerVersion: "4.1.7"
-        topologies: [ sharded-replicaset ]
+        topologies: [ sharded ]
     operations:
       - name: failPoint
         object: testRunner

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

@@ -0,0 +1,62 @@
+description: ElectionId is considered higher precedence than setVersion
+uri: "mongodb://a/?replicaSet=rs"
+phases:
+  - responses:
+      - - "a:27017"
+        - ok: 1
+          helloOk: true
+          isWritablePrimary: true
+          hosts:
+            - "a:27017"
+            - "b:27017"
+          setName: rs
+          setVersion: 1
+          electionId:
+            $oid: "000000000000000000000001"
+          minWireVersion: 0
+          maxWireVersion: 17
+      - - "b:27017"
+        - ok: 1
+          helloOk: true
+          isWritablePrimary: true
+          hosts:
+            - "a:27017"
+            - "b:27017"
+          setName: rs
+          setVersion: 2 # Even though "B" reports the newer setVersion, "A" will report the newer electionId which should allow it to remain the primary
+          electionId:
+            $oid: "000000000000000000000001"
+          minWireVersion: 0
+          maxWireVersion: 17
+      - - "a:27017"
+        - ok: 1
+          helloOk: true
+          isWritablePrimary: true
+          hosts:
+            - "a:27017"
+            - "b:27017"
+          setName: rs
+          setVersion: 1
+          electionId:
+            $oid: "000000000000000000000002"
+          minWireVersion: 0
+          maxWireVersion: 17
+    outcome:
+      servers:
+        "a:27017":
+          type: RSPrimary
+          setName: rs
+          setVersion: 1
+          electionId:
+            $oid: "000000000000000000000002"
+        "b:27017":
+          type: Unknown
+          setName: null
+          setVersion: null
+          electionId: null
+      topologyType: ReplicaSetWithPrimary
+      logicalSessionTimeoutMinutes: null
+      setName: rs
+      maxSetVersion: 1
+      maxElectionId:
+        $oid: "000000000000000000000002"

data/spec/spec_tests/data/sdam/rs/null_election_id-pre-6.0.yml

@@ -0,0 +1,175 @@
+description: "Pre 6.0 Primaries with and without electionIds"
+
+uri: "mongodb://a/?replicaSet=rs"
+
+phases: [
+
+    # Primary A has no electionId.
+    {
+        responses: [
+            ["a:27017", {
+                ok: 1,
+                helloOk: true,
+                isWritablePrimary: true,
+                hosts: ["a:27017", "b:27017", "c:27017"],
+                setVersion: 1,
+                setName: "rs",
+                minWireVersion: 0,
+                maxWireVersion: 6
+            }]
+        ],
+
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId:
+                },
+                "b:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                },
+                "c:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                }
+            },
+            topologyType: "ReplicaSetWithPrimary",
+            logicalSessionTimeoutMinutes: null,
+            setName: "rs",
+            maxSetVersion: 1,
+        }
+    },
+
+    # B is elected, it has an electionId.
+    {
+        responses: [
+            ["b:27017", {
+                ok: 1,
+                helloOk: true,
+                isWritablePrimary: true,
+                hosts: ["a:27017", "b:27017", "c:27017"],
+                setName: "rs",
+                setVersion: 1,
+                electionId: {"$oid": "000000000000000000000002"},
+                minWireVersion: 0,
+                maxWireVersion: 6
+            }]
+        ],
+
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                },
+                "b:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId: {"$oid": "000000000000000000000002"}
+                },
+                "c:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                }
+            },
+            topologyType: "ReplicaSetWithPrimary",
+            logicalSessionTimeoutMinutes: null,
+            setName: "rs",
+            maxSetVersion: 1,
+            maxElectionId: {"$oid": "000000000000000000000002"},
+        }
+    },
+
+    # A still claims to be primary, no electionId, we have to trust it.
+    {
+        responses: [
+            ["a:27017", {
+                ok: 1,
+                helloOk: true,
+                isWritablePrimary: true,
+                hosts: ["a:27017", "b:27017", "c:27017"],
+                setVersion: 1,
+                setName: "rs",
+                minWireVersion: 0,
+                maxWireVersion: 6
+            }]
+        ],
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId:
+                },
+                "b:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                },
+                "c:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                }
+            },
+            topologyType: "ReplicaSetWithPrimary",
+            logicalSessionTimeoutMinutes: null,
+            setName: "rs",
+            maxSetVersion: 1,
+            maxElectionId: {"$oid": "000000000000000000000002"},
+        }
+    },
+
+    # But we remember B's electionId, so when we finally hear from C
+    # claiming it is primary, we ignore it due to its outdated electionId
+    {
+        responses: [
+            ["c:27017", {
+                ok: 1,
+                helloOk: true,
+                isWritablePrimary: true,
+                hosts: ["a:27017", "b:27017", "c:27017"],
+                setName: "rs",
+                setVersion: 1,
+                electionId: {"$oid": "000000000000000000000001"},
+                minWireVersion: 0,
+                maxWireVersion: 6
+            }]
+        ],
+        outcome: {
+            servers: {
+                # Still primary.
+                "a:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId:
+                },
+                "b:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                },
+                "c:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                }
+            },
+            topologyType: "ReplicaSetWithPrimary",
+            logicalSessionTimeoutMinutes: null,
+            setName: "rs",
+            maxSetVersion: 1,
+            maxElectionId: {"$oid": "000000000000000000000002"},
+        }
+    }
+]

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

@@ -15,7 +15,7 @@ phases: [
                 setVersion: 1,
                 setName: "rs",
                 minWireVersion: 0,
-                maxWireVersion: 6
+                maxWireVersion: 17
             }]
         ],
 
@@ -57,7 +57,7 @@ phases: [
                 setVersion: 1,
                 electionId: {"$oid": "000000000000000000000002"},
                 minWireVersion: 0,
-                maxWireVersion: 6
+                maxWireVersion: 17
             }]
         ],
 
@@ -88,7 +88,7 @@ phases: [
         }
     },
 
-    # A still claims to be primary, no electionId, we have to trust it.
+    # A still claims to be primary, no electionId, we don't trust it.
     {
         responses: [
             ["a:27017", {
@@ -99,22 +99,24 @@ phases: [
                 setVersion: 1,
                 setName: "rs",
                 minWireVersion: 0,
-                maxWireVersion: 6
+                maxWireVersion: 17
             }]
         ],
         outcome: {
             servers: {
+                # A ignored for missing electionId
                 "a:27017": {
-                    type: "RSPrimary",
-                    setName: "rs",
-                    setVersion: 1,
-                    electionId:
-                },
-                "b:27017": {
                     type: "Unknown",
                     setName: ,
+                    setVersion: ,
                     electionId:
                 },
+                "b:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId: { "$oid": "000000000000000000000002" }
+                },
                 "c:27017": {
                     type: "Unknown",
                     setName: ,
@@ -142,23 +144,23 @@ phases: [
                 setVersion: 1,
                 electionId: {"$oid": "000000000000000000000001"},
                 minWireVersion: 0,
-                maxWireVersion: 6
+                maxWireVersion: 17
             }]
         ],
         outcome: {
             servers: {
-                # Still primary.
                 "a:27017": {
-                    type: "RSPrimary",
-                    setName: "rs",
-                    setVersion: 1,
-                    electionId:
-                },
-                "b:27017": {
                     type: "Unknown",
                     setName: ,
+                    setVersion: ,
                     electionId:
                 },
+                "b:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId: { "$oid": "000000000000000000000002" }
+                },
                 "c:27017": {
                     type: "Unknown",
                     setName: ,

data/spec/spec_tests/data/sdam/rs/secondary_ignore_ok_0-pre-6.0.yml

@@ -0,0 +1,87 @@
+description: "Pre 6.0 New primary"
+
+uri: "mongodb://a,b/?replicaSet=rs"
+
+phases: [
+
+    {
+        responses: [
+
+                ["a:27017", {
+
+                    ok: 1,
+                    helloOk: true,
+                    isWritablePrimary: true,
+                    setName: "rs",
+                    hosts: ["a:27017", "b:27017"],
+                    minWireVersion: 0,
+                    maxWireVersion: 6
+                }],
+                ["b:27017", {
+
+                    ok: 1,
+                    helloOk: true,
+                    isWritablePrimary: false,
+                    secondary: true,
+                    setName: "rs",
+                    hosts: ["a:27017", "b:27017"],
+                    minWireVersion: 0,
+                    maxWireVersion: 6
+                }]
+        ],
+
+        outcome: {
+
+            servers: {
+
+                "a:27017": {
+
+                    type: "RSPrimary",
+                    setName: "rs"
+                },
+
+                "b:27017": {
+
+                    type: "RSSecondary",
+                    setName: "rs"
+                }
+            },
+            topologyType: "ReplicaSetWithPrimary",
+            logicalSessionTimeoutMinutes: null,
+            setName: "rs"
+        }
+    },
+
+    {
+        responses: [
+
+                ["b:27017", {
+
+                    ok: 0,
+                    minWireVersion: 0,
+                    maxWireVersion: 6
+                }]
+        ],
+
+        outcome: {
+
+            servers: {
+                "a:27017": {
+
+                    type: "RSPrimary",
+                    setName: "rs"
+                },
+
+                "b:27017": {
+
+                    type: "Unknown",
+                    setName:
+                }
+
+            },
+            topologyType: "ReplicaSetWithPrimary",
+            logicalSessionTimeoutMinutes: null,
+            setName: "rs"
+        }
+    }
+]

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

@@ -1,4 +1,4 @@
-description: "New primary"
+description: "Secondary ignored when ok is zero"
 
 uri: "mongodb://a,b/?replicaSet=rs"
 

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

@@ -0,0 +1,101 @@
+description: Set version rolls back after new primary with higher election Id
+uri: mongodb://a/?replicaSet=rs
+phases:
+  - responses:
+      - - a:27017
+        - ok: 1
+          helloOk: true
+          isWritablePrimary: true
+          hosts:
+            - a:27017
+            - b:27017
+          setName: rs
+          setVersion: 2
+          electionId:
+            $oid: '000000000000000000000001'
+          minWireVersion: 0
+          maxWireVersion: 17
+    outcome:
+      servers:
+        a:27017:
+          type: RSPrimary
+          setName: rs
+          setVersion: 2
+          electionId:
+            $oid: '000000000000000000000001'
+        b:27017:
+          type: Unknown
+          setName: null
+          electionId: null
+      topologyType: ReplicaSetWithPrimary
+      logicalSessionTimeoutMinutes: null
+      setName: rs
+      maxSetVersion: 2
+      maxElectionId:
+        $oid: '000000000000000000000001'
+  - # Response from new primary with newer election Id
+    responses:
+      - - b:27017
+        - ok: 1
+          helloOk: true
+          isWritablePrimary: true
+          hosts:
+            - a:27017
+            - b:27017
+          setName: rs
+          setVersion: 1
+          electionId:
+            $oid: '000000000000000000000002'
+          minWireVersion: 0
+          maxWireVersion: 17
+    outcome:
+      servers:
+        a:27017:
+          type: Unknown
+          setName: null
+          electionId: null
+        b:27017:
+          type: RSPrimary
+          setName: rs
+          setVersion: 1
+          electionId:
+            $oid: '000000000000000000000002'
+      topologyType: ReplicaSetWithPrimary
+      logicalSessionTimeoutMinutes: null
+      setName: rs
+      maxSetVersion: 1
+      maxElectionId:
+        $oid: '000000000000000000000002'
+  - # Response from stale primary
+    responses:
+      - - a:27017
+        - ok: 1
+          helloOk: true
+          isWritablePrimary: true
+          hosts:
+            - a:27017
+            - b:27017
+          setName: rs
+          setVersion: 2
+          electionId:
+            $oid: '000000000000000000000001'
+          minWireVersion: 0
+          maxWireVersion: 17
+    outcome:
+      servers:
+        a:27017:
+          type: Unknown
+          setName: null
+          electionId: null
+        b:27017:
+          type: RSPrimary
+          setName: rs
+          setVersion: 1
+          electionId:
+            $oid: '000000000000000000000002'
+      topologyType: ReplicaSetWithPrimary
+      logicalSessionTimeoutMinutes: null
+      setName: rs
+      maxSetVersion: 1
+      maxElectionId:
+        $oid: '000000000000000000000002'

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

@@ -0,0 +1,78 @@
+description: "setVersion version that is equal is treated the same as greater than if there is no electionId"
+
+uri: "mongodb://a/?replicaSet=rs"
+
+phases: [
+
+    # Primary A is discovered and tells us about B.
+    {
+        responses: [
+            ["a:27017", {
+                ok: 1,
+                helloOk: true,
+                isWritablePrimary: true,
+                hosts: ["a:27017", "b:27017"],
+                setName: "rs",
+                setVersion: 1,
+                minWireVersion: 0,
+                maxWireVersion: 17
+            }]
+        ],
+
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId:
+                },
+                "b:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                }
+            },
+            topologyType: "ReplicaSetWithPrimary",
+            logicalSessionTimeoutMinutes: null,
+            setName: "rs",
+            maxSetVersion: 1,
+        }
+    },
+
+    # B is elected, its setVersion is older so it is stale
+    {
+        responses: [
+            ["b:27017", {
+                ok: 1,
+                helloOk: true,
+                isWritablePrimary: true,
+                hosts: ["a:27017", "b:27017"],
+                setName: "rs",
+                setVersion: 1,
+                minWireVersion: 0,
+                maxWireVersion: 17
+            }]
+        ],
+
+        outcome: {
+            servers: {
+                "a:27017": {
+                    type: "Unknown",
+                    setName: ,
+                    electionId:
+                },
+                "b:27017": {
+                    type: "RSPrimary",
+                    setName: "rs",
+                    setVersion: 1,
+                    electionId:
+                }
+            },
+            topologyType: "ReplicaSetWithPrimary",
+            logicalSessionTimeoutMinutes: null,
+            setName: "rs",
+            maxSetVersion: 1, # Max is still 1, there wasn't an actual larger setVersion seen
+        }
+    }
+]