cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "sanjeev n" <sanjeev.neelar...@citrix.com>
Subject Re: Review Request 30157: CLOUDSTACK-8174: Fixed network delete issue in test_add_remove_network.py
Date Thu, 22 Jan 2015 08:49:46 GMT


> On Jan. 22, 2015, 8:49 a.m., sanjeev n wrote:
> > Ship It!

commit 8894fc897b9e6d20efda6f18f7e9178fa63e87a6
Author: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
Date:   Wed Jan 21 22:31:24 2015 -0800

    CLOUDSTACK-8174: Fixed cleanup issue in test_add_remove_network.py


- sanjeev


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


On Jan. 22, 2015, 7:03 a.m., Gaurav Aradhye wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30157/
> -----------------------------------------------------------
> 
> (Updated Jan. 22, 2015, 7:03 a.m.)
> 
> 
> Review request for cloudstack, sanjeev n and SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-8174
>     https://issues.apache.org/jira/browse/CLOUDSTACK-8174
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> The test case failed while cleaning up the network.
> The network will be cleaned up as part of account cleanup, no need to add it separately
to the cleanup list.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_add_remove_network.py 2f685b5 
> 
> Diff: https://reviews.apache.org/r/30157/diff/
> 
> 
> Testing
> -------
> 
> Yes.
> 
> LOg:
> Add network to vm with wrong vm id ... === TestName: test_15_add_nic_wrong_vm_id | Status
: SUCCESS ===
> ok
> Add network to vm with wrong network id ... === TestName: test_16_add_nic_wrong_network_id
| Status : SUCCESS ===
> ok
> Add network to vm where both belong to different zones ... === TestName: test_17_add_nic_different_zone
| Status : SUCCESS ===
> ok
> Add network to vm in basic zone ... SKIP: This test requires at least one basic zone
to be present in the setup
> Try to add network to vm with insufficient permission ... === TestName: test_26_add_nic_insufficient_permission
| Status : SUCCESS ===
> ok
> 
> ----------------------------------------------------------------------
> Ran 5 tests in 285.485s
> 
> OK (SKIP=1)
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>


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