hammer_cli_foreman_virt_who_configure 0.0.9 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36e2261f8c32cd0906a348f9920af4f7cbd8acd74ad7c78447c48e020a42a925
-  data.tar.gz: cd292789c8ceea3321fb0a091ba190dbe489f2fcc4b7937ba35a58698748af0b
+  metadata.gz: fce0d55b84baaf7c230353820665c88c6053b7f62253231a3e0d17d57663d0bc
+  data.tar.gz: 22219b792cfe77a3500a149641a579fd28100ca3b084b75e99137b2415cf26f8
 SHA512:
-  metadata.gz: d71a9bf91cde3e5840bf83b4ed13cf00f20ccc5b670fc601b9cce0728806f66bab53ee0b8a5c9a8a5c7b0811648dd3dbc3558f238b2f1055754a37a291814ab2
-  data.tar.gz: 93dfa03a8a0fa04c0fca4a90c4f7352d8d949a2862ae7f18cd9937815aec74cca48c99e1a724e3eed7e5c92c4f02d47e6f61d7804756a609bf1cc03ff52a0799
+  metadata.gz: 71786def3416e05021baa313753105a35e6eee7faa2d6fe259a228539a42e8240bce786a7ca5271b9ec2245a450b404f598d3e802879ebf4cb347ea7c47f2536
+  data.tar.gz: 8cf7a835c06daedd8c44b7907167b22aa245c727e508f9847c60540a9d81d3067ace69d29f983934bfacf49935b563f8ca9cafdc0b2883c847f074f6bcc6dab4

data/lib/hammer_cli_foreman_virt_who_configure/config.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'hammer_cli_foreman_virt_who_configure/system_caller'
 
 module HammerCLIForemanVirtWhoConfigure
@@ -19,34 +21,34 @@ module HammerCLIForemanVirtWhoConfigure
 
     def self.format_status(status)
       case status
-        when 'unknown'
-          _('No Report Yet')
-        when 'ok', 'out_of_date'
-          _('OK')
-        when 'error'
-          _('Error')
-        else
-          _('Unknown configuration status')
+      when 'unknown'
+        _('No Report Yet')
+      when 'ok', 'out_of_date'
+        _('OK')
+      when 'error'
+        _('Error')
+      else
+        _('Unknown configuration status')
       end
     end
 
     def self.format_filtering_mode(mode)
       case mode
-        when MODE_UNLIMITED
-          _('Unlimited')
-        when MODE_WHITELIST
-          _('Whitelist')
-        when MODE_BLACKLIST
-          _('Blacklist')
-        else
-          _('Unknown listing mode')
+      when MODE_UNLIMITED
+        _('Unlimited')
+      when MODE_WHITELIST
+        _('Whitelist')
+      when MODE_BLACKLIST
+        _('Blacklist')
+      else
+        _('Unknown listing mode')
       end
     end
 
     def self.validate_hypervisor_options(conf)
-      options = conf["hypervisor_type"] == 'kubevirt' ? %w(hypervisor_server hypervisor_username) : %w(kubeconfig_path)
-      options.append("prism_flavor", "ahv_update_interval", "ahv_internal_debug") unless conf["hypervisor_type"] == 'ahv'
-      conf.delete_if { |k, v| options.include?(k) }
+      options = conf['hypervisor_type'] == 'kubevirt' ? %w(hypervisor_server hypervisor_username) : %w(kubeconfig_path)
+      options.append('prism_flavor', 'ahv_internal_debug') unless conf['hypervisor_type'] == 'ahv'
+      conf.delete_if { |k, _v| options.include?(k) }
     end
 
     class ListCommand < HammerCLIForeman::ListCommand
@@ -77,7 +79,6 @@ module HammerCLIForemanVirtWhoConfigure
           field :hypervisor_username, _('Hypervisor username')
           field :kubeconfig_path, _('Configuration file'), Fields::Field, :hide_blank => true
           field :prism_flavor, _('AHV Prism flavor'), Fields::Field, :hide_blank => true
-          field :ahv_update_interval, _('AHV update frequency'), Fields::Field, :hide_blank => true
           field :ahv_internal_debug, _('Enable AHV debug'), Fields::Boolean
           field :_status, _('Status')
         end
@@ -114,12 +115,12 @@ module HammerCLIForemanVirtWhoConfigure
         if conf['filtering_mode'] != MODE_WHITELIST
           conf['whitelist'] = nil
         else
-          conf['whitelist'] ||= " "
+          conf['whitelist'] ||= ' '
         end
         if conf['filtering_mode'] != MODE_BLACKLIST
           conf['blacklist'] = nil
         else
-          conf['blacklist'] ||= " "
+          conf['blacklist'] ||= ' '
         end
         VirtWhoConfig.validate_hypervisor_options(conf)
       end
@@ -128,7 +129,7 @@ module HammerCLIForemanVirtWhoConfigure
     end
 
     class FetchCommand < HammerCLIForeman::Command
-      command_name "fetch"
+      command_name 'fetch'
       action :deploy_script
 
       failure_message _('Could not fetch the Virt Who configuration')
@@ -139,15 +140,15 @@ module HammerCLIForemanVirtWhoConfigure
         data = send_request
         if option_output
           path = File.expand_path(option_output)
-          if File.exists?(path)
+          if File.exist?(path)
             # this could be a security issue, the file should be readable only by the user
             output.print_error(
               _('Could not save the script'),
-              _("File at %{path} already exists, please specify a different path.") % { :path => path }
+              _('File at %{path} already exists, please specify a different path.') % { :path => path }
             )
             return HammerCLI::EX_USAGE
           else
-            File.write(path, data, { :perm => 0700, :mode => File::RDWR|File::CREAT|File::EXCL })
+            File.write(path, data, perm: 0700, mode: File::RDWR | File::CREAT | File::EXCL)
             return HammerCLI::EX_OK
           end
         else
@@ -164,7 +165,7 @@ module HammerCLIForemanVirtWhoConfigure
     end
 
     class DeployCommand < HammerCLIForeman::Command
-      command_name "deploy"
+      command_name 'deploy'
       desc _('Download and execute script for the specified virt-who configuration')
       action :deploy_script
 
@@ -195,7 +196,7 @@ module HammerCLIForemanVirtWhoConfigure
         'none' => 0,
         'whitelist' => 1,
         'blacklist' => 2
-      }
+      }.freeze
 
       def self.included(base)
         base.option '--filtering-mode', 'MODE', _('Hypervisor filtering mode'),

data/lib/hammer_cli_foreman_virt_who_configure/i18n.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'hammer_cli/i18n'
 
 module HammerCLIForemanVirtWhoConfigure

data/lib/hammer_cli_foreman_virt_who_configure/system_caller.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 module HammerCLIForemanVirtWhoConfigure
   class SystemCaller
-    WHITELISTED_VARS = %w[HOME USER LANG].freeze
+    WHITELISTED_VARS = %w(HOME USER LANG).freeze
 
     def initialize(tempfile = nil)
       @tempfile = tempfile || Tempfile.new('virt_who')

data/lib/hammer_cli_foreman_virt_who_configure/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module HammerCLIForemanVirtWhoConfigure
   def self.version
-    @version ||= Gem::Version.new '0.0.9'
+    @version ||= Gem::Version.new '0.1.1'
   end
 end

data/lib/hammer_cli_foreman_virt_who_configure.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 require 'hammer_cli'
 require 'hammer_cli_foreman'
 require 'hammer_cli_foreman_virt_who_configure/i18n'
 
 module HammerCLIForemanVirtWhoConfigure
-  HammerCLI::MainCommand.lazy_subcommand('virt-who-config', _("Manage Virt Who configurations"),
-    'HammerCLIForemanVirtWhoConfigure::VirtWhoConfig', 'hammer_cli_foreman_virt_who_configure/config'
-  )
+  HammerCLI::MainCommand.lazy_subcommand('virt-who-config', _('Manage Virt Who configurations'),
+  'HammerCLIForemanVirtWhoConfigure::VirtWhoConfig', 'hammer_cli_foreman_virt_who_configure/config'
+                                        )
 end