[ 
https://issues.apache.org/jira/browse/CASSANDRA-15583?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17206074#comment-17206074
 ] 

Sam Tunnicliffe commented on CASSANDRA-15583:
---------------------------------------------

Thanks [~Bereng], the work so far on the UX tests is a great improvement on 
what we had and having the foundations in place for adding future tests is a 
big win.

In terms of the 'spirit' of the ticket, my interpretation is that we want to be 
confident that none of the bundled tools are completely broken in 4.0 and that 
essential operational tasks can still be performed. Now that the UX tests are 
performing basic smoke tests for the tooling, we're probably done with the 
first part of that and at the point where all of the lhf has been picked.

I agree that fully covering all functions and corner cases of all the tools is 
an unrealistic goal for rc, so perhaps the best way forward is to identify a 
subset of "essential" tools/commands without which operators would be 
completely unable to manage a cluster and ensure those are functionally tested 
(hopefully there may be existing coverage for some). If we make sure of that 
and set the fixver for any follow up tickets to 4.x, then we should be good IMO.

As far as the "essential" tools go, I'm thinking of things like:
 * nodetool
 ** ring
 ** tablestats
 ** tpstats
 ** netstats
 ** disablebinary
 ** enablebinary
 ** gossipinfo
 * sstableupgrade
 * sstablescrub

Things like repair and add/remove/move nodes should already be covered by other 
4.0 testing epics.

What do you think?

> 4.0 quality testing: Tooling, Bundled and First Party
> -----------------------------------------------------
>
>                 Key: CASSANDRA-15583
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15583
>             Project: Cassandra
>          Issue Type: Task
>          Components: Test/dtest/python, Test/unit
>            Reporter: Josh McKenzie
>            Assignee: Berenguer Blasi
>            Priority: Normal
>             Fix For: 4.0-beta, 4.0-triage
>
>
> Reference [doc from 
> NGCC|https://docs.google.com/document/d/1uhUOp7wpE9ZXNDgxoCZHejHt5SO4Qw1dArZqqsJccyQ/edit#]
>  for context.
> *Shepherd: Sam Tunnicliffe*
> Test plans should cover bundled first-party tooling and CLIs such as 
> nodetool, cqlsh, and new tools supporting full query and audit logging 
> (CASSANDRA-13983, CASSANDRA-12151).
> *Progress as of Aug 2020*
> {{ToolRunner}} has been added enabling us to test tools in java unit tests. 
> This includes capturing their stdout/err and stdin i.e. Most tools have a 
> starting unit test testing their cmd line args happy path. Tickets have been 
> created to improve coverage of those  and flagged LHF. Also for those tools 
> big enough they can't be addressed in a simple ticket such as nodetool, a 
> placeholder ticket for future improvements has been created as well. Tickets 
> and status are:
> ||Tool||UX test||UT coverage||dtest coverage||Comments||
> |Nodetool|(x)|(x) CASSANDRA-16026|(!)|Not all the sub commands are tested. 
> Dtest also test nodetool as a side effect|
> |Cqlsh|(x)|(x) CASSANDRA-16025|(!)| |
> |Cassandra-stress|(x)|(x) CASSANDRA-16024|(x)| |
> |debug-cql|(x)|(x) CASSANDRA-16023|(x)| |
> |fqltool|(x)|(/) CASSANDRA-16022|(!)| |
> |auditlogviewer|(/) CASSANDRA-15991|(!) CASSANDRA-16021|(!)| |
> |*Sstable utilities*| | | | |
> |sstabledump|(/) CASSANDRA-15991|(/) CASSANDRA-16020|(!)| |
> |sstableexpiredblockers|(/) CASSANDRA-15991|(x) CASSANDRA-16019|(!)| |
> |sstablelevelreset|(/) CASSANDRA-15991|(x) CASSANDRA-16018|(!)| |
> |sstableloader|(x)|(x) CASSANDRA-16017|(!)| |
> |sstablemetadata|(/) CASSANDRA-15991|(x) CASSANDRA-16016|(x)|Ran in dtests, 
> no dedicated test|
> |sstableofflinerelevel|(/) CASSANDRA-15991|(x) CASSANDRA-16015|(!)| |
> |sstablerepairedset|(/) CASSANDRA-15991|(x) CASSANDRA-16014|(x)|Ran in 
> dtests, no dedicated test|
> |sstablescrub|(/) CASSANDRA-15991|(x) CASSANDRA-16013|(!)| |
> |sstablesplit|(/) CASSANDRA-15991|(x) CASSANDRA-16012|(!)| |
> |sstableupgrade|(/) CASSANDRA-15991|(x) CASSANDRA-16011|(!)| |
> |sstableutil|(/) CASSANDRA-15991|(x) CASSANDRA-16010|(!)| |
> |sstableverify|(/) CASSANDRA-15991|(x) CASSANDRA-16009|(!)| |



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to