From 1dab66327368dab8271ee68db0c0bbb97c8d0555 Mon Sep 17 00:00:00 2001
From: Mark McLoughlin <markmc@redhat.com>
Date: Thu, 26 Jul 2012 13:58:06 +0100
Subject: Reinstate instance locked error logging

Commit eebc64f removed some error logging, presumably by mistake.

If checks_instance_state fails, and the action blocked, users aren't
notified but the error is logged for admins. Changing this behavior
clearly wasn't the intent of the no-db-messaging commit.

Change-Id: I14581171c201770b03cecc77d4a2933c9687ef11
---
 nova/compute/manager.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/nova/compute/manager.py b/nova/compute/manager.py
index 339dbcb5f..836d33452 100644
--- a/nova/compute/manager.py
+++ b/nova/compute/manager.py
@@ -186,6 +186,8 @@ def checks_instance_lock(function):
         if admin or not locked:
             cb(self, context, *args, **kwargs)
         else:
+            LOG.error(_("check_instance_lock: not executing |%s|"),
+                      function, context=context, instance_uuid=instance_uuid)
             return False
 
     @functools.wraps(function)
-- 
cgit