Skip to content

Conversation

tymoteuszblochmobica
Copy link
Contributor

Greentea DNS tests workaround due to random NSAPI DNS bug/deadlock with Nanostack

Summary of changes

Greentea DNS tests in following sequence:

-ASYNCHRONOUS_DNS_INVALID_HOST
-ASYNCHRONOUS_DNS_TIMEOUTS
sometimes causes deadlock.

nsapi_dns_query_multiple_async block on
dns_mutex->lock();
Due to async callbacks and test specific events more detailed debugging is needed
to find root cause.

Swaping tests sequence :
-ASYNCHRONOUS_DNS_TIMEOUTS
-ASYNCHRONOUS_DNS_INVALID_HOST
or add 10 ms delay causing problem disappear.
This is temporary solution to prevent Jenkins jobs fail until root cause will be fixed.

ONME-4465 is still open to find final solution.

Impact of changes

None

Documentation

None


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[x] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Reviewers

@SeppoTakalo
@AnttiKauppila
@michalpasztamobica


…stack.

Tests sequence swaped:
ASYNCHRONOUS_DNS_TIMEOUTS
ASYNCHRONOUS_DNS_INVALID_HOST
This is temporary solution to prevent Jenkins fail until root cause will be fixed.
@ciarmcom
Copy link
Member

@tymoteuszblochmobica, thank you for your changes.
@AnttiKauppila @SeppoTakalo @michalpasztamobica @ARMmbed/mbed-os-test @ARMmbed/mbed-os-ipcore @ARMmbed/mbed-os-maintainers please review.

@mergify mergify bot added needs: CI and removed needs: review labels Jan 27, 2020
@mbed-ci
Copy link

mbed-ci commented Jan 29, 2020

Test run: SUCCESS

Summary: 5 of 5 test jobs passed
Build number : 1
Build artifacts

@kjbracey kjbracey merged commit 617ba27 into ARMmbed:master Jan 29, 2020
@mergify
Copy link

mergify bot commented Jan 29, 2020

This PR does not contain release version label after merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-version: 6.0.0-alpha-2 Second pre-release version of 6.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants