Zac Bergquist

This metric should also exist on auth, for the number of incomplete uploads to external storage.

#15330 is newer, so lets merge that instead.

@gabrielcorado do your new recording mode role options make it possible to disable session recording on a per-role basis?

Are you referring to the teleport daemon or Teleport Connect here? AFAIK, the only exe we ship today is tsh.exe.

Should this be on v9 too?

This one looks good to go, can we get it merged @capnspacehook?

Looks like audit on is flaky: ``` --- FAIL: TestIntegrations (52.44s) --- FAIL: TestIntegrations/AuditOn (52.43s) --- FAIL: TestIntegrations/AuditOn/recording_proxy (14.18s) integration_test.go:379: Skipping mismatching session ed8d74c8-eec4-459f-ae7f-1fbd11f21a08, expecting upload of 63fd3d91-7e47-4db5-abe3-67a6a4130dec. integration_test.go:379: Skipping...

> it looks to be related to the upload completer being late and not any of the timing-related fetching code changed in this PR as it errors on a seperate...

These could be unrelated, but `TestClientDisconnect` (which was flaky back in 2021 but then fixed) is failing: ``` =================================================== OUTPUT github.com/gravitational/teleport/lib/srv/regular.TestClientDisconnect =================================================== === RUN TestClientDisconnect === PAUSE TestClientDisconnect === CONT...

I believe we're also seeing this issue, but there is not high-latency as both the client and server are running in the same process. It's rare (1 out of every...