Skip to content

Add Cloudflare quick tunnel support for manager URLs - #274

Open
hermabr wants to merge 1 commit into
mainfrom
cloudflare-quick-tunnel-manager-url
Open

Add Cloudflare quick tunnel support for manager URLs#274
hermabr wants to merge 1 commit into
mainfrom
cloudflare-quick-tunnel-manager-url

Conversation

@hermabr

@hermabr hermabr commented May 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds a pluggable manager connection abstraction with direct and Cloudflare quick tunnel implementations.
  • Routes manager startup through an advertised URL while keeping a local loopback origin for tunnel setup.
  • Updates Slurm backend defaults to use Cloudflare quick tunnels when no worker connect host is configured.
  • Adds tests for tunnel startup, timeout/exit handling, URL rewriting, and manager connection selection.

Testing

  • Not run
  • Covered by new tests for:
    • direct connection passthrough
    • cloudflared command validation and PATH errors
    • trycloudflare URL parsing from subprocess output
    • timeout and early-exit cleanup behavior
    • Slurm backend URL rewriting and defaults
    • manager/server advertised URL selection and conflict detection

- Introduce a manager connection abstraction with direct and cloudflared-backed implementations
- Advertise a public trycloudflare URL to workers while using loopback locally
- Update Slurm backend defaults and add coverage for tunnel startup and shutdown
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