Fix release log msg, check for owner on release

bonfire/bonfire.py

@@ -625,7 +625,7 @@ def _cmd_namespace_reserve(name, requester, duration, timeout, local):
     "--force",
     is_flag=True,
     default=False,
-    help="Do not check if you own this namespace",
+    help="Do not ask for confirmation",
 )
 @options([_local_option])
 @click_exception_wrapper("namespace release")
@@ -636,6 +636,9 @@ def _cmd_namespace_release(namespace, force, local):
 
     if not force:
         _warn_before_delete()
+        ns = Namespace(name=namespace)
+        if not ns.owned_by_me:
+            _warn_if_not_owned_by_me()
 
     release_reservation(namespace=namespace, local=local)
 

bonfire/namespaces.py

@@ -278,6 +278,7 @@ def reserve_namespace(name, requester, duration, timeout, local=True):
 def release_reservation(name=None, namespace=None, local=True):
     res = get_reservation(name=name, namespace=namespace)
     if res:
+        res_name = res["metadata"]["name"]
         res_config = process_reservation(
             res["metadata"]["name"],
             res["spec"]["requester"],
@@ -286,7 +287,7 @@ def release_reservation(name=None, namespace=None, local=True):
         )
 
         apply_config(None, list_resource=res_config)
-        msg = f"releasing reservation '{name}'"
+        msg = f"releasing reservation '{res_name}'"
         if namespace:
             msg += f" namespace '{namespace}'"
         log.info(msg)