testlab 1.13.0 → 1.14.0

data/lib/commands/testlab.rb

@@ -209,3 +209,20 @@ command :status do |status|
     commands[:container].commands[:status].execute({}, {}, [])
   end
 end
+
+
+# LAB DOCTOR
+#############
+desc 'Check the health of the lab'
+long_desc <<-EOF
+Attempts to analyze the health of the lab and report any issues.
+EOF
+command :doctor do |doctor|
+  doctor.action do |global_options, options, args|
+    if @testlab.doctor == true
+      @testlab.ui.stdout.puts("Everything is OK".green.bold)
+    else
+      @testlab.ui.stdout.puts(format_message("OH NOES!  SOMETHING IS SCREWED UP!".red.bold))
+    end
+  end
+end

data/lib/testlab.rb

@@ -159,6 +159,14 @@ class TestLab
   #
   # @return [Boolean] True if successful.
   def boot
+
+    # Raise how many files we can have open to the hard limit.
+    rlimit_nofile = Process.getrlimit(Process::RLIMIT_NOFILE)
+    if rlimit_nofile[0] != rlimit_nofile[1]
+      self.ui.logger.info { "Changing maximum open file descriptors from #{rlimit_nofile[0].inspect} to #{rlimit_nofile[1].inspect}" }
+      Process.setrlimit(Process::RLIMIT_NOFILE, rlimit_nofile[1])
+    end
+
     @labfile         = TestLab::Labfile.load(labfile_path)
     @labfile.testlab = self
 
@@ -351,6 +359,26 @@ class TestLab
     true
   end
 
+  # Test Lab Doctor
+  #
+  # Attempts to analyze the lab for issues.
+  #
+  # @return [Boolean] True if everything is OK; false otherwise.
+  def doctor
+    results = Array.new
+
+    if ((rlimit_nofile = Process.getrlimit(Process::RLIMIT_NOFILE)[0]) < 1024)
+      @ui.stderr.puts(format_message("The maximum number of file handles is set to #{rlimit_nofile}!  Please raise it to 1024 or higher!".red.bold))
+      results << false
+    end
+
+    results << nodes.all? do |node|
+      node.doctor
+    end
+
+    results.flatten.compact.all?
+  end
+
   # Node Method Proxy
   #
   # Iterates all of the lab nodes, sending the supplied method name and arguments

data/lib/testlab/container/actions.rb

@@ -13,7 +13,6 @@ class TestLab
         @ui.logger.debug { "Container Create: #{self.id}" }
 
         self.node.alive? or return false
-        self.node.ok?
 
         persistent_operation_check(:create)
 
@@ -37,7 +36,6 @@ class TestLab
         @ui.logger.debug { "Container Destroy: #{self.id}" }
 
         self.node.alive? or return false
-        self.node.ok?
 
         please_wait(:ui => @ui, :message => format_object_action(self, 'Destroy', :red)) do
           self.lxc.destroy(%(-f))
@@ -58,7 +56,6 @@ class TestLab
         @ui.logger.debug { "Container Up: #{self.id}" }
 
         self.node.alive? or return false
-        self.node.ok?
 
         please_wait(:ui => @ui, :message => format_object_action(self, 'Up', :green)) do
           configure
@@ -104,7 +101,6 @@ class TestLab
         @ui.logger.debug { "Container Down: #{self.id}" }
 
         self.node.alive? or return false
-        self.node.ok?
 
         please_wait(:ui => @ui, :message => format_object_action(self, 'Down', :red)) do
 

data/lib/testlab/container/io.rb

@@ -60,7 +60,6 @@ class TestLab
         @ui.logger.debug { "Container Export: #{self.id} " }
 
         self.node.alive? or return false
-        self.node.ok?
 
         (self.state == :not_created) and raise ContainerError, 'You must create a container before you can export it!'
 
@@ -115,7 +114,6 @@ EOF
         @ui.logger.debug { "Container Import: #{self.id}" }
 
         self.node.alive? or return false
-        self.node.ok?
 
         import_tempfile = Tempfile.new('import')
         remote_filename = File.basename(import_tempfile.path.dup)

data/lib/testlab/container/provision.rb

@@ -13,7 +13,6 @@ class TestLab
         @ui.logger.debug { "Container Provision: #{self.id} " }
 
         self.node.alive? or return false
-        self.node.ok?
 
         please_wait(:ui => @ui, :message => format_object_action(self, :provision, :green)) do
           do_provisioner_callbacks(self, :provision, @ui)
@@ -32,7 +31,6 @@ class TestLab
         @ui.logger.debug { "Container Deprovision: #{self.id} " }
 
         self.node.alive? or return false
-        self.node.ok?
 
         persistent_operation_check(:deprovision)
 

data/lib/testlab/node.rb

@@ -12,6 +12,7 @@ class TestLab
     # Sub-Modules
     autoload :Actions,       'testlab/node/actions'
     autoload :ClassMethods,  'testlab/node/class_methods'
+    autoload :Doctor,        'testlab/node/doctor'
     autoload :LXC,           'testlab/node/lxc'
     autoload :MethodMissing, 'testlab/node/method_missing'
     autoload :Provision,     'testlab/node/provision'
@@ -19,6 +20,7 @@ class TestLab
     autoload :Status,        'testlab/node/status'
 
     include TestLab::Node::Actions
+    include TestLab::Node::Doctor
     include TestLab::Node::LXC
     include TestLab::Node::MethodMissing
     include TestLab::Node::Provision

data/lib/testlab/node/doctor.rb

@@ -0,0 +1,62 @@
+class TestLab
+  class Node
+
+    module Doctor
+
+      # Node Doctor
+      #
+      # Attempts to analyze the current node and report any issues.
+      #
+      # @result [Boolean] True if everything is OK; false otherwise.
+      def doctor
+        if self.dead?
+          @ui.stderr.puts(format_message("The node #{self.id.inspect} is dead! (Did you forget to up or build the node?)".red.bold))
+          return false
+        end
+
+        if !self.lxc.installed?
+          @ui.stderr.puts(format_message("LXC does not appear to be installed on your TestLab node!  (Did you forget to provision or build the node?)".red.bold))
+          return false
+        end
+
+        result = true
+
+        # make sure the node has some free space
+        free_space_percent = (self.exec(%(df -P /), :ignore_exit_status => true).output.split("\n")[1].split[-2].to_i rescue nil)
+        if free_space_percent.nil?
+          @ui.stderr.puts(format_message("ERROR: We could not determine how much free space node #{self.id.inspect} has!".red.bold))
+          result = false
+        elsif (free_space_percent >= 90)
+          @ui.stderr.puts(format_message("WARNING: Your TestLab node #{self.id.inspect} is using #{free_space_percent}% of its available disk space!".red.bold))
+          result = false
+        end
+
+        # get the names of all of the defined containers
+        my_container_names = self.containers.map(&:id)
+
+        # ephemeral containers parent containers have a "-master" suffix; we need to remove these from the results or we will complain about them
+        node_container_names = self.lxc.containers.map(&:name).delete_if do |node_container_name|
+          my_container_names.any?{ |my_container_name| "#{my_container_name}-master" == node_container_name }
+        end
+
+        unknown_container_names = (node_container_names - my_container_names)
+        unknown_running_container_names = self.lxc.containers.select{ |c| (unknown_container_names.include?(c.name) && (c.state == :running)) }.map(&:name)
+
+        if unknown_container_names.count > 0
+          if unknown_running_container_names.count > 0
+            @ui.stderr.puts(format_message("WARNING: You have *running* containers on your TestLab node #{self.id.inspect} which are not defined in your Labfile!".red.bold))
+            @ui.stderr.puts(format_message(">>> You may need to manually stop the following containers: #{unknown_running_container_names.join(', ')}".red.bold))
+            result = false
+          end
+
+          @ui.stderr.puts(format_message("WARNING: You have containers on your TestLab node #{self.id.inspect} which are not defined in your Labfile!".red.bold))
+          @ui.stderr.puts(format_message(">>> You may need to manually remove the following containers: #{unknown_container_names.join(', ')}".red.bold))
+        end
+
+        result
+      end
+
+    end
+
+  end
+end

data/lib/testlab/node/status.rb

@@ -8,7 +8,7 @@ class TestLab
       # @return [Hash] A hash detailing the status of the node.
       def status
         {
-          :ok => self.ok?.inspect,
+          :ok => self.doctor.inspect,
           :id => self.id,
           :instance_id => @provider.instance_id,
           :state => @provider.state,
@@ -21,52 +21,6 @@ class TestLab
         }
       end
 
-      def ok?
-        self.dead? and return false
-
-        result = true
-
-        if !self.lxc.installed?
-          # @ui.stderr.puts(format_message("ERROR: "))
-          raise NodeError, "LXC does not appear to be installed on your TestLab node!  (Did you forget to provision or build the node?)"
-          # result = false
-        end
-
-        # make sure the node has some free space
-        free_space_percent = (self.exec(%(df -P /), :ignore_exit_status => true).output.split("\n")[1].split[-2].to_i rescue nil)
-        if free_space_percent.nil?
-          @ui.stderr.puts(format_message("ERROR: We could not determine how much free space node #{self.id.inspect} has!".red.bold))
-          result = false
-        elsif (free_space_percent >= 90)
-          @ui.stderr.puts(format_message("WARNING: Your TestLab node #{self.id.inspect} is using #{free_space_percent}% of its available disk space!".red.bold))
-          result = false
-        end
-
-        # get the names of all of the defined containers
-        my_container_names = self.containers.map(&:id)
-
-        # ephemeral containers parent containers have a "-master" suffix; we need to remove these from the results or we will complain about them
-        node_container_names = self.lxc.containers.map(&:name).delete_if do |node_container_name|
-          my_container_names.any?{ |my_container_name| "#{my_container_name}-master" == node_container_name }
-        end
-
-        unknown_container_names = (node_container_names - my_container_names)
-        unknown_running_container_names = self.lxc.containers.select{ |c| (unknown_container_names.include?(c.name) && (c.state == :running)) }.map(&:name)
-
-        if unknown_container_names.count > 0
-          if unknown_running_container_names.count > 0
-            @ui.stderr.puts(format_message("WARNING: You have *running* containers on your TestLab node #{self.id.inspect} which are not defined in your Labfile!".red.bold))
-            @ui.stderr.puts(format_message(">>> You may need to manually stop the following containers: #{unknown_running_container_names.join(', ')}".red.bold))
-            result = false
-          end
-
-          @ui.stderr.puts(format_message("WARNING: You have containers on your TestLab node #{self.id.inspect} which are not defined in your Labfile!".red.bold))
-          @ui.stderr.puts(format_message(">>> You may need to manually remove the following containers: #{unknown_container_names.join(', ')}".red.bold))
-        end
-
-        result
-      end
-
     end
 
   end

data/lib/testlab/version.rb

@@ -1,6 +1,6 @@
 class TestLab
   unless const_defined?(:VERSION)
     # TestLab Gem Version
-    VERSION = "1.13.0"
+    VERSION = "1.14.0"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: testlab
 version: !ruby/object:Gem::Version
-  version: 1.13.0
+  version: 1.14.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-13 00:00:00.000000000 Z
+date: 2013-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gli
@@ -266,6 +266,7 @@ files:
 - lib/testlab/node.rb
 - lib/testlab/node/actions.rb
 - lib/testlab/node/class_methods.rb
+- lib/testlab/node/doctor.rb
 - lib/testlab/node/lxc.rb
 - lib/testlab/node/method_missing.rb
 - lib/testlab/node/provision.rb
@@ -350,7 +351,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3460081152239272666
+      hash: -2303575706409532818
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -359,7 +360,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3460081152239272666
+      hash: -2303575706409532818
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.25