mirror of
https://github.com/containerd/containerd.git
synced 2026-08-10 17:14:49 +00:00
ci: bound Go fuzzing by execution count
Go can report context deadline exceeded when a duration-based fuzz limit
expires (https://go.dev/issue/75804).
Use a 50,000-execution limit based on the roughly 47,000 executions
FuzzImageStore completed in 30 seconds in CI. This keeps work stable
across runners and avoids the duration issue.
Assisted-by: Codex
(cherry picked from commit c1b9b78f47)
Signed-off-by: Chris Henzie <chrishenzie@gmail.com>
This commit is contained in:
@@ -14,16 +14,18 @@
|
||||
# See the License for the specific language governing permissions and
|
||||
# limitations under the License.
|
||||
|
||||
# Running Go 1.18's fuzzing for 30 seconds each. While this would be too
|
||||
# Running Go's fuzzing for 50,000 executions. While this would be too
|
||||
# short to actually find issues, we want to make sure that these fuzzing
|
||||
# tests are not fundamentally broken.
|
||||
# tests are not fundamentally broken. Using an execution limit rather than a
|
||||
# duration also avoids spurious failures when a duration expires
|
||||
# (https://go.dev/issue/75804).
|
||||
|
||||
set -euo pipefail
|
||||
|
||||
fuzztime=30s
|
||||
fuzzlimit=50000x
|
||||
pkgs=$(git grep 'func Fuzz.*testing\.F' | grep -o '.*\/' | sort | uniq)
|
||||
|
||||
for pkg in $pkgs
|
||||
do
|
||||
go test -fuzz=. ./$pkg -fuzztime=$fuzztime
|
||||
go test -fuzz=. ./$pkg -fuzztime=$fuzzlimit
|
||||
done
|
||||
|
||||
Reference in New Issue
Block a user