oslo-incubator

policy enforcer does not delete cached policy file

Bug #1340843 reported by Valeriy Ponomaryov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo-incubator
Fix Released
Undecided
Valeriy Ponomaryov
oslo-incubator juno-2 "j2"

Bug Description

Policy ENFORCER's method 'clear' overwrites rules with empty dict and sets policy_path to None, but does not delete cached policy file, it is expected by docstring text.

https://github.com/openstack/oslo-incubator/blob/569979adfd4237869bb50c7aaa02bc3fd4f0b413/openstack/common/policy.py#L216

See original description
Valeriy Ponomaryov (vponomaryov)
description: updated
description: updated
Changed in oslo:
assignee: nobody → Valeriy Ponomaryov (vponomaryov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo-incubator (master)

Fix proposed to branch: master
Review: https://review.openstack.org/106433

Changed in oslo:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo-incubator (master)

Reviewed: https://review.openstack.org/106433
Committed: https://git.openstack.org/cgit/openstack/oslo-incubator/commit/?id=2b966f94de7367b5d0915d8e301b9b0e78d8c831
Submitter: Jenkins
Branch: master

commit 2b966f94de7367b5d0915d8e301b9b0e78d8c831
Author: vponomaryov <email address hidden>
Date: Fri Jul 11 19:08:58 2014 +0300

    Fix deletion of cached file for policy enforcer

    Policy ENFORCER's method 'clear' does not really
    delete cached file as written in its docstring.

    This commit adds 'delete_cached_file' to fileutils
    and makes 'clear' method of ENFORCER use it.

    Change-Id: I855d5f2001ac721249217d9fbd782c64af9db874
    Closes-Bug: #1340843

Changed in oslo:
status: In Progress → Fix Committed
Russell Bryant (russellb)
Changed in oslo:
milestone: none → juno-2
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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