Skip to content

rpc: samples: Extend verbosity of RPC errors#28384

Open
MarekPorwisz wants to merge 1 commit intonrfconnect:mainfrom
MarekPorwisz:detaled_error_reporting
Open

rpc: samples: Extend verbosity of RPC errors#28384
MarekPorwisz wants to merge 1 commit intonrfconnect:mainfrom
MarekPorwisz:detaled_error_reporting

Conversation

@MarekPorwisz
Copy link
Copy Markdown
Contributor

Added support for more detailed RPC error info to the protocol_serialization sample.
Updated protocols rpc implementation not to loose the file/line/function info due to the usage of a common error reporting function.

@MarekPorwisz MarekPorwisz requested review from a team as code owners April 24, 2026 11:40
@NordicBuilder NordicBuilder requested review from a team April 24, 2026 11:41
Added support for more detailed RPC error info to the
protocol_serialization sample.
Updated protocols rpc implementation not to loose the file/line/function
info due to the usage of a common error reporting function.

Signed-off-by: Marek Porwisz <marek.porwisz@nordicsemi.no>
@MarekPorwisz MarekPorwisz force-pushed the detaled_error_reporting branch from d4588c0 to dbccb18 Compare April 24, 2026 11:46
@MarekPorwisz MarekPorwisz requested review from a team as code owners April 24, 2026 11:46
@NordicBuilder
Copy link
Copy Markdown
Contributor

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

Name Old Revision New Revision Diff
nrfxlib nrfconnect/sdk-nrfxlib@5290128 nrfconnect/sdk-nrfxlib#2092 nrfconnect/sdk-nrfxlib#2092/files

DNM label due to: 1 project with PR revision

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

@NordicBuilder
Copy link
Copy Markdown
Contributor

NordicBuilder commented Apr 24, 2026

CI Information

To view the history of this post, click the 'edited' button above
Build number: 2

Inputs:

Sources:

sdk-nrf: PR head: dbccb1884137e3982f3bc151b73ec4cc6186ffb7
nrfxlib: PR head: 885545fd1065596372e608faab95916773d83b39

more details

sdk-nrf:

PR head: dbccb1884137e3982f3bc151b73ec4cc6186ffb7
merge base: d91ee58723d21ff66e31c3d29046953f2f0c7834
target head (main): dd3c1d9b4b8b15b6a75f035cc69755315933c5ef
Diff

nrfxlib:

PR head: 885545fd1065596372e608faab95916773d83b39
merge base: 529012899ffb2aa8ef69cbbb315eaf2848737aca
target head (main): ce8170de81d9a0d6887194543111d43617c7c6cb
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 (22)
nrfxlib
│  ├── nrf_rpc
│  │  ├── Kconfig
│  │  ├── include
│  │  │  │ nrf_rpc.h
│  │  │ nrf_rpc.c
samples
│  ├── nrf_rpc
│  │  ├── protocols_serialization
│  │  │  ├── client
│  │  │  │  │ verbose.conf
│  │  │  ├── server
│  │  │  │  │ verbose.conf
subsys
│  ├── bluetooth
│  │  ├── rpc
│  │  │  ├── client
│  │  │  │  ├── bt_rpc_conn_client.c
│  │  │  │  ├── bt_rpc_gap_client.c
│  │  │  │  │ bt_rpc_gatt_client.c
│  │  │  ├── common
│  │  │  │  │ bt_rpc_common.h
│  │  │  ├── host
│  │  │  │  ├── bt_rpc_conn_host.c
│  │  │  │  ├── bt_rpc_crypto_host.c
│  │  │  │  ├── bt_rpc_gap_host.c
│  │  │  │  ├── bt_rpc_gatt_host.c
│  │  │  │  │ bt_rpc_internal_host.c
│  ├── dm
│  │  ├── rpc
│  │  │  ├── common
│  │  │  │  │ dm_rpc_common.h
│  │  │  ├── host
│  │  │  │  │ dm_rpc_host.c
│  ├── net
│  │  ├── openthread
│  │  │  ├── rpc
│  │  │  │  ├── client
│  │  │  │  │  │ ot_rpc_udp.c
│  │  │  │  ├── common
│  │  │  │  │  ├── ot_rpc_common.c
│  │  │  │  │  │ ot_rpc_common.h
│  ├── nfc
│  │  ├── rpc
│  │  │  ├── common
│  │  │  │  ├── nfc_rpc_common.c
│  │  │  │  │ nfc_rpc_common.h
west.yml

Outputs:

Toolchain

Version: 53ac68dd62
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:53ac68dd62_5ea73affbf

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ❌ Build twister
    • sdk-nrf test count: 1560
  • ❌ Integration tests
    • ❌ test_ble_nrf_config
    • ❌ test-fw-nrfconnect-ble_samples
    • ❌ test-fw-nrfconnect-nfc
    • ❌ test-fw-nrfconnect-rpc
    • ❌ test-fw-nrfconnect-thread-main
    • ❌ test-sdk-find-my
    • ❌ test-sdk-dfu
    • ❌ test-fw-nrfconnect-ps-main
    • ⚠️ test-fw-nrfconnect-nrf-iot_cloud
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-low-level
    • test-sdk-mcuboot
    • test-sdk-wifi

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants