RDKB-63427 "WIFI_RXDELTA_17" is not available in wifihealth.txt#942
RDKB-63427 "WIFI_RXDELTA_17" is not available in wifihealth.txt#942gsathish86 merged 3 commits intordkcentral:developfrom
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
There was a problem hiding this comment.
Pull request overview
This PR adds support for logging the RX byte delta for the 6GHz private VAP (interface index 17) in aphealth.sh. Previously, only 2.4GHz and 5GHz VAP RX statistics were captured, so the WIFI_RXDELTA_17 marker was absent from wifihealth.txt on devices with a 6GHz radio (specifically CGM4981COM). The fix introduces a new model-specific branch for CGM4981COM that sets a third interface variable (ssid3), and conditionally reads, computes, and logs the RX byte counts and delta for that interface.
Changes:
- Adds a new
elifbranch for theCGM4981COMmodel to definessid3="wl2.1"(the 6GHz VAP interface) in addition tossid1andssid2. - Adds a conditional block (
if [ -n "$ssid3" ]) that reads, persists, computes, and logsWIFI_RX_17andWIFI_RXDELTA_17statistics. - Adds a blank line (cosmetic).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
gsathish86
left a comment
gsathish86
left a comment
There was a problem hiding this comment.
Please update jira details in the PR description and header
SakeVictorDaniel
changed the title
|
"I have read the CLA Document and I hereby sign the CLA" |
SakeVictorDaniel
force-pushed
the
RDKB-63427_dev
branch
from
96c5b62 to
13ccf2e
Compare
Reason for change: Add support for logging RXDELTA for the 6GHz private VAP in aphealth.sh, previously RX statistics were logged only for 2.4 & 5GHz private VAP interfaces, hence RXDELTA marker for 6GHz was not reflected in wifihealth.txt Test Procedure: -- Load the device with build having the fix. -- /rdklogs/logs/wifihealth.txt will not be created as soon as the device boots up aphealth.sh will be launched by aphealth_log.sh (cron job on 35th minute of every hour) -- Either wait until the 35th minute of the hour and ensure system uptime is > 1800 (OR) manually launch the aphealth.sh like the aphealth_log.sh does by executing /usr/ccsp/wifi/aphealth.sh >> /rdklogs/logs/wifihealth.txt -- Check whether /tmp/wifihealth/ directory is created and also check the directory contains rx17_0 file -- Check whether /rdklogs/logs/wifihealth.txt is created and check for the WIFI_RXDELTA_17 marker Risks: Low Priority: P1 Signed-off-by: Sake Victor Daniel <VictorDaniel_Sake@comcast.com>
SakeVictorDaniel
force-pushed
the
RDKB-63427_dev
branch
from
13ccf2e to
61c0255
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
RDKB-63427 "WIFI_RXDELTA_17" is not available in wifihealth.txt
Reason for change:
Add support for logging RXDELTA for the 6GHz private VAP in aphealth.sh, previously RX statistics were logged only for 2.4 & 5GHz private VAP interfaces, hence RXDELTA marker for 6GHz was not reflected in wifihealth.txt
Test Procedure:
Load the device with build having the fix.
/rdklogs/logs/wifihealth.txt will not be created as soon as the device boots up
aphealth.sh will be launched by aphealth_log.sh (cron job on 35th minute of every hour)
Either wait until the 35th minute of the hour and ensure system uptime is > 1800 (OR) manually launch the aphealth.sh like
the aphealth_log.sh does by executing /usr/ccsp/wifi/aphealth.sh >> /rdklogs/logs/wifihealth.txt
Check whether /tmp/wifihealth/ directory is created and also check the directory contains rx17_0 file
Check whether /rdklogs/logs/wifihealth.txt is created and check for the WIFI_RXDELTA_17 marker
Risks: Low
Priority: P1
Signed-off-by: Signed-off-by: Sake Victor Daniel victordaniel_sake@comcast.com