-
Notifications
You must be signed in to change notification settings - Fork 7.5k
Statistics handling rejig #90649
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Statistics handling rejig #90649
Conversation
Fixes a corruption when active traffic is run along with stats in the background. Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
Separate out host(local) and FW(remote) statistics and in case of a FW statistics timeout, return success with a magic value in the FW statistics, this way atleast host statistics would be handy for debugging instead of sending failure up. Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 1 project with PR revision Note: This message is automatically posted and updated by the Manifest GitHub Action. |
|
@kartben this PR has a |
The PR metadata check should have been rerun by github on the relabel event but it didn't, I'll look into adding a delay there, something's broken with github and it's hard to get these to play well together for some reasons. |
As for this PR, the module was approved, merged the change there I'd say just fix the sha in a followup, |
Fixes a corruption seen when active traffic is run with stats in the background (script)