[GitHub] [cloudstack] sureshanaparti commented on pull request #4378: server: Optional destination host when migrate a vm

[GitBox <gi...@apache.org>]

sureshanaparti commented on pull request #4378:
URL: https://github.com/apache/cloudstack/pull/4378#issuecomment-884949132


   > > Sorry for the late reply, @ustcweizhou.
   > > > @GabrielBrascher @sureshanaparti
   > > > we have 3 options
   > > > (1) autoselect is 'true' by default
   > > > (2) autoselect is 'false' by default (current behavior)
   > > > (3) a global setting. if autoselect is not set in api request, then honor the global setting.
   > > > what's your opinion ?
   > > 
   > > 
   > > I like to give autonomy to Admins thus I would go with **(1)**, but I do understand why **(2)** is considered.
   > > Regarding **(3)**, I personally prefer to avoid adding new global settings, if possible. Keeping **(1)** or **(2)** would update the API documentation which would keep users/admins in the same page with the API changes.
   > 
   > @GabrielBrascher thanks for reply. both are ok to me.
   > @sureshanaparti your opinion ?
   
   I'm OK with **(1)**, so auto pick the host to migrate when host/storage are not specified, else ignore the autoselect flag.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org