I will check if it applies properly. It is a valid patch in terms of whether it should be added to repo or not.
Regards, Gaurav On Wed, Sep 17, 2014 at 2:42 PM, SrikanteswaraRao Talluri < srikanteswararao.tall...@citrix.com> wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23819/#review53670 > ----------------------------------------------------------- > > > Is this patch still valid? > > - SrikanteswaraRao Talluri > > > On Sept. 2, 2014, 6:26 a.m., Girish Shilamkar wrote: > > > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > https://reviews.apache.org/r/23819/ > > ----------------------------------------------------------- > > > > (Updated Sept. 2, 2014, 6:26 a.m.) > > > > > > Review request for cloudstack, sanjeev n and SrikanteswaraRao Talluri. > > > > > > Bugs: CLOUDSTACK-2251 > > https://issues.apache.org/jira/browse/CLOUDSTACK-2251 > > > > > > Repository: cloudstack-git > > > > > > Description > > ------- > > > > CLOUDSTACK-2251: Automation tests for dedicated public IP addreses per > tenant feature > > > > > > Diffs > > ----- > > > > test/integration/component/test_dedicate_public_ip_range.py > PRE-CREATION > > tools/marvin/marvin/config/test_data.py ade8657 > > tools/marvin/marvin/lib/base.py 99a541a > > > > Diff: https://reviews.apache.org/r/23819/diff/ > > > > > > Testing > > ------- > > > > Yes > > > > > > Thanks, > > > > Girish Shilamkar > > > > > >
