Skip to content

Conversation

@maxd-nordic
Copy link
Contributor

Use production setup of thingy91x bootloader configuration.

Use production setup of thingy91x bootloader configuration.

Signed-off-by: Maximilian Deubel <[email protected]>
@maxd-nordic maxd-nordic added this to the 2.8.0 milestone Oct 23, 2024
@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 Oct 23, 2024
@NordicBuilder
Copy link
Contributor

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

Name Old Revision New Revision Diff
mcuboot nrfconnect/sdk-mcuboot@68b96b8 nrfconnect/sdk-mcuboot#358 nrfconnect/sdk-mcuboot#358/files

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 23, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 9cd9fa55b5c6c02044d79d55c52f54eeca88c6df
mcuboot: PR head: 55beb1b01bef80d8f689c3a630d04ae177dec39a

more details

sdk-nrf:

PR head: 9cd9fa55b5c6c02044d79d55c52f54eeca88c6df
merge base: b2fbe4262015a1219b0e1fc7dde50bb1c7f80296
target head (main): f3704fc50abc4c043ee20f13c2d0760a247cdc67
Diff

mcuboot:

PR head: 55beb1b01bef80d8f689c3a630d04ae177dec39a
merge base: 68b96b802cdeef77ce4200e776afa46f6d3cfb66
target head (main): 7e010864696446fc42e185161d1f73bff563ee29
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 (15)
applications
│  ├── asset_tracker_v2
│  │  ├── sysbuild
│  │  │  ├── mcuboot
│  │  │  │  ├── boards
│  │  │  │  │  ├── thingy91x_nrf9151.conf
│  │  │  │  │  │ thingy91x_nrf9151.overlay
│  ├── connectivity_bridge
│  │  ├── sysbuild
│  │  │  ├── mcuboot
│  │  │  │  ├── boards
│  │  │  │  │  │ thingy91x_nrf5340_cpuapp.conf
│  ├── serial_lte_modem
│  │  ├── sysbuild
│  │  │  ├── mcuboot
│  │  │  │  ├── boards
│  │  │  │  │  ├── thingy91x_nrf9151.conf
│  │  │  │  │  │ thingy91x_nrf9151.overlay
bootloader
│  ├── mcuboot
│  │  ├── boot
│  │  │  ├── zephyr
│  │  │  │  ├── boards
│  │  │  │  │  ├── thingy91x_nrf5340_cpuapp.conf
│  │  │  │  │  ├── thingy91x_nrf9151.conf
│  │  │  │  │  │ thingy91x_nrf9151.overlay
samples
│  ├── cellular
│  │  ├── nrf_cloud_multi_service
│  │  │  ├── sysbuild
│  │  │  │  ├── mcuboot
│  │  │  │  │  ├── boards
│  │  │  │  │  │  ├── thingy91x_nrf9151.conf
│  │  │  │  │  │  │ thingy91x_nrf9151.overlay
│  ├── net
│  │  ├── aws_iot
│  │  │  ├── sysbuild
│  │  │  │  ├── mcuboot
│  │  │  │  │  ├── boards
│  │  │  │  │  │  ├── thingy91x_nrf9151.conf
│  │  │  │  │  │  │ thingy91x_nrf9151.overlay
│  ├── wifi
│  │  ├── shell
│  │  │  ├── sysbuild
│  │  │  │  ├── mcuboot
│  │  │  │  │  ├── boards
│  │  │  │  │  │  ├── thingy91x_nrf9151.conf
│  │  │  │  │  │  │ thingy91x_nrf9151.overlay
west.yml

Outputs:

Toolchain

Version: 3dd8985b56
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:3dd8985b56_912848a074

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 553
  • ❌ Integration tests
    • ✅ test-fw-nrfconnect-boot
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ❌ test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91
    • ❌ test-sdk-wifi
    • ✅ test-sdk-mcuboot
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-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_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • 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-pmic-samples
    • test-sdk-sidewalk

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 Publishing GitHub Action.

@plskeggs plskeggs requested a review from tony-le-24 October 23, 2024 16:01
CONFIG_MCUBOOT_SERIAL=y
CONFIG_MCUBOOT_SERIAL_DIRECT_IMAGE_UPLOAD=y
CONFIG_BOOT_SERIAL_IMG_GRP_IMAGE_STATE=y

Copy link
Contributor

Choose a reason for hiding this comment

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

Seems ok. @tony-le-24 can you check if the various additional nightly and weekly tests still work with this change -- any of them that run on the Thingy:91X?

Copy link
Contributor

Choose a reason for hiding this comment

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

I will, thanks

Copy link
Contributor

Choose a reason for hiding this comment

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

I ran it on my PR for Thingy91:X testing, things seem green. LGTM!

@nordicjm
Copy link
Contributor

sdk-mcuboot PR has been merged, have brought in manifest update and commit here into #17567 to save on CI

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-mcuboot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants