Skip to content

docs(lore-0063): re-diff BE RBAC + draft BE-repo PR#50

Merged
karczuRF merged 2 commits into
developfrom
docs/0063_be-rbac-rediff-pr-draft
Jun 22, 2026
Merged

docs(lore-0063): re-diff BE RBAC + draft BE-repo PR#50
karczuRF merged 2 commits into
developfrom
docs/0063_be-rbac-rediff-pr-draft

Conversation

@karczuRF

Copy link
Copy Markdown
Collaborator

Summary

  • Re-diffed the prices tenant RBAC additions against soroban-block-explorer develop @ 8e4e705d; corrected the drafted services.xml (XML <grants> take no TO <user> clause; added <access_management>0</access_management> to match BE's 5 live users).
  • Confirmed quotas.xml caps are an exact match to BE's live high_write / api_throttle, and profiles.xml needs no change (write_no_ddl + read_only both already in use).
  • Locked two decisions into the runbook: dedicated quota naming prices_write / prices_read (open item 4) and single-CN with no env suffix prices-ingestion / prices-api (open item 5).
  • Added a paste-ready BE-repo PR description (G-be-rbac-pr-description.md) with a why-this-PR context section for BE reviewers / AI agents.
  • Doc-only; no code changes. Remaining open item: pre-PR check that the box CH version applies user-XML <grants> at startup (else init.sql GRANT fallback).

karczuRF added 2 commits June 22, 2026 12:05
Re-diffed the prices tenant additions against soroban-block-explorer
develop @ 8e4e705d. Corrected the drafted services.xml: XML <grants>
take no "TO <user>" clause, and added <access_management>0</access_management>
to match BE's five live users. Confirmed quotas.xml caps are an exact
match to the live high_write / api_throttle values, and that profiles.xml
needs no change (write_no_ddl + read_only both in use).

Locked two decisions into the runbook: dedicated quota naming
(prices_write / prices_read, open item 4) and single-CN with no env
suffix (prices-ingestion / prices-api, open item 5). Remaining open
item 6 is the pre-PR check that the box CH version applies user-XML
<grants> at startup, else fall back to init.sql GRANTs.

Added a paste-ready BE-repo PR description (G-be-rbac-pr-description),
including a why-this-PR context section for BE reviewers / AI agents.
Last check before applying prices.* to Hetzner. Verified close_usd
(task 0061) is wired end-to-end: defined in init.sql (pos 12, all 7
grains via idempotent ALTERs), propagated by argMax through the full
rollups.sql MV chain and preroll.sql, and read by the price_usd_series
views — which the init binary applies on every run. Confirmed MV
positional matching is correct and the views.sql USDC issuer literal is
a byte-exact match to prices_clickhouse::USDC_ISSUER.

Verdict GREEN: schema clear to deploy. Caveat recorded: close_usd is
DEFAULT 0 until enrichment fills it, so price_usd_series is empty
post-apply until ingestion populates it (0026/0038/backfill).
@karczuRF karczuRF merged commit 15d0f91 into develop Jun 22, 2026
3 checks passed
@karczuRF karczuRF deleted the docs/0063_be-rbac-rediff-pr-draft branch June 22, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant