move default rule to policy section in keystone.conf

Bug #1276017 reported by Steve Martinelli
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Won't Fix
Wishlist
Unassigned
oslo-incubator
Won't Fix
Low
Unassigned

Bug Description

The following is currently in the keystone.conf file:

# Rule to check if no matching policy definition is found
# FIXME(dolph): This should really be defined as [policy] default_rule
# policy_default_rule = admin_required

As the comment suggest, we should move the config option to the policy section.
This will also impact oslo and we should ensure the old option is still supported for backwards compatibility.

Changed in keystone:
assignee: nobody → Steve Martinelli (stevemar)
Revision history for this message
Dolph Mathews (dolph) wrote :
Changed in keystone:
importance: Undecided → Low
status: New → In Progress
Changed in oslo:
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Adam Young (ayoung) wrote :

Not certain I agree. This splits the policy file, so that defaults are not honored if policy is fetched from a remote server. Granted, the Keystone policy will use its local one, but the rules should be the same for all, especially with the unified policyfile approach.

Changed in keystone:
importance: Low → Wishlist
Changed in oslo-incubator:
status: Triaged → Won't Fix
Revision history for this message
Steve Martinelli (stevemar) wrote :

these are two disjoint entities. One is for oslo_policy (the default rule), the other is for keystone's policy database (largely unused).

with the release of oslo.policy as it's own project, this relationship had been made more clear, and no longer needs to be grouped together as the initial bug report suggests.

Changed in keystone:
status: In Progress → Won't Fix
assignee: Steve Martinelli (stevemar) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.