----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10009/#review18088 -----------------------------------------------------------
Ship it! Applied to 4.1 branch. Please update the tickets appropriately. - Chip Childers On March 19, 2013, 11:38 a.m., Kishan Kavala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10009/ > ----------------------------------------------------------- > > (Updated March 19, 2013, 11:38 a.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > ------- > > Remove region response corrected to return removeregionresponse instead of > updateregionresponse. > Default region should not contain /api in end_point > > > This addresses bugs CLOUDSTACK-1717 and CLOUDSTACK-1718. > > > Diffs > ----- > > api/src/org/apache/cloudstack/api/command/admin/region/RemoveRegionCmd.java > 79c34d0 > setup/db/db/schema-40to410.sql 865fbd3 > > Diff: https://reviews.apache.org/r/10009/diff/ > > > Testing > ------- > > Tested RemoveRegion API and default region creation. > > > Thanks, > > Kishan Kavala > >