[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

creation/lib/cWParams.py

@@ -106,9 +106,9 @@ def validate(self, base, path_text):
  # list of elements
  if isinstance(self.data[k], OrderedDict):
  if len(list(self.data[k].keys())) == 0:
- self.data[
- k
- ] = [] # XML does not know if an empty list is a dictionary or not.. fix this
+ self.data[k] = (
+ []
+ ) # XML does not know if an empty list is a dictionary or not.. fix this
 
  mylist = self[k]
  if not isinstance(mylist, list):

frontend/glideinFrontendElement.py

@@ -2359,16 +2359,16 @@ def subprocess_count_glidein(self, glidein_list):
  cred_dict = glideinFrontendLib.getClientCondorStatusCredIdOnly(req_dict, cred_id)
 
  if st in ("TotalCores", "IdleCores", "RunningCores"):
- count_status_multi_per_cred[request_name][cred_id][
- st
- ] = glideinFrontendLib.countCoresCondorStatus(cred_dict, st)
+ count_status_multi_per_cred[request_name][cred_id][st] = (
+ glideinFrontendLib.countCoresCondorStatus(cred_dict, st)
+ )
  elif st == "Running":
  # Running counts are computed differently because of
  # the dict composition. Dict also has p-slots
  # corresponding to the dynamic slots
- count_status_multi_per_cred[request_name][cred_id][
- st
- ] = glideinFrontendLib.countRunningCondorStatus(cred_dict)
+ count_status_multi_per_cred[request_name][cred_id][st] = (
+ glideinFrontendLib.countRunningCondorStatus(cred_dict)
+ )
  else:
  count_status_multi_per_cred[request_name][cred_id][st] = glideinFrontendLib.countCondorStatus(
  cred_dict

frontend/glideinFrontendInterface.py

@@ -1388,7 +1388,7 @@ def writeTypedClassadAttrToFile(fd, attr_name, attr_value):
  """
  Given the FD, type check the value and write the info the classad file
  """
- if isinstance(attr_value, (int, int, float)):
+ if isinstance(attr_value, (int, float)):
  # don't quote numeric values
  fd.write(f"{attr_name} = {attr_value}\n")
  else:

lib/config_util.py

@@ -273,7 +273,7 @@ def update(data, update_data, overwrite=True):
  if value is None:
  if key in data:
  del data[key]
- elif isinstance(value, collections.Mapping):
+ elif isinstance(value, collections.abc.Mapping):
  sub_data = data.get(key, {})
  if sub_data is not None:
  data[key] = update(sub_data, value, overwrite)