Skip to content

CHEF-29430 - Updating the chef references to chef-cli and fix chef-cli exec not getting ruby bin in the PATH #278

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.

Sign up for GitHub

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

Jump to bottom
Open
nikhil2611 wants to merge 18 commits into
base: main
Choose a base branch
from
Open

CHEF-29430 - Updating the chef references to chef-cli and fix chef-cli exec not getting ruby bin in the PATH #278

nikhil2611 wants to merge 18 commits into from

Conversation

@nikhil2611
Copy link
Contributor

@nikhil2611 nikhil2611 commented Mar 27, 2025 •
edited
Loading

Description

  • Updating the chef references to chef-cli in the usages
  • Updated the main description from The Chef command line tool for managing your infrastructure from your workstation. to This command line tool is used for managing your infrastructure from your workstation.
  • Updated the --version message from Show Chef Workstation version information to Show Chef CLI version information
  • Updated ➜ chef generate --version
    from Chef Workstation version: 5.6.23 to Chef CLI version: 6.1.16
  • Fixed the chef-cli exec command which was not identifying ruby bin in the PATH

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

  • I have read the CONTRIBUTING document.
  • I have run the pre-merge tests locally and they pass.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • If Gemfile.lock has changed, I have used --conservative to do it and included the full output in the Description above.
  • All new and existing tests passed.
  • All commits have been signed-off for the Developer Certificate of Origin.

@nikhil2611
updating the chef refrences to chef-cli
3926c3a 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611 nikhil2611 requested review from a team as code owners March 27, 2025 13:09
@github-actions
Copy link

github-actions bot commented Mar 27, 2025

Simplecov Report

Covered Threshold
98.51% 90%

@nikhil2611 nikhil2611 changed the title Updating the chef references to chef-cli [DO NOT MERGE] Updating the chef references to chef-cli Mar 27, 2025
@nikhil2611
updating the test cases to fix the chef-cli references
5f42084 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
updating the remaining test cases to fix the chef-cli references
4274156 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@sonarqube-for-infrastructure-prod
Copy link

sonarqube-for-infrastructure-prod bot commented Mar 27, 2025

Quality Gate failed Quality Gate failed

Failed conditions
1 New issue

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarLint SonarLint

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 27, 2025

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
26.7% Duplication on New Code

See analysis details on SonarQube Cloud

@nikhil2611 nikhil2611 changed the title [DO NOT MERGE] Updating the chef references to chef-cli Updating the chef references to chef-cli Dec 30, 2025
@nikhil2611
empty commit to run scans
46d79b8 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
updating the chef ref to chef cli
b9ca336 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
add ruby bin to PATH for hab env
1b13325 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
updated comment
f8e538e 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
updated references from chef to chef-cli
b8f7b28 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
added ruby bin to the PATH so that exec command can find it
e222911 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
empty commit to run scans
be5593f 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
No explicit Ruby PATH needed in plan.ps1 - handled by pkg_deps
29f88cc 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
empty commit to run scans
fc7297e 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
updated comment to run scans
70bb742 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
updated usage description and version message to point to chef cli
be2d33c 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611
updated --version message to chef cli instead o chef workstation
02a3f7e 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611 nikhil2611 changed the title Updating the chef references to chef-cli CHEF-29430 - Updating the chef references to chef-cli and fix chef exec not getting ruby bin in the PATH Jan 6, 2026
@nikhil2611 nikhil2611 changed the title CHEF-29430 - Updating the chef references to chef-cli and fix chef exec not getting ruby bin in the PATH CHEF-29430 - Updating the chef references to chef-cli and fix chef-cli exec not getting ruby bin in the PATH Jan 6, 2026
@nikhil2611
fixing the test cases
ce6f3f2 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@sanghinitin
Copy link
Contributor

sanghinitin commented Jan 6, 2026

please update https://github.com/chef/chef-cli/tree/main/lib/chef-cli/skeletons/code_generator/files/default/repo/policyfiles

@nikhil2611
updated Readme.md
23d473d 
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611 nikhil2611 requested a review from a team as a code owner January 6, 2026 12:27
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 6, 2026

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
11.1% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ashiqueps ashiqueps ashiqueps approved these changes

@sanghinitin sanghinitin sanghinitin approved these changes

@sanjain-progress sanjain-progress sanjain-progress approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@nikhil2611 @sanghinitin @ashiqueps @sanjain-progress