NCSDK-30461: Move SUIT MPI to sysbuild #19016

tomchy · 2024-11-21T13:01:01Z

No description provided.

NordicBuilder · 2024-11-21T13:01:44Z

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

Name	Old Revision	New Revision	Diff
suit-generator	nrfconnect/suit-generator@`0e66051`	nrfconnect/suit-generator@`9bd5f35` (`ncs`)	nrfconnect/[email protected]
zephyr	nrfconnect/sdk-zephyr@`711346a`	nrfconnect/sdk-zephyr@`6e53c57` (`main`)	nrfconnect/[email protected]

✅ All manifest checks OK

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

NordicBuilder · 2024-11-21T13:04:14Z

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 98de316f410945fa666d4954ecf2a39d3821e3c7
suit-generator: PR head: 9bd5f35396bea61f6c8e6aaebc00635364522ce6
zephyr: PR head: 6e53c5702ecb1f3dddfdc051ff487550d6c24672

more details

sdk-nrf:

PR head: 98de316f410945fa666d4954ecf2a39d3821e3c7
merge base: cabe5e5d26fe555fdc9851634bc5779d16faab6d
target head (main): cabe5e5d26fe555fdc9851634bc5779d16faab6d
Diff

suit-generator:

PR head: 9bd5f35396bea61f6c8e6aaebc00635364522ce6
merge base: 0e660519a58adba119d552e90fcbca8e430490a4
Diff

zephyr:

PR head: 6e53c5702ecb1f3dddfdc051ff487550d6c24672
merge base: 711346aa377a65263784cd0d37efc9230e68229b
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 (70)

CODEOWNERS
applications
│  ├── machine_learning
│  │  ├── remote
│  │  │  ├── boards
│  │  │  │  │ nrf54h20dk_nrf54h20_cpuppr.conf
cmake
│  ├── sysbuild
│  │  ├── suit.cmake
│  │  ├── suit_provisioning.cmake
│  │  ├── suit_provisioning_nrf54h20.cmake
│  │  │ suit_utilities.cmake
config
│  ├── suit
│  │  ├── templates
│  │  │  ├── nrf54h20
│  │  │  │  ├── default
│  │  │  │  │  ├── v1
│  │  │  │  │  │  ├── app_envelope.yaml.jinja2
│  │  │  │  │  │  ├── app_recovery_envelope_app_local.yaml.jinja2
│  │  │  │  │  │  ├── app_recovery_envelope_direct.yaml.jinja2
│  │  │  │  │  │  ├── app_recovery_local_envelope.yaml.jinja2
│  │  │  │  │  │  ├── rad_envelope.yaml.jinja2
│  │  │  │  │  │  ├── rad_recovery_envelope.yaml.jinja2
│  │  │  │  │  │  │ root_with_binary_nordic_top.yaml.jinja2
│  │  │  │  ├── matter
│  │  │  │  │  ├── v1
│  │  │  │  │  │  ├── app_envelope.yaml.jinja2
│  │  │  │  │  │  ├── rad_envelope.yaml.jinja2
│  │  │  │  │  │  │ root_with_binary_nordic_top.yaml.jinja2
doc
│  ├── nrf
│  │  ├── app_dev
│  │  │  ├── config_and_build
│  │  │  │  ├── sysbuild
│  │  │  │  │  │ sysbuild_forced_options.rst
│  │  │  ├── device_guides
│  │  │  │  ├── nrf54h
│  │  │  │  │  ├── ug_nrf54h20_suit_customize_dfu.rst
│  │  │  │  │  ├── ug_nrf54h20_suit_customize_qsg.rst
│  │  │  │  │  │ ug_nrf54h20_suit_recovery.rst
modules
│  ├── lib
│  │  ├── suit-generator
│  │  │  ├── ncs
│  │  │  │  ├── Kconfig
│  │  │  │  ├── app_envelope_encrypted.yaml.jinja2
│  │  │  │  │ root_with_nordic_top_envelope.yaml.jinja2
│  │  │  ├── suit_generator
│  │  │  │  │ cmd_image.py
│  │  │  ├── tests
│  │  │  │  │ test_cmd_image.py
samples
│  ├── suit
│  │  ├── flash_companion
│  │  │  │ prj.conf
│  │  ├── recovery
│  │  │  ├── prj.conf
│  │  │  ├── sysbuild
│  │  │  │  │ hci_ipc.conf
│  │  ├── smp_transfer
│  │  │  ├── suit
│  │  │  │  ├── nrf54h20
│  │  │  │  │  ├── app_envelope_extflash.yaml.jinja2
│  │  │  │  │  ├── rad_envelope_extflash.yaml.jinja2
│  │  │  │  │  │ root_with_binary_nordic_top_extflash.yaml.jinja2
│  │  │  ├── sysbuild
│  │  │  │  │ hci_ipc.conf
subsys
│  ├── suit
│  │  ├── CMakeLists.txt
│  │  ├── Kconfig
│  │  ├── provisioning
│  │  │  │ CMakeLists.txt
sysbuild
│  ├── CMakeLists.txt
│  ├── Kconfig.suit_provisioning
│  ├── Kconfig.sysbuild
│  ├── suit_provisioning
│  │  ├── Kconfig.nrf54h20
│  │  │ Kconfig.template.manifest_config
west.yml
zephyr
│  ├── scripts
│  │  ├── west_commands
│  │  │  ├── runners
│  │  │  │  ├── __init__.py
│  │  │  │  ├── blackmagicprobe.py
│  │  │  │  ├── bossac.py
│  │  │  │  ├── canopen_program.py
│  │  │  │  ├── core.py
│  │  │  │  ├── dfu.py
│  │  │  │  ├── esp32.py
│  │  │  │  ├── ezflashcli.py
│  │  │  │  ├── intel_adsp.py
│  │  │  │  ├── intel_cyclonev.py
│  │  │  │  ├── jlink.py
│  │  │  │  ├── linkserver.py
│  │  │  │  ├── mdb.py
│  │  │  │  ├── native.py
│  │  │  │  ├── nios2.py
│  │  │  │  ├── nrf_common.py
│  │  │  │  ├── nrfutil.py
│  │  │  │  ├── nsim.py
│  │  │  │  ├── openocd.py
│  │  │  │  ├── pyocd.py
│  │  │  │  ├── renode-robot.py
│  │  │  │  ├── renode.py
│  │  │  │  ├── silabs_commander.py
│  │  │  │  ├── spi_burn.py
│  │  │  │  ├── stm32cubeprogrammer.py
│  │  │  │  ├── stm32flash.py
│  │  │  │  ├── teensy.py
│  │  │  │  │ uf2.py
│  │  │  ├── tests
│  │  │  │  │ test_canopen_program.py

Outputs:

Toolchain

Version:
Build docker image:

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

◻️ Toolchain
◻️ Build twister
◻️ Integration tests
- ◻️ test-sdk-audio
- ◻️ desktop52_verification
- ◻️ test-fw-nrfconnect-boot
- ◻️ test-fw-nrfconnect-apps
- ◻️ test_ble_nrf_config
- ◻️ test-fw-nrfconnect-ble_mesh
- ◻️ test-fw-nrfconnect-ble_samples
- ◻️ test-fw-nrfconnect-chip
- ◻️ test-fw-nrfconnect-nfc
- ◻️ test-fw-nrfconnect-nrf-iot_libmodem-nrf
- ◻️ test-fw-nrfconnect-nrf-iot_serial_lte_modem
- ◻️ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- ◻️ test-fw-nrfconnect-nrf-iot_samples
- ◻️ test-fw-nrfconnect-nrf-iot_lwm2m
- ◻️ test-fw-nrfconnect-nrf-iot_thingy91
- ◻️ test-fw-nrfconnect-nrf_crypto
- ◻️ test-fw-nrfconnect-rpc
- ◻️ test-fw-nrfconnect-rs
- ◻️ test-fw-nrfconnect-fem
- ◻️ test-fw-nrfconnect-tfm
- ◻️ test-fw-nrfconnect-thread
- ◻️ test-fw-nrfconnect-zigbee
- ◻️ test-sdk-find-my
- ◻️ test-fw-nrfconnect-nrf-iot_mosh
- ◻️ test-fw-nrfconnect-nrf-iot_positioning
- ◻️ test-sdk-sidewalk
- ◻️ test-sdk-wifi
- ◻️ test-low-level
- ◻️ test-fw-nrfconnect-nrf-iot_nrf_provisioning
- ◻️ test-sdk-pmic-samples
- ◻️ test-sdk-mcuboot
- ◻️ test-sdk-dfu
- ◻️ test-fw-nrfconnect-ps
- ◻️ test-secdom-samples-public
- ⚠️ test-sdk-dfu

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

NordicBuilder · 2024-11-21T13:18:32Z

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.

b-gent

doc looks good

NordicBuilder · 2024-12-05T10:53:38Z

Memory footprint analysis revealed the following potential issues

sample.matter.template.release[nrf7002dk/nrf5340/cpuapp]: High ROM usage: 820678[B] - link (cc: @kkasperczyk-no @ArekBalysNordic @markaj-nordic)
sample.matter.template.debug[nrf7002dk/nrf5340/cpuapp]: High ROM usage: 911846[B] - link (cc: @kkasperczyk-no @ArekBalysNordic @markaj-nordic)
applications.sdp.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 8568[B] - link (cc: @nrfconnect/ncs-ll-ursus)

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

Remove usage of suit-install-legacy from manifest templates. Ref: NCSDK-NONE Signed-off-by: Tomasz Chyrowicz <[email protected]>

Use sysbuild to build and configure SUIT Manifest Provisioning Information. Ref: NCSDK-30461 Signed-off-by: Tomasz Chyrowicz <[email protected]>

Pull-in required changes in suit-generator and sdk-zephyr. Ref: NCSDK-30461 Signed-off-by: Tomasz Chyrowicz <[email protected]>

Add a warning if user tries to build an app with default VID and CID values. Ref: NCSDK-29986 Signed-off-by: Tomasz Chyrowicz <[email protected]>

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 21, 2024

NordicBuilder added manifest-zephyr manifest-suit-generator DNM labels Nov 21, 2024

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch 4 times, most recently from eeb00bd to a8e5016 Compare November 25, 2024 16:16

github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Nov 25, 2024

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from a8e5016 to 3199797 Compare November 26, 2024 12:53

tomchy marked this pull request as ready for review November 27, 2024 09:36

tomchy requested a review from a team as a code owner November 27, 2024 09:36

tomchy requested a review from a team November 27, 2024 09:36

tomchy requested review from a team as code owners November 27, 2024 09:36

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from 3199797 to cdb5e5d Compare November 27, 2024 09:37

tomchy requested a review from a team November 27, 2024 09:37

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from cdb5e5d to d71648a Compare November 27, 2024 09:46

nordicjm approved these changes Nov 27, 2024

View reviewed changes

tomchy requested review from adsz-nordic, ahasztag, kszromek-nordic and robertstypa November 27, 2024 12:19

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from 80cc908 to e720e78 Compare November 29, 2024 13:07

tomchy requested a review from b-gent November 29, 2024 13:07

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from e720e78 to 3ec3cd2 Compare November 29, 2024 13:43

b-gent approved these changes Nov 29, 2024

View reviewed changes

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch 4 times, most recently from a52d80f to f067e4c Compare December 5, 2024 08:20

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch 2 times, most recently from a1d8c6c to a5083b3 Compare December 6, 2024 08:15

tomchy added this to the 2.9.0-nRF54H20 milestone Dec 6, 2024

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch 4 times, most recently from d2d2954 to a0cd85d Compare December 9, 2024 15:10

adsz-nordic approved these changes Dec 10, 2024

View reviewed changes

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch 3 times, most recently from 805fe83 to 772e928 Compare December 11, 2024 15:39

tomchy mentioned this pull request Dec 12, 2024

manifest: sdk-zephyr: suit: Move MPI to sysbuild Kconfig #19014

Closed

tomchy added 4 commits December 12, 2024 10:44

suit: Remove suit-install-legacy

2537529

Remove usage of suit-install-legacy from manifest templates. Ref: NCSDK-NONE Signed-off-by: Tomasz Chyrowicz <[email protected]>

suit: Move MPI to sysbuild Kconfig

5343c02

Use sysbuild to build and configure SUIT Manifest Provisioning Information. Ref: NCSDK-30461 Signed-off-by: Tomasz Chyrowicz <[email protected]>

manifest: Update suit-generator and sdk-zephyr

be9bab7

Pull-in required changes in suit-generator and sdk-zephyr. Ref: NCSDK-30461 Signed-off-by: Tomasz Chyrowicz <[email protected]>

suit: Add warning about default VID/CID

98de316

Add a warning if user tries to build an app with default VID and CID values. Ref: NCSDK-29986 Signed-off-by: Tomasz Chyrowicz <[email protected]>

tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from 772e928 to 98de316 Compare December 12, 2024 09:46

NordicBuilder removed the DNM label Dec 12, 2024

rlubos merged commit 3a51fda into nrfconnect:main Dec 12, 2024
13 of 14 checks passed

tomchy mentioned this pull request Dec 12, 2024

suit: nRF9280 SUIT support #18496

Merged

tomchy deleted the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch January 24, 2025 12:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

NCSDK-30461: Move SUIT MPI to sysbuild #19016

NCSDK-30461: Move SUIT MPI to sysbuild #19016

Uh oh!

tomchy commented Nov 21, 2024

Uh oh!

NordicBuilder commented Nov 21, 2024 •

edited

Loading

Uh oh!

NordicBuilder commented Nov 21, 2024 •

edited

Loading

Uh oh!

NordicBuilder commented Nov 21, 2024

Uh oh!

b-gent left a comment

Uh oh!

NordicBuilder commented Dec 5, 2024 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

NCSDK-30461: Move SUIT MPI to sysbuild #19016

NCSDK-30461: Move SUIT MPI to sysbuild #19016

Uh oh!

Conversation

tomchy commented Nov 21, 2024

Uh oh!

NordicBuilder commented Nov 21, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

NordicBuilder commented Nov 21, 2024 • 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!

NordicBuilder commented Nov 21, 2024

Uh oh!

b-gent left a comment

Choose a reason for hiding this comment

Uh oh!

NordicBuilder commented Dec 5, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Memory footprint analysis revealed the following potential issues

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

NordicBuilder commented Nov 21, 2024 •

edited

Loading

NordicBuilder commented Nov 21, 2024 •

edited

Loading

NordicBuilder commented Dec 5, 2024 •

edited

Loading