fixup! fix: Don't clean sandboxes which are on S3

chrisburr · chrisburr · commit d8ca5421c3a5 · 2025-05-09T14:20:38.000+02:00
diff --git a/src/DIRAC/WorkloadManagementSystem/DB/SandboxMetadataDB.py b/src/DIRAC/WorkloadManagementSystem/DB/SandboxMetadataDB.py
@@ -310,7 +310,7 @@ def getUnusedSandboxes(self):
             f"! Assigned AND TIMESTAMPDIFF( DAY, LastAccessTime, UTC_TIMESTAMP() ) >= {self.__unassignedSBGraceDays}",
         ]
         # Exclude sandboxes that are in S3 as those are handled by DiracX
-        sqlCmd = f"SELECT SBId, SEName, SEPFN FROM `sb_SandBoxes` WHERE SEPFN not like '/S3/%' AND ( {' ) OR ( '.join(sqlCond)} )"
+        sqlCmd = f"SELECT SBId, SEName, SEPFN FROM `sb_SandBoxes` WHERE SEPFN not like '/S3/%' AND (( {' ) OR ( '.join(sqlCond)} ))"
         return self._query(sqlCmd)
 
     def deleteSandboxes(self, SBIdList):

Original file line number	Diff line number	Diff line change
`@@ -310,7 +310,7 @@ def getUnusedSandboxes(self):`
`310`	`310`	`f"! Assigned AND TIMESTAMPDIFF( DAY, LastAccessTime, UTC_TIMESTAMP() ) >= {self.__unassignedSBGraceDays}",`
`311`	`311`	`]`
`312`	`312`	`# Exclude sandboxes that are in S3 as those are handled by DiracX`
`313`		- sqlCmd = f"SELECT SBId, SEName, SEPFN FROM `sb_SandBoxes` WHERE SEPFN not like '/S3/%' AND ( {' ) OR ( '.join(sqlCond)} )"
	`313`	+ sqlCmd = f"SELECT SBId, SEName, SEPFN FROM `sb_SandBoxes` WHERE SEPFN not like '/S3/%' AND (( {' ) OR ( '.join(sqlCond)} ))"
`314`	`314`	`return self._query(sqlCmd)`
`315`	`315`
`316`	`316`	`def deleteSandboxes(self, SBIdList):`