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

Add support for SUSE SLE12 #28

Merged
merged 1 commit into from
Jun 24, 2015
Merged

Conversation

jwennerberg
Copy link

No description provided.

@ghoneycutt
Copy link
Owner

@jwennerberg Thanks!

I don't have ready access to Suse 12. Has this code been tested on that platform?

@ghoneycutt
Copy link
Owner

@Phil-Friderici You have any Suse 12 systems around by chance?

@ghoneycutt
Copy link
Owner

The metadata.json and README also need to be updated. I can do that, just a note so I don't forget.

@Phil-Friderici
Copy link

@ghoneycutt I'll ask our Suse 12 testing specialists to help us.

@ofsole
Copy link

ofsole commented Jun 10, 2015

Hey guys, I tested this module on our SLES12 servers, it works great, in addition, there may be one line could be updated, you can check the link here:
https://github.com/jwennerberg/puppet-module-nscd/blob/sle12/manifests/init.pp#L172

@Phil-Friderici
Copy link

@ofsole Thanks a lot for testing and the additional hint :)

@ghoneycutt
Copy link
Owner

Thanks @jwennerberg @Phil-Friderici @ofsole !!

Could you please squash this commits and I'll get them merged

@jwennerberg
Copy link
Author

Done! =)

ghoneycutt added a commit that referenced this pull request Jun 24, 2015
Add support for SUSE SLE12
@ghoneycutt ghoneycutt merged commit 98a6a40 into ghoneycutt:master Jun 24, 2015
@ghoneycutt
Copy link
Owner

Released in v1.4.0

Thanks everyone!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants