AA-2472 : Audit monitoring not showing any graph

Review Request #664 — Created Feb. 20, 2025 and submitted Feb. 24, 2025, 6:28 p.m.

Information
Owner:	pmurugaiyan
Repository:	AMP
Branch:	amp_3_7
Bugs:
Depends On:
Reviewers
Groups:
People:	apoorva.sn, prajesh, shuinvy

Description

Problem statement

AA-2472 : Audit monitoring not showing any graph

Fix

Earlier the Database query was made using the virtual server name which was not indexed. Now, I have updated the
logic to query using the server IP address.
RTT chart - Updated the index value from adc.responsetime to adc.duration to render the chart
SSL related charts - We don't have the SSL statistics as part of the APV system logs, so I have removed the SSL
related charts from the Audit logging section.
Processing Time charts - We don't have enough data to represent the client/server processing time for the requests
with AN_WELF log format. Hence, removed that as well.

Testing Done

The changes has been tested locally.

Files

Issues

Description	From	Last Updated
can you update the review with what the issue was and what changes was done to fix this bug Can …	apoorva.sn	Feb. 21, 2025, 10:07 p.m.
Do we require top 3 values - RTT P50.0, RTT P95.0, RTT P99.0 in "Client to Device Time" chart	apoorva.sn	Feb. 24, 2025, 4:34 p.m.
this line is commented. shld we uncomment ?	apoorva.sn	Feb. 24, 2025, 1:29 p.m.
Same as above	apoorva.sn	Feb. 24, 2025, 1:28 p.m.

Added Files:

Ship it!

```
Ship It!
```

Show all issues

can you update the review with what the issue was and what changes was done to fix this bug

Can we follow some pattern similar to

Eg:

Problem:

Solution:

Testing Done:

Description:

~		Audit monitoring not showing any graph
	~	Problem statement
	+
	+	AA-2472 : Audit monitoring not showing any graph
	+
	+	Fix
	+
	+	Earlier the Database query was made using the virtual server name which was not indexed. Now, I have updated the logic to query using the server IP address.
	+	RTT chart - Updated the index value from adc.responsetime to adc.duration to render the chart
	+	SSL related charts - We don't have the SSL statistics as part of the APV system logs, so I have removed the SSL related charts from the Audit logging section.
	+	Processing Time charts - We don't have enough data to represent the client/server processing time for the requests with AN_WELF log format. Hence, removed that as well.

branches/amp_3_7/extensions/monitoring/webui/resource/resource.service.js (Diff revision 1)

Show all issues

this line is commented. shld we uncomment ?

pmurugaiyan Feb. 24, 2025, 1:29 p.m.

These are comment from earlier also, we can perform the code cleanup later. I don't want to uncomment/remove this without the full picture.

branches/amp_3_7/extensions/monitoring/webui/resource/resource.service.js (Diff revision 1)
Show all issues
```
Same as above
```

Show all issues

Do we require top 3 values - RTT P50.0, RTT P95.0, RTT P99.0 in "Client to Device Time" chart

pmurugaiyan Feb. 24, 2025, 4:34 p.m.

Yes Apoorva, Its the earlier implementation. So we haven't made any changes on that. If required we can revisit the charts later during the UI revamp.

Ship it!

```
Ship It!
```

You have a pending review.

Review Board 5.0.5

AA-2472 : Audit monitoring not showing any graph

Problem statement

Fix

Added Files:

Description:

Problem statement

Fix

Status: Closed (submitted)