Hi Asanka,

Great work and thanks for the PR.

Just glanced at it. There seem to be several places where you have hard
coded GCE zones, project ids, etc. where you have added TODOs remove them
later. Also, there are places that you are using System.outs to print
information. Shall we clean up these? Please check on formatting as well.
That will help the community to focus on the functionality, code
improvements and detect possible issues.

On Fri, Aug 7, 2015 at 1:03 AM, Asanka Sanjaya Herath <angal...@gmail.com>
wrote:

> I have created the PR [1].
>
> [1]. https://github.com/apache/stratos/pull/419
>
> On Fri, Aug 7, 2015 at 11:43 AM, Asanka Sanjaya Herath <angal...@gmail.com
> > wrote:
>
>> Hi Gayan,
>>
>> I'll send the PR today. Sorry for the delay.
>>
>> On Thu, Aug 6, 2015 at 1:02 PM, Gayan Gunarathne <gay...@wso2.com> wrote:
>>
>>> Hi Asanka,
>>>
>>> Once you have done with the above code review changes , please send us
>>> the PR for the implementation.
>>>
>>> Thanks,
>>> Gayan
>>>
>>> On Thu, Jul 30, 2015 at 7:14 PM, Asanka Sanjaya Herath <
>>> angal...@gmail.com> wrote:
>>>
>>>> Code review notes:
>>>>
>>>> 1. Without throwing run time exceptions always try to handle the
>>>> exception
>>>> 2. Try to throw more specific exceptions without just throwing generic
>>>> exceptions so that will be easy to troubleshoot
>>>> 3. Always put generic comments
>>>> 4. Add debug log messages
>>>>
>>>>
>>>> On Thu, Jul 30, 2015 at 12:06 PM, Asanka Sanjaya Herath <
>>>> angal...@gmail.com> wrote:
>>>>
>>>>> Hi All,
>>>>>
>>>>> We had the hangout and demo as we planned. We discussed about issues
>>>>> in current code like exception handling commenting etc. And we discussed
>>>>> about modifications I did, future plans, limitations in GCE like domain
>>>>> mapping, statistic publishing etc.
>>>>>
>>>>> I'm planing to do all the modifications requested in my code and do a
>>>>> screen cast  again including the changes I have done.
>>>>>
>>>>> Participants: Stratos Team, Asanka
>>>>>
>>>>> On Mon, Jul 27, 2015 at 12:16 PM, Asanka Sanjaya Herath <
>>>>> angal...@gmail.com> wrote:
>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> We are planning to have a hangout in this week for discuss the
>>>>>> progress, changes I have made and issues about the GCE LB extension
>>>>>> currently I'm working on. It has been scheduled on 30th Thursday at 11 AM
>>>>>> IST.
>>>>>>
>>>>>> On Wed, Jul 22, 2015 at 12:31 PM, Isuru Haththotuwa <
>>>>>> isu...@apache.org> wrote:
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Tue, Jul 21, 2015 at 10:51 PM, Gayan Gunarathne <gay...@wso2.com>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> Hi Asanka,
>>>>>>>>
>>>>>>>> We can check further with their forum[1]
>>>>>>>>
>>>>>>>> Also did you check with the [2]. Seems there are inbuilt health
>>>>>>>> checking option.If they don't provide option to get in flight request
>>>>>>>> count,IMO we can use their inbuilt health checking option.
>>>>>>>>
>>>>>>>> [1] https://groups.google.com/forum/#!forum/gce-discussion
>>>>>>>> [2]
>>>>>>>> https://cloud.google.com/compute/docs/load-balancing/health-checks
>>>>>>>>
>>>>>>> Yes. If GCE API does not provide a way to get this information, we
>>>>>>> have to look in to alternate methods. Not sure how the health check 
>>>>>>> works
>>>>>>> here, but maybe its just doing a port check. If so, we won't be able to
>>>>>>> directly use in Autoscaling decision making. But that is fine IMO, we 
>>>>>>> can
>>>>>>> only support what is available.
>>>>>>>
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>> Gayan
>>>>>>>>
>>>>>>>> On Tue, Jul 21, 2015 at 8:22 PM, Asanka Sanjaya Herath <
>>>>>>>> angal...@gmail.com> wrote:
>>>>>>>>
>>>>>>>>> Hi Isuru,
>>>>>>>>>
>>>>>>>>> Unfortunately I was not able to figure out a way to do this. I
>>>>>>>>> really appreciate if you can help me with this.
>>>>>>>>>
>>>>>>>>> On Tue, Jul 21, 2015 at 8:20 PM, Isuru Haththotuwa <
>>>>>>>>> isu...@apache.org> wrote:
>>>>>>>>>
>>>>>>>>>> Hi Asanka,
>>>>>>>>>>
>>>>>>>>>> On Sat, Jul 18, 2015 at 10:13 AM, Asanka Sanjaya Herath <
>>>>>>>>>> angal...@gmail.com> wrote:
>>>>>>>>>>
>>>>>>>>>>> Thanks Isuru. Currently I'm fixing an issue related to member
>>>>>>>>>>> termination. Meanwhile I'm struggling to find a way to get "in 
>>>>>>>>>>> flight
>>>>>>>>>>> request count" from GCE. I'll do some more research and update the 
>>>>>>>>>>> thread.
>>>>>>>>>>>
>>>>>>>>>> Were you able to find a way to get the In Flight Request count
>>>>>>>>>> from GCE APIs? Please share your findings.
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> On Sat, Jul 18, 2015 at 8:33 PM, Isuru Haththotuwa <
>>>>>>>>>>> isu...@apache.org> wrote:
>>>>>>>>>>>
>>>>>>>>>>>> Great work so far Asanka!
>>>>>>>>>>>>
>>>>>>>>>>>> On Wed, Jul 15, 2015 at 6:01 AM, Asanka Sanjaya Herath <
>>>>>>>>>>>> angal...@gmail.com> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>>> Thanks Gayan for the feedback. As we discussed on hangout I
>>>>>>>>>>>>> have done some screen recoding about the LB extension that I have 
>>>>>>>>>>>>> created.
>>>>>>>>>>>>> I have published that on YouTube. Here is the link.
>>>>>>>>>>>>> https://youtu.be/dAWYQL1Vleo .
>>>>>>>>>>>>>
>>>>>>>>>>>>> On Mon, Jul 13, 2015 at 12:25 PM, Gayan Gunarathne <
>>>>>>>>>>>>> gay...@wso2.com> wrote:
>>>>>>>>>>>>>
>>>>>>>>>>>>>> Thanks Asanka for the progress update.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Nice to hear that you are writing the blog on this. Please
>>>>>>>>>>>>>> share the blog post once you have done.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Yeah now you can work on the statistic publishing part.Also
>>>>>>>>>>>>>> please share us the information about how do you plan to gather 
>>>>>>>>>>>>>> GCE related
>>>>>>>>>>>>>> statistics for your extension.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Thanks,
>>>>>>>>>>>>>> Gayan
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> On Mon, Jul 6, 2015 at 6:12 PM, Asanka Sanjaya Herath <
>>>>>>>>>>>>>> angal...@gmail.com> wrote:
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Hi,
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Last week I implemented a way to read configuration from a
>>>>>>>>>>>>>>> configuration file instead of reading from system arguments as 
>>>>>>>>>>>>>>> we discussed
>>>>>>>>>>>>>>> in last hangout [1]. And I did some testing related to topology 
>>>>>>>>>>>>>>> issue and
>>>>>>>>>>>>>>> end to end testing as well. I have started writing blog post 
>>>>>>>>>>>>>>> related to
>>>>>>>>>>>>>>> extension as we discussed in last hangout. I'm willing to do 
>>>>>>>>>>>>>>> the statistics
>>>>>>>>>>>>>>> publishing part in this week and I'm supposed to recode a demo 
>>>>>>>>>>>>>>> too.
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> [1].
>>>>>>>>>>>>>>> https://github.com/apache/stratos/commit/d8256222b959721763354e8149a6f975dd8e3d76
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> On Mon, Jun 29, 2015 at 2:58 PM, Asanka Sanjaya Herath <
>>>>>>>>>>>>>>> angal...@gmail.com> wrote:
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Hi All,
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Last week I was working on debugging, testing and finding
>>>>>>>>>>>>>>>> an error related to topology as we discussed in hangout. After 
>>>>>>>>>>>>>>>> debugging I
>>>>>>>>>>>>>>>> opened a discussion thread in dev list in order to discuss 
>>>>>>>>>>>>>>>> findings. And I
>>>>>>>>>>>>>>>> added a timeout for operation completions [1]. And as we 
>>>>>>>>>>>>>>>> discussed in
>>>>>>>>>>>>>>>> hangout I did some changes in message processor side in order 
>>>>>>>>>>>>>>>> to expose the
>>>>>>>>>>>>>>>> instance ID of the member to other components too [2].
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> [1].
>>>>>>>>>>>>>>>> https://github.com/apache/stratos/commit/9df565f1296707165e96755ca6a0b1c37bea53b1
>>>>>>>>>>>>>>>> [2].
>>>>>>>>>>>>>>>> https://github.com/apache/stratos/commit/9f44791884a54499997a0e5d00f7a4b46047b3b2
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> On Tue, Jun 23, 2015 at 2:09 PM, Imesh Gunaratne <
>>>>>>>>>>>>>>>> im...@apache.org> wrote:
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> On Mon, Jun 22, 2015 at 4:00 PM, Asanka Sanjaya Herath <
>>>>>>>>>>>>>>>>> angal...@gmail.com> wrote:
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> Hi All,
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> We had the hangout today and we discussed on following.
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> Participants: IsuruH, GayanG, Imesh, Asanka
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>>>> Imesh Gunaratne
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> Senior Technical Lead, WSO2
>>>>>>>>>>>>>>>>> Committer & PMC Member, Apache Stratos
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>>> Thanks,
>>>>>>>>>>>>>>>> Regards,
>>>>>>>>>>>>>>>> ASH
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>> Thanks,
>>>>>>>>>>>>>>> Regards,
>>>>>>>>>>>>>>> ASH
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Gayan Gunarathne
>>>>>>>>>>>>>> Technical Lead, WSO2 Inc. (http://wso2.com)
>>>>>>>>>>>>>> Committer & PMC Member, Apache Stratos
>>>>>>>>>>>>>> email : gay...@wso2.com  | mobile : +94 766819985
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> --
>>>>>>>>>>>>> Thanks,
>>>>>>>>>>>>> Regards,
>>>>>>>>>>>>> ASH
>>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> --
>>>>>>>>>>>> Thanks and Regards,
>>>>>>>>>>>>
>>>>>>>>>>>> Isuru H.
>>>>>>>>>>>> +94 716 358 048* <http://wso2.com/>*
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> --
>>>>>>>>>>> Thanks,
>>>>>>>>>>> Regards,
>>>>>>>>>>> ASH
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>> Thanks and Regards,
>>>>>>>>>>
>>>>>>>>>> Isuru H.
>>>>>>>>>> +94 716 358 048* <http://wso2.com/>*
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>> Thanks,
>>>>>>>>> Regards,
>>>>>>>>> ASH
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> --
>>>>>>>>
>>>>>>>> Gayan Gunarathne
>>>>>>>> Technical Lead, WSO2 Inc. (http://wso2.com)
>>>>>>>> Committer & PMC Member, Apache Stratos
>>>>>>>> email : gay...@wso2.com  | mobile : +94 766819985
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Thanks and Regards,
>>>>>>>
>>>>>>> Isuru H.
>>>>>>> +94 716 358 048* <http://wso2.com/>*
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Thanks,
>>>>>> Regards,
>>>>>> ASH
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Thanks,
>>>>> Regards,
>>>>> ASH
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Thanks,
>>>> Regards,
>>>> ASH
>>>>
>>>
>>>
>>>
>>> --
>>>
>>> Gayan Gunarathne
>>> Technical Lead, WSO2 Inc. (http://wso2.com)
>>> Committer & PMC Member, Apache Stratos
>>> email : gay...@wso2.com  | mobile : +94 775030545 <%2B94%20766819985>
>>>
>>>
>>>
>>
>>
>>
>> --
>> Thanks,
>> Regards,
>> ASH
>>
>
>
>
> --
> Thanks,
> Regards,
> ASH
>



-- 
Thanks and Regards,

Isuru H.
+94 716 358 048* <http://wso2.com/>*

Reply via email to