10885
66%

Ran 08 Jul 2019 05:59PM UTC

Jobs 1

Files 95

Run time 2min

Badge

Embed ▾

Committed 08 Jul 2019 05:47PM UTC coverage: 61.33% (+0.03%) from 61.305%

Build # 10885

Build Type

push

travis-ci

Committed by

cpu

Commit Message

grpc: Add better error message for timeouts. (#4324)

Right now we sometimes get errors like:

rpc error: code = Unknown desc = rpc error:
  code = DeadlineExceeded desc = context deadline exceeded

For instance, when an SA call times out, and the RA returns that
timed-out error to the WFE. These are kind of confusing because they
have two layers of nested gRPC error, and they don't provide additional
information about which SA call timed out.

This change replaces DeadlineExceeded errors with our own error type
that includes the service and the method that were called, as well as
the amount of time it took (which helps understand if timeouts are
happening because earlier calls ate up time towards the deadline).

When the RA->SA NewOrder call times out, and the RA returns that error to WFE:

"InternalErrors":["rpc error: code = Unknown desc =
  sa.StorageAuthority.NewOrder timed out after 14954 ms"]

When the WFE->RA NewOrder call times out:

"InternalErrors":["ra.RegistrationAuthority.NewOrder timed out after 15000 ms"]

Note that this change only handles timeouts at one level deep, which I
think is sufficient for our needs.

Run Details

11289 of 18407 relevant lines covered (61.33%)

0.68 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
7	10885.7 (RUN="coverage" CONTAINER="netaccess")	08 Jul 2019 05:59PM UTC	0	61.33	Travis Job 10885.7

Source Files on build 10885

Detailed source file information is not available for this build.

letsencrypt / boulder / 10885
66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 10885

letsencrypt / boulder / 10885 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 10885

letsencrypt / boulder / 10885
66%

README BADGES
x