RE: Testing stratos 4.1: cartridge group deployment broken ? ... was ... RE: Validating the service group definition

2015-05-11 Thread Martin Eppel (meppel)
Ok, thanks From: Imesh Gunaratne [mailto:im...@apache.org] Sent: Monday, May 11, 2015 6:39 PM To: dev Cc: Vishanth Balasubramaniam Subject: Re: Testing stratos 4.1: cartridge group deployment broken ? ... was ... RE: Validating the service group definition Hi Martin, If you have dependencies

Re: Testing stratos 4.1: cartridge group deployment broken ? ... was ... RE: Validating the service group definition

2015-05-11 Thread Imesh Gunaratne
va:271) > > at > org.apache.cxf.transport.ChainInitiationObserver.onMessage(ChainInitiationObserver.java:121) > > at > org.apache.cxf.transport.http.AbstractHTTPDestination.invoke(AbstractHTTPDestination.java:239) > > at > org.apache.cxf.transport

Re: Validating the service group definition

2015-05-11 Thread Imesh Gunaratne
Thanks Vishanth! I have now merged this PR to master branch. On Mon, May 11, 2015 at 11:15 AM, Vishanth Balasubramaniam < vishan...@wso2.com> wrote: > Hi Imesh, > > Followed the feedback and made the changes to the PR wherever needed. > Please merge the PR [1]. > > [1] - https://github.com/apach

Re: Validating the service group definition

2015-05-10 Thread Vishanth Balasubramaniam
Hi Imesh, Followed the feedback and made the changes to the PR wherever needed. Please merge the PR [1]. [1] - https://github.com/apache/stratos/pull/296 Thank you, Vishanth On Wed, May 6, 2015 at 7:39 PM, Imesh Gunaratne wrote: > Hi Vishanth, > > Thanks for your effort. I have added some co

Re: Validating the service group definition

2015-05-06 Thread Imesh Gunaratne
Hi Vishanth, Thanks for your effort. I have added some comments in the PR, please review. Thanks On Wed, May 6, 2015 at 6:11 PM, Vishanth Balasubramaniam wrote: > Hi Devs, > > I have added the validation when adding a service group to check the > following > >- duplicate cartridge name in

Re: Validating the service group definition

2015-05-06 Thread Reka Thirunavukkarasu
Thanks Vishanth for fixing those issues. Will verify it and push it master.. Thanks, Reka On Wed, May 6, 2015 at 6:11 PM, Vishanth Balasubramaniam wrote: > Hi Devs, > > I have added the validation when adding a service group to check the > following > >- duplicate cartridge name in the same

Validating the service group definition

2015-05-06 Thread Vishanth Balasubramaniam
Hi Devs, I have added the validation when adding a service group to check the following - duplicate cartridge name in the same group level - duplicate group name in the same group level - cyclic behaviour of groups, i.e. when a parent group name is given again in any of its sub-groups