cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Antonio Fornie" <antonio.for...@gmail.com>
Subject Re: Review Request 18213: Fix and add some Rest unit and IT tests. Remove duplication of site pugin declaration. Fix by adding a missing type of RoutingConfig.
Date Tue, 18 Feb 2014 16:09:13 GMT

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

(Updated Feb. 18, 2014, 4:09 p.m.)


Review request for cloudstack, daan Hoogland and Hugo Trippaers.


Changes
-------

Can you check that this time there is no collision with Hugo's change on root pom (removal
of site plugin duplication)?


Repository: cloudstack-git


Description
-------

Fix and add some Rest unit and IT tests. Remove duplication of site pugin
declaration. Fix by adding a missing type of RoutingConfig.


Diffs (updated)
-----

  plugins/network-elements/nicira-nvp/src/com/cloud/network/nicira/NiciraNvpApi.java 3861d63

  plugins/network-elements/nicira-nvp/src/com/cloud/network/nicira/RoutingTableRoutingConfig.java
PRE-CREATION 
  plugins/network-elements/nicira-nvp/test/com/cloud/network/nicira/NiciraNvpApiTest.java
fe41545 
  utils/test/com/cloud/utils/rest/RESTServiceConnectorTest.java 17864b4 

Diff: https://reviews.apache.org/r/18213/diff/


Testing
-------

Full maven lifecycle with unit and integration tests. Execute most of the functionality against
our Ncira server.


Thanks,

Antonio Fornie


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