rhtyd commented on pull request #21: URL: https://github.com/apache/cloudstack-go/pull/21#issuecomment-979946839
Considering all the discussions we've had, I think it's safe to say that moving forward the specific datatype of the mgmt server ID is string. The only cons are that whoever may be using this response attribute will have to handle the typecast of how it is consumed by their code, however, this is not something most people would use out of an API request. I think it makes sense to rename the struct to something like ManagementServerID or UUID and have the changes such that at least it can handle all past/new ACS releases wherein the response object was integer and string. -- 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]
