You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
fix(ci): surface gcloud output on read-replica Data API limit errors - #4006
Release capgo-22.28.1 (run 41068982301) failed in Reconcile production read-replica schema with "Cloud SQL Data API rejected the response at its 10 MB response limit". That message comes from a broad regex over gcloud stderr and drops the real output, so the cause is hidden. That job gates supabase_deploy, deploy_api, deploy_files, deploy_plugin_regions and deploy_webapp, so nothing deployed.
This keeps the classified message and appends the trimmed, 4 KB capped gcloud output so the next run shows the real root cause. No workflow changes.
Need help on this PR? Tag @codesmith-bot with what you need.
Summary by CodeRabbit
Bug Fixes
Errors triggered by request or response limits now include normalized cloud command output, providing additional diagnostic context. Limit detection and fallback error behavior remain unchanged, so this update affects the information shown in these errors without changing how limit conditions are handled.
Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.
Walkthrough
The request-limit and response-limit errors in dataApiCommandError now include normalized gcloud output. Limit detection and fallback behavior remain unchanged.
Changes: Data API error diagnostics
Layer / File(s)
Summary
Include output in limit errors scripts/sync-read-replica-schema.ts
The request-limit and response-limit errors now include normalized gcloud output. Limit detection and fallback behavior remain unchanged.
Cap-go/capgo.app#3999: Changes dataApiCommandError in the same script, including request-limit and response-limit classifications.
Cap-go/capgo.app#4003: Adds subscription and publication checks that use the same Cloud SQL Data API execution path.
Suggested reviewers:riderx
Merge Risk: Minimal · up to 75512
The change improves failure diagnostics without a reported change to limit detection or fallback behavior. No merge-blocking issue is identified.
Pre-merge checks: 3 · 2
Failed checks: 2 warnings
Check name
Status
Explanation
Resolution
Description check
The description explains the failure and intended fix, but it does not follow the required template. It lacks the Summary heading, Test plan, Screenshots section, and Checklist responses.
Add the required template sections. Include a Test plan with validation steps, complete the Checklist, and state whether screenshots are applicable.
Docstring Coverage
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed checks: 3 passed
Check name
Status
Explanation
Title check
The title clearly and concisely describes the main change: exposing gcloud output for read-replica Data API limit errors.
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage
Explanation
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 1 functions across 1 files.
Resolution
Write docstrings for the functions missing them to satisfy the coverage threshold.
Finishing Touches: 💡 1
📝 Generate docstrings: 💡
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Warning
Some tools did not complete. Review the errors below.
ESLint
If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.
ESLint install timed out. The project may have too many dependencies for the sandbox.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Already reviewed the last commit. Use @coderabbitai full review to rerun a review of the entire changeset.
Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.
Superseded by #4007, which keeps the gcloud diagnostics from this PR and adds tighter Data API limit matching plus deploy workflow unblock so read_replica_schema no longer gates production deploys.
Note: this PR is already merged on main; #4007 rebases on that and carries the remaining workflow fix.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release capgo-22.28.1 (run 41068982301) failed in
Reconcile production read-replica schemawith "Cloud SQL Data API rejected the response at its 10 MB response limit". That message comes from a broad regex over gcloud stderr and drops the real output, so the cause is hidden. That job gates supabase_deploy, deploy_api, deploy_files, deploy_plugin_regions and deploy_webapp, so nothing deployed.This keeps the classified message and appends the trimmed, 4 KB capped gcloud output so the next run shows the real root cause. No workflow changes.
Need help on this PR? Tag
@codesmith-botwith what you need.Summary by CodeRabbit