Skip to content

Commit

Permalink
logictest: triage tests that block 3node-tenant from running
Browse files Browse the repository at this point in the history
This is done by adding an associated issue number to all directives that
block 3node-tenant.

Release note: None (testing change)
  • Loading branch information
asubiotto committed Jun 11, 2020
1 parent 43ba173 commit f107077
Show file tree
Hide file tree
Showing 20 changed files with 24 additions and 28 deletions.
4 changes: 1 addition & 3 deletions pkg/sql/logictest/testdata/logic_test/alter_primary_key
Original file line number Diff line number Diff line change
@@ -1,6 +1,4 @@
# 3node-tenant fails due to
# https://github.com/cockroachdb/cockroach/issues/47900.
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(50048)

statement ok
SET experimental_enable_hash_sharded_indexes = true
Expand Down
8 changes: 3 additions & 5 deletions pkg/sql/logictest/testdata/logic_test/builtin_function
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
# 3node-tenant fails due to:
# https://github.com/cockroachdb/cockroach/issues/48375
# Specifically, a status server is unavailable when attempting to set a zone
# config.
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(49854)
# TODO(asubiotto): Possibly split out the test that attempts to set a zone
# config in order to run the other ones.

statement ok
CREATE TABLE foo (a int)
Expand Down
3 changes: 1 addition & 2 deletions pkg/sql/logictest/testdata/logic_test/cascade
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# This test times out when run with 3node-tenant.
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(50048)

# The tests in this file target the legacy FK paths.
statement ok
Expand Down
3 changes: 1 addition & 2 deletions pkg/sql/logictest/testdata/logic_test/cascade_opt
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# This test times out when run with 3node-tenant.
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(50048)

# The tests in this file target the new optimizer-driven FK paths (with
# fall back on the legacy paths for unsupported cases).
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/crdb_internal
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(47895)
query error database "crdb_internal" does not exist
ALTER DATABASE crdb_internal RENAME TO not_crdb_internal

Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/delete
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(50048)
statement ok
CREATE TABLE kv (
k INT PRIMARY KEY,
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/enums
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(50049)

statement ok
SET experimental_enable_enums=true;
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/event_log
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(50047)
##################
# TABLE DDL
##################
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/family
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(50048)
# a is the primary key so b gets optimized into a one column value. The c, d
# family has two columns, so it's encoded as a tuple
statement ok
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/fk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(49582)
# The tests in this file target the legacy FK paths.
statement ok
SET optimizer_foreign_keys = false
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/interleaved
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(49854)
# Grandparent table
statement ok
CREATE TABLE p2 (i INT PRIMARY KEY, s STRING)
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/interleaved_join
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(50050)
# The following tables form the interleaved hierarchy:
# name: primary key: # rows: 'a' = id mod X :
# parent1 (pid1) 40 8
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/lookup_join
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(49582)
statement ok
CREATE TABLE abc (a INT, b INT, c INT, PRIMARY KEY (a, c));
INSERT INTO abc VALUES (1, 1, 2), (2, 1, 1), (2, NULL, 2)
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/run_control
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(47895)
query error job with ID 1 does not exist
PAUSE JOB 1

Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/schema_change_in_txn
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(49854)
# Disable automatic stats to avoid flakiness (sometimes causes retry errors).
statement ok
SET CLUSTER SETTING sql.stats.automatic_collection.enabled = false
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/select_index
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(49582)
statement ok
CREATE TABLE t (
a INT PRIMARY KEY,
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/sqlsmith
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(47899)
# This file contains regression tests discovered by sqlsmith.


Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/system
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(49854)
query T
SHOW DATABASES
----
Expand Down
4 changes: 3 additions & 1 deletion pkg/sql/logictest/testdata/logic_test/system_namespace
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(48778)
# When run with a tenant, system.namespace has an extra entry for
# descriptor_id_seq.
query IITI rowsort
SELECT * FROM system.namespace
----
Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/zone_config
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# LogicTest: !3node-tenant
# LogicTest: !3node-tenant(49854)
# Check that we can alter the default zone config.

statement ok
Expand Down

0 comments on commit f107077

Please sign in to comment.