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

gracefully handle winrm errors #1872

Merged

Conversation

jasonwbarnett
Copy link
Contributor

@jasonwbarnett jasonwbarnett commented Mar 1, 2022

Description

This change allows for gracefully handling odd scenarios such as the Chef Infra Client convergence restarting the winrm service mid convergence or reconfiguring, etc.

# kitchen.yml

provisioner:
  name: chef_zero
  max_retries: 6
  wait_for_retry: 10

transport:
  name: winrm
  connection_retries: 180
  connection_retry_sleep: 5

This magic config ^^ plus these changes resolved all of our issues. In other words it made test-kitchen resilient to random windows server reboots and random winrm service reboots.

Issues Resolved

N/A

Check List

  • All tests pass. See TESTING.md for details.
  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@jasonwbarnett jasonwbarnett force-pushed the feature/gracefully-handle-winrm-500 branch 7 times, most recently from 03eeb4e to de2f368 Compare March 2, 2022 01:21
@jasonwbarnett jasonwbarnett force-pushed the feature/gracefully-handle-winrm-500 branch 6 times, most recently from cc5b698 to 2e8df0a Compare March 2, 2022 16:31
Signed-off-by: Jason Barnett <jason.w.barnett@gmail.com>
@jasonwbarnett jasonwbarnett force-pushed the feature/gracefully-handle-winrm-500 branch from 2e8df0a to ba40ef3 Compare March 2, 2022 16:34
@jasonwbarnett jasonwbarnett changed the title gracefully handle winrm 500 errors gracefully handle winrm errors Mar 2, 2022
@jasonwbarnett jasonwbarnett marked this pull request as ready for review March 2, 2022 16:35
@tas50 tas50 merged commit d907b98 into test-kitchen:main Mar 3, 2022
@jasonwbarnett jasonwbarnett deleted the feature/gracefully-handle-winrm-500 branch March 3, 2022 22:27
@ashiqueps ashiqueps mentioned this pull request Jun 10, 2022
3 tasks
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.

None yet

3 participants