CI test on Upmerge22073 #22148

michalek-no · 2025-05-05T11:22:24Z

No description provided.

Run migrate_posix_kconfigs.py to rename deprecated POSIX API Kconfig options. GETOPT was replaced by POSIX_C_LIB_EXT which default to n. Remove the obsolete default value change and use the replacement POSIX_C_LIB_EXT. Signed-off-by: Tomasz Moń <[email protected]> Signed-off-by: Johann Fischer <[email protected]>

Update version to upmerged sdk-zephyr and sdk-mcuboot. Signed-off-by: Johann Fischer <[email protected]>

Update Matter SDK revision. Signed-off-by: Adrian Gielniewski <[email protected]>

The combination of AF_PACKET and IPPROTO_RAW is not correct as packet sockets should only be used with IEEE 802.3 protocol numbers. As the support for this combination will likely be soon removed from Zephyr, switch to use DGRAM packet socket instead. Using ETH_P_ALL is fine in this case, as OpenThread interface only provides IPv6 packets anyway. Signed-off-by: Robert Lubos <[email protected]>

The combination of AF_PACKET and IPPROTO_RAW is not correct as packet sockets should only be used with IEEE 802.3 protocol numbers. As the support for this combination will likely be soon removed from Zephyr, switch to use DGRAM packet socket instead. As PPP L2 only supports IP/IPv6 packets, specify the L2 protocol type field accordingly so that the L2 can interpret the packet data correctly. Signed-off-by: Robert Lubos <[email protected]>

Totally unexpected, the exact same Kconfig options and C designators are introduced in the upstream Zephyr repository. Prefix the Kconfig options of the NCS implementation with NRF to resolve the conflicts. Signed-off-by: Johann Fischer <[email protected]>

Refer to commit zephyrproject-rtos/zephyr@d382fca Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>

last_erased_page_start_offset has been removed by zephyrproject-rtos/zephyr#82326. Need to change accordingly to not break the build. Signed-off-by: Maximilian Deubel <[email protected]>

This has been renamed upstream. Signed-off-by: Timothy Keys <[email protected]>

BT_ISO_CHAN_TYPE_CONNECTED has been refactored into BT_ISO_CHAN_TYPE_CENTRAL and BT_ISO_CHAN_TYPE_PERIPHERAL. Signed-off-by: Timothy Keys <[email protected]>

Signature was changed to have const bt_conn upstream. Signed-off-by: Timothy Keys <[email protected]>

This is needed after changes in upstream zephyr. Signed-off-by: Timothy Keys <[email protected]>

Adjust for new net_linkaddr definition. Signed-off-by: Maximilian Deubel <[email protected]>

Moved OpenThread-related Kconfigs from L2 layer to modules/openthread. All of those configs were not strictly related to the L2 layer, so they fits better to OpenThread module. This operation allows using OpenThread Kconfigs even if L2 layer is disabled. Enabling NET_L2_OPENTHREAD also configures those configs by selecting the OPENTHREAD kconfig, so there is no change regarding backwards compatibility. The only change is that the Kconfigs related to Thread were moved to the modules space, so their location in the menuconfig also changed. Once it is done, a choice for setting different L2 implementations seems to be redundant. Signed-off-by: Arkadiusz Balys <[email protected]> Signed-off-by: Adrian Gielniewski <[email protected]>

Due to a change in the crypto library, the expected JWT signatures have changed. Signed-off-by: Maximilian Deubel <[email protected]>

The signature of bt_conn_get_remote_info changed upstream to use const conn *, but in this usecase conn * should not be const. Signed-off-by: Timothy Keys <[email protected]>

BT_GATT_CCC_INITIALIZER was renamed upstream. It is now called BT_GATT_CCC_MANAGED_USER_DATA_INIT. Signed-off-by: Timothy Keys <[email protected]>

Upstream refactor means it is no longer needed by default. Signed-off-by: Timothy Keys <[email protected]>

CURL and IPERF3 depend on POSIX_THREADS not being selected but POSIX_API selects POSIX_BASE_DEFINITIONS and subsequently POSIX_THREADS, resulting in a dependency conflict. There is no justification provided by the responsible team as to what the exact problem was with PTHREAD_IPC (now POSIX_THREADS) and whatever it still is or is not. Signed-off-by: Johann Fischer <[email protected]>

naming adjustment Signed-off-by: Mateusz Michalek <[email protected]>

NordicBuilder · 2025-05-05T11:25:08Z

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

Name	Old Revision	New Revision	Diff
matter	nrfconnect/sdk-connectedhomeip@`0539fd4`	nrfconnect/sdk-connectedhomeip#603	nrfconnect/sdk-connectedhomeip#603/files
mcuboot	nrfconnect/sdk-mcuboot@`9d9d524` (`main`)	nrfconnect/sdk-mcuboot#425	nrfconnect/sdk-mcuboot#425/files
zephyr	nrfconnect/sdk-zephyr@`aba3d9c`	nrfconnect/sdk-zephyr#2817	nrfconnect/sdk-zephyr#2817/files

⛔ DNM label due to: 3 projects with PR revision

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

NordicBuilder · 2025-05-05T11:27:13Z

CI Information

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

Inputs:

Sources:

hal_nordic: PR head: 119ff5b5ec7a413ca2f64ae1928c79eee1e7b7b2
nrf_hw_models: PR head: d5b95fdd0260e8189e788d40d2863d1e2d4be159
sdk-nrf: PR head: 416f02fc2c7fb1ce4f2ffb630a0f702218d0053c
mcuboot: PR head: 7bdc0e1ebfd84bdbb06dd282e10deae3394442f2
mbedtls: PR head: 5f889934359deccf421554c7045a8381ef75298f
matter: PR head: 0ab678dde1fbbb748df1ddd0a26aea7f1df34e85
hostap: PR head: 8412f4b23b6267ee6035d25515a23aaf243f6ad7
zephyr: PR head: 76517990079cb0b93328e1f70932b8da17e263ea

more details

hal_nordic:

PR head: 119ff5b5ec7a413ca2f64ae1928c79eee1e7b7b2
merge base: 8cdcc33c6e5c7618ca4e77ff19a0eab557cc4bd5
Diff

nrf_hw_models:

PR head: d5b95fdd0260e8189e788d40d2863d1e2d4be159
merge base: b84bd7314a239f818e78f6927f5673247816df53
Diff

sdk-nrf:

PR head: 416f02fc2c7fb1ce4f2ffb630a0f702218d0053c
merge base: 363445b47f552b6c90152223fcce230d78f447f4
target head (main): e764da8a9cf1f6c41b8bf40f2c54181445dddb97
Diff

mcuboot:

PR head: 7bdc0e1ebfd84bdbb06dd282e10deae3394442f2
merge base: 9d9d524407df6da0815c79c5b9cf645f1ce25592
target head (main): 9d9d524407df6da0815c79c5b9cf645f1ce25592
Diff

mbedtls:

PR head: 5f889934359deccf421554c7045a8381ef75298f
merge base: 4952e1328529ee549d412b498ea71c54f30aa3b1
Diff

matter:

PR head: 0ab678dde1fbbb748df1ddd0a26aea7f1df34e85
merge base: 0539fd4f072724d8ad6355045d24f5e6510505a9
target head (master): c7a5f2f76f8f728ec3844e692005b3b5b8a45d9b
Diff

hostap:

PR head: 8412f4b23b6267ee6035d25515a23aaf243f6ad7
merge base: 219b1f75d6452aafd1aff1912cc07dd132f753fa
Diff

zephyr:

PR head: 76517990079cb0b93328e1f70932b8da17e263ea
merge base: aba3d9c72a3aa57bb53b42d6cdfa0c1e4a8b0927
target head (main): 08fd4a2d291aaa10b75675d17b0ee6ec93250d6e
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 (12945)

Error generating tree on changed files

Outputs:

Toolchain

Version: 1d2c5d0ab5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:1d2c5d0ab5_8bf7ca4353

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

◻️ Toolchain - Skipped: existing toolchain is used
❌ Build twister
- sdk-nrf test count: 31
- sdk-zephyr test count: 10799
❌ Integration tests
- ❌ test-sdk-audio
- ❌ desktop52_verification
- ❌ test-fw-nrfconnect-boot
- ❌ test-fw-nrfconnect-apps
- ❌ test_ble_nrf_config
- ❌ test-fw-nrfconnect-ble_mesh
- ❌ test-fw-nrfconnect-ble_samples
- ❌ test-fw-nrfconnect-chip
- ❌ test-fw-nrfconnect-nfc
- ❌ test-fw-nrfconnect-nrf-iot_cloud
- ❌ test-fw-nrfconnect-nrf-iot_libmodem-nrf
- ❌ 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_thingy91
- ❌ test-fw-nrfconnect-nrf_crypto
- ❌ test-fw-nrfconnect-rpc
- ❌ test-fw-nrfconnect-rs
- ❌ test-fw-nrfconnect-fem
- ❌ test-fw-nrfconnect-tfm
- ❌ test-fw-nrfconnect-thread
- ❌ test-sdk-find-my
- ❌ test-fw-nrfconnect-nrf-iot_mosh
- ❌ test-fw-nrfconnect-nrf-iot_positioning
- ❌ test-sdk-wifi
- ❌ test-low-level
- ❌ test-sdk-pmic-samples
- ❌ test-sdk-mcuboot
- ❌ test-fw-nrfconnect-ps
- ⚠️ test-fw-nrfconnect-fw-update
- ⚠️ test-sdk-dfu

Disabled integration tests

- doc-internal
- test-fw-nrfconnect-proprietary_esb
- test-secdom-samples-public

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

peknis · 2025-05-05T11:30:44Z

applications/nrf_desktop/doc/cpu_meas.rst

+This option selects the :kconfig:option:`CONFIG_NRF_CPU_LOAD` option.
+The :kconfig:option:`CONFIG_NRF_CPU_LOAD` option enables the :ref:`cpu_load` library that is used to perform the measurements.


Suggested change

This option selects the :kconfig:option:`CONFIG_NRF_CPU_LOAD` option.

The :kconfig:option:`CONFIG_NRF_CPU_LOAD` option enables the :ref:`cpu_load` library that is used to perform the measurements.

This option selects the :kconfig:option:`CONFIG_NRF_CPU_LOAD` option that enables the :ref:`cpu_load` library used for performing the measurements.

thanks but that's just a CI test. No need for reviews here.

pdunaj · 2025-05-05T12:47:34Z

samples/bluetooth/peripheral_lbs/prj_minimal.conf

@@ -107,5 +107,6 @@ CONFIG_BT_BUF_EVT_RX_COUNT=2
 CONFIG_BT_CONN_TX_MAX=3
 CONFIG_BT_L2CAP_TX_BUF_COUNT=2
 CONFIG_BT_ATT_TX_COUNT=2
+CONFIG_BT_BUF_EVT_RX_COUNT=4


where this nr comes from?

thanks but that's just a CI test. No need for reviews here.

pdunaj · 2025-05-05T12:50:01Z

subsys/bluetooth/rpc/client/bt_rpc_conn_client.c

@@ -403,7 +403,7 @@ int bt_conn_get_remote_info(struct bt_conn *conn,

 	bt_rpc_encode_bt_conn(&ctx, conn);

-	result.conn = conn;
+	result.conn = (struct bt_conn *)conn;


struct bt_conn_get_remote_info_rpc_res needs to be updated instead

pdunaj · 2025-05-05T12:52:54Z

applications/nrf_desktop/doc/cpu_meas.rst

+This option selects the :kconfig:option:`CONFIG_NRF_CPU_LOAD` option.
+The :kconfig:option:`CONFIG_NRF_CPU_LOAD` option enables the :ref:`cpu_load` library that is used to perform the measurements.


tmon-nordic and others added 21 commits May 5, 2025 10:45

manifest: update sdk-zephyr and sdk-mcuboot revision

303680d

Update version to upmerged sdk-zephyr and sdk-mcuboot. Signed-off-by: Johann Fischer <[email protected]>

manifest: Update sdk-connectedhomeip revision

a395822

Update Matter SDK revision. Signed-off-by: Adrian Gielniewski <[email protected]>

Bluetooth: Controller: Port HCI Command buffer allocation

0088952

Refer to commit zephyrproject-rtos/zephyr@d382fca Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>

subsys: dfu: dfu_target: Update for upstream changes

6531883

last_erased_page_start_offset has been removed by zephyrproject-rtos/zephyr#82326. Need to change accordingly to not break the build. Signed-off-by: Maximilian Deubel <[email protected]>

Bluetooth: Host: Rename _bt_gatt_ccc to bt_gatt_ccc_managed_user_data

7f2c423

This has been renamed upstream. Signed-off-by: Timothy Keys <[email protected]>

Samples: Bluetooth: Refactor removed BT_ISO_CHAN_TYPE_CONNECTED

019b52c

BT_ISO_CHAN_TYPE_CONNECTED has been refactored into BT_ISO_CHAN_TYPE_CENTRAL and BT_ISO_CHAN_TYPE_PERIPHERAL. Signed-off-by: Timothy Keys <[email protected]>

Bluetooth: rpc: Change signature of bt_conn_get_remote_info

e3a858f

Signature was changed to have const bt_conn upstream. Signed-off-by: Timothy Keys <[email protected]>

Samples: Bluetooth: Explicitly set CONFIG_BT_BUF_EVT_RX_COUNT

d600392

This is needed after changes in upstream zephyr. Signed-off-by: Timothy Keys <[email protected]>

tests: lib: hw_id: upmerge fixes

04aad10

Adjust for new net_linkaddr definition. Signed-off-by: Maximilian Deubel <[email protected]>

tests: app_jwt: update expected JWT sigs

1faf1e6

Due to a change in the crypto library, the expected JWT signatures have changed. Signed-off-by: Maximilian Deubel <[email protected]>

Bluetooth: rpc: cast conn to not const

ba42838

The signature of bt_conn_get_remote_info changed upstream to use const conn *, but in this usecase conn * should not be const. Signed-off-by: Timothy Keys <[email protected]>

Bluetooth: Services: Rename BT_GATT_CCC_INITIALIZER

0e10415

BT_GATT_CCC_INITIALIZER was renamed upstream. It is now called BT_GATT_CCC_MANAGED_USER_DATA_INIT. Signed-off-by: Timothy Keys <[email protected]>

kconfig: Remove large default command buffer for 5340

34b0ee3

Upstream refactor means it is no longer needed by default. Signed-off-by: Timothy Keys <[email protected]>

subsys: dfu: ctx struct fix

416f02f

naming adjustment Signed-off-by: Mateusz Michalek <[email protected]>

michalek-no requested review from a team, nordic-krch and Frodevan as code owners May 5, 2025 11:22

michalek-no requested review from muraliThokala, bama-nordic, sachinthegreen, rado17, a team and nordic-auko as code owners May 5, 2025 11:22

michalek-no requested review from koffes, rick1082, alexsven and gWacey May 5, 2025 11:22

github-actions bot added doc-required PR must not be merged without tech writer approval. manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels May 5, 2025

michalek-no added DNM and removed doc-required PR must not be merged without tech writer approval. manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels May 5, 2025

NordicBuilder added manifest manifest-mcuboot manifest-zephyr manifest-matter labels May 5, 2025

peknis requested changes May 5, 2025

View reviewed changes

pdunaj requested changes May 5, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CI test on Upmerge22073 #22148

CI test on Upmerge22073 #22148

michalek-no commented May 5, 2025 •

edited by nordic-segl

Loading

NordicBuilder commented May 5, 2025

NordicBuilder commented May 5, 2025 •

edited

Loading

peknis May 5, 2025

michalek-no May 5, 2025

pdunaj May 5, 2025

pdunaj May 5, 2025

michalek-no May 5, 2025

pdunaj May 5, 2025

pdunaj May 5, 2025

		This option selects the :kconfig:option:`CONFIG_NRF_CPU_LOAD` option.
		The :kconfig:option:`CONFIG_NRF_CPU_LOAD` option enables the :ref:`cpu_load` library that is used to perform the measurements.

CI test on Upmerge22073 #22148

Are you sure you want to change the base?

CI test on Upmerge22073 #22148

Conversation

michalek-no commented May 5, 2025 • edited by nordic-segl Loading

NordicBuilder commented May 5, 2025

NordicBuilder commented May 5, 2025 • edited Loading

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

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

peknis May 5, 2025

Choose a reason for hiding this comment

michalek-no May 5, 2025

Choose a reason for hiding this comment

pdunaj May 5, 2025

Choose a reason for hiding this comment

pdunaj May 5, 2025

Choose a reason for hiding this comment

michalek-no May 5, 2025

Choose a reason for hiding this comment

pdunaj May 5, 2025

Choose a reason for hiding this comment

pdunaj May 5, 2025

Choose a reason for hiding this comment

michalek-no commented May 5, 2025 •

edited by nordic-segl

Loading

NordicBuilder commented May 5, 2025 •

edited

Loading