Skip to content

Conversation

Damian-Nordic
Copy link
Contributor

zperf server functionality has recently been opted-in so our zperf snippet should be updated to enable the server explicitly.

Also, pull the fix for building zperf server with UDP only.

@Damian-Nordic Damian-Nordic requested review from a team as code owners May 15, 2025 09:42
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels May 15, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 15, 2025

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@36ffd80 nrfconnect/sdk-zephyr@b1c7699 (main) nrfconnect/[email protected]

All manifest checks OK

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 15, 2025

CI Information

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

Inputs:

Sources:

hal_nordic: PR head: 243c6708278364516c6ca1dcccc00c5f721095fa
sdk-nrf: PR head: 987bee0e5e0ac20ffab34740c693853e9af7503a
zephyr: PR head: b1c76998edb2252cc10bb911ca7a9973972d1888

more details

hal_nordic:

PR head: 243c6708278364516c6ca1dcccc00c5f721095fa
merge base: a5a2277d4399939e9aa67a71ca9eb411c85a1a7d
Diff

sdk-nrf:

PR head: 987bee0e5e0ac20ffab34740c693853e9af7503a
merge base: fbf8be836402bbb32458dd3ab7c8af82672992b9
target head (main): fbf8be836402bbb32458dd3ab7c8af82672992b9
Diff

zephyr:

PR head: b1c76998edb2252cc10bb911ca7a9973972d1888
merge base: 36ffd80983a7804fb6ed142f11279dbe283996d9
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 (7)
modules
│  ├── hal
│  │  ├── nordic
│  │  │  ├── nrfx
│  │  │  │  ├── drivers
│  │  │  │  │  ├── src
│  │  │  │  │  │  ├── nrfx_nvmc.c
│  │  │  │  │  │  │ nrfx_rramc.c
snippets
│  ├── zperf
│  │  │ zperf.conf
west.yml
zephyr
│  ├── subsys
│  │  ├── net
│  │  │  ├── lib
│  │  │  │  ├── lwm2m
│  │  │  │  │  │ lwm2m_message_handling.c
│  │  │  │  ├── zperf
│  │  │  │  │  │ zperf_shell.c
│  │ west.yml

Outputs:

Toolchain

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

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 18
    • sdk-zephyr test count: 1451
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ✅ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • ✅ test-fw-nrfconnect-nrf-iot_samples
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-fw-nrfconnect-thread-main
    • ✅ test-sdk-find-my
    • ✅ test-fw-nrfconnect-nrf-iot_mosh
    • ✅ test-fw-nrfconnect-nrf-iot_positioning
    • ✅ test-low-level
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-sdk-audio
    • 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

You can find the documentation preview for this PR here.

zperf server functionality has recently been opted-in so
our zperf snippet should be updated to enable the server
explicitly.

Also, pull the fix for building zperf server with UDP only.

Signed-off-by: Damian Krolik <[email protected]>
@NordicBuilder NordicBuilder removed the DNM label May 15, 2025
@rlubos rlubos merged commit 97e8103 into nrfconnect:main May 15, 2025
14 of 15 checks passed
@Damian-Nordic Damian-Nordic deleted the zperf_server branch May 15, 2025 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants