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