This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/master by this push:
     new 49be7ee  CLOUDSTACK-9800: Enabled netscaler inline mode. Added 
corrsponding changes in UI (#1964)
49be7ee is described below

commit 49be7eecd2e7db33a421b8680e4e8806a3d63952
Author: kishankavala <kishankav...@users.noreply.github.com>
AuthorDate: Tue Dec 19 04:18:55 2017 +0530

    CLOUDSTACK-9800: Enabled netscaler inline mode. Added corrsponding changes 
in UI (#1964)
    
    NetScaler can work in inline mode also along with other Firewall devices. 
Enabled netscaler inline mode capability
---
 .../netscaler/src/com/cloud/network/element/NetscalerElement.java     | 3 +++
 ui/scripts/configuration.js                                           | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java
 
b/plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java
index 2041735..38a836d 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/network/element/NetscalerElement.java
@@ -509,6 +509,9 @@ public class NetscalerElement extends 
ExternalLoadBalancerDeviceManagerImpl
         // Supports only Public load balancing
         lbCapabilities.put(Capability.LbSchemes, 
LoadBalancerContainer.Scheme.Public.toString());
 
+        // Support inline mode with firewall
+        lbCapabilities.put(Capability.InlineMode, "true");
+
         // Specifies that load balancing rules can support autoscaling and the 
list of counters it supports
         // list of counters it supports
         AutoScaleCounter counter;
diff --git a/ui/scripts/configuration.js b/ui/scripts/configuration.js
index 00972b6..cd4bad4 100644
--- a/ui/scripts/configuration.js
+++ b/ui/scripts/configuration.js
@@ -2634,7 +2634,7 @@
                                         //hide/show service fields ***** (end) 
*****
 
                                         //show LB InlineMode dropdown only 
when (1)LB service is checked and LB service provider is F5BigIp (2)Firewall 
service is checked and Firewall service provider is JuniperSRX
-                                        if 
((args.$form.find('.form-item[rel=\"service.Lb.isEnabled\"]').find('input[type=checkbox]').is(':checked')
 == true) && 
(args.$form.find('.form-item[rel=\"service.Lb.provider\"]').find('select').val()
 == 'F5BigIp') &&
+                                        if 
((args.$form.find('.form-item[rel=\"service.Lb.isEnabled\"]').find('input[type=checkbox]').is(':checked')
 == true) && 
((args.$form.find('.form-item[rel=\"service.Lb.provider\"]').find('select').val()
 == 'F5BigIp') || 
(args.$form.find('.form-item[rel=\"service.Lb.provider\"]').find('select').val()
 == 'Netscaler')) &&
                                             
(args.$form.find('.form-item[rel=\"service.Firewall.isEnabled\"]').find('input[type=checkbox]').is(':checked')
 == true) && 
(args.$form.find('.form-item[rel=\"service.Firewall.provider\"]').find('select').val()
 == 'JuniperSRX')) {
                                             
args.$form.find('.form-item[rel=\"service.Lb.inlineModeDropdown\"]').css('display',
 'inline-block');
                                         } else {
@@ -3267,7 +3267,7 @@
                                             inputData['servicecapabilitylist[' 
+ serviceCapabilityIndex + '].capabilitytype'] = 'ElasticLb';
                                             inputData['servicecapabilitylist[' 
+ serviceCapabilityIndex + '].capabilityvalue'] = true; //because this 
checkbox's value == "on"
                                             serviceCapabilityIndex++;
-                                        } else if ((key == 
'service.Lb.inlineModeDropdown') && ("Lb" in serviceProviderMap) && 
(serviceProviderMap.Lb == "F5BigIp")) {
+                                        } else if ((key == 
'service.Lb.inlineModeDropdown') && ("Lb" in serviceProviderMap) && 
((serviceProviderMap.Lb == "F5BigIp") || (serviceProviderMap.Lb == 
"Netscaler"))) {
                                             if (value == 'true') { //CS-16605 
do not pass parameter if value is 'false'(side by side)
                                                 
inputData['servicecapabilitylist[' + serviceCapabilityIndex + '].service'] = 
'lb';
                                                 
inputData['servicecapabilitylist[' + serviceCapabilityIndex + 
'].capabilitytype'] = 'InlineMode';

-- 
To stop receiving notification emails like this one, please contact
['"commits@cloudstack.apache.org" <commits@cloudstack.apache.org>'].

Reply via email to