Skip to content
This repository has been archived by the owner on Nov 29, 2024. It is now read-only.

fix: adjust default producer/end-to-end latency buckets. #194

Merged
merged 1 commit into from
Jun 29, 2022

Conversation

k-wall
Copy link
Contributor

@k-wall k-wall commented Jun 26, 2022

following #188, the default producer/end-to-end histogram buckets used by
ihe canary are no longer ideal with all of the obseverations falling into
the <=100ms bucket. This change introduces smaller buckets and drops the
largest ones.

Signed-off-by: kwall kwall@apache.org

@k-wall k-wall force-pushed the adjust-bucket branch 3 times, most recently from be67d56 to d676e8d Compare June 27, 2022 11:08
@ppatierno ppatierno requested a review from a team June 29, 2022 08:17
@ppatierno ppatierno added this to the 0.4.0 milestone Jun 29, 2022
Copy link
Member

@ppatierno ppatierno left a comment

Choose a reason for hiding this comment

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

LGTM. @k-wall can you resolve the conflict please?

@ppatierno ppatierno requested a review from a team June 29, 2022 08:19
following strimzi#188, the default producer/end-to-end histogram buckets used by
ihe canary are no longer ideal with all of the obseverations falling into
the first (<=100ms) bucket.  This change introduces several smaller buckets
and drops the largest ones.

Signed-off-by: kwall <kwall@apache.org>
@ppatierno ppatierno merged commit d459874 into strimzi:main Jun 29, 2022
melchiormoulin pushed a commit to melchiormoulin/strimzi-canary that referenced this pull request Jun 29, 2022
following strimzi#188, the default producer/end-to-end histogram buckets used by
ihe canary are no longer ideal with all of the obseverations falling into
the first (<=100ms) bucket.  This change introduces several smaller buckets
and drops the largest ones.

Signed-off-by: kwall <kwall@apache.org>
Signed-off-by: Melchior Moulin <melchior.moulin@gmail.com>
melchiormoulin pushed a commit to melchiormoulin/strimzi-canary that referenced this pull request Jul 3, 2022
following strimzi#188, the default producer/end-to-end histogram buckets used by
ihe canary are no longer ideal with all of the obseverations falling into
the first (<=100ms) bucket.  This change introduces several smaller buckets
and drops the largest ones.

Signed-off-by: kwall <kwall@apache.org>
Signed-off-by: Melchior Moulin <melchior.moulin@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants