Skip to content

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Jul 5, 2020

A temporary workaround for #19448.

@DrahtBot DrahtBot added the Tests label Jul 5, 2020
@maflcko maflcko merged commit f4301e9 into bitcoin:master Jul 5, 2020
@hebasto hebasto deleted the 200705-tsan-bd branch July 6, 2020 09:49
maflcko pushed a commit that referenced this pull request Jul 17, 2020
0cdf2a7 ci: add tsan debug symbols option (Russell Yanofsky)
9a2f126 ci: Add tsan suppression for race in DatabaseBatch (Hennadii Stepanov)

Pull request description:

  Since #19325 was merged, the corresponding change in TSan suppression file gets required.

  This PR is:
  - an analogous to #19226 and #19450, and
  - a temporary workaround for CI fail like https://cirrus-ci.com/task/5741795508224000?command=ci#L4993

ACKs for top commit:
  MarcoFalke:
    ACK 0cdf2a7

Tree-SHA512: 7832f143887c8a0df99dea03e00694621710378fbe923e3592185fcd3658546a590693b513abffc5ab96e9ef76c9c4bff3330eeee69a0c5dbe7574f34c417220
jasonbcox pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Sep 25, 2020
Summary:
ci: Add tsan suppression for race in BerkeleyBatch (Hennadii Stepanov)

Pull request description:

  A temporary workaround for #19448.

---

Backport of Core [[bitcoin/bitcoin#19450 | PR19450]]

Test Plan:
on a CI VM:
  build-configuration.py build-tsan

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D7560
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants