commit 97c42afccc32042ef3be4c4c4ebf01196d36633f
author Zuul <zuul@review.openstack.org> Tue Jan 30 05:56:52 2018 +0000
committer Gerrit Code Review <review@openstack.org> Tue Jan 30 05:56:52 2018 +0000
tree a70573f3cbbe6ccfd379ef4e83f12a4b3a4f9d09
parent 233b943a3c8a8b03407137ef3e4845cc62521801
parent 1882e9bffc09fe20dcc186354883c707dc995251

Merge "override_role cleanup: Remove superfluous call in rbac_rule_validation"

patrole_tempest_plugin/rbac_rule_validation.py

diff --git a/patrole_tempest_plugin/rbac_rule_validation.py b/patrole_tempest_plugin/rbac_rule_validation.py
index 75d1baa..daf03e4 100644
--- a/patrole_tempest_plugin/rbac_rule_validation.py
+++ b/patrole_tempest_plugin/rbac_rule_validation.py
@@ -175,10 +175,6 @@
                         "OverPermission: Role %s was allowed to perform %s" %
                         (role, rule))
             finally:
-                # TODO(felipemonteiro): Remove the call below once all the
-                # tests have migrated over to `override_role` public method.
-                test_obj.rbac_utils._override_role(test_obj,
-                                                   toggle_rbac_role=False)
                 if CONF.patrole_log.enable_reporting:
                     RBACLOG.info(
                         "[Service]: %s, [Test]: %s, [Rule]: %s, "