Skip to content
This repository has been archived by the owner on Mar 8, 2023. It is now read-only.

Ability to set owner & group on directory within a server::export resource #30

Closed
JoshiiSinfield opened this issue Jul 29, 2015 · 8 comments
Closed

Ability to set owner & group on directory within a server::export resource #30

JoshiiSinfield opened this issue Jul 29, 2015 · 8 comments

Comments

@JoshiiSinfield
Copy link

Hi,

We have the need to set the owner&group on the folder location for server::export folder, however server::export doesn't have parameters to set owner & group.

I notice that nfs::mkdir has params to support it.

Cheers,
Josh
@JoshiiSinfield
Copy link
Author

I'll push a patchset up - just wondering if you have any views/preferences on how it should be done or if simply adding parameters to server::export and passing through to nfs::mkdir will suffice?

Josh.

@JoshiiSinfield JoshiiSinfield changed the title Ability to set owner & group on a server::export resource Ability to set owner & group on directory within a server::export resource Jul 29, 2015
@dwerder
Copy link
Member

dwerder commented Jul 29, 2015

I have no preference. Sounds ok to me. Waiting for the PR.

@dwerder
Copy link
Member

dwerder commented Jul 29, 2015

Please add an example in the README.

JoshiiSinfield pushed a commit to CDLSoftware/puppet-nfs that referenced this issue Jul 29, 2015
Issue echocat#30 Add ability to set owner,group,perms on server::export
ac2b4b8 
When using the nfs::server::export resource it creates a bindmount resource,
which in turn creates a nfs::mkdir resource. Adding these parameters and
passing them through provides the ability to set owner, group and mode
metaparams on the file resource within nfs::mkdir from a server::export resource.
dwerder added a commit that referenced this issue Jul 29, 2015
@dwerder
Merge pull request #31 from CDLSoftware/issue30_setOwnershipThroughEx…
89a4ee9 
…port

Issue #30 Add ability to set owner,group,perms on server::export
@JoshiiSinfield JoshiiSinfield mentioned this issue Jul 29, 2015
Merged
@InfernoZeus
Copy link

Any chance of a release including this patch?

@dwerder
Copy link
Member

dwerder commented Sep 25, 2015

I will create the next puppetforge release next week. Until then u can use the github master

@InfernoZeus
Copy link

Awesome, thanks! :)

On Fri, 25 Sep 2015 15:13 Daniel Werdermann notifications@github.com
wrote:

I will create the next puppetforge release next week. Until then u can use
the github master

—
Reply to this email directly or view it on GitHub
#30 (comment).

@dwerder
Copy link
Member

dwerder commented Sep 29, 2015

Version 1.6.0 is released: https://forge.puppetlabs.com/echocat/nfs

@dwerder dwerder closed this as completed Sep 29, 2015
@InfernoZeus
Copy link

Awesome, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@InfernoZeus @dwerder @JoshiiSinfield