Save error status if err data is present in dhcp checker response
Change-Id: I0e109e28b292c95bed25aa55302c7200a48e7fd0 Related-Bug: #1569339
This commit is contained in:
parent
882847a7e2
commit
d6a40e0590
|
@ -223,7 +223,8 @@ module Astute
|
|||
if response.results[:data][:out].present?
|
||||
Astute.logger.debug("DHCP checker received:\n#{response.pretty_inspect}")
|
||||
node_result[:data] = JSON.parse(response.results[:data][:out])
|
||||
elsif response.results[:data][:err].present?
|
||||
end
|
||||
if response.results[:data][:err].present?
|
||||
Astute.logger.debug("DHCP checker errred with:\n#{response.pretty_inspect}")
|
||||
node_result[:status] = 'error'
|
||||
node_result[:error_msg] = 'Error in dhcp checker. Check logs for details'
|
||||
|
|
|
@ -137,8 +137,9 @@ describe Astute::Network do
|
|||
mock_and_verify_check_dhcp(nodes, [res1, res2], expected)
|
||||
|
||||
# check case when the check failed for one of the nodes
|
||||
err_res = mock_mc_result({:sender => "1", :data => {:err => 'Test err'}})
|
||||
expected = {"nodes" => [{:uid => "1", :status => "error",
|
||||
err_res = mock_mc_result({:sender => "1",
|
||||
:data => {:out => json_output, :err => 'Test err'}})
|
||||
expected = {"nodes" => [{:uid => "1", :status => "error", :data=>expected_data,
|
||||
:error_msg => "Error in dhcp checker. Check logs for details"}],
|
||||
"status" => "error"}
|
||||
mock_and_verify_check_dhcp([nodes[0]], [err_res], expected)
|
||||
|
|
Loading…
Reference in New Issue