onering-report 0.6.6 → 0.6.7

checksums.yaml

@@ -1,7 +1,15 @@
 ---
-SHA1:
-  metadata.gz: 52a32d9a3ce8d7cf810b725c475de4f73897be43
-  data.tar.gz: 2d9baebdcbc0bf1f33e9c13ad0aa8c1137f208c6
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    NDQ4ZmMyNGFmMTg1ZTUzM2YyMDQ4ZDU4ZWJmYmFkMmE1OTkzMjQ0NA==
+  data.tar.gz: !binary |-
+    MjdjMzJlN2E1M2ExMDhkZGZhZmUxOTg2NjQwZjg2NTdhMjYxOTU5Zg==
 SHA512:
-  metadata.gz: 68f3bf0756d41ac91af178c26c19dbfe06ae7e3bd4ef43407e4fcb039fe10e2b6b399199d075d5255efc360436aad6efd94849e2ae4a6a8f065c9995fdd5f118
-  data.tar.gz: 2ae2657444af6fdf86949cd232e381dc12fcb369e42da71fa21bdffce857bf533f51d877b35e8143f59afd79b01a2e2f3ccb2f3e5e061c94d85fcfd24b70fdb3
+  metadata.gz: !binary |-
+    YjFkNjdjNWVjYmE0YjExZTNkNTk0ZGRiMjAxNTBjOGMwOWJhMWY1OGFjMzVj
+    YjBmMzM1ODY3M2JiZDc4Y2U2ZWFlODkyYzA0MGFkYTFjNmEzYjMzMzQxM2Yz
+    NTBhODhhNDgzNmU1ZThmMDM5MzcwYzcwNGNjNTdlYjliZTEyYzU=
+  data.tar.gz: !binary |-
+    MGM0MDNjMWFkNzQzZDEyOGVhMzRjNmNhYTUwODZhYjY3ZDBlNGZmZWUyN2E3
+    ODM0YWI5YmQ1OWY2NTIwMzQ1YmI2ZmZlOTJhMDVkYTA4MDc4N2U3YjJiZjVl
+    NGE1ODMxMjJmNzY3ZDE4NzJkNWQ2OTJjMTVhZmM2MGE4ZjlhMWQ=

data/.ruby-version

@@ -1 +1 @@
-ruby-2.1.2
+ruby-1.9.3-p547

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    onering-report (0.6.5)
+    onering-report (0.6.6)
       onering-client
 
 GEM

data/lib/facter/onering_properties_physical.rb

@@ -48,11 +48,21 @@ Facter.add('slot') do
   end
 end
 
+def count_ipmi_instances(cmd)
+  Dir['/proc/[0-9]*/cmdline'].select {|p| File.read(p).include?(cmd)}.count
+end
+
+ALLOWED_IPMI = 5
+
 def ipmi_ok?
   begin
     ipmi = IPMI_PATH if defined? IPMI_PATH 
     ipmi ||= 'ipmiutil'
-    Timeout::timeout(1) {
+    if count_ipmi_instances(ipmi) > ALLOWED_IPMI
+	$stderr.puts "More then #{ALLOWED_IPMI} #{ipmi} instances found"
+	return false
+    end
+    Timeout::timeout(30) {
 	Facter::Util::Resolution.exec("#{ipmi} health 2> /dev/null | tail -n1 | grep failed").nil?
     }
   rescue Timeout::Error

data/onering-report.gemspec

@@ -1,7 +1,7 @@
 # -*- encoding: utf-8 -*-
 Gem::Specification.new do |gem|
   gem.name          = 'onering-report'
-  gem.version       = '0.6.6'
+  gem.version       = '0.6.7'
   gem.authors       = ['rnarkis']
   gem.email         = ['rnarkis@outbrain.com']
   gem.description   = %q{Onering facter facts}

data/test/bin/ipmiutil

@@ -1,2 +1,2 @@
 #!/usr/bin/env bash
-sleep 10
+sleep 60

data/test/ipmi_test.rb

@@ -34,4 +34,10 @@ class IpmiTest < MiniTest::Unit::TestCase
     assert_equal  false, ipmi_ok?
   end
 
+  def test_count_ipmi
+    res = count_ipmi_instances('ruby')
+    # this test is running within a ruby process to more then one is expected
+    assert_operator res, :>, 1
+  end
+
 end

metadata

@@ -1,55 +1,55 @@
 --- !ruby/object:Gem::Specification
 name: onering-report
 version: !ruby/object:Gem::Version
-  version: 0.6.6
+  version: 0.6.7
 platform: ruby
 authors:
 - rnarkis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-03 00:00:00.000000000 Z
+date: 2014-09-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: onering-client
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: puppet
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: Onering facter facts
@@ -59,9 +59,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".ruby-gemset"
-- ".ruby-version"
+- .gitignore
+- .ruby-gemset
+- .ruby-version
 - Gemfile
 - Gemfile.lock
 - Makefile
@@ -113,12 +113,12 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []