-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Support map types in auto.master #67
Conversation
README.md
Outdated
|
||
This Mapping describes the name and path of the autofs map file. | ||
This mapping is used in the auto.master generation, as well as generating the map | ||
file from the auto.map.erb template. This parameter is no longer required. | ||
When anything other than a simle file path isused `mapfile_manage` must be false. |
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.
misspelling ;)
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.
Two corrections there.
@@ -42,7 +51,7 @@ | |||
define autofs::mount ( | |||
Stdlib::Absolutepath $mount, | |||
Integer $order, | |||
Optional[Stdlib::Absolutepath] $mapfile = undef, | |||
Optional[Variant[Stdlib::Absolutepath,Autofs::Mapentry]] $mapfile = undef, |
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.
Should we add some logic here to to fail if type($mapfile) == Autofs::Mapentry && $mapfile_manage == true
?
I'm guessing on the syntax, but I could see that being a good idea.
Thoughts?
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.
Good idea $mapfile.is_a(Autofs::Mapentry)
is the magic.
@traylenator could you look into why the travis tests are failing as well? Thanks! |
It is desirable to be able to configure map types. /smb program:/etc/auto.smb ```puppet mount{'smb': mount => '/smb', mapfile => 'program:/etc/auto.smb', mapfile_manage => false, } ```
Failing tests was just somthing transient unrelated. |
It is desirable to be able to configure map types.
/smb program:/etc/auto.smb