11349
66%

Ran 20 Oct 2019 06:26AM UTC

Jobs 1

Files 109

Run time 6s

Badge

Embed ▾

Committed 18 Oct 2019 05:53PM UTC coverage: 61.393%. Remained the same

Build # 11349

Build Type

cron

travis-ci

Committed by

web-flow

Commit Message

log: panic if bothWriter write to stdout errs. (#4491)

If we can't write to stdout we prefer to panic immediately rather than
potentially lose logs we capture from redirecting stdout as a syslog backup.

A unit test is included to verify the panic behaviour. Prior to the `log` diff
in this branch the test failed because the non-nil `err` result from
`fmt.Printf` was being away:

```
=== RUN   TestStdoutFailure
=== PAUSE TestStdoutFailure
=== CONT  TestStdoutFailure
FAIL	github.com/letsencrypt/boulder/log	0.011s
FAIL
```

After the `log` package diff in this branch is applied the test passes.

I additionally tested this end-to-end by redirecting stdout to a full
filesystem volume mounted into the Boulder docker image. It provoked the
expected panic when a component tried to write to stdout and the filesystem was
full.

Run Details

11882 of 19354 relevant lines covered (61.39%)

0.69 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
7	11349.7 (RUN="coverage" CONTAINER="netaccess")	20 Oct 2019 06:26AM UTC	0	61.39	Travis Job 11349.7

Source Files on build 11349

Detailed source file information is not available for this build.

letsencrypt / boulder / 11349
66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 11349

letsencrypt / boulder / 11349 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 11349

letsencrypt / boulder / 11349
66%

README BADGES
x