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

Don't run glide install in travis builds. #236

Merged
merged 1 commit into from
Dec 25, 2017
Merged

Conversation

jlewi
Copy link
Contributor

@jlewi jlewi commented Dec 21, 2017

  • We now check in the vendor directory so there shouldn't be any reason to run
    glide install in our travis builds.

This change is Reviewable

* We now check in the vendor directory so there shouldn't be any reason to run
  glide install in our travis builds.
@jlewi
Copy link
Contributor Author

jlewi commented Dec 21, 2017

@jimexist I just realized that in #232 you removed

- go test -v ./pkg/...

Was that intentional? Does goveralls actually run the tests?

@jlewi
Copy link
Contributor Author

jlewi commented Dec 21, 2017

Looks like it does

$ $GOPATH/bin/goveralls -service=travis-ci -v -package ./pkg/...

?   	github.com/tensorflow/k8s/pkg/controller	[no test files]

=== RUN   TestAddAccelertor

--- PASS: TestAddAccelertor (0.00s)

=== RUN   TestSetDefaults

=== RUN   TestSetDefaults/0

=== RUN   TestSetDefaults/1

=== RUN   TestSetDefaults/2

--- PASS: TestSetDefaults (0.00s)

    --- PASS: TestSetDefaults/0 (0.00s)

    --- PASS: TestSetDefaults/1 (0.00s)

    --- PASS: TestSetDefaults/2 (0.00s)

=== RUN   TestValidate

--- PASS: TestValidate (0.00s)

PASS

coverage: 7.4% of statements in github.com/tensorflow/k8s/pkg/controller, github.com/tensorflow/k8s/pkg/spec, github.com/tensorflow/k8s/pkg/trainer, github.com/tensorflow/k8s/pkg/util, github.com/tensorflow/k8s/pkg/util/k8sutil, github.com/tensorflow/k8s/pkg/util/k8sutil/fake, github.com/tensorflow/k8s/pkg/util/retryutil

ok  	github.com/tensorflow/k8s/pkg/spec	0.106s

@coveralls
Copy link

coveralls commented Dec 21, 2017

Coverage Status

Coverage decreased (-0.1%) to 37.782% when pulling 7a649e8 on jlewi:travis_glide into cb1e053 on tensorflow:master.

@jlewi
Copy link
Contributor Author

jlewi commented Dec 21, 2017

@jimexist can you review this please?

@jimexist
Copy link
Member

LGTM

And yes it does, I followed some official docs so I guessed it should work. The report showed that tests are still run everytime.

@jlewi jlewi merged commit 60b8f7a into kubeflow:master Dec 25, 2017
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.

4 participants