Skip to content

Commit

Permalink
separate vm store and datastore scan supports checks
Browse files Browse the repository at this point in the history
  • Loading branch information
nasarak_786@hotmail.com authored and nasark committed Dec 21, 2023
1 parent 999306b commit 4538922
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 12 deletions.
1 change: 1 addition & 0 deletions app/models/storage.rb
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,7 @@ class Storage < ApplicationRecord

supports(:delete) { _("Only storage without VMs and Hosts can be removed") if vms_and_templates.any? || hosts.any? }


def to_s
name
end
Expand Down
16 changes: 4 additions & 12 deletions app/models/vm_scan/dispatcher.rb
Original file line number Diff line number Diff line change
Expand Up @@ -246,18 +246,10 @@ def get_eligible_proxies_for_job(job)
return []
end

if @vm.requires_storage_for_scan?
if @vm.storage.nil?
msg = "Vm [#{@vm.path}] is not located on a storage, aborting job [#{job.guid}]."
queue_signal(job, {:args => [:abort, msg, "error"]})
return []
else
unless @vm.storage.supports?(:smartstate_analysis)
msg = @vm.storage.unsupported_reason(:smartstate_analysis)
queue_signal(job, {:args => [:abort, msg, "error"]})
return []
end
end
unless @vm.supports?(:smartstate_analysis)
msg = @vm.unsupported_reason(:smartstate_analysis)
queue_signal(job, {:args => [:abort, msg, "error"]})
return []
end

vm_proxies, = Benchmark.realtime_block(:get_eligible_proxies_for_job__proxies4job) { @vm.proxies4job(job) }
Expand Down

0 comments on commit 4538922

Please sign in to comment.