foreman_fog_proxmox 0.15.1 → 0.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7fb26099667e555c1234f7a330cf9914f6efc3c874b22cd2dbff180f1e544ebc
-  data.tar.gz: 4c7dc234f8e677f91c898cd9bb1fa52bd5d074998481aeeee7fde0c64a6ef4d5
+  metadata.gz: 66a00f7786605946bb2388d497b7a2f512fb29fcf00a12f35bc7059e8b0825f4
+  data.tar.gz: 3e2bf00ae5430cad3c5927ba4636582cd78d83cefa7065088fa4458da638b173
 SHA512:
-  metadata.gz: 4c1585871bc12fd7b7d6a806dc42c1ae975e4c79765d0a753a9823a2a7c0919e3e7a5b4997217b441f98b208d025166a0ce7f25411c9b058bc874dfd2786a4cb
-  data.tar.gz: 0b6d569fb3dfb400f65caf1fedf74e03d7044a3158c98ad9daee258860b9080362431e250ddd49fa79d2e75f7a852ab0dd972029a45e54bcfc20b59eb8dacd19
+  metadata.gz: 9534979808ea0e6ad262d696497189b3129e6594478601649d74d475ef1cee060898988e662d50db447f04257b6e6cdcfa6d2d144fea0ef498e66c3a899bd140
+  data.tar.gz: 981cfaa3815ce58182a465490541051b4a79612f0153e3419142bbf5e3c7e58c68b7758b2cac8eaa4bae31a428717d239315695b6847f2d1d207b04f26ff8a41

data/app/helpers/proxmox_compute_resources_helper.rb

@@ -34,7 +34,7 @@ module ProxmoxComputeResourcesHelper
   def cluster_nodes(compute_resource)
     nodes = compute_resource.nodes ? compute_resource.nodes.collect(&:node) : []
   rescue ::Foreman::Exception => e
-    return [] if e.message == 'User token expired'
+    [] if e.message == 'User token expired'
   rescue StandardError => e
     logger.warn("failed to get cluster nodes: #{e}")
     raise e

data/app/helpers/proxmox_form_helper.rb

@@ -30,7 +30,7 @@ module ProxmoxFormHelper
         addClass options, 'form-control'
         pass = f.password_field(attr, options) +
                tag(:span, '', class: 'glyphicon glyphicon-warning-sign input-addon', title: 'Caps lock ON',
-style: 'display:none')
+                 style: 'display:none')
         if unset_button
           button = link_to_function(icon_text('edit', '', :kind => 'pficon'), 'toggle_input_group(this)',
             :id => 'disable-pass-btn', :class => 'btn btn-default', :title => _('Change the password'))
@@ -56,7 +56,7 @@ style: 'display:none')
     options[:form_builder_attrs] ||= {}
 
     content_tag(:div, :class => "#{options[:type]}_#{association}_fields_template form_template",
-:style => 'display: none;') do
+      :style => 'display: none;') do
       form_builder.fields_for(association, options[:object],
         :child_index => "new_#{options[:type]}_#{association}") do |f|
         render(:partial => options[:partial], :layout => options[:layout],

data/app/helpers/proxmox_vm_attrs_helper.rb

@@ -0,0 +1,131 @@
+# frozen_string_literal: true
+
+# Copyright 2018 Tristan Robert
+
+# This file is part of ForemanFogProxmox.
+
+# ForemanFogProxmox is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+
+# ForemanFogProxmox is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
+
+require 'fog/proxmox/helpers/disk_helper'
+require 'fog/proxmox/helpers/nic_helper'
+require 'fog/proxmox/helpers/cpu_helper'
+require 'foreman_fog_proxmox/value'
+require 'foreman_fog_proxmox/hash_collection'
+
+# Convert a foreman form server hash into a fog-proxmox server attributes hash
+module ProxmoxVmAttrsHelper
+  def object_to_attributes_hash(vms, from_profile, start_checked)
+    param_scope = from_profile ? "compute_attribute[vm_attrs]" : "host[compute_attributes]"
+    vm_h = ActiveSupport::HashWithIndifferentAccess.new
+    keys = [:vmid, :node_id, :type, :pool]
+    main = vms.attributes.select { |key, _value| keys.include? key }
+    vms.config.all_attributes.each do |key, value|
+      camel_key = key.to_s.include?('_') ? snake_to_camel(key.to_s).to_sym : key
+      vm_h[camel_key] = { :name => "#{param_scope}[config_attributes][#{key}]", :value => value } unless keys.include? key
+    end
+    main.each do |key, value|
+      camel_key = key.to_s.include?('_') ? snake_to_camel(key.to_s).to_sym : key
+      vm_h[camel_key] = { :name => "#{param_scope}[#{key}]", :value => value } if keys.include? key
+    end
+
+    vm_h.merge!(additional_attrs(vms, param_scope, start_checked))
+    vm_h[:interfaces] = network_attrs(param_scope, vms.interfaces)
+    vm_h[:disks] = volumes_attrs(param_scope, vms.volumes)
+    vm_h.merge(cpu_flags_attrs(param_scope, vms.config))
+  end
+
+  def cpu_flags_attrs(param_scope, config)
+    flag_attrs = ActiveSupport::HashWithIndifferentAccess.new
+    Fog::Proxmox::CpuHelper.flags.each do |key, _val|
+      flag_attrs.merge!({ key => { :name => "#{param_scope}[config_attributes][#{key}]", :value => config.public_send(key) } })
+    end
+    flag_attrs
+  end
+
+  def volumes_attrs(param_scope, volumes)
+    vol_attrs = []
+    volumes.each_with_index do |vol, id|
+      keys = []
+      type = ""
+      if vol.rootfs?
+        keys = ['id', 'volid', 'storage', 'size', 'storage_type']
+        type = 'rootfs'
+      elsif vol.hard_disk?
+        keys = ['id', 'volid', 'storage_type', 'storage', 'controller', 'device', 'cache', 'size']
+        type = 'hard_disk'
+      elsif  vol.cdrom?
+        keys = ['id', 'storage_type', 'cdrom', 'storage', 'volid']
+        type = 'cdrom'
+      elsif vol.cloud_init?
+        keys = ['id', 'volid', 'storage_type', 'storage', 'controller', 'device']
+        type = 'cloud_init'
+      elsif vol.mount_point?
+        keys = ['id', 'volid', 'storage_type', 'storage', 'device', 'mp', 'size']
+        type = 'mount_point'
+      end
+      vol_attrs << { :name => type, :value => vol_keys(param_scope, keys, vol, id) }
+    end
+    vol_attrs
+  end
+
+  def vol_keys(param_scope, keys, vol, id)
+    attrs = ActiveSupport::HashWithIndifferentAccess.new
+    keys.each do |key|
+      camel_key = key.to_s.include?('_') ? snake_to_camel(key.to_s).to_sym : key
+      attrs[camel_key] = { :name => "#{param_scope}[volumes_attributes][#{id}][#{key}]", :value => vol.public_send(key) }
+    end
+    attrs
+  end
+
+  def network_attrs(param_scope, interfaces)
+    networks_attrs = []
+    interfaces.each_with_index do |interface, id|
+      attrs = ActiveSupport::HashWithIndifferentAccess.new
+      interface.all_attributes.each do |key, value|
+        camel_key = key.to_s.include?('_') ? snake_to_camel(key.to_s).to_sym : key
+        attrs[camel_key] = { :name => "#{param_scope}[interfaces_attributes][#{id}][#{key}]", :value => value }
+      end
+      networks_attrs << { :name => 'interface', :value => attrs }
+    end
+    networks_attrs
+  end
+
+  def additional_attrs(vms, param_scope, start_checked)
+    attributes = {
+      pool: vms.pool,
+      image_id: vms.image_id,
+      cpu_type: vms.config.cpu_type,
+      nameserver: vms.config.nameserver,
+      searchdomain: vms.config.searchdomain,
+      hostname: vms.config.hostname,
+      ostemplate_storage: vms.ostemplate_storage,
+      ostemplate_file: vms.ostemplate_file,
+      start_after_create: vms.start_after_create,
+      templated: vms.templated,
+    }
+    vms_keys = [:pool, :start_after_create]
+    extra_attrs = ActiveSupport::HashWithIndifferentAccess.new
+    attributes.each do |key, value|
+      camel_key = key.to_s.include?('_') ? snake_to_camel(key.to_s).to_sym : key
+      nested_key = vms_keys.include?(key) ? key : "config_attributes[#{key}]"
+      value = start_checked if key == :start_after_create
+      extra_attrs[camel_key] = { name: "#{param_scope}[#{nested_key}]", value: value }
+    end
+    extra_attrs
+  end
+
+  def snake_to_camel(str)
+    str.split('_').inject([]) { |buffer, e| buffer.push(buffer.empty? ? e : e.capitalize) }.join
+  end
+end

data/app/helpers/proxmox_vm_interfaces_helper.rb

@@ -56,13 +56,13 @@ module ProxmoxVmInterfacesHelper
   end
 
   def interface_common_typed_keys(type)
-    ['id', type == 'qemu' ? 'macaddr' : 'hwaddr']
+    ['id', (type == 'qemu') ? 'macaddr' : 'hwaddr']
   end
 
   def compute_dhcps(interface_attributes_h)
-    interface_attributes_h[:dhcp] = interface_attributes_h[:ip] == 'dhcp' ? '1' : '0'
+    interface_attributes_h[:dhcp] = (interface_attributes_h[:ip] == 'dhcp') ? '1' : '0'
     interface_attributes_h[:ip] = '' if interface_attributes_h[:dhcp] == '1'
-    interface_attributes_h[:dhcp6] = interface_attributes_h[:ip6] == 'dhcp' ? '1' : '0'
+    interface_attributes_h[:dhcp6] = (interface_attributes_h[:ip6] == 'dhcp') ? '1' : '0'
     interface_attributes_h[:ip6] = '' if interface_attributes_h[:dhcp6] == '1'
   end
 

data/app/helpers/proxmox_vm_volumes_helper.rb

@@ -52,9 +52,9 @@ module ProxmoxVmVolumesHelper
     logger.debug(format(_('parse_hard_disk_volume(): args=%<args>s'), args: args))
     disk = {}
     disk[:id] = args['id'] if args.key?('id')
-    disk[:volid] = args['volid'] if args.key?('volid')
-    disk[:storage] = args['storage'].to_s if args.key?('storage')
-    disk[:size] = args['size'].to_i if args.key?('size')
+    disk[:volid] = args['volid'] if args.key?('volid') && !args['volid'].empty?
+    disk[:storage] = args['storage'].to_s if args.key?('storage') && !args['storage'].empty?
+    disk[:size] = args['size'].to_i if args.key?('size') && !args['size'].empty?
     add_disk_options(disk, args) unless args.key?('options')
     disk[:options] = args['options'] if args.key?('options')
     disk.key?(:storage) ? disk : {}

data/app/models/concerns/fog_extensions/proxmox/node.rb

@@ -31,7 +31,7 @@ module FogExtensions
             # We need to explicitly use the base 'each' method here on the page,
             #  otherwise we get infinite recursion
             base_each = Fog::Collection.instance_method(:each)
-            base_each.bind(page).call(&block)
+            base_each.bind_call(page, &block)
           end
         end
         self

data/app/models/foreman_fog_proxmox/proxmox_compute_attributes.rb

@@ -30,7 +30,7 @@ module ForemanFogProxmox
         unless compute_os_types(host).include?(ostype)
           raise ::Foreman::Exception,
             format(_('Operating system family %<type>s is not consistent with %<ostype>s'), type: host.operatingsystem.type,
-ostype: ostype)
+              ostype: ostype)
         end
       end
       super
@@ -42,8 +42,8 @@ ostype: ostype)
 
     def interface_compute_attributes(interface_attributes)
       vm_attrs = ForemanFogProxmox::HashCollection.new_hash_reject_keys(interface_attributes, [:identifier, :macaddr, :hwaddr])
-      vm_attrs[:dhcp] = interface_attributes[:ip] == 'dhcp' ? '1' : '0'
-      vm_attrs[:dhcp6] = interface_attributes[:ip6] == 'dhcp' ? '1' : '0'
+      vm_attrs[:dhcp] = (interface_attributes[:ip] == 'dhcp') ? '1' : '0'
+      vm_attrs[:dhcp6] = (interface_attributes[:ip6] == 'dhcp') ? '1' : '0'
       vm_attrs
     end
 

data/app/models/foreman_fog_proxmox/proxmox_images.rb

@@ -48,9 +48,14 @@ module ForemanFogProxmox
       volumes = []
       nodes.each do |node|
         storages(node.node).each do |storage|
+          # fetches volumes of QEMU servers for images
           volumes += storage.volumes.list_by_content_type('images')
+
+          # fetches volumes of KVM containers for images
+          volumes += storage.volumes.list_by_content_type('rootdir')
         end
       end
+      # for creating image, only list volumes which are templated
       volumes.select(&:template?)
     end
 

data/app/models/foreman_fog_proxmox/proxmox_version.rb

@@ -36,7 +36,7 @@ module ForemanFogProxmox
       v = identity_client.read_version if identity_client
       v ? v['version'] : 'Unknown'
     rescue ::Foreman::Exception => e
-      return 'Unkown' if e.message == 'User token expired'
+      'Unkown' if e.message == 'User token expired'
     rescue StandardError => e
       logger.warn("failed to get identity client version: #{e}")
       raise e

data/app/models/foreman_fog_proxmox/proxmox_vm_commands.rb

@@ -55,7 +55,7 @@ module ForemanFogProxmox
     end
 
     def assign_vmid(vmid, node)
-      vmid < 1 ? node.servers.next_id : vmid
+      (vmid < 1) ? node.servers.next_id : vmid
     end
 
     def compute_clone_attributes(args, container, type)

data/app/models/foreman_fog_proxmox/proxmox_vm_new.rb

@@ -191,7 +191,7 @@ module ForemanFogProxmox
       new_attr_type ||= type
       logger.debug("new_typed_vm(#{type}): new_attr_type=#{new_attr_type}")
       logger.debug("new_typed_vm(#{type}): new_attr=#{new_attr}'")
-      options = !new_attr.key?('vmid') || ForemanFogProxmox::Value.empty?(new_attr['vmid']) ? vm_typed_instance_defaults(type).merge(new_attr).merge(type: type) : new_attr
+      options = (!new_attr.key?('vmid') || ForemanFogProxmox::Value.empty?(new_attr['vmid'])) ? vm_typed_instance_defaults(type).merge(new_attr).merge(type: type) : new_attr
       logger.debug("new_typed_vm(#{type}): options=#{options}")
       vm_h = parse_typed_vm(options, type).deep_symbolize_keys
       logger.debug("new_typed_vm(#{type}): vm_h=#{vm_h}")

data/app/overrides/compute_resources_vms/form/add_from_profile_to_compute_attributes_form.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright 2018 Tristan Robert
 
 # This file is part of ForemanFogProxmox.
@@ -15,6 +17,12 @@
 # You should have received a copy of the GNU General Public License
 # along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
 
+Deface::Override.new(
+  virtual_path: 'compute_attributes/_compute_form',
+  name: 'remove_networks_and_volumes_partial',
+  remove: "erb[loud]:contains('compute_resources_vms/form/networks'), erb[loud]:contains('compute_resources_vms/form/volumes')"
+)
+
 Deface::Override.new(
   :virtual_path => 'compute_attributes/_form',
   :name => 'add_from_profile_to_compute_attributes_form',

data/app/overrides/compute_resources_vms/form/add_react_component_to_host.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+# Copyright 2018 Tristan Robert
+
+# This file is part of ForemanFogProxmox.
+
+# ForemanFogProxmox is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+
+# ForemanFogProxmox is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
+
+Deface::Override.new(
+  virtual_path: 'hosts/_form',
+  name: 'add_react_component_to_virtual_machine_tab',
+  insert_after: "li.active",
+  partial: 'compute_resources_vms/form/proxmox/add_react_component_to_host_form'
+)

data/app/overrides/compute_resources_vms/form/update_react_component_to_host_form.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+# Copyright 2018 Tristan Robert
+
+# This file is part of ForemanFogProxmox.
+
+# ForemanFogProxmox is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+
+# ForemanFogProxmox is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
+
+Deface::Override.new(
+  :virtual_path => 'hosts/_compute',
+  :name => 'update_react_component_to_virtual_machine_tab',
+  :replace => "erb[loud]:contains('hosts/compute_detail')",
+  :partial => 'compute_resources_vms/form/proxmox/update_react_component_to_host_form'
+)

data/app/views/compute_resources_vms/form/proxmox/_add_react_component_to_host_form.html.erb

@@ -0,0 +1,5 @@
+<% content_for(:javascripts) do %>
+  <%= webpacked_plugins_js_for :foreman_fog_proxmox %>
+  <%= javascript_include_tag 'foreman_fog_proxmox/proxmox_vm', "data-turbolinks-track" => true %>
+<% end %>
+<%= react_component('ProxmoxVmType', { registerComp: true }) unless @host.managed %>

data/app/views/compute_resources_vms/form/proxmox/_base.html.erb

@@ -14,25 +14,25 @@ GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
 along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>. %>
-
-<%= javascript_include_tag 'foreman_fog_proxmox/proxmox_vm', "data-turbolinks-track" => true %>
-
-<%= select_f f, :type, proxmox_types_map, :id, :name, { }, :label => _('Type'), :label_size => "col-md-2", :required => true, :onchange => "vmTypeSelected()", :disabled => !new_vm %>
-
-<!-- VM General Settings -->
-<%= render :partial => "compute_resources_vms/form/proxmox/general", :locals => { :f => f, :compute_resource =>  compute_resource, :new_vm => new_vm, :from_profile => from_profile } %>
-
-<!-- VM Extended Settings -->
-<%= render :partial => "compute_resources_vms/form/proxmox/container/extended", :locals => { :f => f, :compute_resource =>  compute_resource, :new_vm => new_vm } %>
-
-<!-- VM Advanced Settings -->
-<%= render :partial => "compute_resources_vms/form/proxmox/server/advanced", :locals => { :f => f, :compute_resource =>  compute_resource, :new_vm => new_vm } %>
-<%= render :partial => "compute_resources_vms/form/proxmox/container/advanced", :locals => { :f => f, :compute_resource =>  compute_resource, :new_vm => new_vm } %>
-
-<!-- Config Options Settings-->
-<%= f.fields_for :config, compute_resource.new_typed_vm(object_to_config_hash(f.object, 'qemu'), 'qemu').config, include_id: false do |server_config|%>
-  <%= render :partial => "compute_resources_vms/form/proxmox/server/config", :locals => { :f => server_config, :cloudinit => f.object.cloud_init?, :compute_resource =>  compute_resource, :host => @host, :new_vm => new_vm, :item_layout => 'removable', :type => f.object.type, :node_id => f.object.node_id } %>
+<% nodes = compute_resource.nodes %>
+<% unless local_assigns[:hide_image] && !new_vm %>
+    <%
+      arch ||= nil ; os ||= nil
+      images = possible_images(compute_resource, arch, os)
+    -%>
+    <div id='image_selection'>
+    <%= select_f f, :image_id, images, :uuid, :name, { :include_blank => true },
+      :disabled => true,
+      :help_inline => :indicator,
+      :class => ('hide' if from_profile),
+      :label => _('Image'),
+      :no_label => from_profile,
+      :label_size => "col-md-2" %>
+    </div>
+<% end %>
+<% untemplatable = new_vm || f.object.templated? %>
+<% checked = params[:host] && params[:host][:compute_attributes] && params[:host][:compute_attributes][:start_after_create] || '1' %>
+<% content_for(:javascripts) do %>
+  <%= webpacked_plugins_js_for :foreman_fog_proxmox %>
 <% end %>
-<%= f.fields_for :config, compute_resource.new_typed_vm(object_to_config_hash(f.object, 'lxc'), 'lxc').config, include_id: false do |container_config|%>
-  <%= render :partial => "compute_resources_vms/form/proxmox/container/config", :locals => { :f => container_config, :compute_resource =>  compute_resource, :host => @host, :new_vm => new_vm, :item_layout => 'removable', :type => f.object.type, :node_id => f.object.node_id } %>
-<% end %>
+<%= react_component('ProxmoxVmType',  { vmAttrs: object_to_attributes_hash(f.object, from_profile, checked), nodes: nodes.as_json, images: images.as_json, pools: compute_resource.pools.as_json, storages: compute_resource.storages.as_json, fromProfile: from_profile, newVm: new_vm, bridges: compute_resource.bridges.as_json, untemplatable: untemplatable }) %>

data/app/views/compute_resources_vms/form/proxmox/_update_react_component_to_host_form.html.erb

@@ -0,0 +1,26 @@
+<%# Copyright 2018 Tristan Robert
+
+This file is part of ForemanFogProxmox.
+
+ForemanFogProxmox is free software: you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation, either version 3 of the License, or
+(at your option) any later version.
+
+ForemanFogProxmox is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+GNU General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>. %>
+
+<% if compute_resource.class == ForemanFogProxmox::Proxmox %>
+<%= render :partial => provider_partial(compute_resource, 'base'),
+            locals: { f: compute, host: host, compute_resource: compute_resource, new_host: host.new_record?, new_vm: !compute.object.persisted?,
+                     arch: host.architecture_id, os: host.operatingsystem_id, from_profile: false } %>
+<% else %>
+<%= render :partial => provider_partial(compute_resource, 'base'),
+            locals: { f: compute, host: host, compute_resource: compute_resource, new_host: host.new_record?, new_vm: !compute.object.persisted?,
+                     arch: host.architecture_id, os: host.operatingsystem_id } %>
+<% end %>

data/config/routes.rb

@@ -20,15 +20,15 @@
 Rails.application.routes.draw do
   namespace :foreman_fog_proxmox do
     match 'isos/:compute_resource_id/:node_id/:storage', :to => 'compute_resources#isos_by_id_and_node_and_storage',
-:via => 'get'
+      :via => 'get'
     match 'ostemplates/:compute_resource_id/:node_id/:storage',
       :to => 'compute_resources#ostemplates_by_id_and_node_and_storage', :via => 'get'
     match 'isos/:compute_resource_id/:node_id', :to => 'compute_resources#isos_by_id_and_node', :via => 'get'
     match 'ostemplates/:compute_resource_id/:node_id', :to => 'compute_resources#ostemplates_by_id_and_node',
-:via => 'get'
+      :via => 'get'
     match 'storages/:compute_resource_id/:node_id', :to => 'compute_resources#storages_by_id_and_node', :via => 'get'
     match 'isostorages/:compute_resource_id/:node_id', :to => 'compute_resources#iso_storages_by_id_and_node',
-:via => 'get'
+      :via => 'get'
     match 'bridges/:compute_resource_id/:node_id', :to => 'compute_resources#bridges_by_id_and_node', :via => 'get'
   end
 end

data/lib/foreman_fog_proxmox/version.rb

@@ -18,5 +18,5 @@
 # along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
 
 module ForemanFogProxmox
-  VERSION = '0.15.1'
+  VERSION = '0.16.0'
 end

data/package.json

@@ -0,0 +1,42 @@
+{
+  "name": "foreman_fog_proxmox",
+  "version": "1.0.0",
+  "description": "DESCRIPTION",
+  "main": "index.js",
+  "scripts": {
+    "lint": "tfm-lint --plugin -d webpack",
+    "test": "tfm-test --config jest.config.js",
+    "test:watch": "tfm-test --plugin --watchAll",
+    "test:current": "tfm-test --plugin --watch",
+    "publish-coverage": "tfm-publish-coverage",
+    "stories": "tfm-stories --plugin",
+    "stories:build": "tfm-build-stories --plugin",
+    "create-react-component": "yo react-domain"
+  },
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/theforeman/foreman_fog_proxmox.git"
+  },
+  "bugs": {
+    "url": "http://projects.theforeman.org/projects/foreman_fog_proxmox/issues"
+  },
+  "peerDependencies": {
+    "@theforeman/vendor": "^12.0.1"
+  },
+  "dependencies": {
+    "react-intl": "^2.8.0"
+  },
+  "devDependencies": {
+    "@babel/core": "^7.7.0",
+    "@testing-library/react": "^10.4.9",
+    "@theforeman/builder": ">=12.0.1",
+    "@theforeman/eslint-plugin-foreman": ">=12.0.1",
+    "@theforeman/find-foreman": ">=12.0.1",
+    "@theforeman/test": ">=12.0.1",
+    "@theforeman/vendor-dev": ">=12.0.1",
+    "babel-eslint": "^10.0.3",
+    "eslint": "^6.7.2",
+    "prettier": "^1.19.1",
+    "react-redux-test-utils": "^0.2.0"
+  }
+}

data/test/factories/proxmox_factory.rb

@@ -20,7 +20,7 @@
 require 'fog/proxmox/compute/models/node'
 
 FactoryBot.define do
-  factory :proxmox_resource, :class => ComputeResource do
+  factory :proxmox_resource, :class => 'ComputeResource' do
     sequence(:name) { |n| "compute_resource#{n}" }
     organizations { [Organization.find_by(name: 'Organization 1')] }
     locations { [Location.find_by(name: 'Location 1')] }
@@ -32,10 +32,10 @@ FactoryBot.define do
       url { 'https://192.168.56.101:8006/api2/json' }
     end
 
-    factory :proxmox_cr, :class => ForemanFogProxmox::Proxmox, :traits => [:proxmox]
+    factory :proxmox_cr, :class => 'ForemanFogProxmox::Proxmox', :traits => [:proxmox]
   end
 
-  factory :node, :class => Fog::Proxmox::Compute::Node do
+  factory :node, :class => 'Fog::Proxmox::Compute::Node' do
     sequence(:identity) { |n| "node#{n}" }
     trait :proxmox do
       identity { 'proxmox' }
@@ -43,7 +43,7 @@ FactoryBot.define do
     trait :service do
       service { :proxmox_cr }
     end
-    factory :proxmox_node, :class => Fog::Proxmox::Compute::Node, :traits => [:proxmox, :service]
+    factory :proxmox_node, :class => 'Fog::Proxmox::Compute::Node', :traits => [:proxmox, :service]
   end
 
   def deferred_nic_attrs
@@ -59,16 +59,16 @@ FactoryBot.define do
     host
   end
 
-  factory :nic_base_empty, :class => Nic::Base do
+  factory :nic_base_empty, :class => 'Nic::Base' do
     type { 'Nic::Base' }
   end
 
-  factory :nic_managed_empty, :class => Nic::Managed, :parent => :nic_base_empty do
+  factory :nic_managed_empty, :class => 'Nic::Managed', :parent => :nic_base_empty do
     type { 'Nic::Managed' }
     identifier { 'net0' }
   end
 
-  factory :host_empty, :class => Host do
+  factory :host_empty, :class => 'Host' do
     sequence(:name) { |n| "host#{n}" }
     sequence(:hostname) { |n| "host#{n}" }
     trait :compute_attributes do

data/test/unit/foreman_fog_proxmox/proxmox_compute_attributes_test.rb

@@ -62,7 +62,7 @@ module ForemanFogProxmox
 
       it 'sets container hostname with host name' do
         physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'net0', :primary => true,
-:compute_attributes => { 'dhcp' => '1', 'dhcp6' => '1' })
+          :compute_attributes => { 'dhcp' => '1', 'dhcp6' => '1' })
         host = FactoryBot.build(
           :host_empty,
           :interfaces => [physical_nic],

data/test/unit/foreman_fog_proxmox/proxmox_interfaces_test.rb

@@ -40,7 +40,7 @@ module ForemanFogProxmox
       it 'raises Foreman::Exception when server proxmox NIC id does not match net[k] with k integer' do
         compute_attributes = ActiveSupport::HashWithIndifferentAccess.new({ 'id' => 'dsfqsfqzef' })
         physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'eth0',
-:compute_attributes => compute_attributes)
+          :compute_attributes => compute_attributes)
         host = FactoryBot.build(
           :host_empty,
           :interfaces => [physical_nic],
@@ -58,7 +58,7 @@ module ForemanFogProxmox
         ip6 = Array.new(4) { format('%<x>s', x: rand(16**4)) }.join(':') + '::1'
         compute_attributes = ActiveSupport::HashWithIndifferentAccess.new({ 'id' => 'net0' })
         physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'net0', :ip => ip, :ip6 => ip6,
-:mac => mac_address, :compute_attributes => compute_attributes)
+          :mac => mac_address, :compute_attributes => compute_attributes)
         host = FactoryBot.build(
           :host_empty,
           :interfaces => [physical_nic],
@@ -75,7 +75,7 @@ module ForemanFogProxmox
       it 'raises Foreman::Exception when container proxmox NIC id does not match net[k] with k integer' do
         compute_attributes = ActiveSupport::HashWithIndifferentAccess.new({ 'id' => 'dsfqsfqzef' })
         physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'eth0',
-:compute_attributes => compute_attributes)
+          :compute_attributes => compute_attributes)
         host = FactoryBot.build(
           :host_empty,
           :interfaces => [physical_nic],
@@ -94,7 +94,7 @@ module ForemanFogProxmox
         compute_attributes = ActiveSupport::HashWithIndifferentAccess.new({ 'id' => 'net0', 'cidr' => cidr, 'gw' => ip,
 'ip' => ip, 'dhcp6' => '1' })
         physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'net0', :ip => ip, :ip6 => ip6,
-:compute_attributes => compute_attributes)
+          :compute_attributes => compute_attributes)
         host = FactoryBot.build(
           :host_empty,
           :interfaces => [physical_nic],
@@ -115,7 +115,7 @@ module ForemanFogProxmox
         compute_attributes = ActiveSupport::HashWithIndifferentAccess.new({ 'id' => 'net0', 'cidr6' => cidr6,
 'dhcp' => '1', 'gw6' => ip6 })
         physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'net0', :ip => ip, :ip6 => ip6,
-:compute_attributes => compute_attributes)
+          :compute_attributes => compute_attributes)
         host = FactoryBot.build(
           :host_empty,
           :interfaces => [physical_nic],
@@ -137,7 +137,7 @@ module ForemanFogProxmox
         compute_attributes = ActiveSupport::HashWithIndifferentAccess.new({ 'id' => 'net0', 'dhcp' => '1',
 'ip6' => ip6, 'firewall' => firewall })
         physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'net0', :ip => ip, :ip6 => ip6,
-:mac => mac_address, :compute_attributes => compute_attributes)
+          :mac => mac_address, :compute_attributes => compute_attributes)
         host = FactoryBot.build(
           :host_empty,
           :interfaces => [physical_nic],