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

[1.4] In WaterHeaterManagement Cluster, The Featuremap and ClusterRevision values are Failure instead of Success. #36417

Open
Ashwinigrl opened this issue Nov 7, 2024 · 3 comments
Labels
bug Something isn't working needs triage water heater

Comments

@Ashwinigrl
Copy link

Ashwinigrl commented Nov 7, 2024

Description

In the WaterHeaterManagement Cluster, When we read the FeatureMap and ClusterRevision attributes from the DUT returns a failure response instead of a success response with the expected values. The sample app(Energy-Management-app) needs to be updated to correct the FeatureMap and ClusterRevision attributes response.

Reproduction steps

  1. Execute the below mentioned command to put DUT into a commissionable state :
    ./chip-energy-management-app

  2. Once DUT reach the commissionable state sent below mentioned command on TH.
    ./chip-tool pairing onnetwork 1 20202021

Once after successful pairing provide the below commands:

  1. TH reads FeatureMap value from the DUT.

./chip-tool waterheatermanagement read feature-map 1 1

Actual Outcome:

[1730974880.590] [65803:65805] [DMG] {
[1730974880.590] [65803:65805] [DMG] status = 0x01 (FAILURE),
[1730974880.590] [65803:65805] [DMG] },
[1730974880.590] [65803:65805] [DMG]
[1730974880.590] [65803:65805] [DMG] },
[1730974880.590] [65803:65805] [DMG]
[1730974880.590] [65803:65805] [DMG] },
[1730974880.590] [65803:65805] [DMG]
[1730974880.590] [65803:65805] [DMG] ],
[1730974880.590] [65803:65805] [DMG]
[1730974880.590] [65803:65805] [DMG] SuppressResponse = true,
[1730974880.590] [65803:65805] [DMG] InteractionModelRevision = 11
[1730974880.590] [65803:65805] [DMG] }

Expected Outcome:

[1730975412.558] [65847:65849] [DMG] },
[1730975412.558] [65847:65849] [DMG]
[1730975412.558] [65847:65849] [DMG] },
[1730975412.558] [65847:65849] [DMG]
[1730975412.558] [65847:65849] [DMG] ],
[1730975412.558] [65847:65849] [DMG]
[1730975412.558] [65847:65849] [DMG] SuppressResponse = true,
[1730975412.558] [65847:65849] [DMG] InteractionModelRevision = 11
[1730975412.558] [65847:65849] [DMG] }
[1730975412.558] [65847:65849] [TOO] Endpoint: 1 Cluster: 0x0000_0094 Attribute 0x0000_FFFC DataVersion: 3823610553
[1730975412.559] [65847:65849] [TOO] FeatureMap:

  1. TH reads ClusterRevision value from the DUT.

./chip-tool waterheatermanagement read cluster-revision 1 1

Actual Outcome:

[1730978688.118] [66624:66626] [DMG] {
[1730978688.118] [66624:66626] [DMG] status = 0x01 (FAILURE),
[1730978688.119] [66624:66626] [DMG] },
[1730978688.119] [66624:66626] [DMG]
[1730978688.119] [66624:66626] [DMG] },
[1730978688.119] [66624:66626] [DMG]
[1730978688.119] [66624:66626] [DMG] },
[1730978688.119] [66624:66626] [DMG]
[1730978688.119] [66624:66626] [DMG] ],
[1730978688.119] [66624:66626] [DMG]
[1730978688.119] [66624:66626] [DMG] SuppressResponse = true,
[1730978688.119] [66624:66626] [DMG] InteractionModelRevision = 11
[1730978688.119] [66624:66626] [DMG] }

Expected Outcome:

[1730978939.117] [66655:66657] [DMG] Data = 2 (unsigned),
[1730978939.117] [66655:66657] [DMG] },
[1730978939.117] [66655:66657] [DMG]
[1730978939.117] [66655:66657] [DMG] },
[1730978939.117] [66655:66657] [DMG]
[1730978939.117] [66655:66657] [DMG] ],
[1730978939.117] [66655:66657] [DMG]
[1730978939.117] [66655:66657] [DMG] SuppressResponse = true,
[1730978939.117] [66655:66657] [DMG] InteractionModelRevision = 11
[1730978939.117] [66655:66657] [DMG] }
[1730978939.117] [66655:66657] [TOO] Endpoint: 1 Cluster: 0x0000_0094 Attribute 0x0000_FFFD DataVersion: 2387229930
[1730978939.117] [66655:66657] [TOO] ClusterRevision: 2

Sample App used to read the FeatureMap value: EnergyManagementApp

Bug prevalence

Whenever we do

GitHub hash of the SDK that was being used

9c389ba

Platform

raspi

Spec References:

Log References:
WaterHeaterManagement.txt

@Ashwinigrl Ashwinigrl added bug Something isn't working needs triage labels Nov 7, 2024
@Ashwinigrl Ashwinigrl changed the title [1.4] In WaterHeaterManagement Cluster, The Featuremap value is Failure instead of Success. [1.4] In WaterHeaterManagement Cluster, The Featuremap and ClusterRevision values are Failure instead of Success. Nov 7, 2024
@bzbarsky-apple
Copy link
Contributor

@jamesharrow

@lboue
Copy link
Contributor

lboue commented Nov 11, 2024

Did you try against PR #36201?

@jamesharrow
Copy link
Contributor

The issue is that you need to use --application water-heater to enable the Water Heater cluster to respond on EP1 on Linux (or compile this for an embedded target with water heater enabled).

@Ashwinigrl - please retest with the extra command line option.

Note that the PR #36201 will move the Water Heater to a different endpoint once merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage water heater
Projects
Status: Todo
Development

No branches or pull requests

4 participants