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

Always run chef install even if the lock file exists. #1103

Merged
merged 4 commits into from
Aug 20, 2016

Conversation

coderanger
Copy link
Contributor

This is required to make sure the cookbooks are cached before we run
chef export in CI or other non-workstation environments.

Also refactored things a tad.

This is required to make sure the cookbooks are cached before we run
`chef export` in CI or other non-workstation environments.
@mwrock
Copy link
Member

mwrock commented Aug 20, 2016

👍 on green

@coderanger
Copy link
Contributor Author

Not going to do it in this PR, but we should probably consider having an option to run chef update as part of the kitchen run automatically, would be good for development flows I think. /cc @danielsdeleo

@coderanger coderanger merged commit f77ea22 into test-kitchen:master Aug 20, 2016
@coderanger coderanger deleted the policyinstall branch August 20, 2016 01:16
coderanger added a commit to poise/yolover-example that referenced this pull request Aug 20, 2016
@danielsdeleo
Copy link
Contributor

@coderanger I think what I'd like is an opt- in for the chef update where we configure it to be enabled by default in the generators. That way, most people get auto updates, but it's easier to discover why it happens and easy to turn off.

Anyway thanks for this fix.

@coderanger
Copy link
Contributor Author

That sounds about right :) Will poke it later tonight after mayhaps.

@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.

4 participants