centurion 1.3.1 → 1.4.0

data/lib/centurion/version.rb

@@ -1,3 +1,3 @@
 module Centurion
-  VERSION = '1.3.1'
+  VERSION = '1.4.0'
 end

data/lib/tasks/deploy.rake

@@ -49,25 +49,18 @@ namespace :deploy do
         s3_region: fetch(:s3_region) || 'us-east-1',
       )
 
-      # Download image from S3 to local /tmp/directory
-      Dir.mktmpdir("dogestry") do |local_dir|
-
-        info "** Downloading image(#{fetch(:image)}:#{fetch(:tag)}) from S3 to local directory"
-        registry.download_image_to_temp_dir("#{fetch(:image)}:#{fetch(:tag)}", local_dir)
-
-        # Upload image from local /tmp/directory to specified Docker hosts
-        target_servers = Centurion::DockerServerGroup.new(fetch(:hosts), fetch(:docker_path))
-        target_servers.each_in_parallel do |target_server|
-
-          docker_host = "tcp://#{target_server.hostname}:#{target_server.port}"
+      target_servers = Centurion::DockerServerGroup.new(fetch(:hosts), fetch(:docker_path))
+      pull_hosts = []
 
-          image_and_tag = "#{fetch(:image)}:#{fetch(:tag)}"
+      target_servers.each do |target_server|
+        docker_host = "tcp://#{target_server.hostname}:#{target_server.port}"
+        pull_hosts.push(docker_host)
+      end
 
-          info "** Pushing image(#{image_and_tag}) from #{local_dir} to Docker: #{docker_host}"
+      image_and_tag = "#{fetch(:image)}:#{fetch(:tag)}"
+      info "** Pulling image(#{image_and_tag}) from S3 to Docker Hosts: #{pull_hosts}"
 
-          registry.upload_temp_dir_image_to_docker(image_and_tag, local_dir, docker_host)
-        end
-      end
+      registry.pull(image_and_tag, pull_hosts)
     end
   end
 
@@ -187,7 +180,9 @@ namespace :deploy do
     if fetch(:registry) == 'dogestry'
       invoke 'deploy:dogestry:pull_image'
     else
-      target_servers = Centurion::DockerServerGroup.new(fetch(:hosts), fetch(:docker_path))
+      hosts, docker_path = fetch(:hosts, []), fetch(:docker_path)
+      target_servers = Centurion::DockerServerGroup.new(hosts, docker_path,
+                                                        build_tls_params)
       target_servers.each_in_parallel do |target_server|
         target_server.pull(fetch(:image), fetch(:tag))
       end

data/spec/capistrano_dsl_spec.rb

@@ -61,7 +61,7 @@ describe Capistrano::DSL do
       end
 
       DSLTest.set(:foo, NoAny.new)
-      DSLTest.any?(:foo).should eq('oh no')
+      expect(DSLTest.any?(:foo)).to eq('oh no')
     end
   end
 end

data/spec/deploy_dsl_spec.rb

@@ -115,7 +115,7 @@ describe Centurion::DeployDSL do
   end
 
   it 'gets current tags for an image' do
-    Centurion::DockerServer.any_instance.stub(current_tags_for: [ 'foo' ])
+    allow_any_instance_of(Centurion::DockerServer).to receive(:current_tags_for).and_return([ 'foo' ])
     DeployDSLTest.set(:hosts, [ 'host1' ])
 
     expect(DeployDSLTest.get_current_tags_for('asdf')).to eq [ { server: 'host1', tags: [ 'foo'] } ]

data/spec/deploy_spec.rb

@@ -3,9 +3,11 @@ require 'centurion/deploy_dsl'
 require 'centurion/logging'
 
 describe Centurion::Deploy do
-  let(:mock_ok_status)  { double('http_status_ok').tap { |s| s.stub(status: 200) } }
-  let(:mock_bad_status) { double('http_status_ok').tap { |s| s.stub(status: 500) } }
-  let(:server)          { double('docker_server').tap { |s| s.stub(hostname: 'host1'); s.stub(:attach) } }
+  let(:mock_ok_status)  { double('http_status_ok').tap { |s| allow(s).to receive(:status).and_return(200) } }
+  let(:mock_bad_status) { double('http_status_ok').tap { |s| allow(s).to receive(:status).and_return(500) } }
+  let(:server)          { double('docker_server').tap { |s|
+                            allow(s).to receive(:hostname).and_return(hostname)
+                            allow(s).to receive(:attach) } }
   let(:port)            { 8484 }
   let(:container)       { { 'Ports' => [{ 'PublicPort' => port }, 'Created' => Time.now.to_i ], 'Id' => '21adfd2ef2ef2349494a', 'Names' => [ 'name1' ] } }
   let(:endpoint)        { '/status/check' }
@@ -16,6 +18,12 @@ describe Centurion::Deploy do
       o.send(:extend, Centurion::Logging)
     end
   end
+  let(:hostname) { 'host1' }
+
+  before do
+    allow(test_deploy).to receive(:fetch).and_return nil
+    allow(test_deploy).to receive(:fetch).with(:container_hostname, hostname).and_return(hostname)
+  end
 
   describe '#http_status_ok?' do
     it 'validates HTTP status checks when the response is good' do
@@ -25,7 +33,7 @@ describe Centurion::Deploy do
 
     it 'identifies bad HTTP responses' do
       expect(Excon).to receive(:get).and_return(mock_bad_status)
-      test_deploy.stub(:warn)
+      allow(test_deploy).to receive(:warn)
       expect(test_deploy.http_status_ok?(server, port, endpoint)).to be_falsey
     end
 
@@ -46,32 +54,32 @@ describe Centurion::Deploy do
     it 'recognizes when no containers are running' do
       expect(server).to receive(:find_containers_by_public_port).and_return([])
 
-      test_deploy.container_up?(server, port).should be_falsey
+      expect(test_deploy.container_up?(server, port)).to be_falsey
     end
 
     it 'complains when more than one container is bound to this port' do
       expect(server).to receive(:find_containers_by_public_port).and_return([1,2])
       expect(test_deploy).to receive(:error).with /More than one container/
 
-      test_deploy.container_up?(server, port).should be_falsey
+      expect(test_deploy.container_up?(server, port)).to be_falsey
     end
 
     it 'recognizes when the container is actually running' do
       expect(server).to receive(:find_containers_by_public_port).and_return([container])
       expect(test_deploy).to receive(:info).with /Found container/
 
-      test_deploy.container_up?(server, port).should be_truthy
+      expect(test_deploy.container_up?(server, port)).to be_truthy
     end
   end
 
   describe '#wait_for_http_status_ok?' do
     before do
-      test_deploy.stub(:info)
+      allow(test_deploy).to receive(:info)
     end
 
     it 'identifies that a container is up' do
-      test_deploy.stub(:container_up? => true)
-      test_deploy.stub(:http_status_ok? => true)
+      allow(test_deploy).to receive(:container_up?).and_return(true)
+      allow(test_deploy).to receive(:http_status_ok?).and_return(true)
 
       test_deploy.wait_for_http_status_ok(server, port, '/foo', 'image_id', 'chaucer')
       expect(test_deploy).to have_received(:info).with(/Waiting for the port/)
@@ -79,9 +87,9 @@ describe Centurion::Deploy do
     end
 
     it 'waits when the container is not yet up' do
-      test_deploy.stub(:container_up? => false)
-      test_deploy.stub(:error)
-      test_deploy.stub(:warn)
+      allow(test_deploy).to receive(:container_up?).and_return(false)
+      allow(test_deploy).to receive(:error)
+      allow(test_deploy).to receive(:warn)
       expect(test_deploy).to receive(:exit)
       expect(test_deploy).to receive(:sleep).with(0)
 
@@ -90,10 +98,10 @@ describe Centurion::Deploy do
     end
 
     it 'waits when the HTTP status is not OK' do
-      test_deploy.stub(:container_up? => true)
-      test_deploy.stub(:http_status_ok? => false)
-      test_deploy.stub(:error)
-      test_deploy.stub(:warn)
+      allow(test_deploy).to receive(:container_up?).and_return(true)
+      allow(test_deploy).to receive(:http_status_ok?).and_return(false)
+      allow(test_deploy).to receive(:error)
+      allow(test_deploy).to receive(:warn)
       expect(test_deploy).to receive(:exit)
 
       test_deploy.wait_for_http_status_ok(server, port, '/foo', 'image_id', 'chaucer', 1, 0)
@@ -215,18 +223,19 @@ describe Centurion::Deploy do
     let(:bindings) { {'80/tcp'=>[{'HostIp'=>'0.0.0.0', 'HostPort'=>'80'}]} }
 
     it 'pass host_config to start_container' do
-      server.stub(:container_config_for).and_return({
+      allow(server).to receive(:container_config_for).and_return({
         'Image'        => 'image_id',
         'Hostname'     => server.hostname,
       })
 
-      server.stub(:create_container).and_return({
+      allow(server).to receive(:create_container).and_return({
         'Id' => 'abc123456'
       })
 
-      server.stub(:inspect_container)
+      allow(server).to receive(:inspect_container)
 
       allow(test_deploy).to receive(:fetch).with(:custom_dns).and_return('8.8.8.8')
+      allow(test_deploy).to receive(:fetch).with(:name).and_return(nil)
 
       expect(server).to receive(:start_container).with(
         'abc123456',
@@ -242,14 +251,14 @@ describe Centurion::Deploy do
 
   describe '#start_new_container' do
     let(:bindings) { {'80/tcp'=>[{'HostIp'=>'0.0.0.0', 'HostPort'=>'80'}]} }
-    let(:env) { { 'FOO' => 'BAR' } }
-    let(:volumes) { ['/foo:/bar'] }
-    let(:command) { ['/bin/echo', 'hi'] }
+    let(:env)      { { 'FOO' => 'BAR' } }
+    let(:volumes)  { ['/foo:/bar'] }
+    let(:command)  { ['/bin/echo', 'hi'] }
 
     it 'configures the container' do
       expect(test_deploy).to receive(:container_config_for).with(server, 'image_id', bindings, nil, {}, nil).once
 
-      test_deploy.stub(:start_container_with_config)
+      allow(test_deploy).to receive(:start_container_with_config)
 
       test_deploy.start_new_container(server, 'image_id', bindings, {}, nil)
     end
@@ -260,22 +269,44 @@ describe Centurion::Deploy do
       test_deploy.start_new_container(server, 'image_id', bindings, {})
     end
 
+    it 'sets the container hostname when asked' do
+      allow(test_deploy).to receive(:fetch).with(:container_hostname, anything()).and_return('chaucer')
+
+      expect(server).to receive(:create_container).with(
+        hash_including(
+          'Image'        => 'image_id',
+          'Hostname'     => 'chaucer',
+          'ExposedPorts' => {'80/tcp'=>{}},
+          'Cmd'          => command,
+          'Env'          => ['FOO=BAR'],
+          'Volumes'      => {'/bar' => {}},
+        ),
+        nil
+      ).and_return(container)
+
+      expect(server).to receive(:start_container)
+      expect(server).to receive(:inspect_container)
+      test_deploy.start_new_container(server, 'image_id', bindings, volumes, env, command)
+    end
+
     it 'ultimately asks the server object to do the work' do
       allow(test_deploy).to receive(:fetch).with(:custom_dns).and_return(nil)
+      allow(test_deploy).to receive(:fetch).with(:name).and_return('app1')
 
-      server.should_receive(:create_container).with(
+      expect(server).to receive(:create_container).with(
         hash_including(
-          'Image'=>'image_id',
-          'Hostname'=>'host1',
-          'ExposedPorts'=>{'80/tcp'=>{}},
-          'Cmd' => command,
-          'Env' => ['FOO=BAR'],
-          'Volumes' => {'/bar' => {}},
-        )
+          'Image'        => 'image_id',
+          'Hostname'     => hostname,
+          'ExposedPorts' => {'80/tcp'=>{}},
+          'Cmd'          => command,
+          'Env'          => ['FOO=BAR'],
+          'Volumes'      => {'/bar' => {}},
+        ),
+        'app1'
       ).and_return(container)
 
-      server.should_receive(:start_container)
-      server.should_receive(:inspect_container)
+      expect(server).to receive(:start_container)
+      expect(server).to receive(:inspect_container)
 
       new_container = test_deploy.start_new_container(server, 'image_id', bindings, volumes, env, command)
       expect(new_container).to eq(container)
@@ -290,7 +321,7 @@ describe Centurion::Deploy do
 
     it 'configures the container' do
       expect(test_deploy).to receive(:container_config_for).with(server, 'image_id', bindings, env, volumes, command).once
-      test_deploy.stub(:start_container_with_config)
+      allow(test_deploy).to receive(:start_container_with_config)
 
       test_deploy.start_new_container(server, 'image_id', bindings, volumes, env, command)
     end

data/spec/docker_server_group_spec.rb

@@ -23,7 +23,7 @@ describe Centurion::DockerServerGroup do
   end
 
   it 'can run parallel operations' do
-    item = double('item').tap { |i| i.stub(:dummy_method) }
+    item = double('item').tap { |i| allow(i).to receive(:dummy_method) }
     expect(item).to receive(:dummy_method).twice
 
     expect { group.each_in_parallel { |host| item.dummy_method } }.not_to raise_error

data/spec/docker_server_spec.rb

@@ -28,7 +28,7 @@ describe Centurion::DockerServer do
       it "delegates '#{method}' to #{delegate}" do
         dummy_result = double
         dummy_delegate = double(method => dummy_result)
-        server.stub(delegate => dummy_delegate)
+        allow(server).to receive(delegate).and_return(dummy_delegate)
         expect(dummy_delegate).to receive(method)
         expect(server.send(method)).to be(dummy_result)
       end
@@ -37,7 +37,7 @@ describe Centurion::DockerServer do
 
   it 'returns tags associated with an image' do
     image_names = %w[target:latest target:production other:latest]
-    server.stub(ps: image_names.map {|name| { 'Image' => name } })
+    allow(server).to receive(:ps).and_return(image_names.map {|name| { 'Image' => name } })
     expect(server.current_tags_for('target')).to eq(%w[latest production])
   end
 end

data/spec/docker_via_api_spec.rb

@@ -4,8 +4,6 @@ require 'centurion/docker_via_api'
 describe Centurion::DockerViaApi do
   let(:hostname) { 'example.com' }
   let(:port) { '2375' }
-  let(:api) { Centurion::DockerViaApi.new(hostname, port) }
-  let(:excon_uri) { "http://#{hostname}:#{port}/" }
   let(:json_string) { '[{ "Hello": "World" }]' }
   let(:json_value) { JSON.load(json_string) }
   let(:inspected_containers) do
@@ -16,90 +14,227 @@ describe Centurion::DockerViaApi do
     ]
   end
 
-  it 'lists processes' do
-    expect(Excon).to receive(:get).
-                     with(excon_uri + "v1.7" + "/containers/json").
-                     and_return(double(body: json_string, status: 200))
-    expect(api.ps).to eq(json_value)
-  end
+  context 'without TLS certificates' do
+    let(:excon_uri) { "http://#{hostname}:#{port}/" }
+    let(:api) { Centurion::DockerViaApi.new(hostname, port) }
 
-  it 'lists all processes' do
-    expect(Excon).to receive(:get).
-                     with(excon_uri + "v1.7" + "/containers/json?all=1").
-                     and_return(double(body: json_string, status: 200))
-    expect(api.ps(all: true)).to eq(json_value)
-  end
+    it 'lists processes' do
+      expect(Excon).to receive(:get).
+                       with(excon_uri + 'v1.7/containers/json', {}).
+                       and_return(double(body: json_string, status: 200))
+      expect(api.ps).to eq(json_value)
+    end
 
-  it 'inspects an image' do
-    expect(Excon).to receive(:get).
-                     with(excon_uri + "v1.7" + "/images/foo:bar/json",
-                          headers: {'Accept' => 'application/json'}).
-                     and_return(double(body: json_string, status: 200))
-    expect(api.inspect_image('foo', 'bar')).to eq(json_value)
-  end
+    it 'lists all processes' do
+      expect(Excon).to receive(:get).
+                       with(excon_uri + 'v1.7/containers/json?all=1', {}).
+                       and_return(double(body: json_string, status: 200))
+      expect(api.ps(all: true)).to eq(json_value)
+    end
 
-  it 'creates a container' do
-    configuration_as_json = double
-    configuration = double(:to_json => configuration_as_json)
-    expect(Excon).to receive(:post).
-                         with(excon_uri + "v1.10" + "/containers/create",
-                              body: configuration_as_json,
-                              headers: {'Content-Type' => 'application/json'}).
-                         and_return(double(body: json_string, status: 201))
-    api.create_container(configuration)
-  end
+    it 'creates a container' do
+      configuration_as_json = double
+      configuration = double(:to_json => configuration_as_json)
+      expect(Excon).to receive(:post).
+                           with(excon_uri + "v1.10" + "/containers/create",
+                                query: nil,
+                                body: configuration_as_json,
+                                headers: {'Content-Type' => 'application/json'}).
+                           and_return(double(body: json_string, status: 201))
+      api.create_container(configuration)
+    end
 
-  it 'starts a container' do
-    configuration_as_json = double
-    configuration = double(:to_json => configuration_as_json)
-    expect(Excon).to receive(:post).
-                         with(excon_uri + "v1.10" + "/containers/12345/start",
-                              body: configuration_as_json,
-                              headers: {'Content-Type' => 'application/json'}).
-                         and_return(double(body: json_string, status: 204))
-    api.start_container('12345', configuration)
-  end
+    it 'creates a container with a name' do
+      configuration_as_json = double
+      configuration = double(:to_json => configuration_as_json)
+      expect(Excon).to receive(:post).
+                           with(excon_uri + "v1.10" + "/containers/create",
+                                query: { :name => match(/^app1-[a-f0-9]+$/) },
+                                body: configuration_as_json,
+                                headers: {'Content-Type' => 'application/json'}).
+                           and_return(double(body: json_string, status: 201))
+      api.create_container(configuration, 'app1')
+    end
 
-  it 'stops a container' do
-    expect(Excon).to receive(:post).
-                     with(excon_uri + "v1.7" + "/containers/12345/stop?t=300").
-                     and_return(double(status: 204))
-    api.stop_container('12345', 300)
-  end
+    it 'starts a container' do
+      configuration_as_json = double
+      configuration = double(:to_json => configuration_as_json)
+      expect(Excon).to receive(:post).
+                           with(excon_uri + "v1.10" + "/containers/12345/start",
+                                body: configuration_as_json,
+                                headers: {'Content-Type' => 'application/json'}).
+                           and_return(double(body: json_string, status: 204))
+      api.start_container('12345', configuration)
+    end
 
-  it 'stops a container with a custom timeout' do
-    expect(Excon).to receive(:post).
-                     with(excon_uri + "v1.7" + "/containers/12345/stop?t=30").
-                     and_return(double(status: 204))
-    api.stop_container('12345')
-  end
+    it 'stops a container' do
+      expect(Excon).to receive(:post).
+                       with(excon_uri + 'v1.7/containers/12345/stop?t=300', {}).
+                       and_return(double(status: 204))
+      api.stop_container('12345', 300)
+    end
 
-  it 'inspects a container' do
-    expect(Excon).to receive(:get).
-                         with(excon_uri + "v1.7" + "/containers/12345/json").
-                         and_return(double(body: json_string, status: 200))
-    expect(api.inspect_container('12345')).to eq(json_value)
-  end
+    it 'stops a container with a custom timeout' do
+      expect(Excon).to receive(:post).
+                       with(excon_uri + 'v1.7/containers/12345/stop?t=30', {}).
+                       and_return(double(status: 204))
+      api.stop_container('12345')
+    end
+
+    it 'inspects a container' do
+      expect(Excon).to receive(:get).
+                           with(excon_uri + 'v1.7/containers/12345/json', {}).
+                           and_return(double(body: json_string, status: 200))
+      expect(api.inspect_container('12345')).to eq(json_value)
+    end
+
+    it 'removes a container' do
+      expect(Excon).to receive(:delete).
+                           with(excon_uri + 'v1.7/containers/12345', {}).
+                           and_return(double(status: 204))
+      expect(api.remove_container('12345')).to eq(true)
+    end
+
+    it 'lists old containers for a port' do
+      expect(Excon).to receive(:get).
+                           with(excon_uri + 'v1.7/containers/json?all=1', {}).
+                           and_return(double(body: inspected_containers.to_json, status: 200))
+      expect(Excon).to receive(:get).
+                           with(excon_uri + 'v1.7/containers/123/json', {}).
+                           and_return(double(body: inspected_container_on_port("123", 8485).to_json, status: 200))
+      expect(Excon).to receive(:get).
+                           with(excon_uri + 'v1.7/containers/789/json', {}).
+                           and_return(double(body: inspected_container_on_port("789", 8486).to_json, status: 200))
+
+      expect(api.old_containers_for_port(8485)).to eq([{"Id" => "123", "Status" => "Exit 0"}])
+    end
+
+    it 'inspects an image' do
+      expect(Excon).to receive(:get).
+                       with(excon_uri + "v1.7" + "/images/foo:bar/json",
+                            headers: {'Accept' => 'application/json'}).
+                       and_return(double(body: json_string, status: 200))
+      expect(api.inspect_image('foo', 'bar')).to eq(json_value)
+    end
 
-  it 'removes a container' do
-    expect(Excon).to receive(:delete).
-                         with(excon_uri + "v1.7" + "/containers/12345").
-                         and_return(double(status: 204))
-    expect(api.remove_container('12345')).to eq(true)
   end
 
-  it 'lists old containers for a port' do
-    expect(Excon).to receive(:get).
-                         with(excon_uri + "v1.7" + "/containers/json?all=1").
-                         and_return(double(body: inspected_containers.to_json, status: 200))
-    expect(Excon).to receive(:get).
-                         with(excon_uri + "v1.7" + "/containers/123/json").
-                         and_return(double(body: inspected_container_on_port("123", 8485).to_json, status: 200))
-    expect(Excon).to receive(:get).
-                         with(excon_uri + "v1.7" + "/containers/789/json").
-                         and_return(double(body: inspected_container_on_port("789", 8486).to_json, status: 200))
-
-    expect(api.old_containers_for_port(8485)).to eq([{"Id" => "123", "Status" => "Exit 0"}])
+  context 'with TLS certificates' do
+    let(:excon_uri) { "https://#{hostname}:#{port}/" }
+    let(:tls_args)  { { tls: true, tlscacert: '/certs/ca.pem',
+                       tlscert: '/certs/cert.pem', tlskey: '/certs/key.pem' } }
+    let(:api)       { Centurion::DockerViaApi.new(hostname, port, tls_args) }
+
+    it 'lists processes' do
+      expect(Excon).to receive(:get).
+                       with(excon_uri + 'v1.7/containers/json',
+                            client_cert: '/certs/cert.pem',
+                            client_key: '/certs/key.pem').
+                       and_return(double(body: json_string, status: 200))
+      expect(api.ps).to eq(json_value)
+    end
+
+    it 'lists all processes' do
+      expect(Excon).to receive(:get).
+                       with(excon_uri + 'v1.7/containers/json?all=1',
+                            client_cert: '/certs/cert.pem',
+                            client_key: '/certs/key.pem').
+                       and_return(double(body: json_string, status: 200))
+      expect(api.ps(all: true)).to eq(json_value)
+    end
+
+    it 'inspects an image' do
+      expect(Excon).to receive(:get).
+                       with(excon_uri + 'v1.7/images/foo:bar/json',
+                            client_cert: '/certs/cert.pem',
+                            client_key: '/certs/key.pem',
+                            headers: {'Accept' => 'application/json'}).
+                       and_return(double(body: json_string, status: 200))
+      expect(api.inspect_image('foo', 'bar')).to eq(json_value)
+    end
+
+    it 'creates a container' do
+      configuration_as_json = double
+      configuration = double(:to_json => configuration_as_json)
+      expect(Excon).to receive(:post).
+                           with(excon_uri + 'v1.10/containers/create',
+                                client_cert: '/certs/cert.pem',
+                                client_key: '/certs/key.pem',
+                                query: nil,
+                                body: configuration_as_json,
+                                headers: {'Content-Type' => 'application/json'}).
+                           and_return(double(body: json_string, status: 201))
+      api.create_container(configuration)
+    end
+
+    it 'starts a container' do
+      configuration_as_json = double
+      configuration = double(:to_json => configuration_as_json)
+      expect(Excon).to receive(:post).
+                           with(excon_uri + 'v1.10/containers/12345/start',
+                                client_cert: '/certs/cert.pem',
+                                client_key: '/certs/key.pem',
+                                body: configuration_as_json,
+                                headers: {'Content-Type' => 'application/json'}).
+                           and_return(double(body: json_string, status: 204))
+      api.start_container('12345', configuration)
+    end
+
+    it 'stops a container' do
+      expect(Excon).to receive(:post).
+                       with(excon_uri + 'v1.7/containers/12345/stop?t=300',
+                            client_cert: '/certs/cert.pem',
+                            client_key: '/certs/key.pem').
+                       and_return(double(status: 204))
+      api.stop_container('12345', 300)
+    end
+
+    it 'stops a container with a custom timeout' do
+      expect(Excon).to receive(:post).
+                       with(excon_uri + 'v1.7/containers/12345/stop?t=30',
+                            client_cert: '/certs/cert.pem',
+                            client_key: '/certs/key.pem').
+                       and_return(double(status: 204))
+      api.stop_container('12345')
+    end
+
+    it 'inspects a container' do
+      expect(Excon).to receive(:get).
+                           with(excon_uri + 'v1.7/containers/12345/json',
+                                client_cert: '/certs/cert.pem',
+                                client_key: '/certs/key.pem').
+                           and_return(double(body: json_string, status: 200))
+      expect(api.inspect_container('12345')).to eq(json_value)
+    end
+
+    it 'removes a container' do
+      expect(Excon).to receive(:delete).
+                           with(excon_uri + 'v1.7/containers/12345',
+                                client_cert: '/certs/cert.pem',
+                                client_key: '/certs/key.pem').
+                           and_return(double(status: 204))
+      expect(api.remove_container('12345')).to eq(true)
+    end
+
+    it 'lists old containers for a port' do
+      expect(Excon).to receive(:get).
+                           with(excon_uri + 'v1.7/containers/json?all=1',
+                                client_cert: '/certs/cert.pem',
+                                client_key: '/certs/key.pem').
+                           and_return(double(body: inspected_containers.to_json, status: 200))
+      expect(Excon).to receive(:get).
+                           with(excon_uri + 'v1.7/containers/123/json',
+                                client_cert: '/certs/cert.pem',
+                                client_key: '/certs/key.pem').
+                           and_return(double(body: inspected_container_on_port("123", 8485).to_json, status: 200))
+      expect(Excon).to receive(:get).
+                           with(excon_uri + 'v1.7/containers/789/json',
+                                client_cert: '/certs/cert.pem',
+                                client_key: '/certs/key.pem').
+                           and_return(double(body: inspected_container_on_port("789", 8486).to_json, status: 200))
+
+      expect(api.old_containers_for_port(8485)).to eq([{"Id" => "123", "Status" => "Exit 0"}])
+    end
   end
 
   def inspected_container_on_port(id, port)