fix failing api failover tests

[lukasIO]

livekit/protocol#1766 caused the tests to fail

[devin-ai-integration]

🔍 Devin Review: 1 flag

Not posted on this PR by your GitHub settings — view it in Devin Review. (Configure)

[stephen-derosa]

@lukasIO i think we need to revert the upstream PR since this will be extended to a bunch of client sdks (rust included)

[ChelseaBradbury]

I think this line from the readme should also be changed:

python-sdks/README.md

Line 134 in 3f3babb

print(e) # e.g. "SIP call failed: 486 Busy Here (resource_exhausted)"

[lukasIO]

@ChelseaBradbury can you give more context on the change and why it is required? from the API's perspective this could/should be interpreted as a breaking change?

[ChelseaBradbury]

@ChelseaBradbury can you give more context on the change and why it is required? from the API's perspective this could/should be interpreted as a breaking change?

Basically ResourceExhausted isn't the right code for this, since it's likely to be interpreted as an infrastructure capacity issue rather than an expected potential call outcome. We had some discussion back and forth and we're pretty confident this isn't likely to break anything, so we decided to move forward.

[ChelseaBradbury]

non-blocking -> docs update