Remove .no_timeout as a timeout option from GitalyClient
Problem to solve
no_timeout currently will become the default timeout for a Gitaly call from a Sidekiq worker if no
timeout: argument was passed to
Gitaly.no_timeoutwith a very high timeout value. All cases of
Gitaly.no_timeoutin the app should use this new high timeout instead. This new high timeout value will be called
Up for discussion
Note: These were the initial set of questions, however, please read the discussions in this Issue as some answers are being formed there.
- Should the default timeout for Sidekiq jobs become the same as for non-Sidekiq calls? Or,
- Should there be a new user-configurable default timeout created for Sidekiq jobs? (a
- Should the default timeout for Sidekiq jobs become
slow_timeoutbe set to 24 hours?
To be updated: