Review Request 13268: fix bug CLOUDSTACK-4088 cloudstack will stop and delete vm which not belongs to cs
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13268/ --- 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
Re: Review Request 13268: fix bug CLOUDSTACK-4088 cloudstack will stop and delete vm which not belongs to cs
--- 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