mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
zfs_delegate_admin: drop choices from permissions (#2540)
instead of whitelisting some subset of known existing permissions, just allow any string to be used as permissions. this way, any permission supported by the underlying zfs commands can be used, eg. 'bookmark', 'load-key', 'change-key' and all property permissions, which were missing from the choices list.
This commit is contained in:
parent
345d5f2dfa
commit
dc0a56141f
2 changed files with 5 additions and 5 deletions
2
changelogs/fragments/2540-zfs-delegate-choices.yml
Normal file
2
changelogs/fragments/2540-zfs-delegate-choices.yml
Normal file
|
@ -0,0 +1,2 @@
|
|||
minor_changes:
|
||||
- zfs_delegate_admin - drop choices from permissions, allowing any permission supported by the underlying zfs commands (https://github.com/ansible-collections/community.general/pull/2540).
|
|
@ -51,8 +51,9 @@ options:
|
|||
permissions:
|
||||
description:
|
||||
- The list of permission(s) to delegate (required if C(state) is C(present)).
|
||||
- Supported permissions depend on the ZFS version in use. See for example
|
||||
U(https://openzfs.github.io/openzfs-docs/man/8/zfs-allow.8.html) for OpenZFS.
|
||||
type: list
|
||||
choices: [ allow, clone, create, destroy, diff, hold, mount, promote, readonly, receive, release, rename, rollback, send, share, snapshot, unallow ]
|
||||
elements: str
|
||||
local:
|
||||
description:
|
||||
|
@ -248,10 +249,7 @@ def main():
|
|||
users=dict(type='list', elements='str'),
|
||||
groups=dict(type='list', elements='str'),
|
||||
everyone=dict(type='bool', default=False),
|
||||
permissions=dict(type='list', elements='str',
|
||||
choices=['allow', 'clone', 'create', 'destroy', 'diff', 'hold', 'mount', 'promote',
|
||||
'readonly', 'receive', 'release', 'rename', 'rollback', 'send', 'share',
|
||||
'snapshot', 'unallow']),
|
||||
permissions=dict(type='list', elements='str'),
|
||||
local=dict(type='bool'),
|
||||
descendents=dict(type='bool'),
|
||||
recursive=dict(type='bool', default=False),
|
||||
|
|
Loading…
Reference in a new issue