Re: [Discuss] Stratos 4.0.0 Instance Activated Extension

2015-03-26 Thread Imesh Gunaratne
Thanks Anuruddha, this is now merged to master branch. On Thu, Mar 26, 2015 at 12:12 PM, Anuruddha Liyanarachchi < anurudd...@wso2.com> wrote: > Hi, > > I created the pull request [1] for the above. > > [1] https://github.com/apache/stratos/pull/278/files > > On Thu, Mar 26, 2015 at 11:50 AM, Anu

Re: [Discuss] Stratos 4.0.0 Instance Activated Extension

2015-03-25 Thread Anuruddha Liyanarachchi
Hi, I created the pull request [1] for the above. [1] https://github.com/apache/stratos/pull/278/files On Thu, Mar 26, 2015 at 11:50 AM, Anuruddha Liyanarachchi < anurudd...@wso2.com> wrote: > Hi Chamila, > > Please find the attached diff for the changes in master branch. > > > > On Thu, Mar 26

Re: [Discuss] Stratos 4.0.0 Instance Activated Extension

2015-03-25 Thread Anuruddha Liyanarachchi
Hi Chamila, Please find the attached diff for the changes in master branch. On Thu, Mar 26, 2015 at 11:07 AM, Anuruddha Liyanarachchi < anurudd...@wso2.com> wrote: > Hi Chamila, > > I will resend the diff rebased for master. > > On Thu, Mar 26, 2015 at 11:01 AM, Chamila De Alwis > wrote: > >

Re: [Discuss] Stratos 4.0.0 Instance Activated Extension

2015-03-25 Thread Anuruddha Liyanarachchi
Hi Chamila, I will resend the diff rebased for master. On Thu, Mar 26, 2015 at 11:01 AM, Chamila De Alwis wrote: > Hi Anuruddha, > > This seems to be based on 4.0.0-rc4 tag. Can you resend it by rebasing for > master? > > > Regards, > Chamila de Alwis > Software Engineer | WSO2 | +94772207163

Re: [Discuss] Stratos 4.0.0 Instance Activated Extension

2015-03-25 Thread Chamila De Alwis
Hi Anuruddha, This seems to be based on 4.0.0-rc4 tag. Can you resend it by rebasing for master? Regards, Chamila de Alwis Software Engineer | WSO2 | +94772207163 Blog: code.chamiladealwis.com On Tue, Mar 24, 2015 at 2:35 PM, Anuruddha Liyanarachchi < anurudd...@wso2.com> wrote: > Hi Chamila

Re: [Discuss] Stratos 4.0.0 Instance Activated Extension

2015-03-25 Thread Imesh Gunaratne
Great work Anuruddha! Will you be able to send a pull request for this? Thanks On Tue, Mar 24, 2015 at 2:35 PM, Anuruddha Liyanarachchi < anurudd...@wso2.com> wrote: > Hi Chamila, > > Please find the diff with changes to fix the issue. > WDYT ? > > On Tue, Mar 24, 2015 at 1:50 PM, Chamila De Alw

Re: [Discuss] Stratos 4.0.0 Instance Activated Extension

2015-03-24 Thread Anuruddha Liyanarachchi
Hi Chamila, Please find the diff with changes to fix the issue. WDYT ? On Tue, Mar 24, 2015 at 1:50 PM, Chamila De Alwis wrote: > Hi Anuruddha, > > Yes, this seems to be missing from the ArtifactUpdatedEvent processing. If > the extension is being manually executed in the non-repo based path, i

Re: [Discuss] Stratos 4.0.0 Instance Activated Extension

2015-03-24 Thread Chamila De Alwis
Hi Anuruddha, Yes, this seems to be missing from the ArtifactUpdatedEvent processing. If the extension is being manually executed in the non-repo based path, it should also be the case in the repo based path. It might have been dropped while being refactored. Regards, Chamila de Alwis Software E

[Discuss] Stratos 4.0.0 Instance Activated Extension

2015-03-24 Thread Anuruddha Liyanarachchi
Hi, Found that the instance activated extension is not getting called when onArtifactUpdatedEvent is publishing Instance activated event. However artifactUpdate extension is getting called. Shouldn't artifactUpdatedEvent trigger both instanceActivated and artifactUpdated Extensions? -- *Thank