vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Make pop on realm work even if the realm isn't present

Browse source 
This is cleanup code and, occasionally, we get a cleanup without the realm having been added, leading to a KeyError
This commit is contained in:
Joseph Schorr 2017-02-21 10:47:11 -05:00
parent 2ef55a5842
commit fd4a31d3c3
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
buildman/server.py
View file

@ -132,9 +132,13 @@ class BuilderServer(object):
logger.debug('Unregistering component with realm %s and token %s', logger.debug('Unregistering component with realm %s and token %s',
component.builder_realm, component.expected_token) component.builder_realm, component.expected_token)
self._realm_map.pop(component.builder_realm) self._realm_map.pop(component.builder_realm, None)
self._current_components.remove(component)
self._session_factory.remove(component) if component in self._current_components:
self._current_components.remove(component)
if component in self._session_factory:
self._session_factory.remove(component)
def _job_heartbeat(self, build_job): def _job_heartbeat(self, build_job):
self._queue.extend_processing(build_job.job_item, seconds_from_now=JOB_TIMEOUT_SECONDS, self._queue.extend_processing(build_job.job_item, seconds_from_now=JOB_TIMEOUT_SECONDS,