Too many 'Enforcing rule' logged on the gate

Bug #1385266 reported by Attila Fazekas
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Wishlist
Eugene Nikanorov

Bug Description

After #1356679 the debug logging became too verbose.
Logging the policy rule checking is useful when you are editing the policy.json, but for general usage it is too verbose.

http://logs.openstack.org/53/130753/1/check/check-tempest-dsvm-neutron-full/4ad9772/logs/screen-q-svc.txt.gz#_2014-10-24_11_44_41_049

$ wget http://logs.openstack.org/53/130753/1/check/check-tempest-dsvm-neutron-full/4ad9772/logs/screen-q-svc.txt.gz
$ wc -l screen-q-svc.txt.gz
94283 screen-q-svc.txt.gz
$ grep 'Enforcing rule' screen-q-svc.txt.gz | wc -l
25320

26.85% of log lines contains the 'Enforcing rule'.

These 'Enforcing rule' log messages should be disabled by default (even with debug=True).

Note:
Maybe 'default_log_levels' can be used for make it less verbose by default.

Changed in neutron:
assignee: nobody → Eugene Nikanorov (enikanorov)
importance: Undecided → Low
status: New → Confirmed
tags: added: low-hanging-fruit
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
status: Confirmed → In Progress
Changed in neutron:
importance: Low → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/130836
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=b9c200c13544f991681763f23b3a5adfb529e593
Submitter: Jenkins
Branch: master

commit b9c200c13544f991681763f23b3a5adfb529e593
Author: Eugene Nikanorov <email address hidden>
Date: Fri Oct 24 18:40:15 2014 +0400

    Decrease policy logging verbosity

    Log enforced rules only in case policy check has failed.

    Change-Id: I1fe8cbc1c9b5084b9cef6aa8329780512d8c7ec2
    Closes-Bug: #1385266

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → kilo-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: kilo-1 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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