opennebula-cli 6.6.2 → 6.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 21c8a6e2683191cfb58031afd725ea75ddad8d6034c2002c93e5a5d9f86c61c2
-  data.tar.gz: 8aa8cfbbac6a3f522c7e04646c63681580608b417ab87b1804dd9b28284c0eca
+  metadata.gz: 7e74dc25cf00cc6c901701dcfef74bb0d5f34145ca4ce12d3a2aae23f4f0ce83
+  data.tar.gz: fabdc4659737fb9adc7219d04b0057034ad04503f1c28e68912c748f02405952
 SHA512:
-  metadata.gz: 784f2bc0129a173ce3ce6e9699544fc835e4999524907a751d160713306744aee8bb3c73987f6af0665a34882fbb8f9ac5792f08f79e505ef7770ab371df5d63
-  data.tar.gz: ac3df996ca1205fc96fc1e0fc23fa66e91f78d75b2cf2d58fc2d7750ff51742c605b896dcc16c07b8e8bb9f0fa04208d0bfe24cb5072fab360930aa26ad69e6f
+  metadata.gz: a2cd0831f008fb2b52ea545c2d1615a8cc0e31e376199e6b52dab49553b1677e6628dbafd0fde22544562d4c8bea00572ce1b20e353dc45e096e5f721d5a35c9
+  data.tar.gz: 10d16e841304506cb3978fd67d8349e7f7684bbe70fc2f7351a2fcc1129f41af9a626a8580099f24415d879d280b1e085ce7b29ff523e35b13e6c04d66e0a71d

data/bin/oneacl

@@ -113,7 +113,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists the ACL rule set
+        Lists the ACL rule set. #{OneAclHelper.list_layout_help}
     EOT
 
     command :list, list_desc, :options => [CLIHelper::OPTIONS,

data/bin/onecluster

@@ -121,7 +121,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Clusters in the pool
+        Lists Clusters in the pool. #{OneClusterHelper.list_layout_help}
     EOT
 
     command :list, list_desc, :options => list_options do

data/bin/onedatastore

@@ -163,7 +163,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Datastores in the pool
+        Lists Datastores in the pool. #{OneDatastoreHelper.list_layout_help}
     EOT
 
     command :list, list_desc, :options => list_options do

data/bin/onegroup

@@ -168,7 +168,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Groups in the pool
+        Lists Groups in the pool. #{OneGroupHelper.list_layout_help}
     EOT
 
     command :list, list_desc, :options => list_options do

data/bin/onehook

@@ -229,7 +229,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Hooks in the pool
+        Lists Hooks in the pool. #{OneHookHelper.list_layout_help}
     EOT
 
     command :list, list_desc, [:filterflag, nil], :options => list_options do

data/bin/onehost

@@ -297,7 +297,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Hosts in the pool
+        Lists Hosts in the pool. #{OneHostHelper.list_layout_help}
     EOT
 
     command :list, list_desc,

data/bin/oneimage

@@ -478,7 +478,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Images in the pool
+        Lists Images in the pool. #{OneImageHelper.list_layout_help}
     EOT
 
     command :list, list_desc, [:filterflag, nil], :options => list_options do

data/bin/onemarket

@@ -156,7 +156,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Marketplaces
+        Lists Marketplaces. #{OneMarketPlaceHelper.list_layout_help}
     EOT
 
     command :list, list_desc, :options => list_options do

data/bin/onemarketapp

@@ -403,7 +403,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists marketplace apps
+        Lists marketplace apps. #{OneMarketPlaceAppHelper.list_layout_help}
     EOT
 
     command :list, list_desc, [:filterflag, nil], :options => list_options do

data/bin/onesecgroup

@@ -224,7 +224,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Security Group in the pool
+        Lists Security Group in the pool. #{OneSecurityGroupHelper.list_layout_help}
     EOT
 
     command :list, list_desc, [:filterflag, nil], :options => list_options do

data/bin/onetemplate

@@ -404,7 +404,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Templates in the pool
+        Lists Templates in the pool. #{OneTemplateHelper.list_layout_help}
     EOT
 
     rename_desc = <<-EOT.unindent

data/bin/oneuser

@@ -562,7 +562,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Users in the pool
+        Lists Users in the pool. #{OneUserHelper.list_layout_help}
     EOT
 
     command :list, list_desc, :options => list_options do

data/bin/onevdc

@@ -300,7 +300,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists VDCs in the pool
+        Lists VDCs in the pool. #{OneVdcHelper.list_layout_help}
     EOT
 
     command :list, list_desc, :options => list_options do

data/bin/onevm

@@ -1278,7 +1278,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists VMs in the pool
+        Lists VMs in the pool. #{OneVMHelper.list_layout_help}
     EOT
 
     command :list, list_desc, [:filterflag, nil],
@@ -1485,11 +1485,11 @@ CommandParser::CmdParser.new(ARGV) do
     create_chart_desc = <<-EOT.unindent
         Adds a charter to the VM, these are some consecutive scheduled actions
 
-        You can configure the actions in /etc/one/cli/onevm.yaml
+        You can configure the actions in #{OneVMHelper.conf_file}
     EOT
 
     command :'create-chart', create_chart_desc, :vmid do
-        charters = helper.get_charters
+        charters = helper.charters
 
         if charters.nil?
             STDERR.puts 'No charters found on configuration file'
@@ -1512,10 +1512,7 @@ CommandParser::CmdParser.new(ARGV) do
                                              action.to_s,
                                              time[:warning][:time] + acc_w)
 
-                if OpenNebula.is_error?(rc)
-                    STDERR.puts rc.message
-                    exit(-1)
-                end
+                exit(-1) if rc < 0
 
                 acc_t += time[:time].to_i
                 acc_w += time[:warning][:time].to_i
@@ -1578,7 +1575,7 @@ CommandParser::CmdParser.new(ARGV) do
         reset = options[:reset] == true
 
         if !options[:schedule].nil?
-            options[:args] = options[:datastore]
+            options[:args] = "#{options[:datastore]},#{reset ? 1 : 0}"
 
             helper.schedule_actions([args[0]], options, @comm_name)
         else

data/bin/onevmgroup

@@ -151,7 +151,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists VM Group in the pool
+        Lists VM Group in the pool. #{OneVMGroupHelper.list_layout_help}
     EOT
 
     command :list, list_desc, [:filterflag, nil], :options => list_options do

data/bin/onevnet

@@ -347,7 +347,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Virtual Networks in the pool
+        Lists Virtual Networks in the pool. #{OneVNetHelper.list_layout_help}
     EOT
 
     command :list, list_desc, [:filterflag, nil],

data/bin/onevntemplate

@@ -340,7 +340,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists VN Templates in the pool
+        Lists VN Templates in the pool. #{OneVNTemplateHelper.list_layout_help}
     EOT
 
     rename_desc = <<-EOT.unindent

data/bin/onevrouter

@@ -341,7 +341,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists the Virtual Routers in the pool
+        Lists the Virtual Routers in the pool. #{OneVirtualRouterHelper.list_layout_help}
     EOT
 
     command :list, list_desc, [:filterflag, nil], :options => list_options do

data/bin/onezone

@@ -211,7 +211,7 @@ CommandParser::CmdParser.new(ARGV) do
     end
 
     list_desc = <<-EOT.unindent
-        Lists Zones in the pool
+        Lists Zones in the pool. #{OneZoneHelper.list_layout_help}
     EOT
 
     command :list, list_desc, :options => list_options do

data/lib/one_helper/onevm_helper.rb

@@ -267,8 +267,8 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
         end
 
         vm_nics.each do |nic|
-            %w[IP EXTERNAL_IP IP6_GLOBAL IP6_ULA IP6
-               VROUTER_IP VROUTER_IP6_GLOBAL VROUTER_IP6_ULA].each do |attr|
+            ['IP', 'EXTERNAL_IP', 'IP6_GLOBAL', 'IP6_ULA', 'IP6', 'VROUTER_IP',
+             'VROUTER_IP6_GLOBAL', 'VROUTER_IP6_ULA'].each do |attr|
                 if nic.key?(attr)
                     ips.push(nic[attr])
                 end
@@ -333,7 +333,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
             end
         end
 
-        table = CLIHelper::ShowTable.new(config_file, self) do
+        CLIHelper::ShowTable.new(config_file, self) do
             column :ID, 'ONE identifier for Virtual Machine', :size => 6 do |d|
                 d['ID']
             end
@@ -377,7 +377,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
                    :left, :size => 10 do |d|
                 if d['HISTORY_RECORDS'] && d['HISTORY_RECORDS']['HISTORY']
                     state_str = VirtualMachine::VM_STATE[d['STATE'].to_i]
-                    if %w[ACTIVE SUSPENDED POWEROFF].include? state_str
+                    if ['ACTIVE', 'SUSPENDED', 'POWEROFF'].include? state_str
                         d['HISTORY_RECORDS']['HISTORY']['HOSTNAME']
                     end
                 end
@@ -414,8 +414,6 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
             default :ID, :USER, :GROUP, :NAME, :STAT, :CPU, :MEM, :HOST,
                     :TIME
         end
-
-        table
     end
 
     def schedule_actions(ids, options, action, warning = nil)
@@ -428,8 +426,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
                       "#{action} scheduled after #{options[:schedule]}s from start"
                   end
 
-        perform_actions( ids, options, message) do |vm|
-
+        perform_actions(ids, options, message) do |vm|
             str_periodic = ''
 
             if options.key?(:weekly)
@@ -457,7 +454,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
                 str_periodic << ', END_TYPE = 0'
             end
 
-            tmp_str = "SCHED_ACTION = ["
+            tmp_str = 'SCHED_ACTION = ['
             tmp_str << "ACTION = #{action}, "
             tmp_str << "WARNING = #{warning}," if warning
             tmp_str << "ARGS = \"#{options[:args]}\"," if options[:args]
@@ -467,7 +464,8 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
             rc = vm.sched_action_add(tmp_str)
 
             if OpenNebula.is_error?(rc)
-                return rc
+                STDERR.puts rc.message
+                return -1
             end
         end
     end
@@ -600,7 +598,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
                 EOF
 
                 ans = ''
-                until %w[n a r q].include?(ans)
+                until ['n', 'a', 'r', 'q'].include?(ans)
                     printf '> '
                     ans = STDIN.gets.strip.downcase
 
@@ -618,7 +616,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
 
                         if result == 'SUCCESS'
                             success = true
-                            puts result.to_s
+                            puts result
                         else
                             puts
                             puts "#{result}. Repeat command.".red
@@ -660,7 +658,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
     #       - action
     #           - time
     #           - warning
-    def get_charters
+    def charters
         YAML.load_file(self.class.table_conf)[:charters]
     end
 
@@ -725,9 +723,9 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
             options[:ssh_opts].nil? ? opts = '' : opts = options[:ssh_opts]
 
             if opts.empty?
-                exec(*%W[ssh #{login}@#{ip} -p #{port} #{cmd}])
+                exec('ssh', "#{login}@#{ip}", '-p', port.to_s, cmd.to_s)
             else
-                exec('ssh', *opts.split, *%W[#{login}@#{ip} -p #{port} #{cmd}])
+                exec('ssh', *opts.split, "#{login}@#{ip}", '-p', port.to_s, cmd.to_s)
             end
         end
 
@@ -759,7 +757,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
 
         vm_hash = vm.to_hash
 
-        if %w[ACTIVE SUSPENDED POWEROFF].include? vm.state_str
+        if ['ACTIVE', 'SUSPENDED', 'POWEROFF'].include? vm.state_str
             cluster_id = vm['/VM/HISTORY_RECORDS/HISTORY[last()]/CID']
         else
             cluster_id = nil
@@ -794,7 +792,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
                     OpenNebulaHelper.level_lock_to_str(vm['LOCK/LOCKED']))
         puts format(str, 'RESCHED',
                     OpenNebulaHelper.boolean_to_str(vm['RESCHED']))
-        if %w[ACTIVE SUSPENDED POWEROFF].include? vm.state_str
+        if ['ACTIVE', 'SUSPENDED', 'POWEROFF'].include? vm.state_str
             puts format(str, 'HOST',
                         vm['/VM/HISTORY_RECORDS/HISTORY[last()]/HOSTNAME'])
         end
@@ -823,7 +821,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
             end
         end
 
-        order_attrs = %w[CPU MEMORY NETTX NETRX]
+        order_attrs = ['CPU', 'MEMORY', 'NETTX', 'NETRX']
 
         vm_monitoring_sort = []
         order_attrs.each do |key|
@@ -834,7 +832,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
 
         vm_monitoring_sort.sort_by {|a| a[0] }
 
-        filter_attrs = %w[STATE DISK_SIZE SNAPSHOT_SIZE]
+        filter_attrs = ['STATE', 'DISK_SIZE', 'SNAPSHOT_SIZE']
         vm_monitoring.each do |key, val|
             if !filter_attrs.include?(key)
                 vm_monitoring_sort << [key, val]
@@ -856,7 +854,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
 
         CLIHelper.print_header(str_h1 % 'PERMISSIONS', false)
 
-        %w[OWNER GROUP OTHER].each do |e|
+        ['OWNER', 'GROUP', 'OTHER'].each do |e|
             mask = '---'
             mask[0] = 'u' if vm["PERMISSIONS/#{e}_U"] == '1'
             mask[1] = 'm' if vm["PERMISSIONS/#{e}_M"] == '1'
@@ -969,9 +967,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
             end.show(vm_disks, {})
 
             if !options[:all]
-                while vm.has_elements?('/VM/TEMPLATE/DISK')
-                    vm.delete_element('/VM/TEMPLATE/DISK')
-                end
+                vm.delete_element('/VM/TEMPLATE/DISK') while vm.has_elements?('/VM/TEMPLATE/DISK')
             end
         end
 
@@ -1006,17 +1002,17 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
         # array. Duplicate IPs are not shown.
         extra_ips = []
 
-        if (val = vm['/VM/MONITORING/GUEST_IP'])
-            extra_ips << val if val && !val.empty?
+        if (val = vm['/VM/MONITORING/GUEST_IP']) && (val && !val.empty?)
+            extra_ips << val
         end
 
-        if (val = vm['/VM/MONITORING/GUEST_IP_ADDRESSES'])
-            extra_ips += val.split(',') if val && !val.empty?
+        if (val = vm['/VM/MONITORING/GUEST_IP_ADDRESSES']) && (val && !val.empty?)
+            extra_ips += val.split(',')
         end
 
         extra_ips.uniq!
 
-        %w[NIC NIC_ALIAS].each do |type|
+        ['NIC', 'NIC_ALIAS'].each do |type|
             next unless vm.has_elements?("/VM/TEMPLATE/#{type}") ||
                         vm.has_elements?('/VM/TEMPLATE/PCI[NIC_ID>-1]') ||
                         !extra_ips.empty?
@@ -1052,7 +1048,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
             vm_nics.each do |nic|
                 next if nic.key?('CLI_DONE')
 
-                %w[EXTERNAL_IP IP6_LINK IP6_ULA IP6_GLOBAL IP6].each do |attr|
+                ['EXTERNAL_IP', 'IP6_LINK', 'IP6_ULA', 'IP6_GLOBAL', 'IP6'].each do |attr|
                     next unless nic.key?(attr)
 
                     shown_ips << nic[attr]
@@ -1065,8 +1061,8 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
                     array_id += 1
                 end
 
-                %w[VROUTER_IP VROUTER_IP6_LINK
-                   VROUTER_IP6_ULA VROUTER_IP6_GLOBAL].each do |attr|
+                ['VROUTER_IP', 'VROUTER_IP6_LINK', 'VROUTER_IP6_ULA',
+                 'VROUTER_IP6_GLOBAL'].each do |attr|
                     next unless nic.key?(attr)
 
                     shown_ips << nic[attr]
@@ -1148,15 +1144,13 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
 
             next if options[:all]
 
-            while vm.has_elements?("/VM/TEMPLATE/#{type}")
-                vm.delete_element("/VM/TEMPLATE/#{type}")
-            end
+            vm.delete_element("/VM/TEMPLATE/#{type}") while vm.has_elements?("/VM/TEMPLATE/#{type}")
         end
 
         if vm_hash['VM']['TEMPLATE']['NIC']
             nic = [vm_hash['VM']['TEMPLATE']['NIC']]
             nic = nic.flatten
-            nic = nic.select {|v| !v['EXTERNAL_PORT_RANGE'].nil? }[0]
+            nic = nic.reject {|v| v['EXTERNAL_PORT_RANGE'].nil? }[0]
 
             if nic
                 ip   = vm_hash['VM']['HISTORY_RECORDS']['HISTORY']
@@ -1174,9 +1168,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
         end
 
         if !options[:all]
-            while vm.has_elements?('/VM/TEMPLATE/NIC')
-                vm.delete_element('/VM/TEMPLATE/NIC')
-            end
+            vm.delete_element('/VM/TEMPLATE/NIC') while vm.has_elements?('/VM/TEMPLATE/NIC')
         end
 
         if vm.has_elements?('/VM/TEMPLATE/SECURITY_GROUP_RULE') && !is_hybrid
@@ -1322,15 +1314,17 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
                 column :REPEAT, '', :adjust => true do |d|
                     str_rep = ''
                     if !d.nil? && d.key?('REPEAT')
-                        if d['REPEAT'] == '0'
+                        case d['REPEAT']
+                        when '0'
                             str_rep << 'Weekly '
-                        elsif d['REPEAT'] == '1'
+                        when '1'
                             str_rep << 'Monthly '
-                        elsif d['REPEAT'] == '2'
+                        when '2'
                             str_rep << 'Yearly '
-                        elsif d['REPEAT'] == '3'
+                        when '3'
                             str_rep << 'Each ' << d['DAYS'] << ' hours'
                         end
+
                         if d['REPEAT'] != '3'
                             str_rep << d['DAYS']
                         end
@@ -1341,20 +1335,19 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
                 column :END, '', :adjust => true do |d|
                     str_end = ''
                     if !d.nil? && d.key?('END_TYPE')
-                        if d['END_TYPE'] == '0'
+                        case d['END_TYPE']
+                        when '0'
                             str_end << 'None'
-                        elsif d['END_TYPE'] == '1'
+                        when '1'
                             str_end << 'After ' << d['END_VALUE'] << ' times'
-                        elsif d['END_TYPE'] == '2'
+                        when '2'
                             str_end << 'On ' << \
-                                OpenNebulaHelper.time_to_str(d['END_VALUE'],
-                                                             false, false,
-                                                             true)
+                                OpenNebulaHelper.time_to_str(d['END_VALUE'], false, false, true)
                         end
                     end
                     str_end unless d.nil?
                 end
-                column :STATUS, '', :left, :size => 50  do |d|
+                column :STATUS, '', :left, :size => 50 do |d|
                     if d['DONE'] && !d['REPEAT']
                         "Done on #{OpenNebulaHelper.time_to_str(d['DONE'], false)}"
                     elsif d['MESSAGE']
@@ -1377,7 +1370,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
                         elsif minutes > 0
                             "Next in #{minutes} minutes"
                         else
-                            "Overdue!"
+                            'Overdue!'
                         end
                     end
                 end
@@ -1427,18 +1420,18 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
     def print_backups(vm, vm_hash)
         if vm.has_elements?('/VM/BACKUPS/BACKUP_CONFIG')
             puts
-            CLIHelper.print_header('%-80s' % 'BACKUP CONFIGURATION', false)
+            CLIHelper.print_header(format('%-80s', 'BACKUP CONFIGURATION'), false)
             puts vm.template_like_str('BACKUPS/BACKUP_CONFIG')
         end
 
-        if vm.has_elements?('/VM/BACKUPS/BACKUP_IDS')
-            puts
-            CLIHelper.print_header('%-80s' % 'VM BACKUPS', false)
+        return unless vm.has_elements?('/VM/BACKUPS/BACKUP_IDS')
 
-            ids = [vm_hash['VM']['BACKUPS']['BACKUP_IDS']['ID']].flatten
+        puts
+        CLIHelper.print_header(format('%-80s', 'VM BACKUPS'), false)
 
-            puts format('IMAGE IDS: %s', ids.join(','))
-        end
+        ids = [vm_hash['VM']['BACKUPS']['BACKUP_IDS']['ID']].flatten
+
+        puts format('IMAGE IDS: %s', ids.join(','))
     end
 
     def print_numa_nodes(numa_nodes)