Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Isuru Haththotuwa
Hi Imesh, Sorry for the late response. On Fri, Nov 7, 2014 at 12:28 PM, Imesh Gunaratne wrote: > Hi Isuru, > > On Fri, Nov 7, 2014 at 11:19 AM, Isuru Haththotuwa > wrote: > >> >> IMHO we should have the context name 'stratos' in the url. Then it is >> more intuitive. having just /api doesn't

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Reka Thirunavukkarasu
Hi all, After fixing few issues, i could get the application undeployment working with one cluster now. So the basic undeployment is working fine...Will continue testing with a complex application... Thanks, Reka On Sun, Nov 9, 2014 at 1:05 PM, Isuru Haththotuwa wrote: > > > On Sun, Nov 9, 201

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Isuru Haththotuwa
On Sun, Nov 9, 2014 at 1:09 PM, Reka Thirunavukkarasu wrote: > Hi all, > > After fixing few issues, i could get the application undeployment working > with one cluster now. So the basic undeployment is working fine...Will > continue testing with a complex application... > Great! I will test the s

Re: NPE while undeploying an application

2014-11-08 Thread Reka Thirunavukkarasu
This is fixed now... On Fri, Nov 7, 2014 at 7:27 PM, Udara Liyanage wrote: > > Hi Reka, Isuru, > > I got the following NPE when I undeployed an app. While debuging I noticed > that the appId of the moniter is not set, thus application become null. > > >1. String appId = monitor.getAp

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Isuru Haththotuwa
On Sun, Nov 9, 2014 at 8:38 AM, Imesh Gunaratne wrote: > No, there seems to be a problem with application un-deploy function. For > me the un-deploy call hangs in the autoscaler. This is currerntly under > investigation. > Got the same issue, will check in this. > > Since the basic workflow of s

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Isuru Haththotuwa
On Sat, Nov 8, 2014 at 12:15 PM, Rajkumar Rajaratnam wrote: > Does anyone know why cluster status filed is commented out on > Cluster.java? > > //private ClusterStatus status; > Sorry for the late response, there is a LifeCycleStateManager which was added recently, and currently status is mainta

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Reka Thirunavukkarasu
I too fixed few application undeployment issues..Will hold the commit now onwards.. On Sun, Nov 9, 2014 at 12:46 PM, Lahiru Sandaruwan wrote: > > > On Sun, Nov 9, 2014 at 12:38 PM, Manula Chathurika Thantriwatte < > manu...@wso2.com> wrote: > >> Hi Lahiru, >> >> Sure, please let us know when you

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Lahiru Sandaruwan
On Sun, Nov 9, 2014 at 12:38 PM, Manula Chathurika Thantriwatte < manu...@wso2.com> wrote: > Hi Lahiru, > > Sure, please let us know when you finish it. > Done just now. > > > > On Sun, Nov 9, 2014 at 12:17 PM, Lahiru Sandaruwan > wrote: > >> Can you guys hold until i push cluster context change

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Manula Chathurika Thantriwatte
Hi Lahiru, Sure, please let us know when you finish it. On Sun, Nov 9, 2014 at 12:17 PM, Lahiru Sandaruwan wrote: > Can you guys hold until i push cluster context change. Will do it asap. > > On Sun, Nov 9, 2014 at 12:13 PM, Manula Chathurika Thantriwatte < > manu...@wso2.com> wrote: > >> Hi,

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Lahiru Sandaruwan
Can you guys hold until i push cluster context change. Will do it asap. On Sun, Nov 9, 2014 at 12:13 PM, Manula Chathurika Thantriwatte < manu...@wso2.com> wrote: > Hi, > > +1, then shall we merge docker-grouping-merge branch to master branch and > continue testing on it. > > Thanks ! > > On Sun,

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Manula Chathurika Thantriwatte
Hi, +1, then shall we merge docker-grouping-merge branch to master branch and continue testing on it. Thanks ! On Sun, Nov 9, 2014 at 12:05 PM, Reka Thirunavukkarasu wrote: > Hi Imesh, > > On Sun, Nov 9, 2014 at 8:38 AM, Imesh Gunaratne wrote: > >> No, there seems to be a problem with applica

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Reka Thirunavukkarasu
Hi Imesh, On Sun, Nov 9, 2014 at 8:38 AM, Imesh Gunaratne wrote: > No, there seems to be a problem with application un-deploy function. For > me the un-deploy call hangs in the autoscaler. This is currerntly under > investigation. > > Since the basic workflow of service grouping and docker are w

Commit New UI changes to docker-grouping-merge

2014-11-08 Thread Manula Chathurika Thantriwatte
Hi Dakshika, Is it possible to do $subject ? Thanks ! -- Regards, Manula Chathurika Thantriwatte Software Engineer WSO2 Inc. : http://wso2.com lean . enterprise . middleware email : manu...@wso2.com / man...@apache.org phone : +94 772492511 blog : http://manulachathurika.blogspot.com/

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Rajkumar Rajaratnam
Hi Imesh, - Couldn't access the service via LB. LB routes requests to port 80 instead of service proxy port. Looking into it currently. - Manual scaling is not working But I guess we can merge to master and fix these there. These are minor issues. Thanks. On Sun, Nov 9, 2014 at 8:38 AM

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Imesh Gunaratne
No, there seems to be a problem with application un-deploy function. For me the un-deploy call hangs in the autoscaler. This is currerntly under investigation. Since the basic workflow of service grouping and docker are working now shall we merge docker-grouping-merge branch to master branch now?

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Rajaratnam Rajkumar
Hi all, Did anyone undeploy the application successfully? Were all clusters wiped out? Thanks On Saturday, November 8, 2014, Imesh Gunaratne wrote: > On Sat, Nov 8, 2014 at 11:13 PM, Chamila De Alwis > wrote: > >> >> Don't we also need the event processing chain since it constructs the >> even

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Imesh Gunaratne
On Sat, Nov 8, 2014 at 11:13 PM, Chamila De Alwis wrote: > > Don't we also need the event processing chain since it constructs the > event objects from the Jsons? Additionally this is done through the Gson > library, so we would have some difficulty in getting that functionality in > to python as

Re: CLI - DefaultHttpClient is deprecreated

2014-11-08 Thread Imesh Gunaratne
Yes this is something we need to fix. On Sat, Nov 8, 2014 at 11:21 PM, Udara Liyanage wrote: > Hi, > > DefaultHttpClient used in CLI is shown as deprecated, shall we upgrade it > to newer version. We can decouple the REST client from metadata service and > use the same client in both CLI and met

Re: Maintaining Better Coding Standards

2014-11-08 Thread Imesh Gunaratne
On Sat, Nov 8, 2014 at 11:14 PM, Udara Liyanage wrote: > Hi Imesh, > > Also in below methods entityName is used only for printing messages. We > can remove them, right? > No, this is how we have generalized CRUD operations and their error messages. Entity name is needed.

CLI - DefaultHttpClient is deprecreated

2014-11-08 Thread Udara Liyanage
Hi, DefaultHttpClient used in CLI is shown as deprecated, shall we upgrade it to newer version. We can decouple the REST client from metadata service and use the same client in both CLI and metadataservice. -- Udara Liyanage Software Engineer WSO2, Inc.: http://wso2.com lean. enterprise. middl

Re: Maintaining Better Coding Standards

