Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Commit

Permalink
Merge pull request #338 from dmcassel/issue-335
Browse files Browse the repository at this point in the history
#335 adding 'raise e' so that exit statuses will be returned
  • Loading branch information
paxtonhare committed Jan 20, 2015
2 parents ace44fc + 3c2a233 commit b962ba6
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions deploy/lib/ml.rb
Original file line number Diff line number Diff line change
Expand Up @@ -156,25 +156,32 @@ def need_help?
case e.response
when Net::HTTPUnauthorized then
@logger.error "Invalid login credentials for #{@properties["environment"]} environment!!"
raise e
else
@logger.error e
@logger.error e.response.body
raise e
end
rescue Net::HTTPFatalError => e
@logger.error e
@logger.error e.response.body
raise e
rescue DanglingVarsException => e
@logger.error "WARNING: The following configuration variables could not be validated:"
e.vars.each do |k,v|
@logger.error "#{k}=#{v}"
end
raise e
rescue HelpException => e
Help.doHelp(@logger, e.command, e.message)
raise e
rescue ExitException => e
@logger.error e
raise e
rescue Exception => e
@logger.error e
@logger.error e.backtrace
raise e
end

if @profile then
Expand Down

0 comments on commit b962ba6

Please sign in to comment.