-
Notifications
You must be signed in to change notification settings - Fork 2
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
IT-1686: add NFS module, nfs profile #25
Conversation
Puppetfile
Outdated
@@ -38,6 +38,8 @@ mod 'puppetlabs/mailalias_core', '1.0.5' | |||
mod 'puppetlabs/docker', '3.8.0' | |||
mod 'puppetlabs/reboot', '2.2.0' | |||
mod 'puppet/python', '3.0.1' | |||
mod 'derdanne/nfs', '2.1.2' | |||
mod 'puppet-posix_acl', '1.0.0' |
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.
This is a nit-pick but I would like to consistently use /
or -
as the separator.
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.
I've reverted this commit, please take another look.
ca60579
to
02f965a
Compare
02f965a
to
f25e127
Compare
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.
LGTM (but I am the original author...)
This pull request adds an NFS module for managing nfs exports and mounts, and
adds in some shims in our profile module to define NFS exports and mounts in
hiera.
I propose that we fork the derdanne/nfs module, apply some fixups, and inject
the hiera resource creation directly into the module. We can push these changes
upstream.