Skip to content

Conversation

@simensrostad
Copy link
Contributor

@simensrostad simensrostad commented Jun 30, 2025

Make sure errors are forwarded to the application if reconnection fails.

@simensrostad simensrostad requested a review from a team as a code owner June 30, 2025 02:21
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jun 30, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jun 30, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: cbf0e22bc1a71d720dc1460d59b8c1f68043f145

more details

sdk-nrf:

PR head: cbf0e22bc1a71d720dc1460d59b8c1f68043f145
merge base: 15d1605416f56a5053397351ccf934d623c63b05
target head (main): e398867be1a71dbf382a0ec2fc129ae782ebdc40
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 (1)
subsys
│  ├── net
│  │  ├── lib
│  │  │  ├── downloader
│  │  │  │  ├── src
│  │  │  │  │  │ downloader.c

Outputs:

Toolchain

Version: 11ad62c2a5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:11ad62c2a5_bba2ea5f2e

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 361
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ✅ 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-sdk-dfu
    • ⚠️ test-fw-nrfconnect-nrf_lrcs_mosh
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • 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_thingy91
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • 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

Copy link
Contributor

@eivindj-nordic eivindj-nordic left a comment

Choose a reason for hiding this comment

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

Please add changelog entry :)

@simensrostad simensrostad force-pushed the preventfinishedmessage branch from b84c35a to 8811505 Compare July 1, 2025 20:27
@simensrostad simensrostad requested a review from a team as a code owner July 1, 2025 20:27
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. and removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Jul 1, 2025
@simensrostad
Copy link
Contributor Author

Please add changelog entry :)

@github-actions
Copy link

github-actions bot commented Jul 1, 2025

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-23038/nrf/releases_and_maturity/releases/release-notes-changelog.html

@simensrostad simensrostad force-pushed the preventfinishedmessage branch 4 times, most recently from f7bb3b2 to 7c7f2f0 Compare July 1, 2025 21:41
@simensrostad simensrostad changed the title net: lib: downloader: Add additional errno check net: lib: downloader: Fix error check Jul 1, 2025
@simensrostad simensrostad force-pushed the preventfinishedmessage branch from 7c7f2f0 to b9603eb Compare July 1, 2025 23:29
@simensrostad simensrostad force-pushed the preventfinishedmessage branch 2 times, most recently from 3b5bc6a to ba3f042 Compare July 2, 2025 14:11
@simensrostad simensrostad added the bug Something isn't working label Aug 4, 2025
@simensrostad simensrostad added this to the 3.1.0 milestone Aug 4, 2025
Make sure errors are forwarded to the application if
reconnection fails.

Signed-off-by: Simen S. Røstad <[email protected]>
@simensrostad simensrostad force-pushed the preventfinishedmessage branch from ba3f042 to cbf0e22 Compare August 4, 2025 09:00
@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. and removed doc-required PR must not be merged without tech writer approval. labels Aug 4, 2025
@divipillai
Copy link
Contributor

FYI - The release note entry is added in PR #23631.

@rlubos rlubos merged commit f7406fd into nrfconnect:main Aug 4, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport v3.1-branch bug Something isn't working changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants