Skip to content

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

Merged
merged 2 commits into from
May 29, 2025
Merged

Conversation

krish2718
Copy link
Collaborator

Fixes a corruption seen when active traffic is run with stats in the background (script)

krish2718 added 2 commits May 27, 2025 15:08
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>
Copy link

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
nrf_wifi zephyrproject-rtos/nrf_wifi@5e185a2 (main) zephyrproject-rtos/nrf_wifi#53 zephyrproject-rtos/nrf_wifi#53/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-nrf_wifi DNM (manifest) This PR should not be merged (controlled by action-manifest) labels May 27, 2025
Copy link

@kartben kartben merged commit 5601af3 into zephyrproject-rtos:main May 29, 2025
30 checks passed
@krish2718
Copy link
Collaborator Author

@kartben this PR has a DNM (manifest) the manifest still has a PR reference.

@fabiobaltieri
Copy link
Member

@kartben this PR has a DNM (manifest) the manifest still has a PR reference.

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.

@fabiobaltieri
Copy link
Member

fabiobaltieri commented May 29, 2025

As for this PR, the module was approved, merged the change there I'd say just fix the sha in a followup, 0cd7f28d34a5279cd839940c199658a294165722

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Wi-Fi Wi-Fi DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-nrf_wifi platform: nRF Nordic nRFx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants