Add builder header timeout flag

[eserilev]

Issue Addressed

#4709

Proposed Changes

Add a new flag builder-header-timeout which allows for configuring timeouts for the get builder header api call. Ive added a max limit of 3 seconds to the timeout to prevent potential liveness issues.

[eserilev]

Do we want to force minimum and maximum values for this flag? I don't see why anyone would want this set to less than one second, or to more than 2-3 seconds

[dapplion]

Do we want to force minimum and maximum values for this flag? I don't see why anyone would want this set to less than one second, or to more than 2-3 seconds

If you set a very low value you will prioritize local building, which you can do today already.
If you set a very high value you will never consider local building and potentially cause liveness issues if builders misbehave. In extreme cases the failover should kick in switching to local building if the failures are common.

I have a soft opinion towards not specifying neither a minimum nor a maximum

[jimmygchen]

Looks good - I don't have anything else to add!

[michaelsproul]

Looks good modulo capitalisation fix

[michaelsproul]

@Mergifyio queue

[mergify]

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at fbc230e