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

Fixes #29187 - using winrm on EC2 #29205

Merged
merged 1 commit into from Nov 25, 2015

Conversation

Projects
None yet
3 participants
@trevor-h
Contributor

trevor-h commented Nov 25, 2015

Ensures that use_winrm: True is respected when bootstrapping with salt-cloud on EC2. Adds the option to specify winrm port to use with 'winrm_port' option in cloud configuration. This is optional and defaults to WinRM 2.0 HTTP port tcp/5986. Only HTTPS (as before) is supported.

There is also a documentation update with a working example using default EC2 Windows images. The existing examples don't account for opening WinRM on the Windows firewall or ensuring auth type is basic.

Additionally the example uses the following Technet 'trick':
http://social.technet.microsoft.com/wiki/contents/articles/28753.powershell-trick-copy-certificates-from-one-store-to-another.aspx

This allows you to enable HTTPS as previously there was no store available on EC2 Windows images resulting in an error to enable HTTPS. This borrows from RDP.

cachedout added a commit that referenced this pull request Nov 25, 2015

@cachedout cachedout merged commit fffcf9f into saltstack:2015.8 Nov 25, 2015

4 of 6 checks passed

jenkins/salt-pr-rs-cent6-n Salt PR - RS CentOS 6 #227 — FAILURE
Details
default Merged build started.
Details
jenkins/salt-pr-clone Salt PR - Clone Repository #11515 — SUCCESS
Details
jenkins/salt-pr-linode-ubuntu14.04-n Salt PR - Linode Ubuntu 14.04 #2582 — SUCCESS
Details
jenkins/salt-pr-lint-n Salt PR - Code Lint #11223 — SUCCESS
Details
jenkins/salt-pr-rs-cent7-n Salt PR - RS CentOS 7 #10052 — SUCCESS
Details
@cachedout

This comment has been minimized.

Show comment
Hide comment
@cachedout

cachedout Nov 25, 2015

Contributor

Looks great! Thanks, @trevor-h

Contributor

cachedout commented Nov 25, 2015

Looks great! Thanks, @trevor-h

@jfindlay

This comment has been minimized.

Show comment
Hide comment
@jfindlay

jfindlay Nov 25, 2015

Contributor

Fixes #29187.

Contributor

jfindlay commented Nov 25, 2015

Fixes #29187.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment