[PR] Better naming for timeouts + connection timeout

[kopf-archiver]

A pull request by nolar at 2020-03-27 21:48:24+00:00
Original URL: zalando-incubator/kopf#337
Merged by nolar at 2020-03-29 23:28:50+00:00

What do these changes do?

Follow-ups for #336 with better naming of the timeout settings.

Description

#336 was merged but not released yet, so it is fine to quickly change the names without backward compatibility implemented.

In addition, a connection timeout is added for watching streams — to distinguish it from the total timeout on the client side (which is supposed to be infinite in a perfect world). The connection timeout is needed to handle the cases of broken networking in some cluster setups — for which kopf.config.WatchersConfig.session_timeout was originally introduced in #322 .

Issues/PRs

Issues: #23

Related: #322 #192 #109

Type of changes

• New feature (non-breaking change which adds functionality)
• Mostly documentation and examples (no code changes)

Checklist

• The code addresses only the mentioned problem, and this problem only
• I think the code is well written
• Unit tests for the changes exist
• Documentation reflects the changes
• If you provide code modification, please add yourself to CONTRIBUTORS.txt