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.

Sign up for GitHub

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

Jump to bottom

Get rid of AllowSlowTests #8896

Open

mbautin opened this issue Jun 14, 2021 · 0 comments

Assignees

Collaborator

mbautin commented Jun 14, 2021

We have ~50 occurrences of AllowSlowTests across the codebase. Our test suite does not run in the "slow tests" mode so those tests are not being executed. We should enable some of them, where it makes sense, and delete the rest, and remove the AllowSlowTests flag.

mbautin self-assigned this

mbautin added a commit to mbautin/yugabyte-db that referenced this issue


          [yugabyte#8896] [WIP] Remove divison between fast and slow tests

6bf5150

Summary: Make AllowSlowTests always return true.

Test Plan: Jenkins: test regex: .*(all_types-itest|alter_table-randomized-test|alter_table-test|catalog_manager-test|client-stress-test|client-test|crc-test|create-table-stress-test|delete_table-test|full_stack-insert-scan-test|linked_list-test|log-test|master_failover-itest|monotime-test|raft_consensus-itest|remote_bootstrap-itest|rle-test|rpc-test|rwc_lock-test|tablet-pushdown-test|tablet_random_access-test|tablet_replacement-itest|tablet_server-stress-test|tablet_server-test|test_util|test_util.h:bool|test_utilbool|thread-test|trace-test|update_scan_delta_compact-test|verifyrows-tablet-test|).*

Reviewers: jenkins-bot

Differential Revision: https://phabricator.dev.yugabyte.com/D11927

mbautin added a commit to mbautin/yugabyte-db that referenced this issue


          [yugabyte#8896] [WIP] Remove the separation between "fast" and "slow"…

a7c3117

… C++ tests

Summary: Make AllowSlowTests always return true.

Test Plan: Jenkins: test regex: .*(all_types-itest|alter_table-randomized-test|alter_table-test|catalog_manager-test|client-stress-test|client-test|crc-test|create-table-stress-test|delete_table-test|full_stack-insert-scan-test|linked_list-test|log-test|master_failover-itest|monotime-test|raft_consensus-itest|remote_bootstrap-itest|rle-test|rpc-test|rwc_lock-test|tablet-pushdown-test|tablet_random_access-test|tablet_replacement-itest|tablet_server-stress-test|tablet_server-test|test_util|test_util.h:bool|test_utilbool|thread-test|trace-test|update_scan_delta_compact-test|verifyrows-tablet-test).*

Reviewers: jenkins-bot

Differential Revision: https://phabricator.dev.yugabyte.com/D11927

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment