11016
66%

Ran 31 Jul 2019 08:00PM UTC

Jobs 1

Files 103

Run time 46s

Badge

Embed ▾

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

Build # 11016

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

Jobs

ID	Job ID	Ran	Files	Coverage
7	11016.7 (RUN="coverage" CONTAINER="netaccess")	31 Jul 2019 08:00PM UTC	0	61.63	Travis Job 11016.7

Source Files on build 11016

Detailed source file information is not available for this build.

letsencrypt / boulder / 11016
66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 11016

letsencrypt / boulder / 11016 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 11016

letsencrypt / boulder / 11016
66%

README BADGES
x