centurion 1.10.1 → 1.10.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 51af32612106564d2523f46f98c8a862d855ec94
-  data.tar.gz: b92dfa1483449ac9d90be790a11eb79ea50ebd4f
+  metadata.gz: fc7802eda143e2a4c3ea296957448e864b044dfd
+  data.tar.gz: 4aefded54c32d42cdb9d2e3a661e3aa087ee1bcc
 SHA512:
-  metadata.gz: 264d38524264717e972df446a5dddd570ae7a913e05dee5be7306cfde7b03be221b48c8a6a3630d3b9e18f5b1345d34d66f09dc002185114a66f3f7a470f3b9d
-  data.tar.gz: '0609e9e3ffbf3cafb689e2ba4f47bd49eaa6ab0af32423132eb53901c8979d87d2beb13b5ab3b84f9fbb8e51f4288950d2adeaf86435717cc365bbbe5686ed7d'
+  metadata.gz: e4dfe15cb4dda8e19c72a7870a4e6a987ca12ee4ac775b15443b97cc3ffbd3100bef6430b7a5be930e192dc4e818825e65fc22f76f1f2fb4df964e9c9b09ec10
+  data.tar.gz: 696f5cb9fb26a33a5bb4283ffd8c273c793bbb725bd1f2072ac3f662195af5401aaabe2e0a60eaf1f0c20942a4b90677b7b8b70f00fd865e6c2bc36ab76275f9

data/.travis.yml

@@ -1,6 +1,6 @@
 language: ruby
 rvm:
-  - 1.9.3
+  - 1.9.3-p551
   - 2.0.0
   - 2.1
   - 2.2
@@ -9,4 +9,7 @@ rvm:
   - 2.5
 before_install:
   - gem update
-  - gem install bundler
+  # downgrade bundler to fix installation error with ruby <= 2.2
+  # https://docs.travis-ci.com/user/languages/ruby/#bundler-20
+  - gem uninstall -v '>= 2' -i $(rvm gemdir)@global -ax bundler || true
+  - gem install bundler -v '< 2'

data/README.md

@@ -401,9 +401,20 @@ You can configure it with a few options:
     set :ssh, true # enable ssh connections
     set :ssh_user, "myuser" # if you want to specify the user to connect as, otherwise your current user
     set :ssh_log_level, Logger::DEBUG # passed on to net/ssh, can be noisy; defaults to Logger::WARN
