roachtest: add startOpts.RoachprodOpts.SkipWaitForSQL
#141115
Labels
C-enhancement
Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)
E-quick-win
Likely to be a quick win for someone experienced.
T-testeng
TestEng Team
In [1],
WaitForSQLReady
was added immediately afterroachprod.Start
to ensure freshly started nodes are ready to serve (and authenticate) SQL sessions. This behavior is specific toroachtest
; i.e.,roachprod start
remains unchanged. The reason forWaitForSQLReady
is described in [2]. Essentially, part of the SQL subsystem is initialized asynchronously; session authentication isn't available until it's fully initialized.Some roachtests, e.g.,
cluster-init
, setSkipInit
, effectively disabling automated cluster initialization. In that case,WaitForSQLReady
is also skipped, since we can't place a definite upper bound on when SQL will eventually be ready. More recently, another roachtest, namelynetwork/authentication
was found to flake due toWaitForSQLReady
[3]. While the test was made compatible withWaitForSQLReady
, it's likely we will run into other examples in the future. Thus, we should add an opt-out viastartOpts.RoachprodOpts.SkipWaitForSQL
.[1] #138109
[2] #137988
[3] #138806
Jira issue: CRDB-47618
The text was updated successfully, but these errors were encountered: