From e38cb779b1f4110c25920f0aa98581e791254af3 Mon Sep 17 00:00:00 2001
From: Yuriy Zveryanskyy <yzveryanskyy@mirantis.com>
Date: Mon, 3 Oct 2016 10:09:00 +0300
Subject: [PATCH] No need to clear "target_provision_state" again from
 conductor

The event processing for "fail" event clears the node's
"target_provision_state", so there is no need for the conductor to
explicitly clear it again.

Change-Id: I18e7c1dbcec6ad48bfea242b6201d900fd0c3461
---
 ironic/conductor/manager.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/ironic/conductor/manager.py b/ironic/conductor/manager.py
index a83c71b7b4..98ea7f4216 100644
--- a/ironic/conductor/manager.py
+++ b/ironic/conductor/manager.py
@@ -971,8 +971,6 @@ class ConductorManager(base_manager.BaseConductorManager):
             LOG.error(error)
             node.last_error = error
             task.process_event('fail')
-            node.target_provision_state = None
-            node.save()
 
     @task_manager.require_exclusive_lock
     def _do_node_clean_abort(self, task, step_name=None):