+    set :ssh_socket_heartbeat, 5 # passed on to net/ssh (Net::SSH::Connection::Session#loop); defaults to 30
   end
 ```
 
+#### Troubleshooting SSH connections
+
+In some cases you may notice your SSH commands are completing successfully,
+but the connection isn't aware they are done. This will manifest as a tunneled
+command taking much longer than anticipated - minutes depending on your
+server's configuration. You may need to set `:ssh_socket_heartbeat` to a
+smaller number. This will check more frequently if the command has completed
+which can alleviate this issue, though it will consume more CPU as it wakes up
+its threads more frequently.
+
 Deploying
 ---------
 

data/lib/centurion/deploy_dsl.rb

@@ -195,6 +195,7 @@ module Centurion::DeployDSL
       # nil is OK for both of these, defaults applied internally
       opts[:ssh_user] = fetch(:ssh_user)
       opts[:ssh_log_level] = fetch(:ssh_log_level)
+      opts[:ssh_socket_heartbeat] = fetch(:ssh_socket_heartbeat)
     end
 
     opts

data/lib/centurion/docker_via_api.rb

@@ -12,8 +12,7 @@ class Centurion::DockerViaApi
     if connection_opts[:ssh]
       @base_uri = hostname
       @ssh = true
-      @ssh_user = connection_opts[:ssh_user]
-      @ssh_log_level = connection_opts[:ssh_log_level]
+      @connection_opts = connection_opts
     else
       @base_uri = "http#{'s' if tls_enable?}://#{hostname}:#{port}"
     end
@@ -194,7 +193,7 @@ class Centurion::DockerViaApi
   end
 
   def with_excon_via_ssh
-    Centurion::SSH.with_docker_socket(@base_uri, @ssh_user, @ssh_log_level) do |socket|
+    Centurion::SSH.with_docker_socket(@base_uri, @connection_opts[:ssh_user], @connection_opts[:ssh_log_level], @connection_opts[:ssh_socket_heartbeat]) do |socket|
       conn = Excon.new('unix:///', socket: socket)
       yield conn
     end

data/lib/centurion/docker_via_cli.rb

@@ -97,7 +97,7 @@ class Centurion::DockerViaCli
 
   def connect
     if @connection_opts[:ssh]
-      Centurion::SSH.with_docker_socket(@docker_host, @connection_opts[:ssh_user], @connection_opts[:ssh_log_level]) do |socket|
+      Centurion::SSH.with_docker_socket(@docker_host, @connection_opts[:ssh_user], @connection_opts[:ssh_log_level], @connection_opts[:ssh_socket_heartbeat]) do |socket|
         @socket = socket
         ret = yield
         @socket = nil

data/lib/centurion/ssh.rb

@@ -7,7 +7,7 @@ module Centurion; end
 module Centurion::SSH
   extend self
 
-  def with_docker_socket(hostname, user, log_level = nil)
+  def with_docker_socket(hostname, user, log_level = nil, ssh_socket_heartbeat = 30)
     log_level ||= Logger::WARN
 
     with_sshkit(hostname, user) do
@@ -24,7 +24,7 @@ module Centurion::SSH
           yield local_socket_path
         end
 
-        ssh.loop { t.alive? }
+        ssh.loop(ssh_socket_heartbeat) { t.alive? }
         ssh.forward.cancel_local_socket local_socket_path
         local_socket_path_file.delete
         t.value

data/lib/centurion/version.rb

@@ -1,3 +1,3 @@
 module Centurion
-  VERSION = '1.10.1'
+  VERSION = '1.10.2'
 end

data/spec/docker_via_api_spec.rb

@@ -134,6 +134,7 @@ describe Centurion::DockerViaApi do
     let(:port) { nil }
     let(:ssh_user) { 'myuser' }
     let(:ssh_log_level) { nil }
+    let(:ssh_socket_heartbeat) { nil }
     let(:base_req) { {
       socket: '/tmp/socket/path'
     } }
@@ -142,12 +143,13 @@ describe Centurion::DockerViaApi do
       p = { ssh: true}
       p[:ssh_user] = ssh_user if ssh_user
       p[:ssh_log_level] = ssh_log_level if ssh_log_level
+      p[:ssh_socket_heartbeat] = ssh_socket_heartbeat if ssh_socket_heartbeat
       p
     end
 
     context 'with no log level' do
       before do
-        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, nil).and_yield('/tmp/socket/path')
+        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, nil, nil).and_yield('/tmp/socket/path')
       end
 
       it_behaves_like 'docker API'
@@ -157,7 +159,7 @@ describe Centurion::DockerViaApi do
       let(:ssh_user) { nil }
 
       before do
-        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, nil, nil).and_yield('/tmp/socket/path')
+        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, nil, nil, nil).and_yield('/tmp/socket/path')
       end
 
       it_behaves_like 'docker API'
@@ -167,7 +169,17 @@ describe Centurion::DockerViaApi do
       let(:ssh_log_level) { Logger::DEBUG }
 
       before do
-        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, Logger::DEBUG).and_yield('/tmp/socket/path')
+        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, Logger::DEBUG, nil).and_yield('/tmp/socket/path')
+      end
+
+      it_behaves_like 'docker API'
+    end
+
+    context 'with a socket heartbeat set' do
+      let(:ssh_socket_heartbeat) { 5 }
+
+      before do
+        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, nil, 5).and_yield('/tmp/socket/path')
       end
 
       it_behaves_like 'docker API'

data/spec/docker_via_cli_spec.rb

@@ -69,18 +69,20 @@ describe Centurion::DockerViaCli do
     let(:hostname) { 'host1' }
     let(:ssh_user) { 'myuser' }
     let(:ssh_log_level) { nil }
+    let(:ssh_socket_heartbeat) { nil }
     let(:docker_via_cli) { Centurion::DockerViaCli.new(hostname, nil, docker_path, params) }
     let(:prefix) { "-H=unix:///tmp/socket/path" }
     let(:params) do
-      p = { ssh: true}
+      p = { ssh: true }
       p[:ssh_user] = ssh_user if ssh_user
       p[:ssh_log_level] = ssh_log_level if ssh_log_level
+      p[:ssh_socket_heartbeat] = ssh_socket_heartbeat if ssh_socket_heartbeat
       p
     end
 
     context 'with no log level' do
       before do
-        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, nil).and_yield('/tmp/socket/path')
+        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, nil, nil).and_yield('/tmp/socket/path')
       end
 
       it_behaves_like 'docker CLI'
@@ -90,7 +92,7 @@ describe Centurion::DockerViaCli do
       let(:ssh_user) { nil }
 
       before do
-        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, nil, nil).and_yield('/tmp/socket/path')
+        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, nil, nil, nil).and_yield('/tmp/socket/path')
       end
 
       it_behaves_like 'docker CLI'
@@ -100,7 +102,17 @@ describe Centurion::DockerViaCli do
       let(:ssh_log_level) { Logger::DEBUG }
 
       before do
-        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, Logger::DEBUG).and_yield('/tmp/socket/path')
+        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, Logger::DEBUG, nil).and_yield('/tmp/socket/path')
+      end
+
+      it_behaves_like 'docker CLI'
+    end
+
+    context 'with an ssh loop wait set' do
+      let(:ssh_socket_heartbeat) { 5 }
+
+      before do
+        expect(Centurion::SSH).to receive(:with_docker_socket).with(hostname, ssh_user, nil, 5).and_yield('/tmp/socket/path')
       end
 
       it_behaves_like 'docker CLI'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: centurion
 version: !ruby/object:Gem::Version
-  version: 1.10.1
+  version: 1.10.2
 platform: ruby
 authors:
 - Nic Benders
@@ -21,7 +21,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-12-06 00:00:00.000000000 Z
+date: 2019-07-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: trollop