beaker-docker 1.3.0 → 1.5.0

data/Rakefile

@@ -1,176 +1,79 @@
+# frozen_string_literal: true
+
 require 'rspec/core/rake_task'
 
-namespace :test do
+begin
+  require 'rubocop/rake_task'
+rescue LoadError
+  # RuboCop is an optional group
+else
+  RuboCop::RakeTask.new(:rubocop) do |task|
+    # These make the rubocop experience maybe slightly less terrible
+    task.options = ['--display-cop-names', '--display-style-guide', '--extra-details']
+    # Use Rubocop's Github Actions formatter if possible
+    task.formatters << 'github' if ENV['GITHUB_ACTIONS'] == 'true'
+  end
+end
 
+namespace :test do
   namespace :spec do
-
-    desc "Run spec tests"
+    desc 'Run spec tests'
     RSpec::Core::RakeTask.new(:run) do |t|
       t.rspec_opts = ['--color', '--format documentation']
       t.pattern = 'spec/'
     end
 
-    desc "Run spec tests with coverage"
+    desc 'Run spec tests with coverage'
     RSpec::Core::RakeTask.new(:coverage) do |t|
       ENV['BEAKER_DOCKER_COVERAGE'] = 'y'
       t.rspec_opts = ['--color', '--format documentation']
       t.pattern = 'spec/'
     end
-
   end
 
   namespace :acceptance do
-
-    desc <<-EOS
-A quick acceptance test, named because it has no pre-suites to run
-    EOS
+    desc 'A quick acceptance test, named because it has no pre-suites to run'
     task :quick do
-
-      # setup & load_path of beaker's acceptance base and lib directory
-      beaker_gem_spec = Gem::Specification.find_by_name('beaker')
-      beaker_gem_dir = beaker_gem_spec.gem_dir
-      beaker_test_base_dir = File.join(beaker_gem_dir, 'acceptance/tests/base')
-      load_path_option = File.join(beaker_gem_dir, 'acceptance/lib')
+      ## setup & load_path of beaker's acceptance base and lib directory
+      ## see below for the reason why it's commented out atm
+      # beaker_gem_spec = Gem::Specification.find_by_name('beaker')
+      # beaker_gem_dir = beaker_gem_spec.gem_dir
+      # beaker_test_base_dir = File.join(beaker_gem_dir, 'acceptance/tests/base')
+      # load_path_option = File.join(beaker_gem_dir, 'acceptance/lib')
 
       ENV['BEAKER_setfile'] = 'acceptance/config/nodes/hosts.yaml'
-      sh("beaker",
-         "--hosts", "acceptance/config/nodes/hosts.yaml",
-          # We can't run these tests until the rsync support in the main
-          # beaker/host.rb is updated to work with passwords.
-          # "--tests", beaker_test_base_dir,
-          # "--load-path", load_path_option,
-         "--tests", 'acceptance/tests/',
-         "--log-level", "debug",
-         "--debug")
+      sh('beaker',
+         '--hosts', 'acceptance/config/nodes/hosts.yaml',
+         ## We can't run these tests until the rsync support in the main
+         ## beaker/host.rb is updated to work with passwords.
+         # '--tests', beaker_test_base_dir,
+         # '--load-path', load_path_option,
+         '--tests', 'acceptance/tests/',
+         '--log-level', 'debug',
+         '--debug')
     end
-
   end
-
 end
 
 # namespace-named default tasks.
 # these are the default tasks invoked when only the namespace is referenced.
 # they're needed because `task :default` in those blocks doesn't work as expected.
-task 'test:spec' => 'test:spec:run'
-task 'test:acceptance' => 'test:acceptance:quick'
+task 'test:spec': %i[test:spec:run]
+task 'test:acceptance': %i[test:acceptance:quick]
 
 # global defaults
-task :test => 'test:spec'
-task :default => :test
-
-###########################################################
-#
-#   Documentation Tasks
-#
-###########################################################
-DOCS_DAEMON = "yard server --reload --daemon --server thin"
-FOREGROUND_SERVER = 'bundle exec yard server --reload --verbose --server thin lib/beaker'
-
-def running?( cmdline )
-  ps = `ps -ef`
-  found = ps.lines.grep( /#{Regexp.quote( cmdline )}/ )
-  if found.length > 1
-    raise StandardError, "Found multiple YARD Servers. Don't know what to do."
-  end
-
-  yes = found.empty? ? false : true
-  return yes, found.first
-end
-
-def pid_from( output )
-  output.squeeze(' ').strip.split(' ')[1]
-end
-
-desc 'Start the documentation server in the foreground'
-task :docs => 'docs:clear' do
-  original_dir = Dir.pwd
-  Dir.chdir( File.expand_path(File.dirname(__FILE__)) )
-  sh FOREGROUND_SERVER
-  Dir.chdir( original_dir )
-end
-
-namespace :docs do
-
-  desc 'Clear the generated documentation cache'
-  task :clear do
-    original_dir = Dir.pwd
-    Dir.chdir( File.expand_path(File.dirname(__FILE__)) )
-    sh 'rm -rf docs'
-    Dir.chdir( original_dir )
-  end
-
-  desc 'Generate static documentation'
-  task :gen => 'docs:clear' do
-    original_dir = Dir.pwd
-    Dir.chdir( File.expand_path(File.dirname(__FILE__)) )
-    output = `bundle exec yard doc`
-    puts output
-    if output =~ /\[warn\]|\[error\]/
-      fail "Errors/Warnings during yard documentation generation"
-    end
-    Dir.chdir( original_dir )
-  end
-
-  desc 'Run the documentation server in the background, alias `bg`'
-  task :background => 'docs:clear' do
-    yes, output = running?( DOCS_DAEMON )
-    if yes
-      puts "Not starting a new YARD Server..."
-      puts "Found one running with pid #{pid_from( output )}."
-    else
-      original_dir = Dir.pwd
-      Dir.chdir( File.expand_path(File.dirname(__FILE__)) )
-      sh "bundle exec #{DOCS_DAEMON}"
-      Dir.chdir( original_dir )
-    end
-  end
-
-  task(:bg) { Rake::Task['docs:background'].invoke }
-
-  desc 'Check the status of the documentation server'
-  task :status do
-    yes, output = running?( DOCS_DAEMON )
-    if yes
-      pid = pid_from( output )
-      puts "Found a YARD Server running with pid #{pid}"
-    else
-      puts "Could not find a running YARD Server."
-    end
-  end
-
-  desc "Stop a running YARD Server"
-  task :stop do
-    yes, output = running?( DOCS_DAEMON )
-    if yes
-      pid = pid_from( output )
-      puts "Found a YARD Server running with pid #{pid}"
-      `kill #{pid}`
-      puts "Stopping..."
-      yes, output = running?( DOCS_DAEMON )
-      if yes
-        `kill -9 #{pid}`
-        yes, output = running?( DOCS_DAEMON )
-        if yes
-          puts "Could not Stop Server!"
-        else
-          puts "Server stopped."
-        end
-      else
-        puts "Server stopped."
-      end
-    else
-      puts "Could not find a running YARD Server"
-    end
-  end
-end
+task lint: %i[lint:rubocop]
+task test: %i[test:spec]
+task default: %i[test]
 
 begin
   require 'rubygems'
   require 'github_changelog_generator/task'
 rescue LoadError
+  # Do nothing if no required gem installed
 else
   GitHubChangelogGenerator::RakeTask.new :changelog do |config|
-    config.exclude_labels = %w{duplicate question invalid wontfix wont-fix skip-changelog}
+    config.exclude_labels = %w[duplicate question invalid wontfix wont-fix skip-changelog]
     config.user = 'voxpupuli'
     config.project = 'beaker-docker'
     gem_version = Gem::Specification.load("#{config.project}.gemspec").version

data/acceptance/tests/00_default_spec.rb

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
+
 require 'beaker'
-require 'beaker-rspec'
 
-RSpec.describe 'it can connect' do
+test_name 'Ensure docker container is accessible' do
   hosts.each do |host|
-    context "on #{host}" do
-      on(host, 'ls /tmp')
+    step "on #{host}" do
+      on(host, 'true')
     end
   end
 end

data/beaker-docker.gemspec

@@ -1,35 +1,41 @@
-# -*- encoding: utf-8 -*-
-$LOAD_PATH.unshift File.expand_path("../lib", __FILE__)
+# frozen_string_literal: true
+
+$LOAD_PATH.unshift File.expand_path('lib', __dir__)
 require 'beaker-docker/version'
 
 Gem::Specification.new do |s|
-  s.name        = "beaker-docker"
+  s.name        = 'beaker-docker'
   s.version     = BeakerDocker::VERSION
   s.authors     = [
-    "Vox Pupuli",
-    "Rishi Javia",
-    "Kevin Imber",
-    "Tony Vu"
+    'Vox Pupuli',
+    'Rishi Javia',
+    'Kevin Imber',
+    'Tony Vu',
   ]
-  s.email       = ["voxpupuli@groups.io"]
-  s.homepage    = "https://github.com/voxpupuli/beaker-docker"
-  s.summary     = %q{Beaker DSL Extension Helpers!}
-  s.description = %q{For use for the Beaker acceptance testing tool}
+  s.email       = ['voxpupuli@groups.io']
+  s.homepage    = 'https://github.com/voxpupuli/beaker-docker'
+  s.summary     = 'Docker hypervisor for Beaker acceptance testing framework'
+  s.description = 'Allows running Beaker tests using Docker'
   s.license     = 'Apache-2.0'
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  s.require_paths = ["lib"]
+  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
+  s.require_paths = ['lib']
+
+  s.required_ruby_version = '>= 2.4', '< 4'
 
   # Testing dependencies
-  s.add_development_dependency 'rspec', '~> 3.0'
-  s.add_development_dependency 'rspec-its', '~> 1.3'
-  s.add_development_dependency 'fakefs', '~> 1.3'
+  s.add_development_dependency 'fakefs', '>= 1.3', '< 3.0'
   s.add_development_dependency 'rake', '~> 13.0'
+  s.add_development_dependency 'rspec', '~> 3.0'
+  s.add_development_dependency 'rubocop', '~> 1.12.0'
+  s.add_development_dependency 'rubocop-performance', '~> 1.10'
+  s.add_development_dependency 'rubocop-rake', '~> 0.2'
+  s.add_development_dependency 'rubocop-rspec', '>= 1.44'
 
   # Run time dependencies
-  s.add_runtime_dependency 'stringify-hash', '~> 0.0.0'
-  s.add_runtime_dependency 'docker-api', '~> 2.1'
   s.add_runtime_dependency 'beaker', '>= 4.34'
+  s.add_runtime_dependency 'docker-api', '~> 2.1'
+  s.add_runtime_dependency 'stringify-hash', '~> 0.0.0'
 end

data/bin/beaker-docker

@@ -1,10 +1,10 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require 'rubygems' unless defined?(Gem)
 require 'beaker-docker'
 
-VERSION_STRING =
-"
+VERSION_STRING = <<'VER'
                                  _ .--.
                                 ( `    )
         beaker-docker         .-'      `--,
@@ -14,19 +14,17 @@ VERSION_STRING =
               ;|  _|  _|  _|  '-'__,--'`--'
               | _|  _|  _|  _| |
           _   ||  _|  _|  _|  _| %s
-        _( `--.\\_|  _|  _|  _|/
+        _( `--.\_|  _|  _|  _|/
      .-'       )--,|  _|  _|.`
     (__, (_      ) )_|  _| /
-      `-.__.\\ _,--'\\|__|__/
+      `-.__.\ _,--'\|__|__/
                     ;____;
-                     \\YT/
+                     \YT/
                       ||
-                     |\"\"|
+                     |""|
                      '=='
-"
+VER
 
-
-
-puts BeakerDocker::VERSION
+puts VERSION_STRING % BeakerDocker::VERSION
 
 exit 0

data/lib/beaker/hypervisor/docker.rb

@@ -30,14 +30,14 @@ module Beaker
           * Check your DOCKER_HOST variable has been set
           * If you are on OSX or Windows, you might not have Docker Machine setup correctly: https://docs.docker.com/machine/
           * If you are using rootless podman, you might need to set up your local socket and service
-          ERRMSG
+        ERRMSG
       end
 
       # Pass on all the logging from docker-api to the beaker logger instance
       ::Docker.logger = @logger
 
       # Find out what kind of remote instance we are talking against
-      if @docker_version['Version'] =~ /swarm/
+      if /swarm/.match?(@docker_version['Version'])
         @docker_type = 'swarm'
         unless ENV['DOCKER_REGISTRY']
           raise "Using Swarm with beaker requires a private registry. Please setup the private registry and set the 'DOCKER_REGISTRY' env var"
@@ -58,7 +58,7 @@ module Beaker
 
       # If the container is running ssh as its init process then this method
       # will cause issues.
-      if host[:docker_cmd] =~ /sshd/
+      if /sshd/.match?(Array(host[:docker_cmd]).first)
         def host.ssh_service_restart
           self[:docker_container].exec(%w(kill -1 1))
         end
@@ -142,9 +142,9 @@ module Beaker
         port: nil
       }
 
-      container_json = container.json
+      container_json   = container.json
       network_settings = container_json['NetworkSettings']
-      host_config = container_json['HostConfig']
+      host_config      = container_json['HostConfig']
 
       ip = nil
       port = nil
@@ -161,31 +161,28 @@ module Beaker
 
           # Host to Container
           port22 = network_settings.dig('PortBindings','22/tcp')
-          if port22.nil? && network_settings.key?('Ports')
+          if port22.nil? && network_settings.key?('Ports') && !nested_docker?
             port22 = network_settings.dig('Ports','22/tcp')
           end
+
           ip = port22[0]['HostIp'] if port22
           port = port22[0]['HostPort'] if port22
 
           # Container to container
           unless ip && port
-            ip = nil
-            port = nil
-
             ip = network_settings['IPAddress']
-            port = 22 if ip && !ip.empty?
+            port = ip && !ip.empty? ? 22 : nil
           end
 
           # Container through gateway
           unless ip && port
-            ip = nil
-            port = nil
-
             ip = network_settings['Gateway']
 
             if ip && !ip.empty?
               port22 = network_settings.dig('PortBindings','22/tcp')
               port = port22[0]['HostPort'] if port22
+            else
+              port = nil
             end
           end
 
@@ -216,7 +213,6 @@ module Beaker
       @hosts.each do |host|
         @logger.notify "provisioning #{host.name}"
 
-
         image = get_container_image(host)
 
         if host['tag']
@@ -510,145 +506,129 @@ module Beaker
 
     def dockerfile_for(host)
       # specify base image
-      dockerfile = <<-EOF
+      dockerfile = <<~DF
         FROM #{host['image']}
         ENV container docker
-      EOF
+      DF
 
       # Commands before any other commands. Can be used for eg. proxy configuration
-      dockerfile += (host['docker_image_first_commands'] || []).map { |command|
-        "RUN #{command}\n"
-      }.join('')
-
-      # additional options to specify to the sshd
-      # may vary by platform
-      sshd_options = ''
+      dockerfile += (host['docker_image_first_commands'] || []).map { |cmd| "RUN #{cmd}\n" }.join('')
 
       # add platform-specific actions
-      service_name = "sshd"
+      service_name = 'sshd'
       additional_packages = host_packages(host)
       case host['platform']
       when /ubuntu/, /debian/
-        service_name = "ssh"
-        dockerfile += <<~EOF
-          RUN apt-get update
-          RUN apt-get install -y openssh-server openssh-client #{additional_packages.join(' ')}
-          RUN sed 's@session\s*required\s*pam_loginuid.so@session optional pam_loginuid.so@g' -i /etc/pam.d/*
-          EOF
-      when  /cumulus/
-          dockerfile += <<~EOF
-          RUN apt-get update
-          RUN apt-get install -y openssh-server openssh-client #{additional_packages.join(' ')}
-          EOF
+        service_name = 'ssh'
+        dockerfile += <<~DF
+          RUN apt-get update \
+          && apt-get install -y openssh-server openssh-client #{additional_packages.join(' ')} \
+          && sed 's@session\s*required\s*pam_loginuid.so@session optional pam_loginuid.so@g' -i /etc/pam.d/*
+        DF
+      when /cumulus/
+        dockerfile += <<~DF
+          RUN apt-get update \
+          && apt-get install -y openssh-server openssh-client #{additional_packages.join(' ')}
+        DF
       when /el-[89]/, /fedora-(2[2-9]|3)/
-        dockerfile += <<~EOF
-          RUN dnf clean all
-          RUN dnf install -y sudo openssh-server openssh-clients #{additional_packages.join(' ')}
-          RUN ssh-keygen -t rsa -f /etc/ssh/ssh_host_rsa_key
-          RUN ssh-keygen -t dsa -f /etc/ssh/ssh_host_dsa_key
-          RUN sed 's@session\s*required\s*pam_loginuid.so@session optional pam_loginuid.so@g' -i /etc/pam.d/*
-          EOF
+        dockerfile += <<~DF
+          RUN dnf clean all \
+          && dnf install -y sudo openssh-server openssh-clients #{additional_packages.join(' ')} \
+          && ssh-keygen -A \
+          && sed 's@session\s*required\s*pam_loginuid.so@session optional pam_loginuid.so@g' -i /etc/pam.d/*
+        DF
       when /^el-/, /centos/, /fedora/, /redhat/, /eos/
-        dockerfile += <<~EOF
-          RUN yum clean all
-          RUN yum install -y sudo openssh-server openssh-clients #{additional_packages.join(' ')}
-          RUN ssh-keygen -t rsa -f /etc/ssh/ssh_host_rsa_key
-          RUN ssh-keygen -t dsa -f /etc/ssh/ssh_host_dsa_key
-          RUN sed 's@session\s*required\s*pam_loginuid.so@session optional pam_loginuid.so@g' -i /etc/pam.d/*
-          EOF
+        dockerfile += <<~DF
+          RUN yum clean all \
+          && yum install -y sudo openssh-server openssh-clients #{additional_packages.join(' ')} \
+          && ssh-keygen -A \
+          && sed 's@session\s*required\s*pam_loginuid.so@session optional pam_loginuid.so@g' -i /etc/pam.d/*
+        DF
       when /opensuse/, /sles/
-        dockerfile += <<~EOF
-          RUN zypper -n in openssh #{additional_packages.join(' ')}
-          RUN ssh-keygen -t rsa -f /etc/ssh/ssh_host_rsa_key
-          RUN ssh-keygen -t dsa -f /etc/ssh/ssh_host_dsa_key
-          RUN sed -ri 's/^#?UsePAM .*/UsePAM no/' /etc/ssh/sshd_config
-          RUN sed 's@session\s*required\s*pam_loginuid.so@session optional pam_loginuid.so@g' -i /etc/pam.d/*
-          EOF
+        dockerfile += <<~DF
+          RUN zypper -n in openssh #{additional_packages.join(' ')} \
+          && ssh-keygen -A \
+          && sed -ri 's/^#?UsePAM .*/UsePAM no/' /etc/ssh/sshd_config \
+          && sed 's@session\s*required\s*pam_loginuid.so@session optional pam_loginuid.so@g' -i /etc/pam.d/*
+        DF
       when /archlinux/
-        dockerfile += <<~EOF
-          RUN pacman --sync --refresh --noconfirm archlinux-keyring
-          RUN pacman --sync --refresh --noconfirm --sysupgrade
-          RUN pacman --sync --noconfirm #{additional_packages.join(' ')}
-          RUN ssh-keygen -A
-          RUN sed -ri 's/^#?UsePAM .*/UsePAM no/' /etc/ssh/sshd_config
-          RUN systemctl enable sshd
-          EOF
+        dockerfile += <<~DF
+          RUN pacman --sync --refresh --noconfirm archlinux-keyring \
+          && pacman --sync --refresh --noconfirm --sysupgrade \
+          && pacman --sync --noconfirm #{additional_packages.join(' ')} \
+          && ssh-keygen -A \
+          && sed -ri 's/^#?UsePAM .*/UsePAM no/' /etc/ssh/sshd_config \
+          && systemctl enable sshd
+        DF
       else
-        # TODO add more platform steps here
+        # TODO: add more platform steps here
         raise "platform #{host['platform']} not yet supported on docker"
       end
 
       # Make sshd directory, set root password
-      dockerfile += <<~EOF
-        RUN mkdir -p /var/run/sshd
-        RUN echo root:#{root_password} | chpasswd
-        EOF
+      dockerfile += <<~DF
+        RUN mkdir -p /var/run/sshd \
+        && echo root:#{root_password} | chpasswd
+      DF
 
       # Configure sshd service to allowroot login using password
       # Also, disable reverse DNS lookups to prevent every. single. ssh
       # operation taking 30 seconds while the lookup times out.
-      dockerfile += <<~EOF
-        RUN sed -ri 's/^#?PermitRootLogin .*/PermitRootLogin yes/' /etc/ssh/sshd_config
-        RUN sed -ri 's/^#?PasswordAuthentication .*/PasswordAuthentication yes/' /etc/ssh/sshd_config
-        RUN sed -ri 's/^#?UseDNS .*/UseDNS no/' /etc/ssh/sshd_config
-        EOF
-
+      # Also unbreak users with a bunch of SSH keys loaded in their keyring.
+      dockerfile += <<~DF
+        RUN sed -ri \
+        -e 's/^#?PermitRootLogin .*/PermitRootLogin yes/' \
+        -e 's/^#?PasswordAuthentication .*/PasswordAuthentication yes/' \
+        -e 's/^#?UseDNS .*/UseDNS no/' \
+        -e 's/^#?MaxAuthTries.*/MaxAuthTries 1000/' \
+        /etc/ssh/sshd_config
+      DF
 
       # Any extra commands specified for the host
-      dockerfile += (host['docker_image_commands'] || []).map { |command|
-        "RUN #{command}\n"
-      }.join('')
+      dockerfile += (host['docker_image_commands'] || []).map { |cmd| "RUN #{cmd}\n" }.join('')
 
       # Override image entrypoint
-      if host['docker_image_entrypoint']
-        dockerfile += "ENTRYPOINT #{host['docker_image_entrypoint']}\n"
-      end
+      dockerfile += "ENTRYPOINT #{host['docker_image_entrypoint']}\n" if host['docker_image_entrypoint']
 
       # How to start a sshd on port 22.  May be an init for more supervision
       # Ensure that the ssh server can be restarted (done from set_env) and container keeps running
-      cmd = host['docker_cmd'] || ["sh","-c","service #{service_name} start ; tail -f /dev/null"]
-      dockerfile += <<-EOF
+      cmd = host['docker_cmd'] || ['sh', '-c', "service #{service_name} start; tail -f /dev/null"]
+      dockerfile += <<~DF
         EXPOSE 22
         CMD #{cmd}
-      EOF
-
-    # end
+      DF
 
       @logger.debug("Dockerfile is #{dockerfile}")
-      return dockerfile
+
+      dockerfile
     end
 
     # a puppet run may have changed the ssh config which would
     # keep us out of the container.  This is a best effort to fix it.
     # Optionally pass in a host object to to determine which ssh
     # restart command we should try.
-    def fix_ssh(container, host=nil)
+    def fix_ssh(container, host = nil)
       @logger.debug("Fixing ssh on container #{container.id}")
-      container.exec(['sed','-ri',
-                      's/^#?PermitRootLogin .*/PermitRootLogin yes/',
-                      '/etc/ssh/sshd_config'])
-      container.exec(['sed','-ri',
-                      's/^#?PasswordAuthentication .*/PasswordAuthentication yes/',
-                      '/etc/ssh/sshd_config'])
-      container.exec(['sed','-ri',
-                      's/^#?UseDNS .*/UseDNS no/',
-                      '/etc/ssh/sshd_config'])
-      # Make sure users with a bunch of SSH keys loaded in their keyring can
-      # still run tests
-      container.exec(['sed','-ri',
-                     's/^#?MaxAuthTries.*/MaxAuthTries 1000/',
-                     '/etc/ssh/sshd_config'])
-
-      if host
-        if host['platform'] =~ /alpine/
-          container.exec(%w(/usr/sbin/sshd))
-        else
-          container.exec(%w(service ssh restart))
-        end
+      container.exec(['sed', '-ri',
+                      '-e', 's/^#?PermitRootLogin .*/PermitRootLogin yes/',
+                      '-e', 's/^#?PasswordAuthentication .*/PasswordAuthentication yes/',
+                      '-e', 's/^#?UseDNS .*/UseDNS no/',
+                      # Unbreak users with a bunch of SSH keys loaded in their keyring.
+                      '-e', 's/^#?MaxAuthTries.*/MaxAuthTries 1000/',
+                      '/etc/ssh/sshd_config',])
+
+      return unless host
+
+      case host['platform']
+      when /alpine/
+        container.exec(%w[/usr/sbin/sshd])
+      when /ubuntu/, /debian/
+        container.exec(%w[service ssh restart])
+      else
+        container.exec(%w[service sshd restart])
       end
     end
 
-
     # return the existing container if we're not provisioning
     # and docker_container_name is set
     def find_container(host)
@@ -679,6 +659,5 @@ module Beaker
     def in_container?
       return File.file?('/.dockerenv')
     end
-
   end
 end

data/lib/beaker-docker/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module BeakerDocker
-  VERSION = '1.3.0'
+  VERSION = '1.5.0'
 end

data/lib/beaker-docker.rb

@@ -0,0 +1 @@
+# frozen_string_literal: true