Uploaded image for project: 'Kudu'
  1. Kudu
  2. KUDU-2455

Don't crash when flushing cmeta with empty config

Attach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 1.7.0
    • Fix Version/s: None
    • Component/s: consensus
    • Labels:
      None

      Description

      We saw a case in the wild where there was a missing cmeta file due to a crash in the middle of a tablet copy, resulting in the replica starting up in a tombstoned state. When the replica attempted to vote while tombstoned, it crashed when syncing its vote to disk with the message "RaftConfig must have at least one peer" from VerifyRaftConfig().

      I think we can support this scenario. In order to do that, we should remove that check from that location, put it somewhere else (perhaps in ChangeConfig()), and add a test for this scenario.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              mpercy Mike Percy

              Dates

              • Created:
                Updated:

                Issue deployment