Support configuration repositories when using mount

[traylenator]

Pull Request (PR) description

This patch only comes into play when using the mount_method of mount rather than the default autofs.

It is now possible to specify that one particular repository on a client is the configuration repository and must be mounted first before all other repositories. This should happen during puppets initial configuration of CvmFS and also at reboot.

class{'cvmfs':
  mount_method => 'mount',
  config_repo  => 'cvmfs-config.example.org',
}
cmvfs::mount{'myrepo.example.org':}
cmvfs::mount{'cvmfs-config.example.org':}

In this example the repository cvmfs-config.example.org will be mounted before myrepo.example.org.

Note there is all ways at most one configuration repository per client.

This Pull Request (PR) fixes the following issues

Fixes: #176

In addition a couple of parameters to cvmfs::mount are made bettter:

• mount_method to cvmfs::mount never made sense and setting it is pointless unless also set in the main class.
  deprecate the parameter. It will still be accepted now but is redundant. It was already documented as 'DO NOT SET'.
• mount_options now defaults to an Array of options but a String is still accepted for now.

[olifre]

👍
We have also hit this, and for now, we've used:

        # always mount config repository containing information about other repositories
        cvmfs::mount { 'cvmfs-config.cern.ch': }

        # mount requested repositories
        cvmfs::mount { $mounts:
                # ensure cvmfs-config.cern.ch is mounted first,
                # see https://cvmfs.readthedocs.io/en/stable/cpt-configure.html#mounting
                mount_options   => 'defaults,_netdev,nodev,x-systemd.requires-mounts-for=/cvmfs/cvmfs-config.cern.ch',
                require         => Cvmfs::Mount['cvmfs-config.cern.ch'],
        }

Having this in the module proper is a good improvement.

[olifre]

This looks great, I also really like the two improvements (mount_options as array and deprecation of mount_method on cvmfs::mount.
I only added two small nits, but these should not prevent this from being merged 😉 .

[olifre]

Thanks, that lookgs good to me, nice to finally have all occurrences of CVMFS consistent 👍 .