gaptool-server 0.7.4 → 0.8.0

data/tasks/ec2.rb

@@ -1,42 +1,41 @@
 namespace :ec2 do
-
-  desc "Tag all (untagged) instances with role and environment"
-  task :retag do |t|
+  desc 'Tag all (untagged) instances with role and environment'
+  task :retag do |_t|
     EC2.retag
   end
 
-  desc "Rehash instances"
-  task :rehash do |t|
+  desc 'Rehash instances'
+  task :rehash do |_t|
     EC2.rehash
   end
 
-  desc "Rehash single attribute of all instances"
-  task :rehash_attr_all, [:property] do |t, args|
+  desc 'Rehash single attribute of all instances'
+  task :rehash_attr_all, [:property] do |_t, args|
     if args[:property].nil?
-      puts "Missing property"
+      puts 'Missing property'
       return 1
     end
     EC2.rehash_property(args[:property])
   end
 
-  desc "Rehash ec2 attributes for a single instance"
-  task :rehash_attrs_for, [:instance] do |t, args|
+  desc 'Rehash ec2 attributes for a single instance'
+  task :rehash_attrs_for, [:instance] do |_t, args|
     if args[:instance].nil?
-      puts "Missing instance"
+      puts 'Missing instance'
       return 1
     end
     res = EC2.rehash_properties_for_instance(args[:instance])
     return 1 unless res
   end
 
-  desc "Rehash a single property for an instance"
-  task :rehash_attr_for, [:instance, :property] do |t, args|
+  desc 'Rehash a single property for an instance'
+  task :rehash_attr_for, [:instance, :property] do |_t, args|
     if args[:instance].nil?
-      puts "Missing instance"
+      puts 'Missing instance'
       return 1
     end
     if args[:property].nil?
-      puts "Missing property"
+      puts 'Missing property'
       return 1
     end
     res = EC2.rehash_property_for_instance(args[:property], args[:instance])
@@ -46,5 +45,4 @@ namespace :ec2 do
       return 1
     end
   end
-
 end

data/tasks/gem.rb

@@ -1,21 +1,20 @@
-unless File.exists?('/.dockerenv')
+unless File.exist?('/.dockerenv')
   namespace :gem do
-
-    desc "Build the gem"
-    task :build => :clean do
+    desc 'Build the gem'
+    task build: :clean do
       sys(%w(gem build gaptool-server.gemspec))
     end
 
-    desc "Clean built file"
+    desc 'Clean built file'
     task :clean do
-      Dir.glob("*.gem") do |f|
+      Dir.glob('*.gem') do |f|
         puts " * #{f}"
         File.unlink(f)
       end
     end
 
