Default mount name should be undef so it gets filled in the module

[ghost]

Otherwise it tries to mount the share by its title unless you have explicitly specified the path in the node manifest. And if you have, autogeneration of mount paths ("${nfs::client::nfs_v4_mount_root}/${share}") won't work.

[ghost]

Hmm, the build is failing and I'm struggling to find the reason. Is everything all right with Travis?

[dwerder]

The travis CI is red, because of failures in coding conventions.

manifests/client/debian/service.pp - WARNING: ensure found on line but it's not the first attribute on line 15
manifests/client/redhat/service.pp - WARNING: selector inside resource block on line 12
manifests/client/redhat/service.pp - WARNING: selector inside resource block on line 18
manifests/client/redhat/service.pp - WARNING: selector inside resource block on line 28

[dwerder]

But thats not a problem right now.

[dwerder]

I was wrong. It seems that travis has a problem with cloning stdlib for spec/fixtures. strange.

[ghost]

Yeah, I couldn't get to the bottom of it. Thanks for your quick reaction.