[
https://issues.apache.org/jira/browse/CASSANDRA-16683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17349555#comment-17349555
]
David Capwell commented on CASSANDRA-16683:
-------------------------------------------
Starting commit
CI Results (pending):
||Branch||Source||Circle CI||Jenkins||
|cassandra-3.0|[branch|https://github.com/dcapwell/cassandra/tree/commit_remote_branch/CASSANDRA-16683-cassandra-3.0-EE2779F4-1496-4C96-B603-E79E9C948014]|[build|https://app.circleci.com/pipelines/github/dcapwell/cassandra?branch=commit_remote_branch%2FCASSANDRA-16683-cassandra-3.0-EE2779F4-1496-4C96-B603-E79E9C948014]|[build|https://ci-cassandra.apache.org/job/Cassandra-devbranch/797/]|
|cassandra-3.11|[branch|https://github.com/dcapwell/cassandra/tree/commit_remote_branch/CASSANDRA-16683-cassandra-3.11-EE2779F4-1496-4C96-B603-E79E9C948014]|[build|https://app.circleci.com/pipelines/github/dcapwell/cassandra?branch=commit_remote_branch%2FCASSANDRA-16683-cassandra-3.11-EE2779F4-1496-4C96-B603-E79E9C948014]|[build|https://ci-cassandra.apache.org/job/Cassandra-devbranch/798/]|
|cassandra-4.0|[branch|https://github.com/dcapwell/cassandra/tree/commit_remote_branch/CASSANDRA-16683-cassandra-4.0-EE2779F4-1496-4C96-B603-E79E9C948014]|[build|https://app.circleci.com/pipelines/github/dcapwell/cassandra?branch=commit_remote_branch%2FCASSANDRA-16683-cassandra-4.0-EE2779F4-1496-4C96-B603-E79E9C948014]|[build|https://ci-cassandra.apache.org/job/Cassandra-devbranch/799/]|
|trunk|[branch|https://github.com/dcapwell/cassandra/tree/commit_remote_branch/CASSANDRA-16683-trunk-EE2779F4-1496-4C96-B603-E79E9C948014]|[build|https://app.circleci.com/pipelines/github/dcapwell/cassandra?branch=commit_remote_branch%2FCASSANDRA-16683-trunk-EE2779F4-1496-4C96-B603-E79E9C948014]|[build|https://ci-cassandra.apache.org/job/Cassandra-devbranch/800/]|
> StandaloneVerifier does not fail when unable to verify SSTables, it only
> fails if Corruption is thrown
> ------------------------------------------------------------------------------------------------------
>
> Key: CASSANDRA-16683
> URL: https://issues.apache.org/jira/browse/CASSANDRA-16683
> Project: Cassandra
> Issue Type: Bug
> Components: Test/dtest/python, Tool/sstable
> Reporter: David Capwell
> Assignee: David Capwell
> Priority: Normal
> Fix For: 3.0.x, 3.11.x, 4.0-rc
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> offline_tools_test.py::TestOfflineTools::test_sstableverify has the following
> check
> {code}
> try:
> (out, error, rc) = node1.run_sstableverify("keyspace1",
> "standard1", options=['-v'])
> except ToolError as e:
> # Process sstableverify output to normalize paths in string to
> Python casing as above
> error = re.sub("(?<=Corrupted: ).*", lambda match:
> os.path.normcase(match.group(0)), str(e))
> assert re.search("Corrupted: " + sstable1, error)
> assert e.exit_status == 1, str(e.exit_status)
> {code}
> This checks if the corrupt log is present IFF ToolError is thrown, but does
> not validate that the error is actually thrown. I tried calling the same
> logic before the try to validate and see that it does not fail. If we fix
> the test to check for error we also see that the log that is returned to the
> user does not match 2.2’s behavior but instead returns different logic as
> digest validation throws IOException, which we do not convert to a
> CorruptSSTableException (which is the message the test checks for).
> This also shows another big issue, that when the digest fails verify passes
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
