Revert "Fix LVHDSR.load: test other_conf to prevent mypy error"

This reverts commit 85b9b23.
xcp-ng · Jan 29, 2025 · 5b6b951 · 5b6b951
1 parent a574fcc
commit 5b6b951
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/drivers/LVHDSR.py b/drivers/LVHDSR.py
@@ -167,7 +167,8 @@ def load(self, sr_uuid) -> None:
         self.provision = self.PROVISIONING_DEFAULT
 
         self.other_conf = None
-        if self.srcmd.params.get("sr_ref"):
+        has_sr_ref = self.srcmd.params.get("sr_ref")
+        if has_sr_ref:
             self.other_conf = self.session.xenapi.SR.get_other_config(self.sr_ref)
 
         self.lvm_conf = None
@@ -181,7 +182,7 @@ def load(self, sr_uuid) -> None:
                         opterr='Failed to initialise the LVMCache')
         self.lvActivator = LVActivator(self.uuid, self.lvmCache)
         self.journaler = Journaler(self.lvmCache)
-        if not self.other_conf:
+        if not has_sr_ref:
             return  # must be a probe call
         # Test for thick vs thin provisioning conf parameter
         if 'allocation' in self.dconf: