[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8832?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14953330#comment-14953330
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8832:
--------------------------------------------

Github user jburwell commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/801#discussion_r41776195
  
    --- Diff: 
plugins/network-elements/nuage-vsp/src/com/cloud/agent/api/guru/ImplementNetworkVspCommand.java
 ---
    @@ -112,13 +147,212 @@ public boolean isDefaultEgressPolicy() {
             return _defaultEgressPolicy;
         }
     
    -    public Collection<String> getIpAddressRange() {
    +    public Collection<String[]> getIpAddressRange() {
             return _ipAddressRange;
         }
     
    +    public String getDomainTemplateName() {
    +        return _domainTemplateName;
    +    }
    +
    +    public static class Builder extends 
CmdBuilder<ImplementNetworkVspCommand> {
    +        private String _networkDomainName;
    +        private String _networkDomainPath;
    +        private String _networkDomainUuid;
    +        private String _networkAccountName;
    +        private String _networkAccountUuid;
    +        private String _networkName;
    +        private String _networkCidr;
    +        private String _networkGateway;
    +        private Long _networkAclId;
    +        private List<String> _dnsServers;
    +        private List<String> _gatewaySystemIds;
    +        private String _networkUuid;
    +        private boolean _isL3Network;
    +        private boolean _isVpc;
    +        private boolean _isSharedNetwork;
    +        private String _vpcName;
    +        private String _vpcUuid;
    +        private boolean _defaultEgressPolicy;
    +        private List<String[]> _ipAddressRange;
    +        private String _domainTemplateName;
    +
    +        public Builder networkDomainName(String networkDomainName) {
    +            this._networkDomainName = networkDomainName;
    +            return this;
    +        }
    +
    +        public Builder networkDomainPath(String networkDomainPath) {
    +            this._networkDomainPath = networkDomainPath;
    +            return this;
    +        }
    +
    +        public Builder networkDomainUuid(String networkDomainUuid) {
    +            this._networkDomainUuid = networkDomainUuid;
    +            return this;
    +        }
    +
    +        public Builder networkAccountName(String networkAccountName) {
    +            this._networkAccountName = networkAccountName;
    +            return this;
    +        }
    +
    +        public Builder networkAccountUuid(String networkAccountUuid) {
    +            this._networkAccountUuid = networkAccountUuid;
    +            return this;
    +        }
    +
    +        public Builder networkName(String networkName) {
    +            this._networkName = networkName;
    +            return this;
    +        }
    +
    +        public Builder networkCidr(String networkCidr) {
    +            this._networkCidr = networkCidr;
    +            return this;
    +        }
    +
    +        public Builder networkGateway(String networkGateway) {
    +            this._networkGateway = networkGateway;
    +            return this;
    +        }
    +
    +        public Builder networkAclId(Long networkAclId) {
    +            this._networkAclId = networkAclId;
    +            return this;
    +        }
    +
    +        public Builder dnsServers(List<String> dnsServers) {
    +            this._dnsServers = dnsServers;
    +            return this;
    +        }
    +
    +        public Builder gatewaySystemIds(List<String> gatewaySystemIds) {
    +            this._gatewaySystemIds = gatewaySystemIds;
    +            return this;
    +        }
    +
    +        public Builder networkUuid(String networkUuid) {
    +            this._networkUuid = networkUuid;
    +            return this;
    +        }
    +
    +        public Builder isL3Network(boolean isL3Network) {
    +            this._isL3Network = isL3Network;
    +            return this;
    +        }
    +
    +        public Builder isVpc(boolean isVpc) {
    +            this._isVpc = isVpc;
    +            return this;
    +        }
    +
    +        public Builder isSharedNetwork(boolean isSharedNetwork) {
    +            this._isSharedNetwork = isSharedNetwork;
    +            return this;
    +        }
    +
    +        public Builder vpcName(String vpcName) {
    +            this._vpcName = vpcName;
    +            return this;
    +        }
    +
    +        public Builder vpcUuid(String vpcUuid) {
    +            this._vpcUuid = vpcUuid;
    +            return this;
    +        }
    +
    +        public Builder defaultEgressPolicy(boolean defaultEgressPolicy) {
    +            this._defaultEgressPolicy = defaultEgressPolicy;
    +            return this;
    +        }
    +
    +        public Builder ipAddressRange(List<String[]> ipAddressRange) {
    +            this._ipAddressRange = ipAddressRange;
    +            return this;
    +        }
    +
    +        public Builder domainTemplateName(String domainTemplateName) {
    +            this._domainTemplateName = domainTemplateName;
    +            return this;
    +        }
    +
    +        @Override
    +        public ImplementNetworkVspCommand build() {
    +            return new ImplementNetworkVspCommand(_networkDomainName, 
_networkDomainPath, _networkDomainUuid, _networkAccountName, 
_networkAccountUuid, _networkName,
    +                    _networkCidr, _networkGateway, _networkAclId, 
_dnsServers, _gatewaySystemIds, _networkUuid, _isL3Network, _isVpc, 
_isSharedNetwork, _vpcName, _vpcUuid,
    +                    _defaultEgressPolicy, _ipAddressRange, 
_domainTemplateName);
    +        }
    +    }
    +
         @Override
         public boolean executeInSequence() {
             return false;
         }
     
    +    @Override
    +    public boolean equals(Object o) {
    +        if (this == o) return true;
    +        if (o == null || getClass() != o.getClass()) return false;
    +
    +        ImplementNetworkVspCommand that = (ImplementNetworkVspCommand) o;
    +
    +        if (_defaultEgressPolicy != that._defaultEgressPolicy) return 
false;
    +        if (_isL3Network != that._isL3Network) return false;
    +        if (_isSharedNetwork != that._isSharedNetwork) return false;
    +        if (_isVpc != that._isVpc) return false;
    +        if (_dnsServers != null ? !_dnsServers.equals(that._dnsServers) : 
that._dnsServers != null) return false;
    +        if (_domainTemplateName != null ? 
!_domainTemplateName.equals(that._domainTemplateName) : 
that._domainTemplateName != null)
    +            return false;
    +        if (_gatewaySystemIds != null ? 
!_gatewaySystemIds.equals(that._gatewaySystemIds) : that._gatewaySystemIds != 
null)
    +            return false;
    +        if (_ipAddressRange != null ? 
!_ipAddressRange.equals(that._ipAddressRange) : that._ipAddressRange != null)
    +            return false;
    +        if (_networkAccountName != null ? 
!_networkAccountName.equals(that._networkAccountName) : 
that._networkAccountName != null)
    +            return false;
    +        if (_networkAccountUuid != null ? 
!_networkAccountUuid.equals(that._networkAccountUuid) : 
that._networkAccountUuid != null)
    +            return false;
    +        if (_networkAclId != null ? 
!_networkAclId.equals(that._networkAclId) : that._networkAclId != null)
    +            return false;
    +        if (_networkCidr != null ? !_networkCidr.equals(that._networkCidr) 
: that._networkCidr != null) return false;
    +        if (_networkDomainName != null ? 
!_networkDomainName.equals(that._networkDomainName) : that._networkDomainName 
!= null)
    +            return false;
    +        if (_networkDomainPath != null ? 
!_networkDomainPath.equals(that._networkDomainPath) : that._networkDomainPath 
!= null)
    +            return false;
    +        if (_networkDomainUuid != null ? 
!_networkDomainUuid.equals(that._networkDomainUuid) : that._networkDomainUuid 
!= null)
    +            return false;
    +        if (_networkGateway != null ? 
!_networkGateway.equals(that._networkGateway) : that._networkGateway != null)
    +            return false;
    +        if (_networkName != null ? !_networkName.equals(that._networkName) 
: that._networkName != null) return false;
    +        if (_networkUuid != null ? !_networkUuid.equals(that._networkUuid) 
: that._networkUuid != null) return false;
    +        if (_vpcName != null ? !_vpcName.equals(that._vpcName) : 
that._vpcName != null) return false;
    +        if (_vpcUuid != null ? !_vpcUuid.equals(that._vpcUuid) : 
that._vpcUuid != null) return false;
    +
    +        return true;
    +    }
    +
    +    @Override
    +    public int hashCode() {
    +        int result = _networkDomainName != null ? 
_networkDomainName.hashCode() : 0;
    +        result = 31 * result + (_networkDomainPath != null ? 
_networkDomainPath.hashCode() : 0);
    +        result = 31 * result + (_networkDomainUuid != null ? 
_networkDomainUuid.hashCode() : 0);
    +        result = 31 * result + (_networkAccountName != null ? 
_networkAccountName.hashCode() : 0);
    +        result = 31 * result + (_networkAccountUuid != null ? 
_networkAccountUuid.hashCode() : 0);
    +        result = 31 * result + (_networkName != null ? 
_networkName.hashCode() : 0);
    +        result = 31 * result + (_networkCidr != null ? 
_networkCidr.hashCode() : 0);
    +        result = 31 * result + (_networkGateway != null ? 
_networkGateway.hashCode() : 0);
    +        result = 31 * result + (_networkAclId != null ? 
_networkAclId.hashCode() : 0);
    +        result = 31 * result + (_dnsServers != null ? 
_dnsServers.hashCode() : 0);
    +        result = 31 * result + (_gatewaySystemIds != null ? 
_gatewaySystemIds.hashCode() : 0);
    +        result = 31 * result + (_networkUuid != null ? 
_networkUuid.hashCode() : 0);
    +        result = 31 * result + (_isL3Network ? 1 : 0);
    +        result = 31 * result + (_isVpc ? 1 : 0);
    +        result = 31 * result + (_isSharedNetwork ? 1 : 0);
    +        result = 31 * result + (_vpcName != null ? _vpcName.hashCode() : 
0);
    +        result = 31 * result + (_vpcUuid != null ? _vpcUuid.hashCode() : 
0);
    +        result = 31 * result + (_defaultEgressPolicy ? 1 : 0);
    +        result = 31 * result + (_ipAddressRange != null ? 
_ipAddressRange.hashCode() : 0);
    +        result = 31 * result + (_domainTemplateName != null ? 
_domainTemplateName.hashCode() : 0);
    +        return result;
    +    }
    --- End diff --
    
    Please implement ``toString()`` to provide debugging state information for 
debugging purposes.


> Update Nuage VSP plugin to work with Nuage VSP release 3.2
> ----------------------------------------------------------
>
>                 Key: CLOUDSTACK-8832
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8832
>             Project: CloudStack
>          Issue Type: Improvement
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: Management Server
>    Affects Versions: 4.6.0
>            Reporter: Nick Livens
>            Assignee: Nick Livens
>         Attachments: nuageVspMarvinLogs.tar.gz
>
>
> Nuage VSP 3.2 is being released, we want to bring the plugin up to date for 
> this release



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to