Issue #8343 has been updated by Jeff McCune.

Status changed from Accepted to In Topic Branch Pending Merge

# Topic Branch #

Mailed tech for review.  In topic branch in my github repository pending merge.

<pre>
>From a0a49b729a3ec7424baf7e3528d27d15f0d239d7 Mon Sep 17 00:00:00 2001
From: Jeff McCune <[email protected]>
Date: Sun, 10 Jul 2011 17:55:43 -0700
Subject: [PATCH/Module/cloudpack 1/1] (#8343) Create action prints dns name
 asap

This small change gives the end user useful information about the
instance ID and the public DNS name as soon as they are available inside
the create action.  Without this change, the end user may not have the
information they need at the time they need it.

This also allows the end user to "do stuff" while waiting for the
fingerprints.

Here is the new output:

    $ puppet cloudnode create --image ami-2342a94a \
      --keypair jeff_initial --type t1.micro
    Connecting to AWS us-east-1 Done
    Instance Type: t1.micro
    Creating new instance ... Done
    Instance identifier: i-a80be6c9
    Creating tags for instance ... Done
    Starting up ............. Done
    Instance public dns name: ec2-50-17-178-240.compute-1.amazonaws.com
    Waiting for host fingerprints .....................................

Signed-off-by: Jeff McCune <[email protected]>
---
 lib/puppet/cloudpack.rb |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/lib/puppet/cloudpack.rb b/lib/puppet/cloudpack.rb
index 3f6c393..80196fc 100644
--- a/lib/puppet/cloudpack.rb
+++ b/lib/puppet/cloudpack.rb
@@ -309,6 +309,9 @@ module Puppet::CloudPack
         :flavor_id  => options[:type]
       )
 
+      # This is the earliest point we have knowledge of the instance ID
+      puts "Instance identifier: #{server.id}"
+
       Signal.trap(:EXIT) do
         if options[:_destroy_server_at_exit]
           server.destroy rescue nil
@@ -332,6 +335,9 @@ module Puppet::CloudPack
         return nil
       end
 
+      # This is the earliest point we have knowledge of the DNS name
+      puts "Instance public dns name: #{server.dns_name}"
+
       # TODO: Find a better way of getting the Fingerprints
       begin
         print 'Waiting for host fingerprints '
-- 
1.7.5.4
</pre>
----------------------------------------
Feature #8343: CloudPack should print the server hostname as soon as possible
https://projects.puppetlabs.com/issues/8343

Author: Jeff McCune
Status: In Topic Branch Pending Merge
Priority: Low
Assignee: Jeff McCune
Category: 
Target version: 
Keywords: 
Branch: ticket/8185_list_instances_action/8343_create_should_print_dnsname_asap
Roadmapped: No


# Overview #

As the end user, In the event "waiting for fingerprints" takes too long or 
times out it's useful to know what the public hostname of the new instance is.

To help the end user, cloudpack should print the hostname as early as possible.

I suspect this could happen immediately after Starting up and before Waiting 
for host fingerprints:

<pre>
Connecting to AWS us-west-1 Done
Instance Type: t1.micro
Creating new instance ... Done
Creating tags for instance ... Done
Starting up .................. Done
Waiting for host fingerprints ........................
</pre>


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to