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

Improve virtualization detection on Solaris #760

Merged
merged 5 commits into from
Mar 9, 2016

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Mar 5, 2016

Detect additional paravirt KVM CPU strings:

  • Common KVM processor
  • Common 32-bit KVM processor

Use the same node[:virtualization][:systems] multiple virtualization system attribute structure we use on other systems

Use our shared DMI Decode library for detection virtualization systems. This adds detection for Solaris running on the following systems:

  • VirtualBox
  • OpenStack
  • Hyper-V
  • KVM
  • Xen
  • VirtualServer

Copy and past from Linux Virtualization plugin
Linux / BSD use 'Product Name:' while Solaris uses 'Product:'  Product.*: should be just fine
This way anything we add for Linux will automatically get added to Solaris as well.  We're already using this in *BSD as well. This will add Hyper-V, KVM, Xen, VirtualServer, VirtualBox, and Openstack detection. Not bad for a net reduction in code.
We'll want a few more tests to be added here
@lamont-granquist
Copy link
Contributor

👍

1 similar comment
@smurawski
Copy link
Contributor

👍

tas50 added a commit that referenced this pull request Mar 9, 2016
Improve virtualization detection on Solaris
@tas50 tas50 merged commit 07ba106 into chef:master Mar 9, 2016
@thommay thommay added Type: Enhancement Adds new functionality. and removed Enhancement labels Jan 24, 2017
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Enhancement Adds new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants