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

Test-kitchen support #114

Closed
josephholsten opened this issue May 27, 2013 · 6 comments
Closed

Test-kitchen support #114

josephholsten opened this issue May 27, 2013 · 6 comments
Milestone

Comments

@josephholsten
Copy link

To get ourselves a safety net, we need automated testing for this cookbook. Test-kitchen has the most momentum behind it today, and what it lacks in multi-node testing it mostly makes up for in tooling and support.

If we discover that we need multi-node support before it gets added to tk, we should consider chef-workflow.

@josephholsten
Copy link
Author

I vaguely remember someone mentioning a preference to minitest over bats, but I only have experience with bats and couldn't find any examples of test-kitchen + minitest use. I'll happily convert to minitest if someone points me at an example.

@jamesonjlee
Copy link

I liked mini-test more because I was told it was able to introspect the node variables. Here is riak's minitest basho-labs/riak-chef-cookbook#66

a possible way to do multi-node setup with tk. Looks like it basically spins up all nodes and wait for everything to converge and then runs a (integration) test.
https://github.com/bryanwb/tk-demo

@josephholsten
Copy link
Author

Awesome! I'll review and update the patch.

@josephholsten
Copy link
Author

BTW: I'm using opscode's current recommendation of a provisioner-less base box along with the vagrant-omnibus to install chef before the converge phase. This makes it more identical to a standard bootstrap approach.

If we start having issues supporting legacy chef installs, it may make sense to have separate boxes for say chef-10 and chef-11 clients. But as the title of this PR implies, I'm just trying to get something minimal in place.

@jamesonjlee
Copy link

yes.

@josephholsten
Copy link
Author

Oooh, I heard multi-node was in the works. Very nice.

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

No branches or pull requests

2 participants