-
-
Save donnoman/251078 to your computer and use it in GitHub Desktop.
# Allows Tasks that have no servers to be skipped instead of raising a NoMatchingServersError | |
module Capistrano | |
class Configuration | |
module Connections | |
def execute_on_servers(options={}) | |
raise ArgumentError, "expected a block" unless block_given? | |
if task = current_task | |
servers = find_servers_for_task(task, options) | |
if servers.empty? | |
#raise Capistrano::NoMatchingServersError, "`#{task.fully_qualified_name}' is only run for servers matching #{task.options.inspect}, but no servers matched" | |
logger.info "skipping `#{task.fully_qualified_name}' because no servers matched" | |
return | |
end | |
if task.continue_on_error? | |
servers.delete_if { |s| has_failed?(s) } | |
return if servers.empty? | |
end | |
else | |
servers = find_servers(options) | |
raise Capistrano::NoMatchingServersError, "no servers found to match #{options.inspect}" if servers.empty? | |
end | |
servers = [servers.first] if options[:once] | |
logger.trace "servers: #{servers.map { |s| s.host }.inspect}" | |
max_hosts = (options[:max_hosts] || (task && task.max_hosts) || servers.size).to_i | |
is_subset = max_hosts < servers.size | |
# establish connections to those servers in groups of max_hosts, as necessary | |
servers.each_slice(max_hosts) do |servers_slice| | |
begin | |
establish_connections_to(servers_slice) | |
rescue ConnectionError => error | |
raise error unless task && task.continue_on_error? | |
error.hosts.each do |h| | |
servers_slice.delete(h) | |
failed!(h) | |
end | |
end | |
begin | |
yield servers_slice | |
rescue RemoteError => error | |
raise error unless task && task.continue_on_error? | |
error.hosts.each { |h| failed!(h) } | |
end | |
# if dealing with a subset (e.g., :max_hosts is less than the | |
# number of servers available) teardown the subset of connections | |
# that were just made, so that we can make room for the next subset. | |
teardown_connections_to(servers_slice) if is_subset | |
end | |
end | |
end | |
end | |
end |
Yes I had, I think it was submitted to whatever was used prior to github, I had a branch and even added tests for it.
donnoman/capistrano@skip_empty_servers...master
Your welcome to pursue it, I tried several times to get it included, so now I just patch my own installations and move on.
I'll second this initiative. We deploy a shared codebase to servers running in different roles. In some environments (e.g. development sandboxes where we won't have an entire server deployment configured) cap failing when there are no servers matching is prohibitive.
Did you submit a pull request for your patch? Would love to see the option in there.
I haven't had time to write a patch but I opened a ticket on github: capistrano/capistrano#51
An update for people who come from Google (like me), a patch to capistrano/capistrano#51 was accepted in August (cap 2.7.0). That means you can now do something like the following in your tasks:
namespace :unicorn
task :deploy, :roles => :unicorn, :on_no_matching_servers => :continue do
# Task definition goes here
end
end
after "deploy:restart", "unicorn:deploy"
@metavida Thanks for the followup, 1 year later !
@metavida Thank you!! Very, very helpful :)
@metavida Thanks you, that resolves my big issue
Has this been logged as an issue and likely to be contributed as a patch to the capistrano codebase? I wasn't able to find anything in the github project.
Cheers
L