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

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 4.0.6, 4.1.7
    • Affects Version/s: None
    • Component/s: None
    • Fully Compatible
    • ALL
    • v4.0
    • Storage NYC 2018-12-17
    • 12
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      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.

            Assignee:
            Louis Williams
            Reporter:
            Louis Williams
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: