-
Notifications
You must be signed in to change notification settings - Fork 299
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Watchers time out after 100 seconds #165
Comments
I'm not entirely sure, but this appears to be an issue when the server does not send any content at all in the first 100 seconds after the GET request is issued. When the server does send a first response, and subsequent updates come in intervals greater than 100 seconds, the code appears to work fine. |
@qmfrederik is there an action we can take here? Or is this to document the issue for people who encounter it? |
We could implement the approach described here, which allows us to set a timeout on a per-request level, but that would require updating the generator, so it's not a trivial fix. Alternatively the watchers can catch timeout exceptions and try again. I encountered it and for now have try/catch in my calling code. I'm not very happy with that and may want to revisit it in the future, and this issue mainly acts as a reminder to myself/documentation to others 😄 . |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
@fejta-bot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
HttpClient
is 100 seconds, so the watchers will timeout after this amount of time.The text was updated successfully, but these errors were encountered: