Fix TypeError in sync_ironic by adding missing request_id parameter (#1849)

Two calls to push_task_output() were missing the required request_id
parameter, causing "missing 1 required positional argument: 'line'" errors.

- Fixed line 232: Added request_id parameter for node removal failure message
- Fixed line 396: Added request_id parameter for lock acquisition failure message
- Changed line 396 to use f-string for proper device.name interpolation

AI-assisted: Claude Code

Signed-off-by: Christian Berendt <berendt@osism.tech>

Author: berendt

osism/tasks/conductor/ironic.py

@@ -230,7 +230,8 @@ def sync_ironic(request_id, get_ironic_parameters, node_name=None, force=False):
                 openstack.baremetal_node_delete(node["uuid"])
             else:
                 osism_utils.push_task_output(
-                    f"Cannot remove baremetal node because it is still provisioned or running: {node}"
+                    request_id,
+                    f"Cannot remove baremetal node because it is still provisioned or running: {node}\n",
                 )
 
     # NOTE: Find nodes in NetBox which are not present in Ironic and add them
@@ -394,7 +395,7 @@ def sync_ironic(request_id, get_ironic_parameters, node_name=None, force=False):
 
         else:
             osism_utils.push_task_output(
-                "Could not acquire lock for node {device.name}"
+                request_id, f"Could not acquire lock for node {device.name}\n"
             )
 
     osism_utils.finish_task_output(request_id, rc=0)