opennebula-cli 7.1.80.pre → 7.2.1

data/lib/one_helper/onevdc_helper.rb

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------- #
-# Copyright 2002-2025, OpenNebula Project, OpenNebula Systems                #
+# Copyright 2002-2026, OpenNebula Project, OpenNebula Systems                #
 #                                                                            #
 # Licensed under the Apache License, Version 2.0 (the "License"); you may    #
 # not use this file except in compliance with the License. You may obtain    #
@@ -18,14 +18,15 @@ require 'one_helper'
 require 'opennebula/vdc'
 require 'opennebula/vdc_pool'
 
+# Helper class for VDC commands
 class OneVdcHelper < OpenNebulaHelper::OneHelper
 
     def self.rname
-        "VDC"
+        'VDC'
     end
 
     def self.conf_file
-        "onevdc.yaml"
+        'onevdc.yaml'
     end
 
     def id_list_size(list, resource)
@@ -39,20 +40,20 @@ class OneVdcHelper < OpenNebulaHelper::OneHelper
         end
     end
 
-    def format_pool(options)
+    def format_pool(_options)
         config_file = self.class.table_conf
 
-        table = CLIHelper::ShowTable.new(config_file, self) do
-            column :ID, "ONE identifier for the VDC", :size=>5 do |d|
-                d["ID"]
+        CLIHelper::ShowTable.new(config_file, self) do
+            column :ID, 'ONE identifier for the VDC', :size=>5 do |d|
+                d['ID']
             end
 
-            column :NAME, "Name of the VDC", :left, :size=>30 do |d|
-                d["NAME"]
+            column :NAME, 'Name of the VDC', :left, :size=>30 do |d|
+                d['NAME']
             end
 
-            column :GROUPS, "Number of Groups", :size=>6 do |d|
-                ids = d["GROUPS"]["ID"]
+            column :GROUPS, 'Number of Groups', :size=>6 do |d|
+                ids = d['GROUPS']['ID']
                 case ids
                 when String
                     1
@@ -63,31 +64,29 @@ class OneVdcHelper < OpenNebulaHelper::OneHelper
                 end
             end
 
-            column :CLUSTERS, "Number of Clusters", :size=>8 do |d|
-                @ext.id_list_size(d["CLUSTERS"]["CLUSTER"], "CLUSTER") rescue "-"
+            column :CLUSTERS, 'Number of Clusters', :size=>8 do |d|
+                @ext.id_list_size(d['CLUSTERS']['CLUSTER'], 'CLUSTER') rescue '-'
             end
 
-            column :HOSTS, "Number of Hosts", :size=>5 do |d|
-                @ext.id_list_size(d["HOSTS"]["HOST"], "HOST") rescue "-"
+            column :HOSTS, 'Number of Hosts', :size=>5 do |d|
+                @ext.id_list_size(d['HOSTS']['HOST'], 'HOST') rescue '-'
             end
 
-            column :VNETS, "Number of Networks", :size=>5 do |d|
-                @ext.id_list_size(d["VNETS"]["VNET"], "VNET") rescue "-"
+            column :VNETS, 'Number of Networks', :size=>5 do |d|
+                @ext.id_list_size(d['VNETS']['VNET'], 'VNET') rescue '-'
             end
 
-            column :DATASTORES, "Number of Datastores", :size=>10 do |d|
-                @ext.id_list_size(d["DATASTORES"]["DATASTORE"], "DATASTORE") rescue "-"
+            column :DATASTORES, 'Number of Datastores', :size=>10 do |d|
+                @ext.id_list_size(d['DATASTORES']['DATASTORE'], 'DATASTORE') rescue '-'
             end
 
             default :ID, :NAME, :GROUPS, :CLUSTERS, :HOSTS, :VNETS, :DATASTORES
         end
-
-        table
     end
 
     private
 
-    def factory(id=nil)
+    def factory(id = nil)
         if id
             OpenNebula::Vdc.new_with_id(id, @client)
         else
@@ -96,26 +95,26 @@ class OneVdcHelper < OpenNebulaHelper::OneHelper
         end
     end
 
-    def factory_pool(user_flag=-2)
+    def factory_pool(_user_flag = -2)
         OpenNebula::VdcPool.new(@client)
     end
 
-    def format_resource(vdc, options = {})
-        str="%-18s: %-20s"
-        str_h1="%-80s"
+    def format_resource(vdc, _options = {})
+        str='%-18s: %-20s'
+        str_h1='%-80s'
 
         CLIHelper.print_header(str_h1 % "VDC #{vdc['ID']} INFORMATION")
-        puts str % ["ID",   vdc.id.to_s]
-        puts str % ["NAME", vdc.name]
+        puts format(str, 'ID', vdc.id.to_s)
+        puts format(str, 'NAME', vdc.name)
 
         vdc_hash = vdc.to_hash
 
         groups = vdc_hash['VDC']['GROUPS']['ID']
-        if(groups != nil)
+        if(!groups.nil?)
             puts
 
             CLIHelper::ShowTable.new(nil, self) do
-                column :"GROUPS", "", :right, :size=>7 do |d|
+                column :GROUPS, '', :right, :size=>7 do |d|
                     d
                 end
             end.show([groups].flatten, {})
@@ -123,24 +122,25 @@ class OneVdcHelper < OpenNebulaHelper::OneHelper
 
         ['CLUSTER', 'HOST', 'DATASTORE', 'VNET'].each do |resource|
             res_array = vdc_hash['VDC']["#{resource}S"][resource]
-            if(res_array != nil)
-                puts
-                CLIHelper.print_header(str_h1 % "#{resource}S", false)
-
-                CLIHelper::ShowTable.new(nil, self) do
-                    column :"ZONE", "", :right, :size=>7 do |d|
-                        d['ZONE_ID']
-                    end
-
-                    column :"#{resource}", "", :right, :size=>9 do |d|
-                        d["#{resource}_ID"] == Vdc::ALL_RESOURCES ? 'ALL' : d["#{resource}_ID"]
-                    end
-                end.show([res_array].flatten, {})
-            end
+            next if res_array.nil?
+
+            puts
+            CLIHelper.print_header(str_h1 % "#{resource}S", false)
+
+            CLIHelper::ShowTable.new(nil, self) do
+                column :ZONE, '', :right, :size=>7 do |d|
+                    d['ZONE_ID']
+                end
+
+                column :"#{resource}", '', :right, :size=>9 do |d|
+                    d["#{resource}_ID"] == Vdc::ALL_RESOURCES ? 'ALL' : d["#{resource}_ID"]
+                end
+            end.show([res_array].flatten, {})
         end
 
         puts
-        CLIHelper.print_header(str_h1 % "VDC TEMPLATE", false)
+        CLIHelper.print_header(str_h1 % 'VDC TEMPLATE', false)
         puts vdc.template_str
     end
+
 end

data/lib/one_helper/onevm_helper.rb

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------- #
-# Copyright 2002-2025, OpenNebula Project, OpenNebula Systems                #
+# Copyright 2002-2026, OpenNebula Project, OpenNebula Systems                #
 #                                                                            #
 # Licensed under the Apache License, Version 2.0 (the "License"); you may    #
 # not use this file except in compliance with the License. You may obtain    #
@@ -135,13 +135,13 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
     }
 
     POFF = {
-        :name => 'poweroff',
+        :name => 'poff',
         :large => '--poff',
         :description => 'Do the migrate by poweringoff the vm'
     }
 
     POFFHARD = {
-        :name => 'poweroff_hard',
+        :name => 'poff-hard',
         :large => '--poff-hard',
         :description => 'Do the migrate by poweringoff hard the vm'
     }
@@ -558,9 +558,9 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
     end
 
     def get_migration_type(options)
-        if options[:poweroff]
+        if options[:poff]
             1
-        elsif options[:poweroff_hard]
+        elsif options[:'poff-hard']
             2
         else
             0
@@ -644,9 +644,7 @@ class OneVMHelper < OpenNebulaHelper::OneHelper
             end
         end
 
-        # rubocop:disable Style/SpecialGlobalVars
-        $?.exitstatus
-        # rubocop:enable Style/SpecialGlobalVars
+        $CHILD_STATUS.exitstatus
     end
 
     def get_backup_mode(vm_id)

data/lib/one_helper/onevmgroup_helper.rb

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------- #
-# Copyright 2002-2025, OpenNebula Project, OpenNebula Systems                #
+# Copyright 2002-2026, OpenNebula Project, OpenNebula Systems                #
 #                                                                            #
 # Licensed under the Apache License, Version 2.0 (the "License"); you may    #
 # not use this file except in compliance with the License. You may obtain    #
@@ -18,69 +18,69 @@ require 'one_helper'
 require 'opennebula/vm_group'
 require 'opennebula/vm_group_pool'
 
+# Helper class for VMGroup commands
 class OneVMGroupHelper < OpenNebulaHelper::OneHelper
+
     def self.rname
-        "VM_GROUP"
+        'VM_GROUP'
     end
 
     def self.conf_file
-        "onevmgroup.yaml"
+        'onevmgroup.yaml'
     end
 
     def format_pool(options)
         config_file = self.class.table_conf
 
-        table = CLIHelper::ShowTable.new(config_file, self) do
-            column :ID, "ONE identifier for the VM Group", :size=>4 do |d|
-                d["ID"]
+        CLIHelper::ShowTable.new(config_file, self) do
+            column :ID, 'ONE identifier for the VM Group', :size=>4 do |d|
+                d['ID']
             end
 
-            column :NAME, "Name of the VM Group", :left, :size=>15 do |d|
-                d["NAME"]
+            column :NAME, 'Name of the VM Group', :left, :size=>15 do |d|
+                d['NAME']
             end
 
-            column :USER, "Username of the VM Group owner", :left,
-                    :size=>8 do |d|
+            column :USER, 'Username of the VM Group owner', :left,
+                   :size=>8 do |d|
                 helper.user_name(d, options)
             end
 
-            column :GROUP, "Group of the VM Group", :left, :size=>8 do |d|
+            column :GROUP, 'Group of the VM Group', :left, :size=>8 do |d|
                 helper.group_name(d, options)
             end
 
-            column :VMS, "Number of VMs in the VM Group", :left, :size=>4 do |d|
-                vms   = 0
+            column :VMS, 'Number of VMs in the VM Group', :left, :size=>4 do |d|
+                vms = 0
 
-                if !d["ROLES"]["ROLE"].nil?
-	        	roles = [d["ROLES"]["ROLE"]].flatten
-                    roles.each { |r|
-                        vms += r["VMS"].split(',').size if !r["VMS"].nil?
-                    }
+                if !d['ROLES']['ROLE'].nil?
+                    roles = [d['ROLES']['ROLE']].flatten
+                    roles.each do |r|
+                        vms += r['VMS'].split(',').size unless r['VMS'].nil?
+                    end
                 end
 
                 vms
             end
 
-            column :ROLES, "Roles in the VM Group", :left, :size=>36 do |d|
+            column :ROLES, 'Roles in the VM Group', :left, :size=>36 do |d|
                 begin
-                    roles = [d["ROLES"]["ROLE"]].flatten
-                    roles_names = ""
+                    roles = [d['ROLES']['ROLE']].flatten
+                    roles_names = ''
 
                     if !roles.nil?
-                        rnames = roles.collect { |i| i["NAME"] }
-                        roles_names = rnames.join(", ") if !rnames.empty?
+                        rnames = roles.collect {|i| i['NAME'] }
+                        roles_names = rnames.join(', ') unless rnames.empty?
                     end
 
                     roles_names
-                rescue
-                    "-"
+                rescue StandardError
+                    '-'
                 end
             end
 
             default :ID, :USER, :GROUP, :NAME, :VMS, :ROLES
         end
-
-        table
     end
 
     def retrieve_role_id(vmg_id, id)
@@ -100,7 +100,7 @@ class OneVMGroupHelper < OpenNebulaHelper::OneHelper
 
     private
 
-    def factory(id=nil)
+    def factory(id = nil)
         if id
             OpenNebula::VMGroup.new_with_id(id, @client)
         else
@@ -109,84 +109,86 @@ class OneVMGroupHelper < OpenNebulaHelper::OneHelper
         end
     end
 
-    def factory_pool(user_flag=-2)
+    def factory_pool(user_flag = -2)
         OpenNebula::VMGroupPool.new(@client, user_flag)
     end
 
-    def format_resource(vmgroup, options = {})
-        str="%-15s: %-20s"
-        str_h1="%-80s"
+    def format_resource(vmgroup, _options = {})
+        str='%-15s: %-20s'
+        str_h1='%-80s'
 
         CLIHelper.print_header(
-            str_h1 % "VM GROUP #{vmgroup['ID']} INFORMATION")
-        puts str % ["ID", vmgroup.id.to_s]
-        puts str % ["NAME", vmgroup.name]
-        puts str % ["USER", vmgroup['UNAME']]
-        puts str % ["GROUP", vmgroup['GNAME']]
-        puts str % ["LOCK", OpenNebulaHelper.level_lock_to_str(vmgroup['LOCK/LOCKED'])]
+            str_h1 % "VM GROUP #{vmgroup['ID']} INFORMATION"
+        )
+        puts format(str, 'ID', vmgroup.id.to_s)
+        puts format(str, 'NAME', vmgroup.name)
+        puts format(str, 'USER', vmgroup['UNAME'])
+        puts format(str, 'GROUP', vmgroup['GNAME'])
+        puts format(str, 'LOCK', OpenNebulaHelper.level_lock_to_str(vmgroup['LOCK/LOCKED']))
 
-        CLIHelper.print_header(str_h1 % "PERMISSIONS",false)
+        CLIHelper.print_header(str_h1 % 'PERMISSIONS', false)
 
         puts
 
-        ["OWNER", "GROUP", "OTHER"].each { |e|
-            mask = "---"
-            mask[0] = "u" if vmgroup["PERMISSIONS/#{e}_U"] == "1"
-            mask[1] = "m" if vmgroup["PERMISSIONS/#{e}_M"] == "1"
-            mask[2] = "a" if vmgroup["PERMISSIONS/#{e}_A"] == "1"
+        ['OWNER', 'GROUP', 'OTHER'].each do |e|
+            mask = '---'
+            mask[0] = 'u' if vmgroup["PERMISSIONS/#{e}_U"] == '1'
+            mask[1] = 'm' if vmgroup["PERMISSIONS/#{e}_M"] == '1'
+            mask[2] = 'a' if vmgroup["PERMISSIONS/#{e}_A"] == '1'
 
-            puts str % [e,  mask]
-        }
+            puts format(str, e, mask)
+        end
 
         puts
 
-        CLIHelper.print_header(str_h1 % "ROLES", false)
+        CLIHelper.print_header(str_h1 % 'ROLES', false)
 
         if !vmgroup.to_hash['VM_GROUP']['ROLES']['ROLE'].nil?
             roles = [vmgroup.to_hash['VM_GROUP']['ROLES']['ROLE']].flatten
         end
 
         CLIHelper::ShowTable.new(nil, self) do
-            column :ID, "", :left, :size=>4 do |d|
-                d["ID"]
+            column :ID, '', :left, :size=>4 do |d|
+                d['ID']
             end
 
-            column :NAME, "", :left, :size=>8 do |d|
-                d["NAME"]
+            column :NAME, '', :left, :size=>8 do |d|
+                d['NAME']
             end
 
-            column :POLICY, "", :left, :size=>12 do |d|
-                if d["POLICY"].nil?
+            column :POLICY, '', :left, :size=>12 do |d|
+                if d['POLICY'].nil?
                     '-'
                 else
-                    d["POLICY"]
+                    d['POLICY']
                 end
             end
 
-            column :AFFINED_HOSTS, "", :left, :size=>18 do |d|
-                if d["HOST_AFFINED"].nil?
+            column :AFFINED_HOSTS, '', :left, :size=>18 do |d|
+                if d['HOST_AFFINED'].nil?
                     '-'
                 else
-                    d["HOST_AFFINED"]
+                    d['HOST_AFFINED']
                 end
             end
 
-            column :ANTI_AFFINED_HOST, "", :left, :size=>18 do |d|
-                if d["HOST_ANTI_AFFINED"].nil?
+            column :ANTI_AFFINED_HOST, '', :left, :size=>18 do |d|
+                if d['HOST_ANTI_AFFINED'].nil?
                     '-'
                 else
-                    d["HOST_ANTI_AFFINED"]
+                    d['HOST_ANTI_AFFINED']
                 end
             end
 
-            column :VIRTUAL_MACHINES, "", :left, :size=>20 do |d|
-                d["VMS"]
+            column :VIRTUAL_MACHINES, '', :left, :size=>20 do |d|
+                d['VMS']
             end
         end.show(roles, {})
 
         puts
 
-        CLIHelper.print_header(str_h1 % "TEMPLATE CONTENTS",false)
+        CLIHelper.print_header(str_h1 % 'TEMPLATE CONTENTS', false)
         puts vmgroup.template_str
     end
+
 end

data/lib/one_helper/onevnet_helper.rb

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------- #
-# Copyright 2002-2025, OpenNebula Project, OpenNebula Systems                #
+# Copyright 2002-2026, OpenNebula Project, OpenNebula Systems                #
 #                                                                            #
 # Licensed under the Apache License, Version 2.0 (the "License"); you may    #
 # not use this file except in compliance with the License. You may obtain    #

data/lib/one_helper/onevntemplate_helper.rb

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------- #
-# Copyright 2002-2025, OpenNebula Project, OpenNebula Systems                #
+# Copyright 2002-2026, OpenNebula Project, OpenNebula Systems                #
 #                                                                            #
 # Licensed under the Apache License, Version 2.0 (the "License"); you may    #
 # not use this file except in compliance with the License. You may obtain    #
@@ -19,39 +19,41 @@ require 'one_helper/onetemplate_helper'
 require 'opennebula/vntemplate'
 require 'opennebula/vntemplate_pool'
 
