foreman_bootdisk 10.0.2 → 11.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8e1f729480f6663624d1fb96b994a381e9926c1d99f7239d02fca9a376d6ccaa
-  data.tar.gz: e94de3dd4d1dd78abac381ca057e6da44a42e0a5f9b5afbd493c29ce6310861a
+  metadata.gz: 7090fb0dd77db85fc9aa172b0a785f2d97c37a3dc5597a689a5c79fd7fbd0f2a
+  data.tar.gz: d35862a0f641367cc7bfd1fa95a801e0ab5afd056b19995fa6f05aae97bf4bdd
 SHA512:
-  metadata.gz: 15348dcc0e22146f014239eae49e90a0e4fd95050d44c42cab4ee461e165ea63f407b797c70001c49180d36c0fef47ca166aa01e26253d08ca2412b3cbeac7b0
-  data.tar.gz: b3f5b11e82ed73e2d0c7373f61af61fd1b861c45e2399b02e010399d24139d89ded5ecaae5663d17b19fec370e9ccec592e3dff2734f2431c9f493291ad965a5
+  metadata.gz: c7c3a1e1f5e951b2252087d2064c9886a779a08c96a4c52d15c5d7711a542d5c9195e96bd86aecf2d82f780b633f11680c91368304d7874e8130b181e0f1606f
+  data.tar.gz: 278854b3bcfbd844c72d0c817e1479d47a47125f95f47cb2b08b7dc257f0ece53f73070296681952c7bfeced060329e30033bf021a371420727b3b942461fb92

data/CHANGES.md

@@ -1,12 +1,11 @@
 # Changelog
 
-## v10.0.2
+## v11.0.0
 * bootdisk_raise template marco works in preview mode (#22136)
 * host image works with subnet without gateway (#21106)
-
-## v10.0.1
 * Fix rake tasks are Taxonomy and User aware (#21877)
-* Revert alias_method_chain with Module#prepend for Katello 3.4 compatibility
+* database migrations support rails 5 (#21821)
+* changes to support rails 5.1
 
 ## v10.0.0
 * retry vmware iso boot 10 times before giving up (#19013)

data/README.md

@@ -45,6 +45,7 @@ gPXE images are unsupported due to lack of initrd support.
 | >= 1.12         | ~> 8.0         |
 | >= 1.13         | ~> 9.0         |
 | >= 1.15         | ~> 10.0        |
+| >= 1.17         | ~> 11.0        |
 
 # Usage
 

data/app/controllers/concerns/foreman_bootdisk/unattended_controller_ext.rb

@@ -1,12 +1,6 @@
 module ForemanBootdisk::UnattendedControllerExt
-  extend ActiveSupport::Concern
-
-  included do
-    alias_method_chain :find_host_by_ip_or_mac, :param_mac
-  end
-
-  def find_host_by_ip_or_mac_with_param_mac
+  def find_host_by_ip_or_mac
     request.env['HTTP_X_RHN_PROVISIONING_MAC_0'] = "unknown #{params['mac']}" unless request.env.has_key?('HTTP_X_RHN_PROVISIONING_MAC_0') || params['mac'].nil?
-    find_host_by_ip_or_mac_without_param_mac
+    super
   end
 end

data/app/controllers/foreman_bootdisk/disks_controller.rb

@@ -9,7 +9,7 @@ module ForemanBootdisk
         tmpl = ForemanBootdisk::Renderer.new.generic_template_render
       rescue => e
         error_rendering(e)
-        redirect_to :back
+        redirect_back(fallback_location: '/')
         return
       end
 
@@ -24,7 +24,7 @@ module ForemanBootdisk
         tmpl = host.bootdisk_template_render
       rescue => e
         error_rendering(e)
-        redirect_to :back
+        redirect_back(fallback_location: '/')
         return
       end
 
@@ -48,7 +48,7 @@ module ForemanBootdisk
         tmpl = ForemanBootdisk::Renderer.new.generic_template_render(subnet)
       rescue => e
         error_rendering(e)
-        redirect_to :back
+        redirect_back(fallback_location: '/')
         return
       end
 

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

@@ -1,11 +1,5 @@
 module ForemanBootdisk::HostsHelperExt
-  extend ActiveSupport::Concern
-
-  included do
-    alias_method_chain :host_title_actions, :bootdisk
-  end
-
-  def host_title_actions_with_bootdisk(*args)
+  def host_title_actions(*args)
     if @host.bootdisk_downloadable?
       title_actions(
         button_group(
@@ -24,7 +18,7 @@ module ForemanBootdisk::HostsHelperExt
       bootdisk_button_disabled
     end
 
-    host_title_actions_without_bootdisk(*args)
+    super
   end
 
   def bootdisk_button_disabled

data/app/models/concerns/foreman_bootdisk/compute_resources/vmware.rb

@@ -1,19 +1,12 @@
 module ForemanBootdisk
   module ComputeResources
     module Vmware
-      extend ActiveSupport::Concern
-
-      included do
-        alias_method_chain :capabilities, :bootdisk
-        alias_method_chain :parse_args, :bootdisk
-      end
-
-      def capabilities_with_bootdisk
-        capabilities_without_bootdisk + [:bootdisk]
+      def capabilities
+        super + [:bootdisk]
       end
 
-      def parse_args_with_bootdisk(args = {})
-        args = parse_args_without_bootdisk args
+      def parse_args(args = {})
+        args = super
         if args[:provision_method] == 'bootdisk'
           args[:cdroms] = [new_cdrom]
           args[:boot_order] = ['cdrom', 'disk']

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

@@ -1,13 +1,6 @@
 require 'uri'
 
 module ForemanBootdisk::HostExt
-  extend ActiveSupport::Concern
-
-  included do
-    alias_method_chain :validate_media?, :bootdisk
-    alias_method_chain :can_be_built?, :bootdisk
-  end
-
   def bootdisk_template
     ProvisioningTemplate.unscoped.find_by_name(Setting[:bootdisk_host_template]) || raise(::Foreman::Exception.new(N_('Unable to find template specified by %s setting'), 'bootdisk_host_template'))
   end
@@ -37,11 +30,11 @@ module ForemanBootdisk::HostExt
     /i.86|x86[_-]64/ =~ architecture.name
   end
 
-  def validate_media_with_bootdisk?
-    validate_media_without_bootdisk? || (managed && bootdisk_build? && build?)
+  def validate_media?
+    super || (managed && bootdisk_build? && build?)
   end
 
-  def can_be_built_with_bootdisk?
-    can_be_built_without_bootdisk? || (managed? and SETTINGS[:unattended] and bootdisk_build? and !build?)
+  def can_be_built?
+    super || (managed? and SETTINGS[:unattended] and bootdisk_build? and !build?)
   end
 end

data/app/models/setting/bootdisk.rb

@@ -1,6 +1,6 @@
 class Setting::Bootdisk< ::Setting
   def self.load_defaults
-    return unless ActiveRecord::Base.connection.table_exists?('settings')
+    return unless ApplicationRecord.connection.table_exists?('settings')
     return unless super
 
     ipxe = ['/usr/lib/ipxe'].find { |p| File.exist?(p) } || '/usr/share/ipxe'

data/db/migrate/20130914211030_create_host_bootdisk_template.rb

@@ -1,4 +1,4 @@
-class CreateHostBootdiskTemplate < ActiveRecord::Migration
+class CreateHostBootdiskTemplate < ActiveRecord::Migration[4.2]
   def self.up
   end
 

data/db/migrate/20130915104500_edit_host_bootdisk_template_multinic.rb

@@ -1,4 +1,4 @@
-class EditHostBootdiskTemplateMultinic < ActiveRecord::Migration
+class EditHostBootdiskTemplateMultinic < ActiveRecord::Migration[4.2]
   def self.up
   end
 

data/db/migrate/20130915133321_create_kickstart_bootdisk_template.rb

@@ -1,4 +1,4 @@
-class CreateKickstartBootdiskTemplate < ActiveRecord::Migration
+class CreateKickstartBootdiskTemplate < ActiveRecord::Migration[4.2]
   def self.up
   end
 

data/db/migrate/20130915201457_create_generic_host_bootdisk_template.rb

@@ -1,4 +1,4 @@
-class CreateGenericHostBootdiskTemplate < ActiveRecord::Migration
+class CreateGenericHostBootdiskTemplate < ActiveRecord::Migration[4.2]
   def self.up
   end
 

data/db/migrate/20131021095100_edit_host_bootdisk_template_dns_secondary.rb

@@ -1,4 +1,4 @@
-class EditHostBootdiskTemplateDnsSecondary < ActiveRecord::Migration
+class EditHostBootdiskTemplateDnsSecondary < ActiveRecord::Migration[4.2]
   def self.up
   end
 

data/db/migrate/20140522185700_change_templatekind_to_bootdisk.rb

@@ -1,5 +1,5 @@
-class ChangeTemplatekindToBootdisk < ActiveRecord::Migration
-  class FakeConfigTemplate < ActiveRecord::Base
+class ChangeTemplatekindToBootdisk < ActiveRecord::Migration[4.2]
+  class FakeConfigTemplate < ApplicationRecord
     if ActiveRecord::Migrator.get_all_versions.include?(20150514072626)
       self.table_name = 'templates'
     else

data/db/migrate/20171009225200_remove_duplicate_bootdisk_templates.rb

@@ -1,4 +1,4 @@
-class RemoveDuplicateBootdiskTemplates < ActiveRecord::Migration
+class RemoveDuplicateBootdiskTemplates < ActiveRecord::Migration[4.2]
   def up
     template_names = ['Boot disk iPXE - host', 'Boot disk iPXE - generic host']
     template_names.each do |template_name|

data/lib/foreman_bootdisk/engine.rb

@@ -48,7 +48,7 @@ module ForemanBootdisk
 
     initializer 'foreman_bootdisk.register_plugin', :before => :finisher_hook do |app|
       Foreman::Plugin.register :foreman_bootdisk do
-        requires_foreman '>= 1.15'
+        requires_foreman '>= 1.17'
 
         security_block :bootdisk do |map|
           permission :download_bootdisk, {:'foreman_bootdisk/disks' => [:generic, :host, :full_host, :subnet, :help],
@@ -76,11 +76,11 @@ module ForemanBootdisk
 
     config.to_prepare do
       begin
-        Host::Managed.send(:include, ForemanBootdisk::HostExt)
+        Host::Managed.send(:prepend, ForemanBootdisk::HostExt)
         Host::Managed.send(:include, ForemanBootdisk::Orchestration::Compute) if SETTINGS[:unattended]
-        HostsHelper.send(:include, ForemanBootdisk::HostsHelperExt)
-        UnattendedController.send(:include, ForemanBootdisk::UnattendedControllerExt)
-        Foreman::Model::Vmware.send(:include, ForemanBootdisk::ComputeResources::Vmware) if Foreman::Model::Vmware.available?
+        HostsHelper.send(:prepend, ForemanBootdisk::HostsHelperExt)
+        UnattendedController.send(:prepend, ForemanBootdisk::UnattendedControllerExt)
+        Foreman::Model::Vmware.send(:prepend, ForemanBootdisk::ComputeResources::Vmware) if Foreman::Model::Vmware.available?
       rescue => e
         puts "#{ForemanBootdisk::ENGINE_NAME}: skipping engine hook (#{e.to_s})"
       end

data/lib/foreman_bootdisk/version.rb

@@ -1,3 +1,3 @@
 module ForemanBootdisk
-  VERSION = '10.0.2'
+  VERSION = '11.0.0'
 end

data/test/functional/foreman_bootdisk/api/v2/disks_controller_test.rb

@@ -7,7 +7,7 @@ class ForemanBootdisk::Api::V2::DisksControllerTest < ActionController::TestCase
   test "should generate generic image" do
     ForemanBootdisk::ISOGenerator.expects(:generate).with(has_entry(:ipxe => regexp_matches(/disk generic host template/))).yields("temp ISO")
     @controller.expects(:read_file).with("temp ISO").returns("ISO image")
-    get :generic, {}
+    get :generic
     assert_response :success
     assert_equal "ISO image", @response.body
   end
@@ -19,7 +19,7 @@ class ForemanBootdisk::Api::V2::DisksControllerTest < ActionController::TestCase
     test "should generate host image" do
       ForemanBootdisk::ISOGenerator.expects(:generate).with(has_entry(:ipxe => regexp_matches(/disk host template/))).yields("temp ISO")
       @controller.expects(:read_file).with("temp ISO").returns("ISO image")
-      get :host, {:id => @host.name}
+      get :host, params: {:id => @host.name}
       assert_response :success
       assert_equal "ISO image", @response.body
     end
@@ -27,7 +27,7 @@ class ForemanBootdisk::Api::V2::DisksControllerTest < ActionController::TestCase
     test "should generate full host image" do
       ForemanBootdisk::ISOGenerator.expects(:generate_full_host).with(@host).yields("temp ISO")
       @controller.expects(:read_file).with("temp ISO").returns("ISO image")
-      get :host, {:id => @host.name, :full => true}
+      get :host, params: {:id => @host.name, :full => true}
       assert_response :success
       assert_equal "ISO image", @response.body
     end

data/test/functional/foreman_bootdisk/api/v2/subnet_disks_controller_test.rb

@@ -14,7 +14,7 @@ class ForemanBootdisk::Api::V2::SubnetDisksControllerTest < ActionController::Te
       ForemanBootdisk::ISOGenerator.expects(:generate).with(has_entry(:ipxe => regexp_matches(/disk generic host template/))).yields("temp ISO")
       ForemanBootdisk::Renderer.any_instance.stubs(:bootdisk_chain_url).yields("http://smart-proxy.lan")
       File.expects(:read).with("temp ISO").returns("ISO image")
-      get :subnet, {:id => @host.subnet.id}
+      get :subnet, params: {:id => @host.subnet.id}
       assert_response :success
       assert_equal "ISO image", @response.body
     end

data/test/functional/foreman_bootdisk/disks_controller_test.rb

@@ -7,7 +7,7 @@ class ForemanBootdisk::DisksControllerTest < ActionController::TestCase
   test "should generate generic image" do
     ForemanBootdisk::ISOGenerator.expects(:generate).with(has_entry(:ipxe => regexp_matches(/disk generic host template/))).yields("temp ISO")
     @controller.expects(:read_file).with("temp ISO").returns("ISO image")
-    get :generic, {}, set_session_user
+    get :generic, session: set_session_user
     assert_response :success
     assert_equal "ISO image", @response.body
   end
@@ -21,7 +21,7 @@ class ForemanBootdisk::DisksControllerTest < ActionController::TestCase
     test "should generate host image" do
       ForemanBootdisk::ISOGenerator.expects(:generate).with(has_entry(:ipxe => regexp_matches(/disk host template/))).yields("temp ISO")
       @controller.expects(:read_file).with("temp ISO").returns("ISO image")
-      get :host, {:id => @host.name}, set_session_user
+      get :host, params: {:id => @host.name}, session: set_session_user
       assert_response :success
       assert_equal "ISO image", @response.body
     end
@@ -29,7 +29,7 @@ class ForemanBootdisk::DisksControllerTest < ActionController::TestCase
     test "should generate full host image" do
       ForemanBootdisk::ISOGenerator.expects(:generate_full_host).with(@host).yields("temp ISO")
       @controller.expects(:read_file).with("temp ISO").returns("ISO image")
-      get :full_host, {:id => @host.name}, set_session_user
+      get :full_host, params: {:id => @host.name}, session: set_session_user
       assert_response :success
       assert_equal "ISO image", @response.body
     end
@@ -38,7 +38,7 @@ class ForemanBootdisk::DisksControllerTest < ActionController::TestCase
       ForemanBootdisk::ISOGenerator.expects(:generate).with(has_entry(:ipxe => regexp_matches(/disk generic host template/))).yields("temp ISO")
       ForemanBootdisk::Renderer.any_instance.stubs(:bootdisk_chain_url).yields("http://smart-proxy.lan")
       @controller.expects(:read_file).with("temp ISO").returns("ISO image")
-      get :subnet, {:id => @host.name}, set_session_user
+      get :subnet, params: {:id => @host.name}, session: set_session_user
       assert_empty flash[:error]
       assert_response :success
       assert_equal "ISO image", @response.body
@@ -52,14 +52,14 @@ class ForemanBootdisk::DisksControllerTest < ActionController::TestCase
     setup :setup_host
 
     test "should not generate subnet image" do
-      get :subnet, {:id => @host.name}, set_session_user
+      get :subnet, params: {:id => @host.name}, session: set_session_user
       assert_match(/Failed.*: TFTP feature not enabled/, flash[:error])
       assert_response :redirect
     end
   end
 
   test "should render help" do
-    get :help, {}, set_session_user
+    get :help, session: set_session_user
     assert_response :success
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_bootdisk
 version: !ruby/object:Gem::Version
-  version: 10.0.2
+  version: 11.0.0
 platform: ruby
 authors:
 - Dominic Cleal
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-08 00:00:00.000000000 Z
+date: 2018-01-15 00:00:00.000000000 Z
 dependencies: []
 description: Plugin for Foreman that creates iPXE-based boot disks to provision hosts
   without the need for PXE infrastructure.