From eb8d9e0f407e3a748eae6a43407613da9012eb4b Mon Sep 17 00:00:00 2001
From: kubajj <vilouskubajj@gmail.com>
Date: Wed, 22 Jan 2025 14:04:40 +0000
Subject: [PATCH] Fix errors in the function erase_devices_express

Prevents the UnboundLocalError in erase_devices_express clean step.

Closes-Bug: #2095499
Change-Id: I01ce5005a62638ff960d2a75f225f882b2d56973
(cherry picked from commit 018a5f6253ce1bb30a65e491e4365b7c2444937b)
---
 ironic_python_agent/hardware.py                              | 1 +
 ...fix_errors_in_erase_devices_express-f425c76b485a875f.yaml | 5 +++++
 2 files changed, 6 insertions(+)
 create mode 100644 releasenotes/notes/fix_errors_in_erase_devices_express-f425c76b485a875f.yaml

diff --git a/ironic_python_agent/hardware.py b/ironic_python_agent/hardware.py
index fd89fc06b..d3f0a0774 100644
--- a/ironic_python_agent/hardware.py
+++ b/ironic_python_agent/hardware.py
@@ -1791,6 +1791,7 @@ class GenericHardwareManager(HardwareManager):
             return
         for dev in self._list_erasable_devices(node):
             safety_check_block_device(node, dev.name)
+            secure_erase_error = None
             try:
                 if self._is_nvme(dev):
                     execute_nvme_erase = info.get(
diff --git a/releasenotes/notes/fix_errors_in_erase_devices_express-f425c76b485a875f.yaml b/releasenotes/notes/fix_errors_in_erase_devices_express-f425c76b485a875f.yaml
new file mode 100644
index 000000000..d0eb471e4
--- /dev/null
+++ b/releasenotes/notes/fix_errors_in_erase_devices_express-f425c76b485a875f.yaml
@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    Prevent the UnboundLocalError in erase_devices_express,
+    for example, on a disk failure.