New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(maint) Remove ruby requirement for creating certs #129
Conversation
cyberious
commented
Aug 19, 2015
- Add install_ca_certs for windows
- Install from forge instead of git
@@ -1,34 +1,26 @@ | |||
require 'beaker-rspec/spec_helper' | |||
require 'beaker-rspec/helpers/serverspec' | |||
require 'beaker/puppet_install_helper' | |||
require 'pry' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does pry need to be in the Gemfile ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed
- Add install_ca_certs for windows platforms - Install from forge instead of git - Increases ability to test across older versions of windows - No longer needs to search for ruby to generate pem files
@@ -28,7 +28,7 @@ group :system_tests do | |||
gem 'beaker-rspec', :require => false | |||
end | |||
gem 'serverspec', :require => false | |||
gem 'beaker-puppet_install_helper', :require => false | |||
gem 'beaker-puppet_install_helper', '~> 0.3', :require => false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this need to be bound this tightly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it needs to be at least 0.3 as that is when we added install_ca_certs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but seems like >= 0.3.0 would cause fewer future issues.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More so it protects us from what may happen in 1.0 but anything up until 1.0
(maint) Remove ruby requirement for creating certs