From bec3f3c8a2a4db09433db4d5315840a4cb29f511 Mon Sep 17 00:00:00 2001 From: Edison Su Date: Fri, 4 Mar 2011 13:02:44 -0500 Subject: [PATCH] bug 8843: add vm into security group when vm is created status 8843: resolved fixed --- server/src/com/cloud/vm/UserVmManagerImpl.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/server/src/com/cloud/vm/UserVmManagerImpl.java b/server/src/com/cloud/vm/UserVmManagerImpl.java index 6d7b43b5c47..10792cadf7d 100755 --- a/server/src/com/cloud/vm/UserVmManagerImpl.java +++ b/server/src/com/cloud/vm/UserVmManagerImpl.java @@ -2183,6 +2183,8 @@ public class UserVmManagerImpl implements UserVmManager, UserVmService, Manager throw new CloudRuntimeException("Unable to assign Vm to the group " + group); } + _networkGroupMgr.addInstanceToGroups(vm.getId(), cmd.getSecurityGroupIdList()); + return vm; } @@ -2230,9 +2232,6 @@ public class UserVmManagerImpl implements UserVmManager, UserVmService, Manager updateVmStateForFailedVmCreation(vm.getId()); } - _networkGroupMgr.addInstanceToGroups(vm.getId(), cmd.getSecurityGroupIdList()); - - if (template.getEnablePassword()) { //this value is not being sent to the backend; need only for api dispaly purposes vm.setPassword(password);