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.

Sign up for GitHub

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

Jump to bottom

Refactor winrm communicator to use latest winrm gems and v2 api #8102

Merged
merged 1 commit into from
Jan 5, 2017
Merged

Refactor winrm communicator to use latest winrm gems and v2 api #8102

merged 1 commit into from
Jan 5, 2017

Conversation

mwrock
Copy link
Contributor

@mwrock mwrock commented Dec 9, 2016 •
edited
Loading

This updates the winrm dependencies to the latest versions. The latest versions address several bugs and introduce some enhancements. Here are the highlights

  • Powershell commands use the PSRP protocol which lifts the 8k command line limitation and adds compatibility with Windows Nano server.
  • Due to the above, winrm-fs is orders of magnitude faster and more reliable
  • The elevated shell via scheduled task logic now lives in winrm-elevated which is also used by other projects
  • Updated winrm configuration documentation.

cc @sneal

Signed-off-by: Matt Wrock matt@mattwrock.com

@mwrock mwrock mentioned this pull request Dec 9, 2016
Closed
@chrisroberts chrisroberts mentioned this pull request Dec 9, 2016
Closed
@hedgehog hedgehog mentioned this pull request Dec 10, 2016
Closed
@mwrock
Copy link
Contributor Author

mwrock commented Dec 11, 2016

Fixes #7626, #7717, #7526, #8050, #7476, #7953

@mwrock mwrock changed the title WIP: refactor winrm communicator to use latest winrm gems and v2 api Refactor winrm communicator to use latest winrm gems and v2 api Dec 11, 2016
@mwrock
refactor winrm communicator to use latest winrm gems and v2 api
725824e 
Signed-off-by: Matt Wrock <matt@mattwrock.com>
chrisroberts
chrisroberts approved these changes Dec 13, 2016
View reviewed changes
Copy link
Member

@chrisroberts chrisroberts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mwrock This is amazing! 👍

@chrisroberts chrisroberts merged commit c11534e into hashicorp:master Jan 5, 2017
taylormonacelli pushed a commit to gkwa/vagrant-provision-windows that referenced this pull request Feb 1, 2017
Workaround provision problem on Win10
d7c5fcd 
hashicorp/vagrant#8102
hashicorp/vagrant#7717
https://github.com/mitchellh/vagrant/tree/master/plugins/communicators
taylormonacelli pushed a commit to gkwa/vagrant-provision-windows that referenced this pull request Feb 1, 2017
Workaround provision problem on Win10
b017a3b 
hashicorp/vagrant#8102
hashicorp/vagrant#7717
https://github.com/mitchellh/vagrant/tree/master/plugins/communicators
taylormonacelli pushed a commit to gkwa/vagrant-provision-windows that referenced this pull request Feb 1, 2017
Workaround provision problem on Win10
547718d 
hashicorp/vagrant#8102
hashicorp/vagrant#7717
https://github.com/mitchellh/vagrant/tree/master/plugins/communicators
taylormonacelli pushed a commit to gkwa/vagrant-provision-windows that referenced this pull request Feb 1, 2017
Workaround provision problem on Win10
a80e6e6 
hashicorp/vagrant#8102
hashicorp/vagrant#7717
https://github.com/mitchellh/vagrant/tree/master/plugins/communicators
taylormonacelli pushed a commit to gkwa/vagrant-provision-windows that referenced this pull request Feb 1, 2017
Workaround provision problem on Win10
9d812e2 
hashicorp/vagrant#8102
hashicorp/vagrant#7717
https://github.com/mitchellh/vagrant/tree/master/plugins/communicators
taylormonacelli pushed a commit to gkwa/vagrant-provision-windows that referenced this pull request Feb 1, 2017
Workaround provision problem on Win10
e915b5b 
hashicorp/vagrant#8102
hashicorp/vagrant#7717
https://github.com/mitchellh/vagrant/tree/master/plugins/communicators
This was referenced Mar 2, 2017
Closed
Closed
Closed
@ghost ghost locked and limited conversation to collaborators Apr 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@chrisroberts chrisroberts chrisroberts approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mwrock @chrisroberts