Skip to content

Simplify tunneled E2E setup - #1079

Open
j0sh wants to merge 5 commits into
mainfrom
ja/streamline-ci
Open

Simplify tunneled E2E setup#1079
j0sh wants to merge 5 commits into
mainfrom
ja/streamline-ci

Conversation

@j0sh

@j0sh j0sh commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • restore concurrent in-a-box recording tests
  • consolidate synchronized output, MinIO readiness, and object-store URL helpers
  • wait for Catalyst API and orchestrator readiness before submitting each VOD request exactly once
  • redact trycloudflare URLs from captured container and command output
  • keep the catalyst-api and go-livepeer revisions plus the signing, misspell, and checksum changes unchanged

Known blocker

The E2E suite still requires public Cloudflare quick tunnels because the current catalyst-api requires callback_url and rejects private object-store destinations. The focused TODO documents the follow-up: optional VOD callbacks plus an empty-by-default exact host:port allowlist enforced during validation and dialing, followed by configuring in-a-box for its local MinIO and Task Runner endpoints.

Local validation

  • go test -race ./test/e2e -short
  • go test ./... -short
  • go vet ./...
  • git diff --check

The complete Docker-backed E2E suite will run in CI because Docker and cloudflared are not installed on the development host.

@j0sh
j0sh requested a review from emranemran September 7, 2026 19:04
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