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

Support Travis CI #28

Merged
merged 8 commits into from
Mar 21, 2019
Merged

Support Travis CI #28

merged 8 commits into from
Mar 21, 2019

Conversation

TommyLike
Copy link
Contributor

@TommyLike TommyLike commented Mar 20, 2019

for #27
Now Travis consists of two jobs:

  • Gofmt & Golint check
  • E2E tests in kind cluster with volcano chart.

.travis.yml Outdated Show resolved Hide resolved
@TommyLike TommyLike changed the title Update travis file [WIP] Support Travis CI Mar 20, 2019
@TommyLike TommyLike changed the title [WIP] Support Travis CI Support Travis CI Mar 20, 2019
@TommyLike
Copy link
Contributor Author

Four tests are failing and need more investigation.

• Failure [14.611 seconds]
Queue E2E Test
/home/travis/gopath/src/volcano.sh/volcano/test/e2e/queue.go:26
  Reclaim [It]
  /home/travis/gopath/src/volcano.sh/volcano/test/e2e/queue.go:27
  Expected error:
      <*errors.errorString | 0xc000156c30>: {
          s: "expected replica <1> is too small",
      }
      expected replica <1> is too small
  not to have occurred
  /home/travis/gopath/src/volcano.sh/volcano/test/e2e/queue.go:57

•! Panic [5.227 seconds]
Job E2E Test
/home/travis/gopath/src/volcano.sh/volcano/test/e2e/job_scheduling.go:26
  Statement [It]
  /home/travis/gopath/src/volcano.sh/volcano/test/e2e/job_scheduling.go:252
  Test Panicked
  runtime error: slice bounds out of range
  /home/travis/.gimme/versions/go1.11.6.linux.amd64/src/runtime/panic.go:513
  Full Stack Trace
  	/home/travis/.gimme/versions/go1.11.6.linux.amd64/src/runtime/panic.go:513 +0x1b9
  volcano.sh/volcano/test/e2e.createContainers(0x121282e, 0x5, 0x0, 0x0, 0xc0002d0d80, 0xc000000000, 0xe, 0x0, 0x0)
  	/home/travis/gopath/src/volcano.sh/volcano/test/e2e/util.go:536 +0x2fa
  volcano.sh/volcano/test/e2e.createJob(0xc0005d54f0, 0xc000773208, 0xc000000003)
  	/home/travis/gopath/src/volcano.sh/volcano/test/e2e/util.go:350 +0x242
  volcano.sh/volcano/test/e2e.glob..func3.8()
  	/home/travis/gopath/src/volcano.sh/volcano/test/e2e/job_scheduling.go:272 +0x1a4
  volcano.sh/volcano/vendor/github.com/onsi/ginkgo/internal/leafnodes.(*runner).runSync(0xc0003372c0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
  	/home/travis/gopath/src/volcano.sh/volcano/test/e2e/e2e_test.go:28 +0xa2
  testing.tRunner(0xc00013cc00, 0x129af20)
  	/home/travis/.gimme/versions/go1.11.6.linux.amd64/src/testing/testing.go:827 +0xbf
  created by testing.(*T).Run
  	/home/travis/.gimme/versions/go1.11.6.linux.amd64/src/testing/testing.go:878 +0x35c

• Failure [84.015 seconds]
Job E2E Test
/home/travis/gopath/src/volcano.sh/volcano/test/e2e/job_scheduling.go:26
  Preemption [It]
  /home/travis/gopath/src/volcano.sh/volcano/test/e2e/job_scheduling.go:149
  Expected error:
      <*errors.errorString | 0xc0002ab5a0>: {
          s: "timed out waiting for the condition",
      }
      timed out waiting for the condition
  not to have occurred
  /home/travis/gopath/src/volcano.sh/volcano/test/e2e/job_scheduling.go:178
------------------------------
• Failure [77.625 seconds]
Job E2E Test
/home/travis/gopath/src/volcano.sh/volcano/test/e2e/job_scheduling.go:26
  Multiple Preemption [It]
  /home/travis/gopath/src/volcano.sh/volcano/test/e2e/job_scheduling.go:181
  Expected error:
      <*errors.errorString | 0xc0002ab5a0>: {
          s: "timed out waiting for the condition",
      }
      timed out waiting for the condition
  not to have occurred
  /home/travis/gopath/src/volcano.sh/volcano/test/e2e/job_scheduling.go:216
------------------------------

@TommyLike
Copy link
Contributor Author

Same cases are failing for the second build........

@k82cn
Copy link
Member

k82cn commented Mar 20, 2019

If ci process is ok, we can get it merged firstly. For the error of e2e, we can open another PR to trace it.

@TommyLike
Copy link
Contributor Author

Try it again, if these tests are still failing, I can open a new issue.

@TommyLike TommyLike mentioned this pull request Mar 21, 2019
@k82cn k82cn merged commit 154b31b into master Mar 21, 2019
@k82cn k82cn deleted the bug/fix_ci_issue branch March 21, 2019 03:57
kevin-wangzefeng pushed a commit to kevin-wangzefeng/volcano that referenced this pull request Apr 30, 2019
kevin-wangzefeng pushed a commit to kevin-wangzefeng/volcano that referenced this pull request Jun 28, 2019
yolgun added a commit to yolgun/volcano that referenced this pull request Sep 14, 2022
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