Skip to content

[DNM] [nrf toup] Network Commissioning: Add hysteresis to status reporting - #620

Open
LuDuda wants to merge 1 commit into
nrfconnect:masterfrom
LuDuda:pr/network-comm-last-status
Open

LuDuda wants to merge 1 commit into
nrfconnect:masterfrom
LuDuda:pr/network-comm-last-status

Conversation

@LuDuda

@LuDuda LuDuda commented Jun 23, 2025

Copy link
Copy Markdown
Collaborator

This is just a PoC for delaying reporting the error status - for review.

This patch delays error reporting in NetworkCommissioning cluster if a success was recently observed, premature notifications to mislead clients. Functionality is controlled by
CHIP_CONFIG_NETWORK_COMMISSIONING_IMMEDIATE_STATUS_REPORTING

This patch delays error reporting in NetworkCommissioning cluster if a
success was recently observed, premature notifications to mislead
clients. Functionality is controlled by
CHIP_CONFIG_NETWORK_COMMISSIONING_IMMEDIATE_STATUS_REPORTING

Signed-off-by: Łukasz Duda <lukasz.duda@nordicsemi.no>
Comment thread src/lib/core/CHIPConfig.h
* Default: 1 (enabled)
*/
#ifndef CHIP_CONFIG_NETWORK_COMMISSIONING_IMMEDIATE_STATUS_REPORTING
#define CHIP_CONFIG_NETWORK_COMMISSIONING_IMMEDIATE_STATUS_REPORTING 0

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: Disabled for CI testing of a new behavior.

@github-actions

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity.

@github-actions github-actions Bot added the Stale label Dec 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant