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.

Sign up for GitHub

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

Jump to bottom

WIP: Prometheus metrics and endpoint #9

Draft
wants to merge 7 commits into
base: develop
Choose a base branch
from
Draft

WIP: Prometheus metrics and endpoint #9

wants to merge 7 commits into from

Conversation

HazardDede
Copy link
Owner

@HazardDede HazardDede commented Oct 24, 2019 •
edited

Ideas for metrics:

  • Generated events by a pull
  • Failed polls
  • Execution time for poll / cron
  • Events processed by push
  • Failed pushes
  • Execution time for push (sync / async)
  • Events processed by udf
  • Failed udfs
  • Execution time for udf
  • Poll restarts

Other stuff:

  • Start metrics server with console argument (--metrics=8080)
  • Start metrics server with environment variable (for docker)
  • Add type of the component as a label

Bugs:

  • Calling the async method from sync (or vice versa) will track the calling twice

@HazardDede HazardDede added the enhancement New feature or request label Oct 24, 2019
@HazardDede HazardDede self-assigned this Oct 24, 2019
@HazardDede HazardDede changed the base branch from master to develop January 25, 2020 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@HazardDede HazardDede

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@HazardDede