Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement JENKINS-38269 #54

Merged
merged 2 commits into from
Sep 22, 2016
Merged

Conversation

pjdarton
Copy link
Member

Enhances existing vSphere cloud-template slave functionality to allow the launcher method to be configurable (allowing one to have a Java Web Start slave instead of SSHLauncher, for example), and/or to choose a a retention-policy of "use until it's idle" as an alternative to "use once".

launch method used by cloud template-based slaves.
Added help text for numerous cloud-template fields.
Improved description in launchDelay field's help text.
@pjdarton
Copy link
Member Author

Yes you can - That's what the (new) VSphereCloudRetentionStrategy does :-)

Prior to this enhancement, the only option available for cloud provisioned
slaves was the
RunOnceCloudRetentionStrategy which only runs a single build on the slave
before destroying it.
With this enhancement, you now have the option of choosing a
VSphereCloudRetentionStrategy
which keeps the slave alive after the build and only destroys it if it
remains idle for more than a period of time.
I chose a default of 5 minutes idle time, but you could configure it to be
20 minutes if you prefer.

On 18 September 2016 at 10:06, segator [email protected] wrote:

could you add an idle timeout? as a retention-policy?

Example:
wait to destroy slave 20 min if is in idle, this allow an other job to
reuse slave and avoid to deploy an other machine.
if the slave is idle 20 min automatically be destroyed.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#54 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8F2bsKtCeMyMN8r6nYh4ldOknbgCgKks5qrP8jgaJpZM4J_Hjw
.

@jswager jswager merged commit f5c2d4c into jenkinsci:master Sep 22, 2016
@pjdarton pjdarton deleted the implement_jenkins_38269 branch September 5, 2017 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants