Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi Salim,
We started utilizing test-kitchen and kitchen-ec2 quite heavily, as it makes our job so much easier to test and verify our Windows cookbooks properly in Amazon. So here's my new batch of contribution for the Windows / WinRm support to your review - this time for test-kitchen. :)
The main change is that - I've realized during
kitchen verify
executions, thatbusser_sync
method is responsible for pushing the test files, which breaks with WinRM, if the test files are getting larger - due to the command length limitations of the WinRM service. So my idea was to move them with the standard transport layer (both for ssh and winrm) and letbusser_sync
only do the preparation on the already transported files from a temp folder.The changes namely:
Exit
commandkitchen setup
repeatable by removing and reinstalling pluginsI've tested the changes both with SSH and WinRM. Thank you again for your work and of course for your time reviewing this! :)
Have a great day and Regards,
Akos