Repository navigation
Bump API schema to 2ae04130 - #20000
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Please update the IS YOUR CHANGE URGENT? section of the PR description:
This information helps the Docs team prioritize your review. |
|
The plan checks API documentation page loads, API-path not-found events, and client errors on those pages. An issue escalates when API page loads fall while other pages hold. It also escalates when API not-found events or API-page errors rise. Services: Mention PlanWhat changedThe build now fetches the public API OpenAPI document from git SHA 2ae04130. Category and endpoint pages under The schema commit at that SHA is SDK client contract work. It did not list RiskA bad or empty OpenAPI document omits Intended effectAPI documentation pages must keep loading after the new pin.
Regression watchA wrong schema can drop or rename
Not observableThe OpenAPI path and endpoint delta versus the last good production pin is not measured here. Pageload p95 for |
…ng (#20001) ## DESCRIBE YOUR PR The Bump API Schema SHA workflow got rate-limited by the GitHub API and wrote the error body (`{"message":"API rate limit exceeded for ..."}`) into `SENTRY_API_SCHEMA_SHA`. It then admin-merged the change ([#19999](#19999)). That broke `master` for about 10 minutes, until the next run ([#20000](#20000)) replaced it with a real SHA. The workflow called the commits API without authentication and never checked what came back. It now: - Sends the GitHub App token it already creates. That token has access to `sentry-api-schema`, so the request gets the much higher authenticated rate limit. - Uses `curl --fail`, so an HTTP error fails the job instead of returning the error body. - Fails the job unless the response is a 40-character hex SHA, so nothing else gets written into the file or merged. It also fixes `-sSl` to `-sSL`. Lowercase `-l` is curl's FTP `--list-only` flag; `-L` follows redirects, which is what was meant. ## IS YOUR CHANGE URGENT? - [x] No deadline: Not urgent, can wait up to 1 week+ ## PRE-MERGE CHECKLIST - [ ] Checked Vercel preview for correctness, including links - [ ] PR was reviewed and approved by any necessary SMEs (subject matter experts) - [ ] PR was reviewed and approved by a member of the [Sentry docs team](https://github.com/orgs/getsentry/teams/docs) <!-- junior-request-attribution:start --> via @sergical. <!-- junior-request-attribution:end --> <!-- junior-session-footer:start --> <!-- junior-conversation-id:slack%3AC8H0BQRDJ%3A1792070558.468179 --> -- [View Junior Session](https://junior-prod.sentry.dev/conversations/slack%3AC8H0BQRDJ%3A1792070558.468179) [[Sentry]](https://sentry.sentry.io/explore/conversations/slack%3AC8H0BQRDJ%3A1792070558.468179/?project=4510944073809921) <!-- junior-session-footer:end --> Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com> Co-authored-by: Sergiy Dybskiy <sergiy.dybskiy@sentry.io>
No description provided.