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