13319
66%
master: 66%

Ran 26 Jan 2021 12:21AM UTC

Jobs 2

Files 110

Run time 1min

Badge

Embed ▾

Committed 25 Jan 2021 11:34PM UTC coverage: 65.256% (-0.2%) from 65.407%

Build # 13319

Build Type

push

travis-pro

Committed by

web-flow

Commit Message

Enhancements to SQL driver tuning via JSON config (#5235)

Historically the only database/sql driver setting exposed via JSON
config was maxDBConns. This change adds support for maxIdleConns,
connMaxLifetime, connMaxIdleTime, and renames maxDBConns to
maxOpenConns. The addition of these settings will give our SRE team a
convenient method for tuning the reuse/closure of database connections.

A new struct, DBSettings, has been added to SA. The struct, and each of
it's fields has been commented.

All new fields have been plumbed through to the relevant Boulder
components and exported as Prometheus metrics. Tests have been
added/modified to ensure that the fields are being set. There should be
no loss in coverage

Deployability concerns for the migration from maxDBConns to maxOpenConns
have been addressed with the temporary addition of the helper method
cmd.DBConfig.GetMaxOpenConns(). This method can be removed once
test/config is defaulted to using maxOpenConns. Relevant sections of the
code have TODOs added that link back to an newly opened issue.

Fixes #5199

Run Details

13311 of 20398 relevant lines covered (65.26%)

1.46 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
13	13319.13 (TESTFLAGS="--coverage" CONTAINER="netaccess")	26 Jan 2021 12:21AM UTC	0	65.26	Travis Job 13319.13
14	13319.14 (TESTFLAGS="--coverage" CONTAINER="netaccess")	26 Jan 2021 12:22AM UTC	0	65.26	Travis Job 13319.14

Source Files on build 13319

Detailed source file information is not available for this build.

letsencrypt / boulder / 13319
66%
master: 66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 13319

letsencrypt / boulder / 13319 66% master: 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 13319

letsencrypt / boulder / 13319
66%
master: 66%

README BADGES
x