VSTS IsHostedAgent detection is flawed

When running CakeBuild in VSTS and rely on Context.TFBuild().IsRunningOnVSTS. However we have 2 hosted build agents and second build agent is called “Hosted Agent 2”. And build is always failing on the second agent.

Here code insists that agent name is exactly “Hosted Agent”.

Also We are setting up private build agent and this code does not agree with private build agents. It is still VSTS build, only not run in hosted agent.

I propose to change the check for the agent name to .StartsWith("Hosted Agent") and remove check for IsHostedAgent from IsRunningOnVSTS option.

@agc93 what do you think? (as far as I remember you have done work on this)

I can do PR for this.

[kevbite]

We’ve just stumbled across this issue too since we’ve added a couple more hosted agents to our team. As a workaround we’re just checking that we’ve got a agent name:

var isRunningOnVSTS = !string.IsNullOrEmpty(EnvironmentVariable("AGENT_NAME"));

[agc93]

Yep, I’ll have a look at this (a few different parts of the logic need improvements)