Fixes for STM32H5 demo. Fixes MQTT broker and improrves testing/docs. Expands CI test#73
Fixes for STM32H5 demo. Fixes MQTT broker and improrves testing/docs. Expands CI test#73dgarske wants to merge 3 commits intowolfSSL:masterfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the STM32H563 demo to better support the wolfMQTT broker on bare-metal (wolfIP TLS + UART logging), expands documentation for broker usage, and adds a fuller CI job that boots the firmware in m33mu and integration-tests multiple services.
Changes:
- Configure wolfMQTT broker to use wolfIP-provided TLS context and route broker logs to UART via a custom
PRINTF. - Add
wolfmqtt_log()UART printf-style logger for WBLOG/broker logging. - Expand STM32H563 CI to build and integration-test echo/HTTPS/TLS-echo/MQTT/SSH in emulation.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| src/port/stm32h563/user_settings.h | Enables broker custom net/TLS integration and custom UART logging macros; raises broker log level. |
| src/port/stm32h563/main.c | Adds wolfmqtt_log() implementation using vsnprintf + UART output. |
| src/port/stm32h563/README.md | Updates build/flash instructions and adds detailed MQTT broker usage/testing docs. |
| .github/workflows/stm32h563-m33mu.yml | Adds a “full” CI job that emulates the firmware and performs network service checks incl. MQTT. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 21 out of 21 changed files in this pull request and generated 10 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| # --------------------------------------------------------------------------- | ||
| # 5) MQTT Broker | ||
| # --------------------------------------------------------------------------- | ||
| banner "5. MQTT Broker (Port 8883) - TLS 1.3" |
There was a problem hiding this comment.
This needs to use wolfMQTT
| } | ||
| trap cleanup EXIT | ||
|
|
||
| sudo modprobe tun |
| runs-on: ubuntu-latest | ||
| timeout-minutes: 30 | ||
| container: | ||
| image: ghcr.io/danielinux/m33mu-ci:1.5 |
| } | ||
| trap cleanup EXIT | ||
|
|
||
| sudo modprobe tun |
There was a problem hiding this comment.
no moprobe in container: this will break the test
No description provided.