cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kishan Kavala" <Kishan.Kav...@citrix.com>
Subject Re: Review Request: Included VpnUsers in Add state while listing. Added state to Vpn user response
Date Wed, 20 Mar 2013 06:21:53 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message