cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "bharat kumar" <bharat.ku...@citrix.com>
Subject Review Request 13112: upgrade to ASF 4.2 "cloud" Database Schema Inconsistencies on the Upgraded Setup - Table "volumes"
Date Wed, 31 Jul 2013 14:53:28 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13112/
-----------------------------------------------------------

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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message