Re: Review Request 12962: CLOUDSTACK-3594: Fix params of AffinityGroup.delete API

2013-08-14 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12962/#review25209 --- Commit 682e995a4a21e2d74b79bc26513055b913f3545d in branch refs/heads

Re: Review Request 12962: CLOUDSTACK-3594: Fix params of AffinityGroup.delete API

2013-08-14 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12962/#review25208 --- Commit 0d0409bddbcd9d8543d8dcf8d4648b9fcf6f5b2c in branch refs/heads

Re: Review Request 12962: CLOUDSTACK-3594: Fix params of AffinityGroup.delete API

2013-07-26 Thread Prasanna Santhanam
> On July 26, 2013, 8:21 a.m., Prasanna Santhanam wrote: > > I've changed the library method to only accept delete by id last night. Can > > you reuse that? We need to keep the delete operations consistent across the > > integration library. > > If a custom delete is required, then that can be

Re: Review Request 12962: CLOUDSTACK-3594: Fix params of AffinityGroup.delete API

2013-07-26 Thread Girish Shilamkar
> On July 26, 2013, 8:21 a.m., Prasanna Santhanam wrote: > > I've changed the library method to only accept delete by id last night. Can > > you reuse that? We need to keep the delete operations consistent across the > > integration library. > > If a custom delete is required, then that can be

Re: Review Request 12962: CLOUDSTACK-3594: Fix params of AffinityGroup.delete API

2013-07-26 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12962/#review23919 --- Review 12962 PASSED the build test The url of build cloudstack-maste

Re: Review Request 12962: CLOUDSTACK-3594: Fix params of AffinityGroup.delete API

2013-07-26 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12962/#review23918 --- I've changed the library method to only accept delete by id last nig

Review Request 12962: CLOUDSTACK-3594: Fix params of AffinityGroup.delete API

2013-07-26 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12962/ --- Review request for cloudstack and Prasanna Santhanam. Bugs: CLOUDSTACK-3594 R