Skip to content

Conversation

@rlubos
Copy link
Contributor

@rlubos rlubos commented Mar 27, 2025

Pull MQTT 5.0 support from upstream and align NCS codebase.

@rlubos rlubos requested review from a team as code owners March 27, 2025 09:07
@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 Mar 27, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 27, 2025

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@9442059 nrfconnect/sdk-zephyr@1a2d42b (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 Mar 27, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: dd85ee122467874abc4e8d13c894ab23b0ecf412
zephyr: PR head: 1a2d42b0aa3b4bc2148d4c1ae2ccc977cf6282c5

more details

sdk-nrf:

PR head: dd85ee122467874abc4e8d13c894ab23b0ecf412
merge base: 8beb4738db1eca7e95e1f3143762811a5859a321
target head (main): 8beb4738db1eca7e95e1f3143762811a5859a321
Diff

zephyr:

PR head: 1a2d42b0aa3b4bc2148d4c1ae2ccc977cf6282c5
merge base: 9442059bfd79de2f8e8ad858e7ad064694493bc1
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 (27)
applications
│  ├── serial_lte_modem
│  │  ├── src
│  │  │  ├── mqtt_c
│  │  │  │  │ slm_at_mqtt.c
subsys
│  ├── net
│  │  ├── lib
│  │  │  ├── mqtt_helper
│  │  │  │  │ mqtt_helper.c
│  │  │  ├── nrf_cloud
│  │  │  │  ├── src
│  │  │  │  │  │ nrf_cloud_transport.c
tests
│  ├── subsys
│  │  ├── net
│  │  │  ├── lib
│  │  │  │  ├── mqtt_helper
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ mqtt_helper_test.c
west.yml
zephyr
│  ├── doc
│  │  ├── connectivity
│  │  │  ├── networking
│  │  │  │  ├── api
│  │  │  │  │  │ mqtt.rst
│  ├── include
│  │  ├── zephyr
│  │  │  ├── net
│  │  │  │  │ mqtt.h
│  ├── samples
│  │  ├── net
│  │  │  ├── cloud
│  │  │  │  ├── aws_iot_mqtt
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ main.c
│  │  │  ├── mqtt_publisher
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  ├── secure_mqtt_sensor_actuator
│  │  │  │  ├── src
│  │  │  │  │  │ mqtt_client.c
│  ├── subsys
│  │  ├── net
│  │  │  ├── lib
│  │  │  │  ├── mqtt
│  │  │  │  │  ├── Kconfig
│  │  │  │  │  ├── mqtt.c
│  │  │  │  │  ├── mqtt_decoder.c
│  │  │  │  │  ├── mqtt_encoder.c
│  │  │  │  │  ├── mqtt_internal.h
│  │  │  │  │  │ mqtt_rx.c
│  │  ├── shell
│  │  │  ├── backends
│  │  │  │  │ shell_mqtt.c
│  ├── tests
│  │  ├── net
│  │  │  ├── lib
│  │  │  │  ├── mqtt
│  │  │  │  │  ├── v3_1_1
│  │  │  │  │  │  ├── mqtt_client
│  │  │  │  │  │  │  ├── prj.conf
│  │  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │  │ main.c
│  │  │  │  │  │  │  │ testcase.yaml
│  │  │  │  │  │  ├── mqtt_packet
│  │  │  │  │  │  │  ├── prj.conf
│  │  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │  │ mqtt_packet.c
│  │  │  │  │  │  │  │ testcase.yaml
│  │  │  │  │  ├── v5_0
│  │  │  │  │  │  ├── mqtt_packet
│  │  │  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  │  │  ├── prj.conf
│  │  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │  │ mqtt_packet.c
│  │  │  │  │  │  │  │ testcase.yaml

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: 669
    • sdk-zephyr test count: 1801
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ✅ 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-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-fw-nrfconnect-thread
    • ✅ test-sdk-find-my
    • ✅ test-fw-nrfconnect-nrf-iot_mosh
    • ✅ test-fw-nrfconnect-nrf-iot_positioning
    • ✅ test-low-level
    • ✅ test-sdk-dfu
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • 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_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

@github-actions
Copy link

You can find the documentation preview for this PR here.

@rlubos rlubos force-pushed the net/mqtt-5 branch 2 times, most recently from aed54fd to fb49a4b Compare March 27, 2025 11:24
@MarkusLassila MarkusLassila self-requested a review March 28, 2025 08:56
Pull MQTT 5.0 support from upstream and align NCS codebase.

Signed-off-by: Robert Lubos <[email protected]>
@NordicBuilder NordicBuilder removed the DNM label Mar 28, 2025
@rlubos rlubos merged commit adc5732 into nrfconnect:main Mar 28, 2025
14 of 15 checks passed
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.

9 participants