Merge pull request #412 from consensus-shipyard/ipc-304-permissioning

IPC-304: add permissioned flag for new contract version

Author: adlrocha

Cargo.lock

@@ -41,6 +41,7 @@ impl CreateSubnet {
                     .active_validators_limit
                     .unwrap_or(DEFAULT_ACTIVE_VALIDATORS),
                 f64_to_token_amount(arguments.min_cross_msg_fee)?,
+                arguments.permissioned,
             )
             .await?;
 
@@ -96,4 +97,9 @@ pub struct CreateSubnetArgs {
         help = "Minimum fee for cross-net messages in subnet (in whole FIL)"
     )]
     pub min_cross_msg_fee: f64,
+    #[arg(
+        long,
+        help = "Deploy static network where validators can't join in a permissionless manner"
+    )]
+    pub permissioned: bool,
 }

ipc/cli/src/commands/subnet/create.rs

@@ -40,7 +40,7 @@ pub struct GlobalArguments {
     config_path: Option<String>,
 
     /// Set the FVM Address Network. It's value affects whether `f` (main) or `t` (test) prefixed addresses are accepted.
-    #[arg(short, long, default_value = "mainnet", env = "NETWORK", value_parser = parse_network)]
+    #[arg(short, long, default_value = "testnet", env = "NETWORK", value_parser = parse_network)]
     pub network: Network,
 }
 

ipc/cli/src/lib.rs

@@ -239,6 +239,7 @@ impl IpcProvider {
         bottomup_check_period: ChainEpoch,
         active_validators_limit: u16,
         min_cross_msg_fee: TokenAmount,
+        permissioned: bool,
     ) -> anyhow::Result<Address> {
         let conn = match self.connection(&parent) {
             None => return Err(anyhow!("target parent subnet not found")),
@@ -251,12 +252,13 @@ impl IpcProvider {
         let constructor_params = ConstructParams {
             parent,
             ipc_gateway_addr: subnet_config.gateway_addr(),
-            consensus: ConsensusType::Mir,
+            consensus: ConsensusType::Fendermint,
             min_validators,
             min_validator_stake,
             bottomup_check_period,
             active_validators_limit,
             min_cross_msg_fee,
+            permissioned,
         };
 
         conn.manager()

ipc/provider/src/lib.rs

@@ -262,6 +262,7 @@ impl SubnetManager for EthSubnetManager {
             active_validators_limit: params.active_validators_limit,
             power_scale: 3,
             min_cross_msg_fee: ethers::types::U256::from(min_cross_msg_fee),
+            permissioned: params.permissioned,
         };
 
         log::info!("creating subnet on evm with params: {params:?}");

ipc/provider/src/manager/evm/manager.rs

@@ -24,16 +24,12 @@ pub struct ConstructParams {
     pub bottomup_check_period: ChainEpoch,
     pub active_validators_limit: u16,
     pub min_cross_msg_fee: TokenAmount,
+    pub permissioned: bool,
 }
 
 /// Consensus types supported by hierarchical consensus
 #[derive(PartialEq, Eq, Clone, Copy, Debug, Deserialize_repr, Serialize_repr)]
 #[repr(u64)]
 pub enum ConsensusType {
-    Mir,
-}
-
-#[derive(Clone, Debug, Serialize_tuple, Deserialize_tuple, PartialEq, Eq)]
-pub struct JoinParams {
-    pub validator_net_addr: String,
+    Fendermint,
 }