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

Turn auto-retries on by default for Chef provisioners #1310

Merged
merged 2 commits into from
Nov 6, 2017

Conversation

coderanger
Copy link
Contributor

I think that's the right codes from the RFC, but easy to change.

Copy link
Contributor

@btm btm left a comment

Choose a reason for hiding this comment

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

Woo, definitely interested in getting 35 added to the default. Lot's of people struggle with getting these values set right in their .kitchen.yml.

I'm not sure about 37. It should be set when the system needs a reboot to complete an operation, but that often doesn't indicate it is going to happen any time soon. I don't think I would included it.

@coderanger
Copy link
Contributor Author

Roger, I'll leave 213 in there for some future when Chef can self-upgrade even though testing that should probably not happen in TK.

…machine needs to reboot but isn't going to any time soon, which is probably a recipe error.
@coderanger
Copy link
Contributor Author

Ignoring appveryor because it looks jammed on something.

@coderanger coderanger merged commit 782f3f7 into test-kitchen:master Nov 6, 2017
@coderanger coderanger deleted the retry-default branch November 6, 2017 22:52
@test-kitchen test-kitchen locked and limited conversation to collaborators Mar 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants