Skip to content
This repository has been archived by the owner on Aug 28, 2023. It is now read-only.

Spike: Metrics/Monitoring #5

Open
arein opened this issue Jun 29, 2022 · 5 comments
Open

Spike: Metrics/Monitoring #5

arein opened this issue Jun 29, 2022 · 5 comments
Assignees
Labels
enhancement New feature or request gotrue Related to GoTrue help wanted Extra attention is needed ops Terraform

Comments

@arein
Copy link
Member

arein commented Jun 29, 2022

AC:

  • export metrics somewhere (I remember @Harry saying that GoTrue is integrated w/Datadog)
  • integrate with our Grafana instance
  • define top-level metrics
  • setup dashboards
  • setup monitors
@HarryET
Copy link
Contributor

HarryET commented Jun 30, 2022

Metrics can be done, and GoTrue uses OpenTracing. So I could edit our fork to add Prometheus support but that then starts to become a hard fork and that is not what we want to do. I will research today the possibility of using Datadog and check if we already have it...

@HarryET HarryET self-assigned this Jun 30, 2022
@HarryET HarryET added enhancement New feature or request help wanted Extra attention is needed ops Terraform gotrue Related to GoTrue labels Jun 30, 2022
@HarryET
Copy link
Contributor

HarryET commented Jun 30, 2022

@arein, what metrics were we looking to get from this?

  • Requests count
  • 2xx vs 4xx vs 5xx

@arein
Copy link
Member Author

arein commented Jun 30, 2022

Yup and latency. Ideally we have two top-level metrics: availability (derived from 2xx/5xx) and latency.
As discussed offline these can be gathered from other sources e.g. the load balancer

@HarryET
Copy link
Contributor

HarryET commented Jul 18, 2022

Open Telem is now working with a sidecar container the IAM permissions just need to be looked at for the execution role as I'm not sure it has permissions to create new X-Ray/Cloudwatch traces

@arein
Copy link
Member Author

arein commented Jul 23, 2022

https://github.com/WalletConnect/rs-relay/pull/156/files is adding that access (not really following least-privilege in that example tho)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request gotrue Related to GoTrue help wanted Extra attention is needed ops Terraform
Projects
None yet
Development

No branches or pull requests

2 participants