DK101010 commented on pull request #5848: URL: https://github.com/apache/cloudstack/pull/5848#issuecomment-1055242247
> code looks kind of ok (one remark about the upgrade code. I wonder about the usefulness of this change, @DK101010 . The UUID is mostly usefull if you can search for it is it. Shouldn't there be a searchfield added for it on the listVolumes API? In our workflow (at least until now) it isn't necessary to add this as search parameter, admins searching volumes with already existing parameter but need this uuids to find these volumes on hard disk for further actions. It is a very special value and I have no clue in which use case a user goes to the hard disk, pick up a uuid and searching with that a volume in CS. -- 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]
