Skip to content

Commit

Permalink
Revoke sketchy giving up on jobs
Browse files Browse the repository at this point in the history
Revokes f31504d.

This sketchy giving up is needed while #194 is open. Once it is closed,
it should be removed.
  • Loading branch information
j-mao committed Jan 9, 2020
1 parent f31504d commit f957164
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions infrastructure/worker/app/subscription.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,7 @@ def subscribe(subscription_name, worker):

# The process is finished
else:
give_up = (int(time.time()) - message.message.publish_time.seconds) > 600
if process.exitcode == 0 or give_up:
if process.exitcode == 0:
# Success; acknowledge and return
client.acknowledge(subscription_path, [message.ack_id])
logging.info('Ending and acknowledged: {}'.format(message.message.data.decode()))
Expand Down

0 comments on commit f957164

Please sign in to comment.