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.

Sign up for GitHub

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

Jump to bottom

No option to remove an already defined autofs::mount #91

Closed
danyel2014 opened this issue Jan 16, 2018 · 3 comments · Fixed by #95
Closed

No option to remove an already defined autofs::mount #91

danyel2014 opened this issue Jan 16, 2018 · 3 comments · Fixed by #95
Assignees
@dhollinger
Labels
bug Something isn't working

Comments

@danyel2014
Copy link

Dear developers,

today I just wanted to remove an existing autofs::mount, but, there is no ensure => absent option available. The only workaround I found was to set mapcontents => [], but this doesn't remove the entry in auto.master and also the mapfile is not removed.

Would be possible to extend the module having in mind also the removal for already defined autofs::mount entries ?

Thank you
Dan
@danyel2014 danyel2014 changed the title No option to remove and already defined autofs::mount No option to remove an already defined autofs::mount Jan 16, 2018
@dhollinger
Copy link
Member

dhollinger commented Feb 6, 2018 •
edited
Loading

I'll take a look at this week.

@dhollinger dhollinger added the bug Something isn't working label Feb 12, 2018
@dhollinger
Copy link
Member

This is definitely a bug as it was intended to be able to remove mounts and maps as well

@dhollinger dhollinger self-assigned this Feb 12, 2018
@dhollinger dhollinger mentioned this issue Feb 21, 2018
Merged
@dhollinger
Copy link
Member

@danyel2014 added enhancements. See #95 for the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dhollinger dhollinger

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add ability to remove mounts and maps. dhollinger/puppet-autofs
2 participants
@dhollinger @danyel2014