summaryrefslogtreecommitdiff
path: root/oslo_policy/opts.py
diff options
context:
space:
mode:
authorSteve Martinelli <stevemar@ca.ibm.com>2015-03-04 15:28:52 -0500
committerSteve Martinelli <stevemar@ca.ibm.com>2015-03-05 16:31:35 +0000
commitd391fd1fef1b9a57ef11ad0c170ec9be6b63fa4f (patch)
treeab911c88fb61e329688a0e19b85d42b62af46c34 /oslo_policy/opts.py
parent66d732628b98fa8604b524d9ff35e8385e3a497b (diff)
downloadoslo-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
Diffstat (limited to 'oslo_policy/opts.py')
-rw-r--r--oslo_policy/opts.py4
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),
]