Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-37078

Use awaitSecondaryNodes() instead of waitForState() to confirm a node is a secondary in repair_invalidates_replica_set_nodes.js

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0.6, 4.1.7
    • Component/s: None
    • Labels:
    • Backwards Compatibility:
      Fully Compatible
    • Operating System:
      ALL
    • Backport Requested:
      v4.0
    • Sprint:
      Storage NYC 2018-12-17
    • Linked BF Score:
      12

      Description

      The test disk/repair_invalidates_replica_set_config.js calls in to wt_file_helper.js to wait for a repaired node to initial sync and become a secondary.

      This uses waitForState(node, SECONDARY), which calls "replSetGetStatus" on the primary. If the primary is unaware a secondary was shut down, this will succeed, and the subsequent call to "replSetGetStatus" on the secondary will fail.

      This should be replaced with a call to ReplSetTest.awaitSecondaryNodes(), which calls isMaster on each node to confirm its state, instead of just the primary.

        Attachments

          Activity

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: