Hi All, +1 for having minimal set of components. Shall we create list of components we need here. AFAIU following components must be there. Please add/ remove what i missed here. So we can come up with final list and continue refactoring.
autoscaler autoscaler-service lb-endpoint org.wso2.carbon.lb.common account-mgt org.wso2.carbon.activation org.wso2.carbon.stratos.common status-monitor usage artifact-deployment-coordinator org.wso2.carbon.cartridge.mgt.ui org.wso2.carbon.keystore.mgt org.wso2.carbon.stratos.deployment tenant-activity org.wso2.carbon.stratos.landing.page.deployer redirector tenant-mgt cartridge-agent org.wso2.carbon.email.sender org.wso2.carbon.migration org.wso2.carbon.tenant.dispatcher theme-mgt domain-validator org.wso2.carbon.stratos.cloud.controller throttling On Sun, Jul 7, 2013 at 10:02 AM, Pradeep Fernando <[email protected]>wrote: > Hi Devs, > > I think we should give priority to identify the minimal set of components. > It will help in the refactoring process as well. At the moment I am doing > some component refactoring and find it hard to what to re-factor and what > not.. > > thanks, > --Pradeep > > > On Sat, Jul 6, 2013 at 9:13 AM, Shariq Muhammed <[email protected]> wrote: > >> On Sat, Jul 6, 2013 at 9:06 AM, Amila Maha Arachchi <[email protected]>wrote: >> >>> >>> >>> >>> On Sat, Jul 6, 2013 at 8:58 AM, Shariq Muhammed <[email protected]> wrote: >>> >>>> If I am not mistaken the following components also could be removed >>>> since they are only needed for Carbon based products (cartridges in >>>> future ?!) >>>> >>>> - components/stratos/org.wso2.carbon.ui.menu.stratos - provides a link >>>> in the services to direct to CC >>>> - components/stratos/org.wso2.carbon.stratos.landing.page.deployer - >>>> deploys the carbon service specific landing page webapp >>>> >>> >>> Yes. I'll consider these as well when creating the patch. >>> >> >> Great, guess there is one or two more, need to check the code ... >> >>> >>>> Also how about getting rid of components/*stratos *dir and moving its >>>> contents under components/? The reason to the "stratos" dir in carbon >>>> code base was to distinguish between stratos and other components, >>>> that's not valid here .. >>>> >>> >>> This will be done eventually when the cleanup and re-factoring is >>> completed. We are starting from the code which was available in Carbon. >>> >> >> Gotcha ..! >> >>> >>>> >>>> >>>> >>>> On Sat, Jul 6, 2013 at 8:23 AM, Amila Maha Arachchi <[email protected]>wrote: >>>> >>>>> Hi all, >>>>> >>>>> I was going through the incubator code to provide some help on >>>>> cleaning it and making it ready for the first release. I have identified >>>>> some components which can be deleted from the code. Some of these are not >>>>> used anymore and some others are not valid for Apache Stratos. >>>>> >>>>> components/stratos/org.wso2.carbon.db.keep.alive (unused) >>>>> components/stratos/org.wso2.carbon.gapp.registration.ui (not valid) >>>>> components/stratos/org.wso2.carbon.google.analytics.ui (not valid) >>>>> components/stratos/org.wso2.carbon.migration (unused) >>>>> components/stratos/sample-installer (not valid) >>>>> >>>>> You can simple get rid of the unused ones. For the invalid >>>>> components, I will check for dependencies in the >>>>> product/stratos2/stratos_controller and provide a patch to remove them. >>>>> >>>>> There are some other components which will be used as dependencies >>>>> from Carbon code. We can get rid of them too. We'll discuss them >>>>> separately. >>>>> >>>>> Regards, >>>>> Amila. >>>>> >>>>> -- >>>>> *Amila Maharachchi* >>>>> Senior Technical Lead >>>>> WSO2, Inc.; http://wso2.com >>>>> >>>>> Blog: http://maharachchi.blogspot.com >>>>> Mobile: +94719371446 >>>>> >>>>> >>>> >>>> >>>> -- >>>> Thanks, >>>> Shariq. >>>> Phone: +94 777 202 225 >>>> >>> >>> >>> >>> -- >>> *Amila Maharachchi* >>> Senior Technical Lead >>> WSO2, Inc.; http://wso2.com >>> >>> Blog: http://maharachchi.blogspot.com >>> Mobile: +94719371446 >>> >>> >> >> >> -- >> Thanks, >> Shariq. >> Phone: +94 777 202 225 >> > > > > -- > Pradeep Fernando. > http://pradeepfernando.blogspot.com/ > -- *Sanjeewa Malalgoda* http://sanjeewamalalgoda.blogspot.com/ <http://sanjeewamalalgoda.blogspot.com/>
