Skip to content

Commit

Permalink
Update tests for new meaning of xattr=on (#14661)
Browse files Browse the repository at this point in the history
In OpenZFS 2.3 the meaning of xattr=on changed from dir to sa
which means that "ON" is now the expected value for the top-level
dataset of a new zpool and for docker datasets.
  • Loading branch information
anodos325 authored Oct 11, 2024
1 parent 00dfd2c commit e01b813
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
'overlay': 'on',
'setuid': 'on',
'snapdir': 'hidden',
'xattr': 'sa',
'xattr': 'on',
}


Expand Down
2 changes: 1 addition & 1 deletion tests/api2/test_006_pool_and_sysds.py
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,7 @@ def test__check_root_level_dataset_properties():
ds = call('pool.dataset.get_instance', pool_name)
assert ds['acltype']['value'] == 'POSIX'
assert ds['aclmode']['value'] == 'DISCARD'
assert ds['xattr']['value'] == 'SA'
assert ds['xattr']['value'] == 'ON'
assert ds['deduplication']['value'] == 'OFF'
assert ds['casesensitivity']['value'] == 'SENSITIVE'
assert ds['compression']['value'] == 'LZ4'
Expand Down

0 comments on commit e01b813

Please sign in to comment.