Skip to content
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

Add default values to proxy-*-connect-timeout annotations docs #12155

Merged
merged 2 commits into from
Mar 14, 2024
Merged

Add default values to proxy-*-connect-timeout annotations docs #12155

merged 2 commits into from
Mar 14, 2024

Conversation

doubletooth
Copy link
Contributor

Finding these values in the docs site is pretty painful

Update the documentation to follow other proxy fields in providing a default value at the end of the description for the field

Fixes #12154

Finding these values in the docs site is pretty painful

Update the documentation to follow other proxy fields in providing
a default value at the end of the description for the field

Fixes #12154

Signed-off-by: Akshay Dongaonkar <[email protected]>
@doubletooth doubletooth requested a review from a team as a code owner February 27, 2024 02:10
Copy link
Member

@alpeb alpeb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is useful, thanks!
I think however that the default for inbound connect timeout is 100ms (see charts/linkerd-control-plane/values.yaml. Also can you use the same unit (ms) for the outbound connect timeout?

cli/cmd/doc.go Outdated Show resolved Hide resolved
cli/cmd/doc.go Outdated Show resolved Hide resolved
@alpeb alpeb changed the title Add default values to proxy connect description Add default values to proxy-*-connect-timeout annotations docs Mar 14, 2024
@alpeb alpeb merged commit e920389 into linkerd:main Mar 14, 2024
34 checks passed
@doubletooth doubletooth deleted the akd-update-docs branch March 14, 2024 22:49
@doubletooth
Copy link
Contributor Author

Hey @alpeb thanks for applying the suggestion and merging in the submission! I really appreciate it. Sorry I couldn't respond sooner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add default connection timeout values to proxy configuration documentation.
2 participants