Merge #17469: test: Remove fragile assert_memory_usage_stable
ClosedPublic
Actions

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

Details

Reviewers

nakihito

Group Reviewers

Restricted Project

Commits

rABCa382f87da683: Merge #17469: test: Remove fragile assert_memory_usage_stable

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

jasonbcox created this revision.May 21 2020, 17:49

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 21 2020, 17:49

jasonbcox requested review of this revision.May 21 2020, 17:49

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

jasonbcox planned changes to this revision.May 21 2020, 17:50

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

Build Bitcoin ABC Diffs / Diff Testing (build-asan) started.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

jasonbcox added inline comments.May 21 2020, 17:53

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.

jasonbcox requested review of this revision.May 21 2020, 17:53

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) passed.

Harbormaster completed remote builds in B10947: Diff 20475.May 21 2020, 17:59

Build Bitcoin ABC Diffs / Diff Testing (build-asan) passed.

nakihito accepted this revision.May 21 2020, 18:21

nakihito added a subscriber: nakihito.

nakihito added inline comments.

test/functional/p2p_invalid_messages.py
86 ↗	(On Diff #20475)	https://github.com/bitcoin/bitcoin/pull/16888/files#diff-944d172fa38f5aa35a89412794d08d06R88 This is actually the only remaining change from this PR.
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

jasonbcox added inline comments.May 21 2020, 18:26

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.