foreman_fog_proxmox 0.15.0 → 0.15.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b41c2a927bcce7383e1fd55c588247e935e0c6e11ba967fe6628db72a80730df
-  data.tar.gz: ce7634ce750c8317bc28906e4394888d50b66931b9c77586f3af90c339ca07c2
+  metadata.gz: 7fb26099667e555c1234f7a330cf9914f6efc3c874b22cd2dbff180f1e544ebc
+  data.tar.gz: 4c7dc234f8e677f91c898cd9bb1fa52bd5d074998481aeeee7fde0c64a6ef4d5
 SHA512:
-  metadata.gz: 65a3b9ac47bb382cc0c2b30430151a9db92bf64dcc9c64ed6c9fffb1fbb5dc525316d3475a7c5d88c21d30e3ffb8ad8eaf6ef5d6676cff53cfbd43dd5a086378
-  data.tar.gz: e210dad89239964322d0dc1b5f3a326a11f449ada2ad61d23f87f2047d421454020aa4b14ffed1efc173f31fd7cb0c68d92a76a9df0757133c810993a49a5a85
+  metadata.gz: 4c1585871bc12fd7b7d6a806dc42c1ae975e4c79765d0a753a9823a2a7c0919e3e7a5b4997217b441f98b208d025166a0ce7f25411c9b058bc874dfd2786a4cb
+  data.tar.gz: 0b6d569fb3dfb400f65caf1fedf74e03d7044a3158c98ad9daee258860b9080362431e250ddd49fa79d2e75f7a852ab0dd972029a45e54bcfc20b59eb8dacd19

data/app/assets/javascripts/foreman_fog_proxmox/proxmox_vm.js

@@ -15,8 +15,6 @@
 // You should have received a copy of the GNU General Public License
 // along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
 
-//= require jquery-ui/widgets/accordion
-
 $(document).ready(vmTypeSelected);
 
 function vmTypeSelected() {
@@ -25,13 +23,21 @@ function vmTypeSelected() {
   var host_uuid = $("input[id='host_uuid']").val();
   var new_vm =  host_uuid == undefined;
   var fieldsets = [];
+  var fieldconfig = [];
   fieldsets.push({id: 'config_advanced_options', toggle: true, new_vm: new_vm, selected: selected});
   fieldsets.push({id: 'config_ext', toggle: true, new_vm: new_vm, selected: selected});
   fieldsets.push({id: 'volume', toggle: true, new_vm: new_vm, selected: selected});
   fieldsets.push({id: 'network', toggle: true, new_vm: true, selected: selected});
+  fieldconfig.push({id: 'config_options', new_vm: new_vm, selected: selected});
+  fieldconfig.push({id: 'config_cpu', new_vm: new_vm, selected: selected});
+  fieldconfig.push({id: 'config_memory', new_vm: new_vm, selected: selected});
+  fieldconfig.push({id: 'config_cdrom', new_vm: new_vm, selected: selected});
+  fieldconfig.push({id: 'config_os', new_vm: new_vm, selected: selected});
+  fieldconfig.push({id: 'config_dns', new_vm: new_vm, selected: selected});
   fieldsets.forEach(toggleFieldsets);
-  toggleAccordion(selected);
   toggleVolumes(selected);
+  fieldconfig.forEach(toggleConfigs);
+  toggleAccordions();
   return false;
 }
 
@@ -131,7 +137,7 @@ function enableFieldset(fieldsetId, fieldset) {
   input_hidden_id(fieldsetId).removeAttr('disabled');
 }
 
-function disableFieldset(fieldsetId, fieldset) {  
+function disableFieldset(fieldsetId, fieldset) {
   if (fieldset.toggle && fieldset.new_vm){
     fieldset_id(fieldsetId, fieldset).hide();
   }
@@ -139,57 +145,72 @@ function disableFieldset(fieldsetId, fieldset) {
   input_hidden_id(fieldsetId).attr('disabled','disabled');
 }
 
-function enableConfigOptions(fieldsetId) {
-  var field = $("#" + fieldsetId + "_advanced_options");
-  field.accordion({collapsible : true, heightStyle: "content"});
-  field.removeClass('disabled').find("*").prop("disabled", false);
-  field.removeClass('hide');
+function toggleFieldset(fieldsetId, fieldset, type1, type2) {
+  type1 === type2 ? enableFieldset(fieldsetId, fieldset) : disableFieldset(fieldsetId, fieldset);
 }
 
-function disableConfigOptions(fieldsetId) {
-  var field = $("#" + fieldsetId + "_advanced_options");
-  field.addClass('disabled').find("*").prop("disabled", true);
-  field.addClass('hide');
+function input_hidden_id(volume_id){
+  return $("div[id^='"+ volume_id +"_volumes']" + " + input:hidden");
 }
 
-function toggleConfigOptions(fieldsetId, type1, type2) {
-  if (type1 === type2) {
-    enableConfigOptions(fieldsetId);
-  } else {
-    disableConfigOptions(fieldsetId);
-  }
+function fieldset_id(fieldsetId, fieldset){
+  return $("fieldset[id^='" + fieldsetId + "_"+fieldset.id+"']");
 }
 
-function toggleAccordion(selected){
+function fieldsets(type){
+  return type === 'qemu' ? ['server'] : ['container'];
+}
+
+function toggleFieldsets(fieldset){
+  var removable_input_hidden = $("div.removable-item[style='display: none;']" + " + input:hidden");
+  removable_input_hidden.attr('disabled','disabled');
   ['qemu', 'lxc'].forEach(function(type){
     fieldsets(type).forEach(function(fieldsetId){
-      toggleConfigOptions(fieldsetId, selected, type);
+      toggleFieldset(fieldsetId, fieldset, fieldset.selected, type);
     });
   });
 }
 
-function toggleFieldset(fieldsetId, fieldset, type1, type2) {  
-  type1 === type2 ? enableFieldset(fieldsetId, fieldset) : disableFieldset(fieldsetId, fieldset);
+function toggleAccordions() {
+  $('.accordion-content').hide();
+  $('.accordion-section').off('click').on('click', function(event) {
+    var $content = $(this).find('.accordion-content');
+    $content.slideToggle();
+    $(this).toggleClass('active');
+    $('.accordion-content').not($content).slideUp();
+    $('.accordion-section').not($(this)).removeClass('active');
+    event.stopPropagation();
+  });
+  $('.accordion-content').on('click', function(event) {
+    event.stopPropagation();
+  });
 }
 
-function input_hidden_id(volume_id){
-  return $("div[id^='"+ volume_id +"_volumes']" + " + input:hidden");
+function enableConfig(fieldsetId, fieldset) {
+  fieldset_id(fieldsetId, fieldset).removeClass('hide');
+  fieldset_id(fieldsetId, fieldset).removeAttr('disabled');
+  input_hidden_id(fieldsetId).removeAttr('disabled');
+
 }
 
-function fieldset_id(fieldsetId, fieldset){
-  return $("fieldset[id^='" + fieldsetId + "_"+fieldset.id+"']");
+function disableConfig(fieldsetId, fieldset) {
+  fieldset_id(fieldsetId, fieldset).addClass('hide');
+  fieldset_id(fieldsetId, fieldset).attr('disabled','disabled');
+  input_hidden_id(fieldsetId).attr('disabled','disabled');
 }
 
-function fieldsets(type){
-  return type === 'qemu' ? ['server'] : ['container'];
+function toggleConfig(fieldsetId, fieldset, type1, type2) {
+  if (type1 === type2) {
+    enableConfig(fieldsetId, fieldset);
+  } else {
+    disableConfig(fieldsetId, fieldset);
+  }
 }
 
-function toggleFieldsets(fieldset){
-  var removable_input_hidden = $("div.removable-item[style='display: none;']" + " + input:hidden");
-  removable_input_hidden.attr('disabled','disabled');  
+function toggleConfigs(fieldset){
   ['qemu', 'lxc'].forEach(function(type){
     fieldsets(type).forEach(function(fieldsetId){
-      toggleFieldset(fieldsetId, fieldset, fieldset.selected, type);
+      toggleConfig(fieldsetId, fieldset, fieldset.selected, type);
     });
   });
 }

data/app/assets/javascripts/foreman_fog_proxmox/proxmox_vm_server.js

@@ -32,7 +32,7 @@ function getIndex(item) {
 }
 
 function isProfile() {
-  return $(volumesAttributesSelector(true,0,'id')) !== undefined;
+  return $(volumesAttributesSelector(true,0,'id')).val() !== undefined;
 }
 
 function controllerSelected(item) {
@@ -42,23 +42,43 @@ function controllerSelected(item) {
   var profile = isProfile();
   var device_selector = volumesAttributesSelector(profile,index,'device');
   var id_selector = volumesAttributesSelector(profile,index,'id');
-  $(device_selector).attr('data-soft-max', max);
-  var device = $(device_selector).limitedSpinner('value');
+  var device = $(device_selector).val();
   var id = controller + device;
+  validateDeviceLimit(device_selector, max);
   $(id_selector).val(id);
 }
 
 function deviceSelected(item) {
-  var device = $(item).limitedSpinner('value');
+  var device = $(item).val();
   var index = getIndex(item);
   var profile = isProfile();
   var controller_selector = volumesAttributesSelector(profile,index,'controller');
   var id_selector = volumesAttributesSelector(profile,index,'id');
   var controller = $(controller_selector).val();
   var id = controller + device;
+  var max = computeControllerMaxDevice(controller);
+  validateDeviceLimit(item, max);
   $(id_selector).val(id);
 }
 
+function validateDeviceLimit(item, maxLimit) {
+  var warningContainer = $(item).closest('form').find('.warning-container');
+  var deviceNumber = $(item).val();
+  var submitButton = $(item).closest('form').find('[data-disable-with]');
+  if (!isNaN(maxLimit) && !isNaN(deviceNumber) && deviceNumber > maxLimit) {
+    if (warningContainer.length === 0) {
+      warningContainer = $('<div class="warning-container" style="color: red;"><span style="color: red;">&#9888;</span> Warning: Value exceeds the maximum limit of ' + maxLimit + '.</div>');
+      $(item).closest('.col-md-4').append(warningContainer);
+    }
+    warningContainer.show();
+    submitButton.prop('disabled', true);
+  } else {
+    warningContainer.hide();
+    warningContainer.remove();
+    submitButton.prop('disabled', false);
+  }
+}
+
 function computeControllerMaxDevice(controller) {
   switch (controller) {
     case 'ide':
@@ -72,4 +92,4 @@ function computeControllerMaxDevice(controller) {
     default:
       return 1;
   }
-}
+}

data/app/assets/stylesheets/foreman_fog_proxmox/accordion.css

@@ -0,0 +1,21 @@
+.accordion-section {
+   cursor: pointer;
+}
+
+.accordion-section > legend::before {
+    content: "\f054";
+    font-family: "Font Awesome 5 Free";
+    position: relative;
+    left: -1em;
+    top: 50%;
+    transform: translateY(-50%);
+}
+
+.accordion-section.active > legend::before {
+    content: "\f078";
+    font-family: "Font Awesome 5 Free";
+    position: relative;
+    left: -1em;
+    top: 50%;
+    transform: translateY(-50%);
+}

data/app/helpers/proxmox_vm_cloudinit_helper.rb

@@ -48,7 +48,7 @@ module ProxmoxVmCloudinitHelper
     wd = create_temp_directory(ssh)
 
     configs.each do |config|
-      config_file = ssh.run(%(echo "#{config[1]}" >> "#{wd}/#{config[0]}"))
+      config_file = ssh.run(%(echo '#{config[1]}' >> "#{wd}/#{config[0]}"))
       unless config_file.first.status.zero?
         delete_temp_dir(ssh, wd)
         raise ::Foreman::Exception, "Failed to create file #{config[0]}: #{config_file.first.stdout}"

data/app/models/foreman_fog_proxmox/proxmox.rb

@@ -181,7 +181,7 @@ module ForemanFogProxmox
        your server cannot connect to Proxmox due to network issues."
     end
 
-    def host
+    def proxmox_host
       URI.parse(url).host
     end
   end

data/app/models/foreman_fog_proxmox/proxmox_console.rb

@@ -33,7 +33,7 @@ module ForemanFogProxmox
         vnc_console = vm.start_console(options)
         vmid = extract_vmid(uuid).to_i
         vnc_host_port = vnc_console['port'].to_i + vmid
-        WsProxy.start(:host => host, :host_port => vnc_host_port, :password => vnc_console['ticket']).merge(
+        WsProxy.start(:host => proxmox_host, :host_port => vnc_host_port, :password => vnc_console['ticket']).merge(
           :name => vm.name, :type => type_console
         )
       rescue StandardError => e

data/app/models/foreman_fog_proxmox/proxmox_vm_commands.rb

@@ -32,10 +32,9 @@ module ForemanFogProxmox
     end
 
     def create_vm(args = {})
-      vmid = args[:vmid].to_i
       type = args[:type]
       node = client.nodes.get(args[:node_id])
-      vmid = node.servers.next_id.to_i if vmid < 1
+      vmid = args[:vmid] = assign_vmid(args[:vmid].to_i, node)
       raise ::Foreman::Exception, format(N_('invalid vmid=%<vmid>s'), vmid: vmid) unless node.servers.id_valid?(vmid)
 
       image_id = args[:image_id]
@@ -55,6 +54,10 @@ module ForemanFogProxmox
       raise e
     end
 
+    def assign_vmid(vmid, node)
+      vmid < 1 ? node.servers.next_id : vmid
+    end
+
     def compute_clone_attributes(args, container, type)
       args = parse_cloudinit_config(args) if args[:user_data]
       parsed_args = parse_typed_vm(args, type)

data/app/views/compute_resources_vms/form/proxmox/container/_config.html.erb

@@ -16,33 +16,37 @@ 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_container', "data-turbolinks-track" => true %>
+<%= stylesheet_link_tag 'foreman_fog_proxmox/accordion' %>
 <% container = type == 'lxc' %>
-<%= content_tag :div, :id => "container_advanced_options" do %>
-  <%= content_tag :h3, "Main Options"%>
-  <div>
-    <%= textarea_f f, :description, :label => _('Description'), :label_size => "col-md-2" %>
-    <%= checkbox_f f, :onboot, :label => _('Start at boot') %>
+<%= field_set_tag n_("Main option", "Main options", 2), :id => "container_config_options", :class => 'accordion-section', :disabled => !container do %>
+  <div class="accordion-content">
+  <%= textarea_f f, :description, :label => _('Description'), :label_size => "col-md-2" %>
+  <%= checkbox_f f, :onboot, :label => _('Start at boot') %>
   </div>
-  <%= content_tag :h3, "CPUs"%>
-  <div>
+<% end %>
+<%= field_set_tag _("CPU"), :id => "container_config_cpu", :class => 'accordion-section', :disabled => !container do %>
+  <div class="accordion-content">
   <%= select_f f, :arch, proxmox_archs_map, :id, :name, { }, :label => _('Architecture'), :label_size => "col-md-2" %>
   <%= counter_f f, :cores, :class => "input-mini", :label => _('Cores'), :label_size => "col-md-2" %>
   <%= counter_f f, :cpulimit, :class => "input-mini", :label => _('CPU limit'), :label_size => "col-md-2" %>
   <%= counter_f f, :cpuunits, :class => "input-mini", :label => _('CPU units'), :label_size => "col-md-2" %>
   </div>
-  <%= content_tag :h3, "Memory"%>
-  <div>
+<% end %>
+<%= field_set_tag _("Memory"), :id => "container_config_memory", :class => 'accordion-section', :disabled => !container do %>
+  <div class="accordion-content">
   <%= text_f f, :memory, :class => "input-mini", :label => _('Memory (MB)'), :label_size => "col-md-2" %>
   <%= text_f f, :swap, :class => "input-mini", :label => _('Swap (MB)'), :label_size => "col-md-2" %>
   </div>
-  <%= content_tag :h3, "DNS"%>
-  <div>
+<% end %>
+<%= field_set_tag _("DNS"), :id => "container_config_dns", :class => 'accordion-section', :disabled => !container do %>
+  <div class="accordion-content">
   <%= text_f f, :hostname, :label => _('Hostname'), :label_size => "col-md-2", :disabled => true %>
   <%= text_f f, :nameserver, :label => _('DNS server'), :label_size => "col-md-2" %>
   <%= text_f f, :searchdomain, :label => _('Search domain'), :label_size => "col-md-2" %>
   </div>
-  <%= content_tag :h3, "Operating System"%>
-  <div>
+<% end %>
+<%= field_set_tag _("Operating System"), :id => "container_config_os", :class => 'accordion-section', :disabled => !container do %>
+  <div class="accordion-content">
   <%= select_f f, :ostype, proxmox_ostypes_map, :id, :name, { }, :label => _('OS type'), :label_size => "col-md-2" %>
   </div>
 <% end %>

data/app/views/compute_resources_vms/form/proxmox/server/_config.html.erb

@@ -16,25 +16,23 @@ 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_server', "data-turbolinks-track" => true %>
-
+<%= stylesheet_link_tag 'foreman_fog_proxmox/accordion' %>
 <% server = type == 'qemu' %>
 <% logger.debug("_config.html.erb server_config=#{f.object.inspect}") %>
-
-<%= content_tag :div, :id => "server_advanced_options" do %>
-  <%= content_tag :h3, "Main Options" %>
-  <div>
-    <%= textarea_f f, :description, :label => _('Description'), :label_size => "col-md-2" %>
-    <%= text_f f, :boot, :label => _('Boot device order'), :label_size => "col-md-2", :label_help => _('Order your devices, e.g. order=net0;ide2;scsi0. Default empty (any)')  %>
-    <%= checkbox_f f, :onboot, :label => _('Start at boot') %>
-    <%= checkbox_f f, :agent, :label => _('Qemu Agent') %>
-    <%= checkbox_f f, :kvm, :label => _('KVM'), :label_help => _('Enable/disable KVM hardware virtualization') %>
-    <%= select_f f, :vga, proxmox_vgas_map, :id, :name, { :include_blank => true }, :label => _('VGA'), :label_size => "col-md-2" %>
-    <%= select_f f, :scsihw, proxmox_scsi_controllers_map, :id, :name, { }, :label => _('SCSI Controller'), :label_size => "col-md-2" %>
-    <%= select_f f, :bios, proxmox_bios_map, :id, :name, { }, :label => _('BIOS'), :label_size => "col-md-2" %>
+<%= field_set_tag n_("Main option", "Main options", 2), :id => "server_config_options", :class => 'accordion-section', :disabled => !server do %>
+  <div class="accordion-content">
+  <%= textarea_f f, :description, :label => _('Description'), :label_size => "col-md-2" %>
+  <%= text_f f, :boot, :label => _('Boot device order'), :label_size => "col-md-2", :label_help => _('Order your devices, e.g. order=net0;ide2;scsi0. Default empty (any)')  %>
+  <%= checkbox_f f, :onboot, :label => _('Start at boot') %>
+  <%= checkbox_f f, :agent, :label => _('Qemu Agent') %>
+  <%= checkbox_f f, :kvm, :label => _('KVM'), :label_help => _('Enable/disable KVM hardware virtualization') %>
+  <%= select_f f, :vga, proxmox_vgas_map, :id, :name, { :include_blank => true }, :label => _('VGA'), :label_size => "col-md-2" %>
+  <%= select_f f, :scsihw, proxmox_scsi_controllers_map, :id, :name, { }, :label => _('SCSI Controller'), :label_size => "col-md-2" %>
+  <%= select_f f, :bios, proxmox_bios_map, :id, :name, { }, :label => _('BIOS'), :label_size => "col-md-2" %>
   </div>
-
-  <%= content_tag :h3, "CPUs" %>
-  <div>
+<% end %>
+<%= field_set_tag _("CPUs"), :id => "server_config_cpu", :class => 'accordion-section', :disabled => !server do %>
+  <div class="accordion-content">
   <%= select_f f, :cpu_type, proxmox_cpus_map, :id, :name, { }, :label => _('Type'), :label_size => "col-md-2" %>
   <%= counter_f f, :sockets, :class => "input-mini", :label => _('Sockets'), :label_size => "col-md-2" %>
   <%= counter_f f, :cores, :class => "input-mini", :label => _('Cores'), :label_size => "col-md-2" %>
@@ -48,14 +46,16 @@ along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>. %>
     <% end %>
   <% end %>
   </div>
-  <%= content_tag :h3, "Memory" %>
-  <div>
+<% end %>
+<%= field_set_tag _("Memory"), :id => "server_config_memory", :class => 'accordion-section', :disabled => !server do %>
+  <div class="accordion-content">
   <%= text_f f, :memory, :class => "input-mini", :label => _('Memory (MB)'), :label_size => "col-md-2" %>
   <%= text_f f, :balloon, :class => "input-mini", :label => _('Minimum memory (MB)'), :label_size => "col-md-2" %>
   <%= text_f f, :shares, :class => "input-mini", :label => _('Shares (MB)'), :label_size => "col-md-2" %>
   </div>
-  <%= content_tag :h3, "Operating System" %>
-  <div>
+<% end %>
+<%= field_set_tag _("Operating System"), :id => "server_config_os", :class => 'accordion-section', :disabled => !server do %>
+  <div class="accordion-content">
   <%= select_f f, :ostype, proxmox_operating_systems_map, :id, :name, { :include_blank => true }, :label => _('OS type'), :label_size => "col-md-2" %>
   </div>
 <% end %>

data/app/views/compute_resources_vms/form/proxmox/server/_volume_hard_disk.html.erb

@@ -26,7 +26,10 @@ along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>. %>
     <%= f.hidden_field :storage_type %>
     <%= select_f f, :storage, compute_resource.storages(node_id), :storage, :storage, { }, :label => _('Storage'), :label_size => "col-md-2" %>
     <%= select_f f, :controller, proxmox_controllers_map, :id, :name, { }, :label => _('Controller'), :label_size => "col-md-2", :disabled => !new_volume, :onchange => 'controllerSelected(this)' %>
-    <%= text_f f, :device, :label => _('Device'), :label_size => "col-md-2", :disabled => !new_volume, :'data-min' => 0, :'data-soft-max' => proxmox_max_device(f.object.controller), :onchange => 'deviceSelected(this)' %>
+    <div>
+       <%= text_f f, :device, :label => _('Device'), :label_size => "col-md-2", :disabled => !new_volume, :onchange => 'deviceSelected(this)' %>
+    <span class="warning-container"></span>
+    </div>
     <%= select_f f, :cache, proxmox_caches_map, :id, :name, { include_blank: true }, :label => _('Cache'), :label_size => "col-md-2" %>
     <%= text_f f, :size, :class => "input-mini", :label => _("Size (GB)"), :label_size => "col-md-2", :disabled => !hard_disk %>
 <% 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.0'
+  VERSION = '0.15.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_fog_proxmox
 version: !ruby/object:Gem::Version
-  version: 0.15.0
+  version: 0.15.1
 platform: ruby
 authors:
 - Tristan Robert
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-11-15 00:00:00.000000000 Z
+date: 2024-04-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface
@@ -114,6 +114,7 @@ files:
 - app/assets/javascripts/foreman_fog_proxmox/proxmox_volume.js
 - app/assets/javascripts/foreman_fog_proxmox/proxmox_volume_cdrom.js
 - app/assets/javascripts/foreman_fog_proxmox/proxmox_volume_cloudinit.js
+- app/assets/stylesheets/foreman_fog_proxmox/accordion.css
 - app/controllers/concerns/foreman_fog_proxmox/compute_resources_vms_controller.rb
 - app/controllers/concerns/foreman_fog_proxmox/controller/parameters/compute_resource.rb
 - app/controllers/concerns/foreman_fog_proxmox/hosts_controller.rb
@@ -262,29 +263,29 @@ signing_key:
 specification_version: 4
 summary: Foreman plugin that adds Proxmox VE compute resource using fog-proxmox
 test_files:
-- test/factories/proxmox_factory.rb
+- test/test_plugin_helper.rb
+- test/factories/foreman_fog_proxmox/proxmox_node_mock_factory.rb
 - test/factories/foreman_fog_proxmox/proxmox_container_mock_factory.rb
 - test/factories/foreman_fog_proxmox/proxmox_server_mock_factory.rb
-- test/factories/foreman_fog_proxmox/proxmox_node_mock_factory.rb
-- test/test_plugin_helper.rb
-- test/functional/compute_resources_controller_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_create_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_container_test.rb
+- test/factories/proxmox_factory.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_images_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_interfaces_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_test.rb
 - test/unit/foreman_fog_proxmox/proxmox_vm_queries_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_new_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_test.rb
 - test/unit/foreman_fog_proxmox/semver_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_cdrom_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_hard_disk_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_create_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_version_test.rb
 - test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_cloudinit_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_new_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_images_test.rb
 - test/unit/foreman_fog_proxmox/proxmox_compute_attributes_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_container_test.rb
+- test/unit/foreman_fog_proxmox/helpers/proxmox_vm_volumes_helper_test.rb
 - test/unit/foreman_fog_proxmox/helpers/proxmox_server_helper_test.rb
-- test/unit/foreman_fog_proxmox/helpers/proxmox_vm_helper_test.rb
 - test/unit/foreman_fog_proxmox/helpers/proxmox_container_helper_test.rb
+- test/unit/foreman_fog_proxmox/helpers/proxmox_vm_helper_test.rb
 - test/unit/foreman_fog_proxmox/helpers/proxmox_vm_uuid_helper_test.rb
-- test/unit/foreman_fog_proxmox/helpers/proxmox_vm_volumes_helper_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_version_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_interfaces_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_hard_disk_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_cdrom_test.rb
+- test/functional/compute_resources_controller_test.rb