Skip to content
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

[CERT-TEST-FAILURE] - TC-DEM-1.1 - ClusterRevision value has to be updated for Device Energy Management Cluster #34297

Closed
KishokG opened this issue Jul 11, 2024 · 3 comments · Fixed by #35067
Assignees
Labels

Comments

@KishokG
Copy link
Contributor

KishokG commented Jul 11, 2024

Feature Area

Other

Test Case

TC-DEM-1.1

Reproduction steps

  1. Bring up the DUT with energy-management-app.
  2. Pair the DUT using chip-tool through "Onnetwork" transport.
  3. Run the command ./chip-tool deviceenergymanagement read cluster-revision 1 1

Expected Outcome:

[1720690440.539] [50435:50437] [TOO] Endpoint: 1 Cluster: 0x0000_0098 Attribute 0x0000_FFFD DataVersion: 521028924
[1720690440.539] [50435:50437] [TOO] ClusterRevision: 4
[1720690440.540] [50435:50437] [EM] <<< [E:5099i S:17862 M:203971832 (Ack:69830139)] (S) Msg TX to 1:0000000000000001 [C5DD] [UDP:[fe80::e65f:1ff:fe0f:2878%eth0]:5540] --- Type 0000:10 (SecureChannel:StandaloneAck)

Actual Outcome:

[1720690440.539] [50435:50437] [TOO] Endpoint: 1 Cluster: 0x0000_0098 Attribute 0x0000_FFFD DataVersion: 521028924
[1720690440.539] [50435:50437] [TOO] ClusterRevision: 3
[1720690440.540] [50435:50437] [EM] <<< [E:5099i S:17862 M:203971832 (Ack:69830139)] (S) Msg TX to 1:0000000000000001 [C5DD] [UDP:[fe80::e65f:1ff:fe0f:2878%eth0]:5540] --- Type 0000:10 (SecureChannel:StandaloneAck)

Bug prevalence

Whenever I try to read ClusterRevision.

GitHub hash of the SDK that was being used

6e31453

Platform

raspi

Anything else?

In SPEC, the ClusterRevision value has been updated to 4. As per spec changes, SDK(energy-management-app) has to update the ClusterRevision value from 3 to 4.

Specification References: https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/app_clusters/DeviceEnergyManagement.adoc#revision-history

@KishokG KishokG added bug Something isn't working cert blocker needs triage labels Jul 11, 2024
@github-project-automation github-project-automation bot moved this to Open Cert Blockers in [Certification] Blockers Jul 11, 2024
@KishokG KishokG changed the title [CERT-TEST-FAILURE] - TC-DEM-1.1 - Test failed [CERT-TEST-FAILURE] - TC-DEM-1.1 - ClusterRevision value has to be updated for Device Energy Management Cluster Jul 12, 2024
@cjandhyala
Copy link
Contributor

@KishokG is this still an issue ? Can we close this ticket. ?

@KishokG
Copy link
Contributor Author

KishokG commented Aug 20, 2024

Yes @cjandhyala It's yet to be fixed

@cjandhyala cjandhyala added the app label Aug 20, 2024
@jamesharrow
Copy link
Contributor

Note that we have removed TC DEM 1.1 https://github.com/CHIP-Specifications/chip-test-plans/pull/4555
The cluster revision however has been fixed in the SDK in another SDK PR:
#35067

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants