cockroach
cockroach copied to clipboard
roachtest: use non root authentication by default
This PR attempts to minimize the usage of root user authentication in roachtests, as the root user skips certain authentication paths.
This is done through changing the default mode of authentication to AuthUserCert instead of AuthRootCert. This establishes non root user auth as the preferred default for roachtests and forces tests to explicitly opt into root auth. Relevant roachtest/roachprod helpers such as c.Conn and {pgurl} will now default to using the DefaultUser instead of root.
The majority of tests should now be authenticating with a non root user. The exceptions are:
- Tests that run in insecure mode.
multitenanttests that use the old API. The old API does not create a default admin user for the tenant or copy certs to the tenant. While thec2ctests have helpers that do this, it would be easier/better to just switch the problematic tests to the new API than to try and reuse the c2c API.- Roachprod cluster setup that cannot use a non root user, i.e. the command to create the default non root user.
Release note: none Epic: none Fixes:
Ran the GCE nightly suite here. Only a few related test failures that have been fixed. Rerunning those + preempted tests here
tpchvec/direct_scans/mt-shared-process failure was due to a rebase mistake, fixed.
TFTRs!
bors r=srosenberg, renatolabs