Sandesh message statistics request not displaying correct information

Bug #1552854 reported by Megh Bhatt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
High
Megh Bhatt
Trunk
Fix Committed
High
Megh Bhatt

Bug Description

Sandesh message statistics HTTP request on the client is not displaying correct information due to a bug in sandesh_statistics.cc. The code is using basic statistics map instead of detail statistics map when iterating to return detail stats.

Tags: analytics
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/18142
Submitter: Megh Bhatt (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/18142
Committed: http://github.org/Juniper/contrail-sandesh/commit/4393f943864a3e50292d4d2134c1f96b97b1f47e
Submitter: Zuul
Branch: master

commit 4393f943864a3e50292d4d2134c1f96b97b1f47e
Author: Megh Bhatt <email address hidden>
Date: Thu Mar 3 10:41:19 2016 -0800

Fix detail sandesh message statistics

1. Maintain only detail stats map in SandeshMessageStatistics, extract
basic stats from the detail stats on request/Get.
2. Add unit tests to verify both detail and basic stats

Change-Id: Ibc11b918b5be32463d826ccc5945b64bdd2b4365
Closes-Bug: #1552854

Changed in juniperopenstack:
milestone: r3.0-fcs → r3.1.0.0-fcs
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/18231
Submitter: Megh Bhatt (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/18231
Committed: http://github.org/Juniper/contrail-sandesh/commit/f22bde799706f6d51a3a1db8053839efb12458c8
Submitter: Zuul
Branch: R3.0

commit f22bde799706f6d51a3a1db8053839efb12458c8
Author: Megh Bhatt <email address hidden>
Date: Thu Mar 3 10:41:19 2016 -0800

Fix detail sandesh message statistics

1. Maintain only detail stats map in SandeshMessageStatistics, extract
basic stats from the detail stats on request/Get.
2. Add unit tests to verify both detail and basic stats

Conflicts:
 library/cpp/sandesh_statistics.cc

Change-Id: Ibc11b918b5be32463d826ccc5945b64bdd2b4365
Closes-Bug: #1552854
(cherry picked from commit 4393f943864a3e50292d4d2134c1f96b97b1f47e)

information type: Proprietary → Public
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.