You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by da...@apache.org on 2014/06/11 16:11:06 UTC
git commit: updated refs/heads/4.4 to 0667f67
Repository: cloudstack
Updated Branches:
refs/heads/4.4 891ec9bd5 -> 0667f678b
CLOUDSTACK-6869: SSH Public key content is overridden by template's meta data when you create a instance
(cherry picked from commit aa75b4388554a502b1073dd78050cd4b364a803e)
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/0667f678
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/0667f678
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/0667f678
Branch: refs/heads/4.4
Commit: 0667f678b81eb45e7b64cb22bee37f16882250e5
Parents: 891ec9b
Author: Harikrishna Patnala <ha...@citrix.com>
Authored: Wed Jun 11 16:33:00 2014 +0530
Committer: Daan Hoogland <da...@onecht.net>
Committed: Wed Jun 11 16:10:49 2014 +0200
----------------------------------------------------------------------
server/src/com/cloud/template/TemplateManagerImpl.java | 1 +
server/src/com/cloud/vm/UserVmManagerImpl.java | 11 ++++++-----
2 files changed, 7 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/0667f678/server/src/com/cloud/template/TemplateManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/template/TemplateManagerImpl.java b/server/src/com/cloud/template/TemplateManagerImpl.java
index 3ebb43a..dc076a8 100755
--- a/server/src/com/cloud/template/TemplateManagerImpl.java
+++ b/server/src/com/cloud/template/TemplateManagerImpl.java
@@ -1656,6 +1656,7 @@ public class TemplateManagerImpl extends ManagerBase implements TemplateManager,
}
}
if (cmd.getDetails() != null) {
+ details.remove("Encrypted.Password"); // new password will be generated during vm deployment from password enabled template
details.putAll(cmd.getDetails());
}
if (!details.isEmpty()) {
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/0667f678/server/src/com/cloud/vm/UserVmManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/vm/UserVmManagerImpl.java b/server/src/com/cloud/vm/UserVmManagerImpl.java
index 58709ec..673e035 100755
--- a/server/src/com/cloud/vm/UserVmManagerImpl.java
+++ b/server/src/com/cloud/vm/UserVmManagerImpl.java
@@ -2898,6 +2898,12 @@ public class UserVmManagerImpl extends ManagerBase implements UserVmManager, Vir
.getLimitCpuUse(), owner.getDomainId(), owner.getId(), offering.getId(), userData, hostName, diskOfferingId);
vm.setUuid(uuidName);
vm.setDynamicallyScalable(template.isDynamicallyScalable());
+
+ Map<String, String> details = template.getDetails();
+ if (details != null && !details.isEmpty()) {
+ vm.details.putAll(details);
+ }
+
if (sshPublicKey != null) {
vm.setDetail("SSH.PublicKey", sshPublicKey);
}
@@ -2971,11 +2977,6 @@ public class UserVmManagerImpl extends ManagerBase implements UserVmManager, Vir
}
}
- Map<String, String> details = template.getDetails();
- if (details != null && !details.isEmpty()) {
- vm.details.putAll(details);
- }
-
_vmDao.persist(vm);
if (customParameters != null && customParameters.size() > 0) {
for (String key : customParameters.keySet()) {