----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13268/ -----------------------------------------------------------
(Updated Aug. 5, 2013, 3:59 p.m.) Review request for cloudstack, Abhinandan Prateek, edison su, and mice xia. Bugs: CLOUDSTACK-4088 Repository: cloudstack-git Description ------- fullSync will check if the vm is following CS naming and will stop the vms which are not belongs to CS and having the same name with CS vms. but in deltaSync it will stop all vms not belongs to CS. It is unreasonable to stop or remove a vm that not belongs to CS. Diffs ----- server/src/com/cloud/vm/VirtualMachineManagerImpl.java b33ee49 Diff: https://reviews.apache.org/r/13268/diff/ Testing ------- vm which not belongs to CS will not removed or stopped Thanks, Hongtu Zang