2014-11-08 Thread Udara Liyanage
Hi Imesh, Also in below methods entityName is used only for printing messages. We can remove them, right? 1. public Object getEntity(String serviceEndpoint, Class responseJsonClass, String identifier, String entityName) { 2. try { 3. return executeGet(serviceEndp

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Chamila De Alwis
Hi Imesh, On Sat, Nov 8, 2014 at 7:44 PM, Imesh Gunaratne wrote: > My understanding is that we only need the domain classes converted to > python. They may only contain list of properties. Therefore the generated > code should be simple enough to validate. Don't we also need the event processi

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Lahiru Sandaruwan
I'm on scaling based on dependency ratio. We need a place to keep runtime information of cluster, which should be also persisted. So it should be the same place where we keep updated autoscale/deployment policies at runtime. (manual scaling) I'll send mails and get the community idea as well. Im f

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Reka Thirunavukkarasu
Hi I could bring an application with more than one cluster as Active. Currently testing the faulty member scenario for VM with the Composite application and fixing issues. Will update on that.. Thanks, Reka On Sat, Nov 8, 2014 at 10:39 PM, Imesh Gunaratne wrote: > I have now done the followin

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Imesh Gunaratne
I have now done the following changes: - Added get applications resource to REST API - Added list-applications command to CLI - Removed CLI commands subscribe-cartridge, list-cartridge-subscriptions, unsubscribe-cartridge and deprecated command classes - Fixed coding convention issues in the CLI -

Re: Maintaining Better Coding Standards

2014-11-08 Thread Imesh Gunaratne
I have now fixed this with commit revision: 414a93b7394de6eb3494f7d164e599006032ffcf On Sat, Nov 8, 2014 at 10:04 PM, Imesh Gunaratne wrote: > Hi, > > I noticed following domain classes introduced recently in the CLI: > > > ​ > It seems like the naming convention used in above classes Applicatio

Maintaining Better Coding Standards

2014-11-08 Thread Imesh Gunaratne
Hi, I noticed following domain classes introduced recently in the CLI: ​ It seems like the naming convention used in above classes Applications, Clusters, Groups and SubGroups is not correct. These classes look like collection types but they are single object types. Class: RestCommandLineServic

Re: Manual scaling effects are not persisted

2014-11-08 Thread Lahiru Sandaruwan
Hi, It seems we need to introduce cluster context(which will be persisted) for keeping these runtime changes to policies and also the run time data such as required count. I will work on this. Thanks. On Sat, Oct 18, 2014 at 10:58 AM, Imesh Gunaratne wrote: > Hi, > > @Raj: What I pointed out

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Lahiru Sandaruwan
On Sat, Nov 8, 2014 at 9:06 PM, Reka Thirunavukkarasu wrote: > Hi > > On Sat, Nov 8, 2014 at 7:57 PM, Lahiru Sandaruwan > wrote: > >> Hi all, >> >> I'm working on $subject. When we predict the instance count from >> statistics and dependency based ratio in application, we will get numbers >> wit

[GitHub] stratos pull request: Review changes

2014-11-08 Thread R-Rajkumar
Github user R-Rajkumar closed the pull request at: https://github.com/apache/stratos/pull/82 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Reka Thirunavukkarasu
Hi On Sat, Nov 8, 2014 at 7:57 PM, Lahiru Sandaruwan wrote: > Hi all, > > I'm working on $subject. When we predict the instance count from > statistics and dependency based ratio in application, we will get numbers > with fraction. E.g. 1.2, 3.9, 4.5. Therefore we have to round the instance > co

Re: Manual Scaling Support for Stratos

2014-11-08 Thread Lahiru Sandaruwan
Thanks. On Sat, Nov 8, 2014 at 5:18 PM, Nirmal Fernando wrote: > Nope, currently. There was a discussion here: Manual scaling effects are > not persisted > > On Sat, Nov 8, 2014 at 12:28 PM, Lahiru Sandaruwan > wrote: > >> Hi Nirmal, >> >> Do we persist the values set by Manual-scaling command?

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Rajkumar Rajaratnam
On Sat, Nov 8, 2014 at 8:32 PM, Lahiru Sandaruwan wrote: > > > On Sat, Nov 8, 2014 at 8:26 PM, Rajkumar Rajaratnam > wrote: > >> Hi Lahiru, >> >> Are you going to make all three configurable via autoscaler xml? or How >> you are going to do? >> >> > IMO it should be a part of Autoscaling policy.

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Rajkumar Rajaratnam
On Sat, Nov 8, 2014 at 8:24 PM, Lahiru Sandaruwan wrote: > > > On Sat, Nov 8, 2014 at 8:14 PM, Rajkumar Rajaratnam > wrote: > >> Oh, I think I am confused here. Ignore the previous mail. >> >> But #3 can cover all the scenario :) >> > > Yes. User can input 0 for #1 and 1 for #2. May be we can ju

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Lahiru Sandaruwan
On Sat, Nov 8, 2014 at 8:26 PM, Rajkumar Rajaratnam wrote: > Hi Lahiru, > > Are you going to make all three configurable via autoscaler xml? or How > you are going to do? > > IMO it should be a part of Autoscaling policy. > Thanks. > > On Sat, Nov 8, 2014 at 8:22 PM, Lahiru Sandaruwan > wrote:

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Rajkumar Rajaratnam
Hi Lahiru, Are you going to make all three configurable via autoscaler xml? or How you are going to do? Thanks. On Sat, Nov 8, 2014 at 8:22 PM, Lahiru Sandaruwan wrote: > > > On Sat, Nov 8, 2014 at 8:06 PM, Nirmal Fernando > wrote: > >> Why not simply round the value to nearest integer? >> ht

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Lahiru Sandaruwan
On Sat, Nov 8, 2014 at 8:14 PM, Rajkumar Rajaratnam wrote: > Oh, I think I am confused here. Ignore the previous mail. > > But #3 can cover all the scenario :) > Yes. User can input 0 for #1 and 1 for #2. May be we can just have one option to the users then. > > Thanks. > > On Sat, Nov 8, 2014

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Lahiru Sandaruwan
On Sat, Nov 8, 2014 at 8:06 PM, Nirmal Fernando wrote: > Why not simply round the value to nearest integer? > http://docs.oracle.com/javase/7/docs/api/java/lang/Math.html#round(float) > It is not the industry practice. AWS guys use a more complex algorithms. But we can start with suggested optio

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Rajkumar Rajaratnam
Oh, I think I am confused here. Ignore the previous mail. But #3 can cover all the scenario :) Thanks. On Sat, Nov 8, 2014 at 8:07 PM, Rajkumar Rajaratnam wrote: > Hi Lahiru, > > #3 includes #1 and #2. > > If user input < 0.5 implies #1 > If user input >= 0.5 implies #2 > > IMO, we can go with

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Nirmal Fernando
Why not simply round the value to nearest integer? http://docs.oracle.com/javase/7/docs/api/java/lang/Math.html#round(float) On Sat, Nov 8, 2014 at 3:28 PM, Lahiru Sandaruwan wrote: > /s/fives/gives > > On Sat, Nov 8, 2014 at 7:57 PM, Lahiru Sandaruwan > wrote: > >> Hi all, >> >> I'm working on

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Rajkumar Rajaratnam
Hi Lahiru, #3 includes #1 and #2. If user input < 0.5 implies #1 If user input >= 0.5 implies #2 IMO, we can go with only #3. wdyt? Thanks. On Sat, Nov 8, 2014 at 7:57 PM, Lahiru Sandaruwan wrote: > Hi all, > > I'm working on $subject. When we predict the instance count from > statistics an

Re: [Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Lahiru Sandaruwan
/s/fives/gives On Sat, Nov 8, 2014 at 7:57 PM, Lahiru Sandaruwan wrote: > Hi all, > > I'm working on $subject. When we predict the instance count from > statistics and dependency based ratio in application, we will get numbers > with fraction. E.g. 1.2, 3.9, 4.5. Therefore we have to round the i

[Autoscaling]Instance count rounding algorithm

2014-11-08 Thread Lahiru Sandaruwan
Hi all, I'm working on $subject. When we predict the instance count from statistics and dependency based ratio in application, we will get numbers with fraction. E.g. 1.2, 3.9, 4.5. Therefore we have to round the instance count for creating instances in IaaS. I have thought of three types to cho

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Imesh Gunaratne
My understanding is that we only need the domain classes converted to python. They may only contain list of properties. Therefore the generated code should be simple enough to validate. On Sat, Nov 8, 2014 at 7:10 PM, Chamila De Alwis wrote: > Hi Imesh, > > Is it reliable to follow a code conve

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Rajkumar Rajaratnam
Update: - Auto-scaling works fine for container scenario - Faulty members are successfully terminated and new containers are created successfully Thanks. On Sat, Nov 8, 2014 at 7:44 PM, Imesh Gunaratne wrote: > My understanding is that we only need the domain classes converted to > py

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Rajkumar Rajaratnam
On Sat, Nov 8, 2014 at 7:10 PM, Chamila De Alwis wrote: > Hi Imesh, > > Is it reliable to follow a code conversion tool? Given that this tool > doesn't guarantee working code, I'm not sure this would be a better option. > > *"The generated Python code is not guaranteed to run,"* > If it is not g

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Chamila De Alwis
Hi Imesh, Is it reliable to follow a code conversion tool? Given that this tool doesn't guarantee working code, I'm not sure this would be a better option. *"The generated Python code is not guaranteed to run,"* Regards, Chamila de Alwis Software Engineer | WSO2 | +94772207163 Blog: code.chamil

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Rajkumar Rajaratnam
Yes Java agent. Thanks. On Sat, Nov 8, 2014 at 6:32 PM, Imesh Gunaratne wrote: > Thanks for the update Raj, were you using Java cartridge agent? > > On Sat, Nov 8, 2014 at 6:21 PM, Rajkumar Rajaratnam > wrote: > >> Hi all, >> >> Just to give an update; >> >> Simple container scenario is workin

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Imesh Gunaratne
Thanks for the update Raj, were you using Java cartridge agent? On Sat, Nov 8, 2014 at 6:21 PM, Rajkumar Rajaratnam wrote: > Hi all, > > Just to give an update; > > Simple container scenario is working now with m2 image. Members are > activated and publishing health stats to CEP. Seems undeployi

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Rajkumar Rajaratnam
Hi all, Just to give an update; Simple container scenario is working now with m2 image. Members are activated and publishing health stats to CEP. Seems undeploying application is not working. Looking into it currently. Thanks. On Sat, Nov 8, 2014 at 1:50 PM, Imesh Gunaratne wrote: > I think w

Re: stratos git commit: Fixing a typo

2014-11-08 Thread Lahiru Sandaruwan
:) will change On Sat, Nov 8, 2014 at 5:34 PM, Nirmal Fernando wrote: > /s/particuler/particular > > On Sat, Nov 8, 2014 at 12:51 PM, wrote: > >> Repository: stratos >> Updated Branches: >> refs/heads/docker-grouping-merge 3930b18b0 -> 34a3a8f51 >> >> >> Fixing a typo >> >> >> Project: http:/

Re: stratos git commit: Fixing a typo

2014-11-08 Thread Nirmal Fernando
/s/particuler/particular On Sat, Nov 8, 2014 at 12:51 PM, wrote: > Repository: stratos > Updated Branches: > refs/heads/docker-grouping-merge 3930b18b0 -> 34a3a8f51 > > > Fixing a typo > > > Project: http://git-wip-us.apache.org/repos/asf/stratos/repo > Commit: http://git-wip-us.apache.org/rep

Re: Manual Scaling Support for Stratos

2014-11-08 Thread Nirmal Fernando
Nope, currently. There was a discussion here: Manual scaling effects are not persisted On Sat, Nov 8, 2014 at 12:28 PM, Lahiru Sandaruwan wrote: > Hi Nirmal, > > Do we persist the values set by Manual-scaling command? > > > Thanks. > > On Mon, Oct 13, 2014 at 4:32 PM, Nirmal Fernando > wrote: >

Re: Manual Scaling Support for Stratos

2014-11-08 Thread Lahiru Sandaruwan
Hi Nirmal, Do we persist the values set by Manual-scaling command? Thanks. On Mon, Oct 13, 2014 at 4:32 PM, Nirmal Fernando wrote: > Hi Lahiru, > > Yes, VM cluster monitor needs to read relevant properties and update them > accordingly. > > On Mon, Oct 13, 2014 at 12:03 PM, Lahiru Sandaruwan

Re: [Important] Merging Grouping Changes to Master Branch

2014-11-08 Thread Imesh Gunaratne
I think we can use [1] to convert messaging component data model to python at the build time and the agent can refer that directly. WDYT? [1] https://github.com/natural/java2python On Nov 8, 2014 1:28 PM, "Chamila De Alwis" wrote: > Hi Raj, > > On Sat, Nov 8, 2014 at 1:19 PM, Rajkumar Rajaratnam