Skip to content

Commit

Permalink
Restore default GOGC for CI fuzz tests (#562)
Browse files Browse the repository at this point in the history
Remove the GOGC override for CI fuzz tests, because:
 * without it fuzz tests run faster,
 * we are close to the runner's memory limit, and we get false positive
   failures caused bo GitHub Actions SIGTERM if we go over the limit.
  • Loading branch information
masih authored Aug 11, 2024
1 parent fee0328 commit eef348e
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ test/cover: GOTEST_ARGS=-coverprofile=coverage.txt -covermode=atomic -coverpkg=.
.PHONY: test/cover

fuzz: FUZZTIME ?= 10s # The duration to run fuzz testing, default to 10s if unset.
fuzz: GOGC ?= 400 # Reduce GC frequency during testing, default to 400 if unset.
fuzz: # List all fuzz tests across the repo, and run them one at a time with the configured fuzztime.
@set -e; \
go list ./... | while read -r package; do \
Expand Down

0 comments on commit eef348e

Please sign in to comment.