Differential D10984

Bump the timeout for the debug log messages in test_seed_peers
ClosedPublic
Actions

Authored by Fabien on Feb 4 2022, 09:48.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCb4ac4601d570: Bump the timeout for the debug log messages in test_seed_peers

Summary

There have been a few CI failures on this test. From investigation the failures occurs on assertions that are expecting several debug log messages to appear upon node startup. The default timeout is used which is 2s and sometimes this is not enough for all the messages to be printed. This diff bumps the timeout for the cases where several messages are expected after a node restart to get rid of the failures.

Test Plan

./test/functional/test_runner.py feature_config_args

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Feb 4 2022, 09:48

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 4 2022, 09:49

Fabien requested review of this revision.Feb 4 2022, 09:49

Fabien mentioned this in D10979: Refactor EraseLastKElements to be a special case of EraseLastKElementsIf.Feb 4 2022, 09:53

PiRK accepted this revision.Feb 4 2022, 09:54

This revision is now accepted and ready to land.Feb 4 2022, 09:54

Harbormaster completed remote builds in B18150: Diff 32190.Feb 4 2022, 10:11

Closed by commit rABCb4ac4601d570: Bump the timeout for the debug log messages in test_seed_peers (authored by Fabien). · Explain WhyFeb 4 2022, 10:11

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCb4ac4601d570: Bump the timeout for the debug log messages in test_seed_peers.

Revision Contents
Changeset List

Path

Size

test/

functional/

feature_config_args.py

8 lines

Diff 32191

View Options

test/functional/feature_config_args.py

Bump the timeout for the debug log messages in test_seed_peersClosedPublicActions