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

Remove unused exception #428

Merged
merged 1 commit into from
May 23, 2018

Conversation

pavolloffay
Copy link
Member

Resolves #426

Signed-off-by: Pavol Loffay ploffay@redhat.com

Signed-off-by: Pavol Loffay <ploffay@redhat.com>
@pavolloffay
Copy link
Member Author

xdock is annoying, 4x restarted and it's still failing. Locally it failed twice on

2018/05/23 14:07:13 HTTP: Service go:8080 timed out. Last error: expecting 200, got 403 Forbidden
2018/05/23 14:07:13 Error: One or more services timed out after 1m0s
make: *** [jaeger-crossdock/rules.mk:12: crossdock] Error 1

@codecov
Copy link

codecov bot commented May 23, 2018

Codecov Report

Merging #428 into master will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #428      +/-   ##
============================================
+ Coverage     87.07%   87.15%   +0.08%     
  Complexity      510      510              
============================================
  Files            66       65       -1     
  Lines          1980     1978       -2     
  Branches        258      258              
============================================
  Hits           1724     1724              
+ Misses          166      164       -2     
  Partials         90       90

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ac8bd93...333aa41. Read the comment docs.

@pavolloffay pavolloffay merged commit 41399fa into jaegertracing:master May 23, 2018
@ghost ghost removed the review label May 23, 2018
@yurishkuro
Copy link
Member

xdock is annoying, 4x restarted and it's still failing. Locally it failed twice on

I suspect it's all due to Cassandra.

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 this pull request may close these issues.

3 participants