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

    https://github.com/apache/cloudstack/pull/801#discussion_r43031009
  
    --- Diff: 
plugins/network-elements/nuage-vsp/src/com/cloud/agent/api/element/ShutDownVpcVspCommand.java
 ---
    @@ -40,9 +43,61 @@ public String getVpcUuid() {
             return _vpcUuid;
         }
     
    +    public String getDomainTemplateName() {
    +        return _domainTemplateName;
    +    }
    +
    +    public static class Builder implements 
CmdBuilder<ShutDownVpcVspCommand> {
    +        private String _domainUuid;
    +        private String _vpcUuid;
    +        private String _domainTemplateName;
    +
    +        public Builder domainUuid(String domainUuid) {
    +            this._domainUuid = domainUuid;
    +            return this;
    +        }
    +
    +        public Builder vpcUuid(String vpcUuid) {
    +            this._vpcUuid = vpcUuid;
    +            return this;
    +        }
    +
    +        public Builder domainTemplateName(String domainTemplateName) {
    +            this._domainTemplateName = domainTemplateName;
    +            return this;
    +        }
    +
    +        @Override
    +        public ShutDownVpcVspCommand build() {
    +            return new ShutDownVpcVspCommand(_domainUuid, _vpcUuid, 
_domainTemplateName);
    +        }
    +    }
    +
         @Override
         public boolean executeInSequence() {
             return false;
         }
     
    +    @Override
    +    public boolean equals(Object o) {
    --- End diff --
    
    This ``equals`` implementation should include attributes from the 
``Command`` super class.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to