Skip to content

Disable folly_synchronization_distributed_mutex_test on ARM for now#6126

Closed
pdillinger wants to merge 1 commit intofacebook:masterfrom
pdillinger:distributed-mutex-test-arm
Closed

Disable folly_synchronization_distributed_mutex_test on ARM for now#6126
pdillinger wants to merge 1 commit intofacebook:masterfrom
pdillinger:distributed-mutex-test-arm

Conversation

@pdillinger
Copy link
Copy Markdown
Contributor

Summary: This test is crashing on ARM but is not yet production code.
Let's not let it block ARM CI. See PR #5932

Test Plan: ./folly_synchronization_distributed_mutex_test, on Linux/ARM,
on Linux/x86_64, and with LITE=1 on Linux/x86_64 (also disabled)

Summary: This test is crashing on ARM but is not yet production code.
Let's not let it block ARM CI.

Test Plan: ./folly_synchronization_distributed_mutex_test, on Linux/ARM,
on Linux/x86_64, and with LITE=1 on Linux/x86_64 (also disabled)
Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@pdillinger has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@pdillinger pdillinger requested a review from siying December 5, 2019 19:56
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@pdillinger merged this pull request in ad528fe.

wolfkdy pushed a commit to wolfkdy/rocksdb that referenced this pull request Dec 22, 2019
…acebook#6126)

Summary:
This test is crashing on ARM but is not yet production code.
Let's not let it block ARM CI. See PR facebook#5932
Pull Request resolved: facebook#6126

Test Plan:
./folly_synchronization_distributed_mutex_test, on Linux/ARM,
on Linux/x86_64, and with LITE=1 on Linux/x86_64 (also disabled)

Differential Revision: D18836576

Pulled By: pdillinger

fbshipit-source-id: d8a36eea2f048e8330411d994435d1c58a15d978
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants