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

to include NIS/LDAP map #142

Open
xm1234567 opened this issue Feb 19, 2019 · 5 comments
Open

to include NIS/LDAP map #142

xm1234567 opened this issue Feb 19, 2019 · 5 comments

Comments

@xm1234567
Copy link

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet:
  • Ruby:
  • Distribution:
  • Module version:

How to reproduce (e.g Puppet code you use)

What are you seeing

Hello, what I am asking is not an issue. I am testing how to add NIS/LDAP maps:
To add the following two lines in /etc/auto.master:
/nethome ldap:nisMapName=auto.nethome,dc=mydom,dc=com
/net yp:auto.net

I use :
include autofs
autofs::mount { 'add_nis_map':
ensure => 'present',
mount => '/net',
mapfile => 'yp:auto.net',
}
autofs::mount { 'ldap_map':
ensure => 'present',
mount => '/nethome',
mapfile => 'ldap:nisMapName=auto.nethome,dc=mydom,dc=com',
}

It is working well.

But if I want to add just one line '+auto.master' in /etc/auto.master is not possible. Do you plan to add this possibility in the future or not?

Thanks

What behaviour did you expect instead

Output log

Any additional information you'd like to impart

@bastelfreak
Copy link
Member

I don't think that we've specific plans for it. Do you want to provide a PR for it?

@xm1234567
Copy link
Author

Sorry for the late reply. I am not ready to provide a PR yet :)
I am wondering, maybe just using "mapfile => 'yp:auto.net'," is an alternative way to add specific NIS map, although it is not exactly as '+auto.master'.

Before I was using this module ( https://forge.puppet.com/pdxcat/autofs ) to add our NIS maps auto.master; which has not been updated for 3 years. So I plan to change to your module.

Now we are changing from NIS to IDM and LDAP, probably we will use ldap in the future, not use +auto.master. But during my test, I notice this, thinking maybe someone else could have this need. That is why I pose this question.

@hubizx
Copy link

hubizx commented Mar 14, 2020

hi, Almost started using voxpupuli/puppet-autofs, but missing support +auto.master stopped me doing that.

@xm1234567
Copy link
Author

maybe you can provide an patch, but please make ii as an option, not mandatory in the conf.

@erobhal
Copy link
Contributor

erobhal commented Jul 28, 2022

Pull request for this feature has been sent.

#183

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

No branches or pull requests

4 participants