small fixes to #3985

Author: david-crespo

nexus/tests/integration_tests/ip_pools.rs

@@ -288,9 +288,9 @@ async fn test_ip_pool_with_silo(cptestctx: &ControlPlaneTestContext) {
     };
     let created_pool = create_pool(client, &params).await;
     assert_eq!(created_pool.identity.name, "p0");
-    assert!(created_pool.silo_id.is_some());
 
-    let silo_id = created_pool.silo_id.unwrap();
+    let silo_id =
+        created_pool.silo_id.expect("Expected pool to have a silo_id");
 
     // now we'll create another IP pool using that silo ID
     let params = IpPoolCreate {

nexus/types/src/external_api/params.rs

@@ -736,7 +736,7 @@ pub struct IpPoolCreate {
     /// silo can draw from that pool.
     pub silo: Option<NameOrId>,
 
-    /// Whether the IP pool is considered a default pool for its scope (fleet,
+    /// Whether the IP pool is considered a default pool for its scope (fleet
     /// or silo). If a pool is marked default and is associated with a silo,
     /// instances created in that silo will draw IPs from that pool unless
     /// another pool is specified at instance create time.