Unbootstrap: delete node from chef server only if /etc/chef directory was successfully moved
This commit is contained in:
parent
c016d777e2
commit
57f5bf0f2e
@ -213,7 +213,10 @@ module Devops
|
|||||||
DevopsLogger.logger.info msg
|
DevopsLogger.logger.info msg
|
||||||
r
|
r
|
||||||
else
|
else
|
||||||
@out << "Can not bootstrap node '#{@server.id}', error code: #{r}"
|
@out << "Can not bootstrap node '#{@server.id}', error code: #{r}\n"
|
||||||
|
if r == 256
|
||||||
|
@out << "Maybe old chef client configuration still exists?\n"
|
||||||
|
end
|
||||||
@out.flush
|
@out.flush
|
||||||
result_code(:server_bootstrap_fail)
|
result_code(:server_bootstrap_fail)
|
||||||
end
|
end
|
||||||
@ -309,7 +312,7 @@ module Devops
|
|||||||
k = Devops::Db.connector.key(@server.key)
|
k = Devops::Db.connector.key(@server.key)
|
||||||
cert_path = k.path
|
cert_path = k.path
|
||||||
i = 0
|
i = 0
|
||||||
res = delete_from_chef_server(@server.chef_node_name)
|
res = {}
|
||||||
begin
|
begin
|
||||||
new_name = "/etc/chef.backup_#{Time.now.strftime("%d-%m-%Y_%H.%M.%S")}"
|
new_name = "/etc/chef.backup_#{Time.now.strftime("%d-%m-%Y_%H.%M.%S")}"
|
||||||
# r = `ssh -i #{cert_path} -q #{@server.remote_user}@#{@server.private_ip} rm -Rf /etc/chef`
|
# r = `ssh -i #{cert_path} -q #{@server.remote_user}@#{@server.private_ip} rm -Rf /etc/chef`
|
||||||
@ -329,7 +332,9 @@ module Devops
|
|||||||
retry unless i == 5
|
retry unless i == 5
|
||||||
return {error: e.message}
|
return {error: e.message}
|
||||||
end
|
end
|
||||||
res
|
|
||||||
|
deleting_chef_node_and_chef_client_result = delete_from_chef_server(@server.chef_node_name)
|
||||||
|
res.merge!(deleting_chef_node_and_chef_client_result)
|
||||||
end
|
end
|
||||||
|
|
||||||
def deploy_server_with_tags tags, deploy_info
|
def deploy_server_with_tags tags, deploy_info
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user