rafaelweingartner commented on issue #2639: [CLOUDSTACK-10276] listVolumes not 
working when storage UUID is not a UUID
URL: https://github.com/apache/cloudstack/pull/2639#issuecomment-390929227
 
 
   I really do not see any other smaller/simpler solution for this issue. 
Moreover, it does not affect/break anything in the client side. The client is 
already seeing an ID, which is a UUID.
   
   The changes are only internal to ACS. I changed the type of `storageId` from 
`long` to `String` to avoid using that method that converts the given UUID to 
an internal database ID. That method is validating the input, and ignoring 
anything that is not an UUID. Then, I am using the UUID as the filter at 
`QueryManagerImpl.java`.
   
   The changed lines are only 2. The other lines are a result of a code cleanup 
that I did, and the code formatting I applied.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to