Skip to content
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

[MODULES-4598] Revert "Revert "fix default params for SUSE family OSes"" #863

Merged
merged 2 commits into from Apr 14, 2017
Merged

[MODULES-4598] Revert "Revert "fix default params for SUSE family OSes"" #863

merged 2 commits into from Apr 14, 2017

Conversation

mmoll
Copy link
Contributor

@mmoll mmoll commented Apr 13, 2017

This reverts commit 63ad343, which reverted
commit b6fbbb3.

@mmoll
Copy link
Contributor Author

mmoll commented Apr 13, 2017

@wilson208 @eputnam
this is working fine for me on SLES 12 SP2

@eputnam
Copy link
Contributor

eputnam commented Apr 13, 2017

@mmoll I'm going to run this through our CI again and try to see what was up.

@@ -111,10 +111,10 @@
'Suse' => $::operatingsystem ? {
'SLES' => $::operatingsystemrelease ? {
/11\.[0-4]/ => '91',
default => '93',
default => '94',
Copy link
Contributor

@eputnam eputnam Apr 14, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mmoll turns out we need a separate condition. SLES 12.0 needs to stay at 93 but SLES >=12.2 works with 94.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see.. I'll whip something up

@mmoll
Copy link
Contributor Author

mmoll commented Apr 14, 2017

I checked on the 12SP1 DVD that SP1 already has postgresql94, the second commit should fix that

@eputnam
Copy link
Contributor

eputnam commented Apr 14, 2017

thanks @mmoll !

@eputnam eputnam merged commit bb3b054 into puppetlabs:master Apr 14, 2017
@mmoll mmoll deleted the suse_defaults branch April 14, 2017 21:39
@Ramesh7 Ramesh7 added the bugfix label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants