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.

Sign up for GitHub

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

Jump to bottom

docs/nia: Update CTS default transport options #9549

Merged
merged 1 commit into from
Jan 11, 2021
Merged

docs/nia: Update CTS default transport options #9549

merged 1 commit into from
Jan 11, 2021

Conversation

findkim
Copy link
Contributor

@findkim findkim commented Jan 11, 2021 •
edited
Loading

image

@findkim findkim added type/docs Documentation needs to be created/updated/clarified theme/consul-terraform-sync Relating to Consul Terraform Sync and Network Infrastructure Automation labels Jan 11, 2021
@findkim findkim requested a review from a team January 11, 2021 23:21
eikenb
eikenb approved these changes Jan 11, 2021
View reviewed changes
Copy link
Contributor

@eikenb eikenb left a comment

Choose a reason for hiding this comment

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

👍
Nice explanation of how this interacts with http_max_conns_per_client.
Should have noticed this in previous PR. Sorry.

@findkim findkim merged commit 41e0db1 into hashicorp:nia/docs-0.1.0-beta Jan 11, 2021
@findkim findkim deleted the docs-cts-transport-defaults branch January 11, 2021 23:27
findkim added a commit that referenced this pull request Jan 13, 2021
@findkim
docs/nia: Update CTS default transport options (#9549)
89ec971
findkim added a commit to findkim/consul that referenced this pull request Feb 23, 2021
@findkim
docs/nia: Update CTS default transport options (hashicorp#9549)
80f2843
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eikenb eikenb eikenb approved these changes

Assignees
No one assigned
Labels
theme/consul-terraform-sync Relating to Consul Terraform Sync and Network Infrastructure Automation type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@findkim @eikenb