Skip to content
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

[Test] Skip tests for dotnet, java and ml/openvino for ethreads=1 #801

Merged
merged 1 commit into from
Aug 19, 2020
Merged

[Test] Skip tests for dotnet, java and ml/openvino for ethreads=1 #801

merged 1 commit into from
Aug 19, 2020

Conversation

hukoyu
Copy link
Collaborator

@hukoyu hukoyu commented Aug 18, 2020

In sgx-lkl (nightly) build we run tests for different ethreads (SGXLKL_ETHREADS) values: 1, 4, 8

5 tests are failing in sgx-lkl nightly build with SGXLKL_ETHREADS=1 ( 3 java, 1 dotnet, 1 ml/openvino)

#415 (ml/openvino)
#795 (java, helloworld)
#796 (java, read_file, thread)
#797 (dotnet)

We are skipping (disabling) these 5 tests for ETHREADS=1. They will continue to run for ETHREADS > 1 ( 4 and 8)
We will enable these 5 tests after underlying bugs fixed.

@hukoyu hukoyu changed the title Skip tests for dotnet, java and ml/openvino for ethreads=1 [Test] Skip tests for dotnet, java and ml/openvino for ethreads=1 Aug 18, 2020
@hukoyu
Copy link
Collaborator Author

hukoyu commented Aug 18, 2020

The nighlty build I ran for this PR https://tfsproduks1.visualstudio.com/Ab5352d9a-5ed5-48f2-b767-7f273b55b53c/sgx-lkl/_build/results?buildId=1909&view=results

passed core tests for ethreads 1,4 and 8. These 5 tests are skipped in core tests for SGXLKL_ETHREADS=1 as expected.
(In the pipeline there are 3 ltp test failures which look random failures: time02, lchown02, fcntl16)
Created issue #802 to track enabling back these tests.

@paulcallen can you also approve and merge please, @davidchisnall approved already.

@hukoyu
Copy link
Collaborator Author

hukoyu commented Aug 18, 2020

Also sgx-lkl nightly pipeline run duration drops from 2 hour to 1 hour since ml/openvino test for 1 thread was taking 1 hour untl it timesout. Since we disabled this test for ethreads=1 now pipeline finishes1 hour earlier.

@hukoyu hukoyu requested review from bodzhang and vtikoo August 19, 2020 08:23
@vtikoo vtikoo merged commit 796b346 into lsds:oe_port Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants