You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by bh...@apache.org on 2016/12/08 11:47:48 UTC
[1/6] git commit: updated refs/heads/4.9 to 26a2c9e
Repository: cloudstack
Updated Branches:
refs/heads/4.9 de7a06efe -> 26a2c9ee9
CLOUDSTACK-9637: Template create from snapshot does not populate vm_template_details
Summary: Retrieve and add source template details to VMTemplateVO.
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/545a214a
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/545a214a
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/545a214a
Branch: refs/heads/4.9
Commit: 545a214a4229daad5fa4cc66a512979707dbe747
Parents: f231c8c
Author: Sudhansu <su...@accelerite.com>
Authored: Fri Dec 2 00:08:12 2016 +0530
Committer: Sudhansu <su...@accelerite.com>
Committed: Sat Dec 3 18:45:08 2016 +0530
----------------------------------------------------------------------
.../com/cloud/template/TemplateManagerImpl.java | 10 +-
...template_from_snapshot_with_template_details | 386 +++++++++++++++++++
2 files changed, 395 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/545a214a/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 c03db35..ae1a649 100644
--- a/server/src/com/cloud/template/TemplateManagerImpl.java
+++ b/server/src/com/cloud/template/TemplateManagerImpl.java
@@ -1737,6 +1737,14 @@ public class TemplateManagerImpl extends ManagerBase implements TemplateManager,
// Increment the number of templates
if (template != null) {
Map<String, String> details = new HashMap<String, String>();
+
+ if (sourceTemplateId != null) {
+ VMTemplateVO sourceTemplate = _tmpltDao.findById(sourceTemplateId);
+ if(sourceTemplate != null){
+ details.putAll(sourceTemplate.getDetails());
+ }
+ }
+
if (volume != null) {
Long vmId = volume.getInstanceId();
if (vmId != null) {
@@ -1903,7 +1911,7 @@ public class TemplateManagerImpl extends ManagerBase implements TemplateManager,
sortKey == null &&
isDynamicallyScalable == null &&
isRoutingTemplate == null &&
- details == null);
+ details == null);
if (!updateNeeded) {
return template;
}
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/545a214a/test/integration/component/test_template_from_snapshot_with_template_details
----------------------------------------------------------------------
diff --git a/test/integration/component/test_template_from_snapshot_with_template_details b/test/integration/component/test_template_from_snapshot_with_template_details
new file mode 100644
index 0000000..f8269e2
--- /dev/null
+++ b/test/integration/component/test_template_from_snapshot_with_template_details
@@ -0,0 +1,386 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" P1 tests for Templates
+"""
+# Import Local Modules
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase, unittest
+from marvin.cloudstackAPI import listZones
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ Domain,
+ Template,
+ ServiceOffering,
+ VirtualMachine,
+ Snapshot,
+ Volume)
+from marvin.lib.common import (get_domain,
+ get_zone,
+ get_template,
+ get_builtin_template_info,
+ list_volumes,
+ list_snapshots)
+# Import System modules
+import time
+
+
+class Services:
+
+ """Test Templates Services
+ """
+
+ def __init__(self):
+ self.services = {
+ "account": {
+ "email": "test@test.com",
+ "firstname": "Test",
+ "lastname": "User",
+ "username": "test",
+ # Random characters are appended for unique
+ # username
+ "password": "password",
+ },
+ "testdomain": {"name": "test"},
+ "service_offering": {
+ "name": "Tiny Instance",
+ "displaytext": "Tiny Instance",
+ "cpunumber": 1,
+ "cpuspeed": 100, # in MHz
+ "memory": 128, # In MBs
+ },
+ "disk_offering": {
+ "displaytext": "Small",
+ "name": "Small",
+ "disksize": 1
+ },
+ "virtual_machine": {
+ "displayname": "testVM",
+ "hypervisor": 'XenServer',
+ "protocol": 'TCP',
+ "ssh_port": 22,
+ "username": "root",
+ "password": "password",
+ "privateport": 22,
+ "publicport": 22,
+ },
+ "volume": {
+ "diskname": "Test Volume",
+ },
+ "templates": {
+ # Configs for different Template formats
+ # For Eg. raw image, zip etc
+ 0: {
+ "displaytext": "Public Template",
+ "name": "Public template",
+ "ostype": 'CentOS 5.3 (64-bit)',
+ "url": "http://download.cloud.com/releases/2.0.0/UbuntuServer-10-04-64bit.vhd.bz2",
+ "hypervisor": 'XenServer',
+ "format": 'VHD',
+ "isfeatured": True,
+ "ispublic": True,
+ "isextractable": True,
+ },
+ },
+ "template": {
+ "displaytext": "Cent OS Template",
+ "name": "Cent OS Template",
+ "ostype": 'CentOS 5.3 (64-bit)',
+ "templatefilter": 'self',
+ "isfeatured": True,
+ "ispublic": True,
+ },
+ "templatefilter": 'self',
+ "ostype": 'CentOS 5.3 (64-bit)',
+ "sleep": 60,
+ "timeout": 10,
+ }
+
+
+class TestCreateTemplate(cloudstackTestCase):
+
+ def setUp(self):
+ self.apiclient = self.testClient.getApiClient()
+ self.dbclient = self.testClient.getDbConnection()
+ self.cleanup = []
+
+ if self.unsupportedHypervisor:
+ self.skipTest(
+ "Template creation from root volume is not supported in LXC")
+ return
+
+ def tearDown(self):
+ try:
+ # Clean up, terminate the created templates
+ cleanup_resources(self.apiclient, self.cleanup)
+
+ except Exception as e:
+ raise Exception("Warning: Exception during cleanup : %s" % e)
+ return
+
+ @classmethod
+ def setUpClass(cls):
+ cls.testClient = super(TestCreateTemplate, cls).getClsTestClient()
+ cls.api_client = cls.testClient.getApiClient()
+
+ cls.services = Services().services
+ # Get Zone, Domain and templates
+ cls.domain = get_domain(cls.api_client)
+ cls.zone = get_zone(cls.api_client, cls.testClient.getZoneForTests())
+ cls.services['mode'] = cls.zone.networktype
+ cls._cleanup = []
+ cls.unsupportedHypervisor = False
+ cls.hypervisor = cls.testClient.getHypervisorInfo()
+ if cls.hypervisor.lower() in ['lxc']:
+ cls.unsupportedHypervisor = True
+ return
+ cls.services["virtual_machine"]["zoneid"] = cls.zone.id
+
+ cls.service_offering = ServiceOffering.create(
+ cls.api_client,
+ cls.services["service_offering"]
+ )
+ cls._cleanup.append(cls.service_offering)
+ cls.account = Account.create(
+ cls.api_client,
+ cls.services["account"],
+ domainid=cls.domain.id
+ )
+ cls._cleanup.append(cls.account)
+ cls.services["account"] = cls.account.name
+ return
+
+ @classmethod
+ def tearDownClass(cls):
+ try:
+ cls.api_client = super(
+ TestCreateTemplate,
+ cls).getClsTestClient().getApiClient()
+ # Cleanup resources used
+ cleanup_resources(cls.api_client, cls._cleanup)
+
+ except Exception as e:
+ raise Exception("Warning: Exception during cleanup : %s" % e)
+
+ return
+ @attr(tags=["advanced", "advancedns"], required_hardware="true")
+ def test_01_create_template_snampshot(self):
+
+
+ builtin_info = get_builtin_template_info(self.apiclient, self.zone.id)
+ self.services["templates"][0]["url"] = builtin_info[0]
+ self.services["templates"][0]["hypervisor"] = builtin_info[1]
+ self.services["templates"][0]["format"] = builtin_info[2]
+
+ # Register new template
+ template = Template.register(
+ self.apiclient,
+ self.services["templates"][0],
+ zoneid=self.zone.id,
+ account=self.account.name,
+ domainid=self.account.domainid,
+ hypervisor=self.hypervisor,
+ details=[{"keyboard":"us","nicAdapter":"e1000","rootDiskController":"scsi"}]
+ )
+ self.debug(
+ "Registered a template of format: %s with ID: %s" % (
+ self.services["templates"][0]["format"],
+ template.id
+ ))
+ # Wait for template to download
+ template.download(self.apiclient)
+ self.cleanup.append(template)
+
+ # Wait for template status to be changed across
+ time.sleep(self.services["sleep"])
+ timeout = self.services["timeout"]
+ while True:
+ list_template_response = Template.list(
+ self.apiclient,
+ templatefilter='all',
+ id=template.id,
+ zoneid=self.zone.id,
+ account=self.account.name,
+ domainid=self.account.domainid)
+ if isinstance(list_template_response, list):
+ break
+ elif timeout == 0:
+ raise Exception("List template failed!")
+
+ time.sleep(5)
+ timeout = timeout - 1
+ # Verify template response to check whether template added successfully
+ self.assertEqual(
+ isinstance(list_template_response, list),
+ True,
+ "Check for list template response return valid data"
+ )
+
+ self.assertNotEqual(
+ len(list_template_response),
+ 0,
+ "Check template available in List Templates"
+ )
+
+ template_response = list_template_response[0]
+ self.assertEqual(
+ template_response.isready,
+ True,
+ "Template state is not ready, it is %s" % template_response.isready
+ )
+
+ self.assertIsNotNone(
+ template_response.details,
+ "Template details is %s" % template_response.details
+ )
+
+ # Deploy new virtual machine using template
+ virtual_machine = VirtualMachine.create(
+ self.apiclient,
+ self.services["virtual_machine"],
+ templateid=template.id,
+ accountid=self.account.name,
+ domainid=self.account.domainid,
+ serviceofferingid=self.service_offering.id,
+ mode=self.services["mode"]
+ )
+ self.debug("creating an instance with template ID: %s" % template.id)
+ vm_response = VirtualMachine.list(self.apiclient,
+ id=virtual_machine.id,
+ account=self.account.name,
+ domainid=self.account.domainid)
+ self.assertEqual(
+ isinstance(vm_response, list),
+ True,
+ "Check for list VMs response after VM deployment"
+ )
+ # Verify VM response to check whether VM deployment was successful
+ self.assertNotEqual(
+ len(vm_response),
+ 0,
+ "Check VMs available in List VMs response"
+ )
+ vm = vm_response[0]
+ self.assertEqual(
+ vm.state,
+ 'Running',
+ "Check the state of VM created from Template"
+ )
+
+ volumes = list_volumes(
+ self.apiclient,
+ virtualmachineid=vm.id,
+ type='ROOT',
+ listall=True
+ )
+
+ snapshot = Snapshot.create(
+ self.apiclient,
+ volumes[0].id,
+ account=self.account.name,
+ domainid=self.account.domainid
+ )
+ time.sleep(self.services["sleep"])
+ self.cleanup.append(snapshot)
+ self.debug("Snapshot created: ID - %s" % snapshot.id)
+
+ snapshots = list_snapshots(
+ self.apiclient,
+ id=snapshot.id
+ )
+ self.assertEqual(
+ isinstance(snapshots, list),
+ True,
+ "Check list response returns a valid list"
+ )
+
+ self.assertNotEqual(
+ snapshots,
+ None,
+ "Check if result exists in list item call"
+ )
+ self.assertEqual(
+ snapshots[0].id,
+ snapshot.id,
+ "Check resource id in list resources call"
+ )
+
+ virtual_machine.delete(self.apiclient, expunge=False)
+
+ list_vm_response = VirtualMachine.list(
+ self.apiclient,
+ id=virtual_machine.id
+ )
+
+ self.assertEqual(
+ isinstance(list_vm_response, list),
+ True,
+ "Check list response returns a valid list"
+ )
+
+ self.assertNotEqual(
+ len(list_vm_response),
+ 0,
+ "Check VM avaliable in List Virtual Machines"
+ )
+
+ list_vm_response = VirtualMachine.list(
+ self.apiclient,
+ id=self.small_virtual_machine.id
+ )
+ self.assertEqual(
+ isinstance(list_vm_response, list),
+ True,
+ "Check list response returns a valid list"
+ )
+
+ self.assertNotEqual(
+ len(list_vm_response),
+ 0,
+ "Check VM avaliable in List Virtual Machines"
+ )
+
+ template = Template.create_from_snapshot(
+ self.apiclient,
+ snapshot,
+ self.services["template"]
+ )
+ self.cleanup.append(template)
+ # Verify created template
+ templates = Template.list(
+ self.apiclient ,
+ templatefilter=self.services["template"]["templatefilter"],
+ id=template.id
+ )
+ self.assertNotEqual(
+ templates,
+ None,
+ "Check if result exists in list item call"
+ )
+
+ self.assertEqual(
+ templates[0].id,
+ template.id,
+ "Check new template id in list resources call"
+ )
+
+
+ self.assertIsNotNone(
+ templates[0].details,
+ "Template details is %s" % template_response.details
+ )
+
+ return
[6/6] git commit: updated refs/heads/4.9 to 26a2c9e
Posted by bh...@apache.org.
Merge pull request #1822 from shapeblue/4.9-travis-testfix
CLOUDSTACK-9584: Fix intermittent test failure in `test_volumes`The component/test_volume failures happen when disk offering is random selected to be a custom one. This fixes that.
* pr/1822:
CLOUDSTACK-9584: Fix intermittent test failure in `test_volumes`
Signed-off-by: Rohit Yadav <ro...@shapeblue.com>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/26a2c9ee
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/26a2c9ee
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/26a2c9ee
Branch: refs/heads/4.9
Commit: 26a2c9ee9b8f06d64e2d8ad97eb2c5d1a3a9790b
Parents: 29fa2e0 6ebf024
Author: Rohit Yadav <ro...@shapeblue.com>
Authored: Thu Dec 8 17:17:10 2016 +0530
Committer: Rohit Yadav <ro...@shapeblue.com>
Committed: Thu Dec 8 17:17:10 2016 +0530
----------------------------------------------------------------------
test/integration/component/test_volumes.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
[2/6] git commit: updated refs/heads/4.9 to 26a2c9e
Posted by bh...@apache.org.
CLOUDSTACK-9659: mismatch in traffic type in ip_associations.json and ips.json
As part of the bug 'CLOUDSTACK-9339 Virtual Routers don't handle Multiple Public Interfaces correctly'
issue of mismatch of traffic type represented by 'nw_type' in config sent by management server in
ip_associations.json and how it is persisted in the ips.json data bag are differnet,
is addressed, however missed the change in final merge.
this bug is to add the functionality in cs_ip.py, to lower the traffic type sent by management server before persisting in the ips.json databag
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/8b4c36ef
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/8b4c36ef
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/8b4c36ef
Branch: refs/heads/4.9
Commit: 8b4c36ef501a96742c52b4d532cc3adda25aa71b
Parents: de7a06e
Author: Murali Reddy <mu...@gmail.com>
Authored: Thu Dec 8 14:19:01 2016 +0530
Committer: Murali Reddy <mu...@gmail.com>
Committed: Thu Dec 8 14:19:01 2016 +0530
----------------------------------------------------------------------
systemvm/patches/debian/config/opt/cloud/bin/cs_ip.py | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/8b4c36ef/systemvm/patches/debian/config/opt/cloud/bin/cs_ip.py
----------------------------------------------------------------------
diff --git a/systemvm/patches/debian/config/opt/cloud/bin/cs_ip.py b/systemvm/patches/debian/config/opt/cloud/bin/cs_ip.py
index c34fc01..1ce78f9 100755
--- a/systemvm/patches/debian/config/opt/cloud/bin/cs_ip.py
+++ b/systemvm/patches/debian/config/opt/cloud/bin/cs_ip.py
@@ -41,6 +41,8 @@ def merge(dbag, ip):
ip['network'] = str(ipo.network) + '/' + str(ipo.prefixlen)
if 'nw_type' not in ip.keys():
ip['nw_type'] = 'public'
+ else:
+ ip['nw_type'] = ip['nw_type'].lower()
if ip['nw_type'] == 'control':
dbag['eth' + str(nic_dev_id)] = [ip]
else:
[3/6] git commit: updated refs/heads/4.9 to 26a2c9e
Posted by bh...@apache.org.
CLOUDSTACK-9584: Fix intermittent test failure in `test_volumes`
Signed-off-by: Rohit Yadav <ro...@shapeblue.com>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/6ebf0245
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/6ebf0245
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/6ebf0245
Branch: refs/heads/4.9
Commit: 6ebf02454b36f09d8090ff333ae0e6738397c7e5
Parents: de7a06e
Author: Rohit Yadav <ro...@shapeblue.com>
Authored: Thu Dec 8 16:25:40 2016 +0530
Committer: Rohit Yadav <ro...@shapeblue.com>
Committed: Thu Dec 8 16:25:40 2016 +0530
----------------------------------------------------------------------
test/integration/component/test_volumes.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/6ebf0245/test/integration/component/test_volumes.py
----------------------------------------------------------------------
diff --git a/test/integration/component/test_volumes.py b/test/integration/component/test_volumes.py
index 77d8679..0dccc46 100644
--- a/test/integration/component/test_volumes.py
+++ b/test/integration/component/test_volumes.py
@@ -1261,7 +1261,7 @@ class TestVolumes(cloudstackTestCase):
zoneid=self.zone.id,
account=domuser.name,
domainid=dom.id,
- diskofferingid=diskoffering[0].id
+ diskofferingid=filter(lambda x: not x.iscustomized, diskoffering)[0].id
)
self.assertTrue(
vol is not None, "volume creation fails in domain %s as user %s" %
[5/6] git commit: updated refs/heads/4.9 to 26a2c9e
Posted by bh...@apache.org.
Merge pull request #1821 from murali-reddy/traffic_type_in_lower_case
CLOUDSTACK-9659: mismatch in traffic type in ip_associations.json and ips.jsonAs part of the bug 'CLOUDSTACK-9339 Virtual Routers don't handle Multiple Public Interfaces correctly'
issue of mismatch of traffic type represented by 'nw_type' in config sent by management server in
ip_associations.json and how it is persisted in the ips.json data bag are differnet,
is addressed, however missed the change in final merge.
this bug is to add the functionality in cs_ip.py, to lower the traffic type sent by management server before persisting in the ips.json databag
* pr/1821:
CLOUDSTACK-9659: mismatch in traffic type in ip_associations.json and ips.json
Signed-off-by: Rohit Yadav <ro...@shapeblue.com>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/29fa2e0b
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/29fa2e0b
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/29fa2e0b
Branch: refs/heads/4.9
Commit: 29fa2e0b8b90754e4258c596273a25c483be3a40
Parents: f70428a 8b4c36e
Author: Rohit Yadav <ro...@shapeblue.com>
Authored: Thu Dec 8 17:15:47 2016 +0530
Committer: Rohit Yadav <ro...@shapeblue.com>
Committed: Thu Dec 8 17:15:47 2016 +0530
----------------------------------------------------------------------
systemvm/patches/debian/config/opt/cloud/bin/cs_ip.py | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------
[4/6] git commit: updated refs/heads/4.9 to 26a2c9e
Posted by bh...@apache.org.
Merge pull request #1805 from sudhansu7/CLOUDSTACK-9637
CLOUDSTACK-9637: Template create from snapshot does not populate vm_t**ISSUE**
============
Template create from snapshot does not populate vm_template_details
**REPRO STEPS**
==================
1. Register a template A and specify property:
Root disk controller: scsi
NIC adapter type: E1000
Keyboard type: us
2. Create a vm instance from template A
3. Take volume snapshot for vm instance
4. Delete VM instance
5. Switch to "Storage->Snapshots", convert snapshot to a template B
6. Observe template B does not inherit property from template A, the table vm_template_details is empty
**SOLUTION**: Retrieve and add source template details to VMTemplateVO.
Before Fix:
```
mysql> select id,name,source_template_id from vm_template where id=202;
+-----+--------+--------------------+
| id | name | source_template_id |
+-----+--------+--------------------+
| 202 | Debian | NULL |
+-----+--------+--------------------+
1 row in set (0.00 sec)
mysql> select * from vm_template_details where template_id=202;
+----+-------------+--------------------+-------+---------+
| id | template_id | name | value | display |
+----+-------------+--------------------+-------+---------+
| 1 | 202 | keyboard | us | 1 |
| 2 | 202 | nicAdapter | E1000 | 1 |
| 3 | 202 | rootDiskController | scsi | 1 |
+----+-------------+--------------------+-------+---------+
3 rows in set (0.00 sec)
mysql> select id,name,source_template_id from vm_template where source_template_id=202;
+-----+----------------+--------------------+
| id | name | source_template_id |
+-----+----------------+--------------------+
| 203 | derived-debian | 202 |
+-----+----------------+--------------------+
1 row in set (0.00 sec)
mysql> select * from vm_template_details where template_id=203;
Empty set (0.00 sec)
After Fix:
mysql> select id,name,source_template_id from vm_template where source_template_id=202;
+-----+--------------------------+--------------------+
| id | name | source_template_id |
+-----+--------------------------+--------------------+
| 203 | derived-debian | 202 |
| 204 | debian-derived-after-fix | 202 |
+-----+--------------------------+--------------------+
2 rows in set (0.00 sec)
mysql> select * from vm_template_details where template_id=204;
+----+-------------+--------------------+-------+---------+
| id | template_id | name | value | display |
+----+-------------+--------------------+-------+---------+
| 4 | 204 | keyboard | us | 1 |
| 5 | 204 | nicAdapter | E1000 | 1 |
| 6 | 204 | rootDiskController | scsi | 1 |
+----+-------------+--------------------+-------+---------+
3 rows in set (0.00 sec)
```
**Marvin Test :** test_template_from_snapshot_with_template_details.py
**Result:**
```
test_01_create_template_snampshot (integration.component.test_template_from_snapshot_with_template_details.TestCreateTemplate) ... === TestName: test_01_create_template_snampshot | Status : SUCCESS ===
ok
----------------------------------------------------------------------
Ran 1 test in 864.523s
OK
```
* pr/1805:
CLOUDSTACK-9637: Template create from snapshot does not populate vm_template_details
Signed-off-by: Rohit Yadav <ro...@shapeblue.com>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/f70428a2
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/f70428a2
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/f70428a2
Branch: refs/heads/4.9
Commit: f70428a2321277adc58adbb553d96e19b9ca0727
Parents: de7a06e 545a214
Author: Rohit Yadav <ro...@shapeblue.com>
Authored: Thu Dec 8 17:14:52 2016 +0530
Committer: Rohit Yadav <ro...@shapeblue.com>
Committed: Thu Dec 8 17:14:52 2016 +0530
----------------------------------------------------------------------
.../com/cloud/template/TemplateManagerImpl.java | 10 +-
...template_from_snapshot_with_template_details | 386 +++++++++++++++++++
2 files changed, 395 insertions(+), 1 deletion(-)
----------------------------------------------------------------------