Skip to content
This repository has been archived by the owner on Jun 11, 2019. It is now read-only.

nfs service name broken in centos < 7 #6

Open
koglinjg opened this issue May 1, 2015 · 1 comment
Open

nfs service name broken in centos < 7 #6

koglinjg opened this issue May 1, 2015 · 1 comment

Comments

@koglinjg
Copy link
Contributor

koglinjg commented May 1, 2015

I see issue #3 is to fix the service name for centos 7, however it is now broken for < 7. I would recommend a structure like https://github.com/puppetlabs/puppetlabs-java/blob/1.3.0/manifests/params.pp so you can apply version specific service and package names. I wouldn't mind making a pull request. Just let me know that is a structure that looks ok to you.

My current workaround while using your module is to use

class { '::nfs':
  ...
  service => 'nfs'
  ...
}

There being an easy workaround I would consider this a minor bug.

@alvagante
Copy link
Member

Yes, that approach would be ok. A PR would be very welcomed, thank you

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

No branches or pull requests

2 participants