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:
- before !14411 (merged) there are ops -- https://logs.nomadic-labs.cloud/PRT/master/2024-06-15/x86_executor1/050750-tezt-run.log
- after !14411 (merged) there are no ops -- https://logs.nomadic-labs.cloud/PRT/master/2024-08-14/x86_executor1/090559-tezt-run.log
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.rstfor the protocol and the environment,CHANGES.rstat the root of the repository for everything else). -
Select suitable reviewers using the Reviewersfield below. -
Select as Assigneethe next person who should take action on that MR
Edited by Victor Allombert