openstack
/
nova
Code Issues Proposed changes

log stale allocations as WARNING instead of DEBUG 

Stale resource allocations can prevent Nova from scheduling new
instances on a host that otherwise has sufficient resources available.
Nova would log a message about this situation on the affected compute
node, but it was logged at DEBUG priority and so would typically not
be visible in a production environment.

This commit modifies Nova to log the message as a warning instead.

Change-Id: I347d2ab0077c4d9b62bc169af9e039d00ec214fa
Closes-bug: #1767553
(cherry picked from commit a54e5ed00f)
(cherry picked from commit 12a3b698d1)
This commit is contained in:
Lars Kellogg-Stedman 2018-04-27 22:43:35 -04:00 committed by Artom Lifshitz
parent d75b995d03
commit 852cf3bca7
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
nova/compute/resource_tracker.py
View File

@ -1278,12 +1278,13 @@ class ResourceTracker(object):
# that contains this source compute host information anyway and
# recreate an allocation that only refers to itself. So we
# don't need to do anything in that case. Just log the
# situation here for debugging information but don't attempt to
# delete or change the allocation.
LOG.debug("Instance %s has been moved to another host %s(%s). "
"There are allocations remaining against the source "
"host that might need to be removed: %s.",
instance_uuid, instance.host, instance.node, alloc)
# situation here for information but don't attempt to delete or
# change the allocation.
LOG.warning("Instance %s has been moved to another host "
"%s(%s). There are allocations remaining against "
"the source host that might need to be removed: "
"%s.",
instance_uuid, instance.host, instance.node, alloc)
def delete_allocation_for_evacuated_instance(self, instance, node,
node_type='source'):