connectedhomeip icon indicating copy to clipboard operation
connectedhomeip copied to clipboard

[pre-sve] DGSW-2.2 Implementation for event handler of Software Diagnostics events is missing

Open PSONALl opened this issue 3 years ago • 7 comments

Problem

In Test case DGSW-2

  • expected behavior - DUT sends an event report to TH. TH reads a list of SoftwareFault struct from DUT.

  • actual behavior - No Event Report is sent

  • system configuration - DUT used : ESP32 - app use: all-clusters-app - chip-tool and DUT commit id : 425f1fb916ca154deeb75cd82cd5196778aba152 (pre-sve SHA)

  • logs

Proposed Solution

  • Add code to handle software events

PSONALl avatar Jun 10 '22 11:06 PSONALl

Bumping this issue to cert blocker as it's a request cert blocker discovered during pre-SVE tests.

franck-apple avatar Jun 10 '22 16:06 franck-apple

SoftwareFault event conformance is optional.

While this is useful for testing, I am unsure if this should block cert: event generation is up to the individual device implementation, software fault event generation is optional.

andy31415 avatar Jun 13 '22 14:06 andy31415

Cert Blocker Review: Tagging @franck-apple to follow up with Sankar, to see if there's a test plan update/review needed.

woody-apple avatar Jun 13 '22 17:06 woody-apple

@PSONALl I checked with QA team, they used the following command to generate the event & then read the event. Can you please try and update your observation?

  1. get the pid of the apps you are running by "ps -aef|grep all-clusters-app"
  2. Then kill the app using the following signal "sudo kill -SIGUSR1 "
  3. Then read the event.

It worked fine for us

sankarselvam avatar Jun 14 '22 09:06 sankarselvam

@sankarselvam tried this but I am not getting any Event Report when I read the event this is because OnSoftwareFaultDetect is never called

PSONALl avatar Jun 14 '22 10:06 PSONALl

OK, Seems like its a platform specific issue. We tried in RPI & we could able to see the event is called in DUT. Attached logs for ref [TC-DGSW-2.2]TH.txt [TC-DGSW-2.2]DUT.txt

sankarselvam avatar Jun 14 '22 11:06 sankarselvam

Yes @sankarselvam I am working to fix this issue on the esp32 side

PSONALl avatar Jun 14 '22 11:06 PSONALl

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Apr 25 '23 18:04 stale[bot]

This stale issue has been automatically closed. Thank you for your contributions.

stale[bot] avatar May 08 '23 21:05 stale[bot]