foreman_bootdisk 17.0.2 → 17.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cc43113465fd89f890f9606fb04af3e948b96ac1bd6c1943211a452e38c4cb2a
-  data.tar.gz: 9db1069085f706df776576fc0df25f49a7d09ae71555ea15397d57656ddd0da5
+  metadata.gz: ca80bb43629c291c9d6b27bb8c94ebd10fcc2e6e267378e4852ab42bdcee0088
+  data.tar.gz: 743d4a31beef4d80d6ab5b025cf6261bbbb11cb62eb82b65d8f7c5aaed58c9fb
 SHA512:
-  metadata.gz: 115bddba362f7c53b78d06ca78a1f120babae76cc2f922447cd0e05b1f322f189e30b186e6cb0718b7d9a24bfca67bf2287283fd9495eba4193a9b6bb560ec29
-  data.tar.gz: 1cd764fdc0a939d45934070d6b34e2619870c8c206afd4915cae93f68f7553cfcc9b18d3c04770aba9f4b6f45d62bbd779563798b7175595ee8c2b991f03b660
+  metadata.gz: 1915d2f7f01f58ec795f343707fe7a6d24f49af0ce6fb6ee9577f3279749e74f5cfb74b7ee707f3eea0e972b92344bc7d10a30f7be9556bffe8be2a39315526d
+  data.tar.gz: b21943b356b209d60ad7cd1bc655a81c042ef93534fb94f719c271688f64f42a2f2ea3ba21a10280b388861fd061d1e08df498de216dc233f27a92e10065c942

data/.github/workflows/rubocop.yaml

@@ -0,0 +1,19 @@
+name: Rubocop
+
+on: [pull_request]
+
+jobs:
+  rubocop:
+    runs-on: ubuntu-latest
+    env:
+      BUNDLE_FOREMAN: '0'
+      BUNDLE_WITHOUT: development:test
+    steps:
+      - uses: actions/checkout@v2
+      - name: Setup Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 2.5
+          bundler-cache: true
+      - name: Run rubocop
+        run: bundle exec rubocop -P --format github

data/.tx/config

@@ -1,7 +1,7 @@
 [main]
 host = https://www.transifex.com
 
-[foreman.bootdisk]
+[foreman.foreman_bootdisk]
 file_filter = locale/<lang>/foreman_bootdisk.edit.po
 source_file = locale/foreman_bootdisk.pot
 source_lang = en

data/AUTHORS

@@ -4,6 +4,7 @@ Bryan Kearney
 crito
 Damon Maria
 Dominic Cleal
+Dominik Matoulek
 Greg Sutcliffe
 Ido Kanner
 Jon Fautley
@@ -13,13 +14,16 @@ Lukas Zapletal
 Lukáš Zapletal
 Marek Hulan
 Marek Hulán
+Markus Bucher
 Martin Bačovský
 Michael Hofer
 Michael Moll
 Ohad Levy
+Oleh Fedorenko
 Ondrej Prazak
 Ondřej Ezr
 Ondřej Pražák
+Pat Riehecky
 Robert Antoni Buj Gelonch
 Rohan21Lobo
 Ronald van Zantvoort

data/app/controllers/concerns/allowed_actions.rb

@@ -0,0 +1,16 @@
+module AllowedActions
+  extend ActiveSupport::Concern
+
+  def bootdisk_type_allowed?(action = params[:action])
+    return true if Setting::Bootdisk.allowed_types&.include?(action)
+
+    message = _('This type of bootdisk is not allowed. Please contact administrator.')
+    if api_request?
+      render_error :custom_error, status: :unprocessable_entity, locals: { message: message}
+    else
+      error(message)
+      redirect_back(fallback_location: '/')
+    end
+    false
+  end
+end

data/app/controllers/foreman_bootdisk/api/v2/disks_controller.rb

@@ -7,11 +7,13 @@ module ForemanBootdisk
     module V2
       class DisksController < ::Api::V2::BaseController
         include ::Api::Version2
+        include AllowedActions
 
         resource_description do
           api_base_url '/bootdisk/api'
         end
 
+        before_action :bootdisk_type_allowed?, only: :generic
         before_action :find_resource, only: :host
         skip_after_action :log_response_body
 
@@ -34,10 +36,12 @@ module ForemanBootdisk
         def host
           host = @disk
           if params[:full]
+            return unless bootdisk_type_allowed?('full_host')
             ForemanBootdisk::ISOGenerator.generate_full_host(host) do |iso|
               send_file(iso, filename: "#{host.name}#{ForemanBootdisk::ISOGenerator.token_expiry(host)}.iso")
             end
           else
+            return unless bootdisk_type_allowed?
             # EFI not supported for iPXE host bootdisk
             tmpl = host.bootdisk_template_render
             ForemanBootdisk::ISOGenerator.generate(ipxe: tmpl) do |iso|

data/app/controllers/foreman_bootdisk/api/v2/subnet_disks_controller.rb

@@ -7,11 +7,14 @@ module ForemanBootdisk
     module V2
       class SubnetDisksController < ::Api::V2::BaseController
         include ::Api::Version2
+        include AllowedActions
 
         resource_description do
           api_base_url '/bootdisk/api'
         end
 
+        rescue_from ActiveRecord::RecordNotFound, :with => :subnet_not_found
+        before_action :bootdisk_type_allowed?, only: :subnet
         before_action :find_resource, only: :subnet
 
         skip_after_action :log_response_body
@@ -42,6 +45,10 @@ module ForemanBootdisk
         def resource_scope
           Subnet.authorized('view_subnets')
         end
+
+        def subnet_not_found
+          not_found ("Subnet not found by id '%s'") % params[:id]
+        end
       end
     end
   end

data/app/controllers/foreman_bootdisk/disks_controller.rb

@@ -4,6 +4,9 @@ require 'uri'
 
 module ForemanBootdisk
   class DisksController < ::ApplicationController
+    include AllowedActions
+
+    before_action :bootdisk_type_allowed?, except: :help
     before_action :find_resource, only: %w[host full_host subnet]
 
     # as this engine is isolated, we need to include url helpers from core explicitly

data/app/helpers/concerns/foreman_bootdisk/hosts_helper_ext.rb

@@ -8,42 +8,7 @@ module ForemanBootdisk
           button_group(
             select_action_button(
               _('Boot disk'), { class: 'btn btn-group' },
-              display_bootdisk_link_if_authorized(
-                _("Host '%s' image") % host.name.split('.')[0],
-                {
-                  controller: 'foreman_bootdisk/disks',
-                  action: 'host',
-                  id: host
-                },
-                class: 'la'
-              ),
-              display_bootdisk_link_if_authorized(
-                _("Full host '%s' image") % host.name.split('.')[0],
-                {
-                  controller: 'foreman_bootdisk/disks',
-                  action: 'full_host',
-                  id: host
-                },
-                class: 'la'
-              ),
-              content_tag(:li, '', class: 'divider'),
-              display_bootdisk_link_if_authorized(
-                _('Generic image'),
-                {
-                  controller: 'foreman_bootdisk/disks',
-                  action: 'generic'
-                },
-                class: 'la'
-              ),
-              display_bootdisk_for_subnet(host),
-              content_tag(:li, '', class: 'divider'),
-              display_bootdisk_link_if_authorized(
-                _('Help'), {
-                  controller: 'foreman_bootdisk/disks',
-                  action: 'help'
-                },
-                class: 'la'
-              )
+              host_action_buttons(host)
             )
           )
         )
@@ -95,5 +60,66 @@ module ForemanBootdisk
     def bootdisk_authorized_for(options)
       User.current.allowed_to?(options)
     end
+
+    def host_action_buttons(host)
+      actions = []
+
+      allowed_actions = Setting::Bootdisk.allowed_types
+      return '' unless allowed_actions
+
+      host_image_link = display_bootdisk_link_if_authorized(
+                          _("Host '%s' image") % host.name.split('.')[0],
+                          {
+                              controller: 'foreman_bootdisk/disks',
+                              action: 'host',
+                              id: host
+                          },
+                          class: 'la'
+                        )
+
+      full_host_image_link = display_bootdisk_link_if_authorized(
+                               _("Full host '%s' image") % host.name.split('.')[0],
+                               {
+                                   controller: 'foreman_bootdisk/disks',
+                                   action: 'full_host',
+                                   id: host
+                               },
+                               class: 'la'
+                             )
+
+      generic_image_link = display_bootdisk_link_if_authorized(
+                             _('Generic image'),
+                             {
+                                 controller: 'foreman_bootdisk/disks',
+                                 action: 'generic'
+                             },
+                             class: 'la'
+                           )
+
+      help_link = display_bootdisk_link_if_authorized(
+                    _('Help'),
+                    {
+                    controller: 'foreman_bootdisk/disks',
+                    action: 'help'
+                    },
+                    class: 'la'
+                  )
+
+      divider = tag(:li, '', class: 'divider')
+
+      actions << host_image_link if allowed_actions.include?('host')
+      actions << full_host_image_link if allowed_actions.include?('full_host')
+
+      subnet_link = display_bootdisk_for_subnet(host)
+      actions << divider if allowed_actions.include?('generic') || (allowed_actions.include?('subnet') && subnet_link.present?)
+
+      actions << generic_image_link if allowed_actions.include?('generic')
+      actions << subnet_link if allowed_actions.include?('subnet')
+
+      actions << divider
+      actions << help_link
+
+      actions
+    end
   end
 end

data/app/lib/foreman_bootdisk/scope/bootdisk.rb

@@ -3,6 +3,20 @@
 module ForemanBootdisk
   module Scope
     class Bootdisk < ::Foreman::Renderer::Scope::Provisioning
+      extend ApipieDSL::Class
+
+      apipie :class, 'Macros related to provisioning via boot disk' do
+        name 'Bootdisk'
+        sections only: %w[all provisioning]
+      end
+
+      apipie :method, 'Generates URL for boot chain' do
+        optional :mac, String, 'MAC address of the host', default: 'MAC address of the current host'
+        optional :action, String, 'Bootloader to use', default: 'iPXE'
+        returns String, desc: 'URL for boot chain'
+        example 'bootdisk_chain_url #=> "http://foreman.some.host.fqdn/unattended/iPXE?mac=00%3A11%3A22%3A33%3A44%3A55"'
+        example 'bootdisk_chain_url("00:11:22:33:44:55") #=> "http://foreman.some.host.fqdn/unattended/iPXE?mac=00%3A11%3A22%3A33%3A44%3A55"'
+      end
       def bootdisk_chain_url(mac = host.try(:mac), action = 'iPXE')
         url = foreman_url(action)
         u = URI.parse(url)
@@ -14,6 +28,16 @@ module ForemanBootdisk
         u.to_s
       end
 
+      apipie :method, 'Always raises an error with a description provided as an argument' do
+        desc 'This method is useful for aborting script execution if some of the conditions are not met'
+        list :args, desc: 'Description for the error'
+        raises error: ::Foreman::Exception, desc: 'The error is always being raised'
+        returns nil, desc: "Doesn't return anything"
+        example "<%
+  interface = @host.provision_interface #=> interface = nil
+  bootdisk_raise(N_('Host has no provisioning interface defined')) unless interface #=> Foreman::Exception is raised and the execution of the script is aborted
+%>"
+      end
       def bootdisk_raise(*args)
         raise ::Foreman::Exception.new(*args)
       end

data/app/models/concerns/foreman_bootdisk/host_ext.rb

@@ -4,6 +4,8 @@ require 'uri'
 
 module ForemanBootdisk
   module HostExt
+    extend ApipieDSL::Extension
+
     def bootdisk_template
       template = ProvisioningTemplate.unscoped.find_by(name: Setting[:bootdisk_host_template])
       unless template
@@ -36,6 +38,10 @@ module ForemanBootdisk
     def can_be_built?
       super || (managed? && SETTINGS[:unattended] && bootdisk_build? && !build?)
     end
+
+    apipie_update :class do
+      property :bootdisk_build?, one_of: [true, false], desc: 'Returns true if provision method for this host is bootdisk, false otherwise'
+    end
   end
 end
 

data/app/models/concerns/foreman_bootdisk/orchestration/compute.rb

@@ -42,8 +42,16 @@ module ForemanBootdisk
       end
 
       def bootdisk_generate_iso_image
-        ForemanBootdisk::ISOGenerator.generate(ipxe: bootdisk_template_render, dir: Dir.tmpdir) do |image|
-          FileUtils.mv image, bootdisk_isofile
+        if self.build? && self.provisioning_template(kind: :PXELinux) && self.provisioning_template(kind: :PXEGrub2)
+          logger.info format('Generating FULL HOST ISO image for %s', name)
+          ForemanBootdisk::ISOGenerator.generate_full_host(self, dir: Dir.tmpdir) do |image|
+            FileUtils.mv image, bootdisk_isofile
+          end
+        else
+          logger.info format('Generating HOST ISO image for %s', name)
+          ForemanBootdisk::ISOGenerator.generate(ipxe: bootdisk_template_render, dir: Dir.tmpdir) do |image|
+            FileUtils.mv image, bootdisk_isofile
+          end
         end
       end
 
@@ -60,7 +68,6 @@ module ForemanBootdisk
       end
 
       def setGenerateIsoImage
-        logger.info format('Generating ISO image for %s', name)
         bootdisk_generate_iso_image
       rescue StandardError => e
         failure format(_('Failed to generate ISO image for instance %{name}: %{message}'), name: name, message: e.message), e

data/app/models/setting/bootdisk.rb

@@ -21,12 +21,21 @@ class Setting
         set('bootdisk_generic_efi_host_template', N_('Grub2 template to use for generic EFI host boot disks'),
             'Boot disk Grub2 EFI - generic host', N_('Generic Grub2 EFI image template'), nil, collection: templates),
         set('bootdisk_mkiso_command', N_('Command to generate ISO image, use genisoimage or mkisofs'), 'genisoimage', N_('ISO generation command')),
-        set('bootdisk_cache_media', N_('Installation media files will be cached for full host images'), true, N_('Installation media caching'))
+        set('bootdisk_cache_media', N_('Installation media files will be cached for full host images'), true, N_('Installation media caching')),
+        set('bootdisk_allowed_types', N_('List of allowed bootdisk types, remove type to disable it'), Setting::Bootdisk.bootdisk_types, N_('Allowed bootdisk types'))
       ]
     end
 
     def self.humanized_category
       N_('Boot disk')
     end
+
+    def self.bootdisk_types
+      %w(generic host full_host subnet)
+    end
+
+    def self.allowed_types
+      Setting['bootdisk_allowed_types']
+    end
   end
 end

data/app/services/foreman_bootdisk/iso_generator.rb

@@ -10,6 +10,8 @@ require 'uri'
 # requires syslinux, ipxe/ipxe-bootimgs, genisoimage, isohybrid
 module ForemanBootdisk
   class ISOGenerator
+    extend Foreman::HTTPProxy
+
     def self.generate_full_host(host, opts = {}, &block)
       raise Foreman::Exception.new(N_('Host is not in build mode, so the template cannot be rendered')) unless host.build?
 
@@ -83,8 +85,8 @@ module ForemanBootdisk
         EOT
       end
 
-      # Temporary directory cannot be cleaned inprocess due to send_file offloaded to web server
-      wd = Dir.mktmpdir('bootdisk-iso-')
+      # And create new temporary directory:
+      wd = Dir.mktmpdir('bootdisk-iso-', Rails.root.join('tmp'))
       Dir.mkdir(File.join(wd, 'build'))
 
       if opts[:isolinux]
@@ -132,8 +134,8 @@ module ForemanBootdisk
           File.join(Setting[:bootdisk_grub2_dir], 'grubx64.efi').to_s => 'GRUBX64.EFI',
           File.join(Setting[:bootdisk_grub2_dir], 'shimx64.efi').to_s => 'BOOTX64.EFI',
         }.each do |src, dest|
-          raise(Foreman::Exception.new(N_('Ensure %{path} contains grubx64.efi and shimx64.efi: install TFTP module or copy those files into /var/lib/foreman/bootdisk (Grub2 directory setting)'), path: src)) unless File.exist?(src)
-          raise(Foreman::Exception.new(N_('Unable to mcopy %{path}'), path: src)) unless system("mcopy -m -i #{efibootimg} '#{src}' '#{"::/EFI/BOOT/#{dest}"}'")
+          raise(Foreman::Exception.new(N_('Ensure %{file} is readable (or update "Grub2 directory" setting)'), file: src)) unless File.exist?(src)
+          raise(Foreman::Exception.new(N_('Unable to mcopy %{file}'), file: src)) unless system("mcopy -m -i #{efibootimg} '#{src}' '#{"::/EFI/BOOT/#{dest}"}'")
         end
         mkiso_args = "-eltorito-alt-boot -e efiboot.img -no-emul-boot"
         isohybrod_args = ["--uefi"]
@@ -161,6 +163,15 @@ module ForemanBootdisk
       raise Foreman::Exception.new(N_('ISO hybrid conversion failed: %s'), $?) unless system(*["isohybrid", isohybrod_args, iso].flatten.compact)
 
       yield iso
+    ensure
+      # Clean the working directory (not the ISO file itself)
+      FileUtils.rm_f(File.join(wd, 'build'))
+      # Temporary directory cannot be cleaned in-process due to asynchronous send_file call.
+      # Also we cannot rely on systemd-tmpfiles-clean as private temporary files are not subject
+      # of scheduled cleanups. Let's clean bootdisks from prevous requests manually by finding
+      # and deleting all directories created 30 minutes ago.
+      delete_older_than = Time.now.to_i - (60 * 30)
+      Rails.root.glob('tmp/bootdisk-iso-*').select { |f| File.ctime(f) < delete_older_than }.each { |f| FileUtils.rm_f(f) }
     end
 
     def self.build_mkiso_command(output_file:, source_directory:, extra_commands:)
@@ -200,7 +211,14 @@ module ForemanBootdisk
           ForemanBootdisk.logger.info("Fetching #{uri}")
           write_cache = use_cache
           uri = URI(uri)
-          Net::HTTP.start(uri.host, uri.port) do |http|
+
+          if proxy_http_request?(nil, uri.host, uri.scheme)
+            proxy_uri = URI.parse(http_proxy)
+            http_object = Net::HTTP::Proxy(proxy_uri.host, proxy_uri.port, proxy_uri.user, proxy_uri.password)
+          else
+            http_object = Net::HTTP
+          end
+          http_object.start(uri.host, uri.port) do |http|
             request = Net::HTTP::Get.new(uri.request_uri, 'Accept-Encoding' => 'plain')
 
             http.request(request) do |response|

data/app/views/foreman_bootdisk/generic_efi_host.erb

@@ -44,6 +44,10 @@ net_bootp
 net_ls_addr
 net_ls_routes
 net_ls_dns
+echo Trying to ping proxy_httpboot_host: <%= proxy_httpboot_host %>
+ping --count 1 <%= proxy_httpboot_host %> || echo Ping to proxy_httpboot_host failed or ping command not available.
+echo Trying to ping proxy_template_host: <%= proxy_template_host %>
+ping --count 1 <%= proxy_template_host %> || echo Ping to proxy_template_host failed or ping command not available.
 sleep 5
 set root=(<%= proxy_proto %>,<%= proxy_httpboot_host %>:<%= proxy_port %>)
 # The variable will not survive configfile fetch, therefore absolute path

data/app/views/foreman_bootdisk/host.erb

@@ -58,6 +58,13 @@ set dns <%= dns %>
 set domain <%= interface.domain.to_s %>
 <% end %>
 
+echo Trying to ping Gateway: ${netX/gateway}
+ping --count 1 ${netX/gateway} || echo Ping to Gateway failed or ping command not available.
+<% if dns.present? -%>
+echo Trying to ping DNS: ${netX/dns}
+ping --count 1 ${netX/dns} || echo Ping to DNS failed or ping command not available.
+<% end %>
+
 # Chainload from Foreman rather than embedding OS info here, so the behaviour
 # is entirely dynamic.
 chain <%= bootdisk_chain_url %>

data/lib/foreman_bootdisk/engine.rb

@@ -51,6 +51,9 @@ module ForemanBootdisk
         add_all_permissions_to_default_roles
 
         apipie_documented_controllers ["#{ForemanBootdisk::Engine.root}/app/controllers/foreman_bootdisk/api/v2/*.rb"]
+        ApipieDSL.configuration.dsl_classes_matchers += [
+          "#{ForemanBootdisk::Engine.root}/app/lib/foreman_bootdisk/scope/*.rb"
+        ]
         provision_method 'bootdisk', N_('Boot disk based')
         template_labels 'Bootdisk' => N_('Boot disk embedded template')
         allowed_template_helpers :bootdisk_chain_url, :bootdisk_raise

data/lib/foreman_bootdisk/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ForemanBootdisk
-  VERSION = '17.0.2'
+  VERSION = '17.1.0'
 end

data/lib/tasks/bootdisk.rake

@@ -88,23 +88,5 @@ end
 
 Rake::Task[:test].enhance ['test:foreman_bootdisk']
 
-namespace :foreman_bootdisk do
-  task :rubocop do
-    begin
-      require 'rubocop/rake_task'
-      RuboCop::RakeTask.new(:rubocop_foreman_bootdisk) do |task|
-        task.patterns = ["#{ForemanBootdisk::Engine.root}/app/**/*.rb",
-                         "#{ForemanBootdisk::Engine.root}/lib/**/*.rb",
-                         "#{ForemanBootdisk::Engine.root}/test/**/*.rb"]
-        task.options << '--auto-correct' if ENV['RUBOCOP_AUTOCORRECT']
-      end
-    rescue StandardError
-      puts 'Rubocop not loaded.'
-    end
-
-    Rake::Task['rubocop_foreman_bootdisk'].invoke
-  end
-end
-
 load 'tasks/jenkins.rake'
-Rake::Task['jenkins:unit'].enhance ['test:foreman_bootdisk', 'foreman_bootdisk:rubocop'] if Rake::Task.task_defined?(:'jenkins:unit')
+Rake::Task['jenkins:unit'].enhance ['test:foreman_bootdisk'] if Rake::Task.task_defined?(:'jenkins:unit')

data/locale/action_names.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 # Autogenerated!
 _("Remote action:")
 _("Import facts")
 _("Import Puppet classes")
-_("Action with sub plans")
+_("Action with sub plans")

metadata

@@ -1,59 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: foreman_bootdisk
 version: !ruby/object:Gem::Version
-  version: 17.0.2
+  version: 17.1.0
 platform: ruby
 authors:
 - Dominic Cleal
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-06-15 00:00:00.000000000 Z
+date: 2021-04-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rubocop-minitest
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rubocop-performance
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rubocop-rails
+  name: theforeman-rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -90,12 +48,14 @@ extra_rdoc_files:
 - LICENSE
 - README.md
 files:
+- ".github/workflows/rubocop.yaml"
 - ".tx/config"
 - AUTHORS
 - CHANGES.md
 - LICENSE
 - README.md
 - app/assets/javascripts/foreman_bootdisk/host_edit.js
+- app/controllers/concerns/allowed_actions.rb
 - app/controllers/foreman_bootdisk/api/v2/disks_controller.rb
 - app/controllers/foreman_bootdisk/api/v2/subnet_disks_controller.rb
 - app/controllers/foreman_bootdisk/disks_controller.rb
@@ -180,7 +140,7 @@ homepage: http://github.com/theforeman/foreman_bootdisk
 licenses:
 - GPL-3.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -188,15 +148,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Create boot disks to provision hosts with Foreman
 test_files: []