Skip to content

docs: update the defaults for request limits#17437

Merged
jmurret merged 5 commits into
mainfrom
jm/update-rl-defaults
May 25, 2023
Merged

docs: update the defaults for request limits#17437
jmurret merged 5 commits into
mainfrom
jm/update-rl-defaults

Conversation

@jmurret
Copy link
Copy Markdown
Member

@jmurret jmurret commented May 23, 2023

Description

Changing the documentation for the default values for read_rate and write_rate in request_limits to reflect what is in code.

Testing & Reproduction steps

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

@jmurret jmurret added pr/no-changelog PR does not need a corresponding .changelog entry backport/1.15 labels May 23, 2023
@jmurret jmurret requested a review from a team as a code owner May 23, 2023 19:23
@github-actions github-actions Bot added the type/docs Documentation needs to be created/updated/clarified label May 23, 2023
@jmurret jmurret mentioned this pull request May 23, 2023
Merged
2 tasks
@jmurret jmurret requested a review from curtbushko May 23, 2023 19:30
@jmurret jmurret enabled auto-merge (squash) May 25, 2023 14:46
@jmurret jmurret merged commit 127eba6 into main May 25, 2023
@jmurret jmurret deleted the jm/update-rl-defaults branch May 25, 2023 15:00
@hc-github-team-consul-core hc-github-team-consul-core mentioned this pull request May 25, 2023
Merged
4 tasks
jmurret added a commit that referenced this pull request Jun 12, 2023
@jmurret
docs: update the defaults for request limits (#17437)
9b7b51a
jmurret added a commit that referenced this pull request Jun 12, 2023
@hc-github-team-consul-core @jmurret
Backport of docs: update the defaults for request limits into release…
7792c27 
…/1.15.x (#17466)

* no-op commit due to failed cherry-picking

* docs: update the defaults for request limits (#17437)

---------

Co-authored-by: temp <temp@hashicorp.com>
Co-authored-by: John Murret <john.murret@hashicorp.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@curtbushko curtbushko curtbushko approved these changes

Assignees

No one assigned

Labels

pr/no-changelog PR does not need a corresponding .changelog entry 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

@jmurret @curtbushko