rudy 0.9.5.003 → 0.9.5.004

data/CHANGES.txt

@@ -4,6 +4,7 @@ RUDY, CHANGES
 #### 0.9.6 (2009-10-??) ###########################
 
 * FIXED: Include Gibbler for REXML
+* FIXED: Elastic IP binding on startup (GH-40)
 * CHANGE: Increased max instances from 5 to 20. 
 
 

data/lib/rudy.rb

@@ -13,19 +13,19 @@ begin; require 'json'; rescue LoadError; end # Silence!
 require 'digest/md5'
 require 'stringio'
 require 'ostruct'
-autoload :YAML, 'yaml'
+require 'yaml'
 require 'logger'
 require 'socket'
 require 'resolv'
 require 'timeout'
-autoload :Gibbler, 'gibbler/aliases'
+require 'gibbler/aliases'
 require 'tempfile'
 require 'rudy/mixins'
 require 'storable'
-autoload :SysInfo, 'sysinfo'
+require 'sysinfo'
 require 'attic'
 require 'annoy'
-autoload :Rye, 'rye'
+require 'rye'
 
 # = Rudy
 #
@@ -42,7 +42,7 @@ module Rudy
       MAJOR = 0.freeze
       MINOR = 9.freeze
       TINY  = 5.freeze
-      PATCH  = '003'.freeze
+      PATCH  = '004'.freeze
     end
     def self.to_s; [MAJOR, MINOR, TINY, PATCH].join('.'); end
     def self.to_f; self.to_s.to_f; end

data/lib/rudy/cli/routines.rb

@@ -20,6 +20,15 @@ module Rudy; module CLI;
         routine = {}
         routine.merge! @@config.routines.find_deferred(@@global.environment, @@global.role) || {}
         routine.merge! @@config.routines.find(@@global.role) || {}
+        #envs, roles = @@global.environment || [], @@global.role || []
+        #envs.each do |env|
+        #  roles.each do |role|
+        #    routine.merge! @@config.routines.find_deferred(env, role) || {}
+        #  end
+        #end
+        #roles.each do |role|
+        #  routine.merge! @@config.routines.find(role) || {}
+        #end
       end
       
       outform = @@global.format == :json ? :to_json : :to_yaml

data/lib/rudy/metadata/machine.rb

@@ -158,21 +158,6 @@ module Rudy
         @state = inst.state
         # We need to be safe when creating machines because if an exception is
         # raised, instances will have been created but the calling class won't know. 
-        begin
-          # Assign IP address only if we have one for that position
-          if @address
-            # Make sure the address is associated to the current account
-            if Rudy::AWS::EC2::Addresses.exists?(@address)
-              li "Associating #{@address} to #{@instid}"
-              Rudy::AWS::EC2::Addresses.associate(@address, @instid)
-            else
-              le "Unknown address: #{@address}"
-            end
-          end
-        rescue => ex
-          le "Error: #{ex.message}"
-          le ex.backtrace if Rudy.debug?
-        end
       end
       
       self.save

data/lib/rudy/routines/handlers/host.rb

@@ -24,26 +24,63 @@ module Rudy; module Routines; module Handlers;
     # it's also important to call here because if a routine was executed
     # and an unexpected exception occurs before this update is executed
     # the machine metadata won't contain the DNS information. Calling it
-    # here ensure that the metadata is always up-to-date.
+    # here ensures that the metadata is always up-to-date.
+    # 
+    # If a machine has an associated elastic IP address, it will also be
+    # assigned in this step. 
+    # 
     # Each Rye:Box instance has a Rudy::Machine instance in its stash so
     # rbox.stash.refresh! == machine.refresh!
     def update_dns(rset)
       raise NoMachines if rset.boxes.empty?
       rset.boxes.each do |rbox|
-        rbox.stash.refresh! 
-        rbox.host = rbox.stash.dns_public
+        mach = rbox.stash
+        # Assign IP address only if we have one for that position
+        if mach.address
+          begin
+            # Make sure the address is associated to the current account
+            if Rudy::AWS::EC2::Addresses.exists?(mach.address)
+              li "Associating #{mach.address} to #{mach.instid}"
+              Rudy::AWS::EC2::Addresses.associate(mach.address, mach.instid)
+            else
+              le "Unknown address: #{mach.address}"
+            end
+          rescue => ex
+            le "Error associating address: #{ex.message}"
+            ld ex.backtrace
+          end
+        end
+        
+        # Give EC2 some time to update their metadata
+        msg = "Waiting for public DNS on #{rbox.nickname} ..."
+        multi = rbox.stash.windows? ? 3 : 2
+        interval, max = 2*multi, 60*multi
+        Rudy::Utils.waiter(interval, max, STDOUT, msg, 0) {
+          mach.refresh!
+          if mach.address
+            mach.dns_public.to_s =~ /#{mach.address.to_s.gsub('.', '-')}/
+          else
+            !mach.dns_public.nil? && !mach.dns_public.empty?
+          end
+        }
+        rbox.host = mach.dns_public
       end
     end
     
     def is_available?(rset, port=22)
       raise NoMachines if rset.boxes.empty?
       rset.boxes.each do |rbox|
+        mach = rbox.stash
+        # This updates the DNS. It's important this happens
+        # before and after the address is updated otherwise
+        # certain errors will causes it to not be updated.  
+        mach.refresh!
         msg = "Waiting for port #{port} on #{rbox.nickname} ..."
-        port = 3389 if rbox.stash.windows?
-        multi = rbox.stash.windows? ? 3 : 2
+        port = 3389 if mach.windows?
+        multi = mach.windows? ? 3 : 2
         interval, max = 1*multi, 30*multi
         Rudy::Utils.waiter(interval, max, STDOUT, msg, 0) {
-          Rudy::Utils.service_available?(rbox.stash.dns_public, port)
+          Rudy::Utils.service_available?(mach.dns_public, port)
         }
       end
     end

data/lib/rudy/routines/startup.rb

@@ -72,6 +72,7 @@ module Rudy; module Routines;
       
       # This is important b/c the machines will not 
       # have DNS info until after they are running. 
+      # This will also assigns elastic IP addresses. 
       Rudy::Routines.rescue { Rudy::Routines::Handlers::Host.update_dns @@rset }
       
       Rudy::Routines.rescue {

data/rudy.gemspec

@@ -1,7 +1,7 @@
 @spec = Gem::Specification.new do |s|
   s.name = "rudy"
   s.rubyforge_project = 'rudy'
-  s.version = "0.9.5.003"
+  s.version = "0.9.5.004"
   s.summary = "Rudy: Not your grandparents' EC2 deployment tool."
   s.description = s.summary
   s.author = "Delano Mandelbaum"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: rudy
 version: !ruby/object:Gem::Version 
-  version: 0.9.5.003
+  version: 0.9.5.004
 platform: ruby
 authors: 
 - Delano Mandelbaum
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-11-04 00:00:00 -05:00
+date: 2009-12-02 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency