brightbox-cli 1.2.2 → 1.3.0

data/bin/brightbox

@@ -4,8 +4,8 @@ begin
   require "brightbox_cli"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
 end
 
-Brightbox::run ARGV
+Brightbox.run ARGV

data/bin/brightbox-accounts

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("accounts")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("accounts")

data/bin/brightbox-cloudips

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("cloudips")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("cloudips")

data/bin/brightbox-config

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("config")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("config")

data/bin/brightbox-firewall-policies

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("firewall-policies")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("firewall-policies")

data/bin/brightbox-firewall-rules

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("firewall-rules")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("firewall-rules")

data/bin/brightbox-groups

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("groups")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("groups")

data/bin/brightbox-images

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("images")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("images")

data/bin/brightbox-lbs

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("lbs")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("lbs")

data/bin/brightbox-servers

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("servers")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("servers")

data/bin/brightbox-types

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("types")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("types")

data/bin/brightbox-users

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("users")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("users")

data/bin/brightbox-zones

@@ -5,9 +5,9 @@ begin
   require "brightbox-cli/legacy/args_adjuster"
 rescue LoadError
   brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
+  $LOAD_PATH.unshift(brightbox_cli_path)
   require "brightbox_cli"
   require "brightbox-cli/legacy/args_adjuster"
 end
 
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("zones")
+Brightbox.run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("zones")

data/brightbox-cli.gemspec

@@ -8,31 +8,30 @@ Gem::Specification.new do |s|
   s.authors     = ["John Leach"]
   s.email       = ["john@brightbox.co.uk"]
   s.homepage    = "http://docs.brightbox.com/cli"
-  s.summary     = %q{The Brightbox cloud management system}
-  s.description = %q{Scripts to interact with the Brightbox cloud API}
+  s.summary     = %q(The Brightbox cloud management system)
+  s.description = %q(Scripts to interact with the Brightbox cloud API)
   s.license     = "MIT"
 
   s.rubyforge_project = "brightbox-cli"
 
   s.files         = `git ls-files`.split("\n") + `find lib/brightbox-cli/vendor`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
   s.require_paths = ["lib"]
 
+  s.add_dependency "fog-brightbox", "~> 0.3"
   s.add_dependency "gli", "~> 2.9"
-  s.add_dependency "fog-brightbox"
-  s.add_dependency "multi_json"
-  s.add_dependency "mime-types", "~> 1.25"
-
   s.add_dependency "i18n"
-
-  s.add_dependency('hirb','~> 0.6')
-  s.add_dependency('highline', '~> 1.6')
-
-  s.add_development_dependency('rake')
-  s.add_development_dependency('vcr', '~> 2.5')
-  s.add_development_dependency('rspec', '~> 2.14')
-  s.add_development_dependency('mocha')
+  s.add_dependency "mime-types", "~> 1.25"
+  s.add_dependency "multi_json"
+  s.add_dependency "highline", "~> 1.6"
+  s.add_dependency "hirb", "~> 0.6"
+
+  s.add_development_dependency "mocha"
+  s.add_development_dependency "pry"
+  s.add_development_dependency "rake"
+  s.add_development_dependency "rspec", "~> 2.99"
+  s.add_development_dependency "vcr", "~> 2.5"
 
   s.post_install_message = <<-EOS
 The CLI command is now `brightbox` with subcommands for the resources.

data/lib/brightbox-cli/api.rb

@@ -99,9 +99,7 @@ module Brightbox
     def self.find(args = :all, options = {})
       raise InvalidArguments, "find(nil)" if args.nil?
       raise InvalidArguments, "find([])" if args.respond_to?(:empty?) && args.empty?
-      options = {
-        :order => :created_at,
-      }.merge options
+      options = { :order => :created_at }.merge(options)
 
       objects = nil
       object = nil
@@ -110,11 +108,11 @@ module Brightbox
         objects = all
       elsif args.is_a? String
         object = cached_get(args.to_s)
-        raise NotFound, "Couldn't find '#{args.to_s}'" if object.nil?
+        raise NotFound, "Couldn't find '#{args}'" if object.nil?
       elsif args.respond_to? :map
         objects = args.map do |arg|
           o = cached_get(arg.to_s)
-          raise NotFound, "Couldn't find '#{arg.to_s}'" if o.nil?
+          raise NotFound, "Couldn't find '#{arg}'" if o.nil?
           o
         end
       else
@@ -140,7 +138,7 @@ module Brightbox
 
     # Find each id in the given array.  Yield the block with any ids
     # that couldn't be found
-    def self.find_or_call(ids, &block)
+    def self.find_or_call(ids, &_block)
       objects = []
       ids.each do |id|
         begin

data/lib/brightbox-cli/collaboration.rb

@@ -29,9 +29,7 @@ module Brightbox
       @id
     end
 
-    def id
-      @id
-    end
+    attr_reader :id
 
     def attributes
       fog_model.attributes

data/lib/brightbox-cli/commands/accounts/accept-invite.rb

@@ -5,7 +5,7 @@ module Brightbox
     cmd.desc I18n.t("accounts.accept_invite.desc")
     cmd.arg_name "account_id"
     cmd.command [:accept_invite] do |c|
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         account_id = args.first
 
         # Find the collaboration for that account

data/lib/brightbox-cli/commands/accounts/default.rb

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "account-id"
     cmd.command [:default] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |_global_options, _options, args|
         if args.empty?
           raise "You must specify the account-id to set as default."
         end

data/lib/brightbox-cli/commands/accounts/list.rb

@@ -7,7 +7,7 @@ module Brightbox
     cmd.arg_name "[account-id...]"
     cmd.command [:list] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, _args|
         if $config.using_application?
           # Collaborating Accounts are combined from owned and collaborations
           accounts = CollaboratingAccount.all

data/lib/brightbox-cli/commands/accounts/remove.rb

@@ -4,7 +4,7 @@ module Brightbox
     cmd.desc I18n.t("accounts.remove.desc")
     cmd.arg_name "account_id"
     cmd.command [:remove] do |c|
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         account_id = args.first
 
         # Find the collaboration for that account

data/lib/brightbox-cli/commands/accounts/reset-ftp-password.rb

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "account-id..."
     cmd.command [:reset_ftp_password] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
 
         if args.empty?
           raise "You must specify the accounts to reset ftp passwords for"

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

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "account-id..."
     cmd.command [:show] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
 
         if args.empty?
           raise "You must specify the accounts to show"

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

@@ -15,7 +15,7 @@ module Brightbox
       c.desc I18n.t("cloudips.options.port_translators.desc")
       c.flag [:t, :"port-translators"]
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, options, _args|
 
         if options[:i].to_s !~ /^[0-9]+$/
           raise "count must be a number"

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

@@ -8,7 +8,7 @@ module Brightbox
       c.desc "Unmap mapped IPs before destroying them"
       c.switch [:u, "unmap"], :negatable => false
 
-      c.action do |global_options, options, args|
+      c.action do |_global_options, options, args|
 
         if args.empty?
           raise "You must specify the Cloud IPs you want to destroy"

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

@@ -7,7 +7,7 @@ module Brightbox
     cmd.arg_name "[cloudip-id...]"
     cmd.command [:list] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         ips = CloudIP.find_all_or_warn(args)
         render_table(ips.sort, global_options)
       end

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

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "cloudip-id..."
     cmd.command [:show] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
 
         if args.empty?
           raise "You must specify the cloud ips you want to show"

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

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "cloudip-id..."
     cmd.command [:unmap] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
 
         if args.empty?
           raise "You must specify the Cloud IPs you want to unmap"

data/lib/brightbox-cli/commands/collaborations.rb

@@ -7,7 +7,7 @@ module Brightbox
     cmd.desc "List an accounts collaborations"
     cmd.arg_name "[collaboration-id...]"
     cmd.command [:list] do |c|
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         collaborations = Collaboration.find_all_or_warn(args)
         render_table(collaborations, global_options)
       end
@@ -32,7 +32,7 @@ module Brightbox
     cmd.desc "Resends an invitation email for a pending collaboration"
     cmd.arg_name "collaboration-id"
     cmd.command [:resend] do |c|
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         collaborations = Collaboration.find_or_call(args) do |id|
           warn "Couldn't find collaboration #{id}"
         end
@@ -44,7 +44,7 @@ module Brightbox
     cmd.desc "Show details of collaborations"
     cmd.arg_name "[collaboration-id...]"
     cmd.command [:show] do |c|
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         collaborations = Collaboration.find_or_call(args) do |id|
           warn "Couldn't find collaboration #{id}"
         end
@@ -55,7 +55,7 @@ module Brightbox
     cmd.desc "Destroy collaborations"
     cmd.arg_name "[collaboration-id...]"
     cmd.command [:destroy] do |c|
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         collaborations = Collaboration.find_or_call(args) do |id|
           warn "Couldn't find collaboration #{id}"
         end

data/lib/brightbox-cli/commands/config/client-add.rb

@@ -12,7 +12,7 @@ module Brightbox
       c.desc "auth url"
       c.flag [:t, "auth-url"]
 
-      c.action do |global_options, options, args|
+      c.action do |_global_options, options, args|
 
         info "Using config file #{$config.config_filename}"
 

data/lib/brightbox-cli/commands/config/client-default.rb

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "alias"
     cmd.command [:client_default] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |_global_options, _options, args|
 
         info "Using config file #{$config.config_filename}"
         calias = args.shift

data/lib/brightbox-cli/commands/config/client-list.rb

@@ -6,7 +6,7 @@ module Brightbox
     cmd.desc I18n.t("config.client_list.desc")
     cmd.command [:client_list] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, _args|
 
         info "Using config file #{$config.config_filename}"
 

data/lib/brightbox-cli/commands/config/client-remove.rb

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "alias"
     cmd.command [:client_remove] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |_global_options, _options, args|
 
         info "Using config file #{$config.config_filename}"
 

data/lib/brightbox-cli/commands/config/user-add.rb

@@ -11,7 +11,7 @@ module Brightbox
       c.desc "password, if not specified you will be prompted"
       c.flag [:p, "password"]
 
-      c.action do |global_options, options, args|
+      c.action do |_global_options, options, args|
 
         require 'highline'
 

data/lib/brightbox-cli/commands/firewall/policies-destroy.rb

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "[firewall-policy-id...]"
     cmd.command [:destroy] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |_global_options, _options, args|
 
         raise "You must specify firewall-policy-id to destroy" if args.empty?
 

data/lib/brightbox-cli/commands/firewall/policies-list.rb

@@ -7,7 +7,7 @@ module Brightbox
     cmd.arg_name "[firewall-policy-id...]"
     cmd.command [:list] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         firewall_policies = FirewallPolicy.find_all_or_warn(args)
         render_table(firewall_policies, global_options)
       end

data/lib/brightbox-cli/commands/firewall/policies-show.rb

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "firewall-policy-id"
     cmd.command [:show] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         raise "You must specify server groups to show" if args.empty?
 
         policies = FirewallPolicy.find_or_call(args) do |id|

data/lib/brightbox-cli/commands/firewall/rules-destroy.rb

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "[firewall-rule-id...]"
     cmd.command [:destroy] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |_global_options, _options, args|
         raise "You must specify firewall-rule-id to destroy" if args.empty?
 
         firewall_rules = FirewallRule.find_or_call(args) do |id|

data/lib/brightbox-cli/commands/firewall/rules-list.rb

@@ -7,7 +7,7 @@ module Brightbox
     cmd.arg_name "[firewall-policy-id...]"
     cmd.command [:list] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         if args.empty?
           raise "You must specify the firewall policy id as the first argument"
         end

data/lib/brightbox-cli/commands/firewall/rules-show.rb

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "firewall-rule-id"
     cmd.command [:show] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         raise "You must specify server groups to show" if args.empty?
 
         policies = FirewallRule.find_or_call(args) do |id|

data/lib/brightbox-cli/commands/groups/add-server.rb

@@ -6,7 +6,7 @@ module Brightbox
     cmd.arg_name "grp-id [srv-id...]"
     cmd.command [:add_servers] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         grp_id = args.shift
         raise "You must specify the server group and then the server ids to add" unless grp_id && grp_id[/^grp-/] && !args.empty?
 

data/lib/brightbox-cli/commands/groups/create.rb

@@ -9,7 +9,7 @@ module Brightbox
       c.desc I18n.t("options.description.desc")
       c.flag [:d, :description]
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, options, _args|
         params = {}
 
         params[:name] = options[:n] if options[:n]

data/lib/brightbox-cli/commands/groups/destroy.rb

@@ -6,7 +6,7 @@ module Brightbox
     # TODO: add option to remove all servers from group before destroying
     cmd.command [:destroy] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |_global_options, _options, args|
         raise "You must specify the server groups to destroy" if args.empty?
 
         sgs = ServerGroup.find_or_call(args) do |id|

data/lib/brightbox-cli/commands/groups/list.rb

@@ -5,7 +5,7 @@ module Brightbox
 
     cmd.desc I18n.t("groups.list.desc")
     cmd.command [:list] do |c|
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         server_groups = ServerGroup.find_all_or_warn(args)
         render_table(server_groups, global_options)
       end

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

@@ -5,7 +5,7 @@ module Brightbox
     cmd.arg_name "[grp-id..]"
     cmd.command [:show] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
         raise "You must specify server groups to show" if args.empty?
 
         sgs = DetailedServerGroup.find_or_call(args) do |id|

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

@@ -6,7 +6,7 @@ module Brightbox
     cmd.arg_name "image-id..."
     cmd.command [:destroy] do |c|
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, _options, args|
 
         if args.empty?
           raise "You must specify the images you want to destroy"

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

@@ -0,0 +1,37 @@
+module Brightbox
+  command [:images] do |cmd|
+    cmd.desc I18n.t("images.lock.desc")
+    cmd.arg_name "[image-id...]"
+    cmd.command [:lock] do |c|
+      c.action do |_global_options, _options, args|
+        raise "You must specify image as arguments" if args.empty?
+
+        images = Image.find_or_call(args) do |image|
+          raise "Couldn't find #{image}"
+        end
+
+        images.each do |image|
+          info "Locking #{image}"
+          image.lock!
+        end
+      end
+    end
+
+    cmd.desc I18n.t("images.unlock.desc")
+    cmd.arg_name "[image-id...]"
+    cmd.command [:unlock] do |c|
+      c.action do |_global_options, _options, args|
+        raise "You must specify images as arguments" if args.empty?
+
+        images = Image.find_or_call(args) do |image|
+          raise "Couldn't find #{image}"
+        end
+
+        images.each do |image|
+          info "Unlocking #{image}"
+          image.unlock!
+        end
+      end
+    end
+  end
+end

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

@@ -27,7 +27,7 @@ module Brightbox
       c.default_value "false"
       c.flag [:p, "public"]
 
-      c.action do |global_options, options, args|
+      c.action do |global_options, options, _args|
 
         raise "You must specify the architecture" unless options[:a]
         raise "You must specify the source filename" unless options[:s]