cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ritu Sabharwal" <rsabh...@brocade.com>
Subject Re: Review Request 23314: Plugin specific code for the Brocade Network Plugin
Date Tue, 15 Jul 2014 22:06:42 GMT


> On July 15, 2014, 11:21 a.m., Hugo Trippaers wrote:
> > This is looking good, when you addresses the comment on the system.js file in the
other review i'll apply both patches. If the patches apply clean i'll run findbugs and cobertura
to check for any issues and unit test coverage. 
> > 
> > Hugo

Hi Hugo,

I have fixed the system.js file. Also, removed the configuration reading from properties files.
Now the plugin configurations are read as network service provider configurations from GUI
or APIs. The unit tests are also updated for the change.

Please review it and do the needful so that we can make it by July 19.

Thanks & Regards,
Ritu S.


> On July 15, 2014, 11:21 a.m., Hugo Trippaers wrote:
> > plugins/network-elements/brocade-vcs/resources/BrocadeInterfaceSchema.xsd, line
5
> > <https://reviews.apache.org/r/23314/diff/2-3/?file=624832#file624832line5>
> >
> >     Did you check with the rat plugin if the license is accepted like this?
> >     
> >     I think the license is only properly detected with it is located in a comment
like in the spring xml files.

The standard way to add comments to schema files is using annotation element. Reference: http://www.w3schools.com/schema/el_annotation.asp

The build throws parsing error if the comments are given like spring xml files. With annotation
tag , the compilation goes fine and JAXB generation also works fine.

I did not find any xsd files in CloudStack codebase to get an example. What is the rat plugin?


- Ritu


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23314/#review47769
-----------------------------------------------------------


On July 15, 2014, 9:52 p.m., Ritu  Sabharwal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23314/
> -----------------------------------------------------------
> 
> (Updated July 15, 2014, 9:52 p.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Bugs: CLOUDSTACK-6823
>     https://issues.apache.org/jira/browse/CLOUDSTACK-6823
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Plugin specific code.
> 
> 
> Diffs
> -----
> 
>   plugins/network-elements/brocade-vcs/pom.xml PRE-CREATION 
>   plugins/network-elements/brocade-vcs/resources/BrocadeInterfaceSchema.xsd PRE-CREATION

>   plugins/network-elements/brocade-vcs/resources/BrocadePortProfileSchema.xsd PRE-CREATION

>   plugins/network-elements/brocade-vcs/resources/BrocadeShowVcsSchema.xsd PRE-CREATION

>   plugins/network-elements/brocade-vcs/resources/META-INF/cloudstack/vcs/module.properties
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/resources/META-INF/cloudstack/vcs/spring-vcs-context.xml
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/AssociateMacToNetworkAnswer.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/AssociateMacToNetworkCommand.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/CreateNetworkAnswer.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/CreateNetworkCommand.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/DeleteNetworkAnswer.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/DeleteNetworkCommand.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/DisassociateMacFromNetworkAnswer.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/DisassociateMacFromNetworkCommand.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/StartupBrocadeVcsCommand.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/api/commands/AddBrocadeVcsDeviceCmd.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/api/commands/DeleteBrocadeVcsDeviceCmd.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/api/commands/ListBrocadeVcsDeviceNetworksCmd.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/api/commands/ListBrocadeVcsDevicesCmd.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/api/response/BrocadeVcsDeviceResponse.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/BrocadeVcsDeviceVO.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/BrocadeVcsNetworkVlanMappingVO.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/brocade/BrocadeVcsApi.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/brocade/BrocadeVcsApiException.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/brocade/Constants.java PRE-CREATION

>   plugins/network-elements/brocade-vcs/src/com/cloud/network/dao/BrocadeVcsDao.java PRE-CREATION

>   plugins/network-elements/brocade-vcs/src/com/cloud/network/dao/BrocadeVcsDaoImpl.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/dao/BrocadeVcsNetworkVlanMappingDao.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/dao/BrocadeVcsNetworkVlanMappingDaoImpl.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/element/BrocadeVcsElement.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/element/BrocadeVcsElementService.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/guru/BrocadeVcsGuestNetworkGuru.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/src/com/cloud/network/resource/BrocadeVcsResource.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/test/com/cloud/network/brocade/BrocadeVcsApiTest.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/test/com/cloud/network/guru/BrocadeVcsGuestNetworkGuruTest.java
PRE-CREATION 
>   plugins/network-elements/brocade-vcs/test/com/cloud/network/resource/BrocadeVcsResourceTest.java
PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/23314/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ritu  Sabharwal
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message