nrf_rpc: Add command to obtain version of remote.

[canisLupus1313]

Commit adds command to pbtain the remote ncs commit sha version.

[NordicBuilder]

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 1cf2330a882dca5849351a0e576714e567a48117

more details

sdk-nrf:

PR head: 1cf2330a882dca5849351a0e576714e567a48117
merge base: af6d9851d03b8bb5471fe5f2cc107b9876dd083b
target head (main): e2cfdfd0f4c8b30e89463b7f421b4e27f2f1f35e
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 (30)

CODEOWNERS
doc
│  ├── nrf
│  │  ├── libraries
│  │  │  ├── nrf_rpc
│  │  │  │  │ nrf_rpc_dev_info.rst
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
include
│  ├── nrf_rpc
│  │  │ nrf_rpc_dev_info.h
samples
│  ├── nrf_rpc
│  │  ├── protocols_serialization
│  │  │  ├── client
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ dev_info_shell.c
│  │  │  ├── server
│  │  │  │  │ prj.conf
subsys
│  ├── nrf_rpc
│  │  ├── CMakeLists.txt
│  │  ├── Kconfig
│  │  ├── dev_info
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  ├── client
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  │ dev_info_client.c
│  │  │  ├── common
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── dev_info_rpc.c
│  │  │  │  │ dev_info_rpc_ids.h
│  │  │  ├── server
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  │ dev_info_server.c
│  │  ├── include
│  │  │  │ nrf_rpc_os.h
tests
│  ├── subsys
│  │  ├── nrf_rpc
│  │  │  ├── dev_info
│  │  │  │  ├── client
│  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  ├── prj.conf
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ dev_info_suite.c
│  │  │  │  │  │ testcase.yaml
│  │  │  │  ├── common
│  │  │  │  │  ├── nrf_rpc_single_thread.c
│  │  │  │  │  │ test_rpc_env.h
│  │  │  │  ├── server
│  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  ├── prj.conf
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ dev_info_suite.c
│  │  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: b77d8c1312
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:b77d8c1312_912848a074

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

• ◻️ Toolchain - Skipped: existing toolchain is used
• ✅ Build twister - Skipped: Skipping Build & Test as it succeeded in a previous run: 18
• ✅ Integration tests
  • ✅ test_ble_nrf_config - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-fw-nrfconnect-ble_samples - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-fw-nrfconnect-rpc - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-fw-nrfconnect-rs
  • ✅ test-fw-nrfconnect-fem - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-sdk-dfu - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-fw-nrfconnect-ps - Skipped: Job was skipped as it succeeded in a previous run

Disabled integration tests

• • desktop52_verification
  • doc-internal
  • test-fw-nrfconnect-apps
  • test-fw-nrfconnect-ble_mesh
  • test-fw-nrfconnect-boot
  • test-fw-nrfconnect-chip
  • 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_serial_lte_modem
  • test-fw-nrfconnect-nrf-iot_thingy91
  • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
  • test-fw-nrfconnect-nrf_crypto
  • test-fw-nrfconnect-tfm
  • test-fw-nrfconnect-thread
  • test-fw-nrfconnect-zigbee
  • test-low-level
  • test-sdk-audio
  • test-sdk-find-my
  • test-sdk-mcuboot
  • test-sdk-pmic-samples
  • test-sdk-sidewalk
  • test-sdk-wifi
  • test-secdom-samples-public

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

[NordicBuilder]

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

[divipillai]

Isn't a changelog is required?

[Damian-Nordic]

Final nits but lgtm overall.

[peknis]

Remember also the changelog entry.

[nordicjm]

minor nit. for future note

[rakons]

Suggestion - when I see dev_info I would rather expect more information about the device itself. This should be rather ncs_info. What about release minor and major? Sha itself would not give us a lot of feedback.