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

Include definitions directory when uploading the cookbooks #157

Closed
wants to merge 1 commit into from
Closed

Include definitions directory when uploading the cookbooks #157

wants to merge 1 commit into from

Conversation

jasonroelofs
Copy link
Contributor

kitchen converge was not including the definitions directory when uploading my cookbooks into vagrant. This fixes that problem.

I tried to get cp_this_cookbook to use cookbook_files_glob to reduce redundancies but was unable to get it working right.

@fnichol
Copy link
Contributor

fnichol commented Jul 17, 2013

Ha, I can't believe I forgot about definitions, whoops.

Do you have an Opscode CLA signed by chance?

@jsm
Copy link

jsm commented Jul 23, 2013

When will this be available in rubygems?

@jasonroelofs
Copy link
Contributor Author

Sorry for the delay. @fnichol I do have a CLA signed.

@fnichol
Copy link
Contributor

fnichol commented Jul 23, 2013

@jasonroelofs excellent thank you! I'm just waiting on the Opscode page to update then this is in!

@sanmiguelje if not today then this pull request should be in the next couple of days 😄

@fnichol fnichol closed this in 1672b18 Aug 6, 2013
@fnichol
Copy link
Contributor

fnichol commented Aug 6, 2013

@jasonroelofs Thanks again! Unfortunately the source of the fix moved, but I credited you in the currently location with a new commit (hope that's okay).

BrentOnRails pushed a commit to BrentOnRails/test-kitchen that referenced this pull request Jul 17, 2017
Adding block_device_mapping iops parameter, fixes test-kitchen#157
@test-kitchen test-kitchen locked and limited conversation to collaborators Nov 16, 2017
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.

3 participants