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

Switch dnsimple gem installation from gem_package to chef_gem #92

Merged
merged 1 commit into from May 8, 2024

Conversation

san983
Copy link
Member

@san983 san983 commented May 8, 2024

As per https://docs.chef.io/resources/chef_gem/

The chef_gem and gem_package resources are both used to install Ruby gems. For any machine on which Chef Infra Client is installed, there are two instances of Ruby. One is the standard, system-wide instance of Ruby and the other is a dedicated instance that is available only to Chef Infra Client. Use the chef_gem resource to install gems into the instance of Ruby that is dedicated to Chef Infra Client. Use the gem_package resource to install all other gems (i.e. install gems system-wide).

:shipit: Deploy Steps:

  • Merge
  • Bump version

:shipit: Verification Steps

  • Ensure ALL tests pass without errors

As per https://docs.chef.io/resources/chef_gem/

The chef_gem and gem_package resources are both used to install Ruby gems. For any machine on which Chef Infra Client is installed, there are two instances of Ruby. One is the standard, system-wide instance of Ruby and the other is a dedicated instance that is available only to Chef Infra Client. Use the chef_gem resource to install gems into the instance of Ruby that is dedicated to Chef Infra Client. Use the gem_package resource to install all other gems (i.e. install gems system-wide).
@san983 san983 self-assigned this May 8, 2024
@san983 san983 marked this pull request as ready for review May 8, 2024 17:40
@san983 san983 requested a review from a team as a code owner May 8, 2024 17:40
@san983 san983 merged commit 1bc18d0 into main May 8, 2024
16 checks passed
@san983 san983 deleted the task/use_chef_gem_instead_of_gem_package branch May 8, 2024 17:41
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.

None yet

1 participant