-
-
Notifications
You must be signed in to change notification settings - Fork 35
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 CvmFS 2.11.0 parameters #186
Conversation
With the release of CvmFS 2.11.0 there are 7 new client parameters that can be set: For instance: ``` class{'cvmfs': cvmfs_cache_symlinks => 'yes', cvmfs_streaming_cache => 'no', cvmfs_statfs_cache_timeout => 10, cvmfs_world_readable => 'yes', cvmfs_cpu_affinity => [0,1,2], cvmfs_xattr_privileged_gids => [100,101,102], cvmfs_xattr_protected_xattrs => ['user.foo','user.bar'], } ``` * CvmFS 2.11.0 release notes https://cvmfs.readthedocs.io/en/stable/cpt-releasenotes.html * CvmFS parameters https://cvmfs.readthedocs.io/en/stable/apx-parameters.html
This is missing the |
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.
That looks good to me in general, just a few nits in the individual comments. Anf of course I second @jblomer 's question, it would be nice to add CVMFS_CACHE_REFCOUNT
while at it 👍 .
CVMFS_CACHE_REFCOUNT added. ( closed by accident) |
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.
Thanks, that looks good to me 👍 .
Pull Request (PR) description
With the release of CvmFS 2.11.0 there are 8 new client
parameters that can be set:
For instance:
Couple of extra commits fix some typos and lint correction on parameter order.