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.

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

roachtest: unoptimized-query-oracle/disable-rules=half failed #88666

Closed
cockroach-teamcity opened this issue Sep 24, 2022 · 3 comments · Fixed by #88716
Closed

roachtest: unoptimized-query-oracle/disable-rules=half failed #88666

cockroach-teamcity opened this issue Sep 24, 2022 · 3 comments · Fixed by #88716
Assignees
@renatolabs
Labels
branch-master Failures and bugs on the master branch. C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot. T-sql-queries SQL Queries Team
Milestone
22.2

Comments

@cockroach-teamcity
Copy link
Member

cockroach-teamcity commented Sep 24, 2022 •
edited by cockroach-jira-scripts
Loading

roachtest.unoptimized-query-oracle/disable-rules=half failed with artifacts on master @ 89f4ad907a1756551bd6864c3e8516eeff6b0e0a:

test artifacts and logs in: /artifacts/unoptimized-query-oracle/disable-rules=half/run_1
	query_comparison_util.go:147,query_comparison_util.go:66,unoptimized_query_oracle.go:51,test_runner.go:928: pq: use of partitions requires an enterprise license. see https://cockroachlabs.com/pricing?cluster=fc545320-71b5-4382-960b-0e1746d24684 for details on how to enable enterprise features

Parameters: ROACHTEST_cloud=gce , ROACHTEST_cpu=4 , ROACHTEST_encrypted=false , ROACHTEST_fs=ext4 , ROACHTEST_localSSD=true , ROACHTEST_ssd=0

Help

See: roachtest README

See: How To Investigate (internal)

/cc @cockroachdb/sql-queries

This test on roachdash | Improve this report!

Jira issue: CRDB-19890
@cockroach-teamcity cockroach-teamcity added branch-master Failures and bugs on the master branch. C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. labels Sep 24, 2022
@cockroach-teamcity cockroach-teamcity added this to the 22.2 milestone Sep 24, 2022
@blathers-crl blathers-crl bot added the T-sql-queries SQL Queries Team label Sep 24, 2022
@cockroach-teamcity
Copy link
Member Author

roachtest.unoptimized-query-oracle/disable-rules=half failed with artifacts on master @ 51c8aae748d338549400c047796c6c9b892527da:

test artifacts and logs in: /artifacts/unoptimized-query-oracle/disable-rules=half/run_1
	query_comparison_util.go:147,query_comparison_util.go:66,unoptimized_query_oracle.go:51,test_runner.go:928: pq: use of partitions requires an enterprise license. see https://cockroachlabs.com/pricing?cluster=9b0f5345-3ff4-48ba-9169-6c91d0504403 for details on how to enable enterprise features

Parameters: ROACHTEST_cloud=gce , ROACHTEST_cpu=4 , ROACHTEST_encrypted=false , ROACHTEST_fs=ext4 , ROACHTEST_localSSD=true , ROACHTEST_ssd=0

Help

See: roachtest README

See: How To Investigate (internal)

This test on roachdash | Improve this report!

@yuzefovich yuzefovich removed the release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. label Sep 25, 2022
@yuzefovich
Copy link
Member

Probably #88514 is to blame.

@cockroach-teamcity
Copy link
Member Author

roachtest.unoptimized-query-oracle/disable-rules=half failed with artifacts on master @ ff13325e9368c4e8dd9a4d5cf4aa2ad2f33e9ac0:

test artifacts and logs in: /artifacts/unoptimized-query-oracle/disable-rules=half/run_1
	query_comparison_util.go:147,query_comparison_util.go:66,unoptimized_query_oracle.go:51,test_runner.go:928: pq: use of partitions requires an enterprise license. see https://cockroachlabs.com/pricing?cluster=540b25e7-3033-463d-bcf5-079069a304fc for details on how to enable enterprise features

Parameters: ROACHTEST_cloud=gce , ROACHTEST_cpu=4 , ROACHTEST_encrypted=false , ROACHTEST_ssd=0

Help

See: roachtest README

See: How To Investigate (internal)

This test on roachdash | Improve this report!

renatolabs added a commit to renatolabs/cockroach that referenced this issue Sep 26, 2022
@renatolabs
roachprod: set default cluster settings when starting.
fbdf786 
In cockroachdb#88514, the cluster start logic was refactored to reuse the same
code across `init` and `start`, fixing a bug in the former. However,
the refactoring overlooked the fact that we previously always set the
default cluster settings when there's more than one node in the
cluster.

This fixes that by setting the default cluster settings in that case;
one particularly important cluster setting is the license key,
necessary for some roachtests.

Fixes cockroachdb#88660, cockroachdb#88665, cockroachdb#88666.

Release note: None
@renatolabs renatolabs mentioned this issue Sep 26, 2022
Merged
craig bot pushed a commit that referenced this issue Sep 26, 2022
@aliher1911 @ZhouXing19
@ajwerner @renatolabs
Merge #88252 #88295 #88670 #88716
c432bd7 
88252: storage: add gc benchmarks with multiple range tombstones r=erikgrinaker a=aliher1911

Previously we only had tests that verify GC performance with no range tombstones or with a single range tombstone under all the data. This commit extends the benchmark to have many tombstones interleaving data at different keys.

Release note: None

Fixes #87417

88295: sql: add assertion for txn passed to internal executor r=ajwerner a=ZhouXing19

If an internal executor is created with a txn binding to it, it should not be used to execute a statement with a nil txn.

Release note: None

88670: schemachanger/rel: fix race due to failure to clone constraint slots r=ajwerner a=ajwerner

The fundamental race here is that while the slots themselves were being copied by value, the "any" clauses which are a slice were not. The second bug here is that the "inline" values were not being properly reset. That bug could lead to problems when the query was run again in the context of a different element set. We need to reset those inline values too.

Fixes #88628

Release note: None

88716: roachprod: set default cluster settings when starting r=yuzefovich,srosenberg,smg260 a=renatolabs

In #88514, the cluster start logic was refactored to reuse the same code across `init` and `start`, fixing a bug in the former. However, the refactoring overlooked the fact that we previously always set the default cluster settings when there's more than one node in the cluster.

This fixes that by setting the default cluster settings in that case; one particularly important cluster setting is the license key, necessary for some roachtests.

Fixes #88660
Fixes #88665
Fixes #88666
Fixes #88710.

Release note: None

Co-authored-by: Oleg Afanasyev <oleg@cockroachlabs.com>
Co-authored-by: Jane Xing <zhouxing@uchicago.edu>
Co-authored-by: Andrew Werner <awerner32@gmail.com>
Co-authored-by: Renato Costa <renato@cockroachlabs.com>
@craig craig bot closed this as completed in ed443e1 Sep 26, 2022
smg260 pushed a commit to smg260/cockroach that referenced this issue Jan 18, 2023
@renatolabs
roachprod: set default cluster settings when starting
0e614a5 
In cockroachdb#88514, the cluster start logic was refactored to reuse the same
code across `init` and `start`, fixing a bug in the former. However,
the refactoring overlooked the fact that we previously always set the
default cluster settings when there's more than one node in the
cluster.

This fixes that by setting the default cluster settings in that case;
one particularly important cluster setting is the license key,
necessary for some roachtests.

Fixes cockroachdb#88660.
Fixes cockroachdb#88665.
Fixes cockroachdb#88666.
Fixes cockroachdb#88710.

Release note: None
smg260 pushed a commit to smg260/cockroach that referenced this issue Jan 18, 2023
@renatolabs
roachprod: set default cluster settings when starting
9c0b17d 
In cockroachdb#88514, the cluster start logic was refactored to reuse the same
code across `init` and `start`, fixing a bug in the former. However,
the refactoring overlooked the fact that we previously always set the
default cluster settings when there's more than one node in the
cluster.

This fixes that by setting the default cluster settings in that case;
one particularly important cluster setting is the license key,
necessary for some roachtests.

Fixes cockroachdb#88660.
Fixes cockroachdb#88665.
Fixes cockroachdb#88666.
Fixes cockroachdb#88710.

Release note: None
smg260 pushed a commit to smg260/cockroach that referenced this issue Mar 7, 2023
@renatolabs
roachprod: set default cluster settings when starting
3765da1 
In cockroachdb#88514, the cluster start logic was refactored to reuse the same
code across `init` and `start`, fixing a bug in the former. However,
the refactoring overlooked the fact that we previously always set the
default cluster settings when there's more than one node in the
cluster.

This fixes that by setting the default cluster settings in that case;
one particularly important cluster setting is the license key,
necessary for some roachtests.

Fixes cockroachdb#88660.
Fixes cockroachdb#88665.
Fixes cockroachdb#88666.
Fixes cockroachdb#88710.

Release note: None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@renatolabs renatolabs

Labels
branch-master Failures and bugs on the master branch. C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot. T-sql-queries SQL Queries Team
Projects
SQL Queries
Archived in project
Milestone
22.2
Development

Successfully merging a pull request may close this issue.

roachprod: set default cluster settings when starting renatolabs/cockroach
3 participants
@yuzefovich @cockroach-teamcity @renatolabs