-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[SIMI-271] Supabase Cloud: Dashboard, logs, and monitors #22339
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
base: master
Are you sure you want to change the base?
Conversation
|
This PR does not modify any files shipped with the agent. To help streamline the release process, please consider adding the |
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.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e8fbfee510
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
supabase_cloud/assets/monitors/supabase_postgres_server_high_load.json
Outdated
Show resolved
Hide resolved
e8fbfee to
966bb6a
Compare
|
Opened DOCS-13130 to assign a Docs writer and follow up with editorial review. |
2776a8c to
cccd5d4
Compare
cccd5d4 to
aecb189
Compare
b1c6265 to
3c13b24
Compare
3c13b24 to
176bb97
Compare
e37d98a to
176bb97
Compare
evazorro
left a comment
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.
Left some feedback on monitor titles and descriptions. Let me know if you have any questions!
| "version": 2, | ||
| "created_at": "2026-01-08", | ||
| "last_updated_at": "2026-01-08", | ||
| "title": "Postgres Server Filesystem Size Exceeds Alert Threshold", |
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.
Not feedback on this specific PR, but related to some other documentation I'm working on: are there guidelines for monitor titles and descriptions written out somewhere that you follow? In the official docs, on the wiki, somewhere else?
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.
Hi @evazorro Thanks for calling this out. I have been loosely referencing existing integrations, and in this case referenced one with non-sentence casing in the titles. I tend to elaborate in the description but a couple of these seem so self-explanatory I failed to come up with anything useful. Will look again.
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.
Makes sense, thank you! I'm adding some guidelines to the integration developer docs and was curious if there are other docs elsewhere I should reference or combine.
supabase_cloud/assets/monitors/supabase_postgres_filesystem_size.json
Outdated
Show resolved
Hide resolved
supabase_cloud/assets/monitors/supabase_postgres_server_high_load.json
Outdated
Show resolved
Hide resolved
supabase_cloud/assets/monitors/supabase_postgres_server_high_load.json
Outdated
Show resolved
Hide resolved
| "created_at": "2026-01-08", | ||
| "last_updated_at": "2026-01-08", | ||
| "title": "Postgres Server Filesystem Size Exceeds Alert Threshold", | ||
| "description": "The Postgres server filesystem usage is above the alert threshold.", |
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.
The description should ideally not just restate the title; is there additional detail you can add here?
Example of a good monitor description (for title "Unacknowledged messages are higher than usual"):
Unacked messages are those that have been delivered to a consumer but have not been acknowledged as processed or handled. This monitor tracks the ratio of unacked messages to avoid potential bottlenecks which could lead to delays in message processing.
supabase_cloud/assets/monitors/supabase_postgres_server_offline.json
Outdated
Show resolved
Hide resolved
| "created_at": "2026-01-08", | ||
| "last_updated_at": "2026-01-08", | ||
| "title": "Postgres Server is Offline", | ||
| "description": "The Postgres server is offline.", |
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.
Same comment on description here too
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.
@evazorro Just updated three of the 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.
These look great, thank you so much for updating!
Co-authored-by: Eva Parish <eva.parish@datadoghq.com>
New integration, Supabase Cloud, assets for initial release.
Review checklist (to be filled by reviewers)
qa/skip-qalabel if the PR doesn't need to be tested during QA.backport/<branch-name>label to the PR and it will automatically open a backport PR once this one is merged