fix(workflows): simplify re-raising the exception

Just re-raise it instead of wrapping it
rackerlabs · Feb 10, 2025 · 0595e7a · 0595e7a
1 parent 31ede5a
commit 0595e7a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/python/understack-workflows/understack_workflows/bmc_bios.py b/python/understack-workflows/understack_workflows/bmc_bios.py
@@ -52,5 +52,4 @@ def patch_bios_settings(bmc: Bmc, new_settings: dict):
         if "Pending configuration values" in repr(e):
             logger.info("%s BIOS settings job already queued, ignoring.", bmc)
             return
-        else:
-            raise (e) from None
+        raise
diff --git a/python/understack-workflows/understack_workflows/bmc_credentials.py b/python/understack-workflows/understack_workflows/bmc_credentials.py
@@ -105,9 +105,9 @@ def _get_bmc_accounts(host: str, token: str, username: str) -> list[dict]:
         logger.debug("accounts: %s", accounts)
 
         return accounts
-    except Exception as e:
+    except Exception:
         logger.exception("Can't fetch accounts from Redfish account service.")
-        raise e from None
+        raise
 
 
 def _set_bmc_creds(host: str, token: str, username: str, new_password: str):