Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

imp(test): turn on full backtraces and disable frame filtering #3763

Merged
merged 4 commits into from
Mar 8, 2022

Conversation

gustavovalverde
Copy link
Member

Motivation

In #3489, the test failure was hard to diagnose, because COLORBT_SHOW_HIDDEN=1 hid some of the function calls we needed to see.

Fixes #3536

Specifications

COLORBT_SHOW_HIDDEN env variable, disabling frame filtering

https://github.com/athre0z/color-backtrace/blob/master/CHANGELOG.md#v042-2020-05-19

Solution

  • Disable frame filtering
  • Add Rust full backtraces

Review

Anyone can review this

@gustavovalverde gustavovalverde added A-devops Area: Pipelines, CI/CD and Dockerfiles P-Medium ⚡ labels Mar 7, 2022
@gustavovalverde gustavovalverde requested a review from a team as a code owner March 7, 2022 17:53
@gustavovalverde gustavovalverde requested review from teor2345 and removed request for a team March 7, 2022 17:53
Copy link
Contributor

@teor2345 teor2345 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

There are about 7 other instances of RUST_BACKTRACE in Zebra, can you fix them all?

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/test-full-sync.yml Show resolved Hide resolved
.github/workflows/test.yml Show resolved Hide resolved
@teor2345 teor2345 added A-diagnostics Area: Diagnosing issues or monitoring performance P-Low ❄️ and removed P-Medium ⚡ labels Mar 7, 2022
@teor2345
Copy link
Contributor

teor2345 commented Mar 7, 2022

I'm marking this as a low priority, because it only caused that one problem.

@codecov
Copy link

codecov bot commented Mar 7, 2022

Codecov Report

Merging #3763 (9b3a248) into main (0e0aefa) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3763      +/-   ##
==========================================
- Coverage   78.91%   78.88%   -0.04%     
==========================================
  Files         292      292              
  Lines       33311    33311              
==========================================
- Hits        26287    26276      -11     
- Misses       7024     7035      +11     

Copy link
Contributor

@teor2345 teor2345 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy to approve as is, or anyone can approve after the minor fix.

docker/zcash-params/Dockerfile Show resolved Hide resolved
@teor2345
Copy link
Contributor

teor2345 commented Mar 7, 2022

Error: API rate limit exceeded for installation ID 1523599.

mergify bot added a commit that referenced this pull request Mar 8, 2022
mergify bot added a commit that referenced this pull request Mar 8, 2022
mergify bot added a commit that referenced this pull request Mar 8, 2022
@mergify mergify bot merged commit 3923c3d into main Mar 8, 2022
@mergify mergify bot deleted the imp-tests-verbosity branch March 8, 2022 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles A-diagnostics Area: Diagnosing issues or monitoring performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Turn on full backtraces using COLORBT_SHOW_HIDDEN=1
2 participants