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

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

commit e243a31e41ed52a78325a6d7b08351dbf236bb30
Author: Nitin Kumar Maharana <nitinkumar.mahar...@accelerite.com>
AuthorDate: Sun Jun 18 19:10:55 2017 +0530

    CLOUDSTACK-8672 : NCC Integration with CloudStack.
    Improvements.
---
 .../admin/address/AcquirePodIpCmdByAdmin.java      |   3 +-
 .../service/NetworkOrchestrationService.java       |  18 +-
 .../service/VolumeOrchestrationService.java        |   6 +-
 .../src/com/cloud/network/IpAddressManager.java    |  49 +-
 .../offerings/dao/NetworkOfferingDaoImpl.java      |   4 +-
 .../commands/DeleteNetscalerControlCenterCmd.java  |   2 -
 .../commands/DeleteServicePackageOfferingCmd.java  |   2 -
 .../api/commands/RegisterServicePackageCmd.java    |   1 -
 .../cloud/network/NetScalerControlCenterVO.java    |   1 -
 .../network/dao/NetScalerControlCenterDaoImpl.java |   5 -
 .../dao/NetScalerServicePackageDaoImpl.java        |   3 -
 .../cloud/network/element/NetscalerElement.java    |   6 -
 .../resource/NetScalerControlCenterResource.java   |  25 +-
 .../com/cloud/network/vm/NetScalerVMManager.java   |  15 +-
 .../cloud/network/vm/NetScalerVMManagerImpl.java   | 662 ++++++++++-----------
 server/src/com/cloud/alert/AlertManagerImpl.java   |  14 +-
 .../ExternalLoadBalancerDeviceManagerImpl.java     |   5 +-
 .../network/lb/LoadBalancingRulesManagerImpl.java  |   2 +-
 .../cloudstack/network/ssl/CertServiceImpl.java    |   3 +-
 .../test/com/cloud/vpc/dao/MockNetworkDaoImpl.java | 149 +----
 20 files changed, 399 insertions(+), 576 deletions(-)

diff --git 
a/api/src/org/apache/cloudstack/api/command/admin/address/AcquirePodIpCmdByAdmin.java
 
b/api/src/org/apache/cloudstack/api/command/admin/address/AcquirePodIpCmdByAdmin.java
index eba41b4..fea0ca6 100644
--- 
a/api/src/org/apache/cloudstack/api/command/admin/address/AcquirePodIpCmdByAdmin.java
+++ 
b/api/src/org/apache/cloudstack/api/command/admin/address/AcquirePodIpCmdByAdmin.java
@@ -42,7 +42,7 @@ public class AcquirePodIpCmdByAdmin extends BaseCmd {
     //////////////// API parameters /////////////////////
     /////////////////////////////////////////////////////
 
-    @Parameter(name = ApiConstants.ZONE_ID, type = CommandType.STRING, 
entityType = ZoneResponse.class, required = true, description = "the ID of the  
zone")
+    @Parameter(name = ApiConstants.ZONE_ID, type = CommandType.STRING, 
entityType = ZoneResponse.class, required = true, description = "the ID of the 
zone")
     private String zoneId;
 
     @Parameter(name = ApiConstants.POD_ID, type = CommandType.STRING, 
entityType = ZoneResponse.class, required = false, description = "Pod ID")
@@ -86,7 +86,6 @@ public class AcquirePodIpCmdByAdmin extends BaseCmd {
 
     @Override
     public long getEntityOwnerId() {
-        // TODO Auto-generated method stub
         return CallContext.current().getCallingAccount().getAccountId();
     }
 
diff --git 
a/engine/api/src/org/apache/cloudstack/engine/orchestration/service/NetworkOrchestrationService.java
 
b/engine/api/src/org/apache/cloudstack/engine/orchestration/service/NetworkOrchestrationService.java
index b59c9ca..13bfee6 100644
--- 
a/engine/api/src/org/apache/cloudstack/engine/orchestration/service/NetworkOrchestrationService.java
+++ 
b/engine/api/src/org/apache/cloudstack/engine/orchestration/service/NetworkOrchestrationService.java
@@ -60,19 +60,21 @@ import com.cloud.vm.VirtualMachineProfile;
  *
  */
 public interface NetworkOrchestrationService {
-    static final String NetworkLockTimeoutCK = "network.lock.timeout";
-    static final String GuestDomainSuffixCK = "guest.domain.suffix";
-    static final String NetworkThrottlingRateCK = "network.throttling.rate";
-    static final String MinVRVersionCK = "minreq.sysvmtemplate.version";
+    String NetworkLockTimeoutCK = "network.lock.timeout";
+    String GuestDomainSuffixCK = "guest.domain.suffix";
+    String NetworkThrottlingRateCK = "network.throttling.rate";
+    String MinVRVersionCK = "minreq.sysvmtemplate.version";
 
-    static final ConfigKey<String> MinVRVersion = new 
ConfigKey<String>(String.class, MinVRVersionCK, "Advanced", "4.10.0",
+    ConfigKey<String> MinVRVersion = new ConfigKey<String>(String.class, 
MinVRVersionCK, "Advanced", "4.10.0",
             "What version should the Virtual Routers report", true, 
ConfigKey.Scope.Zone, null);
 
-    static final ConfigKey<Integer> NetworkLockTimeout = new 
ConfigKey<Integer>(Integer.class, NetworkLockTimeoutCK, "Network", "600",
+    ConfigKey<Integer> NetworkLockTimeout = new 
ConfigKey<Integer>(Integer.class, NetworkLockTimeoutCK, "Network", "600",
         "Lock wait timeout (seconds) while implementing network", true, 
Scope.Global, null);
-    static final ConfigKey<String> GuestDomainSuffix = new 
ConfigKey<String>(String.class, GuestDomainSuffixCK, "Network", 
"cloud.internal",
+
+    ConfigKey<String> GuestDomainSuffix = new ConfigKey<String>(String.class, 
GuestDomainSuffixCK, "Network", "cloud.internal",
         "Default domain name for vms inside virtualized networks fronted by 
router", true, ConfigKey.Scope.Zone, null);
-    static final ConfigKey<Integer> NetworkThrottlingRate = new 
ConfigKey<Integer>("Network", Integer.class, NetworkThrottlingRateCK, "200",
+
+    ConfigKey<Integer> NetworkThrottlingRate = new 
ConfigKey<Integer>("Network", Integer.class, NetworkThrottlingRateCK, "200",
         "Default data transfer rate in megabits per second allowed in 
network.", true, ConfigKey.Scope.Zone);
 
     List<? extends Network> setupNetwork(Account owner, NetworkOffering 
offering, DeploymentPlan plan, String name, String displayText, boolean 
isDefault)
diff --git 
a/engine/api/src/org/apache/cloudstack/engine/orchestration/service/VolumeOrchestrationService.java
 
b/engine/api/src/org/apache/cloudstack/engine/orchestration/service/VolumeOrchestrationService.java
index 3522c1b..451995f 100644
--- 
a/engine/api/src/org/apache/cloudstack/engine/orchestration/service/VolumeOrchestrationService.java
+++ 
b/engine/api/src/org/apache/cloudstack/engine/orchestration/service/VolumeOrchestrationService.java
@@ -57,20 +57,22 @@ import com.cloud.vm.VirtualMachineProfile;
  */
 public interface VolumeOrchestrationService {
 
-    static final ConfigKey<Long> CustomDiskOfferingMinSize = new 
ConfigKey<Long>("Advanced",
+    ConfigKey<Long> CustomDiskOfferingMinSize = new ConfigKey<Long>("Advanced",
         Long.class,
         "custom.diskoffering.size.min",
         "1",
         "Minimum size in GB for custom disk offering.",
         true
     );
-    static final ConfigKey<Long> CustomDiskOfferingMaxSize = new 
ConfigKey<Long>("Advanced",
+
+    ConfigKey<Long> CustomDiskOfferingMaxSize = new ConfigKey<Long>("Advanced",
         Long.class,
         "custom.diskoffering.size.max",
         "1024",
         "Maximum size in GB for custom disk offering.",
         true
     );
+
     VolumeInfo moveVolume(VolumeInfo volume, long destPoolDcId, Long 
destPoolPodId, Long destPoolClusterId, HypervisorType dataDiskHyperType)
         throws ConcurrentOperationException, StorageUnavailableException;
 
diff --git a/engine/components-api/src/com/cloud/network/IpAddressManager.java 
b/engine/components-api/src/com/cloud/network/IpAddressManager.java
index 30976e5..d469cba 100644
--- a/engine/components-api/src/com/cloud/network/IpAddressManager.java
+++ b/engine/components-api/src/com/cloud/network/IpAddressManager.java
@@ -40,12 +40,12 @@ import com.cloud.vm.NicProfile;
 import com.cloud.vm.VirtualMachineProfile;
 
 public interface IpAddressManager {
-    static final String UseSystemPublicIpsCK = "use.system.public.ips";
-    static final ConfigKey<Boolean> UseSystemPublicIps = new 
ConfigKey<Boolean>("Advanced", Boolean.class, UseSystemPublicIpsCK, "true",
+    String UseSystemPublicIpsCK = "use.system.public.ips";
+    ConfigKey<Boolean> UseSystemPublicIps = new ConfigKey<Boolean>("Advanced", 
Boolean.class, UseSystemPublicIpsCK, "true",
             "If true, when account has dedicated public ip range(s), once the 
ips dedicated to the account have been consumed ips will be acquired from the 
system pool",
             true, ConfigKey.Scope.Account);
 
-    static final ConfigKey<Boolean> RulesContinueOnError = new 
ConfigKey<Boolean>("Advanced", Boolean.class, 
"network.rule.delete.ignoreerror", "true",
+    ConfigKey<Boolean> RulesContinueOnError = new 
ConfigKey<Boolean>("Advanced", Boolean.class, 
"network.rule.delete.ignoreerror", "true",
             "When true, ip address delete (ipassoc) failures are  ignored", 
true);
 
     /**
@@ -53,14 +53,11 @@ public interface IpAddressManager {
      *
      * @param dcId
      * @param podId
-     *            TODO
      * @param owner
      * @param type
      * @param networkId
      * @param requestedIp
-     *            TODO
-     * @param allocatedBy
-     *            TODO
+     * @param isSystem
      * @return
      * @throws InsufficientAddressCapacityException
      */
@@ -72,8 +69,7 @@ public interface IpAddressManager {
      *
      * @param userId
      * @param caller
-     *            TODO
-     * @param IpAddress
+     * @param caller
      * @return true if it did; false if it didn't
      */
     boolean disassociatePublicIpAddress(long id, long userId, Account caller);
@@ -82,17 +78,18 @@ public interface IpAddressManager {
             throws ResourceUnavailableException;
 
     /**
-     * @throws ResourceAllocationException TODO
-     * @throws InsufficientCapacityException
-     *             Associates an ip address list to an account. The list of ip 
addresses are all addresses associated
-     *             with the
-     *             given vlan id.
      * @param userId
      * @param accountId
      * @param zoneId
      * @param vlanId
-     * @throws InsufficientAddressCapacityException
-     * @throws
+     * @param guestNetwork
+     * @throws InsufficientCapacityException
+     * @throws ConcurrentOperationException
+     * @throws ResourceUnavailableException
+     * @throws ResourceAllocationException
+     *             Associates an ip address list to an account. The list of ip 
addresses are all addresses associated
+     *             with the
+     *             given vlan id.
      */
     boolean associateIpAddressListToAccount(long userId, long accountId, long 
zoneId, Long vlanId, Network guestNetwork) throws InsufficientCapacityException,
             ConcurrentOperationException, ResourceUnavailableException, 
ResourceAllocationException;
@@ -104,7 +101,7 @@ public interface IpAddressManager {
 
     IPAddressVO markIpAsUnavailable(long addrId);
 
-    public String acquireGuestIpAddress(Network network, String requestedIp);
+    String acquireGuestIpAddress(Network network, String requestedIp);
 
     boolean applyStaticNats(List<? extends StaticNat> staticNats, boolean 
continueOnError, boolean forRevoke) throws ResourceUnavailableException;
 
@@ -125,9 +122,15 @@ public interface IpAddressManager {
     PublicIp assignSourceNatIpAddressToGuestNetwork(Account owner, Network 
guestNetwork) throws InsufficientAddressCapacityException, 
ConcurrentOperationException;
 
     /**
+     *
      * @param ipAddrId
      * @param networkId
-     * @param releaseOnFailure TODO
+     * @param releaseOnFailure
+     * @return
+     * @throws ResourceAllocationException
+     * @throws ResourceUnavailableException
+     * @throws InsufficientAddressCapacityException
+     * @throws ConcurrentOperationException
      */
     IPAddressVO associateIPToGuestNetwork(long ipAddrId, long networkId, 
boolean releaseOnFailure) throws ResourceAllocationException, 
ResourceUnavailableException,
             InsufficientAddressCapacityException, ConcurrentOperationException;
@@ -169,7 +172,6 @@ public interface IpAddressManager {
     IpAddress allocateIp(Account ipOwner, boolean isSystem, Account caller, 
long callerId, DataCenter zone, Boolean displayIp) throws 
ConcurrentOperationException,
             ResourceAllocationException, InsufficientAddressCapacityException;
 
-
     PublicIp assignPublicIpAddressFromVlans(long dcId, Long podId, Account 
owner, VlanType type, List<Long> vlanDbIds, Long networkId, String requestedIp,
             boolean isSystem) throws InsufficientAddressCapacityException;
 
@@ -179,15 +181,12 @@ public interface IpAddressManager {
 
     int getRuleCountForIp(Long addressId, FirewallRule.Purpose purpose, 
FirewallRule.State state);
 
-    public String allocateGuestIP(Network network, String requestedIp) throws 
InsufficientAddressCapacityException;
+    String allocateGuestIP(Network network, String requestedIp) throws 
InsufficientAddressCapacityException;
 
     String allocatePublicIpForGuestNic(Network network, Long podId, Account 
ipOwner, String requestedIp) throws InsufficientAddressCapacityException;
 
-    public AcquirePodIpCmdResponse allocatePodIp(String zoneId, String podId) 
throws ConcurrentOperationException,
-            ResourceAllocationException;
-
-    public void releasePodIp(Long id) throws CloudRuntimeException;
-
+    AcquirePodIpCmdResponse allocatePodIp(String zoneId, String podId) throws 
ConcurrentOperationException, ResourceAllocationException;
 
+    void releasePodIp(Long id) throws CloudRuntimeException;
 }
 
diff --git 
a/engine/schema/src/com/cloud/offerings/dao/NetworkOfferingDaoImpl.java 
b/engine/schema/src/com/cloud/offerings/dao/NetworkOfferingDaoImpl.java
index c7298f2..b0cf0fe 100644
--- a/engine/schema/src/com/cloud/offerings/dao/NetworkOfferingDaoImpl.java
+++ b/engine/schema/src/com/cloud/offerings/dao/NetworkOfferingDaoImpl.java
@@ -23,6 +23,7 @@ import java.util.Map;
 import javax.inject.Inject;
 import javax.persistence.EntityExistsException;
 
+import org.apache.commons.collections.CollectionUtils;
 import org.springframework.stereotype.Component;
 
 import com.cloud.network.Network;
@@ -214,7 +215,8 @@ public class NetworkOfferingDaoImpl extends 
GenericDaoBase<NetworkOfferingVO, Lo
         sc.addAnd("state", SearchCriteria.Op.EQ, 
NetworkOffering.State.Enabled);
         sc.addAnd("servicePackageUuid", SearchCriteria.Op.EQ, uuid);
         List<NetworkOfferingVO> list = this.search(sc, searchFilter);
-        if(list!=null && !list.isEmpty())
+
+        if(CollectionUtils.isNotEmpty(list))
             return true;
 
         return false;
diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/api/commands/DeleteNetscalerControlCenterCmd.java
 
b/plugins/network-elements/netscaler/src/com/cloud/api/commands/DeleteNetscalerControlCenterCmd.java
index 00d4b32..f572b09 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/api/commands/DeleteNetscalerControlCenterCmd.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/api/commands/DeleteNetscalerControlCenterCmd.java
@@ -84,13 +84,11 @@ public class DeleteNetscalerControlCenterCmd extends 
BaseCmd {
 
     @Override
     public String getCommandName() {
-        // TODO Auto-generated method stub
         return s_name;
     }
 
     @Override
     public long getEntityOwnerId() {
-        // TODO Auto-generated method stub
         return CallContext.current().getCallingAccount().getId();
     }
 
diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/api/commands/DeleteServicePackageOfferingCmd.java
 
b/plugins/network-elements/netscaler/src/com/cloud/api/commands/DeleteServicePackageOfferingCmd.java
index 96cd16e..c6fbec1 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/api/commands/DeleteServicePackageOfferingCmd.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/api/commands/DeleteServicePackageOfferingCmd.java
@@ -81,13 +81,11 @@ public class DeleteServicePackageOfferingCmd extends 
BaseCmd {
 
     @Override
     public String getCommandName() {
-        // TODO Auto-generated method stub
         return s_name;
     }
 
     @Override
     public long getEntityOwnerId() {
-        // TODO Auto-generated method stub
         return CallContext.current().getCallingAccount().getId();
     }
 
diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/api/commands/RegisterServicePackageCmd.java
 
b/plugins/network-elements/netscaler/src/com/cloud/api/commands/RegisterServicePackageCmd.java
index 9fabe8d..fdef005 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/api/commands/RegisterServicePackageCmd.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/api/commands/RegisterServicePackageCmd.java
@@ -58,7 +58,6 @@ public class RegisterServicePackageCmd extends BaseCmd {
         try {
             NetScalerServicePackageResponse response = 
_netsclarLbService.registerNetscalerServicePackage(this);
             if (response != null) {
-                //NetScalerServicePackageResponse response = 
_netsclarLbService.createNetscalerServicePackageResponse(servicePackageVO);
                 response.setObjectName("netscalerservicepackage");
                 response.setResponseName(getCommandName());
                 this.setResponseObject(response);
diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/network/NetScalerControlCenterVO.java
 
b/plugins/network-elements/netscaler/src/com/cloud/network/NetScalerControlCenterVO.java
index 84766bc..e9d4ef5 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/network/NetScalerControlCenterVO.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/network/NetScalerControlCenterVO.java
@@ -121,7 +121,6 @@ public class NetScalerControlCenterVO implements 
InternalIdentity {
 
     @Override
     public long getId() {
-        // TODO Auto-generated method stub
         return id;
     }
 
diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/network/dao/NetScalerControlCenterDaoImpl.java
 
b/plugins/network-elements/netscaler/src/com/cloud/network/dao/NetScalerControlCenterDaoImpl.java
index ec24842..1e4f63d 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/network/dao/NetScalerControlCenterDaoImpl.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/network/dao/NetScalerControlCenterDaoImpl.java
@@ -18,8 +18,6 @@ package com.cloud.network.dao;
 
 import java.util.List;
 
-import javax.ejb.Local;
-
 import org.springframework.stereotype.Component;
 
 import com.cloud.network.NetScalerControlCenterVO;
@@ -27,19 +25,16 @@ import com.cloud.utils.db.DB;
 import com.cloud.utils.db.GenericDaoBase;
 
 @Component
-@Local(value = NetScalerControlCenterDao.class)
 @DB
 public class NetScalerControlCenterDaoImpl extends 
GenericDaoBase<NetScalerControlCenterVO, Long> implements 
NetScalerControlCenterDao {
 
     @Override
     public NetScalerControlCenterVO findByPodId(long podId) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public List<NetScalerControlCenterVO> listByNetScalerDeviceId(long 
netscalerDeviceId) {
-        // TODO Auto-generated method stub
         return null;
     }
 
diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/network/dao/NetScalerServicePackageDaoImpl.java
 
b/plugins/network-elements/netscaler/src/com/cloud/network/dao/NetScalerServicePackageDaoImpl.java
index db3b5c0..25d0f4b 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/network/dao/NetScalerServicePackageDaoImpl.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/network/dao/NetScalerServicePackageDaoImpl.java
@@ -18,8 +18,6 @@ package com.cloud.network.dao;
 
 import java.util.List;
 
-import javax.ejb.Local;
-
 import org.springframework.stereotype.Component;
 
 import com.cloud.network.NetScalerServicePackageVO;
@@ -29,7 +27,6 @@ import com.cloud.utils.db.SearchBuilder;
 import com.cloud.utils.db.SearchCriteria;
 
 @Component
-@Local(value = NetScalerServicePackageDao.class)
 @DB
 public class NetScalerServicePackageDaoImpl extends 
GenericDaoBase<NetScalerServicePackageVO, Long> implements 
NetScalerServicePackageDao {
 
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 3371cf8..2041735 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
@@ -381,7 +381,6 @@ public class NetscalerElement extends 
ExternalLoadBalancerDeviceManagerImpl
             networkDetails.put("name", guestConfig.getName());
             networkPayload.put("network", networkDetails);
         } catch (JSONException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         }
 
@@ -445,7 +444,6 @@ public class NetscalerElement extends 
ExternalLoadBalancerDeviceManagerImpl
             // multple providers available
             return false;
         } catch (ConfigurationException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         }
         return false;
@@ -1345,19 +1343,16 @@ public class NetscalerElement extends 
ExternalLoadBalancerDeviceManagerImpl
 
     @Override
     public NetScalerServicePackageResponse 
listNetscalerServicePackage(RegisterServicePackageCmd cmd) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public NetScalerServicePackageResponse 
deleteNetscalerServicePackage(RegisterServicePackageCmd cmd) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public NetScalerServicePackageResponse 
createNetscalerServicePackageResponse(NetScalerServicePackageVO 
servicePackageVO) {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -1515,7 +1510,6 @@ public class NetscalerElement extends 
ExternalLoadBalancerDeviceManagerImpl
         try {
              resp =  _netScalerVMManager.deployNsVpx(cmd.getAccount(), dest, 
plan, serviceOfferingId, templateId);
         } catch (InsufficientCapacityException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         }
         return resp;
diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/network/resource/NetScalerControlCenterResource.java
 
b/plugins/network-elements/netscaler/src/com/cloud/network/resource/NetScalerControlCenterResource.java
index 1132b6c..347186c 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/network/resource/NetScalerControlCenterResource.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/network/resource/NetScalerControlCenterResource.java
@@ -214,7 +214,7 @@ public class NetScalerControlCenterResource implements 
ServerResource {
             }
     }
 
-    private synchronized String login() throws ExecutionException{// , 
ConfigurationException {
+    private synchronized String login() throws ExecutionException{
         String result = null;
         JSONObject jsonResponse = null;
         try {
@@ -317,7 +317,6 @@ public class NetScalerControlCenterResource implements 
ServerResource {
             getHttpRequest(jsonBody.toString(), agentUri, _sessionid);
             s_logger.debug("Keeping Session Alive");
         } catch (URISyntaxException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         }
     }
@@ -360,7 +359,6 @@ public class NetScalerControlCenterResource implements 
ServerResource {
             String errMsg = "Could not generate URI for NetScaler 
ControlCenter";
             s_logger.error(errMsg, e);
           } catch (JSONException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         }
         return result;
@@ -436,7 +434,7 @@ public class NetScalerControlCenterResource implements 
ServerResource {
             JSONArray lbstatus = res.getJSONArray("lbhealthstatus");
             for(int i=0; i<lbstatus.length(); i++) {
                 JSONObject lbstat = lbstatus.getJSONObject(i);
-                LoadBalancerTO loadBalancer = null;// new 
LoadBalancerTO(lbstat.getString("lb_uuid"));
+                LoadBalancerTO loadBalancer = null;
                 JSONArray dest = lbstat.getJSONArray("destinations");
                 List<DestinationTO> listDestTo = new 
ArrayList<DestinationTO>();
                 for(int d=0; d<dest.length(); d++ ) {
@@ -478,7 +476,6 @@ public class NetScalerControlCenterResource implements 
ServerResource {
             response = getHttpRequest(jsonBody.toString(), agentUri, 
_sessionid);
             s_logger.debug("LBHealthcheck Response :" + response);
         } catch (URISyntaxException e) {
-            // TODO Auto-generated catch block
             e.printStackTrace();
         }
         return response;
@@ -619,10 +616,8 @@ public class NetScalerControlCenterResource implements 
ServerResource {
                 }
                 s_logger.debug("IPStats Response :" + response);
             } catch (URISyntaxException e) {
-                // TODO Auto-generated catch block
                 e.printStackTrace();
             } catch (ExecutionException e) {
-                // TODO Auto-generated catch block
                 s_logger.debug("Seesion Alive" + e.getMessage());
                 e.printStackTrace();
             }
@@ -696,6 +691,10 @@ public class NetScalerControlCenterResource implements 
ServerResource {
     }
 
     @Override
+    public void setName(String name) {
+    }
+
+    @Override
     public boolean start() {
         return true;
     }
@@ -711,33 +710,21 @@ public class NetScalerControlCenterResource implements 
ServerResource {
     }
 
     @Override
-    public void setName(String name) {
-        // TODO Auto-generated method stub
-
-    }
-
-    @Override
     public void setConfigParams(Map<String, Object> params) {
-        // TODO Auto-generated method stub
-
     }
 
     @Override
     public Map<String, Object> getConfigParams() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public int getRunLevel() {
-        // TODO Auto-generated method stub
         return 0;
     }
 
     @Override
     public void setRunLevel(int level) {
-        // TODO Auto-generated method stub
-
     }
 
     public String getSessionID() {
diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/network/vm/NetScalerVMManager.java
 
b/plugins/network-elements/netscaler/src/com/cloud/network/vm/NetScalerVMManager.java
index ae1c9da..c82d2b8 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/network/vm/NetScalerVMManager.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/network/vm/NetScalerVMManager.java
@@ -14,6 +14,7 @@
 //KIND, either express or implied.  See the License for the
 //specific language governing permissions and limitations
 //under the License.
+
 package com.cloud.network.vm;
 import java.util.Map;
 
@@ -27,15 +28,15 @@ import com.cloud.network.router.VirtualRouter;
 import com.cloud.user.Account;
 
 public interface NetScalerVMManager {
+    //RAM/CPU for the system offering used by Internal LB VMs
+    int DEFAULT_NETSCALER_VM_RAMSIZE = 2048; // 2048 MB
+    int DEFAULT_NETSCALER_VM_CPU_MHZ = 1024; // 1024 MHz
 
- //RAM/CPU for the system offering used by Internal LB VMs
- public static final int DEFAULT_NETSCALER_VM_RAMSIZE = 2048;            // 
2048 MB
- public static final int DEFAULT_NETSCALER_VM_CPU_MHZ = 1024;            // 
1024 MHz
+    Map<String, Object> deployNetscalerServiceVm(DeployNetscalerVpxCmd cmd);
 
- public Map<String, Object> deployNetscalerServiceVm(DeployNetscalerVpxCmd 
cmd);
+    VirtualRouter stopNetscalerServiceVm(Long id, boolean forced, Account 
callingAccount, long callingUserId) throws ConcurrentOperationException, 
ResourceUnavailableException;
 
- public VirtualRouter stopNetscalerServiceVm(Long id, boolean forced, Account 
callingAccount, long callingUserId) throws ConcurrentOperationException, 
ResourceUnavailableException;
- public Map<String, Object> deployNsVpx(Account owner, DeployDestination dest, 
DeploymentPlan plan, long svcOffId, long templateId) throws 
InsufficientCapacityException;
+    Map<String, Object> deployNsVpx(Account owner, DeployDestination dest, 
DeploymentPlan plan, long svcOffId, long templateId) throws 
InsufficientCapacityException;
 
- public VirtualRouter stopNetScalerVm(Long id, boolean forced, Account 
callingAccount, long callingUserId);
+    VirtualRouter stopNetScalerVm(Long id, boolean forced, Account 
callingAccount, long callingUserId);
 }
\ No newline at end of file
diff --git 
a/plugins/network-elements/netscaler/src/com/cloud/network/vm/NetScalerVMManagerImpl.java
 
b/plugins/network-elements/netscaler/src/com/cloud/network/vm/NetScalerVMManagerImpl.java
index 63574ad..8d50b59 100644
--- 
a/plugins/network-elements/netscaler/src/com/cloud/network/vm/NetScalerVMManagerImpl.java
+++ 
b/plugins/network-elements/netscaler/src/com/cloud/network/vm/NetScalerVMManagerImpl.java
@@ -98,351 +98,351 @@ import com.cloud.vm.dao.VMInstanceDao;
 
 @Local(value = {NetScalerVMManager.class})
 public class NetScalerVMManagerImpl extends ManagerBase implements 
NetScalerVMManager, VirtualMachineGuru {
- private static final Logger s_logger = 
Logger.getLogger(NetScalerVMManagerImpl.class);
- static final private String NetScalerLbVmNamePrefix = "NS";
-
- @Inject
- IpAddressManager _ipAddrMgr;
- @Inject
- VirtualMachineManager _itMgr;
- @Inject
- DomainRouterDao _internalLbVmDao;
- @Inject
- ConfigurationDao _configDao;
- @Inject
- AgentManager _agentMgr;
- @Inject
- DataCenterDao _dcDao;
- @Inject
- VirtualRouterProviderDao _vrProviderDao;
- @Inject
- ApplicationLoadBalancerRuleDao _lbDao;
- @Inject
- NetworkModel _ntwkModel;
- @Inject
- LoadBalancingRulesManager _lbMgr;
- @Inject
- NicDao _nicDao;
- @Inject
- AccountManager _accountMgr;
- @Inject
- NetworkDao _networkDao;
- @Inject
- NetworkOrchestrationService _networkMgr;
- @Inject
- ServiceOfferingDao _serviceOfferingDao;
- @Inject
- PhysicalNetworkServiceProviderDao _physicalProviderDao;
- @Inject
- NetworkOfferingDao _networkOfferingDao;
- @Inject
- VMTemplateDao _templateDao;
- @Inject
- ResourceManager _resourceMgr;
- @Inject
- VMInstanceDao _vmDao;
- @Inject
- NetworkModel _networkModel;
- @Inject
- PhysicalNetworkDao _physicalNetworkDao;
- @Inject
- VlanDao _vlanDao;
- @Inject
- DomainRouterDao _routerDao;
-
- @Override
- public boolean finalizeVirtualMachineProfile(VirtualMachineProfile profile, 
DeployDestination dest, ReservationContext context) {
-
-     //NetScalerLB vm starts up with 3 Nics
-     //Nic #1 - NS IP
-     //Nic #2 - locallink(guest network)
-     //Nic #3 - public nic
-
-     for (final NicProfile nic : profile.getNics()) {
-         if(nic.getTrafficType() == TrafficType.Control) {
-             nic.setTrafficType(TrafficType.Guest);
-         }
-     }
-     return true;
- }
-
- @Override
- public boolean finalizeDeployment(Commands cmds, VirtualMachineProfile 
profile, DeployDestination dest, ReservationContext context)
-     throws ResourceUnavailableException {
-     return true;
- }
-
- @Override
- public boolean finalizeStart(VirtualMachineProfile profile, long hostId, 
Commands cmds, ReservationContext context) {
-     return true;
- }
-
- @Override
- public boolean finalizeCommandsOnStart(Commands cmds, VirtualMachineProfile 
profile) {
-     return true;
- }
-
- @Override
- public void finalizeStop(VirtualMachineProfile profile, Answer answer) {
- }
-
- @Override
- public void finalizeExpunge(VirtualMachine vm) {
- }
-
- @Override
- public void prepareStop(VirtualMachineProfile profile) {
- }
-
- @Override
- public boolean configure(String name, Map<String, Object> params) throws 
ConfigurationException {
-
-     _itMgr.registerGuru(VirtualMachine.Type.NetScalerVm, this);
-     if (s_logger.isInfoEnabled()) {
-         s_logger.info(getName() + " has been configured");
-     }
-     return true;
- }
-
- @Override
- public String getName() {
-     return _name;
- }
-
- protected VirtualRouter stopInternalLbVm(DomainRouterVO internalLbVm, boolean 
forced, Account caller, long callerUserId) throws ResourceUnavailableException,
-     ConcurrentOperationException {
-     s_logger.debug("Stopping internal lb vm " + internalLbVm);
-     try {
-         _itMgr.advanceStop(internalLbVm.getUuid(), forced);
-         return _internalLbVmDao.findById(internalLbVm.getId());
-     } catch (OperationTimedoutException e) {
-         throw new CloudRuntimeException("Unable to stop " + internalLbVm, e);
-     }
- }
-
- public VirtualRouterProvider addNetScalerLoadBalancerElement(long 
ntwkSvcProviderId) {
-     VirtualRouterProviderVO element = 
_vrProviderDao.findByNspIdAndType(ntwkSvcProviderId, 
com.cloud.network.VirtualRouterProvider.Type.NetScalerVm);
-     if (element != null) {
-         s_logger.debug("There is already an " + getName() + " with service 
provider id " + ntwkSvcProviderId);
-         return element;
-     }
-
-     PhysicalNetworkServiceProvider provider = 
_physicalProviderDao.findById(ntwkSvcProviderId);
-     if (provider == null || 
!provider.getProviderName().equalsIgnoreCase(getName())) {
-         throw new InvalidParameterValueException("Invalid network service 
provider is specified");
-     }
-
-     element = new VirtualRouterProviderVO(ntwkSvcProviderId, 
com.cloud.network.VirtualRouterProvider.Type.NetScalerVm);
-     element.setEnabled(true);
-     element = _vrProviderDao.persist(element);
-     return element;
- }
-
- protected long getNetScalerLbProviderId(long physicalNetworkId) {
-     //final long physicalNetworkId = 
_ntwkModel.getPhysicalNetworkId(guestNetwork);
-
-     final PhysicalNetworkServiceProvider provider = 
_physicalProviderDao.findByServiceProvider(physicalNetworkId, "Netscaler");
-     if (provider == null) {
-         throw new CloudRuntimeException("Cannot find service provider " + 
Provider.Netscaler.toString() + " in physical network " + physicalNetworkId);
-     }
-
-     //TODO: get from type
-     VirtualRouterProvider netScalerLbProvider = 
_vrProviderDao.findByNspIdAndType(provider.getId(), 
com.cloud.network.VirtualRouterProvider.Type.NetScalerVm);
-     if (netScalerLbProvider == null) {
-         //create the vrp for netscalerVM.
-         netScalerLbProvider = 
addNetScalerLoadBalancerElement(provider.getId());
-     }
-
-     return netScalerLbProvider.getId();
- }
- @Override
-    public Map<String, Object> deployNsVpx(Account owner, DeployDestination 
dest, DeploymentPlan plan, long svcOffId, long templateId) throws 
InsufficientCapacityException {
+    private static final Logger s_logger = 
Logger.getLogger(NetScalerVMManagerImpl.class);
+    static final private String NetScalerLbVmNamePrefix = "NS";
+
+    @Inject
+    IpAddressManager _ipAddrMgr;
+    @Inject
+    VirtualMachineManager _itMgr;
+    @Inject
+    DomainRouterDao _internalLbVmDao;
+    @Inject
+    ConfigurationDao _configDao;
+    @Inject
+    AgentManager _agentMgr;
+    @Inject
+    DataCenterDao _dcDao;
+    @Inject
+    VirtualRouterProviderDao _vrProviderDao;
+    @Inject
+    ApplicationLoadBalancerRuleDao _lbDao;
+    @Inject
+    NetworkModel _ntwkModel;
+    @Inject
+    LoadBalancingRulesManager _lbMgr;
+    @Inject
+    NicDao _nicDao;
+    @Inject
+    AccountManager _accountMgr;
+    @Inject
+    NetworkDao _networkDao;
+    @Inject
+    NetworkOrchestrationService _networkMgr;
+    @Inject
+    ServiceOfferingDao _serviceOfferingDao;
+    @Inject
+    PhysicalNetworkServiceProviderDao _physicalProviderDao;
+    @Inject
+    NetworkOfferingDao _networkOfferingDao;
+    @Inject
+    VMTemplateDao _templateDao;
+    @Inject
+    ResourceManager _resourceMgr;
+    @Inject
+    VMInstanceDao _vmDao;
+    @Inject
+    NetworkModel _networkModel;
+    @Inject
+    PhysicalNetworkDao _physicalNetworkDao;
+    @Inject
+    VlanDao _vlanDao;
+    @Inject
+    DomainRouterDao _routerDao;
+
+    @Override
+    public boolean finalizeVirtualMachineProfile(VirtualMachineProfile 
profile, DeployDestination dest, ReservationContext context) {
+        //NetScalerLB vm starts up with 3 Nics
+        //Nic #1 - NS IP
+        //Nic #2 - locallink(guest network)
+        //Nic #3 - public nic
+
+        for (final NicProfile nic : profile.getNics()) {
+            if(nic.getTrafficType() == TrafficType.Control) {
+                nic.setTrafficType(TrafficType.Guest);
+            }
+        }
+        return true;
+    }
 
-     VMTemplateVO template = _templateDao.findById(templateId) ;
-     long id = _vmDao.getNextInSequence(Long.class, "id");
-     Account systemAcct = _accountMgr.getSystemAccount();
-
-     if (template == null) {
-         s_logger.error(" Unable to find the NS VPX template");
-         throw new CloudRuntimeException("Unable to find the Template" + 
templateId);
-     }
-     long dataCenterId = dest.getDataCenter().getId();
-     DataCenterVO dc = _dcDao.findById(dest.getDataCenter().getId());
-
-     String nxVpxName = VirtualMachineName.getSystemVmName(id, "Vpx", 
NetScalerLbVmNamePrefix);
-
-     ServiceOfferingVO vpxOffering =   _serviceOfferingDao.findById(svcOffId); 
//using 2GB and 2CPU  offering
-     if(vpxOffering.getRamSize() < 2048 && vpxOffering.getCpu() <2 ) {
-         throw new InvalidParameterValueException("Specified Service Offering 
:" + vpxOffering.getUuid() + " NS Vpx cannot be deployed. Min 2GB Ram and 2 CPU 
are required");
-     }
-
-     long userId = CallContext.current().getCallingUserId();
-     //TODO change the os bits from 142  103 to the actual guest of bits
-     if(template.getGuestOSId() !=  103 ) {
-         throw new InvalidParameterValueException("Specified Template " + 
template.getUuid()+ " not suitable for NS VPX Deployment. Please register the 
template with guest os type as unknow(64-bit)");
-     }
-     NetworkVO defaultNetwork = null;
-      NetworkVO defaultPublicNetwork = null;
-      if (dc.getNetworkType() == NetworkType.Advanced && 
dc.isSecurityGroupEnabled()) {
-          List<NetworkVO> networks = 
_networkDao.listByZoneSecurityGroup(dataCenterId);
-          if (networks == null || networks.size() == 0) {
-              throw new CloudRuntimeException("Can not found security enabled 
network in SG Zone " + dc);
-          }
-          defaultNetwork = networks.get(0);
-      } else {
-          TrafficType defaultTrafficType = TrafficType.Management;
-          List<NetworkVO> defaultNetworks = 
_networkDao.listByZoneAndTrafficType(dataCenterId, defaultTrafficType);
-
-          TrafficType publicTrafficType = TrafficType.Public;
-          List<NetworkVO> publicNetworks = 
_networkDao.listByZoneAndTrafficType(dataCenterId, publicTrafficType);
-
-          // api should never allow this situation to happen
-          if (defaultNetworks.size() != 1) {
-              throw new CloudRuntimeException("Found " + 
defaultNetworks.size() + " networks of type " + defaultTrafficType + " when 
expect to find 1");
-          }
-
-          if (publicNetworks.size() != 1) {
-              throw new CloudRuntimeException("Found " + 
defaultNetworks.size() + " networks of type " + defaultTrafficType + " when 
expect to find 1");
-          }
-          defaultPublicNetwork = publicNetworks.get(0);
-          defaultNetwork = defaultNetworks.get(0);
-      }
-
-      LinkedHashMap<Network, List<? extends NicProfile>> networks = new 
LinkedHashMap<Network, List<? extends NicProfile>>(4);
-      NicProfile defaultNic = new NicProfile();
-      defaultNic.setDefaultNic(true);
-      defaultNic.setDeviceId(0);
-
-      networks.put(_networkMgr.setupNetwork(_accountMgr.getSystemAccount() , 
_networkOfferingDao.findById(defaultNetwork.getNetworkOfferingId()), plan, 
null, null, false).get(0),
-              new ArrayList<NicProfile>());
+    @Override
+    public boolean finalizeDeployment(Commands cmds, VirtualMachineProfile 
profile, DeployDestination dest, ReservationContext context) throws 
ResourceUnavailableException {
+        return true;
+    }
+
+    @Override
+    public boolean finalizeStart(VirtualMachineProfile profile, long hostId, 
Commands cmds, ReservationContext context) {
+        return true;
+    }
+
+    @Override
+    public boolean finalizeCommandsOnStart(Commands cmds, 
VirtualMachineProfile profile) {
+        return true;
+    }
+
+    @Override
+    public void finalizeStop(VirtualMachineProfile profile, Answer answer) {
+    }
+
+    @Override
+    public void finalizeExpunge(VirtualMachine vm) {
+    }
+
+    @Override
+    public void prepareStop(VirtualMachineProfile profile) {
+    }
+
+    @Override
+    public boolean configure(String name, Map<String, Object> params) throws 
ConfigurationException {
+        _itMgr.registerGuru(VirtualMachine.Type.NetScalerVm, this);
+        if (s_logger.isInfoEnabled()) {
+            s_logger.info(getName() + " has been configured");
+        }
+        return true;
+    }
+
+    @Override
+    public String getName() {
+        return _name;
+    }
+
+    protected VirtualRouter stopInternalLbVm(DomainRouterVO internalLbVm, 
boolean forced, Account caller, long callerUserId) throws 
ResourceUnavailableException, ConcurrentOperationException {
+        s_logger.debug("Stopping internal lb vm " + internalLbVm);
+        try {
+            _itMgr.advanceStop(internalLbVm.getUuid(), forced);
+            return _internalLbVmDao.findById(internalLbVm.getId());
+        } catch (OperationTimedoutException e) {
+            throw new CloudRuntimeException("Unable to stop " + internalLbVm, 
e);
+        }
+    }
 
-      NicProfile defaultNic1 = new NicProfile();
-      defaultNic1.setDefaultNic(false);
-      defaultNic1.setDeviceId(1);
+    public VirtualRouterProvider addNetScalerLoadBalancerElement(long 
ntwkSvcProviderId) {
+        VirtualRouterProviderVO element = 
_vrProviderDao.findByNspIdAndType(ntwkSvcProviderId, 
com.cloud.network.VirtualRouterProvider.Type.NetScalerVm);
+        if (element != null) {
+            s_logger.debug("There is already an " + getName() + " with service 
provider id " + ntwkSvcProviderId);
+            return element;
+        }
+
+        PhysicalNetworkServiceProvider provider = 
_physicalProviderDao.findById(ntwkSvcProviderId);
+        if (provider == null || 
!provider.getProviderName().equalsIgnoreCase(getName())) {
+            throw new InvalidParameterValueException("Invalid network service 
provider is specified");
+        }
+
+        element = new VirtualRouterProviderVO(ntwkSvcProviderId, 
com.cloud.network.VirtualRouterProvider.Type.NetScalerVm);
+        element.setEnabled(true);
+        element = _vrProviderDao.persist(element);
+        return element;
+    }
+
+    protected long getNetScalerLbProviderId(long physicalNetworkId) {
+        //final long physicalNetworkId = 
_ntwkModel.getPhysicalNetworkId(guestNetwork);
+
+        final PhysicalNetworkServiceProvider provider = 
_physicalProviderDao.findByServiceProvider(physicalNetworkId, "Netscaler");
+        if (provider == null) {
+            throw new CloudRuntimeException("Cannot find service provider " + 
Provider.Netscaler.toString() + " in physical network " + physicalNetworkId);
+        }
+
+        //TODO: get from type
+        VirtualRouterProvider netScalerLbProvider = 
_vrProviderDao.findByNspIdAndType(provider.getId(), 
com.cloud.network.VirtualRouterProvider.Type.NetScalerVm);
+        if (netScalerLbProvider == null) {
+            //create the vrp for netscalerVM.
+            netScalerLbProvider = 
addNetScalerLoadBalancerElement(provider.getId());
+        }
+
+        return netScalerLbProvider.getId();
+    }
 
-      NicProfile defaultNic2 = new NicProfile();
-      defaultNic2.setDefaultNic(false);
-      defaultNic2.setDeviceId(2);
-      defaultNic2.setIPv4Address("");
-      defaultNic2.setIPv4Gateway("");
-      defaultNic2.setIPv4Netmask("");
-      String macAddress = 
_networkDao.getNextAvailableMacAddress(defaultPublicNetwork.getId());
-      defaultNic2.setMacAddress(macAddress);
+    @Override
+    public Map<String, Object> deployNsVpx(Account owner, DeployDestination 
dest, DeploymentPlan plan, long svcOffId, long templateId) throws 
InsufficientCapacityException {
 
-      networks.put(_networkMgr.setupNetwork(_accountMgr.getSystemAccount(), 
_networkOfferingDao.findByUniqueName(NetworkOffering.SystemPublicNetwork), 
plan, null, null, false).get(0),
+        VMTemplateVO template = _templateDao.findById(templateId) ;
+        long id = _vmDao.getNextInSequence(Long.class, "id");
+        Account systemAcct = _accountMgr.getSystemAccount();
+
+        if (template == null) {
+            s_logger.error(" Unable to find the NS VPX template");
+            throw new CloudRuntimeException("Unable to find the Template" + 
templateId);
+        }
+        long dataCenterId = dest.getDataCenter().getId();
+        DataCenterVO dc = _dcDao.findById(dest.getDataCenter().getId());
+
+        String nxVpxName = VirtualMachineName.getSystemVmName(id, "Vpx", 
NetScalerLbVmNamePrefix);
+
+        ServiceOfferingVO vpxOffering = 
_serviceOfferingDao.findById(svcOffId); //using 2GB and 2CPU  offering
+        if(vpxOffering.getRamSize() < 2048 && vpxOffering.getCpu() <2 ) {
+            throw new InvalidParameterValueException("Specified Service 
Offering :" + vpxOffering.getUuid() + " NS Vpx cannot be deployed. Min 2GB Ram 
and 2 CPU are required");
+        }
+
+        long userId = CallContext.current().getCallingUserId();
+        //TODO change the os bits from 142  103 to the actual guest of bits
+        if(template.getGuestOSId() !=  103 ) {
+            throw new InvalidParameterValueException("Specified Template " + 
template.getUuid()+ " not suitable for NS VPX Deployment. Please register the 
template with guest os type as unknow(64-bit)");
+        }
+
+        NetworkVO defaultNetwork = null;
+        NetworkVO defaultPublicNetwork = null;
+        if (dc.getNetworkType() == NetworkType.Advanced && 
dc.isSecurityGroupEnabled()) {
+            List<NetworkVO> networks = 
_networkDao.listByZoneSecurityGroup(dataCenterId);
+            if (networks == null || networks.size() == 0) {
+                throw new CloudRuntimeException("Can not found security 
enabled network in SG Zone " + dc);
+            }
+            defaultNetwork = networks.get(0);
+        } else {
+            TrafficType defaultTrafficType = TrafficType.Management;
+            List<NetworkVO> defaultNetworks = 
_networkDao.listByZoneAndTrafficType(dataCenterId, defaultTrafficType);
+
+            TrafficType publicTrafficType = TrafficType.Public;
+            List<NetworkVO> publicNetworks = 
_networkDao.listByZoneAndTrafficType(dataCenterId, publicTrafficType);
+
+            // api should never allow this situation to happen
+            if (defaultNetworks.size() != 1) {
+                throw new CloudRuntimeException("Found " + 
defaultNetworks.size() + " networks of type " + defaultTrafficType + " when 
expect to find 1");
+            }
+
+            if (publicNetworks.size() != 1) {
+                throw new CloudRuntimeException("Found " + 
defaultNetworks.size() + " networks of type " + defaultTrafficType + " when 
expect to find 1");
+            }
+            defaultPublicNetwork = publicNetworks.get(0);
+            defaultNetwork = defaultNetworks.get(0);
+        }
+
+        LinkedHashMap<Network, List<? extends NicProfile>> networks = new 
LinkedHashMap<Network, List<? extends NicProfile>>(4);
+        NicProfile defaultNic = new NicProfile();
+        defaultNic.setDefaultNic(true);
+        defaultNic.setDeviceId(0);
+
+        networks.put(_networkMgr.setupNetwork(_accountMgr.getSystemAccount() , 
_networkOfferingDao.findById(defaultNetwork.getNetworkOfferingId()), plan, 
null, null, false).get(0), new ArrayList<NicProfile>());
+
+        NicProfile defaultNic1 = new NicProfile();
+        defaultNic1.setDefaultNic(false);
+        defaultNic1.setDeviceId(1);
+
+        NicProfile defaultNic2 = new NicProfile();
+        defaultNic2.setDefaultNic(false);
+        defaultNic2.setDeviceId(2);
+        defaultNic2.setIPv4Address("");
+        defaultNic2.setIPv4Gateway("");
+        defaultNic2.setIPv4Netmask("");
+        String macAddress = 
_networkDao.getNextAvailableMacAddress(defaultPublicNetwork.getId(), null);
+        defaultNic2.setMacAddress(macAddress);
+
+        networks.put(_networkMgr.setupNetwork(_accountMgr.getSystemAccount(), 
_networkOfferingDao.findByUniqueName(NetworkOffering.SystemPublicNetwork), 
plan, null, null, false).get(0),
               new ArrayList<NicProfile>(Arrays.asList(defaultNic2)));
 
-      networks.put(_networkMgr.setupNetwork(_accountMgr.getSystemAccount(), 
_networkOfferingDao.findByUniqueName(NetworkOffering.SystemControlNetwork), 
plan, null, null, false).get(0),
+        networks.put(_networkMgr.setupNetwork(_accountMgr.getSystemAccount(), 
_networkOfferingDao.findByUniqueName(NetworkOffering.SystemControlNetwork), 
plan, null, null, false).get(0),
               new ArrayList<NicProfile>());
 
-      long physicalNetworkId = 
_networkModel.findPhysicalNetworkId(dataCenterId, 
_networkOfferingDao.findById(defaultPublicNetwork.getNetworkOfferingId()).getTags(),
 TrafficType.Public);
-      // Validate physical network
-      PhysicalNetwork physicalNetwork = 
_physicalNetworkDao.findById(physicalNetworkId);
-      if (physicalNetwork == null) {
-          throw new InvalidParameterValueException("Unable to find physical 
network with id: " + physicalNetworkId + " and tag: "
+        long physicalNetworkId = 
_networkModel.findPhysicalNetworkId(dataCenterId, 
_networkOfferingDao.findById(defaultPublicNetwork.getNetworkOfferingId()).getTags(),
 TrafficType.Public);
+        // Validate physical network
+        PhysicalNetwork physicalNetwork = 
_physicalNetworkDao.findById(physicalNetworkId);
+        if (physicalNetwork == null) {
+            throw new InvalidParameterValueException("Unable to find physical 
network with id: " + physicalNetworkId + " and tag: "
                   + 
_networkOfferingDao.findById(defaultPublicNetwork.getNetworkOfferingId()).getTags());
-      }
-      String guestvnet = physicalNetwork.getVnetString();
+        }
+        String guestvnet = physicalNetwork.getVnetString();
 
-      final List<VlanVO> vlans = _vlanDao.listByZone(dataCenterId);
-      List<String> pvlan = new ArrayList<String>();
-      for (final VlanVO vlan : vlans) {
-          pvlan.add(vlan.getVlanTag());
-      }
+        final List<VlanVO> vlans = _vlanDao.listByZone(dataCenterId);
+        List<String> pvlan = new ArrayList<String>();
+        for (final VlanVO vlan : vlans) {
+            pvlan.add(vlan.getVlanTag());
+        }
 
-      long netScalerProvider = getNetScalerLbProviderId(physicalNetworkId);
-      DomainRouterVO nsVpx = new DomainRouterVO(id, vpxOffering.getId(), 
netScalerProvider, nxVpxName,
+        long netScalerProvider = getNetScalerLbProviderId(physicalNetworkId);
+        DomainRouterVO nsVpx = new DomainRouterVO(id, vpxOffering.getId(), 
netScalerProvider, nxVpxName,
               template.getId(), template.getHypervisorType(),  
template.getGuestOSId(), owner.getDomainId(), owner.getId(), userId, false, 
RedundantState.UNKNOWN, false,
               false, VirtualMachine.Type.NetScalerVm, null);
 
-          nsVpx.setRole(Role.NETSCALER_VM);
-
-          nsVpx = _routerDao.persist(nsVpx);
-
-      VMInstanceVO vmVO= _vmDao.findVMByHostName(nxVpxName);
-     _itMgr.allocate(nxVpxName, template, vpxOffering, networks, plan, 
template.getHypervisorType());
-     Map<Param, Object> params = new HashMap<VirtualMachineProfile.Param, 
Object>(1);
-     try {
-         if (vmVO != null) {
-             startNsVpx(vmVO, params);
-         } else {
-             throw new NullPointerException();
-         }
-     } catch (StorageUnavailableException e) {
-         e.printStackTrace();
-     } catch (ConcurrentOperationException e) {
-         e.printStackTrace();
-     } catch (ResourceUnavailableException e) {
-         e.printStackTrace();
-     }
-     vmVO= _vmDao.findByUuid(nsVpx.getUuid());
-     Map<String, Object> deployResponse = new HashMap<String, Object>();
-     deployResponse.put("vm", vmVO);
-     deployResponse.put("guestvlan", guestvnet);
-     deployResponse.put("publicvlan", pvlan);
-     return deployResponse;
- }
-
- protected void startNsVpx(VMInstanceVO nsVpx, Map<Param, Object> params) 
throws StorageUnavailableException,
- InsufficientCapacityException, ConcurrentOperationException, 
ResourceUnavailableException {
-     s_logger.debug("Starting NS Vpx " + nsVpx);
-     _itMgr.start(nsVpx.getUuid(), params, null, null);
- }
-
- @Override
- public Map<String,Object> deployNetscalerServiceVm(DeployNetscalerVpxCmd cmd) 
{
-     DataCenter zone = _dcDao.findById(cmd.getZoneId());
-     DeployDestination dest = new DeployDestination(zone, null, null, null);
-     VMInstanceVO vmvo = null;
-     Map<String,Object> resp = new HashMap<String, Object>();
-     Long templateId = cmd.getTemplateId();
-     Long serviceOfferingId = cmd.getServiceOfferingId();
-     DeploymentPlan plan = new 
DataCenterDeployment(dest.getDataCenter().getId());
-     try {
-          resp = deployNsVpx(cmd.getAccount(), dest, plan, serviceOfferingId, 
templateId);
-     } catch (InsufficientCapacityException e) {
-         // TODO Auto-generated catch block
-         e.printStackTrace();
-     }
-     return resp;
- }
-
- protected VirtualRouter stopNetScalerVm(final long vmId, final boolean 
forced, final Account caller, final long callerUserId) throws 
ResourceUnavailableException,
- ConcurrentOperationException {
-     final DomainRouterVO netscalerVm = _routerDao.findById(vmId);
-     s_logger.debug("Stopping NetScaler vm " + netscalerVm);
-
-     if (netscalerVm == null || netscalerVm.getRole() != Role.NETSCALER_VM) {
-         throw new InvalidParameterValueException("Can't find NetScaler vm by 
id specified");
-     }
-     _accountMgr.checkAccess(caller, null, true, netscalerVm);
-     try {
-         _itMgr.expunge(netscalerVm.getUuid());
-         return _routerDao.findById(netscalerVm.getId());
-     } catch (final Exception e) {
-         throw new CloudRuntimeException("Unable to stop " + netscalerVm, e);
-     }
- }
- @Override
- public VirtualRouter stopNetscalerServiceVm(Long id, boolean forced, Account 
callingAccount, long callingUserId) throws ConcurrentOperationException,
-         ResourceUnavailableException {
-     return stopNetScalerVm(id, forced, callingAccount, callingUserId);
- }
-
-@Override
-public VirtualRouter stopNetScalerVm(Long vmId, boolean forced, Account 
caller, long callingUserId) {
-    final DomainRouterVO netscalerVm = _routerDao.findById(vmId);
-    s_logger.debug("Stopping NetScaler vm " + netscalerVm);
-
-    if (netscalerVm == null || netscalerVm.getRole() != Role.NETSCALER_VM) {
-        throw new InvalidParameterValueException("Can't find NetScaler vm by 
id specified");
+        nsVpx.setRole(Role.NETSCALER_VM);
+
+        nsVpx = _routerDao.persist(nsVpx);
+
+        VMInstanceVO vmVO= _vmDao.findVMByHostName(nxVpxName);
+        _itMgr.allocate(nxVpxName, template, vpxOffering, networks, plan, 
template.getHypervisorType());
+        Map<Param, Object> params = new HashMap<VirtualMachineProfile.Param, 
Object>(1);
+        try {
+            if (vmVO != null) {
+                startNsVpx(vmVO, params);
+            } else {
+                throw new NullPointerException();
+            }
+        } catch (StorageUnavailableException e) {
+            e.printStackTrace();
+        } catch (ConcurrentOperationException e) {
+            e.printStackTrace();
+        } catch (ResourceUnavailableException e) {
+            e.printStackTrace();
+        }
+        vmVO= _vmDao.findByUuid(nsVpx.getUuid());
+        Map<String, Object> deployResponse = new HashMap<String, Object>();
+        deployResponse.put("vm", vmVO);
+        deployResponse.put("guestvlan", guestvnet);
+        deployResponse.put("publicvlan", pvlan);
+
+        return deployResponse;
     }
-    _accountMgr.checkAccess(caller, null, true, netscalerVm);
-    try {
-        _itMgr.expunge(netscalerVm.getUuid());
-        return _routerDao.findById(netscalerVm.getId());
-    } catch (final Exception e) {
-        throw new CloudRuntimeException("Unable to stop " + netscalerVm, e);
+
+    protected void startNsVpx(VMInstanceVO nsVpx, Map<Param, Object> params) 
throws StorageUnavailableException,
+    InsufficientCapacityException, ConcurrentOperationException, 
ResourceUnavailableException {
+        s_logger.debug("Starting NS Vpx " + nsVpx);
+        _itMgr.start(nsVpx.getUuid(), params, null, null);
+    }
+
+    @Override
+    public Map<String,Object> deployNetscalerServiceVm(DeployNetscalerVpxCmd 
cmd) {
+        DataCenter zone = _dcDao.findById(cmd.getZoneId());
+        DeployDestination dest = new DeployDestination(zone, null, null, null);
+        VMInstanceVO vmvo = null;
+        Map<String,Object> resp = new HashMap<String, Object>();
+        Long templateId = cmd.getTemplateId();
+        Long serviceOfferingId = cmd.getServiceOfferingId();
+        DeploymentPlan plan = new 
DataCenterDeployment(dest.getDataCenter().getId());
+
+        try {
+            resp = deployNsVpx(cmd.getAccount(), dest, plan, 
serviceOfferingId, templateId);
+        } catch (InsufficientCapacityException e) {
+            e.printStackTrace();
+        }
+
+        return resp;
+    }
+
+    protected VirtualRouter stopNetScalerVm(final long vmId, final boolean 
forced, final Account caller, final long callerUserId) throws 
ResourceUnavailableException, ConcurrentOperationException {
+        final DomainRouterVO netscalerVm = _routerDao.findById(vmId);
+        s_logger.debug("Stopping NetScaler vm " + netscalerVm);
+
+        if (netscalerVm == null || netscalerVm.getRole() != Role.NETSCALER_VM) 
{
+            throw new InvalidParameterValueException("Can't find NetScaler vm 
by id specified");
+        }
+
+        _accountMgr.checkAccess(caller, null, true, netscalerVm);
+
+        try {
+            _itMgr.expunge(netscalerVm.getUuid());
+            return _routerDao.findById(netscalerVm.getId());
+        } catch (final Exception e) {
+            throw new CloudRuntimeException("Unable to stop " + netscalerVm, 
e);
+        }
+    }
+
+    @Override
+    public VirtualRouter stopNetscalerServiceVm(Long id, boolean forced, 
Account callingAccount, long callingUserId) throws 
ConcurrentOperationException, ResourceUnavailableException {
+        return stopNetScalerVm(id, forced, callingAccount, callingUserId);
+    }
+
+    @Override
+    public VirtualRouter stopNetScalerVm(Long vmId, boolean forced, Account 
caller, long callingUserId) {
+        final DomainRouterVO netscalerVm = _routerDao.findById(vmId);
+        s_logger.debug("Stopping NetScaler vm " + netscalerVm);
+
+        if (netscalerVm == null || netscalerVm.getRole() != Role.NETSCALER_VM) 
{
+            throw new InvalidParameterValueException("Can't find NetScaler vm 
by id specified");
+        }
+        _accountMgr.checkAccess(caller, null, true, netscalerVm);
+        try {
+            _itMgr.expunge(netscalerVm.getUuid());
+            return _routerDao.findById(netscalerVm.getId());
+        } catch (final Exception e) {
+            throw new CloudRuntimeException("Unable to stop " + netscalerVm, 
e);
+        }
     }
-}
 }
diff --git a/server/src/com/cloud/alert/AlertManagerImpl.java 
b/server/src/com/cloud/alert/AlertManagerImpl.java
index 93c3b77..c751c6a 100644
--- a/server/src/com/cloud/alert/AlertManagerImpl.java
+++ b/server/src/com/cloud/alert/AlertManagerImpl.java
@@ -90,7 +90,7 @@ public class AlertManagerImpl extends ManagerBase implements 
AlertManager, Confi
     private static final Logger s_logger = 
Logger.getLogger(AlertManagerImpl.class.getName());
     private static final Logger s_alertsLogger = 
Logger.getLogger("org.apache.cloudstack.alerts");
 
-    private static final long INITIAL_CAPACITY_CHECK_DELAY = 30L * 1000L;// 
thirty seconds expressed in milliseconds
+    private static final long INITIAL_CAPACITY_CHECK_DELAY = 30L * 1000L; // 
Thirty seconds expressed in milliseconds.
 
     private static final DecimalFormat DfPct = new DecimalFormat("###.##");
     private static final DecimalFormat DfWhole = new DecimalFormat("########");
@@ -126,7 +126,7 @@ public class AlertManagerImpl extends ManagerBase 
implements AlertManager, Confi
     protected ConfigDepot _configDepot;
 
     private Timer _timer = null;
-    private long _capacityCheckPeriod = 60L * 60L * 1000L;// one hour by 
default
+    private long _capacityCheckPeriod = 60L * 60L * 1000L; // One hour by 
default.
     private double _publicIPCapacityThreshold = 0.75;
     private double _privateIPCapacityThreshold = 0.75;
     private double _secondaryStorageCapacityThreshold = 0.75;
@@ -249,8 +249,8 @@ public class AlertManagerImpl extends ManagerBase 
implements AlertManager, Confi
             if (_emailAlert != null) {
                 _emailAlert.sendAlert(alertType, dataCenterId, podId, null, 
subject, body);
             } else {
-                s_alertsLogger.warn(" alertType:: " + alertType + " // 
dataCenterId:: " + dataCenterId + " // podId:: " + podId +
-                        " // message:: " + subject + " // body:: " + body);
+                s_alertsLogger.warn("AlertType:: " + alertType + " | 
dataCenterId:: " + dataCenterId + " | podId:: " + podId +
+                        " | message:: " + subject + " | body:: " + body);
             }
         } catch (Exception ex) {
             s_logger.error("Problem sending email alert", ex);
@@ -747,8 +747,8 @@ public class AlertManagerImpl extends ManagerBase 
implements AlertManager, Confi
         // TODO:  make sure this handles SSL transport (useAuth is true) and 
regular
         public void sendAlert(AlertType alertType, long dataCenterId, Long 
podId, Long clusterId, String subject, String content) throws 
MessagingException,
                 UnsupportedEncodingException {
-            s_alertsLogger.warn(" alertType:: " + alertType + " // 
dataCenterId:: " + dataCenterId + " // podId:: " +
-                    podId + " // clusterId:: " + clusterId + " // message:: " 
+ subject);
+            s_alertsLogger.warn("AlertType:: " + alertType + " | 
dataCenterId:: " + dataCenterId + " | podId:: " +
+                    podId + " | clusterId:: " + clusterId + " | message:: " + 
subject);
             AlertVO alert = null;
             if ((alertType != AlertManager.AlertType.ALERT_TYPE_HOST) &&
                 (alertType != AlertManager.AlertType.ALERT_TYPE_USERVM) &&
@@ -771,7 +771,7 @@ public class AlertManagerImpl extends ManagerBase 
implements AlertManager, Confi
                 newAlert.setClusterId(clusterId);
                 newAlert.setPodId(podId);
                 newAlert.setDataCenterId(dataCenterId);
-                newAlert.setSentCount(1);// initialize sent count to 1 since 
we are now sending an alert
+                newAlert.setSentCount(1); // Initialize sent count to 1 since 
we are now sending an alert.
                 newAlert.setLastSent(new Date());
                 newAlert.setName(alertType.getName());
                 _alertDao.persist(newAlert);
diff --git 
a/server/src/com/cloud/network/ExternalLoadBalancerDeviceManagerImpl.java 
b/server/src/com/cloud/network/ExternalLoadBalancerDeviceManagerImpl.java
index f1b4a79..1e6271d 100644
--- a/server/src/com/cloud/network/ExternalLoadBalancerDeviceManagerImpl.java
+++ b/server/src/com/cloud/network/ExternalLoadBalancerDeviceManagerImpl.java
@@ -1281,14 +1281,11 @@ public abstract class 
ExternalLoadBalancerDeviceManagerImpl extends AdapterBase
             if (loadBalancersToApply.size() > 0) {
                 int numLoadBalancersForCommand = loadBalancersToApply.size();
                 LoadBalancerTO[] loadBalancersForCommand = 
loadBalancersToApply.toArray(new LoadBalancerTO[numLoadBalancersForCommand]);
-                // LoadBalancerConfigCommand cmd = new
-                // LoadBalancerConfigCommand(loadBalancersForCommand, null);
                 HealthCheckLBConfigCommand cmd = new 
HealthCheckLBConfigCommand(loadBalancersForCommand, network.getId());
                 long guestVlanTag = 
Integer.parseInt(BroadcastDomainType.getValue(network.getBroadcastUri()));
                 cmd.setAccessDetail(NetworkElementCommand.GUEST_VLAN_TAG, 
String.valueOf(guestVlanTag));
 
-                HealthCheckLBConfigAnswer answer = (HealthCheckLBConfigAnswer) 
_agentMgr
-                        .easySend(externalLoadBalancer.getId(), cmd);
+                HealthCheckLBConfigAnswer answer = (HealthCheckLBConfigAnswer) 
_agentMgr.easySend(externalLoadBalancer.getId(), cmd);
                 // easySend will return null on error
                 return answer == null ? null : answer.getLoadBalancers();
             }
diff --git a/server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java 
b/server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java
index 099a274..9b7cf9b 100644
--- a/server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java
+++ b/server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java
@@ -2125,7 +2125,7 @@ public class LoadBalancingRulesManagerImpl<Type> extends 
ManagerBase implements
         boolean success = _lbDao.update(lbRuleId, lb);
 
         // If algorithm is changed, have to reapply the lb config
-        if ((algorithm!= null) && 
(tmplbVo.getAlgorithm().compareTo(algorithm)!=0)){
+        if ((algorithm != null) && 
(tmplbVo.getAlgorithm().compareTo(algorithm) != 0)){
             try {
                 lb.setState(FirewallRule.State.Add);
                 _lbDao.persist(lb);
diff --git a/server/src/org/apache/cloudstack/network/ssl/CertServiceImpl.java 
b/server/src/org/apache/cloudstack/network/ssl/CertServiceImpl.java
index 68e84fb..e142ee5 100644
--- a/server/src/org/apache/cloudstack/network/ssl/CertServiceImpl.java
+++ b/server/src/org/apache/cloudstack/network/ssl/CertServiceImpl.java
@@ -328,8 +328,9 @@ public class CertServiceImpl implements CertService {
         response.setFingerprint(cert.getFingerPrint());
         response.setName(cert.getName());
 
-        if (cert.getChain() != null)
+        if (cert.getChain() != null) {
             response.setCertchain(cert.getChain());
+        }
 
         if (lbCertMap != null && !lbCertMap.isEmpty()) {
             final List<String> lbIds = new ArrayList<String>();
diff --git a/server/test/com/cloud/vpc/dao/MockNetworkDaoImpl.java 
b/server/test/com/cloud/vpc/dao/MockNetworkDaoImpl.java
index b0440f2..4c4a0a1 100644
--- a/server/test/com/cloud/vpc/dao/MockNetworkDaoImpl.java
+++ b/server/test/com/cloud/vpc/dao/MockNetworkDaoImpl.java
@@ -34,12 +34,8 @@ import com.cloud.utils.db.SearchBuilder;
 @DB()
 public class MockNetworkDaoImpl extends GenericDaoBase<NetworkVO, Long> 
implements NetworkDao {
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listByOwner(long)
-     */
     @Override
     public List<NetworkVO> listByOwner(final long ownerId) {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -48,279 +44,151 @@ public class MockNetworkDaoImpl extends 
GenericDaoBase<NetworkVO, Long> implemen
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listBy(long, long, long)
-     */
     @Override
     public List<NetworkVO> listBy(final long accountId, final long offeringId, 
final long dataCenterId) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listBy(long, long, 
java.lang.String, boolean)
-     */
     @Override
     public List<NetworkVO> listBy(final long accountId, final long 
dataCenterId, final String cidr, final boolean skipVpc) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listByZoneAndGuestType(long, 
long, com.cloud.network.Network.GuestType, java.lang.Boolean)
-     */
     @Override
     public List<NetworkVO> listByZoneAndGuestType(final long accountId, final 
long dataCenterId, final GuestType type, final Boolean isSystem) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see 
com.cloud.network.dao.NetworkDao#persist(com.cloud.network.NetworkVO, boolean, 
java.util.Map)
-     */
     @Override
     public NetworkVO persist(final NetworkVO network, final boolean gc, final 
Map<String, String> serviceProviderMap) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#createSearchBuilderForAccount()
-     */
     @Override
     public SearchBuilder<NetworkAccountVO> createSearchBuilderForAccount() {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#getNetworksForOffering(long, 
long, long)
-     */
     @Override
     public List<NetworkVO> getNetworksForOffering(final long offeringId, final 
long dataCenterId, final long accountId) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#getNextAvailableMacAddress(long)
-     */
     @Override
     public String getNextAvailableMacAddress(final long networkConfigId, 
Integer zoneMacIdentifier) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listBy(long, long)
-     */
     @Override
     public List<NetworkVO> listBy(final long accountId, final long networkId) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#countByZoneAndUri(long, 
java.lang.String)
-     */
     @Override
     public long countByZoneAndUri(final long zoneId, final String 
broadcastUri) {
-        // TODO Auto-generated method stub
         return 0;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#countByZoneUriAndGuestType(long, 
java.lang.String, com.cloud.network.Network.GuestType)
-     */
     @Override
     public long countByZoneUriAndGuestType(final long zoneId, final String 
broadcastUri, final GuestType guestType) {
-        // TODO Auto-generated method stub
         return 0;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listByZone(long)
-     */
     @Override
     public List<NetworkVO> listByZone(final long zoneId) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#changeActiveNicsBy(long, int)
-     */
     @Override
     public void changeActiveNicsBy(final long networkId, final int nicsCount) {
-        // TODO Auto-generated method stub
-
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#getActiveNicsIn(long)
-     */
     @Override
     public int getActiveNicsIn(final long networkId) {
-        // TODO Auto-generated method stub
         return 0;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#findNetworksToGarbageCollect()
-     */
     @Override
     public List<Long> findNetworksToGarbageCollect() {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#clearCheckForGc(long)
-     */
     @Override
     public void clearCheckForGc(final long networkId) {
-        // TODO Auto-generated method stub
-
     }
 
-    /* (non-Javadoc)
-     * @see 
com.cloud.network.dao.NetworkDao#listByZoneSecurityGroup(java.lang.Long)
-     */
     @Override
     public List<NetworkVO> listByZoneSecurityGroup(final Long zoneId) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#addDomainToNetwork(long, long, 
java.lang.Boolean)
-     */
     @Override
     public void addDomainToNetwork(final long networkId, final long domainId, 
final Boolean subdomainAccess) {
-        // TODO Auto-generated method stub
-
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listByPhysicalNetwork(long)
-     */
     @Override
     public List<NetworkVO> listByPhysicalNetwork(final long physicalNetworkId) 
{
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listSecurityGroupEnabledNetworks()
-     */
     @Override
     public List<NetworkVO> listSecurityGroupEnabledNetworks() {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see 
com.cloud.network.dao.NetworkDao#listByPhysicalNetworkTrafficType(long, 
com.cloud.network.Networks.TrafficType)
-     */
     @Override
     public List<NetworkVO> listByPhysicalNetworkTrafficType(final long 
physicalNetworkId, final TrafficType trafficType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listBy(long, long, 
com.cloud.network.Network.GuestType, com.cloud.network.Networks.TrafficType)
-     */
     @Override
     public List<NetworkVO> listBy(final long accountId, final long 
dataCenterId, final GuestType type, final TrafficType trafficType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see 
com.cloud.network.dao.NetworkDao#listByPhysicalNetworkAndProvider(long, 
java.lang.String)
-     */
     @Override
     public List<NetworkVO> listByPhysicalNetworkAndProvider(final long 
physicalNetworkId, final String providerName) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see 
com.cloud.network.dao.NetworkDao#persistNetworkServiceProviders(long, 
java.util.Map)
-     */
     @Override
     public void persistNetworkServiceProviders(final long networkId, final 
Map<String, String> serviceProviderMap) {
-        // TODO Auto-generated method stub
-
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#update(java.lang.Long, 
com.cloud.network.NetworkVO, java.util.Map)
-     */
     @Override
     public boolean update(final Long networkId, final NetworkVO network, final 
Map<String, String> serviceProviderMap) {
-        // TODO Auto-generated method stub
         return false;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listByZoneAndTrafficType(long, 
com.cloud.network.Networks.TrafficType)
-     */
     @Override
     public List<NetworkVO> listByZoneAndTrafficType(final long zoneId, final 
TrafficType trafficType) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#setCheckForGc(long)
-     */
     @Override
     public void setCheckForGc(final long networkId) {
-        // TODO Auto-generated method stub
-
     }
 
-    /* (non-Javadoc)
-     * @see 
com.cloud.network.dao.NetworkDao#getNetworkCountByNetworkOffId(long)
-     */
     @Override
     public int getNetworkCountByNetworkOffId(final long networkOfferingId) {
-        // TODO Auto-generated method stub
         return 0;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#countNetworksUserCanCreate(long)
-     */
     @Override
     public long countNetworksUserCanCreate(final long ownerId) {
-        // TODO Auto-generated method stub
         return 0;
     }
 
-    /* (non-Javadoc)
-     * @see 
com.cloud.network.dao.NetworkDao#listSourceNATEnabledNetworks(long, long, 
com.cloud.network.Network.GuestType)
-     */
     @Override
     public List<NetworkVO> listSourceNATEnabledNetworks(final long accountId, 
final long dataCenterId, final GuestType type) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#getNetworkCountByVpcId(long)
-     */
     @Override
     public int getNetworkCountByVpcId(final long vpcId) {
-        // TODO Auto-generated method stub
         return 0;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listByVpc(long)
-     */
     @Override
     public List<NetworkVO> listByVpc(final long vpcId) {
         final List<NetworkVO> networks = new ArrayList<NetworkVO>();
@@ -328,21 +196,13 @@ public class MockNetworkDaoImpl extends 
GenericDaoBase<NetworkVO, Long> implemen
         return networks;
     }
 
-    /* (non-Javadoc)
-     * @see 
com.cloud.network.dao.NetworkDao#getPrivateNetwork(java.lang.String, 
java.lang.String, long, long)
-     */
     @Override
     public NetworkVO getPrivateNetwork(final String broadcastUri, final String 
cidr, final long accountId, final long zoneId, final Long netofferid) {
-        // TODO Auto-generated method stub
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#countVpcNetworks(long)
-     */
     @Override
     public long countVpcNetworks(final long vpcId) {
-        // TODO Auto-generated method stub
         return 0;
     }
 
@@ -351,24 +211,18 @@ public class MockNetworkDaoImpl extends 
GenericDaoBase<NetworkVO, Long> implemen
         return true;
     }
 
-    /* (non-Javadoc)
-     * @see com.cloud.network.dao.NetworkDao#listNetworksByAccount(long, long, 
com.cloud.network.Network.GuestType, boolean)
-     */
     @Override
     public List<NetworkVO> listNetworksByAccount(final long accountId, final 
long zoneId, final GuestType type, final boolean isSystem) {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public List<NetworkVO> listRedundantNetworks() {
-        // TODO Auto-generated method stub
         return null;
     }
 
     @Override
     public List<NetworkVO> listVpcNetworks() {
-        // TODO Auto-generated method stub
         return null;
     }
 
@@ -384,7 +238,6 @@ public class MockNetworkDaoImpl extends 
GenericDaoBase<NetworkVO, Long> implemen
 
     @Override
     public List<NetworkVO> listNetworkVO(List<Long> idset) {
-        // TODO Auto-generated method stub
         return null;
     }
-}
+}
\ No newline at end of file

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

Reply via email to