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

NCSDK-30461: Move SUIT MPI to sysbuild #19016

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

tomchy
Copy link
Contributor

@tomchy tomchy commented Nov 21, 2024

No description provided.

@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 21, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 21, 2024

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

Name Old Revision New Revision Diff
suit-generator nrfconnect/suit-generator@9c5e8f1 (ncs) nrfconnect/suit-generator#156 nrfconnect/suit-generator#156/files
zephyr nrfconnect/sdk-zephyr@01976a7 (main) nrfconnect/sdk-zephyr#2289 nrfconnect/sdk-zephyr#2289/files

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 21, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: d71648a9aca7916bceb3881da65ffefc2feb7972
suit-generator: PR head: 4b93e13b0a586397f0ea49dcf4d8ba0b683c34b3
zephyr: PR head: 5f2f74aa78c2a8fa328ddc855c2ca2c6025ad7cd

more details

sdk-nrf:

PR head: d71648a9aca7916bceb3881da65ffefc2feb7972
merge base: 4c8412c6f2a0a7a3f3a15a9bdf0a73fcd0b046c5
target head (main): 155fd870faae7621b2825301f1b7cee6fb84f188
Diff

suit-generator:

PR head: 4b93e13b0a586397f0ea49dcf4d8ba0b683c34b3
merge base: 7bf6a6bfd3df4249e198f83cf6582c34912cce03
target head (ncs): 9c5e8f16ef49e5ffbdeabc87ffd60b82b1fd72b0
Diff

zephyr:

PR head: 5f2f74aa78c2a8fa328ddc855c2ca2c6025ad7cd
merge base: 01976a7968bd9fa0a601079c35022e06bfc809e7
target head (main): 01976a7968bd9fa0a601079c35022e06bfc809e7
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 (68)
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_direct.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: b77d8c1312
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:b77d8c1312_b7b3768ce5

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ 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
    • ✅ doc-internal
    • ✅ 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
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.

@tomchy 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 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 tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from a8e5016 to 3199797 Compare November 26, 2024 12:53
@tomchy tomchy marked this pull request as ready for review November 27, 2024 09:36
@tomchy tomchy requested review from a team as code owners November 27, 2024 09:36
@tomchy tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from 3199797 to cdb5e5d Compare November 27, 2024 09:37
@tomchy tomchy requested a review from a team as a code owner November 27, 2024 09:37
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]>
@tomchy tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from cdb5e5d to d71648a Compare November 27, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNM doc-required PR must not be merged without tech writer approval. manifest manifest-suit-generator manifest-zephyr
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants