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

Adding hyperv to the hypervisor helpers #4270

Merged
merged 3 commits into from
Jul 10, 2024

Conversation

snohio
Copy link
Contributor

@snohio snohio commented Apr 26, 2024

Signed-off-by: Mike Butler michael.butler@progress.com

Description

Added hyperv to the virtualzation.rb in chef-client and this is the documentation to match.

Definition of Done

Issues Resolved

[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]

Related PRs

chef/chef#14359

Check List

  • Spell Check
  • Local build
  • Examine the local build
  • All tests pass

(Local build on Windows not really working.)

Signed-off-by: Mike Butler <michael.butler@progress.com>
@snohio snohio requested a review from a team as a code owner April 26, 2024 19:35
Copy link

netlify bot commented Apr 26, 2024

Deploy Preview for chef-web-docs ready!

Name Link
🔨 Latest commit ce02449
🔍 Latest deploy log https://app.netlify.com/sites/chef-web-docs/deploys/668efe60ea0f2a0008d94a8c
😎 Deploy Preview https://deploy-preview-4270--chef-web-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

sonarcloud bot commented Apr 26, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@snohio
Copy link
Contributor Author

snohio commented Apr 26, 2024

I'm adding this to the chef-client (probably 18.5) release. This should get merged when that goes out. It's a pretty simple addition.

@snohio
Copy link
Contributor Author

snohio commented Apr 26, 2024

The "misspelling" is that hyperv? isn't going to be in a dictionary.

IanMadd and others added 2 commits July 10, 2024 17:33
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Copy link

sonarcloud bot commented Jul 10, 2024

@IanMadd IanMadd merged commit 9f89e1a into main Jul 10, 2024
11 of 12 checks passed
@IanMadd IanMadd deleted the add-hyperv-to-checking_hypervisors branch July 10, 2024 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants