7
66%
master: 66%

Ran 31 Jul 2019 08:01PM UTC

Files 103

Run time 33s

Badge

Embed ▾

Committed 31 Jul 2019 07:41PM UTC coverage: 61.633% (+0.02%) from 61.61%

Job # RUN="coverage" CONTAINER="netaccess"

Build Type

push

travis-ci

Committed by

rolandshoemaker

Commit Message

sa: wrap transactions in a function for commits/rollbacks (#4373)

In the current SA code, we need to remember to call Rollback on any error.
If we don't, we'll leave dangling transactions, which are hard to spot but eventually
clog up the database and cause availability problems.

This change attempts to deal with rollbacks more rigorously, by implementing a
withTransaction function that takes a closure as input. withTransaction opens
a transaction, applies a context.Context to it, and then runs the closure. If the
closure returns an error, withTransaction rolls back and return the error; otherwise
it commits and returns nil.

One of the quirks of this implementation is that it relies on the closure modifying
variables from its parent scope in order to return values. An alternate implementation
could define the return value of the closure as interface{}, nil, and have the calling
function do a type assertion. I'm seeking feedback on that; not sure yet which is cleaner.

This is a subset of the functions that need this treatment. I've got more coming, but
some of the changes break tests so I'm checking into why.

Updates #4337

Run Details

11531 of 18709 relevant lines covered (61.63%)

0.69 hits per line

letsencrypt / boulder / 11016 / 7
66%
master: 66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11016.7 (RUN="coverage" CONTAINER="netaccess")

letsencrypt / boulder / 11016 / 7 66% master: 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11016.7 (RUN="coverage" CONTAINER="netaccess")

letsencrypt / boulder / 11016 / 7
66%
master: 66%

README BADGES
x