Skip to content

Move Block staging hosts to block.builderlabstaging.xyz - #437

Open
matt2e wants to merge 2 commits into
mainfrom
jitter/block-staging-domain
Open

matt2e wants to merge 2 commits into
mainfrom
jitter/block-staging-domain

Conversation

@matt2e

@matt2e matt2e commented Oct 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

bb-block staging moved from test.blockstaging.build to block.builderlabstaging.xyz. The cutover finished on 2026-10-27 with BPCI #2028. This PR has two commits:

  • bb-cli: add compose-ctrl.block.builderlabstaging.xyz to TRUSTED_CONTROL_PLANE_HOSTS. The old compose-ctrl.test.blockstaging.build host still answers (401 without credentials), so it stays trusted until the old domain is retired.
  • Telemetry: move the staging OTLP gateway to otel.block.builderlabstaging.xyz. Per the note in vite.config.ts, the host changes at all four sites together: the default endpoint, ALLOWED_OTEL_LOGS_HOSTS, the CSP connect-src and the pinned tests. Both otel hosts are routed by the staging ingress, and POST /v1/logs on the new host returns the same 415 as the old one.

Related issue

N/A. Same change for builderlab-cli: block/builderlab-cli#36.

Testing

  • bb apps allowlist test (control_plane_allowlist_is_exact_and_https_only) passes. It covers the new host with and without :443 and rejects its http, attacker-suffix and :444 variants.
  • Telemetry tests pass in both lanes, default and block-telemetry-enforced (46 and 38 tests).
  • just check passes.
  • Local toolchain caveat: the pinned Rust 1.94.1 can't load proc-macro dylibs on macOS 27 (sqlx-macros: "mis-aligned LINKEDIT string pool"). So I ran the Tauri tests with Rust 1.98.1 and pushed with the tauri-check and clippy hooks excluded. Clippy on 1.98.1 flags only pre-existing new lints in crates/berd-call, none in the files changed here. CI with the pinned toolchain is the authoritative check.

🤖 Generated with Claude Code

matt2e and others added 2 commits October 28, 2026 23:20
bb-block staging moved from test.blockstaging.build to
block.builderlabstaging.xyz, so `bb apps` must accept
compose-ctrl.block.builderlabstaging.xyz. The old host still answers
during the transition and stays trusted.

Co-authored-by: Jitter <2b2e6415e748c35180847a37aadae06a11e30dddc76b784e7fd9354c4eb42e7a@buzz.block.builderlab.xyz>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
bb-block staging moved from test.blockstaging.build to
block.builderlabstaging.xyz. Move the staging OTLP endpoint to
otel.block.builderlabstaging.xyz at all four sites that pin it: the
vite.config.ts default, ALLOWED_OTEL_LOGS_HOSTS, the CSP connect-src and
the pinned tests.

Co-authored-by: Jitter <2b2e6415e748c35180847a37aadae06a11e30dddc76b784e7fd9354c4eb42e7a@buzz.block.builderlab.xyz>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@matt2e
matt2e marked this pull request as ready for review October 28, 2026 13:01
@matt2e
matt2e requested a review from a team October 28, 2026 13:01

@morgmart morgmart left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Automated code review

Engineering found no concrete changed-line defects in the exact PR comparison. The new staging domains are applied consistently across the CLI allowlist and tests, the telemetry build endpoint, native allowlist and tests, CSP, and documentation. Supplied GitHub evidence is structurally valid; Frontend checks and unit tests were still in progress while the other reported checks were successful or skipped, so required checks still govern merge readiness.

Deterministic publication result: 0 blocking and 0 non-blocking inline finding(s) publishable; 0 duplicate(s) suppressed; 0 blocking screenshot-evidence requirement(s) in this review body.

Pending checks: 2 check(s) are not complete.

This approval reflects the completed code review only; merge readiness remains governed by the repository's required checks.

This branch has not been deployed

No deployments
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.

2 participants