You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Ashutosh Kelkar <as...@clogeny.com> on 2014/12/15 11:32:03 UTC

Review Request 29046: CLOUDSTACK-8071: Fixed api key issue in test_snapshots_improvement.py

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

Review request for cloudstack and SrikanteswaraRao Talluri.


Bugs: CLOUDSTACK-8071
    https://issues.apache.org/jira/browse/CLOUDSTACK-8071


Repository: cloudstack-git


Description
-------

The test case failed with "Api key does not map with valid account".
That is because the apiclient object was assigned previous account's value in another test case.
It is important that all test cases use different account and their respective api clients.
Made appropriate changes.


Diffs
-----

  test/integration/component/test_snapshots_improvement.py 6638ddd 

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


Testing
-------

Test perform concurrent snapshots and migrate the vm from one host ... === TestName: test_01_concurrent_snapshots_live_migrate | Status : SUCCESS ===
ok
Test stop running VM while performing concurrent snapshot on volume ... === TestName: test_02_stop_vm_concurrent_snapshots | Status : SUCCESS ===
ok
Test while parent concurrent snapshot job in progress,create ... === TestName: test_03_concurrent_snapshots_create_template | Status : SUCCESS ===
ok
Test while parent concurrent snapshot job in progress,create volume ... === TestName: test_04_concurrent_snapshots_create_volume | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 4 tests in 1132.201s

OK


Thanks,

Ashutosh Kelkar


Re: Review Request 29046: CLOUDSTACK-8071: Fixed api key issue in test_snapshots_improvement.py

Posted by SrikanteswaraRao Talluri <sr...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29046/#review65315
-----------------------------------------------------------

Ship it!


ed5bc1c7ea1885e95c7cad0045243c7f43e9f516 4.5
391108ff533bdaa40356538ccf04a77498398ad9 master

- SrikanteswaraRao Talluri


On Dec. 15, 2014, 10:32 a.m., Ashutosh Kelkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29046/
> -----------------------------------------------------------
> 
> (Updated Dec. 15, 2014, 10:32 a.m.)
> 
> 
> Review request for cloudstack and SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-8071
>     https://issues.apache.org/jira/browse/CLOUDSTACK-8071
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> The test case failed with "Api key does not map with valid account".
> That is because the apiclient object was assigned previous account's value in another test case.
> It is important that all test cases use different account and their respective api clients.
> Made appropriate changes.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_snapshots_improvement.py 6638ddd 
> 
> Diff: https://reviews.apache.org/r/29046/diff/
> 
> 
> Testing
> -------
> 
> Test perform concurrent snapshots and migrate the vm from one host ... === TestName: test_01_concurrent_snapshots_live_migrate | Status : SUCCESS ===
> ok
> Test stop running VM while performing concurrent snapshot on volume ... === TestName: test_02_stop_vm_concurrent_snapshots | Status : SUCCESS ===
> ok
> Test while parent concurrent snapshot job in progress,create ... === TestName: test_03_concurrent_snapshots_create_template | Status : SUCCESS ===
> ok
> Test while parent concurrent snapshot job in progress,create volume ... === TestName: test_04_concurrent_snapshots_create_volume | Status : SUCCESS ===
> ok
> 
> ----------------------------------------------------------------------
> Ran 4 tests in 1132.201s
> 
> OK
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>