Page MenuHomePhabricator

Fix the check-seeds.sh wrong evaluation of the live nodes
ClosedPublic

Authored by Fabien on Oct 13 2021, 15:37.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABC2553d209b57e: Fix the check-seeds.sh wrong evaluation of the live nodes
Summary

The script count the number of live nodes vs the number of expected nodes, and returns the result as a percentage. I have been monitoring it recently after this percentage dropped below the warning threshold, and found this in the log:

114% of testnet seeds appear to be online.

That's good news, but that's obviously wrong. The error comes from the bitcoin abc stop message sneaking to stdout and messing the count. Because we don't have so many testnet nodes to check, this can lead to this kind of error.

Test Plan
./contrib/seeds/test-seeds.sh --testnet < ./contrib/seeds/nodes_test.txt > out.txt

The out.txt file should no longer contain the Bitcoin ABC stopping line

Diff Detail

Repository
rABC Bitcoin ABC
Branch
fix_check_seeds_count
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17003
Build 33842: Build Diff
Build 33841: arc lint + arc unit