dotnet / machinelearning

ML.NET is an open source and cross-platform machine learning framework for .NET.
https://dot.net/ml
MIT License
9.02k stars 1.88k forks source link

Hotlinking to blob store #4445

Closed justinormont closed 4 years ago

justinormont commented 4 years ago

We need aka.ms links to the CDN instead of direct links to blob store:

The unit test and sample will stop working in a month as we close down that blob store. (update: found out this is no longer the case -- https://github.com/dotnet/machinelearning/issues/4445#issuecomment-552034148)

We should add a check-in test to ensure that no blob-store links are added in the future. All links should be an aka.ms link to the CDN.

justinormont commented 4 years ago

These six should also follow the aka.ms to CDN route: (consider factoring out this duplicate code)

gvashishtha commented 4 years ago

@yaeldekel It turns out that our storage account will not be affected by US West upgrade, so the only work item is to turn the hotlinks into aka.ms short links. Does this make sense?