stratos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sajith Kariyawasam <saj...@wso2.com>
Subject Re: [Hangout] Design and Code review - Kubernetes Host Registration
Date Fri, 26 Sep 2014 21:28:02 GMT
Hi all,

Another point I would like to highlight, it seems that we are having a
dependency to stub bundles in rest.endpoint bundles, which I feel not
correct as rest.endpoint bundles don't invoke stubs,
instead rest endpoint invokes manager component, which is the one invokes
the back end services via stubs. So in that case we should move the object
conversion (from "common" to stub") in to the manager bundle, not to be
done in the rest endpoint component.
wdyt?

On Sat, Sep 27, 2014 at 2:40 AM, Sajith Kariyawasam <sajith@wso2.com> wrote:

>
>
> On Sat, Sep 27, 2014 at 12:21 AM, Rajkumar Rajaratnam <rajkumarr@wso2.com>
> wrote:
>
>>
>>
>> On Fri, Sep 26, 2014 at 3:32 PM, Sajith Kariyawasam <sajith@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 <sajith@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 <
>>>> rajkumarr@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 <
>>>>> ravihansa@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 <
>>>>>> nirmal070125@gmail.com>
>>>>>> >> wrote:
>>>>>> >> >
>>>>>> >> > On Fri, Sep 26, 2014 at 1:17 PM, Akila Ravihansa Perera
>>>>>> >> > <ravihansa@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*
>



-- 
*Sajith Kariyawasam*


*Committer and PMC member, Apache Stratos,WSO2 Inc., http://wso2.com
<http://wso2.com>AMIE (SL)Mobile: +94772269575*

Mime
View raw message