diff --git a/awx/main/tasks.py b/awx/main/tasks.py index 290f8f4acd..026a91ef17 100644 --- a/awx/main/tasks.py +++ b/awx/main/tasks.py @@ -343,11 +343,19 @@ def cluster_node_heartbeat(self): raise RuntimeError("Shutting down.") for other_inst in lost_instances: try: - other_inst.capacity = 0 - other_inst.save(update_fields=['capacity']) - logger.error("Host {} last checked in at {}, marked as lost.".format( - other_inst.hostname, other_inst.modified)) - if settings.AWX_AUTO_DEPROVISION_INSTANCES: + # Capacity could already be 0 because: + # * It's a new node and it never had a heartbeat + # * It was set to 0 by another tower node running this method + # * It was set to 0 by this node, but auto deprovisioning is off + # + # If auto deprovisining is on, don't bother setting the capacity to 0 + # since we will delete the node anyway. + if other_inst.capacity != 0 and not settings.AWX_AUTO_DEPROVISION_INSTANCES: + other_inst.capacity = 0 + other_inst.save(update_fields=['capacity']) + logger.error("Host {} last checked in at {}, marked as lost.".format( + other_inst.hostname, other_inst.modified)) + elif settings.AWX_AUTO_DEPROVISION_INSTANCES: deprovision_hostname = other_inst.hostname other_inst.delete() logger.info("Host {} Automatically Deprovisioned.".format(deprovision_hostname))