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: declarative_schema_changer/job-compatibility-mixed-version-V232-V241 failed #123979

Closed
cockroach-teamcity opened this issue May 11, 2024 · 0 comments · Fixed by #124001
Closed

roachtest: declarative_schema_changer/job-compatibility-mixed-version-V232-V241 failed #123979

cockroach-teamcity opened this issue May 11, 2024 · 0 comments · Fixed by #124001
Labels
branch-master Failures on the master branch. C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot. T-testeng TestEng Team
Projects
Test Engineering

Comments

@cockroach-teamcity
Copy link
Member

cockroach-teamcity commented May 11, 2024 •
edited by cockroach-jira-scripts

roachtest.declarative_schema_changer/job-compatibility-mixed-version-V232-V241 failed with artifacts on master @ 59b261a579cbe2c032a5dd3e182ff67aeee900b9:

(mixedversion.go:592).Run: mixed-version test failure while running step 2 (start cluster at version "v24.1.0-rc.1"): cluster.Stage: output:
   1: 	<err> COMMAND_PROBLEM: exit status 2
	curl: (56) OpenSSL SSL_read: Connection reset by peer, errno 104
	
	gzip: stdin: unexpected end of file
	tar: Unexpected EOF in archive
	tar: Unexpected EOF in archive
	tar: Error is not recoverable: exiting now
	
   3: 	<ok>: TRANSIENT_ERROR(stage_failure): COMMAND_PROBLEM: exit status 2 [owner=test-eng]
test artifacts and logs in: /artifacts/declarative_schema_changer/job-compatibility-mixed-version-V232-V241/run_1

Parameters:

  • ROACHTEST_arch=amd64
  • ROACHTEST_cloud=gce
  • ROACHTEST_coverageBuild=false
  • ROACHTEST_cpu=4
  • ROACHTEST_encrypted=false
  • ROACHTEST_metamorphicBuild=false
  • ROACHTEST_ssd=0
Help

See: roachtest README

See: How To Investigate (internal)

See: Grafana

/cc @cockroachdb/test-eng

This test on roachdash | Improve this report!

Jira issue: CRDB-38637
@cockroach-teamcity cockroach-teamcity added branch-master Failures 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. T-testeng TestEng Team labels May 11, 2024
@renatolabs renatolabs removed the release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. label May 13, 2024
@renatolabs renatolabs mentioned this issue May 13, 2024
Merged
craig bot pushed a commit that referenced this issue May 13, 2024
@renatolabs
Merge #124001
c8e257e 
124001: roachtest: implement `As` for `ErrorWithOwnership` r=herkolategan a=renatolabs

Previously, the `ErrorWithOwnership` type would wrap the original error but callers would not be able to use `errors.As` to find out if the error was of a specific type (notably, roachprod transient errors).

In this commit, we introduce an appropriate implementation of `As` for the `ErrorWithOwnerhsip` type, allowing us to properly identify transient failures even when the ownership is changed.

Fixes: #123979

Release note: None

Co-authored-by: Renato Costa <renato@cockroachlabs.com>
@craig craig bot closed this as completed in f9395e0 May 13, 2024
Test Engineering automation moved this from Triage to Done May 13, 2024
blathers-crl bot pushed a commit that referenced this issue May 13, 2024
@renatolabs
roachtest: implement As for ErrorWithOwnership
11c9ab4 
Previously, the `ErrorWithOwnership` type would wrap the original
error but callers would not be able to use `errors.As` to find out if
the error was of a specific type (notably, roachprod transient
errors).

In this commit, we introduce an appropriate implementation of `As`
for the `ErrorWithOwnerhsip` type, allowing us to properly identify
transient failures even when the ownership is changed.

Fixes: #123979

Release note: None
@blathers-crl blathers-crl bot mentioned this issue May 13, 2024
Merged
blathers-crl bot pushed a commit that referenced this issue May 13, 2024
@renatolabs
roachtest: implement As for ErrorWithOwnership
bf03c35 
Previously, the `ErrorWithOwnership` type would wrap the original
error but callers would not be able to use `errors.As` to find out if
the error was of a specific type (notably, roachprod transient
errors).

In this commit, we introduce an appropriate implementation of `As`
for the `ErrorWithOwnerhsip` type, allowing us to properly identify
transient failures even when the ownership is changed.

Fixes: #123979

Release note: None
@blathers-crl blathers-crl bot mentioned this issue May 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
branch-master Failures on the master branch. C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot. T-testeng TestEng Team
Projects
Test Engineering
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

roachtest: implement As for ErrorWithOwnership renatolabs/cockroach
2 participants
@cockroach-teamcity @renatolabs