brightbox-cli 0.13.1 → 0.14.0

data/lib/brightbox-cli/api.rb

@@ -71,7 +71,7 @@ module Brightbox
           o
         end
       else
-        raise InvalidArguments, "Couldn't find '#{arg.class}'"
+        raise InvalidArguments, "Couldn't find '#{args.class}'"
       end
       if objects
         # wrap in our objects

data/lib/brightbox-cli/cloud_ips.rb

@@ -42,5 +42,14 @@ module Brightbox
     def <=>(b)
       self.status <=> b.status
     end
+
+    def update(options)
+      self.class.conn.update_cloud_ip(id, options)
+      self.reload
+      self
+    rescue Excon::Errors::BadRequest => e
+      raise Conflict, JSON.parse(e.response.body)['error']['details']
+    end
+
   end
 end

data/lib/brightbox-cli/commands/cloudips-update.rb

@@ -0,0 +1,36 @@
+module Brightbox
+  desc 'update Cloud IPs'
+  arg_name 'cloudip-id'
+  command [:update] do |c|
+    c.desc "Set reverse DNS for this cloud ip"
+    c.flag [:r, "reverse-dns"]
+
+    c.desc "Delete the reverse dns for this cloud ip"
+    c.switch ["delete-reverse-dns"]
+
+    c.action do |global_options,options,args|
+      cip_id = args.shift
+      raise "You must specify the cloud ip id as the first argument" unless cip_id =~ /^cip-/
+
+      if options[:r] && options[:r] != "" && options[:"delete-reverse-dns"]
+        raise "You must either specify a reverse dns record or --delete-reverse-dns"
+      end
+
+      cip = CloudIP.find cip_id
+
+      params = {}
+      if options[:r]
+        params[:reverse_dns] = options[:r]
+      end
+
+      if options[:"delete-reverse-dns"]
+        params[:reverse_dns] = ""
+      end
+
+      cip.update(params)
+      cip.reload
+
+      render_table([cip], global_options)
+    end
+  end
+end

data/lib/brightbox-cli/commands/images-register.rb

@@ -11,8 +11,13 @@ module Brightbox
     c.desc "Source filename of the image you uploaded to the image library"
     c.flag [:s, "source"]
 
-    c.desc "This image does not support virtio so needs 'compatibility mode'"
-    c.switch [:c, "compatibility"]
+    c.desc "Set image mode to be either 'virtio' or 'compatibility'"
+    c.default_value "virtio"
+    c.flag [:m, "mode"]
+
+    c.desc "Set image to be publically visible (true or false)"
+    c.default_value "false"
+    c.flag [:p, "public"]
 
     c.desc "Image description"
     c.flag [:d, "description"]
@@ -21,10 +26,24 @@ module Brightbox
 
       raise "You must specify the architecture" unless options[:a]
       raise "You must specify the source filename" unless options[:s]
+      raise "Mode must be 'virtio' or 'compatibility'" unless options[:m] == "virtio" || options[:m] == "compatibility"
+      raise "Public must be true or false" unless options[:p] == "true" || options[:p] == "false"
+
+      if options[:m] == "compatibility"
+        compatibility_flag = true
+      else
+        compatibility_flag = false
+      end
+
+      if options[:p] == "true"
+        public_flag = true
+      else
+        public_flag = false
+      end
 
       image = Image.register :name => options[:n], :arch => options[:a],
-      :source => options[:s], :compatibility_mode => options[:c],
-      :description => options[:d]
+      :source => options[:s], :compatibility_mode => compatibility_flag,
+      :description => options[:d], :public => public_flag
 
       render_table([image])
 

data/lib/brightbox-cli/commands/images-show.rb

@@ -15,7 +15,7 @@ module Brightbox
 
       table_opts = global_options.merge({
                                           :vertical => true,
-                                          :fields => [:id, :type, :owner, :created_at, :status, :arch, :name, :description, :virtual_size, :disk_size, "compatibility_mode", :official, :ancestor_id ]
+                                          :fields => [:id, :type, :owner, :created_at, :status, :arch, :name, :description, :virtual_size, :disk_size, :public, :"compatibility_mode", :official, :ancestor_id ]
                                         })
 
       render_table(images, table_opts)

data/lib/brightbox-cli/commands/images-update.rb

@@ -0,0 +1,52 @@
+module Brightbox
+  desc 'Update an image'
+  arg_name 'img-id'
+  command [:update] do |c|
+
+    c.desc "Name to give the image"
+    c.flag [:n, "name"]
+
+    c.desc "Architecture of the image (i686 or x86_64)"
+    c.flag [:a, "arch"]
+
+    c.desc "Set image mode to be either 'virtio' or 'compatibility'"
+    c.flag [:m, "mode"]
+
+    c.desc "Set image to be publically visible (true or false)"
+    c.flag [:p, "public"]
+
+    c.desc "Image description"
+    c.flag [:d, "description"]
+
+    c.action do |global_options,options,args|
+      img_id = args.shift
+      raise "You must specify the image to update as the first argument" unless img_id =~ /^img-/
+      if options[:m]
+        raise "Mode must be 'virtio' or 'compatibility'" unless options[:m] == "virtio" || options[:m] == "compatibility"
+      end
+      if options[:p]
+        raise "Public must be true or false" unless options[:p] == "true" || options[:p] == "false"
+      end
+
+      params = {}
+      params[:name]               = options[:n] if options[:n]
+      params[:arch]               = options[:a] if options[:a]
+      params[:source]             = options[:s] if options[:s]
+      params[:description]        = options[:d] if options[:d]
+
+      params[:compatibility_mode] = true if options[:m] == "compatibility"
+      params[:compatibility_mode] = false if options[:m] == "virtio"
+
+      params[:public] = true if options[:p] == "true"
+      params[:public] = false if options[:p] == "false"
+
+      image = Image.find img_id
+
+      info "Updating image #{image}"
+      image.update params
+      image.reload
+      render_table([image], global_options)
+    end
+
+  end
+end

data/lib/brightbox-cli/commands/servers-show.rb

@@ -39,7 +39,7 @@ module Brightbox
 
       display_options = {
         :vertical => true,
-        :fields => [:id, :status, :name, :description, :created_at, :deleted_at,
+        :fields => [:id, :status, :name, :created_at, :deleted_at,
           :zone, :type, :type_name, :type_handle, :ram, :cores,
           :disk, :image, :image_name, :private_ips, :cloud_ips, :ipv6_address,
           :cloud_ip_ids, :hostname, :public_hostname, :snapshots

data/lib/brightbox-cli/commands/servers-update.rb

@@ -0,0 +1,59 @@
+module Brightbox
+  desc 'Update a server'
+  arg_name 'srv-id'
+  command [:update] do |c|
+    c.desc "Friendly name of server"
+    c.flag [:n, :name]
+
+    c.desc "Specify user data"
+    c.flag [:m, "user-data"]
+
+    c.desc "Specify the user data from a local file"
+    c.flag [:f, "user-data-file"]
+
+    c.desc "Don't base64 encode the user data"
+    c.switch [:e, :no_base64]
+
+    c.action do |global_options, options, args|
+      srv_id = args.shift
+      raise "You must specify a valid server id as the first argument" unless srv_id =~ /^srv-/
+
+      server = Server.find srv_id
+
+      user_data = options[:m]
+      user_data_file = options[:f]
+
+      if user_data_file
+        raise "Cannot specify user data on command line and in file at same time" if user_data
+        # Wot we use to read the data, be it from stdin or a file on disk
+        file_handler = lambda do |fh|
+          raise "User data file too big (>16k)" if fh.stat.size > 16 * 1024
+          user_data = fh.read
+        end
+        # Figure out how to invoke file_handler, and then invoke it
+        if user_data_file == "-"
+          file_handler[$stdin]
+        else
+          File.open user_data_file, "r", &file_handler
+        end
+      end
+
+      if user_data
+        unless options[:e]
+          require 'base64'
+          user_data = Base64.encode64(user_data)
+        end
+        raise "User data too big (>16k)" if user_data.size > 16 * 1024
+      end
+
+      params = {}
+      params[:name] = options[:n] if options[:n]
+      params[:user_data] = user_data if user_data
+
+      info "Updating server #{server}#{" with %.2fk of user data" % (user_data.size / 1024.0) if user_data}"
+      server.update params
+      server.reload
+      render_table([server], global_options)
+    end
+  end
+end

data/lib/brightbox-cli/config.rb

@@ -117,7 +117,8 @@ module Brightbox
         :brightbox_api_url => c['api_url'],
         :brightbox_auth_url => c['auth_url'] || c['api_url'],
         :brightbox_client_id => c['client_id'],
-        :brightbox_secret => c['secret']
+        :brightbox_secret => c['secret'],
+        :persistent => (c["persistent"] != nil ? c["persistent"] : true)
       }
     end
 

data/lib/brightbox-cli/fog_extensions.rb

@@ -1,13 +1,3 @@
-# Hack to force persistent connections in fog
-module Fog
-  class Connection
-    def initialize(url, persistent=false)
-      @excon = Excon.new(url)
-      @persistent = true
-    end
-  end
-end
-
 Fog::Compute::Brightbox::Real.class_eval do |klass|
   klass.send(:attr_accessor, :oauth_token)
 end

data/lib/brightbox-cli/gli_global_hooks.rb

@@ -50,7 +50,7 @@ module Brightbox
   desc 'Display version information'
   command [:version] do |c|
     c.action do |global_options, options, args|
-      info "Brightbox CLI version: #{Brightbox::VERSION}, Fog version: #{Fog::VERSION}"
+      info "Brightbox CLI version: #{Brightbox::VERSION}"
     end
   end
 end

data/lib/brightbox-cli/images.rb

@@ -1,6 +1,31 @@
 module Brightbox
   class Image < Api
 
+    def self.all
+      conn.images
+    end
+
+    def self.get(id)
+      conn.images.get(id)
+    end
+
+    def self.register(options = {})
+      image = conn.create_image(options)
+      find image['id']
+    end
+
+    def self.default_field_order
+      [:id, :owner, :type, :created_on, :status, :size, :name]
+    end
+
+    def update options
+      self.class.conn.update_image(id, options)
+      self.reload
+      self
+    rescue Excon::Errors::BadRequest => e
+      raise Conflict, JSON.parse(e.response.body)['error']['details']
+    end
+
     def to_row
       o = fog_model.attributes
       o[:id] = fog_model.id
@@ -25,25 +50,9 @@ module Brightbox
       o
     end
 
-    def self.register(options = {})
-      image = conn.create_image(options)
-      find image['id']
-    end
-
     def public?
       public
     end
 
-    def self.all
-      conn.images
-    end
-
-    def self.get(id)
-      conn.images.get(id)
-    end
-
-    def self.default_field_order
-      [:id, :owner, :type, :created_on, :status, :size, :name]
-    end
   end
 end

data/lib/brightbox-cli/servers.rb

@@ -9,6 +9,36 @@ module Brightbox
       new(conn.servers.create(options))
     end
 
+    def self.all
+      conn.servers
+    end
+
+    def self.get(id)
+      conn.servers.get id
+    end
+
+    def self.default_field_order
+      [:id, :status, :type, :zone, :created_on, :image_id, :cloud_ip_ids, :name]
+    end
+
+    def update options
+      self.class.conn.update_server id, options
+      self.reload
+      self
+    rescue Excon::Errors::BadRequest => e
+      raise Conflict, JSON.parse(e.response.body)['error']['details']
+    end
+
+    def destroy
+      fog_model.destroy
+    rescue Excon::Errors::Conflict => e
+      raise Conflict, "Cannot delete server #{id}"
+    end
+
+    def activate_console
+      self.class.conn.activate_console_server id
+    end
+
     def server_type
       @server_type ||= (Type.new(flavor_id) if flavor_id)
     end
@@ -41,28 +71,6 @@ module Brightbox
       fog_model.status == "deleted"
     end
 
-    def destroy
-      fog_model.destroy
-    rescue Excon::Errors::Conflict => e
-      raise Conflict, "Cannot delete server #{id}"
-    end
-
-    def activate_console
-      self.class.conn.activate_console_server id
-    end
-
-    def self.get(id)
-      conn.servers.get id
-    end
-
-    def self.all
-      conn.servers
-    end
-
-    def self.default_field_order
-      [:id, :status, :type, :zone, :created_on, :image_id,:cloud_ip_ids,:name]
-    end
-
     def hostname
       "#{id}.gb1.brightbox.com"
     end

data/lib/brightbox-cli/tables.rb

@@ -83,7 +83,7 @@ module Brightbox
         if options[:vertical]
           data options[:fields].collect { |k| [k, row[k]].join("\t") }.join("\n")
         else
-          data options[:fields].collect { |k| row[k] }.join("\t")
+          data options[:fields].collect { |k| row[k].is_a?(Array) ? row[k].join(',') : row[k] }.join("\t")
         end
       end
     else

data/lib/brightbox-cli/vendor/fog/docs/about/press.markdown

@@ -5,6 +5,10 @@ title:  Press
 
 Mentions and blog posts from elsewhere in reverse chronological order by day (and alphasorted for same days).
 
+**September 13th, 2011**
+
+* [Libvirt support for fog](http://jedi.be/blog/2011/09/13/libvirt-fog-provider/)
+
 **August 1st, 2011**
 
 * [Using EBS Snapshots with Fog](http://www.mediamolecule.com/lab/article/using_ebs_snapshots_with_fog/)

data/lib/brightbox-cli/vendor/fog/docs/compute/index.markdown

@@ -35,7 +35,7 @@ With that in hand we are ready to start making EC2 calls!
 
 ## Servers the EC2 way
 
-Creating a server on EC2 is very easy if you are willing to accept the defaults (the smallest server size, using Ubuntu 10.04 LTS).
+Creating a server on EC2 is very easy if you are willing to accept the defaults (the smallest server size, using Ubuntu 10.04 LTS). NOTE: the default EC2 image uses the 'ubuntu' username, rather than 'root' like other services.
 
     server = connection.servers.create
 

data/lib/brightbox-cli/vendor/fog/docs/dns/index.markdown

@@ -66,7 +66,7 @@ You can add more specifics if you need to, but reasonable defaults make it just
 
 ## No Zerigo? No Problem
 
-If you already have an account with another service you can just as easily use this same code with different credentials. fog currently supports <a href="http://aws.amazon.com/route53/">AWS Route 53</a>, <a href="http://bluebox.net">Blue Box</a>, <a href="http://dnsimple.com">DNSimple</a>, <a href="http://www.linode.com">Linode</a>, <a href="http://www.slicehost.com">Slicehost</a> and <a href="http://www.zerigo.com/managed-dns">Zerigo</a>; so you can have your pick.  As an example you can connect to AWS instead of Zerigo:
+If you already have an account with another service you can just as easily use this same code with different credentials. fog currently supports <a href="http://aws.amazon.com/route53/">AWS Route 53</a>, <a href="http://bluebox.net">Blue Box</a>, <a href="http://dnsimple.com">DNSimple</a>, <a href="http://www.linode.com">Linode</a>, <a href="http://www.rackspace.com">Rackspace</a>, <a href="http://www.slicehost.com">Slicehost</a> and <a href="http://www.zerigo.com/managed-dns">Zerigo</a>; so you can have your pick.  As an example you can connect to AWS instead of Zerigo:
 
     dns = Fog::DNS.new({
       :provider               => 'AWS',

data/lib/brightbox-cli/vendor/fog/fog.gemspec

@@ -54,6 +54,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency('rspec', '~>1.3.1')
   s.add_development_dependency('shindo', '~>0.3.4')
   s.add_development_dependency('virtualbox', '~>0.9.1')
+#  s.add_development_dependency('ruby-libvirt','~>0.4.0')
 
   s.files = `git ls-files`.split("\n")
   s.test_files = `git ls-files -- {spec,tests}/*`.split("\n")