Update dependency chef-sugar to v5 - #51540
Conversation
|
PRApprover will approve and merge this PR, FAQ, #dx-source-code-management 🛠️ PRApproval Status
➡️ Current phase: CI tests failed. Please fix the failing tests to continue. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e3cfee607d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
|
||
| # Install omnibus | ||
| gem 'chef-sugar', git: 'https://github.com/chef/chef-sugar.git', tag: 'v3.6.0' | ||
| gem 'chef-sugar', git: 'https://github.com/chef/chef-sugar.git', tag: 'v5.1.12' |
There was a problem hiding this comment.
Keep chef-sugar within omnibus-ruby's constraint
The default Omnibus dependency here is still DataDog/omnibus-ruby at datadog-5.5.0, whose gemspec declares chef-sugar, ~> 3.3; pinning the top-level bundle to the git tag whose gem version is 5.1.12 makes Bundler unable to resolve the Omnibus bundle. Since tasks/omnibus.py removes Gemfile.lock and runs bundle install before invoking bundle exec omnibus, the Linux/macOS/Windows packaging jobs that use this Gemfile will fail before the build starts unless the omnibus-ruby ref is updated to allow chef-sugar 5.x.
Useful? React with 👍 / 👎.
|
|
Too much work to swtich APIS while we are actively deprecating this. |
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
This PR contains the following updates:
v3.6.0→v5.1.12Release Notes
chef/chef-sugar (chef-sugar)
v5.1.12Compare Source
Merged Pull Requests
v5.1.11Compare Source
Merged Pull Requests
v5.1.10Compare Source
v5.1.9Compare Source
Merged Pull Requests
v5.1.8Compare Source
Merged Pull Requests
v5.1.7Compare Source
v5.1.6Compare Source
v5.1.5Compare Source
v5.1.4Compare Source
v5.1.3Compare Source
v5.1.2Compare Source
v5.1.1Compare Source
Merged Pull Requests
v5.1.0Compare Source
v5.0.4Compare Source
v5.0.1Compare Source
systemd?so that it works equally well on physical/virtualmachines as well as within containers.
v5.0.0Compare Source
v4.2.1Compare Source
v4.2.0Compare Source
v4.1.0Compare Source
v4.0.1Compare Source
v4.0.0Compare Source
Configuration
📅 Schedule: (UTC)
* 0-4,22-23 * * 1-5)* * * * 0,6)🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.