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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
