Page MenuHomePhabricator

Merge #17469: test: Remove fragile assert_memory_usage_stable
ClosedPublic

Authored by jasonbcox on May 21 2020, 17:49.

Details

Summary

fac942ca57dce6cfa5655a3ac8664d6a051bc01f test: Remove fragile assert_memory_usage_stable (MarcoFalke)

Pull request description:

This test fails on arm64 and a fuzz tests seems inappropriate for the functional test suite anyway, so remove it.

Example failures:

* https://travis-ci.org/bitcoin/bitcoin/jobs/611497963#L14517
* https://travis-ci.org/MarcoFalke/bitcoin-core/jobs/611029104#L3876

ACKs for top commit:

jamesob:
  ACK https://github.com/bitcoin/bitcoin/pull/17469/commits/fac942ca57dce6cfa5655a3ac8664d6a051bc01f

Tree-SHA512: 3577e7ce5891d221cb798454589ba796ed0c06621a26351bb919c23bc6bb46aafcd0b11cb02bbfde64b74d67cb2950da44959a7ecdc436491a34e8b045c1ccf4

Backport of Core PR17469

I've skipped backporting Core PR14794 which attempts to fix the issue using a not-so-robust approach of
looking at env variables to determine if the memory usage threshold should be modified.

This fixes sanitizer failures on CI by removing the problem entirely.

Test Plan

ninja check-functional
and run ASAN on CI

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

test/functional/p2p_invalid_messages.py
86 ↗(On Diff #20475)

This timeout gets bumped over various backports, so we might as well bump it to the value in this PR.

nakihito added a subscriber: nakihito.
nakihito added inline comments.
test/functional/p2p_invalid_messages.py
86 ↗(On Diff #20475)
test/functional/test_framework/test_node.py
190 ↗(On Diff #20475)

This is different from PR17469 and the change isn't made in PR14794. The changes are made in PR14693 and PR14812, but since its being removed, its probably fine to skip them.

This revision is now accepted and ready to land.May 21 2020, 18:21
test/functional/p2p_invalid_messages.py
86 ↗(On Diff #20475)

cool

test/functional/test_framework/test_node.py
190 ↗(On Diff #20475)

Apparently there were more than one attempts to fix this, which makes the argument to avoid replicating that process all the more strong.