-
Notifications
You must be signed in to change notification settings - Fork 240
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
feat(apps/price_pusher): add prom metrics #2491
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
e904ed0
to
e55be6d
Compare
… Grafana dashboard descriptions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the code now. The only request i have is to respect the design structure of the code. I recommend you add the balance tracker interface to the top interfaces.ts and later implement the one for evm in the evm directory.
… into a single location
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! can you bump the version as well?
Summary
Added comprehensive monitoring capabilities to the price_pusher service, including:
Rationale
These changes are necessary to improve the observability of the price_pusher service, enabling operators to:
How has this been tested?
The monitoring functionality has been manually verified by: