ec2-plugin icon indicating copy to clipboard operation
ec2-plugin copied to clipboard

[JENKINS-47681] Don't count slaves we just provisioned multiple times

Open marcmac opened this issue 8 years ago • 2 comments

When provisioning several on demand slaves, provisionOndemand() can return the same slave multiple times while it is launching. Keep a list of already provisioned slaves, and exclude them from the available pool when asking for more slaves.

marcmac avatar Mar 07 '18 23:03 marcmac

I would like to apply this change over PR252 and test it as this PR will probably fix a similar issue we are facing atm.

I tried to cherry-pick this commit but there are loads of conflicts, which I don't pretty understand which ones to keep.

OliverPereira avatar May 17 '18 08:05 OliverPereira

@OliverPereira I'm using this internally, but have not had time to track down the test failures that blew up my pull request. I plan to have someone address this in the next few weeks, and can then get this integrated.

marcmac avatar May 29 '18 20:05 marcmac