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
Add more metrics #6447
Add more metrics #6447
Conversation
fantix
commented
Nov 8, 2023
•
edited
edited
- Current number of databases.
- Time a client connection is open.
- Number of compiled/cached GraphQL queries.
- Number of SQL queries since instance startup.
- Number of SQL compilations since instance startup.
- Time it takes to compile a query or script.
- Queries per connection (EdgeQL / SQL only).
- Number of transaction serialization errors.
- Number of network errors.
- Number of Auth API calls.
- Number of Auth UI form rendered.
- Number of Auth successful logins.
- Number of OAuth providers configured.
- Average size of queries.
Should we consider putting some of this stuff behind authentication? |
Good point! Or perhaps we can serve the /metrics endpoint only on specified host address (like localhost) or from certain client IP |
a13416c
to
cafdfa3
Compare
edb/server/metrics.py
Outdated
auth_providers = registry.new_labeled_gauge( | ||
'auth_providers', | ||
'Number of Auth providers configured.', | ||
labels=('tenant', 'database'), |
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.
Shall we add database
to many other metrics?
(Removing backport-4.x since 4.x doesn't have the auth stuff we need) |
* branches_current * graphql_query_compilations_total * sql_queries_total and sql_compilations_total * query_compilation_duration * client_connection_duration * queries_per_connection * transaction_serialization_errors_total * connection_errors_total * query_size * auth_api_calls_total and auth_api_calls_total * auth_providers * auth_successful_logins_total