-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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.
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
roachtest: follower-reads/mixed-version/single-region failed #83502
Comments
roachtest.follower-reads/mixed-version/single-region failed with artifacts on master @ ac11a55499120ffd33016feebc47b4ed43a21007:
Parameters: |
I would hazard a guess that this is invalid span usage (vs a bug in tracing), and it seems to be coming from a trace span thati s set up and finished as part of the jobs infrastructure. cc @cockroachdb/jobs
|
Similar panic from https://teamcityartifacts.crdb.io/presignedTokenAuth/6d4ebdcf-2092-4104-8d47-d9520028793d/repository/download/Cockroach_Nightlies_RoachtestNightlyGceBazel/5582417:id/follower-reads/mixed-version/single-region/run_1/artifacts.zip!/logs/2.unredacted/cockroach-stderr.log with a different stack:
|
The assertion in question was added very recently: #81079 |
Looks like it is a bug in tracing. When we don't have room for all of the spans in a record, we fallthrough to an assertion that is intended for StructuredRecordings rather than verbose recording. Will put together a fix once I have a unit test for this. |
roachtest.follower-reads/mixed-version/single-region failed with artifacts on master @ e1e99da678e29db4ef21813423f012273c99add2:
Parameters: |
roachtest.follower-reads/mixed-version/single-region failed with artifacts on master @ 7f8fef4c3f24ec2b6f9a3db1eb38756da68b3d5a:
Parameters:
ROACHTEST_cloud=gce
,ROACHTEST_cpu=2
,ROACHTEST_ssd=0
Help
See: roachtest README
See: How To Investigate (internal)
This test on roachdash | Improve this report!
Jira issue: CRDB-17109
The text was updated successfully, but these errors were encountered: