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

manifest: sdk-connectedhomeip: Update revision #18910

Conversation

NordicBuilder
Copy link
Contributor

@NordicBuilder NordicBuilder commented Nov 15, 2024

Automatically created by action-manifest-pr GH action from PR: nrfconnect/sdk-connectedhomeip#512

test_chip: upmerge_matter_1_4

@NordicBuilder NordicBuilder requested a review from a team as a code owner November 15, 2024 10:22
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Nov 15, 2024
@NordicBuilder
Copy link
Contributor Author

NordicBuilder commented Nov 15, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
matter nrfconnect/sdk-connectedhomeip@aa2e056 nrfconnect/sdk-connectedhomeip@9cfd6e1 (master) nrfconnect/[email protected]

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

@NordicBuilder
Copy link
Contributor Author

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 566fd96875a9dca73780ebcc85dd172ddc86fb85

more details

sdk-nrf:

PR head: 566fd96875a9dca73780ebcc85dd172ddc86fb85
merge base: e6443ca70a94483de5ac34318c0028f3e95741e2
target head (main): e6443ca70a94483de5ac34318c0028f3e95741e2
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 (1)
west.yml

Outputs:

Toolchain

Version:
Build docker image:

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

  • ❌ Toolchain
  • ❌ Build twister
  • ❌ Integration tests
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-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • 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-iot_zephyr_lwm2m
    • 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

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

@adigie adigie force-pushed the auto-manifest-sdk-connectedhomeip-512 branch from 4ec3bbf to 058bb97 Compare November 15, 2024 11:01
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-connectedhomeip-512 branch from 058bb97 to eba0a1f Compare November 15, 2024 11:19
@adigie adigie requested a review from a team as a code owner November 15, 2024 12:40
@adigie adigie requested review from a team as code owners November 15, 2024 14:10
@adigie adigie force-pushed the auto-manifest-sdk-connectedhomeip-512 branch 2 times, most recently from 067c44f to 38bd5ed Compare November 15, 2024 15:17
Copy link
Contributor

@kkasperczyk-no kkasperczyk-no left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have few observations that should be fixed:

@adigie adigie force-pushed the auto-manifest-sdk-connectedhomeip-512 branch from 38bd5ed to 44de8b2 Compare November 18, 2024 07:48
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-connectedhomeip-512 branch from 44de8b2 to 79d3413 Compare November 18, 2024 07:49
@NordicBuilder
Copy link
Contributor Author

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.

@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-connectedhomeip-512 branch from 79d3413 to c2c51ea Compare November 18, 2024 08:07
@adigie adigie force-pushed the auto-manifest-sdk-connectedhomeip-512 branch 2 times, most recently from 7a830d7 to 6eb4ffb Compare November 18, 2024 14:59
@kkasperczyk-no kkasperczyk-no requested review from a team as code owners November 19, 2024 07:06
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. and removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Nov 19, 2024
@kkasperczyk-no kkasperczyk-no force-pushed the auto-manifest-sdk-connectedhomeip-512 branch 2 times, most recently from 02bf488 to 0118edf Compare November 19, 2024 07:36
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-connectedhomeip-512 branch from 0118edf to eeb5667 Compare November 19, 2024 07:47
@kkasperczyk-no kkasperczyk-no force-pushed the auto-manifest-sdk-connectedhomeip-512 branch from eeb5667 to 86b11be Compare November 19, 2024 09:52
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-connectedhomeip-512 branch from 1117a46 to 2d74f4f Compare November 28, 2024 08:50
@ArekBalysNordic
Copy link
Contributor

ArekBalysNordic commented Nov 28, 2024

@nrfconnect/ncs-ci Could you please have a look and review? But honestly, I don't see any changes related to CI.

NordicBuilder and others added 23 commits November 28, 2024 11:24
Automatically created by Github Action

Signed-off-by: Nordic Builder <[email protected]>
Remove nlunit-test submodule.

Signed-off-by: Adrian Gielniewski <[email protected]>
Set `ActiveMode` of ICDManager using OnNetworkActivity method.

Signed-off-by: Adrian Gielniewski <[email protected]>
* Update zap files
* Regenerate sources

Signed-off-by: Adrian Gielniewski <[email protected]>
Update `ICD Management` cluster revision to 3.

Signed-off-by: Adrian Gielniewski <[email protected]>
Add `MaximumCheckInBackOff` attribute to `ICD management` cluster in LIT
devices.

Signed-off-by: Adrian Gielniewski <[email protected]>
* `clusterId` is now std::optional.
* Fix includes.

Signed-off-by: Adrian Gielniewski <[email protected]>
Updated documentation to list all new features
and configuration that were introduced in Matter v1.4.

Signed-off-by: Kamil Kasperczyk <[email protected]>
Update `Basic Information` cluster revision to 4.

Signed-off-by: Adrian Gielniewski <[email protected]>
Update device types / clusters revisions.

Signed-off-by: Adrian Gielniewski <[email protected]>
Enable LTO for `nrf7002dk/nrf5340/cpuapp` due to increased flash usage.

Signed-off-by: Adrian Gielniewski <[email protected]>
Set bit 0 (`EXTS`) as Extensions are enabled.

Signed-off-by: Adrian Gielniewski <[email protected]>
Update General Commissioning Cluster revision to 2.

Signed-off-by: Adrian Gielniewski <[email protected]>
Update Identify Cluster revision to 5.

Signed-off-by: Adrian Gielniewski <[email protected]>
Update Thread Network Diagnostics Cluster revision to 3.

Signed-off-by: Adrian Gielniewski <[email protected]>
Update Level Control Cluster revision in Light Bulb sample to 6.

Signed-off-by: Adrian Gielniewski <[email protected]>
Update bridged device On/Off and Switch clusters revisions.

Signed-off-by: Adrian Gielniewski <[email protected]>
* Update On/Off Light and On/Off Light Switch endpoint versions to 3.
* Update Bridged Node endpoint version to 3.

Signed-off-by: Adrian Gielniewski <[email protected]>
* Update Bridged Device Basic Information cluster revision to 5.
* Update Identify cluster revision to 4.

Signed-off-by: Adrian Gielniewski <[email protected]>
Add support for UniqueID attribute in Bridged Device Basic Information
cluster.

Signed-off-by: Adrian Gielniewski <[email protected]>
Fix ICD documentation

Signed-off-by: Adrian Gielniewski <[email protected]>
Change Level Control default value to 0xFE.

Signed-off-by: Adrian Gielniewski <[email protected]>
…arted

Matter Server has to be started before reading attributes from
persistent storage. PWM device initization depends on values of some
attributes.

Signed-off-by: Adrian Gielniewski <[email protected]>
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-connectedhomeip-512 branch from 2d74f4f to 874fe7b Compare November 28, 2024 11:24
@NordicBuilder NordicBuilder removed the DNM label Nov 28, 2024
@thst-nordic
Copy link
Contributor

thst-nordic commented Nov 28, 2024

@nrfconnect/ncs-ci Could you please have a look and review? But honestly, I don't see any changes related to CI.

I don't see anything we need to approve. and you have >2 approvals.
looks like CI will be done in ~30min.

test_chip: upmerge_matter_1_4

Make sure you merge your test branch before or simultaneously.

@ArekBalysNordic ArekBalysNordic merged commit 3712e87 into nrfconnect:main Nov 28, 2024
13 of 14 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.

9 participants