Skip to content

Run CI tests in docker instead of a machine. #8948

Merged
ncabatoff merged 74 commits into
masterfrom
docker-testing
Sep 15, 2020
Merged

Run CI tests in docker instead of a machine. #8948
ncabatoff merged 74 commits into
masterfrom
docker-testing

Conversation

@ncabatoff
Copy link
Copy Markdown
Collaborator

Also stop skipping some tests that we ought to be running.

ncabatoff added 30 commits May 7, 2020 15:35
…other step so circleci sees correct exit code.
…t seem to be working very well for us, let's try with way more parallelism. The CircleCI UI says we've paid for 100-way parallelism, let's get a bit closer to that.
…'re splitting on that don't have tests, and so always get "No timing found". Try a different way to list packages to only get those with tests.
…ce-remote-docker's problem. Also try skipping a couple of tests that have been associated with most of the race test failures. Let's see if that changes anything.
…ntinue not using it (in a less disruptive way.)
…ure the VAULT_CI_GO_TEST_RACE

variable is visible in docker as well.  Only use the xlarge executor for race tests.
@ncabatoff ncabatoff merged commit 2b3aef2 into master Sep 15, 2020
@ncabatoff ncabatoff deleted the docker-testing branch September 17, 2020 13:00
@Nayana-ibm Nayana-ibm mentioned this pull request Nov 23, 2020
sapk pushed a commit to sapk-fork/vault that referenced this pull request Aug 27, 2025
) (hashicorp#8948)

* bump go-getter to 1.7.9 (hashicorp#8899)

* bump go-getter to 1.7.9

* add changelog

* go mod tidy



---------

Signed-off-by: Ryan Cragun <me@ryan.ec>
Co-authored-by: Josh Black <raskchanky@gmail.com>
Co-authored-by: Ryan Cragun <me@ryan.ec>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants