----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21973/ -----------------------------------------------------------
(Updated May 28, 2014, 9:38 a.m.) Review request for cloudstack and Girish Shilamkar. Changes ------- Updated patch. Bugs: CLOUDSTACK-6780 https://issues.apache.org/jira/browse/CLOUDSTACK-6780 Repository: cloudstack-git Description ------- Portable IP address was not getting disassociated. Added it to finally block so that it will get associated always and portable ip range gets cleaned up properly during cleanup. Diffs (updated) ----- test/integration/component/test_portable_ip.py 847bb4a Diff: https://reviews.apache.org/r/21973/diff/ Testing ------- Test disassociating portable ip ... === TestName: test_disassociate_ip_address_no_services | Status : SUCCESS === ok Test disassociating portable IP with non-owner account ... === TestName: test_disassociate_ip_address_other_account | Status : SUCCESS === ok Test disassociating portable ip ... === TestName: test_disassociate_ip_address_services_enabled | Status : SUCCESS === ok ---------------------------------------------------------------------- Ran 3 tests in 263.215s OK Thanks, Ashutosh Kelkar