rename `exit_pool` > `validator_change_pool` (#5679)

The `ExitPool` was renamed to `ValidatorChangePool` with Capella, but
the files were still using the previous name. Rename for consistency.

beacon_chain/beacon_node.nim

@@ -20,7 +20,7 @@ import
   ./el/el_manager,
   ./consensus_object_pools/[
     blockchain_dag, blob_quarantine, block_quarantine, consensus_manager,
-    exit_pool, attestation_pool, sync_committee_msg_pool],
+    attestation_pool, sync_committee_msg_pool, validator_change_pool],
   ./spec/datatypes/[base, altair],
   ./spec/eth2_apis/dynamic_fee_recipients,
   ./sync/[sync_manager, request_manager],
@@ -32,10 +32,10 @@ import
 export
   osproc, chronos, presto, action_tracker,
   beacon_clock, beacon_chain_db, conf, light_client,
-  attestation_pool, sync_committee_msg_pool, validator_pool,
+  attestation_pool, sync_committee_msg_pool, validator_change_pool,
   eth2_network, el_manager, request_manager, sync_manager,
   eth2_processor, optimistic_processor, blockchain_dag, block_quarantine,
-  base, exit_pool,  message_router, validator_monitor,
+  base, message_router, validator_monitor, validator_pool,
   consensus_manager, dynamic_fee_recipients
 
 type

beacon_chain/consensus_object_pools/README.md

@@ -22,5 +22,5 @@ The pools presenet in this folder are:
   - blockchain_dag: an in-memory direct-acyclic graph of fully validated and verified blockchain candidates with the tail being the last finalized epoch. A block in the DAG MUST be in the fork choice and a block in the fork choice MUST be in the DAG (except for orphans following finalization). On finalization non-empty epoch blocks are stored in the beacon_chain_db.
 - attestation_pool:
   Handles the attestation received from gossip and collect them for fork choice.
-- exit_pool:
+- validator_change_pool:
   Handle voluntary exits and forced exits (attester slashings and proposer slashings)

beacon_chain/gossip_processing/eth2_processor.nim

@@ -15,16 +15,18 @@ import
   ../spec/datatypes/[altair, phase0, deneb],
   ../consensus_object_pools/[
     blob_quarantine, block_clearance, block_quarantine, blockchain_dag,
-    exit_pool, attestation_pool, light_client_pool, sync_committee_msg_pool],
+    attestation_pool, light_client_pool, sync_committee_msg_pool,
+    validator_change_pool],
   ../validators/validator_pool,
   ../beacon_clock,
   "."/[gossip_validation, block_processor, batch_validation],
   ../nimbus_binary_common
 
 export
-  results, taskpools, block_clearance, blockchain_dag, exit_pool, attestation_pool,
-  light_client_pool, sync_committee_msg_pool, validator_pool, beacon_clock,
-  gossip_validation, block_processor, batch_validation, block_quarantine
+  results, taskpools, block_clearance, blockchain_dag, attestation_pool,
+  light_client_pool, sync_committee_msg_pool, validator_change_pool,
+  validator_pool, beacon_clock, gossip_validation, block_processor,
+  batch_validation, block_quarantine
 
 logScope: topics = "gossip_eth2"
 

beacon_chain/gossip_processing/gossip_validation.nim

@@ -16,7 +16,8 @@ import
     beaconstate, state_transition_block, forks, helpers, network, signatures],
   ../consensus_object_pools/[
     attestation_pool, blockchain_dag, blob_quarantine, block_quarantine,
-    exit_pool, spec_cache, light_client_pool, sync_committee_msg_pool],
+    spec_cache, light_client_pool, sync_committee_msg_pool,
+    validator_change_pool],
   ".."/[beacon_clock],
   ./batch_validation
 

beacon_chain/rpc/rest_beacon_api.nim

@@ -12,7 +12,7 @@ import
   ./rest_utils,
   ./state_ttl_cache,
   ../beacon_node,
-  ../consensus_object_pools/[blockchain_dag, exit_pool, spec_cache],
+  ../consensus_object_pools/[blockchain_dag, spec_cache, validator_change_pool],
   ../spec/[deposit_snapshots, eth2_merkleization, forks, network, validator],
   ../spec/datatypes/[phase0, altair, deneb],
   ../validators/message_router_mev

beacon_chain/validators/beacon_validators.nim

@@ -32,8 +32,8 @@ import
     eth2_merkleization, forks, helpers, network, signatures, state_transition,
     validator],
   ../consensus_object_pools/[
-    spec_cache, blockchain_dag, block_clearance, attestation_pool, exit_pool,
-    sync_committee_msg_pool, consensus_manager],
+    spec_cache, blockchain_dag, block_clearance, attestation_pool,
+    sync_committee_msg_pool, validator_change_pool, consensus_manager],
   ../el/el_manager,
   ../networking/eth2_network,
   ../sszdump, ../sync/sync_manager,

tests/all_tests.nim

@@ -26,7 +26,6 @@ import # Unit test
   ./test_el_manager,
   ./test_el_conf,
   ./test_eth2_ssz_serialization,
-  ./test_exit_pool,
   ./test_forks,
   ./test_gossip_transition,
   ./test_gossip_validation,
@@ -45,6 +44,7 @@ import # Unit test
   ./test_statediff,
   ./test_sync_committee_pool,
   ./test_sync_manager,
+  ./test_validator_change_pool,
   ./test_validator_pool,
   ./test_zero_signature,
   ./test_signing_node,

tests/test_validator_change_pool.nim

@@ -11,7 +11,7 @@ import
   ../beacon_chain/spec/[
     datatypes/base, forks, presets, signatures, state_transition],
   ../beacon_chain/consensus_object_pools/[
-    block_quarantine, blockchain_dag, exit_pool],
+    block_quarantine, blockchain_dag, validator_change_pool],
   "."/[testutil, testblockutil, testdbutil]
 
 func makeSignedBeaconBlockHeader(