foreman_virt_who_configure 0.1.8 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 002aa4b4c24178a44d9be0a0442a1d94036e4b13
-  data.tar.gz: 07e665f0bb749678d0c79396c3f763a9fc6d5fa6
+  metadata.gz: 8355464b93318e276c9ad72c0f65aa8a2f6cae67
+  data.tar.gz: d89ea732217fd7f6130fa053ee4e7dce8cf799a7
 SHA512:
-  metadata.gz: 76e05459270fb2b46ef83aaca37aa39aed83b3a9b87f81fa96175d1f01627483138b63cec5ffa5d51122ada3fb599fef689aa298424c13cc59b46f973bc67214
-  data.tar.gz: 49527d87daef235217c5bea6ba5e85f195d17448d87657e93d2747a71ad057aad4990fdb7e5b4d472e3b60bda139129d0d9f44278346bb8c263fb851fa85df87
+  metadata.gz: 2cdca574bc96c5758f60e8fe50ef6cda5697b0625b07a0d00927c442a5d90b7bf5db9d202b2c6394a1d025ccc9f524600d58f21618d9675d1160d74e49669d10
+  data.tar.gz: 1e5d4b2f4ecff060df3718c57a7102d03177fe536cd9c4b7f53763b10c5d0a6231dedc82804a686698e1442a261a2f7bf6898afd776b995ead8a4886aef8891f

data/app/models/foreman_virt_who_configure/config.rb

@@ -91,6 +91,7 @@ module ForemanVirtWhoConfigure
     validates :interval, :hypervisor_type, :hypervisor_server, :hypervisor_username,
               :satellite_url, :hypervisor_id, :organization_id, :name,
               :presence => true
+    validates :name, :uniqueness => { :scope => :organization_id }
     validates :hypervisor_password, :presence => true, :if => Proc.new { |c| c.hypervisor_type != 'libvirt' }
     validates :hypervisor_type, :inclusion => HYPERVISOR_TYPES.keys
     validates :hypervisor_id, :inclusion => HYPERVISOR_IDS

data/app/models/foreman_virt_who_configure/output_generator.rb

@@ -128,7 +128,7 @@ if [ $result_code -ne 0 ]; then
   echo "There were some errors during configuration:"
   #{error_handling}
 else
-  echo "Finished successully"
+  echo "Finished successfully"
 fi
 
 EOS

data/app/views/foreman_virt_who_configure/configs/show.html.erb

@@ -8,7 +8,7 @@
         <h1><%= _("Virt-who Configuration %s") % @config.name %></h1>
       </div>
       <div class="col-md-6">
-        <%= _("Use either hammer command or the script below to deploy this configuration. Run one of them on the target host which has access to Red Hat Satellite Tools repository and will run virt-who reporting, preferably Satellite host:") %>
+        <%= _("Use either hammer command or the script below to deploy this configuration. Both require root privileges.  Run one of them on the target host which has access to Red Hat Satellite Tools repository and will run virt-who reporting, preferably Satellite host:") %>
       </div>
       <div class="col-md-2">
         <div class="pull-right">

data/app/views/foreman_virt_who_configure/configs/steps/_connection_form.erb

@@ -10,7 +10,7 @@
         inline_help_popover(_("If you run a hybrid environment, with virtual machines running Red Hat Enterprise Linux and other operating systems, you may want to limit the scope of virt-who’s access to hosts. For example, if some hypervisors host only Microsoft Windows Server instances, there is no benefit in having those hypervisors reported by the virt-who agent."))) %>
   <%= textarea_f f, :whitelist, inline_help_popover(_('Only hosts which uuid (or hostname or hwuuid, based on <code>hypervisor_id</code>) is specified in comma-separated list in this option will be reported. Wildcards and regular expressions are supported, multiple records must be separated by comma.  Put the value into the double-quotes if it contains special characters like comma.  All new line characters will be removed in resulting configuration file, white spaces are removed from beginning and end.')).merge(:label => _('Filter hosts')) %>
   <%= textarea_f f, :blacklist, inline_help_popover(_('Hosts which uuid (or hostname or hwuuid, based on <code>hypervisor_id</code>) is specified in comma-separated list in this option will <b>NOT</b> be reported.  Wildcards and regular expressions are supported, multiple records must be separated by comma.  Put the value into the double-quotes if it contains special characters like comma.  All new line characters will be removed in resulting configuration file, white spaces are removed from beginning and end.')).merge(:label => _('Exclude hosts')) %>
-  <%= checkbox_f f, :debug, :label => _('Do you want to enable debugging output?') %>
+  <%= checkbox_f f, :debug, { :label => _('Enable debugging output') }.merge(inline_help_popover(_("Different debug value can't be set per hypervisor, therefore it will affect all other deployed configurations on the host on which this configuration will be deployed."))) %>
   <%= text_f f, :proxy, inline_help_popover(_('HTTP proxy that should be used for communication between the server on which virt-who is running and the hypervisors and virtualization managers. Leave this blank if no proxy is used.')).merge(:label => _('HTTP proxy')) %>
   <%= text_f f, :no_proxy, inline_help_popover(_('A comma-separated list of hostnames or domains or ip addresses to ignore proxy settings for. Optionally this may be set to <code>*</code> to bypass proxy settings for all hostnames domains or ip addresses.')).merge(:label => _('Ignore proxy')) %>
 </div>

data/app/views/foreman_virt_who_configure/configs/steps/_schedule_form.erb

@@ -1,3 +1,3 @@
 <div id="config_schedule">
-  <%= selectable_f f, :interval, ForemanVirtWhoConfigure::Config::AVAILABLE_INTERVALS.map { |k,v| [_(v), k] }, {}, inline_help_popover(_("How often to check connected hypervisors for changes? Also affects how often a mapping is reported. The recommended value for most environments is every two hours.")) %>
+  <%= selectable_f f, :interval, ForemanVirtWhoConfigure::Config::AVAILABLE_INTERVALS.map { |k,v| [_(v), k] }, {}, inline_help_popover(_("How often to check connected hypervisors for changes? Also affects how often a mapping is reported. The recommended value for most environments is every two hours. Different interval can't be set per hypervisor, therefore it will affect all other deployed configurations on the host on which this configuration will be deployed.")) %>
 </div>

data/lib/foreman_virt_who_configure/version.rb

@@ -1,3 +1,3 @@
 module ForemanVirtWhoConfigure
-  VERSION = '0.1.8'.freeze
+  VERSION = '0.1.9'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_virt_who_configure
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.1.9
 platform: ruby
 authors:
 - Foreman virt-who-configure team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-12 00:00:00.000000000 Z
+date: 2017-12-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: katello