Return-Path: X-Original-To: apmail-incubator-cloudstack-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-cloudstack-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 26CB4E970 for ; Tue, 29 Jan 2013 17:25:13 +0000 (UTC) Received: (qmail 54523 invoked by uid 500); 29 Jan 2013 17:25:12 -0000 Delivered-To: apmail-incubator-cloudstack-dev-archive@incubator.apache.org Received: (qmail 54449 invoked by uid 500); 29 Jan 2013 17:25:12 -0000 Mailing-List: contact cloudstack-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: cloudstack-dev@incubator.apache.org Delivered-To: mailing list cloudstack-dev@incubator.apache.org Received: (qmail 54403 invoked by uid 99); 29 Jan 2013 17:25:10 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 29 Jan 2013 17:25:10 +0000 X-ASF-Spam-Status: No, hits=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of rajesh.battala@citrix.com designates 203.166.19.134 as permitted sender) Received: from [203.166.19.134] (HELO SMTP.CITRIX.COM.AU) (203.166.19.134) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 29 Jan 2013 17:25:04 +0000 X-IronPort-AV: E=Sophos;i="4.84,561,1355097600"; d="scan'208";a="593721" Received: from banpmailmx02.citrite.net ([10.103.128.74]) by SYDPIPO01.CITRIX.COM.AU with ESMTP/TLS/RC4-MD5; 29 Jan 2013 17:24:42 +0000 Received: from BANPMAILBOX01.citrite.net ([10.103.128.71]) by BANPMAILMX02.citrite.net ([10.103.128.74]) with mapi; Tue, 29 Jan 2013 22:54:39 +0530 From: Rajesh Battala To: "cloudstack-dev@incubator.apache.org" Date: Tue, 29 Jan 2013 22:54:25 +0530 Subject: RE: Unit Tests for LoadBalancer functionality in ASF Master Thread-Topic: Unit Tests for LoadBalancer functionality in ASF Master Thread-Index: Ac3+QucItaChnACFTnmCzxw/ynh9QwAAoByg Message-ID: <97F4356AEA71904482CD192135C038F9011CC0D20E7C@BANPMAILBOX01.citrite.net> References: <97F4356AEA71904482CD192135C038F9011CC0D20E6F@BANPMAILBOX01.citrite.net> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-Virus-Checked: Checked by ClamAV on apache.org > -----Original Message----- > From: Chip Childers [mailto:chip.childers@sungard.com] > Sent: Tuesday, January 29, 2013 10:35 PM > To: cloudstack-dev@incubator.apache.org > Subject: Re: Unit Tests for LoadBalancer functionality in ASF Master >=20 > On Tue, Jan 29, 2013 at 11:20 AM, Rajesh Battala > wrote: > > Hi All, > > > > I have almost done with my feature (AWS Style healthchecks for NetScale= r). > > I have got paused, how to write unit tests for healthchecks. > > > > Under cloud-server/test I didn't find any tests for LoadBalancer. Ther= e are not > unit tests which tests the functionality of LB for VR, NetScaler, F5. > > I didn't' find any guide/wiki how to write the tests for network servic= es( Ex for > LB\healthchecks). > > > > As there are only 2 days left for the feature code freeze, Writing a fr= amework > for entire LoadBalancer is not possible. > > > > It would be great if anyone writes unit tests for the network services,= so that > later for any features/bug-fixes we can keep contributing to the unit tes= ts. > > > > Can I go ahead and submit my feature for code-review and once basic uni= t > tests framework is available for networking I can add unittest for Health= Check > feature. > > > > This is holding me to submit my feature for the review. Please let me k= now > how to go about. > > > > > > Thanks > > Rajesh Battala > > >=20 > I think your concerns are valid. Go ahead and submit for review. > Perhaps you can spend a little time looking into how to best unit test > LoadBalancer code while the feature is under review? []=20 Thanks a lot Chip. Will submit my feature for Review by tomorrow.=20 Thanks Rajesh Battala