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.

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: hibernate failed #30877

Closed
cockroach-teamcity opened this issue Oct 2, 2018 · 9 comments
Closed

roachtest: hibernate failed #30877

cockroach-teamcity opened this issue Oct 2, 2018 · 9 comments
Assignees
Labels
A-sql-pgcompat Semantic compatibility with PostgreSQL C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot.
Milestone

Comments

@cockroach-teamcity
Copy link
Member

SHA: https://github.com/cockroachdb/cockroach/commits/0f5a289d49105cd317a2713b71546d84c25547f6

Parameters:

To repro, try:

# Don't forget to check out a clean suitable branch and experiment with the
# stress invocation until the desired results present themselves. For example,
# using stressrace instead of stress and passing the '-p' stressflag which
# controls concurrency.
./scripts/gceworker.sh start && ./scripts/gceworker.sh mosh
cd ~/go/src/github.com/cockroachdb/cockroach && \
make stress TESTS=hibernate PKG=roachtest TESTTIMEOUT=5m STRESSFLAGS='-stderr=false -maxtime 20m -timeout 10m'

Failed test: https://teamcity.cockroachdb.com/viewLog.html?buildId=937459&tab=buildLog

The test failed on master:
	test.go:505,hibernate.go:199,hibernate.go:214: 
		0 tests failed unexpectedly
		2 tests passed unexpectedly
		0 tests were not run that were expected to fail
		See artifacts/hibernate/log/logs/report for more details

@cockroach-teamcity cockroach-teamcity added this to the 2.2 milestone Oct 2, 2018
@cockroach-teamcity cockroach-teamcity added C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. labels Oct 2, 2018
@tbg tbg added the A-sql-pgcompat Semantic compatibility with PostgreSQL label Oct 2, 2018
@tbg tbg modified the milestones: 2.2, 2.1 Oct 2, 2018
@cockroach-teamcity
Copy link
Member Author

SHA: https://github.com/cockroachdb/cockroach/commits/f78cc21bb4d7bd3a79e44b4608c04684edf28179

Parameters:

To repro, try:

# Don't forget to check out a clean suitable branch and experiment with the
# stress invocation until the desired results present themselves. For example,
# using stressrace instead of stress and passing the '-p' stressflag which
# controls concurrency.
./scripts/gceworker.sh start && ./scripts/gceworker.sh mosh
cd ~/go/src/github.com/cockroachdb/cockroach && \
make stress TESTS=hibernate PKG=roachtest TESTTIMEOUT=5m STRESSFLAGS='-stderr=false -maxtime 20m -timeout 10m'

Failed test: https://teamcity.cockroachdb.com/viewLog.html?buildId=939960&tab=buildLog

The test failed on master:
	test.go:505,hibernate.go:199,hibernate.go:214: 
		0 tests failed unexpectedly
		2 tests passed unexpectedly
		0 tests were not run that were expected to fail
		See artifacts/hibernate/log/logs/report for more details

@BramGruneir
Copy link
Member

BramGruneir commented Oct 3, 2018 via email

@BramGruneir
Copy link
Member

Here are two new passing tests:

--- PASS: org.hibernate.test.legacy.FooBarTest.testLimit (unexpected)
--- PASS: org.hibernate.test.legacy.FooBarTest.testPSCache (unexpected)

I'll see if they pass in current master vs one from a week ago.

@cockroach-teamcity
Copy link
Member Author

SHA: https://github.com/cockroachdb/cockroach/commits/6a236533824c797948eaddbd6bedf5e12eee1a13

Parameters:

To repro, try:

# Don't forget to check out a clean suitable branch and experiment with the
# stress invocation until the desired results present themselves. For example,
# using stressrace instead of stress and passing the '-p' stressflag which
# controls concurrency.
./scripts/gceworker.sh start && ./scripts/gceworker.sh mosh
cd ~/go/src/github.com/cockroachdb/cockroach && \
make stress TESTS=hibernate PKG=roachtest TESTTIMEOUT=5m STRESSFLAGS='-stderr=false -maxtime 20m -timeout 10m'

Failed test: https://teamcity.cockroachdb.com/viewLog.html?buildId=942507&tab=buildLog

The test failed on master:
	test.go:505,hibernate.go:199,hibernate.go:214: 
		0 tests failed unexpectedly
		2 tests passed unexpectedly
		0 tests were not run that were expected to fail
		See artifacts/hibernate/log/logs/report for more details

@BramGruneir
Copy link
Member

It looks like this commit fixed 7ba018c both tests. Well done @jordanlewis!

I'll send an update to the hiberante test with the new tests expected to be passing.

@petermattis
Copy link
Collaborator

This makes me wonder how much analysis we've done of the hibernate test failures. Are there other test failures in there that are pointing to real bugs and not just lack of features?

@BramGruneir
Copy link
Member

@jordanlewis and I were just discussing that actually. And probably a number of them are.

The analysis I preformed before was on a sampling of the failures, I looked at 150 of them and categorized them. Most were from features we don't have. Large object storage, cursors, etc. But it is worth the time in the 2.2 timeframe to go through all of the remaining failures now that we have a larger team and this work can be split up.

Until this roachtest was created, we didn't really have a list of what was newly failing and what was newly passing, so without those deltas, it was not possible perform any real analysis on it. Now we have the data and can proceed accordingly.

@petermattis
Copy link
Collaborator

@BramGruneir you might want to file an issue so that this idea doesn't get lost.

BramGruneir added a commit to BramGruneir/cockroach that referenced this issue Oct 4, 2018
Two tests are now passing, see cockroachdb#30877 for details and discussion.

Fixes cockroachdb#30877

Release note: None
@BramGruneir
Copy link
Member

Done, see #30975

craig bot pushed a commit that referenced this issue Oct 4, 2018
30970: roachtest: update hibernate blacklist r=BramGruneir a=BramGruneir

Two tests are now passing, see #30877 for details and discussion.

Fixes #30877

Release note: None

Co-authored-by: Bram Gruneir <[email protected]>
@craig craig bot closed this as completed in #30970 Oct 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-sql-pgcompat Semantic compatibility with PostgreSQL C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot.
Projects
None yet
Development

No branches or pull requests

4 participants