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

samples: matter: Added DSLS support to the smoke co alarm #19151

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

kkasperczyk-no
Copy link
Contributor

Integrated and enabled ICD DSLS feature with the smoke CO alarm sample. Additionally, added documentation of test event triggers dedicated for switching the power source.

@kkasperczyk-no kkasperczyk-no requested review from a team as code owners November 29, 2024 07:00
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Nov 29, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 29, 2024

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
matter nrfconnect/sdk-connectedhomeip@297f762 nrfconnect/sdk-connectedhomeip@6c1a9c5 (master) nrfconnect/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 29, 2024

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 8

Inputs:

Sources:

sdk-nrf: PR head: efac50208db9613785fc1a166e4b182ee123246b
matter: PR head: 6c1a9c51256fa3bea98bcdbf31d8a6aeef006a19

more details

sdk-nrf:

PR head: efac50208db9613785fc1a166e4b182ee123246b
merge base: 3677932b0cff3d9024c37b62ff45b59424659c25
target head (main): c32a7da575b7fc8d751d1745c65efc3bc2d1293e
Diff

matter:

PR head: 6c1a9c51256fa3bea98bcdbf31d8a6aeef006a19
merge base: 297f762debcfe0d983fcef2dd60cc926e715b17c
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 (12)
doc
│  ├── nrf
│  │  ├── protocols
│  │  │  ├── matter
│  │  │  │  ├── end_product
│  │  │  │  │  │ test_event_triggers.rst
│  │  │  │  ├── getting_started
│  │  │  │  │  │ low_power_configuration.rst
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
modules
│  ├── lib
│  │  ├── matter
│  │  │  ├── config
│  │  │  │  ├── nrfconnect
│  │  │  │  │  ├── chip-module
│  │  │  │  │  │  │ CMakeLists.txt
│  │  │  │  ├── zephyr
│  │  │  │  │  │ Kconfig
samples
│  ├── matter
│  │  ├── light_switch
│  │  │  ├── snippets
│  │  │  │  ├── lit_icd
│  │  │  │  │  │ lit_icd.conf
│  │  ├── smoke_co_alarm
│  │  │  ├── Kconfig
│  │  │  ├── README.rst
│  │  │  ├── src
│  │  │  │  ├── default_zap
│  │  │  │  │  ├── smoke_co_alarm.zap
│  │  │  │  │  ├── zap-generated
│  │  │  │  │  │  │ endpoint_config.h
│  │  │  │  │ zcl_callbacks.cpp
west.yml

Outputs:

Toolchain

Version: b77d8c1312
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:b77d8c1312_912848a074

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 120
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
Disabled integration tests
    • 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-boot
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

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

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publish GitHub Action.

samples/matter/smoke_co_alarm/README.rst Outdated Show resolved Hide resolved
samples/matter/smoke_co_alarm/README.rst Outdated Show resolved Hide resolved
samples/matter/smoke_co_alarm/README.rst Outdated Show resolved Hide resolved
samples/matter/smoke_co_alarm/README.rst Outdated Show resolved Hide resolved
samples/matter/smoke_co_alarm/README.rst Outdated Show resolved Hide resolved
@peknis
Copy link
Contributor

peknis commented Dec 2, 2024

Maybe needs a changelog entry.

samples/matter/smoke_co_alarm/README.rst Outdated Show resolved Hide resolved
@kkasperczyk-no
Copy link
Contributor Author

Maybe needs a changelog entry.

Yes, definitely, thanks. I will add an entry.

Integrated and enabled ICD DSLS feature with the smoke CO alarm
sample. Additionally, added documentation of test event triggers
dedicated for switching the power source.

Signed-off-by: Kamil Kasperczyk <[email protected]>
@kkasperczyk-no kkasperczyk-no requested a review from a team as a code owner December 2, 2024 12:58
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 2, 2024
@maciejbaczmanski maciejbaczmanski merged commit eaa215e into nrfconnect:main Dec 3, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval. manifest manifest-matter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants