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

[JENKINS-27260] SPNEGO for Windows in EC2 Plugin #138

Merged
merged 1 commit into from
Apr 11, 2015
Merged

[JENKINS-27260] SPNEGO for Windows in EC2 Plugin #138

merged 1 commit into from
Apr 11, 2015

Conversation

gmittert
Copy link

@gmittert gmittert commented Mar 6, 2015

As per Jenkins-27260:

JENKINS-25385 and JENKINS-4995 both have comments complaining about infinite loops when creating Windows slaves. Because SPNEGO is unregistered for, the httpclient throws an exception that is silently
caught and causes the infinite loop.

I was able to connect to WinRM, but I had to re enable SPNEGO. I'm not sure what repercussions that will have otherwise, I'm hoping someone can comment on that.

I also found that the tmpDir was getting set to "", causing a failure, so I added in a case for that.

Fixed the Windows temp directory getting set to ""
Enabled SPNEGO authentication

JENKINS-25385 and JENKINS-4995 both have comments complaining about
infinite loops when creating Windows slaves. Because SPNEGO is
unregistered for, the httpclient throws and exception that is silently
caught and causes the infinite loop.
@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@buildhive
Copy link

Jenkins » ec2-plugin #90 SUCCESS
This pull request looks good
(what's this?)

@rsennewald
Copy link
Collaborator

We're using this in production at my workplace and it's working perfectly for Windows AMIs to be spun up. @francisu would you mind taking a look and merging if all seems well?

francisu added a commit that referenced this pull request Apr 11, 2015
[JENKINS-27260] SPNEGO for Windows in EC2 Plugin
@francisu francisu merged commit 1ba2206 into jenkinsci:master Apr 11, 2015
@francisu
Copy link
Member

Thanks for your contribution!

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.

5 participants