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 exponential buckets for the Event publisher proxy latency histogram #15643

Merged

Conversation

marcobebway
Copy link
Contributor

@marcobebway marcobebway commented Sep 26, 2022

Description

Add exponential buckets for the Event publisher proxy latency histogram.

Related issue(s)

@kyma-bot kyma-bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 26, 2022
@marcobebway marcobebway self-assigned this Sep 26, 2022
@marcobebway marcobebway added the area/eventing Issues or PRs related to eventing label Sep 26, 2022
@marcobebway
Copy link
Contributor Author

/retest

@marcobebway
Copy link
Contributor Author

/hold

@kyma-bot kyma-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 14, 2022
@marcobebway marcobebway force-pushed the 15473-update-epp-histogram-buckets branch from 89e5382 to 2f29da0 Compare October 17, 2022 07:09
@kyma-bot kyma-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 17, 2022
@marcobebway marcobebway force-pushed the 15473-update-epp-histogram-buckets branch from af38441 to 88215f4 Compare October 21, 2022 05:15
@marcobebway marcobebway requested a review from k15r October 21, 2022 05:41
@kyma-bot
Copy link
Contributor

kyma-bot commented Oct 21, 2022

@marcobebway: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pre-main-kyma-skr-eventing 88215f4 link false /test pre-main-kyma-skr-eventing

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@marcobebway
Copy link
Contributor Author

/retest-required

@kyma-bot kyma-bot added the lgtm Looks good to me! label Oct 21, 2022
@marcobebway
Copy link
Contributor Author

/hold cancel

@kyma-bot kyma-bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 21, 2022
@kyma-bot kyma-bot merged commit ea5e864 into kyma-project:main Oct 21, 2022
@marcobebway marcobebway deleted the 15473-update-epp-histogram-buckets branch October 21, 2022 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/eventing Issues or PRs related to eventing lgtm Looks good to me! size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bucket marks are set too small in histogram metric event_publish_to_messaging_server_latency_bucket
3 participants