You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Rajani Karuturi <ra...@gmail.com> on 2014/05/27 17:25:05 UTC

Review Request 21927: Fixed integration.smoke.test_portable_publicip.TestPortablePublicIPRange.test_createPortablePublicIPRange

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

Review request for cloudstack, Abhinandan Prateek and Murali Reddy.


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


Repository: cloudstack-git


Description
-------

there are multiple @Id columns in the ipaddressvo which was causing a wrong remove query and the test case failed.


Diffs
-----

  engine/schema/src/com/cloud/network/dao/IPAddressVO.java 816e063 

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


Testing
-------

ran the following
$nosetests --with-marvin --marvin-config=setup/dev/advanced.cfg -a tags=simulator test/integration/smoke/test_portable_publicip.py

output:
Test to acquire a provisioned public ip range ... === TestName: test_createPortablePublicIPAcquire | Status : SUCCESS ===
ok
Test to create a portable public ip range ... === TestName: test_createPortablePublicIPRange | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 2 tests in 49.172s

OK


Thanks,

Rajani Karuturi