----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13112/#review24349 -----------------------------------------------------------
Review 13112 failed the build test : FAILURE The url of build cloudstack-master-with-patch #94 is : http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/94/ - Jenkins Cloudstack.org On July 31, 2013, 2:53 p.m., bharat kumar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13112/ > ----------------------------------------------------------- > > (Updated July 31, 2013, 2:53 p.m.) > > > Review request for cloudstack, Abhinandan Prateek and Nitin Mehta. > > > Bugs: Cloudstack-3836 > > > Repository: cloudstack-git > > > Description > ------- > > upgrade to ASF 4.2 "cloud" Database Schema Inconsistencies on the Upgraded > Setup - Table "volumes" > https://issues.apache.org/jira/browse/CLOUDSTACK-3836 > > This extra column is because of the fix > Parent id of template incorrectly recorded as last ISO attached to VM. > > when a template is created form the snapshot the sourcetemplateid is recorded > as the iso attached to the VM at thee time of creation. > > we fix this by adding a iso_id column to the volume table. when a template is > created from the snapshot we fetch the iso_id form the parent volume and set > it as the sourcetemplated id of the newly created template. > > > Diffs > ----- > > engine/schema/src/com/cloud/storage/VolumeVO.java 1445e99 > engine/schema/src/com/cloud/upgrade/dao/DbUpgradeUtils.java c37be3e > engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java 13074cf > server/src/com/cloud/storage/VolumeManager.java 2e44a3c > server/src/com/cloud/storage/VolumeManagerImpl.java d8cbf11 > server/src/com/cloud/template/TemplateManagerImpl.java 8ccc567 > server/src/com/cloud/vm/VirtualMachineManagerImpl.java eedf4d2 > setup/db/db/schema-410to420.sql 9cdf837 > > Diff: https://reviews.apache.org/r/13112/diff/ > > > Testing > ------- > > Tested on 4.2. > > > Thanks, > > bharat kumar > >
