diff options
author | Judah Schvimer <judah.schvimer@10gen.com> | 2019-10-22 20:02:47 +0000 |
---|---|---|
committer | evergreen <evergreen@mongodb.com> | 2019-10-22 20:02:47 +0000 |
commit | b6f1f7f3c3455425d6169e9f9222eb4b9291697a (patch) | |
tree | a50f72cc6b723b31025f3effbc58bb71c350f2eb | |
parent | 992e55fb42122f718d11b80dbcdce95ce208f109 (diff) | |
download | mongo-b6f1f7f3c3455425d6169e9f9222eb4b9291697a.tar.gz |
SERVER-43843 skip validate during the rollback fuzzer "restartNode" command
-rw-r--r-- | jstests/replsets/libs/rollback_test.js | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/jstests/replsets/libs/rollback_test.js b/jstests/replsets/libs/rollback_test.js index e7e28e3502f..2bb45eb2cce 100644 --- a/jstests/replsets/libs/rollback_test.js +++ b/jstests/replsets/libs/rollback_test.js @@ -409,8 +409,7 @@ function RollbackTest(name = "RollbackTest", replSet) { return curSecondary; }; - this.restartNode = function( - nodeId, signal, startOptions, allowedExitCode, {skipValidation = false} = {}) { + this.restartNode = function(nodeId, signal, startOptions, allowedExitCode) { assert(signal === SIGKILL || signal === SIGTERM, `Received unknown signal: ${signal}`); assert.gte(nodeId, 0, "Invalid argument to RollbackTest.restartNode()"); @@ -433,7 +432,10 @@ function RollbackTest(name = "RollbackTest", replSet) { signal = SIGTERM; } - let opts = {skipValidation}; + // We may attempt to restart a node while it is in rollback or recovery, in which case + // the validation checks will fail. We will still validate collections during the + // RollbackTest's full consistency checks, so we do not lose much validation coverage. + let opts = {skipValidation: true}; if (allowedExitCode !== undefined) { Object.assign(opts, {allowedExitCode: allowedExitCode}); |