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: tpcc/w=100/nodes=3/chaos=true failed #99594

Closed
cockroach-teamcity opened this issue Mar 26, 2023 · 1 comment · Fixed by #99660
Closed

roachtest: tpcc/w=100/nodes=3/chaos=true failed #99594

cockroach-teamcity opened this issue Mar 26, 2023 · 1 comment · Fixed by #99660
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. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. T-testeng TestEng Team
Milestone
23.1

Comments

@cockroach-teamcity
Copy link
Member

cockroach-teamcity commented Mar 26, 2023 •
edited by cockroach-jira-scripts
Loading

roachtest.tpcc/w=100/nodes=3/chaos=true failed with artifacts on master @ d107217dac5d817cc115bc0e97b7e53c0f2878bf:

test artifacts and logs in: /artifacts/tpcc/w=100/nodes=3/chaos=true/run_1
(cluster.go:1969).Run: output in run_064814.705530049_n4_cockroach-workload-c: ./cockroach workload check tpcc --warehouses=100 {pgurl:1} returned: COMMAND_PROBLEM: ssh verbose log retained in ssh_064815.405368374_n4_cockroach-workload-c.log: exit status 1

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

Help

See: roachtest README

See: How To Investigate (internal)

/cc @cockroachdb/test-eng

This test on roachdash | Improve this report!

Jira issue: CRDB-25962
@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 Mar 26, 2023
@cockroach-teamcity cockroach-teamcity added this to the 23.1 milestone Mar 26, 2023
@blathers-crl blathers-crl bot added the T-testeng TestEng Team label Mar 26, 2023
@cockroach-teamcity
Copy link
Member Author

roachtest.tpcc/w=100/nodes=3/chaos=true failed with artifacts on master @ 2bd2c806ab3044569b09e0a205b5bc0452ad4e2b:

test artifacts and logs in: /artifacts/tpcc/w=100/nodes=3/chaos=true/run_1
(cluster.go:1977).Run: output in run_065314.449702283_n4_cockroach-workload-c: ./cockroach workload check tpcc --warehouses=100 {pgurl:1} returned: COMMAND_PROBLEM: ssh verbose log retained in ssh_065315.156371791_n4_cockroach-workload-c.log: exit status 1

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 renatolabs mentioned this issue Mar 27, 2023
Merged
@nvanbenschoten nvanbenschoten mentioned this issue Mar 27, 2023
Merged
craig bot pushed a commit that referenced this issue Mar 27, 2023
@rickystewart @ericharmeling
@knz @nvanbenschoten
Merge #99509 #99532 #99642 #99660
b1a20ba 
99509: bazci: protect against `nil` target r=healthy-pod a=rickystewart

Closes #98861.

Epic: none

Release note: None

99532: sql: inject stats into TestExecBuild_sql_statistics_persisted testdata r=ericharmeling a=ericharmeling

Background thread: #99240 (comment)

Part of #99399.

This commit replaces the `CREATE STATISTICS` statements in the `TestExecBuild_sql_statistics_persisted` testdata with `ALTER TABLE ... INJECT STATISTICS` and removes the retry directives added to deflake the test in #99447.

Epic: none

Release note: None

99642: serverccl: skip TestServerControllerHTTP under deadlock r=dhartunian a=knz

Fixes #98046.
Release note: None

99660: workload/tpcc: disable check 3.3.2.11 after workload r=renatolabs a=nvanbenschoten

Fixes #99619.
Fixes #99594.
Fixes #99603.
Fixes #99604.
Fixes #99617.
Fixes #99616.
Fixes #99613.
Fixes #99611.
Fixes #99604.
Fixes #99603.
Fixes #99600.
Fixes #99599.
Fixes #99598.
Fixes #99596.
Fixes #99595.
Fixes #99594.

This commit disables TPC-C's consistency check 3.3.2.11 (added in #99542) after the workload has run. The check asserts a relationship between the number of rows in the "order" table and rows in the "new_order" table. Rows are inserted into these tables transactional by the NewOrder transaction. However, only rows in the "new_order" table are deleted by the Delivery transaction. Consequently, the consistency condition will fail after the first Delivery transaction is run by the workload.

See #99542 (comment) for more details.

Release note: None

Co-authored-by: Ricky Stewart <[email protected]>
Co-authored-by: Eric Harmeling <[email protected]>
Co-authored-by: Raphael 'kena' Poss <[email protected]>
Co-authored-by: Nathan VanBenschoten <[email protected]>
@craig craig bot closed this as completed in 70b1be2 Mar 27, 2023
aadityasondhi pushed a commit to aadityasondhi/cockroach that referenced this issue Mar 28, 2023
@nvanbenschoten @aadityasondhi
workload/tpcc: disable check 3.3.2.11 after workload
ea8209a 
Fixes cockroachdb#99619.
Fixes cockroachdb#99594.
Fixes cockroachdb#99603.
Fixes cockroachdb#99604.

This commit disables TPC-C's consistency check 3.3.2.11 after the
workload has run. The check asserts a relationship between the number of
rows in the "order" table and rows in the "new_order" table. Rows are
inserted into these tables transactional by the NewOrder transaction.
However, only rows in the "new_order" table are deleted by the Delivery
transaction. Consequently, the consistency condition will fail after the
first Delivery transaction is run by the workload.

See cockroachdb#99542 (comment)
for more details.

Release note: None
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 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. T-testeng TestEng Team
Projects
None yet
Milestone
23.1
Development

Successfully merging a pull request may close this issue.

workload/tpcc: disable check 3.3.2.11 after workload nvanbenschoten/cockroach
1 participant
@cockroach-teamcity