Skip to content

[JENKINS-44796] vSphereCloudLauncher enhancement#75

Merged
pjdarton merged 1 commit intojenkinsci:masterfrom
pjdarton:launcher_improvement
Jul 28, 2017
Merged

[JENKINS-44796] vSphereCloudLauncher enhancement#75
pjdarton merged 1 commit intojenkinsci:masterfrom
pjdarton:launcher_improvement

Conversation

@pjdarton
Copy link
Copy Markdown
Member

vSphereCloudLauncher no longer connects to vSphere when its "idleAction" is "nothing". It still connects as normal if it needs to, it just doesn't bother connecting when it's not going to use the connection.
It also now calls VSphere's disconnect method from a finally {...} block to ensure that it will always disconnect from vSphere if it had previously connected; before, it could remain connected if exceptions happened.

@pjdarton pjdarton force-pushed the launcher_improvement branch from fe0570c to 0361263 Compare June 28, 2017 15:33
vSphereCloudLauncher no longer connects to vSphere when its "idleAction" is
"nothing".  It still connects as normal if it needs to, it just doesn't bother
connecting when it's not going to use the connection.
It also now calls VSphere's disconnect method from a finally {...} block to
ensure that it will always disconnect from vSphere if it had previously
connected; before, it could remain connected if exceptions happened.
@pjdarton pjdarton force-pushed the launcher_improvement branch from 0361263 to aa4ba8c Compare June 28, 2017 15:36
@pjdarton pjdarton changed the title vSphereCloudLauncher enhancement [JENKINS-44796] vSphereCloudLauncher enhancement Jun 29, 2017
@pjdarton pjdarton merged commit 421fbe6 into jenkinsci:master Jul 28, 2017
@pjdarton pjdarton deleted the launcher_improvement branch July 28, 2017 12:28
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.

1 participant