-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Consider using a CDN for releases-index.json
links
#9669
Comments
I'm wondering if release-index can be set up to use a CDN, since it relies on GitHub links through the Azure Pipeline task, which is causing a lot of trouble for users trying to access it from China. |
It's already on a CDN, but don't take a dependency on the following link. https://dotnetcli.azureedge.net/dotnet/release-metadata/releases-index.json Note: We are in the middle of changing our CDN per https://learn.microsoft.com/azure/frontdoor/migrate-cdn-to-front-door. We're going to publishing new guidance (hopefully) starting Monday. |
could you take a look at this pull request and give me some advice? Also, which URL should I be using? |
We are talking with the pipelines team in the morning (Pacific time). I already shared this issue with them. We will discuss it. |
This is awesome news! This issue has been bothering Chinese users for so long. Really appreciate it, thank you so much! Sorry to bother you. |
No bother. Our new CDN is working (as of 10mins ago). https://builds.dotnet.microsoft.com/dotnet/release-metadata/releases-index.json |
Is there a blog post that describes this in more detail? I want to understand better. |
Ah, I see the links in the pipeline are from GitHub. Due to the GFW (Great Firewall) in China, many sites, including GitHub, can't be accessed properly. |
Sorry ... I get it now. You are proposing that Azure DevOps (at least for this scenario) should have no dependence on GitHub. I was getting mixed up between Actions and Azure DevOps. I have been talking to both teams over the last couple days. We need to move both teams to our new CDN. We'll be publishing new information about the CDN, shortly. |
We are using direct blob storage links in our JSON files. That is bad for performance and for infra flexibility. We should consider changing to a CDN.
core/release-notes/releases-index.json
Lines 15 to 16 in 218ef74
The text was updated successfully, but these errors were encountered: