Skip to content

Commit

Permalink
removing unused action remove_link
Browse files Browse the repository at this point in the history
  • Loading branch information
chasebolt committed Apr 7, 2016
1 parent 06722d4 commit e4b7b1e
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 18 deletions.
9 changes: 0 additions & 9 deletions libraries/docker_container.rb
Original file line number Diff line number Diff line change
Expand Up @@ -371,15 +371,6 @@ def parsed_hostname
call_action(:delete)
end

action :remove_link do
# Help! I couldn't get this working from the CLI in docker 1.6.2.
# It's of dubious usefulness, and it looks like this stuff is
# changing in 1.7.x anyway.
converge_by "removing links for #{container_name}" do
Chef::Log.info(':remove_link not currently implemented')
end
end

action :commit do
converge_by "committing #{container_name}" do
with_retries do
Expand Down
4 changes: 0 additions & 4 deletions libraries/matchers.rb
Original file line number Diff line number Diff line change
Expand Up @@ -256,10 +256,6 @@ def remove_docker_container(resource_name)
ChefSpec::Matchers::ResourceMatcher.new(:docker_container, :remove, resource_name)
end

def remove_link_docker_container(resource_name)
ChefSpec::Matchers::ResourceMatcher.new(:docker_container, :remove_link, resource_name)
end

def commit_docker_container(resource_name)
ChefSpec::Matchers::ResourceMatcher.new(:docker_container, :commit, resource_name)
end
Expand Down
5 changes: 0 additions & 5 deletions test/cookbooks/docker_test/recipes/container.rb
Original file line number Diff line number Diff line change
Expand Up @@ -651,11 +651,6 @@
action :run_if_missing
end

file '/marker_container_remover' do
notifies :remove_link, 'docker_container[another_link_target]', :immediately
action :create
end

################
# volume removal
################
Expand Down

0 comments on commit e4b7b1e

Please sign in to comment.