Apparently waitInHello fail point signature change made is somewhat incompatible with existing test set_step_params.js. However mind that the test started failing in 2020 a year before this fail point refactoring, so the logic there was not strong anyway.

At least we need to adjust the test to the fail point change, which may not be a 100% fix but at least should make it better, then we can revisit the race.

Old signature:
MONGO_FAIL_POINT_PAUSE_WHILE_SET_OR_INTERRUPTED(opCtx, waitInHello);
new one:
boost::optional<Milliseconds> sleepTimeout;
MONGO_FAIL_POINT_BLOCK(waitInHello, customArgs)

{ sleepTimeout = handleHelloFailPoint(customArgs.getData(), cmdObj); }

if (MONGO_unlikely(sleepTimeout))

{ sleepForDurationOrUntilShutdown(*sleepTimeout); }