#3214 Add description for permissions
Merged 2 years ago by tkopecek. Opened 2 years ago by jcupova.
jcupova/koji issue-335  into  master

No commits found

examples:
$ koji list-permissions
Permission name Description
admin Admin permission allow operations with admin only commands.
appliance Appliance permission allow appliance operation.
...
$ koji list-permissions --mine
Permission name
sign
admin

$ koji call getAllPerms
[{'description': 'Admin permission allow operations with admin only commands.',
'id': 1,
'name': 'admin'},
{'description': 'Appliance permission allow appliance operation.',
'id': 2,
'name': 'appliance'},
...
]

$ koji call getPerms
['sign', 'admin']

$ koji call getUserPerms userID=1
['sign', 'admin']

I can't find any use of this permissions anymore. Maybe we can drop it here?

"Image permission allows spawning image tasks"

"Livecd permissions allows spawning livecd tasks"

The default vm policy requires this permission to trigger Windows builds.

grant-permission --new should have an option to add also description of that new permission.
Furthermore, new editPermission(perm_id, description) call would be valuable as there is no way how to change the description.

rebased onto e754d25d4def9631e809176cb60198817cc635ac

2 years ago

rebased onto 61bd5710e7db0f0ca331c6a47b8b1ed1c283f33b

2 years ago

@tkopecek description for next perms added, grant-permission has option to add description (also for API) and created edit-permission and editPermission.

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

2 years ago

I edited the descriptions to be simpler, please take a look:

'admin', 'Full administrator access. Perform all actions.'
'appliance', 'Create appliance builds - deprecated.'
'dist-repo', 'Create a dist-repo.'
'host', 'Add, remove, enable, disable hosts and channels.'
'image', 'Start image tasks.'
'image-import', 'Import image archives.'
'livecd', 'Start livecd tasks.'
'maven-import', 'Import maven archives.'
'repo', 'Manage repos: newRepo, repoExpire, repoDelete, repoProblem.'
'sign', 'Import RPM signatures and write signed RPMs.'
'tag', 'Manage packages in tags: add, block, remove, and clone tags.'
'target', 'Add, edit, and remove targets.'
'win-admin', 'The default hub policy rule for "vm" requires this permission to trigger Windows builds.'
'win-import', 'Import win archives.'

rebased onto 329a15055a8d64e752afdf18ea0b62b12237fd5e

2 years ago

rebased onto 89397a3a78b33b003d34851361b705e3f66d84c8

2 years ago

Metadata Update from @jcupova:
- Pull-request untagged with: testing-ready

2 years ago

rebased onto e96a426cdff1bbf639702368b4bfe0ea8b8389df

2 years ago

rebased onto 69251ed

2 years ago

Metadata Update from @jcupova:
- Pull-request tagged with: testing-ready

2 years ago

Metadata Update from @jobrauer:
- Pull-request tagged with: testing-done

2 years ago

Commit cac6d20 fixes this pull-request

Pull-Request has been merged by tkopecek

2 years ago
Metadata