[KAFKA-15263] KRaftMigrationDriver can run the migration twice

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 3.5.0, 3.5.1
• Fix Version/s: 3.6.0, 3.5.2
• Assignee: David Arthur
• Reporter: David Arthur
• Created: 27/Jul/23 16:59
• Updated: 28/Jul/23 18:39
• Resolved: 28/Jul/23 18:39

Description

There is a narrow race condition in KRaftMigrationDriver where a PollEvent can run that sees the internal state as ZK_MIGRATION and is immediately followed by another poll event (due to external call to wakeup()) that results in two MigrateMetadataEvent being enqueued. 

Since MigrateMetadataEvent lacks a check on the internal state, this causes the metadata migration to occur twice. 

Issue Links

• links to: GitHub Pull Request #14115