summaryrefslogtreecommitdiff
path: root/jstests/noPassthrough/reconfig_for_psa_set_shell.js
diff options
context:
space:
mode:
Diffstat (limited to 'jstests/noPassthrough/reconfig_for_psa_set_shell.js')
-rw-r--r--jstests/noPassthrough/reconfig_for_psa_set_shell.js62
1 files changed, 58 insertions, 4 deletions
diff --git a/jstests/noPassthrough/reconfig_for_psa_set_shell.js b/jstests/noPassthrough/reconfig_for_psa_set_shell.js
index 2b0ba125432..a40704ffdaf 100644
--- a/jstests/noPassthrough/reconfig_for_psa_set_shell.js
+++ b/jstests/noPassthrough/reconfig_for_psa_set_shell.js
@@ -8,6 +8,8 @@
(function() {
'use strict';
+load("jstests/replsets/rslib.js");
+
// Start up a PSA set with the secondary having 'votes: 0' and 'priority: 0'.
const rst = new ReplSetTest({
name: jsTestName(),
@@ -29,10 +31,62 @@ let reconfigScript = `assert.commandFailedWithCode(rs.reconfig(${
let result = runMongoProgram('mongo', '--port', primary.port, '--eval', reconfigScript);
assert.eq(0, result, `reconfig did not fail with expected error code`);
-// Verify that calling 'reconfigForPSASet()' will succeed.
-reconfigScript = `assert.commandWorked(rs.reconfigForPSASet(1, ${tojson(config)}))`;
-result = runMongoProgram('mongo', '--port', primary.port, '--eval', reconfigScript);
-assert.eq(0, result, `reconfig did not succeed as expected`);
+const runReconfigForPSASet = (memberIndex, config, shouldSucceed, endPriority = 1) => {
+ jsTestLog(`Testing with memberIndex ${memberIndex} and config ${tojson(config)}`);
+
+ const reconfigScript =
+ `assert.commandWorked(rs.reconfigForPSASet(${memberIndex}, ${tojson(config)}))`;
+ const result = runMongoProgram('mongo', '--port', primary.port, '--eval', reconfigScript);
+ if (shouldSucceed) {
+ assert.eq(0, result, 'expected reconfigToPSASet to succeed, but it failed');
+
+ // Wait for the new config to be committed.
+ assert.soonNoExcept(() => isConfigCommitted(primary));
+
+ const replSetGetConfig =
+ assert.commandWorked(primary.adminCommand({replSetGetConfig: 1})).config;
+ assert.eq(1, replSetGetConfig.members[1].votes);
+ assert.eq(endPriority, replSetGetConfig.members[1].priority);
+
+ // Reset the config back to the original config.
+ originalConfig.members[memberIndex].votes = 0;
+ originalConfig.members[memberIndex].priority = 0;
+ const reconfigToOriginalConfig =
+ `assert.commandWorked(rs.reconfig(${tojson(originalConfig)}))`;
+ assert.eq(
+ 0,
+ runMongoProgram('mongo', '--port', primary.port, '--eval', reconfigToOriginalConfig));
+ assert.soonNoExcept(() => isConfigCommitted(primary));
+ } else {
+ assert.neq(0, result, 'expected reconfigToPSASet to fail, but it succeeded');
+ }
+};
+
+// Succeed with a reconfig to a standard PSA set, where the secondary has 'votes: 1' and 'priority:
+// 1'.
+jsTestLog("Testing reconfigForPSASet() succeeded: standard PSA set");
+runReconfigForPSASet(1, config, true /* shouldSucceed */);
+
+// Fail when 'memberIndex' does not refer to a node in the new config.
+jsTestLog("Testing reconfigForPSASet() failed: memberIndex out of bounds");
+runReconfigForPSASet(3, config, false /* shouldSucceed */);
+
+// Fail when the node in the new config at 'memberIndex' is not a voter.
+jsTestLog("Testing reconfigForPSASet() failed: node at memberIndex is not a voter");
+config.members[1].votes = 0;
+config.members[1].priority = 0;
+runReconfigForPSASet(1, config, false /* shouldSucceed */);
+
+// Test that reconfigToPSASet() will succeed when we are adding a new node at a specific
+// 'memberIndex'.
+jsTestLog("Testing reconfigForPSASet() succeeded: adding new node");
+
+// First remove the node at index 1 to simulate a two-node replica set.
+config = rst.getReplSetConfigFromNode();
+const filteredMembers = config.members.filter(member => member._id !== 1);
+config.members = filteredMembers;
+config.version += 1;
+assert.commandWorked(primary.adminCommand({replSetReconfig: config}));
const replSetGetConfig = assert.commandWorked(primary.adminCommand({replSetGetConfig: 1})).config;
assert.eq(1, replSetGetConfig.members[1].votes);