Skip to content

Conversation

@ArekBalysNordic
Copy link
Contributor

Now, we copy the NOC key from volatile to persistent space during the commissioning OP keypair. Currently, psa_copy_key does not work for KMU, so we must export this key and import it again as a workaround. This PR is a workaround and can be fully reverted once the NCSDK-34997 issue is resolved.

@ArekBalysNordic ArekBalysNordic requested review from a team as code owners August 20, 2025 10:02
@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 Aug 20, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 20, 2025

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

Name Old Revision New Revision Diff
matter nrfconnect/sdk-connectedhomeip@fe650a3 (master) nrfconnect/sdk-connectedhomeip#640 nrfconnect/sdk-connectedhomeip#640/files

DNM label due to: 1 project with PR revision

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 20, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 32db8d28792ebb798c7bade3b33c1eb48c80a830
matter: PR head: 075678873285e7c2cc08d5de1e6f2dac22bbafcb

more details

sdk-nrf:

PR head: 32db8d28792ebb798c7bade3b33c1eb48c80a830
merge base: b4058d7d06620b71fff81757ecf8eb119716bc45
target head (main): 7f177c3c099679ecbd965ec8c6fe4766fb5ebc60
Diff

matter:

PR head: 075678873285e7c2cc08d5de1e6f2dac22bbafcb
merge base: fe650a3ee4948ef1a2edd55a7fe4f6eb561c9e64
target head (master): fe650a3ee4948ef1a2edd55a7fe4f6eb561c9e64
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 (8)
modules
│  ├── lib
│  │  ├── matter
│  │  │  ├── src
│  │  │  │  ├── crypto
│  │  │  │  │  ├── PSAOperationalKeystore.cpp
│  │  │  │  │  │ PSAOperationalKeystore.h
│  │  │  │  ├── platform
│  │  │  │  │  ├── nrfconnect
│  │  │  │  │  │  ├── BUILD.gn
│  │  │  │  │  │  ├── KMUKeyAllocator.h
│  │  │  │  │  │  │ KMUOperationalKeystore.h
samples
│  ├── matter
│  │  ├── common
│  │  │  ├── src
│  │  │  │  ├── app
│  │  │  │  │  ├── matter_init.cpp
│  │  │  │  │  │ matter_init.h
west.yml

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: 94
  • ✅ 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
Copy link

You can find the documentation preview for this PR here.

@ArekBalysNordic ArekBalysNordic force-pushed the kmu_workaround branch 2 times, most recently from 18362d4 to 93c8897 Compare August 20, 2025 12:22
Now, we copy the NOC key from volatile to persistent
space during the commissioning OP keypair.
Currently, psa_copy_key does not work for KMU, so we
must export this key and import it again as a workaround.
This PR is a workaround and can be fully reverted once
the NCSDK-34997 issue is resolved.

Signed-off-by: Arkadiusz Balys <[email protected]>
@github-actions
Copy link

This pull request has been marked as stale because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 7 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Sep 23, 2025
@github-actions github-actions bot closed this Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-matter Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants