Skip to content

Use Cloudflare R2 + CDN for videos#285

Merged
sshane merged 5 commits intomasterfrom
smaller-videos
Mar 25, 2026
Merged

Use Cloudflare R2 + CDN for videos#285
sshane merged 5 commits intomasterfrom
smaller-videos

Conversation

@sshane
Copy link
Contributor

@sshane sshane commented Mar 25, 2026

@github-actions
Copy link
Contributor

github-actions bot commented Mar 25, 2026

deployed preview: https://comma-web--pr285-7pvymciz.web.app

Welcome! Make sure to:

  • read the contributing guidelines
  • mark your PR as a draft until it's ready to review
  • post the preview on Discord; feedback from users will speedup the PR review

@adeebshihadeh
Copy link
Collaborator

Does it look worse? I was just thinking we move to a cheaper CDN for the videos

@sshane
Copy link
Contributor Author

sshane commented Mar 25, 2026

yeah looking at cloudflare, it looks very promising

This reverts commit 7a084d9.
@sshane sshane changed the title Smaller videos Use Cloudflare R2 + CDN for videos Mar 25, 2026
@sshane
Copy link
Contributor Author

sshane commented Mar 25, 2026

The public r2.dev link is rate limited and not intended to be used in production. To use the CDN properly Cloudflare requires we use their nameservers on the free plan, or pay $200/month for a partial CNAME setup without touching our Azure nameservers (https://developers.cloudflare.com/dns/zone-setups/partial-setup/#availability) which doesn't seem worth it.

We can either 1) move comma.ai nameservers to Cloudflare. Bigger project, but we get CDN on website and blog automatically w/ no manual bucket storage uploading. Or 2) buy a new cheap domain just for videos here/CDN. Should I just buy a cheap .com for now? @adeebshihadeh

Both options from Cloudflare should be completely free w/ unlimited egress. First 10GB of storage on R2 is free

@sshane sshane merged commit b1c6fc8 into master Mar 25, 2026
4 checks passed
@sshane sshane deleted the smaller-videos branch March 25, 2026 04:02
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