-
Notifications
You must be signed in to change notification settings - Fork 273
Adds publish connect #300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
euripedesrocha
wants to merge
66
commits into
espressif:master
Choose a base branch
from
euripedesrocha:adds_publish_connect
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Adds publish connect #300
euripedesrocha
wants to merge
66
commits into
espressif:master
from
euripedesrocha:adds_publish_connect
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Apply the pre-commit hook whitespace fixes to all files in the repo. (Line endings, blank lines at end of file, trailing whitespace)
Also removes unused references to http-request app size in idf_performance.h
This is needed for testing different connection modes in the mqtt library, specifically: * test case CONFIG_EXAMPLE_CONNECT_CASE_NO_CERT: default server - expect to connect normally
Enables building C3 examples in CI. Fixes related warnings/errors and disables examples that cannot run.
Update ROM API. Port changes from bringup branch.
Adds qemu configuration Generalize the get_dut() to enable choosing DUT class per configuration
- Mqtt client will return error on wrong parameters. This fix the test code to setup the client for each test case.
* Fix thread safe issue in paho-mqtt library * Move the weekend test to ethernet runner
Also increase the default sizes and repeat-counts to send more data and exercise the library more intensly
Publish test cases are prone to fail depending on Ci load.
- Add random client id on each iteration - Make timeout configuration dependent on more scenario parameters
- Makes each case to subscribe/publish to unique topics - Makes test to wait for the successful subscription on the runner
4kB stack size for mosquitto runner task wasn't enough and with recent updates of sock_utils it caused stack overflows
Certificates expired, so need to update for testing to pass.
- Previous fix ommited one of the client certificates by mistaque. - This regenerates all certificates to clean that up.
- Example CA certificates that are used for self-signed client certificates need to include the Key Usage parameter. - Python3.13 changed the default context of the SSL context that is generated using ssl.create_default_context() by enabling the VERIFY_X509_STRICT flag by default
This test case fails mostly for network related issues, with retrys we might avoid the failure of the whole job.
0fbdabd
to
6b23f40
Compare
6b23f40
to
dbc9b2c
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Related
Testing
Checklist
Before submitting a Pull Request, please ensure the following: