weizhouapache edited a comment on pull request #4378: URL: https://github.com/apache/cloudstack/pull/4378#issuecomment-868444598
> changes look good, but these are made to a lt of suspect code. > Also my original functional doubt is unanswerred: > > * one operator makes one host/cluster free or less occupied > * another one makes another host/cluster free > * they end out planning VM-tennis @DaanHoogland yeah, but it could happen even if users migrate vms manually. the choose of destination host in this pr, is same as what cloudstack does when start a vm (capacity check, affinity group check, tag check, ordering, etc). migrate vm without hostid -> simiar as start vm without hostid migrate vm with hostid -> simiar as start vm with hostid We/community provide features, but it is up to users to choose a proper solution which meet their requirements. in your example, users can migrate vms with destination host(again, it is up to them), or use other options like host tags. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
