Differential D9801

Remove the 1s timeouts in p2p_inv_download
ClosedPublic
Actions

Authored by Fabien on Jul 16 2021, 12:19.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCa1d89c232954: Remove the 1s timeouts in p2p_inv_download

Summary

This test fails occasionally on CI but I couldn't reproduce locally, so these timeouts might be too short when running the extended test suite. It's not a problem to wait for a minute (the default) when the test fails, and it doesn't change the test duration on success.

Test Plan

./test/functional/test_runner.py p2p_inv_download

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Jul 16 2021, 12:19

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 16 2021, 12:19

Fabien requested review of this revision.Jul 16 2021, 12:19

Harbormaster completed remote builds in B16204: Diff 29221.Jul 16 2021, 12:34

PiRK accepted this revision.Jul 16 2021, 13:26

This revision is now accepted and ready to land.Jul 16 2021, 13:26

Closed by commit rABCa1d89c232954: Remove the 1s timeouts in p2p_inv_download (authored by Fabien). · Explain WhyJul 16 2021, 19:20

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCa1d89c232954: Remove the 1s timeouts in p2p_inv_download.

Revision Contents
Changeset List

Path

Size

test/

functional/

p2p_inv_download.py

8 lines

Diff 29227

View Options

test/functional/p2p_inv_download.py

Remove the 1s timeouts in p2p_inv_downloadClosedPublicActions