You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2021/07/07 17:45:54 UTC

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

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


   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.
   


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