-    desc "Push"
-    task :push => :build do
+    desc 'Push'
+    task push: :build do
       version = File.read('VERSION').strip
       sys(%W(gem push gaptool-server-#{version}.gem))
     end

data/tasks/role.rb

@@ -1,35 +1,35 @@
 require 'pp'
 
 namespace :role do
-  desc "Set security group for a role"
-  task :set_sg, [:role, :environment, :security_group] do |t, args|
-    abort("Missing parameters") if args[:role].nil? || args[:environment].nil? || args[:role].empty? || args[:environment].empty? || args[:security_group].nil? || args[:security_group].empty?
+  desc 'Set security group for a role'
+  task :set_sg, [:role, :environment, :security_group] do |_t, args|
+    abort('Missing parameters') if args[:role].nil? || args[:environment].nil? || args[:role].empty? || args[:environment].empty? || args[:security_group].nil? || args[:security_group].empty?
     role = DH.get_role_data(args[:role], args[:environment])
     role['sg'].merge!(args[:environment] => args[:security_group])
     DH.save_role_data(args[:role], role)
   end
 
-  desc "Set ami for role"
-  task :set_amis, [:role, :region, :ami_id] do |t, args|
-    abort("Missing parameters") if args[:role].nil? || args[:region].nil? || args[:role].empty? || args[:region].empty? || args[:ami_id].nil? || args[:ami_id].empty?
+  desc 'Set ami for role'
+  task :set_amis, [:role, :region, :ami_id] do |_t, args|
+    abort('Missing parameters') if args[:role].nil? || args[:region].nil? || args[:role].empty? || args[:region].empty? || args[:ami_id].nil? || args[:ami_id].empty?
     role = DH.get_role_data(args[:role])
     role['amis'].merge!(args[:region], args[:ami_id])
     DH.save_role_data(args[:role], role)
   end
 
   desc "Set runlist for role; Runlist must be a comma separated string of recipes and roles. e.g. 'recipe[recipe],role[otherrole]'"
-  task :set_chef_runlist, [:role, :chef_runlist] do |t, args|
-    abort("Missing parameters") if args[:role].nil? || args[:chef_runlist].nil? || args[:role].empty? || args[:chef_runlist].empty?
+  task :set_chef_runlist, [:role, :chef_runlist] do |_t, args|
+    abort('Missing parameters') if args[:role].nil? || args[:chef_runlist].nil? || args[:role].empty? || args[:chef_runlist].empty?
     role = DH.get_role_data(args[:role])
-    rl = args[:chef_runlist].strip.split(",").map{|x| x.strip}.to_json
+    rl = args[:chef_runlist].strip.split(',').map(&:strip).to_json
     role['chef_runlist'] = rl
     DH.save_role_data(args[:role], role)
     puts "Added runlist for role #{role}"
   end
 
-  desc "Remove a role, if there are no instances running"
-  task :remove, [:role] do |t, args|
-    abort("Missing parameters") if args[:role].nil? || args[:role].empty?
+  desc 'Remove a role, if there are no instances running'
+  task :remove, [:role] do |_t, args|
+    abort('Missing parameters') if args[:role].nil? || args[:role].empty?
     res = DH.remove_role(args[:role])
     if res == true
       puts "Removed role #{args[:role]}"
@@ -39,15 +39,15 @@ namespace :role do
   end
 end
 
-desc "List all roles (environment is optional but needed to display apps)"
-task :role, [:environment] do |t, args|
+desc 'List all roles (environment is optional but needed to display apps)'
+task :role, [:environment] do |_t, args|
   env = args[:environment]
-  puts "Roles:"
+  puts 'Roles:'
   DH.roles(env).each do |role, info|
     puts "* #{role}:"
-    puts "  - apps: #{info['apps'].join(", ")}"
-    puts "  - sg: #{info['sg'].values.join(", ")}"
-    puts "  - amis: #{info['amis'].map{|k,v| "#{v} (#{k})"}.join(", ")}"
-    puts "  - runlist: #{info['chef_runlist'].to_a.join(", ")}"
+    puts "  - apps: #{info['apps'].join(', ')}"
+    puts "  - sg: #{info['sg'].values.join(', ')}"
+    puts "  - amis: #{info['amis'].map { |k, v| "#{v} (#{k})" }.join(', ')}"
+    puts "  - runlist: #{info['chef_runlist'].to_a.join(', ')}"
   end
 end

data/tasks/server.rb

@@ -1,18 +1,18 @@
 namespace :server do
-  desc "Set terminable flag to true/false for the instance :instance"
-  task :set_terminable, [:instance, :value] do |t, args|
+  desc 'Set terminable flag to true/false for the instance :instance'
+  task :set_terminable, [:instance, :value] do |_t, args|
     if args[:instance].nil? || args[:value].nil?
-      puts "You must specify instance and true/false"
+      puts 'You must specify instance and true/false'
       return
     end
-    DH.set_server_data_attr args[:instance], 'terminable', (args[:value] == "true" ? "true" : "false")
+    DH.set_server_data_attr args[:instance], 'terminable', (args[:value] == 'true' ? 'true' : 'false')
     puts DH.get_server_data args[:instance]
   end
 
-  desc "Remove a server from the gaptool database (no actions on AWS)"
-  task :rm, [:instance] do |t, args|
+  desc 'Remove a server from the gaptool database (no actions on AWS)'
+  task :rm, [:instance] do |_t, args|
     if args[:instance].nil?
-      puts "You must select an instance"
+      puts 'You must select an instance'
       return 1
     end
     DH.rmserver(args[:instance])

data/tasks/user.rb

@@ -1,11 +1,11 @@
 namespace :user do
-  desc "Add a new user. rake user:create <username>"
-  task :create, [:username] do |t, args|
+  desc 'Add a new user. rake user:create <username>'
+  task :create, [:username] do |_t, args|
     puts DH.useradd(args[:username])[:key]
   end
 
-  desc "Rename a user. rake user:rename <oldname> <newname>"
-  task :rename, [:oldname, :newname] do |t, args|
+  desc 'Rename a user. rake user:rename <oldname> <newname>'
+  task :rename, [:oldname, :newname] do |_t, args|
     user = DH.user(args[:oldname])
     abort("Unknown user #{args[:oldname]}") if user.nil?
     DH.userdel(user[:username])
@@ -13,20 +13,20 @@ namespace :user do
     puts "User #{args[:oldname]} renamed to #{args[:newname]}"
   end
 
-  desc "Delete a user. rake user:delete <username>"
-  task :delete, [:username] do |t, args|
+  desc 'Delete a user. rake user:delete <username>'
+  task :delete, [:username] do |_t, args|
     DH.userdel(args[:username])
   end
 
-  desc "Set user key. rake user:setkey <username> <key>"
-  task :setkey, [:username, :key] do |t, args|
+  desc 'Set user key. rake user:setkey <username> <key>'
+  task :setkey, [:username, :key] do |_t, args|
     user = DH.user(args[:username])
     abort("Unknown user #{args[:username]}") if user.nil?
     puts DH.useradd(args[:username], args[:key])
   end
 end
 
-desc "List users"
+desc 'List users'
 task :user do
   puts DH.users.keys.sort.join("\n")
 end