Skip to content

Adjust metric description.#484

Merged
grampelberg merged 1 commit intolinkerd:masterfrom
naseemkullah:retry-metric-description
Apr 17, 2020
Merged

Adjust metric description.#484
grampelberg merged 1 commit intolinkerd:masterfrom
naseemkullah:retry-metric-description

Conversation

@naseemkullah
Copy link
Contributor

The retryable HTTP responses that were not retries are only those with the skipped: no_budget label. Actual retries that occured fall under this metric category.

Signed-off-by: Naseem [email protected]

The retryable HTTP responses that were not retries are only those with the skipped: no_budget label. Actual retries that occured fall under this metric category.

Signed-off-by: Naseem <[email protected]>
@olix0r olix0r requested a review from a team April 17, 2020 20:26
Copy link
Member

@olix0r olix0r left a comment

Choose a reason for hiding this comment

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

Thanks!

@grampelberg grampelberg merged commit b64e4eb into linkerd:master Apr 17, 2020
@naseemkullah naseemkullah deleted the retry-metric-description branch April 17, 2020 21:37
olix0r added a commit to linkerd/linkerd2 that referenced this pull request Apr 22, 2020
This release introduces a per-endpoint authority-override feature. This
is driven by the destination controller and is needed to support
mutli-cluster gateways.

---

* Update to Rust 1.42.0 (linkerd/linkerd2-proxy#483)
* Adjust metric description. (linkerd/linkerd2-proxy#484)
* Use authority override from metadata (linkerd/linkerd2-proxy#458)
olix0r added a commit to linkerd/linkerd2 that referenced this pull request Apr 22, 2020
This release introduces a per-endpoint authority-override feature. This
is driven by the destination controller and is needed to support
mutli-cluster gateways.

---

* Update to Rust 1.42.0 (linkerd/linkerd2-proxy#483)
* Adjust metric description. (linkerd/linkerd2-proxy#484)
* Use authority override from metadata (linkerd/linkerd2-proxy#458)
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.

3 participants