From dev-return-8909-apmail-continuum-dev-archive=continuum.apache.org@continuum.apache.org Thu Dec 09 02:24:53 2010 Return-Path: Delivered-To: apmail-continuum-dev-archive@www.apache.org Received: (qmail 61590 invoked from network); 9 Dec 2010 02:24:53 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 9 Dec 2010 02:24:53 -0000 Received: (qmail 48981 invoked by uid 500); 9 Dec 2010 02:24:53 -0000 Delivered-To: apmail-continuum-dev-archive@continuum.apache.org Received: (qmail 48930 invoked by uid 500); 9 Dec 2010 02:24:53 -0000 Mailing-List: contact dev-help@continuum.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@continuum.apache.org Delivered-To: mailing list dev@continuum.apache.org Received: (qmail 48922 invoked by uid 99); 9 Dec 2010 02:24:53 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 09 Dec 2010 02:24:53 +0000 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: local policy) Received: from [72.26.105.223] (HELO cell.g2ix.net) (72.26.105.223) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 09 Dec 2010 02:24:47 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by cell.g2ix.net (Postfix) with ESMTP id 6BCFE730003 for ; Wed, 8 Dec 2010 18:24:27 -0800 (PST) X-Virus-Scanned: amavisd-new at cell.g2ix.net Received: from cell.g2ix.net ([127.0.0.1]) by localhost (cell.g2ix.net [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id KYQGxSdvt9QX for ; Wed, 8 Dec 2010 18:24:23 -0800 (PST) Received: from cell.g2ix.net (cell [172.18.10.223]) by cell.g2ix.net (Postfix) with ESMTP id 3CF41730002 for ; Wed, 8 Dec 2010 18:24:23 -0800 (PST) Date: Wed, 8 Dec 2010 18:24:23 -0800 (PST) From: Mark John Kennedy Magallanes To: dev@continuum.apache.org Message-ID: <1036295103.1549.1291861463151.JavaMail.root@cell.g2ix.net> In-Reply-To: <915A8671-75EF-4D1E-ADA3-FB6CF6B9972A@apache.org> Subject: Re: proposal to add platform field in the model used for build agents for continuum master MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Originating-IP: [58.71.19.178] X-Mailer: Zimbra 6.0.5_GA_2213.RHEL5_64 (ZimbraWebClient - FF3.0 (Linux)/6.0.5_GA_2213.RHEL5_64) hi Brett, Point taken will now file an issue for this, thanks all. Thanks Mark Magallanes ----- Original Message ----- From: "Brett Porter" To: dev@continuum.apache.org Sent: Thursday, December 9, 2010 10:04:21 AM Subject: Re: proposal to add platform field in the model used for build agents for continuum master Loks fine, though you might even want to replace ping with an "extended status" request, so you only need to send one req for both. - Brett On 09/12/2010, at 12:33 PM, Mark John Kennedy Magallanes wrote: > Hi all, > > Point taken Wendy, will not touch the enabled part so that manually disabling would > still be there on a side note the suggestion was only when adding it. you can still be > able edit the build agent though and change the enabled state. > > case scenario: when adding > -add build agent using UI > > back-end implementation > > pings the newly enter build agent > > *if ping successful > enable = true > get the platform (may also include what brett mention earlier to include installations) > > *if ping not successful > enabled = false > > case scenario: > -update build agent using UI > > back-end implementation > pings build agent again > > *if ping successful > enable = true > **if platform is empty > get the platform (may also include what brett mention earlier to include installations) > > *if ping not successful > enabled = false > > Thanks Brett and Wendy for your ideas, would the case scenarios mention above work for you? > > > Thanks, > > Mark Magallanes -- Brett Porter brett@apache.org http://brettporter.wordpress.com/ http://au.linkedin.com/in/brettporter