-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7058/
-----------------------------------------------------------

(Updated Sept. 14, 2012, 3:17 p.m.)


Review request for cloudstack and Devdeep Singh.


Changes
-------

Changed name getServiceGroupIfExisits() to nsServiceGroupExists() and it now 
returns boolean.
The changed patch only has the functionality change. Will submit re-org 
separately.


Description
-------

AutoScale. Added clean-up if NetScaler errors out during AutoScale config.
Also re-organized NetScalerResource to keep AutoScale block together.


This addresses bug CS-15727.


Diffs (updated)
-----

  
plugins/network-elements/netscaler/src/com/cloud/network/resource/NetscalerResource.java
 0d13622 

Diff: https://reviews.apache.org/r/7058/diff/


Testing
-------

1. Normal AutoScale config.
2. Login to netscaler. Remove timer policy.
3. Remove AutoScale VmGroup from CloudStack.
4. NS Config should return as it was in "No 1" after the error.


Thanks,

Vijay Venkatachalam

Reply via email to