Repository navigation
Precompute dashboard metrics and tune production workload - #1
Merged
Merged
Conversation
Only QueryMetricsSnapshotWarming had a handler, so the four remaining domain exceptions raised by the repository escaped as unhandled 500s. Since sql/017 ships the matviews WITH NO DATA, this was the normal cold-start path on every fresh deploy. Backoff exceptions now return 503 with Retry-After; capacity exceptions return 503 without it, because retrying cannot fix an exceeded limit. The blanket `except Exception` in both refresh paths also caught the warming signal and armed the failure backoff, escalating an unpopulated matview into a refresh backoff. Warming is now re-raised ahead of it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The generated admin token was 44 base64 chars with no prefix, but
security.py fullmatches ^adv_pat_v1_[A-Za-z0-9_-]{43}$ before comparing
hashes, so every bootstrapped deployment had unusable admin credentials
and only the SHA-256 was kept.
Also removes the hardcoded deploy host, ACME email and basic-auth user
in favour of DASHBOARD_DOMAIN/ACME_EMAIL/DASHBOARD_BASIC_AUTH_USER,
matching the placeholder style the Caddyfile already used; drops the ERP
workload defaults and the workload restart override that would have run
a 500-table load generator against a production source; sets
REGISTER_DEMO_SOURCE=false per the installation guide; and validates the
credentials directory before .env is written, since failing afterwards
left an .env the guard refuses to regenerate.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Drops two tracked PDFs (600 KB), the PDF review whose checks are already automated in scripts/verify.sh, and the dated iteration gap analysis. The one permanent fact in the gap analysis -- which fields PoWA 5.2 statement history carries and which it does not -- moves to the known limits section of docs/ARCHITECTURE.md. erp-load-acceptance.yml targets a self-hosted runner label that has no registered runner; its last six scheduled runs were all cancelled on the 24h queue timeout without executing a step. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Validation
NO_IMPROVEMENT; persisted HypoPG validation passed and no unsafe DDL or false recommendation was produced.