SERVER-31128 Skip validate on NotMaster error response.

This change is in preparation for running the validate command against
secondaries in a replica set.

The fuzzer test suites require that the validate command is only run
when the server's featureCompatibilityVersion=3.4. Otherwise, it is
possible that previously inserted Decimal128 data could lead to an
actually innocuous data consistency failure.

1 files changed, 12 insertions, 2 deletions

diff --git a/jstests/hooks/validate_collections.js b/jstests/hooks/validate_collections.js
index e64bb39d305..c5be1a230e8 100644
--- a/jstests/hooks/validate_collections.js
+++ b/jstests/hooks/validate_collections.js
@@ -56,8 +56,18 @@ function validateCollections(db, obj) {
             }
         }
 
-        setFeatureCompatibilityVersion(
-            adminDB, jsTest.options().forceValidationWithFeatureCompatibilityVersion);
+        try {
+            setFeatureCompatibilityVersion(
+                adminDB, jsTest.options().forceValidationWithFeatureCompatibilityVersion);
+        } catch (e) {
+            if (e.code === ErrorCodes.NotMaster) {
+                print('Skipping collection validation on ' + db.getMongo() + ' because the' +
+                      ' featureCompatibilityVersion cannot be changed while connected to a' +
+                      ' secondary');
+                return true;
+            }
+            throw e;
+        }
     }
 
     // Don't run validate on view namespaces.