Hi Isuru, Thanks I'll look into it.
On Sat, Aug 8, 2015 at 5:37 AM, Isuru Haththotuwa <isu...@apache.org> wrote: > 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/>* > > > -- Thanks, Regards, ASH