cherry pick 6128

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Author: ti-chi-bot

server/schedule/region_scatterer.go

@@ -383,9 +383,11 @@ func allowLeader(fit *placement.RegionFit, peer *metapb.Peer) bool {
 	case metapb.PeerRole_Learner, metapb.PeerRole_DemotingVoter:
 		return false
 	}
-
-	rule := fit.GetRuleFit(peer.GetId()).Rule
-	switch rule.Role {
+	peerFit := fit.GetRuleFit(peer.GetId())
+	if peerFit == nil || peerFit.Rule == nil {
+		return false
+	}
+	switch peerFit.Rule.Role {
 	case placement.Voter, placement.Leader:
 		return true
 	}

server/schedule/region_scatterer_test.go

@@ -709,6 +709,11 @@ func TestBalanceRegion(t *testing.T) {
 		re.Equal(uint64(150), scatterer.ordinaryEngine.selectedPeer.Get(i, group))
 		re.Equal(uint64(50), scatterer.ordinaryEngine.selectedLeader.Get(i, group))
 	}
+	// Test for unhealthy region
+	// ref https://github.com/tikv/pd/issues/6099
+	region := tc.AddLeaderRegion(1500, 2, 3, 4, 6)
+	op := scatterer.scatterRegion(region, group)
+	re.False(isPeerCountChanged(op))
 }
 
 func isPeerCountChanged(op *operator.Operator) bool {