Pushed with commit 75261eb317a763114a6054664f4adf3f7a2c2709 Please build and run it at your end
From: Saksham Srivastava <saksham.srivast...@citrix.com<mailto:saksham.srivast...@citrix.com>> Reply-To: Saksham Srivastava <saksham.srivast...@citrix.com<mailto:saksham.srivast...@citrix.com>> To: Nitin Mehta <nitin.me...@citrix.com<mailto:nitin.me...@citrix.com>>, Devdeep Singh <devdeep.si...@citrix.com<mailto:devdeep.si...@citrix.com>> Cc: Saksham Srivastava <saksham.srivast...@citrix.com<mailto:saksham.srivast...@citrix.com>>, "cloudstack-dev@incubator.apache.org<mailto:cloudstack-dev@incubator.apache.org>" <cloudstack-dev@incubator.apache.org<mailto:cloudstack-dev@incubator.apache.org>> Subject: Re: Review Request: CLOUDSTACK-924: source_template_id set to NULL when a template is created from a snapshot This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8900/ Review request for cloudstack, Devdeep Singh and Nitin Mehta. By Saksham Srivastava. Updated March 11, 2013, 10:43 a.m. Changes New patch after resolving merge conflicts. Description Volumes created from snapshots misses the source template information.Fixed the issue with volumes now getting the template id. Testing Verified on local setup. from template1->instance->snapshot1->volume1->snapshot2->template2 template2 now has source template id. Bugs: CLOUDSTACK-924 Diffs (updated) * server/src/com/cloud/storage/VolumeManagerImpl.java (f0e6028) View Diff<https://reviews.apache.org/r/8900/diff/>
