Add PYTHON-2673.patch

NoahStapp · NoahStapp · commit c78dfc4ad581 · 2025-07-30T11:00:35.000-04:00
diff --git a/.evergreen/spec-patch/PYTHON-2673.patch b/.evergreen/spec-patch/PYTHON-2673.patch
@@ -0,0 +1,71 @@
+diff --git a/test/load_balancer/cursors.json b/test/load_balancer/cursors.json
+index 4b492f7d8..e44fad1bc 100644
+--- a/test/load_balancer/cursors.json
++++ b/test/load_balancer/cursors.json
+@@ -386,4 +386,4 @@
+     },
+     {
++      "description": "pinned connections are not returned after an network error during getMore",
+-      "description": "pinned connections are returned after an network error during getMore",
+       "operations": [
+@@ -419,9 +419,9 @@
+         },
+         {
+           "name": "assertNumberConnectionsCheckedOut",
+           "object": "testRunner",
+           "arguments": {
+             "client": "client0",
++            "connections": 0
+-            "connections": 1
+           }
+         }
+@@ -678,4 +678,4 @@
+     },
+     {
++      "description": "pinned connections are not returned to the pool after a non-network error on getMore",
+-      "description": "pinned connections are returned to the pool after a non-network error on getMore",
+       "operations": [
+@@ -732,9 +732,9 @@
+         },
+         {
+           "name": "assertNumberConnectionsCheckedOut",
+           "object": "testRunner",
+           "arguments": {
+             "client": "client0",
++            "connections": 0
+-            "connections": 1
+           }
+         }
+diff --git a/test/load_balancer/sdam-error-handling.json b/test/load_balancer/sdam-error-handling.json
+index 4b492f7d8..e44fad1bc 100644
+--- a/test/load_balancer/sdam-error-handling.json
++++ b/test/load_balancer/sdam-error-handling.json
+@@ -367,23 +367,23 @@
+       "expectEvents": [
+         {
+           "client": "singleClient",
+           "eventType": "cmap",
+           "events": [
+             {
+               "connectionCreatedEvent": {}
+             },
++            {
++              "poolClearedEvent": {}
++            },
+             {
+               "connectionClosedEvent": {
+                 "reason": "error"
+               }
+             },
+             {
+               "connectionCheckOutFailedEvent": {
+                 "reason": "connectionError"
+               }
++            }
+-            },
+-            {
+-              "poolClearedEvent": {}
+-            }
+           ]
+         }
+       ]
diff --git a/test/load_balancer/cursors.json b/test/load_balancer/cursors.json
@@ -316,7 +316,7 @@
           "object": "testRunner",
           "arguments": {
             "client": "client0",
-            "connections": 0
+            "connections": 1
           }
         }
       ],
@@ -385,7 +385,7 @@
       ]
     },
     {
-      "description": "pinned connections are not returned after an network error during getMore",
+      "description": "pinned connections are returned after an network error during getMore",
       "operations": [
         {
           "name": "failPoint",
@@ -461,7 +461,7 @@
           "object": "testRunner",
           "arguments": {
             "client": "client0",
-            "connections": 0
+            "connections": 1
           }
         }
       ],
@@ -677,7 +677,7 @@
       ]
     },
     {
-      "description": "pinned connections are not returned to the pool after a non-network error on getMore",
+      "description": "pinned connections are returned to the pool after a non-network error on getMore",
       "operations": [
         {
           "name": "failPoint",
@@ -745,7 +745,7 @@
           "object": "testRunner",
           "arguments": {
             "client": "client0",
-            "connections": 0
+            "connections": 1
           }
         }
       ],
@@ -854,7 +854,7 @@
           "object": "testRunner",
           "arguments": {
             "client": "client0",
-            "connections": 0
+            "connections": 1
           }
         }
       ],
diff --git a/test/load_balancer/sdam-error-handling.json b/test/load_balancer/sdam-error-handling.json
@@ -372,9 +372,6 @@
             {
               "connectionCreatedEvent": {}
             },
-            {
-              "poolClearedEvent": {}
-            },
             {
               "connectionClosedEvent": {
                 "reason": "error"
@@ -384,6 +381,9 @@
               "connectionCheckOutFailedEvent": {
                 "reason": "connectionError"
               }
+            },
+            {
+              "poolClearedEvent": {}
             }
           ]
         }

Original file line number	Diff line number	Diff line change
`@@ -316,7 +316,7 @@`
`316`	`316`	`"object": "testRunner",`
`317`	`317`	`"arguments": {`
`318`	`318`	`"client": "client0",`
`319`		`- "connections": 0`
	`319`	`+ "connections": 1`
`320`	`320`	`}`
`321`	`321`	`}`
`322`	`322`	`],`
`@@ -385,7 +385,7 @@`
`385`	`385`	`]`
`386`	`386`	`},`
`387`	`387`	`{`
`388`		`- "description": "pinned connections are not returned after an network error during getMore",`
	`388`	`+ "description": "pinned connections are returned after an network error during getMore",`
`389`	`389`	`"operations": [`
`390`	`390`	`{`
`391`	`391`	`"name": "failPoint",`
`@@ -461,7 +461,7 @@`
`461`	`461`	`"object": "testRunner",`
`462`	`462`	`"arguments": {`
`463`	`463`	`"client": "client0",`
`464`		`- "connections": 0`
	`464`	`+ "connections": 1`
`465`	`465`	`}`
`466`	`466`	`}`
`467`	`467`	`],`
`@@ -677,7 +677,7 @@`
`677`	`677`	`]`
`678`	`678`	`},`
`679`	`679`	`{`
`680`		`- "description": "pinned connections are not returned to the pool after a non-network error on getMore",`
	`680`	`+ "description": "pinned connections are returned to the pool after a non-network error on getMore",`
`681`	`681`	`"operations": [`
`682`	`682`	`{`
`683`	`683`	`"name": "failPoint",`
`@@ -745,7 +745,7 @@`
`745`	`745`	`"object": "testRunner",`
`746`	`746`	`"arguments": {`
`747`	`747`	`"client": "client0",`
`748`		`- "connections": 0`
	`748`	`+ "connections": 1`
`749`	`749`	`}`
`750`	`750`	`}`
`751`	`751`	`],`
`@@ -854,7 +854,7 @@`
`854`	`854`	`"object": "testRunner",`
`855`	`855`	`"arguments": {`
`856`	`856`	`"client": "client0",`
`857`		`- "connections": 0`
	`857`	`+ "connections": 1`
`858`	`858`	`}`
`859`	`859`	`}`
`860`	`860`	`],`
Original file line number	Diff line number	Diff line change
`@@ -372,9 +372,6 @@`
`372`	`372`	`{`
`373`	`373`	`"connectionCreatedEvent": {}`
`374`	`374`	`},`
`375`		`- {`
`376`		`- "poolClearedEvent": {}`
`377`		`- },`
`378`	`375`	`{`
`379`	`376`	`"connectionClosedEvent": {`
`380`	`377`	`"reason": "error"`
`@@ -384,6 +381,9 @@`
`384`	`381`	`"connectionCheckOutFailedEvent": {`
`385`	`382`	`"reason": "connectionError"`
`386`	`383`	`}`
	`384`	`+ },`
	`385`	`+ {`
	`386`	`+ "poolClearedEvent": {}`
`387`	`387`	`}`
`388`	`388`	`]`
`389`	`389`	`}`