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 [email protected] http://brettporter.wordpress.com/ http://au.linkedin.com/in/brettporter
