diff options
author | Lamont Granquist <lamont@opscode.com> | 2021-04-12 10:57:12 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-04-12 10:57:12 -0700 |
commit | afcca93767d5b002603ae9234b5ce0cb5050a49e (patch) | |
tree | 31362033a72409fa97998efb7ddb0bc402e811dc | |
parent | 1dfbd15afb830f8aa8959ea9e79ed2648d3c2997 (diff) | |
parent | dd94465a83f5b5b485f2537472174fdb09727ee9 (diff) | |
download | ohai-afcca93767d5b002603ae9234b5ce0cb5050a49e.tar.gz |
Merge pull request #1647 from chef/aix_integers
lpar_no and wpar_no in AIX Virtualizatin plugin should be Integers
-rw-r--r-- | lib/ohai/plugins/aix/virtualization.rb | 7 | ||||
-rw-r--r-- | spec/unit/plugins/aix/virtualization_spec.rb | 4 |
2 files changed, 6 insertions, 5 deletions
diff --git a/lib/ohai/plugins/aix/virtualization.rb b/lib/ohai/plugins/aix/virtualization.rb index 74c72a5a..2bfa928d 100644 --- a/lib/ohai/plugins/aix/virtualization.rb +++ b/lib/ohai/plugins/aix/virtualization.rb @@ -24,14 +24,15 @@ Ohai.plugin(:Virtualization) do virtualization Mash.new lpar_no, lpar_name = shell_out("uname -L").stdout.split(nil, 2) + lpar_no = lpar_no.to_i - unless lpar_no.to_i == -1 || (lpar_no.to_i == 1 && lpar_name == "NULL") + unless lpar_no == -1 || (lpar_no == 1 && lpar_name == "NULL") virtualization[:lpar_no] = lpar_no virtualization[:lpar_name] = lpar_name end - wpar_no = shell_out("uname -W").stdout.strip - if wpar_no.to_i > 0 + wpar_no = shell_out("uname -W").stdout.strip.to_i + if wpar_no > 0 virtualization[:wpar_no] = wpar_no else # the below parses the output of lswpar in the long format diff --git a/spec/unit/plugins/aix/virtualization_spec.rb b/spec/unit/plugins/aix/virtualization_spec.rb index a96f5278..f8413f23 100644 --- a/spec/unit/plugins/aix/virtualization_spec.rb +++ b/spec/unit/plugins/aix/virtualization_spec.rb @@ -247,7 +247,7 @@ describe Ohai::System, "AIX virtualization plugin" do it "uname -L detects the LPAR number and name" do plugin.run - expect(plugin[:virtualization][:lpar_no]).to eq("29") + expect(plugin[:virtualization][:lpar_no]).to eq(29) expect(plugin[:virtualization][:lpar_name]).to eq("virtlpar03 - 7.1 testers") end @@ -314,7 +314,7 @@ describe Ohai::System, "AIX virtualization plugin" do end it "uname -W detects the WPAR number" do - expect(plugin[:virtualization][:wpar_no]).to eq("42") + expect(plugin[:virtualization][:wpar_no]).to eq(42) end end end |