Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add mirror.facebook.net mirror #939

Merged
merged 1 commit into from
Nov 9, 2023
Merged

Conversation

davide125
Copy link
Contributor

Add config for our mirror

Comment on lines 7 to 10
geolocation:
country: US
state_province: Oregon
city: Prineville
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This isn't entirely correct, as that endpoint is loadbalanced between two regions (Prineville, OR and Forest City, NC), but I couldn't find a way to express that here.

Copy link
Member

Choose a reason for hiding this comment

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

Oh. We have some (pretty darn good) geo-steering when we serve mirrors. Is it possible to bypass the load balancer and just add an entry for each location? Does the load balancer also try to geo-steer traffic?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, this is behind a load balancer that steers requests between two pools of hosts (one pool per region) and terminates SSL. We could theoretically rig per-region endpoints, but that's a non-trivial endeavor and I don't know how feasible it would be in practice.

My recommendation here is to support mirrors that just declare a country, and setup your geosteering to send them some of the traffic directed towards the country (regardless of more specific destination).

Copy link
Member

Choose a reason for hiding this comment

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

Per our chat off-list these appear to be in San Jose/Santa Clara and Ashburn with an anycasted IP based on traces.

mirrors.d/mirror.facebook.net.yml Outdated Show resolved Hide resolved
mirrors.d/mirror.facebook.net.yml Outdated Show resolved Hide resolved
@jonathanspw jonathanspw merged commit 05b098a into AlmaLinux:master Nov 9, 2023
1 check passed
@jonathanspw
Copy link
Member

Thanks for setting these up!

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