CloudyScripts 1.5.16 → 1.5.17

data/Rakefile

@@ -12,7 +12,7 @@ require 'rake/testtask'
 
 spec = Gem::Specification.new do |s|
   s.name = 'CloudyScripts'
-  s.version = '1.5.16' 
+  s.version = '1.5.17'
   s.has_rdoc = true
   s.extra_rdoc_files = ['README.rdoc', 'LICENSE']
   s.summary = 'Scripts to facilitate programming for infrastructure clouds.'

data/lib/help/ec2_helper.rb

@@ -118,4 +118,43 @@ class Ec2Helper
     end
   end
 
+  # Checks if all ports are opened for the security group on range "0.0.0.0/0".
+  # If an additional range is specified in the parameter, a check returns
+  # true if a port is opened for either range 0.0.0.0/0 or the additional
+  # range specified.
+  # Returns true or false.
+  def check_open_port(security_group, port, range = "0.0.0.0/0")
+    res = @ec2_api.describe_security_groups(:group_name => security_group)
+    #puts "res = #{res.inspect}"
+    groups = res['securityGroupInfo']['item']
+    if groups.size == 0
+      raise Exception.new("security group '#{security_group}' not found")
+    end
+    permissions = groups[0]['ipPermissions']['item']
+    if permissions.size == 0
+      # no permissions at all
+      return false
+    end
+    permissions.each() {|permission|
+      #puts "permission: #{permission.inspect}"
+      if permission['ipRanges'] == nil
+        #no IP-Ranges defined (group based mode): ignore
+        next
+      end
+      from_port = permission['fromPort'].to_i
+      to_port = permission['toPort'].to_i
+      prot = permission['ipProtocol']
+      if port >= from_port && port <= to_port && prot == "tcp"
+        permission['ipRanges']['item'].each() {|ipRange|
+          if ipRange['cidrIp'] != "0.0.0.0/0" && ipRange['cidrIp'] != range
+            next
+          else
+            return true
+          end
+        }
+      end
+    }
+    false
+  end
+
 end

data/lib/help/remote_command_handler.rb

@@ -122,10 +122,13 @@ class RemoteCommandHandler
   end
 
   # Zip the complete contents of the source path into the destination file.
+  # Returns the an array with stderr output messages.
   def zip(source_path, destination_file)
     begin
-      exec = "cd #{source_path}; zip -ry #{destination_file} *"
-      remote_execute(exec, nil, true)
+      exec = "cd #{source_path}; zip -ryq #{destination_file} *"
+      stderr = []
+      get_output(exec, nil, nil, stderr)
+      return stderr
     rescue Exception => e
       raise Exception.new("zip failed due to #{e.message}")
     end

data/lib/help/state_transition_helper.rb

@@ -417,7 +417,11 @@ module StateTransitionHelper
   # # zip_file_name => name of the zip file (without .zip suffix)
   def zip_volume(source_dir, zip_file_dest, zip_file_name)
     post_message("going to zip the EBS volume")
-    remote_handler().zip(source_dir, zip_file_dest+"/"+zip_file_name)
+    stderr = remote_handler().zip(source_dir, zip_file_dest+"/"+zip_file_name)
+    if stderr.size > 0
+      @logger.info("zip operation generated error and might not be complete. output: #{stderr.join("\n")}")
+      post_message("zip operation generated error and might not be complete. output: #{stderr.join("\n")}")
+    end
     post_message("EBS volume successfully zipped")
   end
 

data/lib/scripts/ec2/ami2_ebs_conversion.rb

@@ -28,6 +28,13 @@ class Ami2EbsConversion < Ec2Script
   end
 
   def check_input_parameters()
+    if @input_params[:security_group_name] == nil
+      @input_params[:security_group_name] = "default"
+    end
+    ec2_helper = Ec2Helper.new(@input_params[:ec2_api_handler])
+    if !ec2_helper.check_open_port(@input_params[:security_group_name], 22)
+      raise Exception.new("Port 22 must be opened for security group #{@input_params[:security_group_name]} to connect via SSH")
+    end
     if @input_params[:name] == nil
       @input_params[:name] = "Boot EBS (for AMI #{@input_params[:ami_id]}) at #{Time.now.strftime('%d/%m/%Y %H.%M.%S')}"
     else

data/lib/scripts/ec2/copy_ami.rb

@@ -42,6 +42,14 @@ class CopyAmi < Ec2Script
     if ec2_helper.ami_prop(@input_params[:ami_id], 'rootDeviceType') != "ebs"
       raise Exception.new("must be an EBS type image")
     end
+    local_ec2_helper = ec2_helper
+    if !local_ec2_helper.check_open_port('default', 22)
+      raise Exception.new("Port 22 must be opened for security group 'default' to connect via SSH in source-region")
+    end
+    remote_ec2_helper = Ec2Helper.new(@input_params[:target_ec2_handler])
+    if !remote_ec2_helper.check_open_port('default', 22)
+      raise Exception.new("Port 22 must be opened for security group 'default' to connect via SSH in target-region")
+    end
     if @input_params[:root_device_name] == nil
       @input_params[:root_device_name] = "/dev/sda1"
     end

data/lib/scripts/ec2/copy_snapshot.rb

@@ -35,6 +35,14 @@ class CopySnapshot< Ec2Script
   end
 
   def check_input_parameters()
+    local_ec2_helper = Ec2Helper.new(@input_params[:ec2_api_handler])
+    if !local_ec2_helper.check_open_port('default', 22)
+      raise Exception.new("Port 22 must be opened for security group 'default' to connect via SSH in source-region")
+    end
+    remote_ec2_helper = Ec2Helper.new(@input_params[:target_ec2_handler])
+    if !remote_ec2_helper.check_open_port('default', 22)
+      raise Exception.new("Port 22 must be opened for security group 'default' to connect via SSH in target-region")
+    end
     if @input_params[:source_ssh_username] == nil
       @input_params[:source_ssh_username] = "root"
     end

data/lib/scripts/ec2/download_snapshot.rb

@@ -34,6 +34,13 @@ class DownloadSnapshot < Ec2Script
   end
 
   def check_input_parameters()
+    if @input_params[:security_group_name] == nil
+      @input_params[:security_group_name] = "default"
+    end
+    ec2_helper = Ec2Helper.new(@input_params[:ec2_api_handler])
+    if !ec2_helper.check_open_port(@input_params[:security_group_name], 22)
+      raise Exception.new("Port 22 must be opened for security group #{@input_params[:security_group_name]} to connect via SSH")
+    end
     if @input_params[:source_device] == nil
       @input_params[:source_device] = "/dev/sdj1"
     end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 1
   - 5
-  - 16
-  version: 1.5.16
+  - 17
+  version: 1.5.17
 platform: ruby
 authors: 
 - Matthias Jung
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-05-18 00:00:00 +02:00
+date: 2010-05-19 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency