diff options
| author | Tim Smith <tsmith@chef.io> | 2020-04-20 16:42:00 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-04-20 16:42:00 -0700 |
| commit | a0966c5f0a54d592f174319d02b84d916b842454 (patch) | |
| tree | 82afadd14da413cbee42be19a5d67232dc68ea8b | |
| parent | cd6090f256fcadf23c10235ece5e976cdd7c6084 (diff) | |
| parent | 25cf419772763a5f1cd1819bcebffdffb4717abd (diff) | |
| download | ohai-a0966c5f0a54d592f174319d02b84d916b842454.tar.gz | |
Merge pull request #1451 from ramereth/fix-dmi-virtualization-product-name
Use correct DMI attribute name for product name
| -rw-r--r-- | .rubocop.yml | 4 | ||||
| -rw-r--r-- | lib/ohai/plugins/linux/virtualization.rb | 4 | ||||
| -rw-r--r-- | spec/unit/plugins/linux/virtualization_spec.rb | 4 |
3 files changed, 6 insertions, 6 deletions
diff --git a/.rubocop.yml b/.rubocop.yml index cc7160b9..5a772c5f 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -6,7 +6,7 @@ AllCops: - "pkg/**/*" # these have shellout examples that need to have the tabs that come with the shellout -Layout/Tab: +Layout/IndentationStyle: Exclude: - "lib/ohai/plugins/mono.rb" - - "lib/ohai/plugins/darwin/hardware.rb"
\ No newline at end of file + - "lib/ohai/plugins/darwin/hardware.rb" diff --git a/lib/ohai/plugins/linux/virtualization.rb b/lib/ohai/plugins/linux/virtualization.rb index dd512ebb..1cb125aa 100644 --- a/lib/ohai/plugins/linux/virtualization.rb +++ b/lib/ohai/plugins/linux/virtualization.rb @@ -118,8 +118,8 @@ Ohai.plugin(:Virtualization) do # parse dmi to discover various virtualization guests # we do this *after* the kvm detection so that OpenStack isn't detected as KVM - logger.trace("Looking up DMI data manufacturer: '#{get_attribute(:dmi, :system, :manufacturer)}' product: '#{get_attribute(:dmi, :system, :product)}' version: '#{get_attribute(:dmi, :system, :version)}'") - guest = guest_from_dmi_data(get_attribute(:dmi, :system, :manufacturer), get_attribute(:dmi, :system, :product), get_attribute(:dmi, :system, :version)) + logger.trace("Looking up DMI data manufacturer: '#{get_attribute(:dmi, :system, :manufacturer)}' product_name: '#{get_attribute(:dmi, :system, :product_name)}' version: '#{get_attribute(:dmi, :system, :version)}'") + guest = guest_from_dmi_data(get_attribute(:dmi, :system, :manufacturer), get_attribute(:dmi, :system, :product_name), get_attribute(:dmi, :system, :version)) if guest logger.trace("Plugin Virtualization: DMI data indicates #{guest} guest") virtualization[:system] = guest diff --git a/spec/unit/plugins/linux/virtualization_spec.rb b/spec/unit/plugins/linux/virtualization_spec.rb index 468c5e0f..ccd868a4 100644 --- a/spec/unit/plugins/linux/virtualization_spec.rb +++ b/spec/unit/plugins/linux/virtualization_spec.rb @@ -190,7 +190,7 @@ describe Ohai::System, "Linux virtualization platform" do it "sets virtualization attributes if the appropriate DMI data is present" do plugin[:dmi] = { system: { manufacturer: "Amazon EC2", - product: "c5n.large", + product_name: "c5n.large", version: nil, }, } @@ -203,7 +203,7 @@ describe Ohai::System, "Linux virtualization platform" do it "sets empty virtualization attributes if nothing is detected" do plugin[:dmi] = { system: { manufacturer: "Supermicro", - product: "X10SLH-N6-ST031", + product_name: "X10SLH-N6-ST031", version: "0123456789", }, } |