Skip to content

Conversation

@ahasztag
Copy link
Contributor

@ahasztag ahasztag commented Apr 2, 2025

To bring the newest changes - adding an error message if too large sequence number

@ahasztag ahasztag requested review from robertstypa and tomchy April 2, 2025 11:52
@ahasztag ahasztag requested review from a team as code owners April 2, 2025 11:52
@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 Apr 2, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 2, 2025

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

Name Old Revision New Revision Diff
suit-generator nrfconnect/suit-generator@1817ea1 nrfconnect/suit-generator@0725f0c (main) nrfconnect/[email protected]

All manifest checks OK

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 2, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 325d4ad007a7219bfca942ecdc1d4ae72d035c6e
suit-generator: PR head: 0725f0c362ab8428048a977080045bb5e73e0d9d

more details

sdk-nrf:

PR head: 325d4ad007a7219bfca942ecdc1d4ae72d035c6e
merge base: 76bb5cd6f84c6857befa07176e4b3a60400c94f6
target head (main): 76bb5cd6f84c6857befa07176e4b3a60400c94f6
Diff

suit-generator:

PR head: 0725f0c362ab8428048a977080045bb5e73e0d9d
merge base: 1817ea1aac7cd0cfae4d50b8aea897d9140833ef
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 (3)
modules
│  ├── lib
│  │  ├── suit-generator
│  │  │  ├── suit_generator
│  │  │  │  ├── suit
│  │  │  │  │  │ manifest.py
│  │  │  ├── tests
│  │  │  │  │ test_suit_envelope.py
west.yml

Outputs:

Toolchain

Version: 7cbc0036f4
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:7cbc0036f4_8bf7ca4353

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 172
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-sdk-dfu
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-boot
    • 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_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • 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
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-low-level
    • test-sdk-audio
    • 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

github-actions bot commented Apr 2, 2025

You can find the documentation preview for this PR here.

@ahasztag ahasztag added this to the 3.0.0 milestone Apr 3, 2025
@ahasztag ahasztag added the bugfix Fixes a known bug label Apr 3, 2025
@ahasztag ahasztag force-pushed the NCSDK-32183_detect_to_large_seq_number branch from 272fa8d to ab83569 Compare April 4, 2025 08:12
@NordicBuilder NordicBuilder removed the DNM label Apr 4, 2025
To bring the newest changes - adding an error message
if too large sequence number

Signed-off-by: Artur Hadasz <[email protected]>
@ahasztag ahasztag force-pushed the NCSDK-32183_detect_to_large_seq_number branch from ab83569 to 325d4ad Compare April 4, 2025 10:20
@tomchy tomchy merged commit b8c6d1f into nrfconnect:main Apr 8, 2025
16 checks passed
@ahasztag ahasztag deleted the NCSDK-32183_detect_to_large_seq_number branch September 17, 2025 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport v3.0-branch bugfix Fixes a known bug changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-suit-generator

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants