Jump to: navigation, search

Difference between revisions of "Cyborg/Policy"

Line 27: Line 27:
 
| rowspan="2" | /v2/accelerator_requests/{arq_uuid} || GET || x ||  ||  ||  ||  ||  ||  || admin_or_owner || cyborg:arq:get_one ||  
 
| rowspan="2" | /v2/accelerator_requests/{arq_uuid} || GET || x ||  ||  ||  ||  ||  ||  || admin_or_owner || cyborg:arq:get_one ||  
 
|-
 
|-
| PATCH ||  || || x(admin_or_owner) ||  ||  ||  ||  || admin_or_owner || cyborg:arq:update ||  
+
| PATCH ||  || x || x(admin_or_owner) ||  ||  ||  ||  || admin_or_owner || cyborg:arq:update ||  
 
|-
 
|-
| /v2/accelerator_requests?arqs={arq_uuid} || DELETE ||  || || x(admin_or_owner) ||  ||  ||  ||  || admin_or_owner || cyborg:arq:delete ||
+
| /v2/accelerator_requests?arqs={arq_uuid} || DELETE ||  || x || x(admin_or_owner) ||  ||  ||  ||  || admin_or_owner || cyborg:arq:delete ||
 
|-
 
|-
| /v2/accelerator_requests?instance={instance_uuid} || DELETE ||  || || x(admin_or_owner) ||  ||  ||  ||  || admin_or_owner || cyborg:arq:delete ||
+
| /v2/accelerator_requests?instance={instance_uuid} || DELETE ||  || x || x(admin_or_owner) ||  ||  ||  ||  || admin_or_owner || cyborg:arq:delete ||
 
|-
 
|-
 
| rowspan="3" | /v2/devices || GET ||  ||  ||  || x ||  ||  ||  || any_user || cyborg:device:get_all ||
 
| rowspan="3" | /v2/devices || GET ||  ||  ||  || x ||  ||  ||  || any_user || cyborg:device:get_all ||

Revision as of 08:18, 11 February 2020

Project-scope(new policy) System-scope(new policy)
Route Method reader member admin reader member admin no auth legacy policy RBAC Name Notes
/ GET x N/A N/A No restrictions on this route
/v2 GET x N/A N/A No restrictions on this route
/v2/device_profiles GET x admin_or_owner cyborg:device_profile:get_all
POST x admin cyborg:device_profile:create
/v2/device_profiles/{device_profiles_uuid} GET x admin_or_owner cyborg:device_profile:get_one
DELETE x admin_or_owner cyborg:device_profile:delete
/v2/device_profiles?value={device_profile_name1},{device_profile_name2} DELETE x admin_or_owner cyborg:device_profile:delete
/v2/accelerator_requests GET x admin_or_owner cyborg:arq:get_all
POST x any_user cyborg:arq:create current rule: any role is allowed to do post action.This is too permissive,instead it should be at least "role:member" with scope_type ["project"]
/v2/accelerator_requests/{arq_uuid} GET x admin_or_owner cyborg:arq:get_one
PATCH x x(admin_or_owner) admin_or_owner cyborg:arq:update
/v2/accelerator_requests?arqs={arq_uuid} DELETE x x(admin_or_owner) admin_or_owner cyborg:arq:delete
/v2/accelerator_requests?instance={instance_uuid} DELETE x x(admin_or_owner) admin_or_owner cyborg:arq:delete
/v2/devices GET x any_user cyborg:device:get_all
GET x any_user cyborg:device:get_one
PATCH x N/A cyborg:device:update Disable or Enable the specified device
/v2/deployables/{uuid} PATCH x N/A cyborg:deployable:update Update the shell image/FPGA bitstream to custom user logic for the specified deployable.

Reference:

Questions (tied to RBAC Name):

  • cyborg:arq:create
    • (Yumeng) current rule: any role is allowed to do post action. This is too permissive, instead, it should be at least "role:member" with scope_type ["project"]
  • cyborg:device:get_all
  • cyborg:device:update
    • (Yumeng) Is it necessary to allow a system-scope user to read and update one device? For example, when one device is shared by different projects, we should allow a role at a system-scope level to access this device, right?
      • (Yumeng) yes, we agreed on the weekly meeting that a device is a system-level resource, so sys_admin is required for device: update.

http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-02-06-03.01.log.html#l-143

  • cyborg:deployable:update
    • (Yumeng) ditto for deployable update
      • (Yumeng) we agreed that deployable: update requires project_admin.

http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-02-06-03.01.log.html#l-143