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

feat: Prometheus /metrics data exportation #42

Closed
42atomys opened this issue Mar 9, 2022 · 0 comments · Fixed by #45
Closed

feat: Prometheus /metrics data exportation #42

42atomys opened this issue Mar 9, 2022 · 0 comments · Fixed by #45
Assignees
@42atomys
Labels
aspect/dex 🤖 Concerns developers' experience with the codebase domain/obvious 🟩 Represents the "known knowns" issue. It's Obviously priority/medium 🟨 Priority 3 - Not blocking but should be fixed soon type/feature ⭐ Addition of new feature
Milestone
1.0

Comments

@42atomys
Copy link
Owner

42atomys commented Mar 9, 2022

Is your feature request related to a problem? Please describe.
No monitoring is actually available on Webhooked. To provide Webhooked production ready, we need a monitoring endpoint

Describe the solution you'd like
Actually, no data, logs or metrics is exposed about processing of webhook entries

Describe alternatives you've considered
I think create a Prometheus endpoints /metrics with prometheus go package to expose basic informations like req_counts, req_status, req_execution_time, webhook_entrypoint, webhook_kind, webhook_version, has_secure, storage_counts, storage_process_time, storage_kind, etc...
@42atomys 42atomys added aspect/dex 🤖 Concerns developers' experience with the codebase state/confirmed 💜 priority/low 🟩 Priority 4 - Low priority and doesn't need to be rushed type/feature ⭐ Addition of new feature domain/obvious 🟩 Represents the "known knowns" issue. It's Obviously labels Mar 9, 2022
@42atomys 42atomys added this to the 1.0 milestone Mar 9, 2022
@42atomys 42atomys moved this to Triage 🚦 in Stud42 V3 Mar 9, 2022
@42atomys 42atomys added priority/medium 🟨 Priority 3 - Not blocking but should be fixed soon and removed priority/low 🟩 Priority 4 - Low priority and doesn't need to be rushed labels Mar 9, 2022
@42atomys 42atomys changed the title feat: Prometheus /metrics daat exportation feat: Prometheus /metrics data exportation Mar 10, 2022
@42atomys 42atomys self-assigned this Mar 10, 2022
@42atomys 42atomys moved this from Triage 🚦 to Confirmed/To Do💜 in Stud42 V3 Mar 10, 2022
@42atomys 42atomys mentioned this issue Mar 12, 2022
Merged
7 tasks
Repository owner moved this from Confirmed/To Do💜 to Done 🎉 in Stud42 V3 Mar 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@42atomys 42atomys

Labels
aspect/dex 🤖 Concerns developers' experience with the codebase domain/obvious 🟩 Represents the "known knowns" issue. It's Obviously priority/medium 🟨 Priority 3 - Not blocking but should be fixed soon type/feature ⭐ Addition of new feature
Projects
Stud42 V3
Archived in project
Milestone
1.0
Development

Successfully merging a pull request may close this issue.

feat(logging): Add prometheus metrics & structure logs 42atomys/webhooked
1 participant
@42atomys