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

WIP: Enable tests on Windows CI #948

Closed
wants to merge 32 commits into from
Closed

WIP: Enable tests on Windows CI #948

wants to merge 32 commits into from

Conversation

seisman
Copy link
Member

@seisman seisman commented Jun 16, 2019

No description provided.

@seisman
Copy link
Member Author

seisman commented Jun 17, 2019

@joa-quim How long does it take to run all the tests on Windows? It seems it will take several hours here.

@seisman
Copy link
Member Author

seisman commented Jun 17, 2019

Attached is the test log file on Windows CI: 33.txt.

Bad news: it takes 5 hours to run 200 tests.

Good news: most tests pass, while some tests fail due to the missing bc command and something else.

@PaulWessel
Copy link
Member

Wow, I guess this is serious time-slicing?

@joa-quim
Copy link
Member

It takes 273 sec with -j8 and 949 sec with ninja check. But it only runs 207 tests. What about the others?

And worst. The testing system is broken. It doesn't write the test directory with the failures. Now we only know it failed but cannot go see why.

@joa-quim
Copy link
Member

Hmm, deleted the build dir and restarted and now things are back to normal. With -j8 it takes 980 sec

@seisman seisman force-pushed the ci-windows-tests branch 3 times, most recently from 9cde987 to aae11de Compare June 23, 2019 21:06
@seisman seisman changed the base branch from master to 6.0 June 25, 2019 02:22
@seisman seisman mentioned this pull request Jul 2, 2019
28 tasks
@seisman seisman force-pushed the ci-windows-tests branch 3 times, most recently from 3b79a63 to 67ec647 Compare July 5, 2019 16:43
@seisman seisman mentioned this pull request Jul 9, 2019
12 tasks
@seisman seisman closed this Jul 19, 2019
@seisman seisman deleted the ci-windows-tests branch July 22, 2019 19:58
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.

None yet

3 participants