{"action":"created","issue":{"url":"https://api.github.com/repos/envoyproxy/envoy/issues/4903","repository_url":"https://api.github.com/repos/envoyproxy/envoy","labels_url":"https://api.github.com/repos/envoyproxy/envoy/issues/4903/labels{/name}","comments_url":"https://api.github.com/repos/envoyproxy/envoy/issues/4903/comments","events_url":"https://api.github.com/repos/envoyproxy/envoy/issues/4903/events","html_url":"https://github.com/envoyproxy/envoy/issues/4903","id":375244569,"node_id":"MDU6SXNzdWUzNzUyNDQ1Njk=","number":4903,"title":"Exclude connection setup in per try timeout","user":{"login":"snowp","id":2023366,"node_id":"MDQ6VXNlcjIwMjMzNjY=","avatar_url":"https://avatars3.githubusercontent.com/u/2023366?v=4","gravatar_id":"","url":"https://api.github.com/users/snowp","html_url":"https://github.com/snowp","followers_url":"https://api.github.com/users/snowp/followers","following_url":"https://api.github.com/users/snowp/following{/other_user}","gists_url":"https://api.github.com/users/snowp/gists{/gist_id}","starred_url":"https://api.github.com/users/snowp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/snowp/subscriptions","organizations_url":"https://api.github.com/users/snowp/orgs","repos_url":"https://api.github.com/users/snowp/repos","events_url":"https://api.github.com/users/snowp/events{/privacy}","received_events_url":"https://api.github.com/users/snowp/received_events","type":"User","site_admin":false},"labels":[{"id":421403909,"node_id":"MDU6TGFiZWw0MjE0MDM5MDk=","url":"https://api.github.com/repos/envoyproxy/envoy/labels/enhancement","name":"enhancement","color":"84b6eb","default":true},{"id":645516726,"node_id":"MDU6TGFiZWw2NDU1MTY3MjY=","url":"https://api.github.com/repos/envoyproxy/envoy/labels/help%20wanted","name":"help wanted","color":"fbca04","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/envoyproxy/envoy/milestones/8","html_url":"https://github.com/envoyproxy/envoy/milestone/8","labels_url":"https://api.github.com/repos/envoyproxy/envoy/milestones/8/labels","id":3676682,"node_id":"MDk6TWlsZXN0b25lMzY3NjY4Mg==","number":8,"title":"1.9.0","description":"","creator":{"login":"mattklein123","id":6305260,"node_id":"MDQ6VXNlcjYzMDUyNjA=","avatar_url":"https://avatars3.githubusercontent.com/u/6305260?v=4","gravatar_id":"","url":"https://api.github.com/users/mattklein123","html_url":"https://github.com/mattklein123","followers_url":"https://api.github.com/users/mattklein123/followers","following_url":"https://api.github.com/users/mattklein123/following{/other_user}","gists_url":"https://api.github.com/users/mattklein123/gists{/gist_id}","starred_url":"https://api.github.com/users/mattklein123/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattklein123/subscriptions","organizations_url":"https://api.github.com/users/mattklein123/orgs","repos_url":"https://api.github.com/users/mattklein123/repos","events_url":"https://api.github.com/users/mattklein123/events{/privacy}","received_events_url":"https://api.github.com/users/mattklein123/received_events","type":"User","site_admin":false},"open_issues":36,"closed_issues":26,"state":"open","created_at":"2018-09-21T05:26:05Z","updated_at":"2018-10-29T23:48:35Z","due_on":null,"closed_at":null},"comments":8,"created_at":"2018-10-29T22:32:54Z","updated_at":"2018-10-30T00:00:20Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"We've been seeing per try timeouts trigger needlessly during boot due to per try timeout being less than the connection setup (including TLS). To get around this, we need to increase the per try timeout to the point where it becomes meaningless for some of our fast endpoints. \r\n\r\nIt would be nice to be able to specify the per try timeout for the request/response itself, not including the connection setup."},"comment":{"url":"https://api.github.com/repos/envoyproxy/envoy/issues/comments/434124032","html_url":"https://github.com/envoyproxy/envoy/issues/4903#issuecomment-434124032","issue_url":"https://api.github.com/repos/envoyproxy/envoy/issues/4903","id":434124032,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNDEyNDAzMg==","user":{"login":"mattklein123","id":6305260,"node_id":"MDQ6VXNlcjYzMDUyNjA=","avatar_url":"https://avatars3.githubusercontent.com/u/6305260?v=4","gravatar_id":"","url":"https://api.github.com/users/mattklein123","html_url":"https://github.com/mattklein123","followers_url":"https://api.github.com/users/mattklein123/followers","following_url":"https://api.github.com/users/mattklein123/following{/other_user}","gists_url":"https://api.github.com/users/mattklein123/gists{/gist_id}","starred_url":"https://api.github.com/users/mattklein123/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattklein123/subscriptions","organizations_url":"https://api.github.com/users/mattklein123/orgs","repos_url":"https://api.github.com/users/mattklein123/repos","events_url":"https://api.github.com/users/mattklein123/events{/privacy}","received_events_url":"https://api.github.com/users/mattklein123/received_events","type":"User","site_admin":false},"created_at":"2018-10-30T00:00:20Z","updated_at":"2018-10-30T00:00:20Z","author_association":"MEMBER","body":"I'm OK with just modifying the existing behavior (and release noting it) since I think what you are proposing makes more sense for the intention of the timeout, as long as the outer timeout continues to cover the entire thing. @envoyproxy/maintainers any opinions here?"}}
{"actor":{"display_login":"mattklein123"}}