-
Notifications
You must be signed in to change notification settings - Fork 48
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) Use correct providers to install gems #47
(maint) Use correct providers to install gems #47
Conversation
on host, puppet('resource package hocon provider=puppet_gem') | ||
end | ||
else | ||
on host, puppet('resource package hocon provider=gem') |
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.
right above this...
on host, puppet('resource package rubygems')
07d6416
to
831a19d
Compare
@@ -5,7 +5,18 @@ | |||
unless ENV['RS_PROVISION'] == 'no' | |||
run_puppet_install_helper | |||
hosts.each do |host| | |||
on host, puppet('resource package hocon provider=puppet_gem') | |||
puppet_version = (on default, puppet('--version')).output.chomp |
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.
Sorry @HAIL9000 this whole block needs to be
if ENV['PUPPET_INSTALL_TYPE'] == 'pe' and Gem::Version.new(puppet_version) < Gem::Version.new('4.0.0')
on host, puppet('resource package hocon provider=pe_gem')
elsif Gem::Version.new(puppet_version) >= Gem::Version.new('4.0.0')
on host, puppet('resource package hocon provider=puppet_gem')
else
on host, puppet('resource package hocon provider=gem')
end
831a19d
to
9b7b508
Compare
Prior to this commit, the test setup was using the same gem provider for PE and FOSS and all versions. Correct this so that it using the correct provider with certain versions of puppet and PE.
9b7b508
to
a06d600
Compare
@bmjen Updated! Looks like we're finally green! |
Prior to this commit, the test setup was using the same gem
provider for PE and FOSS and all versions. Correct this so that
it using the correct provider with certain versions of puppet and
PE.