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

main/openvswitch: improve test coverage #3928

Open
wants to merge 1 commit into
base: master
from

Conversation

4 participants
@itoffshore
Contributor

itoffshore commented Apr 5, 2018

  • adds diffutils as a $checkdepends

    without this tests incompatible with Busybox diff would be skipped

@itoffshore

This comment has been minimized.

Contributor

itoffshore commented Apr 5, 2018

Can the Travis timeout be made longer so the tests complete ?

@TBK

This comment has been minimized.

Contributor

TBK commented Apr 5, 2018

https://docs.travis-ci.com/user/customizing-the-build#Build-Timeouts

One option would be to replace Travis with CircleCI 2.0 which has a build timeout of 120 min.

@itoffshore

This comment has been minimized.

Contributor

itoffshore commented Apr 7, 2018

Running the test on 6 cores takes almost an hour:

===================================
## Tests completed in 52 minutes ##
===================================
@andypost

This comment has been minimized.

Contributor

andypost commented Apr 8, 2018

Maybe disable some tests? I think 20% are critical to test biild

@andypost

This comment has been minimized.

Contributor

andypost commented Apr 8, 2018

Travis now Ran for 5 min 39 sec

@itoffshore

This comment has been minimized.

Contributor

itoffshore commented Apr 8, 2018

@andypost - I've disabled the tests for now & emailed the ovs mailing list

@nmeum nmeum added the A-improve label Apr 22, 2018

@itoffshore

This comment has been minimized.

Contributor

itoffshore commented Apr 30, 2018

the tests now take 15 minutes on my local machine but I think Travis is single threaded so the tests are disabled for the moment.

main/openvswitch: improve test coverage
* adds diffutils as a $checkdepends

  without this tests incompatible with Busybox diff would be skipped
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment