tim/fluke
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

CID-470: fix error and nice output

Browse Source
This commit is contained in:
Anton Chuchkalov 2016-03-17 18:32:44 +02:00
parent aec662ca83
commit 5fcd74c17c
2 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
devops-service/lib/executors/server_executor.rb
View File

@ -436,7 +436,7 @@ module Devops
end
def delete_from_chef_server node_name
return if node_name.blank?
return {chef_node: nil, chef_client: nil} if node_name.blank?
{
:chef_node => knife_instance.chef_node_delete(node_name),
:chef_client => knife_instance.chef_client_delete(node_name)
@ -454,10 +454,14 @@ module Devops
return 0
end
if @server.chef_node_name.present?
puts_and_flush "Deleting from chef server:"
delete_from_chef_server(@server.chef_node_name).each do |key, result|
@out.puts "#{key} - #{result}"
end
else
puts_and_flush "Chef node name is empty, skipping deleting from chef server."
end
puts_and_flush "Deleting from cloud:"
provider = @server.provider_instance

2
devops-service/spec/executors/server_executor_spec.rb
View File

@ -667,7 +667,9 @@ RSpec.describe Devops::Executor::ServerExecutor, type: :executor, stubbed_connec
it "doesn't call knife if chef node name is empty" do
expect(stubbed_knife).not_to receive(:chef_client_delete)
expect(stubbed_knife).not_to receive(:chef_node_delete)
expect(
executor.delete_from_chef_server('')
).to be_a(Hash).and include(:chef_node, :chef_client)
end
end