Fix wrong usage of the python ironic client

Author: stevekeay

python/understack-workflows/understack_workflows/ironic/client.py

@@ -5,6 +5,7 @@
 from ironicclient.v1.client import Client as IronicV1Client
 from ironicclient.v1.node import Node
 from ironicclient.v1.port import Port
+from ironicclient.v1.runbook import Runbook
 
 from understack_workflows.openstack.client import get_ironic_client
 
@@ -58,8 +59,17 @@ def set_node_target_raid_config(self, node_id: str, raid_config: dict) -> None:
     def get_node_traits(self, node_id: str) -> list[str]:
         return cast(list[str], self.client.node.get_traits(node_id))
 
-    def get_runbook(self, runbook_id: str):
-        return self.client.node.api.runbook.get(runbook_id)
+    def get_runbook(self, runbook_name_or_id: str) -> Runbook:
+        """Get runbook by name or by UUID.
+
+        raises ironicclient.common.apiclient.exceptions.NotFound
+        """
+        runbook = self.client.runbook.get(runbook_name_or_id)
+        # The above raises an error if there was no such runbook.  The following
+        # is just to make the type checker happy:
+        if not runbook:
+            raise ironic_exceptions.NotFound
+        return runbook
 
     def get_node_inventory(self, node_ident: str) -> dict:
         """Fetch node inventory data from Ironic API.