----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9157/#review15839 -----------------------------------------------------------
api/src/com/cloud/configuration/ConfigurationService.java <https://reviews.apache.org/r/9157/#comment34129> There is no need for this file to change, revert the changes core/src/com/cloud/vm/VMInstanceVO.java <https://reviews.apache.org/r/9157/#comment34132> Why is_volatile a property of the vm instance? - Koushik Das On Jan. 30, 2013, 11:32 a.m., Harikrishna Patnala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9157/ > ----------------------------------------------------------- > > (Updated Jan. 30, 2013, 11:32 a.m.) > > > Review request for cloudstack, Abhinandan Prateek and Koushik Das. > > > Description > ------- > > CLOUDSTACK-745: Reset a VM on reboot > To have a method to reset the vm on its reboot. > So on reboot, the vm's disk is discarded and creates a new root disk using > the template. > The use cases for this are > -> Secure environments that need a fresh start on every boot > -> Desktops that should not retain state > > > This addresses bug CLOUDSTACK-745. > > > Diffs > ----- > > api/src/com/cloud/configuration/ConfigurationService.java 48a0607 > api/src/com/cloud/offering/ServiceOffering.java b13346c > api/src/org/apache/cloudstack/api/ApiConstants.java 58a7831 > > api/src/org/apache/cloudstack/api/command/admin/offering/CreateServiceOfferingCmd.java > 54151fe > core/src/com/cloud/vm/UserVmVO.java 05a4bd1 > core/src/com/cloud/vm/VMInstanceVO.java 13c1cf3 > server/src/com/cloud/baremetal/BareMetalVmManagerImpl.java 57cfb39 > server/src/com/cloud/configuration/ConfigurationManager.java 19e74bf > server/src/com/cloud/configuration/ConfigurationManagerImpl.java df6642a > server/src/com/cloud/migration/ServiceOffering21VO.java fdec30e > server/src/com/cloud/network/security/SecurityManagerMBeanImpl.java 9328190 > server/src/com/cloud/service/ServiceOfferingVO.java c199a86 > server/src/com/cloud/vm/UserVmManagerImpl.java 5891056 > server/test/com/cloud/vm/dao/UserVmDaoImplTest.java f07abca > server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 3989ecf > setup/db/create-schema.sql 174f53c > setup/db/db/schema-40to410.sql 79b7f8f > > Diff: https://reviews.apache.org/r/9157/diff/ > > > Testing > ------- > > Did the manual testing locally > > > Thanks, > > Harikrishna Patnala > >