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>