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

[SHACK-250] Pull in new chef-apply gem #228

Merged
merged 4 commits into from
Jul 10, 2018
Merged

Conversation

tyler-ball
Copy link
Contributor

@tyler-ball tyler-ball commented Jul 9, 2018

Description

The chef-run gem was renamed to chef-apply. This PR first removes the existing component since it is no longer needed, then changes the omnibus definition to pull in the new gem instead.

Issues Resolved

SHACK-250

Check List

  • New functionality includes tests
  • All tests pass
  • PR title is a worthy inclusion in the CHANGELOG
  • You have locally validated the change
  • www/site/content/docs/ has been updated with any relevant changes:
      • new or changed error messages in 'troubleshooting.md'
      • new or changed CLI flags in cli-reference.md

@tyler-ball tyler-ball requested a review from a team July 9, 2018 19:26
@tyler-ball tyler-ball force-pushed the chef_apply_rename branch 2 times, most recently from faf319b to 4058cec Compare July 9, 2018 21:33
Chef Expeditor and others added 3 commits July 10, 2018 09:37
…ply gem

Signed-off-by: tyler-ball <tball@chef.io>
Signed-off-by: tyler-ball <tball@chef.io>
Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
The built-in version update is sufficient, and there are no more version
files to update.

This also removes the 'bundle update chef-run' step, because there is no
chef-run component here to bundle update.

Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
@marcparadise marcparadise merged commit 51cdbc4 into master Jul 10, 2018
@chef-ci chef-ci deleted the chef_apply_rename branch July 10, 2018 14:10
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.

3 participants