Fix nodeset return on skipped jobs

Change Ideab6eb5794a01d5c2b70cb87d02d61bb3d41cce altered the
node request algorithm so that nodes are not requested unless
the job is ready to run.  Therefore, this stanza which returned
nodesets for skipped jobs would always fail (since skipped jobs
are jobs which were waiting for their parent to be ready to run
and therefore had no nodeset).

This likely contributes to some test instability as it would cause
the main loop to exit with an exception.  While it would restart
and recover, the test may decide it was finished before that could
happen.

Change-Id: I189bd59fa87460a1097fd6c6ebc04f29a89674cd
This commit is contained in:
James E. Blair 2017-01-26 11:21:44 -08:00
parent 98095cf4f8
commit fef033b4cd
1 changed files with 0 additions and 7 deletions

View File

@ -622,13 +622,6 @@ class PipelineManager(object):
if build.retry:
build.build_set.removeJobNodeSet(build.job.name)
# If any jobs were skipped as a result of this build, return
# their nodes.
for build in build.build_set.getBuilds():
if build.result == 'SKIPPED':
nodeset = build.build_set.getJobNodeSet(build.job.name)
self.sched.nodepool.returnNodeSet(nodeset)
return True
def onMergeCompleted(self, event):