4a09c2210b
The same policy rule (os_compute_api:os-server-groups) is being used for all actions (show, index, delete, create) for server_groups REST APIs. It is thus impossible to provide different RBAC for specific actions based on roles. To address this changes are made to have separate policy rules for each of action. It has been argued that index and show may not need separate policy rules, but most other places in nova (and OpenStack in general) do have separate policy rules for each action. This affords the ultimate flexibility to deployers, who can obviously use the same rule if that is what they want. One example where show and index may be different is that if show is restricted based on some criteria, such that a user is able to see some resources within the tenant but not others, then list would need to be disallowed to prevent the user from using list to see resources they cannot show. Change-Id: Ica9e07f6e80257902b4a0cc44b65fd6bad008bba Closes-Bug: #1636157
50 lines
1.5 KiB
Python
50 lines
1.5 KiB
Python
# Copyright 2016 Cloudbase Solutions Srl
|
|
# All Rights Reserved.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
from oslo_policy import policy
|
|
|
|
from nova.policies import base
|
|
|
|
|
|
BASE_POLICY_NAME = 'os_compute_api:os-server-groups'
|
|
POLICY_ROOT = 'os_compute_api:os-server-groups:%s'
|
|
BASE_POLICY_RULE = 'rule:%s' % BASE_POLICY_NAME
|
|
|
|
|
|
server_groups_policies = [
|
|
policy.RuleDefault(
|
|
name=POLICY_ROOT % 'discoverable',
|
|
check_str=base.RULE_ANY),
|
|
policy.RuleDefault(
|
|
name=BASE_POLICY_NAME,
|
|
check_str=base.RULE_ADMIN_OR_OWNER),
|
|
policy.RuleDefault(
|
|
name=POLICY_ROOT % 'create',
|
|
check_str=BASE_POLICY_RULE),
|
|
policy.RuleDefault(
|
|
name=POLICY_ROOT % 'delete',
|
|
check_str=BASE_POLICY_RULE),
|
|
policy.RuleDefault(
|
|
name=POLICY_ROOT % 'index',
|
|
check_str=BASE_POLICY_RULE),
|
|
policy.RuleDefault(
|
|
name=POLICY_ROOT % 'show',
|
|
check_str=BASE_POLICY_RULE),
|
|
]
|
|
|
|
|
|
def list_rules():
|
|
return server_groups_policies
|