----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12942/#review24127 -----------------------------------------------------------
Commit 8be9a47542e0ae0f481733ac4ddb6476d0e20a18 in branch refs/heads/4.2 from Girish Shilamkar [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=8be9a47 ] CLOUDSTACK-3454: Fix test_portable_publicip Added isportable param to associateIP API. Fixed base class for PortableIP tio call portableip APIs. Removed test_createPortablePublicIPAcquire from basic zone run requires additional network creation handling which can be done in component tests. Signed-off-by: Prasanna Santhanam <t...@apache.org> (cherry picked from commit 4054a8e2ac91eee6cbc0c63094016068bdbe62b8) - ASF Subversion and Git Services On July 25, 2013, 12:50 p.m., Girish Shilamkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12942/ > ----------------------------------------------------------- > > (Updated July 25, 2013, 12:50 p.m.) > > > Review request for cloudstack, Murali Reddy and Prasanna Santhanam. > > > Bugs: CLOUDSTACK-3454 > > > Repository: cloudstack-git > > > Description > ------- > > Added isportable param to associateIP API. Fixed base class > for PortableIP tio call portableip APIs. > Removed test_createPortablePublicIPAcquire from basic zone run > requires additional network creation handling which can be done > in component tests. > > > Diffs > ----- > > test/integration/smoke/test_portable_publicip.py 9a3a398 > tools/marvin/marvin/integration/lib/base.py 0f6fdc5 > > Diff: https://reviews.apache.org/r/12942/diff/ > > > Testing > ------- > > Yes, but disassociating ip address fails. Maybe a product defect, yet to > investigate. > > > Thanks, > > Girish Shilamkar > >