letsencrypt / boulder

63%

master: 66%

LAST BUILD ON BRANCH test-cpu-multiva-test-flake-fix-7

branch: test-cpu-multiva-test-flake-fix-7

CHANGE BRANCH

x

Reset

• test-cpu-multiva-test-flake-fix-7
• HEAD
• certificate_not_null_travis
• main
• master
• release
• staging
• test-authz2-purger
• test-broken-branch
• test-bump-version
• test-cleanup
• test-clearer
• test-codeowners
• test-cpu-fix-ra-va-flake-1
• test-cpu-fix-ra-va-flake-2
• test-cpu-fix-ra-va-flake-3
• test-cpu-fix-ra-va-flake-4
• test-cpu-fix-ra-va-flake-5
• test-cpu-fix-ra-va-flake-6
• test-cpu-fix-ra-va-flake-7
• test-cpu-fix-ra-va-flake-8
• test-cpu-flake-chase-1
• test-cpu-flake-chase-10
• test-cpu-flake-chase-2
• test-cpu-flake-chase-3
• test-cpu-flake-chase-4
• test-cpu-flake-chase-5
• test-cpu-flake-chase-6
• test-cpu-flake-chase-7
• test-cpu-flake-chase-8
• test-cpu-flake-chase-9
• test-cpu-go-1.11
• test-cpu-godeps-whyyyyy
• test-cpu-idn-psl
• test-cpu-missing-scts-reward-if-found
• test-cpu-multiva-test-flake-1
• test-cpu-multiva-test-flake-10
• test-cpu-multiva-test-flake-2
• test-cpu-multiva-test-flake-3
• test-cpu-multiva-test-flake-4
• test-cpu-multiva-test-flake-5
• test-cpu-multiva-test-flake-6
• test-cpu-multiva-test-flake-7
• test-cpu-multiva-test-flake-8
• test-cpu-multiva-test-flake-9
• test-cpu-multiva-test-flake-fix-1
• test-cpu-multiva-test-flake-fix-10
• test-cpu-multiva-test-flake-fix-2
• test-cpu-multiva-test-flake-fix-3
• test-cpu-multiva-test-flake-fix-4
• test-cpu-multiva-test-flake-fix-5
• test-cpu-multiva-test-flake-fix-6
• test-cpu-multiva-test-flake-fix-8
• test-cpu-multiva-test-flake-fix-9
• test-cpu-rm-old-v2-keyrollover
• test-disableauthz2orders
• test-docker-maria101
• test-embedscts
• test-ewf
• test-expiration-mailer-fix
• test-fancy-errors
• test-fix-go-generate
• test-godep-fixup
• test-godep-restore-delete
• test-godep-simpler
• test-gorp-2
• test-gorp-22
• test-goveralls
• test-https-confuse
• test-improvements
• test-instant-verify
• test-integration-quiet
• test-less-ports
• test-multi-va
• test-one-six
• test-parallel
• test-race-speed
• test-rm-amqp
• test-runner-improvements
• test-sa-parallelism
• test-simple-testing
• test-tag
• test-travis-docker-pull
• test-travis-fold
• test-update-certbot-tests
• test-update-grpc-stuff
• test-v2-both-auto
• test-va-tests
• test-vendorvendorvendor2

Build # 10464

Build Type

push

travis-ci

Committed by cpu

Commit Message

tests: rework multi-va integration tests.

Rather than having a middle server bounce/redirect requests based on how
many had arrived it would be more reliable to follow the way we did this
in unit tests and decide based on the request UA. By setting different
UAs for the remote VAs its possible to more precisely define how many
requests from each VA should be redirected to the real key
authorization.

I believe using the count was flaky because occasionally with
a configuration that allowed 2 requests to test that 1 of the 2 remote
VAs failing was an overall success would fail if the 2 remote VAs made
their requests before the primary VA. The primary VA always trumps the
remotes and so if the remotes consume the redirect quota the primary VA
will get the fail result and the overall validation will fail.

Run Details

11003 of 17460 relevant lines covered (63.02%)

0.7 hits per line