Rename allow ip to ip allow

docs/recipes/register_cluster.adoc

@@ -71,10 +71,10 @@ This creates read/write credentials on all of the registered clusters that match
 Finally we need the cluster to allow requests from our IP address, this can be done as follows:
 
 ```
-> curl ipconfig.me | allow ip --clusters .*
+> curl ifconfig.me | ip allow --clusters .*
 ```
 
-The curl command returns our IP address, which we then pipe into `allow ip` and this is run against all of the registered clusters again.
+The curl command returns our IP address, which we then pipe into `ip allow` and this is run against all of the registered clusters again.
 Now all of the healthy clusters are registered and ready to use through the shell.
 
 

src/cli/allow_ip.rs

@@ -14,23 +14,23 @@ use std::sync::{Arc, Mutex};
 use tokio::time::Instant;
 
 #[derive(Clone)]
-pub struct AllowIP {
+pub struct IPAllow {
     state: Arc<Mutex<State>>,
 }
 
-impl crate::cli::AllowIP {
+impl crate::cli::IPAllow {
     pub fn new(state: Arc<Mutex<State>>) -> Self {
         Self { state }
     }
 }
 
-impl Command for crate::cli::AllowIP {
+impl Command for crate::cli::IPAllow {
     fn name(&self) -> &str {
-        "allow ip"
+        "ip allow"
     }
 
     fn signature(&self) -> Signature {
-        Signature::build("allow ip")
+        Signature::build("ip allow")
             .category(Category::Custom("couchbase".to_string()))
             .named(
                 "clusters",
@@ -46,7 +46,7 @@ impl Command for crate::cli::AllowIP {
     }
 
     fn usage(&self) -> &str {
-        "Adds IP address to allowlist on a Capella cluster"
+        "Adds IP address to allow list on a Capella cluster"
     }
 
     fn run(
@@ -56,11 +56,11 @@ impl Command for crate::cli::AllowIP {
         call: &Call,
         input: PipelineData,
     ) -> Result<PipelineData, ShellError> {
-        allow_ip(self.state.clone(), engine_state, stack, call, input)
+        ip_allow(self.state.clone(), engine_state, stack, call, input)
     }
 }
 
-fn allow_ip(
+fn ip_allow(
     state: Arc<Mutex<State>>,
     engine_state: &EngineState,
     stack: &mut Stack,
@@ -73,7 +73,7 @@ fn allow_ip(
     let cluster_identifiers = cluster_identifiers_from(engine_state, stack, &state, call, true)?;
     let guard = state.lock().unwrap();
 
-    debug!("Running allow_ip");
+    debug!("Running ip_allow");
 
     let ip_address = match input.into_value(span)? {
         Value::String { val, .. } => format_ip_address(val),

src/cli/mod.rs

@@ -84,7 +84,7 @@ mod vector_enrich_text;
 mod vector_search;
 mod version;
 
-pub use allow_ip::AllowIP;
+pub use allow_ip::IPAllow;
 pub use analytics::Analytics;
 pub use analytics_buckets::AnalyticsBuckets;
 pub use analytics_datasets::AnalyticsDatasets;

src/main.rs

@@ -708,7 +708,7 @@ fn make_state(
 fn merge_couchbase_delta(context: &mut EngineState, state: Arc<Mutex<State>>) {
     let delta = {
         let mut working_set = StateWorkingSet::new(context);
-        working_set.add_decl(Box::new(AllowIP::new(state.clone())));
+        working_set.add_decl(Box::new(IPAllow::new(state.clone())));
         working_set.add_decl(Box::new(Analytics::new(state.clone())));
         working_set.add_decl(Box::new(AnalyticsBuckets::new(state.clone())));
         working_set.add_decl(Box::new(AnalyticsDatasets::new(state.clone())));