diff options
author | Steve Martinelli <stevemar@ca.ibm.com> | 2015-03-04 15:28:52 -0500 |
---|---|---|
committer | Steve Martinelli <stevemar@ca.ibm.com> | 2015-03-05 16:31:35 +0000 |
commit | d391fd1fef1b9a57ef11ad0c170ec9be6b63fa4f (patch) | |
tree | ab911c88fb61e329688a0e19b85d42b62af46c34 | |
parent | 66d732628b98fa8604b524d9ff35e8385e3a497b (diff) | |
download | oslo-policy-d391fd1fef1b9a57ef11ad0c170ec9be6b63fa4f.tar.gz |
deprecate policy_dirs option
It is not clear why this is a configurable option. The original
blueprint just wanted the ability to specify a policy.d infrastructure
to make complex policies easier to enforce. There was no requirement
on it being configurable.
Being configurable just exposes another way in which incompatibilities
in OpenStack projects might be able to emerge. As this merged late in
the incubator lifecycle for the policy code, there are likely few/no
existing users of the option so it should be safe to deprecate it
for the library. That way we won't break anyone currently using it,
and don't have to commit to supporting it going forward.
Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com>
Change-Id: I40bcbfbd4ab626fa9bfa050792e9ed3cfb587225
-rw-r--r-- | oslo_policy/opts.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/oslo_policy/opts.py b/oslo_policy/opts.py index d247e8a..af7d822 100644 --- a/oslo_policy/opts.py +++ b/oslo_policy/opts.py @@ -34,6 +34,7 @@ _options = [ help=_('Default rule. Enforced when a requested rule is not ' 'found.'), deprecated_group='DEFAULT'), + # NOTE(stevemar): Remove this option in the M cycle, refer to bug 1428332 cfg.MultiStrOpt('policy_dirs', default=['policy.d'], help=_('Directories where policy configuration files are ' @@ -43,7 +44,8 @@ _options = [ 'policy_file must exist for these directories to ' 'be searched. Missing or empty directories are ' 'ignored.'), - deprecated_group='DEFAULT'), + deprecated_group='DEFAULT', + deprecated_for_removal=True), ] |