[go: up one dir, main page]

Tezt/LTF: fix long dynamic bake ring/clique test

This is a followup MR of !14411 (merged) that was actually not starting well the bakers.

Indeed, if IIUC we need to:

  • run the initial nodes set
  • run the initial bakers set
  • at each step of the main loop
    • remove one node/baker
    • add a new node/baker --> In the former MR, the associated baker was actually not run

Indeed, with the master's version, we can see that the test is not progressing and there is no new operations:

This should fix both ring/clique cluster flavours .

Manually testing the MR

Run: dune exec tezt/long_tests/main.exe -- --title 'long dynamic bake: clique' --verbose -- I recommend to set a lower test bound if you don't want to be bored to death.

modified   tezt/long_tests/tenderbake.ml
@@ -385,7 +385,7 @@ module Long_dynamic_bake = struct
   let test topology = "long dynamic bake: " ^ string_of_topology topology
 
   (* Test runs for [test_levels] *)
-  let test_levels = Cli.get_int "test_levels" ~default:120
+  let test_levels = Cli.get_int "test_levels" ~default:10
 
   (* add an operation every [add_operation_period] seconds *)
   let add_operation_period = Cli.get_float "add_operation_period" ~default:2.0

Monitoring the upcoming LTF results could be nice as well.

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR
Edited by Victor Allombert

Merge request reports

Loading