community.cassandra icon indicating copy to clipboard operation
community.cassandra copied to clipboard

Update ansible-test.yml

Open rhysmeister opened this issue 9 months ago • 2 comments

SUMMARY

Try C* 4.1 in CI

ISSUE TYPE
  • Feature Pull Request

rhysmeister avatar May 08 '24 18:05 rhysmeister

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 64.67%. Comparing base (5b0e6df) to head (d7f15a4). Report is 7 commits behind head on master.

:exclamation: Current head d7f15a4 differs from pull request most recent head dd97ea3. Consider uploading reports for the commit dd97ea3 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #278       +/-   ##
===========================================
- Coverage   76.81%   64.67%   -12.15%     
===========================================
  Files          37        9       -28     
  Lines        2661      535     -2126     
  Branches      606      110      -496     
===========================================
- Hits         2044      346     -1698     
+ Misses        323      106      -217     
+ Partials      294       83      -211     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 08 '24 19:05 codecov[bot]

cassandra_interdcstreamthroughput - Error requesting the -d flag is used for the get cmd when C*4.1 is used.

Demo from 4.1.1...

root@34a74dfec4ac:/# nodetool getinterdcstreamthroughput error: Use the -d flag to quiet this error and get the exact throughput in megabits/s -- StackTrace -- java.lang.RuntimeException: Use the -d flag to quiet this error and get the exact throughput in megabits/s at org.apache.cassandra.tools.nodetool.GetInterDCStreamThroughput.execute(GetInterDCStreamThroughput.java:84) at org.apache.cassandra.tools.NodeTool$NodeToolCmd.runInternal(NodeTool.java:371) at org.apache.cassandra.tools.NodeTool$NodeToolCmd.run(NodeTool.java:356) at org.apache.cassandra.tools.NodeTool.execute(NodeTool.java:259) at org.apache.cassandra.tools.NodeTool.main(NodeTool.java:83)

root@34a74dfec4ac:/# nodetool getinterdcstreamthroughput -d Current stream throughput: 201.326592 Mb/s

Need to conditionally add this flag to the interdcstreamthrough get cmd.

rhysmeister avatar May 09 '24 18:05 rhysmeister

Tests pass with 4.1 streamthroughput and interdcstreamthroughput probably require a bit of a clean up.

rhysmeister avatar May 14 '24 19:05 rhysmeister