From df40ad7c8dcffb3394fac0d9a7e2e1bb388ed90d Mon Sep 17 00:00:00 2001 From: Colleen Murphy Date: Fri, 22 May 2015 14:49:43 -0700 Subject: [PATCH] Unpin beaker-rspec The latest version of beaker tries to manage sshd_config[1] in order to pass through environment variables. JJB also tries to manage sshd_config by adding Match blocks to securely allow root SSH[2]. When beaker tries to append the PermitUserEnvironment line after the Match blocks, SSH fails to open port 22 and beaker can't SSH in to the node[3]. This patch unpins beaker-rspec and adds set_env: false to the nodepool nodesets so that the set_env method is never called and does not try to mess with the sshd_config. [1] https://github.com/puppetlabs/beaker/blob/master/lib/beaker/host_prebuilt_steps.rb#L528 [2] http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/puppet-module-jobs.yaml#n42 [3] http://logs.openstack.org/61/185161/1/check/gate-puppet-nova-puppet-beaker-rspec-dsvm-centos7/cd81608/console.html.gz#_2015-05-22_22_05_46_933 [4] https://github.com/puppetlabs/beaker/blob/master/lib/beaker/host_prebuilt_steps.rb#L502 Depends-on: Iefdf96325e11b470e864d5386a3eab0eb655c960 Change-Id: Ic5e44397df5558bb296646b666373ee1dc2343b0 --- Gemfile | 3 +-- spec/acceptance/nodesets/nodepool-centos7.yml | 1 + spec/acceptance/nodesets/nodepool-trusty.yml | 1 + 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/Gemfile b/Gemfile index 4c65317ac..3c0b87530 100644 --- a/Gemfile +++ b/Gemfile @@ -3,7 +3,6 @@ source 'https://rubygems.org' group :development, :test do gem 'puppetlabs_spec_helper', :require => false gem 'rspec-puppet', '~> 2.1.0', :require => false - gem 'minitest', '~> 4.7', :require => 'minitest/unit' gem 'metadata-json-lint' gem 'puppet-lint-param-docs' @@ -17,7 +16,7 @@ group :development, :test do gem 'puppet-lint-variable_contains_upcase' gem 'puppet-lint-numericvariable' - gem 'beaker-rspec', '~> 2.2.4', :require => false + gem 'beaker-rspec', :require => false gem 'json' gem 'webmock' end diff --git a/spec/acceptance/nodesets/nodepool-centos7.yml b/spec/acceptance/nodesets/nodepool-centos7.yml index aed4f0b26..575ae6732 100644 --- a/spec/acceptance/nodesets/nodepool-centos7.yml +++ b/spec/acceptance/nodesets/nodepool-centos7.yml @@ -7,3 +7,4 @@ HOSTS: ip: 127.0.0.1 CONFIG: type: foss + set_env: false diff --git a/spec/acceptance/nodesets/nodepool-trusty.yml b/spec/acceptance/nodesets/nodepool-trusty.yml index a2c1ecc63..a95d9f38d 100644 --- a/spec/acceptance/nodesets/nodepool-trusty.yml +++ b/spec/acceptance/nodesets/nodepool-trusty.yml @@ -7,3 +7,4 @@ HOSTS: ip: 127.0.0.1 CONFIG: type: foss + set_env: false