Skip to content

Conversation

bryk
Copy link
Contributor

@bryk bryk commented Feb 10, 2016

Sample report from an integration test:
https://saucelabs.com/beta/tests/7e6958fd51d44abfbb7a5e7a1efe132a/commands
See, e.g., screencast that you can play or see history.

Sample report from an integration test:
https://saucelabs.com/beta/tests/7e6958fd51d44abfbb7a5e7a1efe132a/commands
See, e.g., screencast that you can play or see history.
@bryk
Copy link
Contributor Author

bryk commented Feb 10, 2016

@floreks @cheld Take a look.

@maciaszczykm
Copy link
Member

Is it possible to display link/results in pull requests like in codecov case?

@floreks
Copy link
Member

floreks commented Feb 11, 2016

Report looks great. Very detailed. Just fix travis and LGTM

@bryk
Copy link
Contributor Author

bryk commented Feb 11, 2016

Is it possible to display link/results in pull requests like in codecov case?

When you open travis build logs you can find the links there. That's all I know.

@bryk
Copy link
Contributor Author

bryk commented Feb 11, 2016

I think I need to merge this PR manually. Saucelabs does not start from a fork.

bryk added a commit that referenced this pull request Feb 11, 2016
Run tests that require a browser on saucelabs (FF/Chrome/IE)
@bryk bryk merged commit 0949f6e into kubernetes:master Feb 11, 2016
@bryk
Copy link
Contributor Author

bryk commented Feb 11, 2016

Wish me luck... :)

@bryk
Copy link
Contributor Author

bryk commented Feb 11, 2016

It worked, you can see the output here: https://travis-ci.org/kubernetes/dashboard/builds/108497768

@floreks
Copy link
Member

floreks commented Feb 11, 2016

Doesn't it mean that all our PRs will also fail?

@bryk
Copy link
Contributor Author

bryk commented Feb 11, 2016

They shouldn't. The problem was that I was adding new addon to travis from an external pull request. Now that it is there, it should work.

@bryk bryk deleted the sauce branch November 24, 2016 09:08
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