-
Notifications
You must be signed in to change notification settings - Fork 12
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
just `docker exec`ing a bunch of times adds a decent amount of overhead during test setup. i was able to reduce setup time from 14s to 8s on my system just by combining all the `vault_exec`s that happen during setup as much as possible. ``` before: make integration-tests 318.37s user 105.12s system 42% cpu 16:32.31 total after: make integration-tests 108.12s user 37.11s system 25% cpu 9:19.14 total ```
- Loading branch information
Matt Kulka
committed
Feb 15, 2021
1 parent
1b22562
commit ece3420
Showing
1 changed file
with
31 additions
and
32 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters