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

Service level objectives support on @Timed annotation #5145

Merged

Conversation

lcavadas
Copy link
Contributor

@lcavadas lcavadas commented May 25, 2024

Adding the ability to specify, on @Timed annotations, the service level objectives.
Using seconds as the unit of time for these values.

Copy link
Member

@shakuzen shakuzen left a comment

Choose a reason for hiding this comment

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

Thanks for the pull request. Would you please add test cases showing the Timer created from the TimedAspect and Timer builder method have the SLOs given in the annotation?

@shakuzen
Copy link
Member

shakuzen commented Jul 4, 2024

Could you also look into the TimedAspectTest failures?

@lcavadas
Copy link
Contributor Author

lcavadas commented Jul 5, 2024

@shakuzen Thanks for taking a look, will do as soon as I can

@shakuzen shakuzen added this to the 1.14.x milestone Jul 7, 2024
@lcavadas lcavadas force-pushed the timed-annotation-slos-support branch from 9b5ffa5 to 1485ba1 Compare July 8, 2024 09:57
@lcavadas lcavadas requested a review from shakuzen July 8, 2024 11:45
@shakuzen shakuzen changed the title Adding service level objectives support to the @Timed annotation Service level objectives support on @Timed annotation Jul 9, 2024
@shakuzen shakuzen modified the milestones: 1.14.x, 1.14.0-M1 Jul 9, 2024
@shakuzen shakuzen merged commit 1061499 into micrometer-metrics:main Jul 9, 2024
6 checks passed
@lcavadas lcavadas deleted the timed-annotation-slos-support branch July 9, 2024 10:43
izeye added a commit to izeye/micrometer that referenced this pull request Sep 11, 2024
izeye added a commit to izeye/micrometer that referenced this pull request Sep 11, 2024
@izeye izeye mentioned this pull request Sep 11, 2024
izeye added a commit to izeye/micrometer that referenced this pull request Sep 11, 2024
izeye added a commit to izeye/micrometer that referenced this pull request Sep 11, 2024
@izeye
Copy link
Contributor

izeye commented Sep 11, 2024

The "enhancement" label seems to be missing for this, and an entry for this is also missing from the release notes for 1.14.0-M1.

@jonatan-ivanov jonatan-ivanov added the enhancement A general enhancement label Sep 11, 2024
@jonatan-ivanov
Copy link
Member

Thank you, I fixed them.

jonatan-ivanov pushed a commit that referenced this pull request Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants