Skip to content

docs: Describe the KRKNWK-22680 issue - #90

Open
ArekBalysNordic wants to merge 2 commits into
nrfconnect:mainfrom
ArekBalysNordic:matter_1_0_1_ki
Open

ArekBalysNordic wants to merge 2 commits into
nrfconnect:mainfrom
ArekBalysNordic:matter_1_0_1_ki

Conversation

@ArekBalysNordic

Copy link
Copy Markdown
Collaborator

The issue is related to moving DAC private key to ITS instead of KMU by default.

@ArekBalysNordic
ArekBalysNordic requested review from a team as code owners October 23, 2026 08:23
@github-actions

Copy link
Copy Markdown

You can find the documentation preview for this PR here.

Preview links for modified Matter add-on documents:

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The documented recovery and protection are absent from the pinned SDK, and the ITS workaround incorrectly includes KMU-backed devices.

2 open issues
What changed in this PR

This PR documents KRKNWK-22680, where nRF54L devices use ITS instead of KMU for DAC private-key storage, and adds v1.0.1 upgrade guidance.

Changes:

  • Adds known-issue details, migration paths, and v1.0.1 release notes.
  • Updates documentation navigation and enables known-issue filtering.
  • Corrects the filtering extension’s JavaScript filename handling.
File Description
docs/​versions.json Adds v1.0.1 to the version list.
docs/​release_notes/​release_notes_v101.rst Describes intended storage fixes and boot protection.
docs/​release_notes.rst Links the v1.0.1 release notes.
docs/​migration/​migration_guide_v1.0.1.rst Adds backend-specific migration and validation guidance.
docs/​migration_guides.rst Links the v1.0.1 migration guide.
docs/​known_issues.rst Documents storage mismatches and adds issue filtering.
docs/​_extensions/​page_filter.py Uses the script object’s filename for module imports.

Review effort: Balanced

Comment thread docs/known_issues.rst Outdated
Comment thread docs/release_notes/release_notes_v101.rst Outdated
Copilot AI balanced review requested due to automatic review settings October 23, 2026 09:08
@NordicBuilder

NordicBuilder commented Oct 23, 2026 •

Copy link
Copy Markdown

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

Name Old Revision New Revision Diff
matter nrfconnect/sdk-connectedhomeip@d92ebd4 (master,ncs-matter-preview-nfc-unpowered-commissioning-branch) nrfconnect/sdk-connectedhomeip#767 nrfconnect/sdk-connectedhomeip#767/files

⛔ DNM label due to: 1 project with PR revision

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

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The workaround needs to distinguish existing key locations, and the Matter dependency needs a validated immutable revision for reproducible builds.

2 open issues
2 resolved since last review

Review effort: Balanced

Comment thread docs/known_issues.rst Outdated
Comment thread west.yml Outdated
Copilot AI balanced review requested due to automatic review settings October 23, 2026 09:19
Updated of sdk-connectedhomeip.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
Document the v1.0.0 nRF54L issue where the DAC private key could be
stored in PSA ITS instead of CRACEN KMU, and how v1.0.1 restores KMU
defaults on NCS v3.4.x.

Update migration guidance, known issues, and release notes for
CONFIG_CHIP_CRYPTO_PSA_DAC_PRIV_KEY_KMU_ITS_FALLBACK (default): units
that already have the key in ITS can run KMU-default firmware, attest
with the ITS copy, and re-provision later when KMU-only storage is
required.

Split KRKNWK-22680 guidance by where the key was first migrated (KMU
vs ITS) so KMU-backed devices are directed to Path A instead of ITS
Kconfig that cannot access a KMU-resident key.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The dependency update drops the existing OTA timeout fix and permits a factory-reset configuration that can permanently erase an ITS-resident DAC key.

2 open issues
2 resolved since last review

Review effort: Balanced

Comment thread west.yml Outdated
Comment thread west.yml Outdated
Copilot AI balanced review requested due to automatic review settings October 23, 2026 09:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The dependency needs an immutable revision, and migration guidance needs to clarify factory-reset defaults and removal of existing ITS keys during re-provisioning.

3 open issues
2 resolved since last review

Review effort: Balanced

Comment thread west.yml
Comment thread docs/migration/migration_guide_v1.0.1.rst
Comment thread docs/migration/migration_guide_v1.0.1.rst

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DNM Do not merge - prevent from merging manifest manifest-matter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants