Fang, We should remove instance-id as well because no user should have access to the CS generated name.
Will ________________________________________ From: Fang Wang [[email protected]] On Behalf Of Fang Wang [[email protected]] Sent: Tuesday, August 28, 2012 3:49 PM To: Kelven Yang Cc: cloudstack; Fang Wang Subject: Review Request: Replace db-id by UUID for user VM's vm-id. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6814/ ----------------------------------------------------------- Review request for cloudstack and Kelven Yang. Description ------- Output uuid instead of db_id for vm_id. This addresses bug cs-15627. Diffs ----- api/src/com/cloud/vm/VirtualMachineProfile.java 30aed4e server/src/com/cloud/network/element/CloudZonesNetworkElement.java 5f93cac server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 0aaad10 server/src/com/cloud/vm/VirtualMachineProfileImpl.java 6028c95 Diff: https://reviews.apache.org/r/6814/diff/ Testing ------- Verified on user VM. Thanks, Fang Wang
