Skip to content

Conversation

@bbaarriiss
Copy link


PR Submission Checklist for internal contributors

  • The PR Title

    • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
    • contains a reference JIRA issue number like SQPIT-764
    • answers the question: If merged, this PR will: ... ³
  • The PR Description

    • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

Issues

We need usage stats for Remind App. So that we can know which request is made how many times.

Solutions

Integrate Prometheus metrics and add some basic usage stats metrics for some commands.

How to Test

Locally -> Run the application and send request(s) on staging via curl. Then verify the metrics with their counts from http://localhost:8080/q/metrics

Stating -> After the deployment on staging, we should see the metrics on Grafana if all the K8s related additions are correct.

Attachments (Optional)

Here is the sample output can be seen from http://localhost:8080/q/metrics
ℹ️ 'q' is the default context root for Quarkus applications

image

@bbaarriiss bbaarriiss self-assigned this Dec 9, 2025
@bbaarriiss bbaarriiss requested a review from a team as a code owner December 9, 2025 16:20
@bbaarriiss bbaarriiss closed this Dec 9, 2025
@bbaarriiss bbaarriiss reopened this Dec 9, 2025
Copy link

@spoonman01 spoonman01 left a comment

Choose a reason for hiding this comment

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

LGTM. Passing the UsageMetrics component into the ReminderEventHandler looks pretty simple

@bbaarriiss bbaarriiss merged commit 5e5317b into main Dec 10, 2025
4 checks passed
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