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.

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

readonly not showing correct information #3351

Closed
chjohnst opened this issue Apr 27, 2015 · 0 comments
Closed

readonly not showing correct information #3351

chjohnst opened this issue Apr 27, 2015 · 0 comments
Milestone

Comments

@chjohnst
Copy link

It looks like the readonly option is supported in ZFSoL, but the temporary information is not being reported correctly. Is this a known issue? Perhaps a similar issue related to #985?

ON (works but not reflecting the change in output):
zfs get readonly nas1.pool.0/test
NAME PROPERTY VALUE SOURCE
nas1.pool.0/test readonly off local
zfs set readonly=on nas1.pool.0/test
touch foo
touch: cannot touch `foo': Read-only file system
zfs get readonly nas1.pool.0/test
NAME PROPERTY VALUE SOURCE
nas1.pool.0/test readonly off temporary

OFF (works but reporting correctly):
zfs set readonly=off nas1.pool.0/test
touch foo
zfs get readonly canas1.pool.0/test
NAME PROPERTY VALUE SOURCE
nas1.pool.0/test readonly off local

behlendorf added a commit to behlendorf/zfs that referenced this issue Sep 1, 2015
Add the required kernel side infrastructure to parse arbitrary
mount options.  This enables us to support temporary mount
options is largely the same way it's handled on other platforms.

See the 'Temporary Mount Point Properties' section of zfs(8)
for complete details.

Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#985
Closes openzfs#3351
behlendorf added a commit to behlendorf/zfs that referenced this issue Sep 1, 2015
Add the required kernel side infrastructure to parse arbitrary
mount options.  This enables us to support temporary mount
options in largely the same way it is handled on other platforms.

See the 'Temporary Mount Point Properties' section of zfs(8)
for complete details.

Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#985
Closes openzfs#3351
behlendorf added a commit to behlendorf/zfs that referenced this issue Sep 1, 2015
Add the required kernel side infrastructure to parse arbitrary
mount options.  This enables us to support temporary mount
options in largely the same way it is handled on other platforms.

See the 'Temporary Mount Point Properties' section of zfs(8)
for complete details.

Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#985
Closes openzfs#3351
@behlendorf behlendorf added this to the 0.6.5 milestone Sep 3, 2015
tomgarcia pushed a commit to tomgarcia/zfs that referenced this issue Sep 11, 2015
Add the required kernel side infrastructure to parse arbitrary
mount options.  This enables us to support temporary mount
options in largely the same way it is handled on other platforms.

See the 'Temporary Mount Point Properties' section of zfs(8)
for complete details.

Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#985
Closes openzfs#3351
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants