On Sat, Sep 27, 2014 at 12:21 AM, Rajkumar Rajaratnam <rajkum...@wso2.com> wrote:
> > > On Fri, Sep 26, 2014 at 3:32 PM, Sajith Kariyawasam <saj...@wso2.com> > wrote: > >> Found that some of the AutoscalerImpl operations are invoked by directly >> instantiating the AutoscalerImpl. I think this is not good. >> AutoscalerImpl meant to be a web service, which needs to be accessed only >> via service stubs, referring to the WSDL. >> > > Fixed it now. It is using kubernetes manager instead of AS API. > >> >> Great! > Another small thing to be highlight, regarding the interface naming. >> Currently Autoscaler service's interface and CC 's interface naming are not >> consistent. >> IMO the part 'interface' should not be go in the name of the interface, >> its no hard and fast rule, but I think its better to maintain the >> consistency, So I would like to suggest to >> rename AutoScalerServiceInterface as AutoScalerService. >> >> thoughts? >> >> On Fri, Sep 26, 2014 at 3:13 PM, Sajith Kariyawasam <saj...@wso2.com> >> wrote: >> >>> +1 for Raj's suggestion. >>> >>> Also, how about in "updateX" operations if the updated object is >>> returned instead of the boolean? That will make the programming in client >>> side easy, otherwise it will need to do another "getX" call to retrieve the >>> updated object ? >>> >>> Thanks, >>> Sajith >>> >>> On Fri, Sep 26, 2014 at 2:44 PM, Rajkumar Rajaratnam <rajkum...@wso2.com >>> > wrote: >>> >>>> Hi, >>>> >>>> One KubernetesHost is associated with only one KubernetesGroup. Hence >>>> ideally KubernetesHost class should have a reference to KubernetesGroup. I >>>> mean KubernetesHost should have a KubernetesGroupId filed. >>>> >>>> There are some advantages of doing this. >>>> >>>> 1. We need to pass only KubernetesHost object when calling >>>> addKubernetesHost() method. Currently we are passing both >>>> KubernetesGroupId >>>> and KubernetesHost object. >>>> 2. removeKubernetesHost() method will become more efficient. I >>>> could see that you are looping through all Kubernetes Groups in-order to >>>> identify which Group has the given host. Then you are removing the host >>>> from that group. If you store the KubernetesGroupId in the >>>> KubernetesHost >>>> object, then no need to loop through all the KubernetesGroups. Of >>>> course we >>>> need to pass the KubernetesHost object to the removeKubernetesHost() >>>> method. >>>> 3. updateKubernetesHost() would work faster because of the same >>>> reason as above. >>>> >>>> It will improve the performance. >>>> >>>> wdyt? >>>> >>>> Thanks. >>>> >>>> On Fri, Sep 26, 2014 at 2:23 PM, Akila Ravihansa Perera < >>>> raviha...@wso2.com> wrote: >>>> >>>>> Hi Nirmal, >>>>> >>>>> >> >>>>> >> Thanks for bringing that up. >>>>> >> >>>>> >> Since Kubernetes Group ID is given in the request body itself, >>>>> > >>>>> > >>>>> > Are we adding it in request body or in the request path? >>>>> >>>>> When deploying a new KubernetesGroup we are specifying the groupId in >>>>> the request body itself. We are basically serializing the >>>>> KubernetesGroup object which contains the groupId field. >>>>> >>>>> But when deploying a Kubernetes host, we are passing Kubernetes >>>>> groupId in the request path. >>>>> >>>>> >>>>> > >>>>> >> >>>>> >> shouldn't we make it PUT? Here is a good discussion about this on >>>>> >> StackOverflow [1]. >>>>> >> >>>>> >> [1] http://stackoverflow.com/questions/630453/put-vs-post-in-rest >>>>> >> >>>>> >> On Fri, Sep 26, 2014 at 1:20 PM, Nirmal Fernando < >>>>> nirmal070...@gmail.com> >>>>> >> wrote: >>>>> >> > >>>>> >> > On Fri, Sep 26, 2014 at 1:17 PM, Akila Ravihansa Perera >>>>> >> > <raviha...@wso2.com> >>>>> >> > wrote: >>>>> >> >> >>>>> >> >> Thanks everyone for joining the call. Really sorry about the >>>>> technical >>>>> >> >> glitches we had to face during the Hangout. >>>>> >> >> >>>>> >> >> Here are the meeting notes; >>>>> >> >> >>>>> >> >> 1. Clean up the REST API. Change the REST APIs to adhere to >>>>> RESTful >>>>> >> >> design >>>>> >> >> (not only limited to Kubernetes host registration) >>>>> >> >> >>>>> >> >> For eg - >>>>> >> >> >>>>> >> >> Deploying a new Kubernetes host - >>>>> >> >> PUT /kubernetes/group/{kubernetesGroupId}/host >>>>> >> >> >>>>> >> >> >>>>> >> >> Deploying a new Kubernetes group - >>>>> >> >> PUT /kubernetes/group/ >>>>> >> > >>>>> >> > >>>>> >> > These should be POST IMO. >>>>> >> >> >>>>> >> >> >>>>> >> >> 2. Remove boolean return types in AutoScalerService interface >>>>> >> >> >>>>> >> >> >>>>> >> >> Adding to that what I could not discuss: >>>>> >> >> >>>>> >> >> 1. Shall we rename KubernetesGroup -> KubernetesCluster to make >>>>> the >>>>> >> >> naming >>>>> >> >> consistent? >>>>> >> > >>>>> >> > >>>>> >> > +1 >>>>> >> >> >>>>> >> >> >>>>> >> >> >>>>> >> >> Thanks. >>>>> >> > >>>>> >> > >>>>> >> > >>>>> >> > >>>>> >> > -- >>>>> >> > Best Regards, >>>>> >> > Nirmal >>>>> >> > >>>>> >> > Nirmal Fernando. >>>>> >> > PPMC Member & Committer of Apache Stratos, >>>>> >> > Senior Software Engineer, WSO2 Inc. >>>>> >> > >>>>> >> > Blog: http://nirmalfdo.blogspot.com/ >>>>> >> >>>>> >> >>>>> >> >>>>> >> -- >>>>> >> Akila Ravihansa Perera >>>>> >> Software Engineer, WSO2 >>>>> >> >>>>> >> Blog: http://ravihansa3000.blogspot.com >>>>> > >>>>> > >>>>> > >>>>> > >>>>> > -- >>>>> > Best Regards, >>>>> > Nirmal >>>>> > >>>>> > Nirmal Fernando. >>>>> > PPMC Member & Committer of Apache Stratos, >>>>> > Senior Software Engineer, WSO2 Inc. >>>>> > >>>>> > Blog: http://nirmalfdo.blogspot.com/ >>>>> >>>>> >>>>> >>>>> -- >>>>> Akila Ravihansa Perera >>>>> Software Engineer, WSO2 >>>>> >>>>> Blog: http://ravihansa3000.blogspot.com >>>>> >>>> >>>> >>>> >>>> -- >>>> Rajkumar Rajaratnam >>>> Software Engineer | WSO2, Inc. >>>> Mobile +94777568639 | +94783498120 >>>> >>> >>> >>> >>> -- >>> *Sajith Kariyawasam* >>> >>> >>> *Committer and PMC member, Apache Stratos,WSO2 Inc., http://wso2.com >>> <http://wso2.com>AMIE (SL)Mobile: +94772269575* >>> >> >> >> >> -- >> *Sajith Kariyawasam* >> >> >> *Committer and PMC member, Apache Stratos,WSO2 Inc., http://wso2.com >> <http://wso2.com>AMIE (SL)Mobile: +94772269575* >> > > > > -- > Rajkumar Rajaratnam > Software Engineer | WSO2, Inc. > Mobile +94777568639 | +94783498120 > -- *Sajith Kariyawasam* *Committer and PMC member, Apache Stratos,WSO2 Inc., http://wso2.com <http://wso2.com>AMIE (SL)Mobile: +94772269575*