From e66373f938146ffe00334489492e8c53546bbcaa Mon Sep 17 00:00:00 2001 From: Ruchitha Rajaghatta <77162985+ruchitharajaghatta@users.noreply.github.com> Date: Tue, 21 Apr 2026 14:12:18 -0400 Subject: [PATCH] SERVER-120732: replSetInitiate should error (#51826) GitOrigin-RevId: 28032828d95915808dd95accc55afb0da2c3ef78 --- jstests/libs/replsettest.js | 9 ++++----- .../query/oplog_scan_optimizations_do_not_conflict.js | 3 ++- .../server_status/server_status_change_stream_metrics.js | 3 ++- 3 files changed, 8 insertions(+), 7 deletions(-) diff --git a/jstests/libs/replsettest.js b/jstests/libs/replsettest.js index 86214a0c361..61ff40c973d 100644 --- a/jstests/libs/replsettest.js +++ b/jstests/libs/replsettest.js @@ -1812,12 +1812,11 @@ export class ReplSetTest { } /** - * Runs replSetInitiate on the first node of the replica set. - * - * TODO (SERVER-109841): Replsetinitiate is currently a no-op command for disagg. Determine the - * next steps for this function if additional functionality is to be incorporated. + * Blocks until the set is initialized with a primary. + * TODO SERVER-124472: Determine if initiateForDisagg needs additional functionality to + * match ASC ReplSetTest.initiate. */ - initiateForDisagg(cfg, initCmd) { + initiateForDisagg() { const startTime = new Date(); // Measure the execution time of this function. // Blocks until there is a primary. We use a faster retry interval here since we expect the diff --git a/jstests/noPassthrough/query/oplog_scan_optimizations_do_not_conflict.js b/jstests/noPassthrough/query/oplog_scan_optimizations_do_not_conflict.js index 566d5dd3196..a7dfd323af3 100644 --- a/jstests/noPassthrough/query/oplog_scan_optimizations_do_not_conflict.js +++ b/jstests/noPassthrough/query/oplog_scan_optimizations_do_not_conflict.js @@ -13,7 +13,8 @@ rst.initiate(); // Background query analysis operations such as index creation may throw off // the checks between the replSetGetStatus result and the last oplog entry. -// TODO SERVER-109841: This should be deleted if we move this into ReplSetTest. +// TODO SERVER-124430: This should be deleted if we move this into +// ReplSetTest initiateForDisagg. rst.waitForQueryAnalysisWriterSetup(); const db = rst.getPrimary().getDB("oplog_scan_optimizations"); diff --git a/jstests/noPassthrough/query/server_status/server_status_change_stream_metrics.js b/jstests/noPassthrough/query/server_status/server_status_change_stream_metrics.js index ebfd51e453a..abdfe170737 100644 --- a/jstests/noPassthrough/query/server_status/server_status_change_stream_metrics.js +++ b/jstests/noPassthrough/query/server_status/server_status_change_stream_metrics.js @@ -23,7 +23,8 @@ rst.initiate(); // Background query analysis operations such as index creation may throw off // the checks between the replSetGetStatus result and the last oplog entry. -// TODO SERVER-109841: This should be deleted if we move this into ReplSetTest. +// TODO SERVER-124430: This should be deleted if we move this into +// ReplSetTest initiateForDisagg. rst.waitForQueryAnalysisWriterSetup(); const db = rst.getPrimary().getDB(jsTest.name());