Skip to content

Commit

Permalink
UseVersionAssignmentTablesV3 flag is required for DQ
Browse files Browse the repository at this point in the history
  • Loading branch information
mystenmark committed Feb 12, 2025
1 parent b5fc2d5 commit c39552c
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 4 additions & 0 deletions crates/sui-core/src/authority/authority_per_epoch_store.rs
Original file line number Diff line number Diff line change
Expand Up @@ -823,6 +823,10 @@ impl AuthorityPerEpochStore {
epoch_id
);
let epoch_start_configuration = Arc::new(epoch_start_configuration);
assert!(
epoch_start_configuration.use_version_assignment_tables_v3(),
"use_version_assignment_tables_v3 must be already be enabled for DataQuarantining"
);
info!("epoch flags: {:?}", epoch_start_configuration.flags());
metrics.current_epoch.set(epoch_id as i64);
metrics
Expand Down
4 changes: 2 additions & 2 deletions crates/sui-e2e-tests/tests/reconfiguration_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -737,8 +737,8 @@ async fn test_epoch_flag_upgrade() {
return None;
}

// start with no flags
let flags: Vec<EpochFlag> = vec![EpochFlag::DataQuarantineFromBeginningOfEpoch];
// start with only UseVersionAssignmentTablesV3
let flags: Vec<EpochFlag> = vec![EpochFlag::UseVersionAssignmentTablesV3];
Some(flags)
});

Expand Down

0 comments on commit c39552c

Please sign in to comment.