Skip to content
This repository has been archived by the owner on Dec 1, 2018. It is now read-only.

timeout error comes up more and more frequently during project CI #1349

Closed
huangyuqi opened this issue Oct 26, 2016 · 4 comments · Fixed by #1353
Closed

timeout error comes up more and more frequently during project CI #1349

huangyuqi opened this issue Oct 26, 2016 · 4 comments · Fixed by #1353

Comments

@huangyuqi
Copy link

Now, the PR checks unsuccessfully more frequently in continuous-integration/travis-ci/pr, with the error log:

Your test run exceeded 120 minutes. 
One possible solution is to split up your test run.

The reason is obvious,

  • the project of Heapster becomes bigger day by day. we need to run CI check go test parallelly.
  • And in current version, the CI script run go test in all file path in the project, including vendor; we need to find out the file path which includes *_test.go, then execute go test.
HEAPSTER_ROOT=./
heapster::test-unit::find_dirs() {
  (
    cd ${HEAPSTER_ROOT}
    find . -not \( \
        \( \
          -path './_artifacts/*' \
          -o -path './docs/*' \
          -o -path './Godeps/*' \
          -o -path './deploy/*' \
          -o -path './version/*' \
          -o -path './vendor/*'\
        \) -prune \
      \) -name '*_test.go' -print0 | xargs -0n1 dirname | sed 's|^\./||' | sort -u
  )
}
set -- $(heapster::test-unit::find_dirs)

@piosz

@huangyuqi
Copy link
Author

I will bring a fix PR with second way.

@piosz
Copy link
Contributor

piosz commented Oct 26, 2016

As a first step we can just get rid off test-unit-cov

@AlmogBaku
Copy link
Contributor

@piosz @huangyuqi this can be solved by using glide #1314 (or specifically see my review comment in #1335 )

@huangyuqi
Copy link
Author

@piosz I agree. there are duplicate contents between test-unit and test-unit-cov.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants