-
Notifications
You must be signed in to change notification settings - Fork 3k
[feature-wisun] Disable sleep on STM32F4 as an workaround for stability issues. #12517
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
[feature-wisun] Disable sleep on STM32F4 as an workaround for stability issues. #12517
Conversation
This workaround is related to Mbed OS issue ARMmbed#12294
@teetak01, thank you for your changes. |
CI started |
Test run: FAILEDSummary: 1 of 9 test jobs failed Failed test jobs:
|
The results are expected, sleep for that target failed. If this is pushed to a feature branch, every test run needs to be checked if there are any other failures besides sleep. |
@0xc0170 is there simple way to disable that specific test (and add clear commit-trail) that it is being ignored? I am not very familiar with the greentea side. |
@jamesbeyond What shall we do to exclude this tests for one platform for one branch? Please advise |
Agreed with @0xc0170 to push this fix in now. Test will be fixed later. |
I don't understand the issue, |
That would be better fix 👍 |
a proper way to turn off a feature would add |
I definitively can't approve any change in targets.json file.... |
the underlying issue #12294 is currently under investigation, once that been resolved, hope this change would be reverted |
Summary of changes
This workaround is related to Mbed OS issue
#12294
Documentation
Pull request type
Test results
Reviewers
@artokin @mikter @mikaleppanen @TuomoHautamaki @yogpan01