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

Add a comment about why we are taking a lock when terminating a builder machine.

Browse source
This commit is contained in:
Jake Moshenko 2015-06-10 15:44:26 -04:00
parent f767fc4d03
commit c435f5c127
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
buildman/manager/ephemeral.py
View file

@ -105,6 +105,8 @@ class EphemeralBuilderManager(BaseManager):
if 'builder_id' in job_metadata: if 'builder_id' in job_metadata:
builder_id = job_metadata['builder_id'] builder_id = job_metadata['builder_id']
# Before we delete the build node, we take a lock to make sure that only one manager
# can terminate the node.
try: try:
lock_key = self._etcd_lock_key(builder_id) lock_key = self._etcd_lock_key(builder_id)
yield From(self._etcd_client.write(lock_key, '', prevExist=False, ttl=self.setup_time())) yield From(self._etcd_client.write(lock_key, '', prevExist=False, ttl=self.setup_time()))