Work on copy of registered rule instead of original object3.6.2

When service register their policy rule oslo policy does not
copy the rule and instead work on the original object.

- https://github.com/openstack/oslo.policy/blob/bd9d47aa36ad6f2f4746f09a267d7ce809a820f4/oslo_policy/policy.py#L1104

policy enforcer modify the default rules in
_handle_deprecated_rule().

- https://github.com/openstack/oslo.policy/blob/bd9d47aa36ad6f2f4746f09a267d7ce809a820f4/oslo_policy/policy.py#L767-L774

In any case, oslo policy should make copy of the registered
rules.

Another thing it fix is setting of flag
RuleDefault._deprecated_rule_handled.
Flag _deprecated_rule_handled is set to True when
_handle_deprecated_rule() is called irrespective of it
actually handle the deprecated rule and add it in OR checks.
We should set this flag when acutally deprecated rule is
handled so that if any condition change like config flag or
file rules we correctly handle deprecated rules.

Closes-Bug: #1914095
Closes-Bug: #1914592
Story: 2008556
Task: 41687
Change-Id: I154213dabd4d9eef760f0a4c9a852d504638ca8d

1 files changed, 7 insertions, 3 deletions

diff --git a/oslo_policy/policy.py b/oslo_policy/policy.py
index 046bec0..903f65b 100644
--- a/oslo_policy/policy.py
+++ b/oslo_policy/policy.py
@@ -747,6 +747,7 @@ class Enforcer(object):
                     self.rules[default.name] = self.file_rules[
                         deprecated_rule.name
                     ].check
+                    default._deprecated_rule_handled = True
 
         # In this case, the default check string is changing. We need to let
         # operators know that this is going to change. If they don't want to
@@ -767,12 +768,11 @@ class Enforcer(object):
             default.check = OrCheck([_parser.parse_rule(cs) for cs in
                                      [default.check_str,
                                       deprecated_rule.check_str]])
+            default._deprecated_rule_handled = True
             if not (self.suppress_deprecation_warnings
                     or self.suppress_default_change_warnings):
                 warnings.warn(deprecated_msg)
 
-        default._deprecated_rule_handled = True
-
     def _undefined_check(self, check):
         '''Check if a RuleCheck references an undefined rule.'''
         if isinstance(check, RuleCheck):
@@ -1101,7 +1101,11 @@ class Enforcer(object):
         if default.name in self.registered_rules:
             raise DuplicatePolicyError(default.name)
 
-        self.registered_rules[default.name] = default
+        # NOTE Always make copy of registered rule because policy engine
+        # update these rules in many places (one example is
+        # self._handle_deprecated_rule() ). This will avoid any conflict
+        # in rule object values when running tests in parallel.
+        self.registered_rules[default.name] = copy.deepcopy(default)
 
     def register_defaults(self, defaults):
         """Registers a list of RuleDefaults.