Skip to content
Snippets Groups Projects

Bump chef version to 12.4.0.rc.0.

Merged Bump chef version to 12.4.0.rc.0.
bump_chef into master
Merged Marin Jankovski requested to merge bump_chef into master

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
config/projects/gitlab.rb
44 44
45 45 override :ruby, version: '2.1.6', source: { md5: "6e5564364be085c45576787b48eeb75f" }
46 46 override :rubygems, version: '2.2.1'
47 override :'chef-gem', version: '11.18.0'
47 override :chef, version: '12.4.0.rc.0'
  • Marin Jankovski
    Marin Jankovski @marin ·
    Author Maintainer

    @jacobvosmaer I've opted to use software definition for chef instead of using chef-gem. The reason for this is that I think it is a better move to not depend on releases on rubygems in case we need to react quickly to an upstream fix. What do you think?

  • username-removed-5302
    username-removed-5302 @jacobvosmaer ·
    Contributor

    Great, I tried installing the gem and failed. Glad that they have something that makes it easier :)

  • Marin Jankovski
    Marin Jankovski @marin ·
    Author Maintainer

    Tested the clean installation, apart from few new warnings in the chef run output everything worked ok.

    Still to test after merge, upgrade from package with old chef version to the new one.

  • username-removed-5302
    username-removed-5302 @jacobvosmaer ·
    Contributor

    Could you paste the warnings? just curious

  • Marin Jankovski
    Marin Jankovski @marin ·
    Author Maintainer

    @jacobvosmaer This https://github.com/chef/chef-server/issues/106, already got fixed.

  • username-removed-5302
    username-removed-5302 @jacobvosmaer ·
    Contributor

    Cool. (Let's see how our users like the warnings...)

  • Marin Jankovski
    Marin Jankovski @marin ·
    Author Maintainer

    Tried upgrading the package, no issues observed.

  • Marin Jankovski Status changed to merged

    Status changed to merged

  • Marin Jankovski mentioned in commit b1a3870b

    mentioned in commit b1a3870b

    • Please register or sign in to reply