net: rpc: openthread: add networkdiagnostic APIs

[maciejbaczmanski]

Add APIs for getting and resetting network diagnostic TLVs.

[NordicBuilder]

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 12

Inputs:

Sources:

sdk-nrf: PR head: 1900bb3d4dbe88eb693fd0032605106e4af867fb

more details

sdk-nrf:

PR head: 1900bb3d4dbe88eb693fd0032605106e4af867fb
merge base: b4b0ece65ef9c45fd32bea41c788eec7556595b0
target head (main): 8798f86aa24582bd18256cbaca76cb25e25096cb
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 (6)

doc
│  ├── nrf
│  │  ├── libraries
│  │  │  ├── networking
│  │  │  │  │ ot_rpc.rst
samples
│  ├── nrf_rpc
│  │  ├── protocols_serialization
│  │  │  ├── client
│  │  │  │  ├── src
│  │  │  │  │  │ ot_shell.c
subsys
│  ├── net
│  │  ├── openthread
│  │  │  ├── rpc
│  │  │  │  ├── client
│  │  │  │  │  │ ot_rpc_netdiag.c
│  │  │  │  ├── common
│  │  │  │  │  │ ot_rpc_ids.h
│  │  │  │  ├── server
│  │  │  │  │  │ ot_rpc_netdiag.c
tests
│  ├── subsys
│  │  ├── net
│  │  │  ├── openthread
│  │  │  │  ├── rpc
│  │  │  │  │  ├── server
│  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │ netdiag_suite.c

Outputs:

Toolchain

Version: 4ffa2202d5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4ffa2202d5_8bf7ca4353

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

• ◻️ Toolchain - Skipped: existing toolchain is used
• ✅ Build twister
  • sdk-nrf test count: 522
• ✅ Integration tests
  • ✅ test-fw-nrfconnect-rpc
  • ✅ test-fw-nrfconnect-thread
  • ✅ test-fw-nrfconnect-ps

Disabled integration tests

• • desktop52_verification
  • doc-internal
  • test-fw-nrfconnect-apps
  • test-fw-nrfconnect-ble_samples
  • test-fw-nrfconnect-boot
  • test-fw-nrfconnect-chip
  • 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_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-rs
  • test-fw-nrfconnect-tfm
  • test-low-level
  • test-sdk-audio
  • test-sdk-dfu
  • 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]

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-21110/nrf/libraries/networking/ot_rpc.html

[Damian-Nordic]

encoding the bstr size as a separate uint seems a bit redundant but I guess we need a better helper than nrf_rpc_decode_buffer for decoding var-length buffers, so we can leave it for now and fix later :)

[maciejbaczmanski]

yeah. I think it would be nice if we could also return actual size in nrf_rpc_decode_buffer, similar as nrf_rpc_decode_buffer_ptr_and_size