fix: add motivation argument to remove API BN command (#513)

dfinity · Jun 21, 2024 · c833710 · c833710
1 parent b48ce72
commit c833710
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/rs/cli/src/cli.rs b/rs/cli/src/cli.rs
@@ -524,9 +524,13 @@ pub mod api_boundary_nodes {
 
  /// Decommission a set of API BNs and turn them again in unassigned nodes
  Remove {
- /// Node IDs to turn into API BNs
+ /// Node IDs of API BNs that should be turned into unassigned nodes again
  #[clap(long, num_args(1..), required = true)]
  nodes: Vec<PrincipalId>,
+
+ /// Motivation for removing the API BNs
+ #[clap(short, long, aliases = ["summary"], required = true)]
+ motivation: Option<String>,
  },
  }
 }

diff --git a/rs/cli/src/main.rs b/rs/cli/src/main.rs
@@ -418,15 +418,15 @@ async fn async_main() -> Result<(), anyhow::Error> {
  .await?;
  Ok(())
  }
- cli::api_boundary_nodes::Commands::Remove { nodes } => {
+ cli::api_boundary_nodes::Commands::Remove { nodes, motivation } => {
  runner_instance
  .ic_admin
  .propose_run(
  ic_admin::ProposeCommand::RemoveApiBoundaryNodes { nodes: nodes.to_vec() },
  ic_admin::ProposeOptions {
  title: Some(format!("Remove {} API boundary node(s)", nodes.clone().len())),
  summary: Some(format!("Remove {} API boundary node(s)", nodes.clone().len())),
- motivation: None,
+ motivation: motivation.clone(),
  },
  dry_run,
  )