This is not committed to stratos-4.1.x branch. On Tue, Dec 1, 2015 at 7:34 PM, Isuru Haththotuwa <isu...@apache.org> wrote:
> Sent a PR [1] with the fix. > > [1]. https://github.com/apache/stratos/pull/498/ > > On Sun, Nov 29, 2015 at 9:18 PM, Isuru Haththotuwa <isu...@apache.org> > wrote: > >> >> >> On Fri, Nov 27, 2015 at 11:51 AM, Akila Ravihansa Perera < >> raviha...@wso2.com> wrote: >> >>> Hi Isuru, >>> >>> Good thought. I'd suggest that we just rename the directory by appending >>> "stratos.adc.bak" to existing one. When removing the signup, rename it back >>> to the original name (remove the suffix). Let's keep things simple as >>> possible :) >>> >> +1, maybe using a different suffix :) >> >>> >>> Thanks. >>> >>> On Fri, Nov 27, 2015 at 11:35 AM, Isuru Haththotuwa <isu...@apache.org> >>> wrote: >>> >>>> Hi, >>>> >>>> On Fri, Nov 27, 2015 at 11:13 AM, Gayan Gunarathne <gay...@wso2.com> >>>> wrote: >>>> >>>>> Hi, >>>>> >>>>> On Fri, Nov 27, 2015 at 6:34 AM, Isuru Haththotuwa <isu...@apache.org> >>>>> wrote: >>>>> >>>>>> Hi Devs, >>>>>> >>>>>> Currently for a multiTenant Application (multiTenant=true), when a >>>>>> signup is removed, all artifacts which are in the local repository path >>>>>> will be deleted from the file system. >>>>>> >>>>> But there are some cases where the runtime of the cartridge instance >>>>>> depends on some of the artifacts which were originally there, and without >>>>>> those artifacts, the cartridge instance runtime would not function >>>>>> properly. I suggest the following to handle this: >>>>>> >>>>>> >>>>>> 1. Prior to the first clone, take a backup of the artifacts in >>>>>> the local repository path to a temporary location >>>>>> 2. When the signup is removed, clean all the artifacts which had >>>>>> been cloned to the local repository path location (from the git >>>>>> repository >>>>>> which was provided when signing up), and restore the artifacts (copy >>>>>> them >>>>>> back from the backup location to local repository path) >>>>>> >>>>>> This will ensure the state of the instance after removal of the >>>>>> signup is equivalent to the state before the signup was created. WDYT? >>>>>> >>>>> >>>>> I assume this is happened for the super tenant sign ups only.Other >>>>> tenants it works fine. Isn't? So we need to do consider this if is only >>>>> the >>>>> super tenant sign up. >>>>> >>>> We can implement this generically for all single tenant and multi >>>> tenant signup removal, without explicitly checking. IMHO that is the best >>>> way. >>>> >>>>> >>>>> >>>>>> -- >>>>>> Thanks and Regards, >>>>>> >>>>>> Isuru H. >>>>>> +94 716 358 048* <http://wso2.com/>* >>>>>> >>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> >>>>> Gayan Gunarathne >>>>> Technical Lead, WSO2 Inc. (http://wso2.com) >>>>> Committer & PMC Member, Apache Stratos >>>>> email : gay...@wso2.com | mobile : +94 775030545 >>>>> >>>>> -- >>>>> <%2B94%20766819985> >>>>> <%2B94%20766819985> >>>>> Thanks and Regards, >>>>> >>>>> Isuru H. >>>>> <%2B94%20766819985> >>>>> +94 716 358 048 <%2B94%20766819985>* <http://wso2.com/>* >>>>> >>>>> >>>>> * <http://wso2.com/>* >>>>> >>>>> >>>>> >>> >>> >>> -- >>> Akila Ravihansa Perera >>> WSO2 Inc.; http://wso2.com/ >>> >>> Blog: http://ravihansa3000.blogspot.com >>> >>> -- >>> <http://ravihansa3000.blogspot.com> >>> <http://ravihansa3000.blogspot.com> >>> Thanks and Regards, >>> >>> Isuru H. >>> <http://ravihansa3000.blogspot.com> >>> +94 716 358 048 <http://ravihansa3000.blogspot.com>* <http://wso2.com/>* >>> >>> >>> * <http://wso2.com/>* >>> >>> >>> > > > -- > Thanks and Regards, > > Isuru H. > +94 716 358 048* <http://wso2.com/>* > > > -- Thanks and Regards, Isuru H. +94 716 358 048* <http://wso2.com/>*