+# Helper class for VNTemplate commands
 class OneVNTemplateHelper < OneTemplateHelper
+
     VN_NAME={
-        :name  => "name",
-        :large => "--name name",
+        :name  => 'name',
+        :large => '--name name',
         :format => String,
-        :description =>  <<-EOT.strip
-Name of the new VN TEMPLATE. When instantiating
-                               multiple VNs you can use the \"%i\" wildcard to produce
-                               different names such as vm-0, vm-1...
-EOT
+        :description =>  <<~EOT.strip
+            Name of the new VN TEMPLATE. When instantiating
+                                           multiple VNs you can use the \"%i\" wildcard to produce
+                                           different names such as vm-0, vm-1...
+        EOT
     }
 
     MULTIPLE={
-        :name  => "multiple",
-        :short => "-m x",
-        :large => "--multiple x",
+        :name  => 'multiple',
+        :short => '-m x',
+        :large => '--multiple x',
         :format => Integer,
-        :description => "Instance multiple VNs"
+        :description => 'Instance multiple VNs'
     }
 
     EXTENDED={
-        :name => "extended",
-        :large => "--extended",
-        :description => "Process the template and included extended "+
-                        "information"
+        :name => 'extended',
+        :large => '--extended',
+        :description => 'Process the template and included extended '+
+                        'information'
     }
 
     def self.rname
-        "VNTEMPLATE"
+        'VNTEMPLATE'
     end
 
     def self.conf_file
-        "onevntemplate.yaml"
+        'onevntemplate.yaml'
     end
 
     INT_EXP = /^-?\d+$/
@@ -59,7 +61,7 @@ EOT
 
     private
 
-    def factory(id=nil)
+    def factory(id = nil)
         if id
             OpenNebula::VNTemplate.new_with_id(id, @client)
         else
@@ -68,38 +70,39 @@ EOT
         end
     end
 
-    def factory_pool(user_flag=-2)
+    def factory_pool(user_flag = -2)
         OpenNebula::VNTemplatePool.new(@client, user_flag)
     end
 
-    def format_resource(template, options = {})
-        str="%-15s: %-20s"
-        str_h1="%-80s"
+    def format_resource(template, _options = {})
+        str='%-15s: %-20s'
+        str_h1='%-80s'
 
         CLIHelper.print_header(
-            str_h1 % "TEMPLATE #{template['ID']} INFORMATION")
-        puts str % ["ID", template.id.to_s]
-        puts str % ["NAME", template.name]
-        puts str % ["USER", template['UNAME']]
-        puts str % ["GROUP", template['GNAME']]
-        puts str % ["LOCK", OpenNebulaHelper.level_lock_to_str(template['LOCK/LOCKED'])]
-        puts str % ["REGISTER TIME",
-            OpenNebulaHelper.time_to_str(template['REGTIME'])]
+            str_h1 % "TEMPLATE #{template['ID']} INFORMATION"
+        )
+        puts format(str, 'ID', template.id.to_s)
+        puts format(str, 'NAME', template.name)
+        puts format(str, 'USER', template['UNAME'])
+        puts format(str, 'GROUP', template['GNAME'])
+        puts format(str, 'LOCK', OpenNebulaHelper.level_lock_to_str(template['LOCK/LOCKED']))
+        puts format(str, 'REGISTER TIME', OpenNebulaHelper.time_to_str(template['REGTIME']))
         puts
 
-        CLIHelper.print_header(str_h1 % "PERMISSIONS",false)
+        CLIHelper.print_header(str_h1 % 'PERMISSIONS', false)
 
-        ["OWNER", "GROUP", "OTHER"].each { |e|
-            mask = "---"
-            mask[0] = "u" if template["PERMISSIONS/#{e}_U"] == "1"
-            mask[1] = "m" if template["PERMISSIONS/#{e}_M"] == "1"
-            mask[2] = "a" if template["PERMISSIONS/#{e}_A"] == "1"
+        ['OWNER', 'GROUP', 'OTHER'].each do |e|
+            mask = '---'
+            mask[0] = 'u' if template["PERMISSIONS/#{e}_U"] == '1'
+            mask[1] = 'm' if template["PERMISSIONS/#{e}_M"] == '1'
+            mask[2] = 'a' if template["PERMISSIONS/#{e}_A"] == '1'
 
-            puts str % [e,  mask]
-        }
+            puts format(str, e, mask)
+        end
         puts
 
-        CLIHelper.print_header(str_h1 % "TEMPLATE CONTENTS",false)
+        CLIHelper.print_header(str_h1 % 'TEMPLATE CONTENTS', false)
         puts template.template_str
     end
+
 end