openstack/deb-murano
Code Issues Proposed changes

Fix the issue with multiple agent call

Browse Source 
This patch fixes the issue related to execution of the same execution plan on
multiple VMs. The root cause was agent handling code which overwrites template
with execution plan.
Now template is deep copied before execution plan creation.
Thanks to Stan to help to fix that.

Change-Id: I1d101ab155c46986ac4eed0e53beed7c8df1b569
Closes-Bug: #1335929
This commit is contained in:
Georgy Okrokvertskhov
2014-07-28 16:05:53 -07:00
parent c542a50e61
commit a5818c95cc
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
murano/engine/system/agent.py
View File

@@ -13,6 +13,7 @@
# See the License for the specific language governing permissions and # See the License for the specific language governing permissions and
# limitations under the License. # limitations under the License.
import copy
import datetime import datetime
import os import os
import types import types
@@ -136,6 +137,7 @@ class Agent(murano_object.MuranoObject):
} }
def buildExecutionPlan(self, template, resources): def buildExecutionPlan(self, template, resources):
template = copy.deepcopy(template)
if not isinstance(template, types.DictionaryType): if not isinstance(template, types.DictionaryType):
raise ValueError('Incorrect execution plan ') raise ValueError('Incorrect execution plan ')
format_version = template.get('FormatVersion') format_version = template.get('FormatVersion')