----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10010/ -----------------------------------------------------------
(Updated March 20, 2013, 6:21 a.m.) Review request for cloudstack and Chip Childers. Changes ------- Included fix for CLOUDSTACK-1718 also in the earlier patch by mistake. Uploaded corrected patch now. thanks Description ------- Included VpnUsers in Add state while listing. Added state to Vpn user response This addresses bug CLOUDSTACK-198. Diffs (updated) ----- api/src/org/apache/cloudstack/api/response/VpnUsersResponse.java e654e8a server/src/com/cloud/api/ApiResponseHelper.java 8c97615 server/src/com/cloud/network/vpn/RemoteAccessVpnManagerImpl.java 82c0015 Diff: https://reviews.apache.org/r/10010/diff/ Testing ------- Tested listVpnUsers API with VPN users in Add state. Unit tests passed. Thanks, Kishan Kavala