matter: update data model and enable mandatory cmds #24029

doublemis1 · 2025-08-18T10:35:56Z

Update matter samples data model.

enable VID verification commands.
set identify cluster revision to 6

NordicBuilder · 2025-08-18T10:39:34Z

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 6

Inputs:

Sources:

sdk-nrf: PR head: 3333bc7ab7ba6ba64c08106a3b9e39e70cfd307d

more details

sdk-nrf:

PR head: 3333bc7ab7ba6ba64c08106a3b9e39e70cfd307d
merge base: 57bb508a3cb441ed905426c77d04abfa942ba42e
target head (main): b4058d7d06620b71fff81757ecf8eb119716bc45
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (84)

applications
│  ├── matter_bridge
│  │  ├── snippets
│  │  │  ├── onoff_plug
│  │  │  │  ├── bridge.matter
│  │  │  │  ├── bridge.zap
│  │  │  │  ├── zap-generated
│  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  ├── access.h
│  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │ gen_config.h
│  │  ├── src
│  │  │  ├── default_zap
│  │  │  │  ├── bridge.matter
│  │  │  │  ├── bridge.zap
│  │  │  │  ├── zap-generated
│  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  ├── access.h
│  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │ gen_config.h
│  ├── matter_weather_station
│  │  ├── src
│  │  │  ├── default_zap
│  │  │  │  ├── weather-station.matter
│  │  │  │  ├── weather-station.zap
│  │  │  │  ├── zap-generated
│  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  ├── access.h
│  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │ gen_config.h
samples
│  ├── matter
│  │  ├── common
│  │  │  ├── src
│  │  │  │  ├── certification
│  │  │  │  │  ├── thread_platform
│  │  │  │  │  │  ├── thread_certification.matter
│  │  │  │  │  │  ├── thread_certification.zap
│  │  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  │  │ endpoint_config.h
│  │  ├── light_bulb
│  │  │  ├── src
│  │  │  │  ├── default_zap
│  │  │  │  │  ├── light_bulb.matter
│  │  │  │  │  ├── light_bulb.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
│  │  ├── light_switch
│  │  │  ├── snippets
│  │  │  │  ├── lit_icd
│  │  │  │  │  ├── light_switch.matter
│  │  │  │  │  ├── light_switch.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
│  │  │  ├── src
│  │  │  │  ├── default_zap
│  │  │  │  │  ├── light_switch.matter
│  │  │  │  │  ├── light_switch.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
│  │  ├── lock
│  │  │  ├── snippets
│  │  │  │  ├── schedules
│  │  │  │  │  ├── lock.matter
│  │  │  │  │  ├── lock.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
│  │  │  ├── src
│  │  │  │  ├── default_zap
│  │  │  │  │  ├── lock.matter
│  │  │  │  │  ├── lock.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
│  │  ├── manufacturer_specific
│  │  │  ├── src
│  │  │  │  ├── default_zap
│  │  │  │  │  ├── manufacturer_specific.matter
│  │  │  │  │  ├── manufacturer_specific.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── PluginApplicationCallbacks.h
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── callback-stub.cpp
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
│  │  ├── smoke_co_alarm
│  │  │  ├── src
│  │  │  │  ├── default_zap
│  │  │  │  │  ├── smoke_co_alarm.matter
│  │  │  │  │  ├── smoke_co_alarm.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
│  │  ├── template
│  │  │  ├── src
│  │  │  │  ├── default_zap
│  │  │  │  │  ├── template.matter
│  │  │  │  │  ├── template.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
│  │  ├── thermostat
│  │  │  ├── src
│  │  │  │  ├── default_zap
│  │  │  │  │  ├── thermostat.matter
│  │  │  │  │  ├── thermostat.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
│  │  ├── window_covering
│  │  │  ├── src
│  │  │  │  ├── default_zap
│  │  │  │  │  ├── window-app.matter
│  │  │  │  │  ├── window-app.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  ├── IMClusterCommandHandler.cpp
│  │  │  │  │  │  ├── access.h
│  │  │  │  │  │  ├── endpoint_config.h
│  │  │  │  │  │  │ gen_config.h
scripts
│  │ quarantine.yaml

Outputs:

Toolchain

Version: c5be9c56c7
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:c5be9c56c7_bba2ea5f2e

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
- sdk-nrf test count: 93
✅ Integration tests
- ✅ test-fw-nrfconnect-chip

Disabled integration tests

- test-fw-nrfconnect-nrf_lrcs_mosh
- test-fw-nrfconnect-nrf_lrcs_positioning
- desktop52_verification
- doc-internal
- test_ble_nrf_config
- test-fw-nrfconnect-apps
- test-fw-nrfconnect-ble_mesh
- test-fw-nrfconnect-ble_samples
- test-fw-nrfconnect-fem
- test-fw-nrfconnect-nfc
- test-fw-nrfconnect-nrf-iot_cloud
- test-fw-nrfconnect-nrf-iot_libmodem-nrf
- test-fw-nrfconnect-nrf-iot_lwm2m
- test-fw-nrfconnect-nrf-iot_samples
- test-fw-nrfconnect-nrf-iot_serial_lte_modem
- test-fw-nrfconnect-nrf-iot_thingy91
- test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- test-fw-nrfconnect-nrf_crypto
- test-fw-nrfconnect-proprietary_esb
- test-fw-nrfconnect-ps-main
- test-fw-nrfconnect-rpc
- test-fw-nrfconnect-rs
- test-fw-nrfconnect-tfm
- test-fw-nrfconnect-thread-main
- test-low-level
- test-sdk-audio
- test-sdk-dfu
- test-sdk-find-my
- test-sdk-mcuboot
- test-sdk-pmic-samples
- test-sdk-wifi
- test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

github-actions · 2025-08-18T11:05:24Z

You can find the documentation preview for this PR here.

kkasperczyk-no

Did you re-generate source files for manufacturer-specifc sample and thread_certification? I can see only change in .zap and .matter files, but not in zap-generated directory

doublemis1 · 2025-08-18T11:36:34Z

Did you re-generate source files for manufacturer-specifc sample and thread_certification? I can see only change in .zap and .matter files, but not in zap-generated directory

will check it twice, thanks!

Update matter samples data model. * enable VID verification commands. * set identify cluster revision to 6 Signed-off-by: Michał Szablowski <[email protected]>

github-actions · 2025-08-19T11:51:17Z

Since quarantine was modified, please make sure you are following the process described in Quarantine Process.

NordicBuilder · 2025-08-20T11:12:00Z

Memory footprint analysis revealed the following potential issues

sample.matter.template.debug[nrf52840dk/nrf52840]: ROM size increased by 3488[B] in comparison to the main[57bb508] branch. - link (cc: @kkasperczyk-no @ArekBalysNordic @markaj-nordic)
sample.matter.template.debug[nrf5340dk/nrf5340/cpuapp]: ROM size increased by 3708[B] in comparison to the main[57bb508] branch. - link (cc: @kkasperczyk-no @ArekBalysNordic @markaj-nordic)
sample.matter.template.release[nrf7002dk/nrf5340/cpuapp]: ROM size increased by 1612[B] in comparison to the main[57bb508] branch. - link (cc: @kkasperczyk-no @ArekBalysNordic @markaj-nordic)
sample.matter.template.debug[nrf7002dk/nrf5340/cpuapp]: ROM size increased by 2068[B] in comparison to the main[57bb508] branch. - link (cc: @kkasperczyk-no @ArekBalysNordic @markaj-nordic)
sample.matter.template.release[nrf52840dk/nrf52840]: ROM size increased by 2960[B] in comparison to the main[57bb508] branch. - link (cc: @kkasperczyk-no @ArekBalysNordic @markaj-nordic)
sample.matter.template.release[nrf5340dk/nrf5340/cpuapp]: ROM size increased by 3076[B] in comparison to the main[57bb508] branch. - link (cc: @kkasperczyk-no @ArekBalysNordic @markaj-nordic)

Note: This message is automatically posted and updated by the CI (latest/sdk-nrf/PR-24029/6)

doublemis1 · 2025-08-22T06:03:01Z

@nrfconnect/ncs-co-build-system Please review

kkasperczyk-no · 2025-08-25T12:29:22Z

@nrfconnect/ncs-co-build-system this change is required to be back-ported to v3.1 branch, please review soon.

nordicjm · 2025-08-26T06:49:38Z

@nrfconnect/ncs-co-build-system Please review

I don't see why the review is required on a bunch of random auto generated code which has no CMake or Kconfig changes

kkasperczyk-no · 2025-08-26T06:55:57Z

@nrfconnect/ncs-co-build-system Please review

I don't see why the review is required on a bunch of random auto generated code which has no CMake or Kconfig changes

@nordicjm I think that this particular PR triggered your review, because it modifies snippets.

doublemis1 requested review from a team as code owners August 18, 2025 10:35

github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 18, 2025

doublemis1 force-pushed the fix_data_model branch from 04327c3 to f8b1080 Compare August 18, 2025 11:15

kkasperczyk-no reviewed Aug 18, 2025

View reviewed changes

doublemis1 force-pushed the fix_data_model branch from f8b1080 to 10403ef Compare August 18, 2025 11:39

matter: update data model and enable mandatory cmds

3333bc7

Update matter samples data model. * enable VID verification commands. * set identify cluster revision to 6 Signed-off-by: Michał Szablowski <[email protected]>

doublemis1 force-pushed the fix_data_model branch from 10403ef to 3333bc7 Compare August 19, 2025 11:51

doublemis1 requested a review from a team as a code owner August 19, 2025 11:51

nordic-piks approved these changes Aug 19, 2025

View reviewed changes

kkasperczyk-no approved these changes Aug 20, 2025

View reviewed changes

LuDuda approved these changes Aug 20, 2025

View reviewed changes

nordicjm approved these changes Aug 26, 2025

View reviewed changes

ArekBalysNordic merged commit 4566a49 into nrfconnect:main Aug 26, 2025
19 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

matter: update data model and enable mandatory cmds #24029

matter: update data model and enable mandatory cmds #24029

Uh oh!

doublemis1 commented Aug 18, 2025

Uh oh!

NordicBuilder commented Aug 18, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Aug 18, 2025

Uh oh!

kkasperczyk-no left a comment

Uh oh!

doublemis1 commented Aug 18, 2025

Uh oh!

github-actions bot commented Aug 19, 2025

Uh oh!

NordicBuilder commented Aug 20, 2025

Uh oh!

doublemis1 commented Aug 22, 2025

Uh oh!

kkasperczyk-no commented Aug 25, 2025

Uh oh!

nordicjm commented Aug 26, 2025

Uh oh!

Uh oh!

kkasperczyk-no commented Aug 26, 2025

Uh oh!

Uh oh!

matter: update data model and enable mandatory cmds #24029

matter: update data model and enable mandatory cmds #24029

Uh oh!

Conversation

doublemis1 commented Aug 18, 2025

Uh oh!

NordicBuilder commented Aug 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

Uh oh!

github-actions bot commented Aug 18, 2025

Uh oh!

kkasperczyk-no left a comment

Choose a reason for hiding this comment

Uh oh!

doublemis1 commented Aug 18, 2025

Uh oh!

github-actions bot commented Aug 19, 2025

Uh oh!

NordicBuilder commented Aug 20, 2025

Memory footprint analysis revealed the following potential issues

Uh oh!

doublemis1 commented Aug 22, 2025

Uh oh!

kkasperczyk-no commented Aug 25, 2025

Uh oh!

nordicjm commented Aug 26, 2025

Uh oh!

Uh oh!

kkasperczyk-no commented Aug 26, 2025

Uh oh!

Uh oh!

NordicBuilder commented Aug 18, 2025 •

edited

Loading