Bug 866677

Summary: [RFE] Improved broker logging for expired messages
Product: Red Hat Enterprise MRG Reporter: Jason Dillaman <jdillama>
Component: qpid-cppAssignee: Chuck Rolke <crolke>
Status: CLOSED ERRATA QA Contact: Leonid Zhaldybin <lzhaldyb>
Severity: unspecified Docs Contact:
Priority: medium    
Version: DevelopmentCC: esammons, jross, lzhaldyb, mcressma
Target Milestone: 2.3Keywords: FutureFeature, Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: qpid-cpp-0.18-4 Doc Type: Enhancement
Doc Text:
Feature: Improved broker logging for expired messages Reason: Messages that are expired by the queue cleaner are not logged as they used to be. Logging should include the queue name and the message properties. Result (if any): Logging is moved to a common place in the source module so all expired messages receive the same logging treatment. The queue name and message properties are included in debug-level log entries.
 Story Points: ---
Clone Of:
: 1339449 (view as bug list) Environment:
Last Closed: 2013-03-06 18:52:16 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 698367, 1339449    
Attachments:
Description Flags
Proposed patch none

Description Jason Dillaman 2012-10-15 20:51:42 UTC 
It would be desirable to have improved logging from the broker when messages are expired from queues.  The desired log message would include at least the queue name and message properties from the message being removed so that it is possible to debug any potential reason why a particular message expired.  Additionally, the queue cleaner previously logged "Message expired from queue 'xyz'", but that functionality was removed during a recent refactor.

Version-Release number of selected component (if applicable):
Qpid 0.18
Comment 1 Jason Dillaman 2012-10-15 21:23:09 UTC 
Created attachment 627709 [details]
Proposed patch

Moved logging of expired messages to a common function to support log filtering and dumped the message properties of the expired message.
Comment 7 Leonid Zhaldybin 2012-11-22 10:48:05 UTC 
Tested on RHEL5.9 and RHEL6.3 (both i386 and x86_64). The broker logs messages expiration and the log contains queue name from which the expired message was deleted along with message's properties.

Packages used for testing:

RHEL5.9
python-qpid-0.18-4.el5
python-qpid-qmf-0.18-9.el5
qpid-cpp-client-0.18-10.el5
qpid-cpp-client-devel-0.18-10.el5
qpid-cpp-server-0.18-10.el5
qpid-cpp-server-devel-0.18-10.el5
qpid-qmf-0.18-9.el5
qpid-tools-0.18-7.el5

RHEL6.3
python-qpid-0.18-4.el6.noarch
python-qpid-qmf-0.18-9.el6_3.i686
qpid-cpp-client-0.18-10.el6_3.i686
qpid-cpp-client-devel-0.18-10.el6_3.i686
qpid-cpp-server-0.18-10.el6_3.i686
qpid-cpp-server-devel-0.18-10.el6_3.i686
qpid-qmf-0.18-9.el6_3.i686
qpid-tools-0.18-7.el6_3.noarch

-> VERIFIED
Comment 9 errata-xmlrpc 2013-03-06 18:52:16 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHSA-2013-0561.html