Skip to content

Commit a46fb9c

Browse files
authored
Merge pull request #229 from cms-sw/fix-missing-build-type
protection against missing build type in stats
2 parents 54eebf1 + 80cb40f commit a46fb9c

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

rmanager.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ def allocResourcesForExternals(self, externalsList, count=1000): # return ordere
1717
ext = ext_full.split('+')[1]
1818
ext_items = ext_full.split("-")
1919
build_type = ext_items[1] if ext_items[1] in ["prep", "build", "install", "srpm", "rpms"] else "build"
20-
pkg_stats = self.esStats["packages"][build_type]
20+
pkg_stats = self.esStats["packages"].get(build_type, {})
2121
if ext not in pkg_stats:
2222
idx = -1
2323
ext = "%s:%s" % (build_type, ext)
@@ -47,7 +47,9 @@ def allocResourcesForExternals(self, externalsList, count=1000): # return ordere
4747
break
4848
elif self.highestPriortyOnly:
4949
break
50-
self.scheduler.log("Available resources %s" % self.machineResources)
50+
if externals_ordered:
51+
self.scheduler.log("Available resources %s" % self.machineResources)
52+
self.scheduler.log("Buildable tasks %s: %s" % (len(externals_ordered), ",".join(externals_ordered)))
5153
return externals_ordered
5254

5355
def releaseResourcesForExternal(self, external):

0 commit comments

Comments
 (0)