foreman_dlm 0.1.0 → 2.0.0

data/test/models/foreman_dlm/dlm_facet_test.rb

@@ -0,0 +1,13 @@
+require 'test_plugin_helper'
+
+module ForemanDlm
+  class DlmFacetTest < ActiveSupport::TestCase
+    setup do
+      User.current = users(:admin)
+    end
+
+    subject { FactoryBot.create(:dlm_facet) }
+    should belong_to(:host)
+    should validate_presence_of(:host)
+  end
+end

data/test/models/foreman_dlm/dlmlock_event_test.rb

@@ -0,0 +1,19 @@
+require 'test_plugin_helper'
+
+module ForemanDlm
+  class DlmlockEventTest < ActiveSupport::TestCase
+    should belong_to(:dlmlock)
+    should belong_to(:host)
+    should belong_to(:user)
+
+    test 'should expire lock events' do
+      event_count = 5
+      FactoryBot.create_list(:dlmlock_event, event_count)
+      FactoryBot.create_list(:dlmlock_event, event_count, :old_event)
+      assert_difference(-> { ForemanDlm::DlmlockEvent.count }, -1 * event_count) do
+        ForemanDlm::DlmlockEvent.expire(created_before: 7.days, batch_size: 2, sleep_time: 0.0001)
+      end
+      assert_equal event_count, ForemanDlm::DlmlockEvent.count
+    end
+  end
+end

data/test/models/foreman_dlm/dlmlock_test.rb

@@ -0,0 +1,307 @@
+require 'test_plugin_helper'
+
+module ForemanDlm
+  class DlmlockTest < ActiveSupport::TestCase
+    setup do
+      User.current = users(:admin)
+    end
+
+    should belong_to(:host)
+    should have_many(:dlmlock_events)
+
+    subject { FactoryBot.create(:dlmlock) }
+    should validate_presence_of(:name)
+    should validate_uniqueness_of(:name)
+
+    let(:host1) { FactoryBot.create(:host, :managed) }
+    let(:host2) { FactoryBot.create(:host, :managed) }
+
+    class HostWithCallbacks < ::Host::Managed
+      attr_accessor :callbacks
+
+      def initialize(*attributes, &block)
+        super
+        @callbacks = []
+      end
+
+      after_lock :callback1
+      after_unlock :callback2
+
+      def callback1
+        Rails.logger.debug "callback1 executed for #{self} (#{self.class})"
+        callbacks << 'callback1'
+      end
+
+      def callback2
+        Rails.logger.debug "callback2 executed for #{self} (#{self.class})"
+        callbacks << 'callback2'
+      end
+    end
+
+    let(:host1_with_callbacks) { HostWithCallbacks.create(:name => 'test1.example.com') }
+    let(:host2_with_callbacks) { HostWithCallbacks.create(:name => 'test2.example.com') }
+
+    context 'a free and enabled DLM lock' do
+      let(:dlmlock) { FactoryBot.create(:dlmlock) }
+
+      test 'should be enabled and unlocked' do
+        assert_equal true, dlmlock.enabled?
+        assert_equal false, dlmlock.disabled?
+        assert_equal false, dlmlock.locked?
+        assert_equal false, dlmlock.taken?
+      end
+
+      test 'can be acquired' do
+        assert_nil dlmlock.host
+        assert dlmlock.acquire!(host1)
+        assert_equal host1, dlmlock.reload.host
+      end
+
+      test 'can be released' do
+        assert_nil dlmlock.host
+        assert dlmlock.release!(host1)
+        assert_nil dlmlock.reload.host
+      end
+
+      test 'creates a dlmlock_event on acquisition by owner' do
+        assert_difference -> { DlmlockEvent.count }, 1 do
+          assert dlmlock.acquire!(host1)
+        end
+
+        event = DlmlockEvent.last
+        assert_equal 'acquire', event.event_type
+        assert_equal host1.id, event.host_id
+        assert_equal users(:admin).id, event.user_id
+        assert_equal dlmlock.id, event.dlmlock_id
+      end
+
+      test 'triggers after_lock callback' do
+        host = HostWithCallbacks.new
+        host.name = 'test.example.com'
+        host.save
+        assert dlmlock.acquire!(host)
+        assert_equal ['callback1'], host.callbacks
+      end
+
+      context 'with a parameter disabeling the locking' do
+        setup do
+          FactoryBot.create(:host_parameter, host: host1, name: 'can_acquire_update_locks', value: 'false')
+        end
+
+        test 'can not be acquired' do
+          assert_nil dlmlock.host
+          assert_not dlmlock.acquire!(host1)
+          assert_nil dlmlock.reload.host
+        end
+      end
+    end
+
+    context 'a free and disabled DLM lock' do
+      let(:dlmlock) { FactoryBot.create(:dlmlock, :enabled => false) }
+
+      test 'should be disabled and unlocked' do
+        assert_equal false, dlmlock.enabled?
+        assert_equal true, dlmlock.disabled?
+        assert_equal false, dlmlock.locked?
+        assert_equal false, dlmlock.taken?
+      end
+
+      test 'can not be acquired' do
+        assert_nil dlmlock.host
+        assert_equal false, dlmlock.acquire!(host1)
+        assert_nil dlmlock.reload.host
+      end
+
+      test 'can not be released' do
+        assert_nil dlmlock.host
+        assert_equal false, dlmlock.release!(host1)
+        assert_nil dlmlock.reload.host
+      end
+
+      test 'triggers no callbacks' do
+        host = HostWithCallbacks.new
+        host.name = 'test.example.com'
+        host.save
+        assert_equal false, dlmlock.release!(host)
+        assert_equal [], host.callbacks
+      end
+    end
+
+    context 'an acquired DLM lock' do
+      let(:dlmlock) { FactoryBot.create(:dlmlock, :host => host1) }
+
+      test 'should be enabled and locked' do
+        assert_equal true, dlmlock.enabled?
+        assert_equal false, dlmlock.disabled?
+        assert_equal true, dlmlock.locked?
+        assert_equal true, dlmlock.taken?
+        assert_equal true, dlmlock.locked_by?(host1)
+        assert_equal true, dlmlock.acquired_by?(host1)
+      end
+
+      test 'can be acquired by owner' do
+        assert_equal host1, dlmlock.host
+        assert dlmlock.acquire!(host1)
+        assert_equal host1, dlmlock.reload.host
+      end
+
+      test 'can not be acquired by other host' do
+        assert_equal host1, dlmlock.host
+        assert_equal false, dlmlock.acquire!(host2)
+        assert_equal host1, dlmlock.reload.host
+      end
+
+      test 'can be released by owner' do
+        assert_equal host1, dlmlock.host
+        assert dlmlock.release!(host1)
+        assert_nil dlmlock.reload.host
+      end
+
+      test 'can not be released by other host' do
+        assert_equal host1, dlmlock.host
+        assert_equal false, dlmlock.release!(host2)
+        assert_equal host1, dlmlock.reload.host
+      end
+
+      test 'records audit change on release by owner' do
+        dlmlock
+        assert_difference -> { DlmlockEvent.count }, 1 do
+          assert dlmlock.release!(host1)
+        end
+
+        event = DlmlockEvent.last
+        assert_equal 'release', event.event_type
+        assert_equal host1.id, event.host_id
+        assert_equal users(:admin).id, event.user_id
+        assert_equal dlmlock.id, event.dlmlock_id
+      end
+
+      test 'records no audit change on acquisition by owner' do
+        assert_no_difference "Audit.where(auditable_type: 'ForemanDlm::Dlmlock', action: 'update').count" do
+          assert dlmlock.acquire!(host1)
+        end
+      end
+
+      test 'triggers after_unlock callback on release by owner' do
+        host = HostWithCallbacks.new
+        host.name = 'test.example.com'
+        host.save
+        dlmlock.host = host
+        dlmlock.save
+        assert dlmlock.release!(host)
+        assert_equal ['callback2'], host.callbacks
+      end
+
+      test 'triggers no callbacks on release attempt by other host' do
+        assert host1_with_callbacks
+        assert host2_with_callbacks
+        dlmlock.update(:host => host1_with_callbacks)
+        assert_equal false, dlmlock.release!(host2_with_callbacks)
+        assert_equal [], host1_with_callbacks.callbacks
+        assert_equal [], host2_with_callbacks.callbacks
+      end
+
+      test 'triggers no callbacks on acquiry attempt by owner' do
+        assert host1_with_callbacks
+        dlmlock.update(:host => host1_with_callbacks)
+        assert dlmlock.acquire!(host1_with_callbacks)
+        assert_equal [], host1_with_callbacks.callbacks
+      end
+    end
+
+    context 'scoped search' do
+      test 'can be searched by name' do
+        dlmlock = FactoryBot.create(:dlmlock)
+        assert_equal Dlmlock::Update.find(dlmlock.id), Dlmlock.search_for("name ~ #{dlmlock.name}").first
+      end
+    end
+
+    describe '#locked' do
+      subject { Dlmlock.locked }
+
+      it 'includes only Distributed Locks that are locked' do
+        locked = FactoryBot.create(:dlmlock, :locked)
+        not_locked = FactoryBot.create(:dlmlock)
+
+        assert_includes subject, locked
+        assert_not_includes subject, not_locked
+      end
+    end
+
+    describe '#stale' do
+      subject { Dlmlock.stale }
+
+      it 'includes only Distributed Locks that are stale' do
+        now = Time.now.utc
+
+        travel_to now do
+          stale = FactoryBot.create(:dlmlock, :locked, updated_at: now - 5.hours)
+          not_stale = FactoryBot.create(:dlmlock, :locked, updated_at: now)
+          not_locked = FactoryBot.create(:dlmlock)
+
+          assert_includes subject, stale
+          assert_not_includes subject, not_stale
+          assert_not_includes subject, not_locked
+        end
+      end
+    end
+
+    context '#log_events' do
+      let(:dlmlock) { FactoryBot.create(:dlmlock) }
+
+      test 'logs acquire event' do
+        assert_difference -> { DlmlockEvent.count }, 1 do
+          dlmlock.acquire!(host1)
+        end
+
+        event = DlmlockEvent.last
+        assert_equal 'acquire', event.event_type
+        assert_equal host1, event.host
+      end
+
+      test 'logs disable event' do
+        assert_difference -> { DlmlockEvent.count }, 1 do
+          dlmlock.disable!
+        end
+
+        event = DlmlockEvent.last
+        assert_equal 'disable', event.event_type
+      end
+
+      test 'logs enable event' do
+        dlmlock.update(enabled: false)
+
+        assert_difference -> { DlmlockEvent.count }, 1 do
+          dlmlock.enable!
+        end
+
+        event = DlmlockEvent.last
+        assert_equal 'enable', event.event_type
+      end
+
+      test 'logs failed event' do
+        dlmlock.acquire!(host1)
+
+        assert_difference -> { DlmlockEvent.count }, 1 do
+          dlmlock.acquire!(host2)
+        end
+
+        event = DlmlockEvent.last
+        assert_equal 'fail', event.event_type
+        assert_equal host1, event.host
+      end
+
+      test 'logs release event' do
+        dlmlock.acquire!(host1)
+
+        assert_difference -> { DlmlockEvent.count }, 1 do
+          assert dlmlock.release!(host1)
+        end
+
+        event = DlmlockEvent.last
+        assert_equal 'release', event.event_type
+        assert_equal host1, event.host
+      end
+    end
+  end
+end

data/test/models/host_managed_test.rb

@@ -0,0 +1,41 @@
+require 'test_plugin_helper'
+
+module Host
+  class ManagedTest < ActiveSupport::TestCase
+    should have_many(:dlmlocks)
+    should have_many(:dlmlock_events)
+    should have_one(:dlm_facet)
+
+    describe '#can_acquire_update_locks?' do
+      let(:host) { FactoryBot.create(:host, :managed) }
+
+      it 'should be true without a host parameter' do
+        assert host.can_acquire_update_locks?
+      end
+
+      it 'should be true if parameter is true' do
+        FactoryBot.create(:host_parameter, host: host, name: 'can_acquire_update_locks', value: 'true')
+        assert host.can_acquire_update_locks?
+      end
+
+      it 'should be false if parameter is false' do
+        FactoryBot.create(:host_parameter, host: host, name: 'can_acquire_update_locks', value: 'false')
+        assert_not host.can_acquire_update_locks?
+      end
+    end
+
+    context 'scoped search on' do
+      context 'a host' do
+        let(:host) { FactoryBot.create(:host, :with_dlm_facet) }
+        setup do
+          host
+        end
+
+        test 'can be searched by dlm checkin time' do
+          results = Host.search_for('last_dlm_checkin_at < "2 weeks from now"')
+          assert_includes results, host
+        end
+      end
+    end
+  end
+end

data/test/models/host_monitoring_test.rb

@@ -27,7 +27,7 @@ class HostMonitoringTest < ActiveSupport::TestCase
   end
 
   context 'an acquired DLM lock' do
-   let(:dlmlock) { FactoryBot.create(:dlmlock, :host => host) }
+    let(:dlmlock) { FactoryBot.create(:dlmlock, :host => host) }
 
     test 'removes monitoring dowmtine' do
       monitoring_mock.expects(:del_downtime_host).once

data/test/models/host_status/dlmlock_status_test.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+class DlmlockStatusTest < ActiveSupport::TestCase
+  let(:host) { FactoryBot.create(:host, :managed) }
+
+  describe '#to_status' do
+    test 'should return STALE if host has any stale dlmlocks' do
+      now = Time.now.utc
+
+      travel_to now do
+        FactoryBot.create(:dlmlock, :locked, host: host, updated_at: now - 5.hours)
+        status = host.get_status(HostStatus::DlmlockStatus)
+
+        assert_not_empty host.dlmlocks.stale
+        assert_equal HostStatus::DlmlockStatus::STALE, status.to_status
+      end
+    end
+
+    test 'should return OK if host has no stale dlmlocks' do
+      status = host.get_status(HostStatus::DlmlockStatus)
+
+      assert_empty host.dlmlocks.stale
+      assert_equal HostStatus::DlmlockStatus::OK, status.to_status
+    end
+  end
+
+  describe '#relevant?' do
+    test 'should return true if host has any dlmlocks' do
+      FactoryBot.create(:dlmlock, host: host)
+      status = host.get_status(HostStatus::DlmlockStatus)
+
+      assert_not_empty host.dlmlocks
+      assert_equal true, status.relevant?
+    end
+
+    test 'should return false if host has no dlmlocks' do
+      status = host.get_status(HostStatus::DlmlockStatus)
+
+      assert_empty host.dlmlocks
+      assert_equal false, status.relevant?
+    end
+  end
+end

data/test/models/user_test.rb

@@ -0,0 +1,5 @@
+require 'test_plugin_helper'
+
+class UserTest < ActiveSupport::TestCase
+  should have_many(:dlmlock_events)
+end

data/test/test_plugin_helper.rb

@@ -3,7 +3,9 @@ require 'test_helper'
 
 # Add plugin to FactoryBot's paths
 FactoryBot.definition_file_paths << File.join(File.dirname(__FILE__), 'factories')
-if ForemanDlm.with_monitoring?
-  FactoryBot.definition_file_paths << "#{ForemanMonitoring::Engine.root}/test/factories"
-end
+
+# Add factories of external plugin dependencies
+FactoryBot.definition_file_paths << "#{ForemanMonitoring::Engine.root}/test/factories" if ForemanDlm.with_monitoring?
+
+# Reload factory paths
 FactoryBot.reload

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: foreman_dlm
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 2.0.0
 platform: ruby
 authors:
 - Timo Goebel
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-04-06 00:00:00.000000000 Z
+date: 2022-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rdoc
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -25,19 +25,47 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rdoc
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.25.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.25.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.13.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.13.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.9.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.9.1
 description: Adds a Distributed Lock Manager to Foreman. This enables painless system
   updates for clusters.
 email:
@@ -49,16 +77,27 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- app/controllers/api/v2/dlmlock_events_controller.rb
 - app/controllers/api/v2/dlmlocks_controller.rb
 - app/controllers/concerns/foreman/controller/parameters/dlmlocks.rb
 - app/controllers/concerns/foreman_dlm/find_host_by_client_cert.rb
 - app/controllers/concerns/foreman_dlm/find_host_by_ip.rb
-- app/controllers/dlmlocks_controller.rb
+- app/controllers/concerns/foreman_dlm/update_checkin_time.rb
+- app/controllers/foreman_dlm/application_controller.rb
+- app/controllers/foreman_dlm/dlmlocks_controller.rb
 - app/helpers/foreman_dlm/dlmlock_helper.rb
+- app/jobs/foreman_dlm/refresh_dlmlock_status.rb
+- app/models/concerns/foreman_dlm/dlm_facet_host_extensions.rb
+- app/models/concerns/foreman_dlm/expirable.rb
 - app/models/concerns/foreman_dlm/host_extensions.rb
 - app/models/concerns/foreman_dlm/host_monitoring_extensions.rb
-- app/models/dlmlock.rb
-- app/models/dlmlock/update.rb
+- app/models/concerns/foreman_dlm/user_extensions.rb
+- app/models/foreman_dlm/dlm_facet.rb
+- app/models/foreman_dlm/dlmlock.rb
+- app/models/foreman_dlm/dlmlock/update.rb
+- app/models/foreman_dlm/dlmlock_event.rb
+- app/models/host_status/dlmlock_status.rb
+- app/views/api/v2/dlmlock_events/index.json.rabl
 - app/views/api/v2/dlmlocks/acquire.json.rabl
 - app/views/api/v2/dlmlocks/base.json.rabl
 - app/views/api/v2/dlmlocks/create.json.rabl
@@ -68,33 +107,60 @@ files:
 - app/views/api/v2/dlmlocks/show.json.rabl
 - app/views/api/v2/dlmlocks/update.json.rabl
 - app/views/api/v2/errors/precondition_failed.json.rabl
-- app/views/dlmlocks/_details.html.erb
-- app/views/dlmlocks/_list.html.erb
-- app/views/dlmlocks/index.html.erb
-- app/views/dlmlocks/show.html.erb
-- app/views/dlmlocks/welcome.html.erb
+- app/views/foreman_dlm/api/v2/dlm_facets/base.json.rabl
+- app/views/foreman_dlm/api/v2/dlm_facets/base_with_root.json.rabl
+- app/views/foreman_dlm/api/v2/dlm_facets/show.json.rabl
+- app/views/foreman_dlm/dlmlocks/_details.html.erb
+- app/views/foreman_dlm/dlmlocks/_list.html.erb
+- app/views/foreman_dlm/dlmlocks/index.html.erb
+- app/views/foreman_dlm/dlmlocks/show.html.erb
+- app/views/foreman_dlm/dlmlocks/welcome.html.erb
+- app/views/hosts/_dlmlocks_tab.html.erb
 - config/routes.rb
+- contrib/systemd/foreman-dlm-expire-events.service
+- contrib/systemd/foreman-dlm-expire-events.timer
 - db/migrate/20170824084100_add_dlmlock.foreman_dlm.rb
+- db/migrate/20180627150003_rename_dlmlock_sti_models.rb
+- db/migrate/20180704162345_add_dlmlock_events.rb
+- db/migrate/20180711090022_add_hosts_fk_to_dlmlocks.rb
+- db/migrate/20180711111903_create_dlm_facets.foreman_dlm.rb
+- db/migrate/20180713113208_update_permissions_for_scoped_models.rb
 - lib/foreman_dlm.rb
 - lib/foreman_dlm/engine.rb
 - lib/foreman_dlm/version.rb
+- lib/tasks/dlmlock_events.rake
 - lib/tasks/foreman_dlm_tasks.rake
 - locale/Makefile
 - locale/en/foreman_dlm.po
 - locale/foreman_dlm.pot
 - locale/gemspec.rb
 - test/controllers/api/v2/dlmlocks_controller_test.rb
-- test/controllers/dlmlocks_test.rb
+- test/controllers/api/v2/dlmlocks_dlmlock_events_controller_test.rb
+- test/controllers/api/v2/hosts_controller_test.rb
 - test/controllers/find_host_by_client_cert_test.rb
+- test/controllers/foreman_dlm/dlmlocks_test.rb
+- test/controllers/hosts_controller_test.rb
+- test/factories/dlm_facets.rb
 - test/factories/dlmlock.rb
-- test/models/dlmlock_test.rb
+- test/factories/dlmlock_events.rb
+- test/factories/host.rb
+- test/integration/foreman_dlm/dlmlocks_test.rb
+- test/jobs/refresh_dlmlock_status_test.rb
+- test/models/foreman_dlm/dlm_facet_test.rb
+- test/models/foreman_dlm/dlmlock_event_test.rb
+- test/models/foreman_dlm/dlmlock_test.rb
+- test/models/host_managed_test.rb
 - test/models/host_monitoring_test.rb
+- test/models/host_status/dlmlock_status_test.rb
+- test/models/user_test.rb
 - test/test_plugin_helper.rb
-homepage: https://github.com/timogoebel/foreman_dlm
+homepage: https://github.com/dm-drogeriemarkt/foreman_dlm
 licenses:
 - GPL-3.0
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
+  is_foreman_plugin: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -109,16 +175,28 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.3
-signing_key: 
+rubygems_version: 3.3.3
+signing_key:
 specification_version: 4
 summary: Distributed Lock Manager for Foreman.
 test_files:
-- test/models/host_monitoring_test.rb
-- test/models/dlmlock_test.rb
-- test/factories/dlmlock.rb
-- test/test_plugin_helper.rb
-- test/controllers/dlmlocks_test.rb
 - test/controllers/api/v2/dlmlocks_controller_test.rb
+- test/controllers/api/v2/dlmlocks_dlmlock_events_controller_test.rb
+- test/controllers/api/v2/hosts_controller_test.rb
 - test/controllers/find_host_by_client_cert_test.rb
+- test/controllers/foreman_dlm/dlmlocks_test.rb
+- test/controllers/hosts_controller_test.rb
+- test/factories/dlm_facets.rb
+- test/factories/dlmlock.rb
+- test/factories/dlmlock_events.rb
+- test/factories/host.rb
+- test/integration/foreman_dlm/dlmlocks_test.rb
+- test/jobs/refresh_dlmlock_status_test.rb
+- test/models/foreman_dlm/dlm_facet_test.rb
+- test/models/foreman_dlm/dlmlock_event_test.rb
+- test/models/foreman_dlm/dlmlock_test.rb
+- test/models/host_managed_test.rb
+- test/models/host_monitoring_test.rb
+- test/models/host_status/dlmlock_status_test.rb
+- test/models/user_test.rb
+- test/test_plugin_helper.rb

data/app/controllers/dlmlocks_controller.rb

@@ -1,13 +0,0 @@
-class DlmlocksController < ::ApplicationController
-  include Foreman::Controller::AutoCompleteSearch
-
-  before_action :setup_search_options, :only => :index
-  before_action :find_resource, :only => [:show]
-
-  def index
-    @dlmlocks = resource_base_search_and_page(:host)
-  end
-
-  def show
-  end
-end

data/app/models/dlmlock/update.rb

@@ -1,5 +0,0 @@
-class Dlmlock::Update < Dlmlock
-  def humanized_type
-    _('Update Lock')
-  end
-end