-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Move SYNCHRONOUS_DNS_CACHE test to extended set #7945
Conversation
@mikaleppanen please review |
@@ -178,7 +178,9 @@ Case cases[] = { | |||
#endif | |||
Case("SYNCHRONOUS_DNS", SYNCHRONOUS_DNS), | |||
Case("SYNCHRONOUS_DNS_MULTIPLE", SYNCHRONOUS_DNS_MULTIPLE), | |||
#ifdef MBED_EXTENDED_TESTS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where is MBED_EXTENDED_TESTS
documented? I noticed few tests have this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think its documented anywhere currently. Temporarily moving the test to extended tests until we are able to solve the issues with it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is documented in the test plan
https://github.com/ARMmbed/mbed-os/tree/master/TESTS/netsocket#building-test-binaries
The commit does not specify why is this being moved under |
Changed description |
Also please update the commit message (more important to have it there than in the description here). IT does not reference/state what issues are there |
…ues fixed with it
a72239b
to
bc2166e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@0xc0170 please merge this soon.. our new test job is failing because of this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As long as there's a task for this to be reenabled again (make the test reliable in regular test group).
/morph build |
Build : FAILUREBuild number : 3002 |
Theres a ticket ready for getting it back from extended test set. |
@0xc0170 Could the build for this be triggered again, previous failure seems to be some jenkins-related issue? |
@0xc0170 |
@kjbracey-arm Please start tests for this. |
Does this need to get in for 5.10-rc2? CI machines having various problems, and need to prioritise rc2 PRs. |
Not specifically.. anytime when there is a slot |
/morph build |
Build : SUCCESSBuild number : 3056 Triggering tests/morph test |
Test : SUCCESSBuild number : 2834 |
Exporter Build : FAILUREBuild number : 2671 |
Poop. ARM network license issues. /morph build |
sigh That was meant to be a rerun of the export job, not the entire thing. My bad. |
Build : SUCCESSBuild number : 3062 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 2676 |
/morph test |
1 similar comment
/morph test |
Test : FAILUREBuild number : 2845 |
Something is funky with the network tests after our network switch had to be restarted. Investigating and will restart. |
/morph test |
Test : FAILUREBuild number : 2853 |
Please restart the test and @ARMmbed/mbed-os-hal please fix the unstable test. |
The error was |
@maciejbocianski Correct. A fix is available for the tool, all that's left is for @ARMmbed/mbed-os-test to deploy the fix. No action from HAL needed. /morph test |
/morph test |
Test : SUCCESSBuild number : 2857 |
Description
Temporarily move SYNCHRONOUS_DNS_CACHE to extended test set until we are able to solve the issues with it.
Pull request type