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

[Update Request]: Microsoft.Dotnet.* (update CDN urls) #202037

Open
a-mnich opened this issue Dec 24, 2024 · 10 comments
Open

[Update Request]: Microsoft.Dotnet.* (update CDN urls) #202037

a-mnich opened this issue Dec 24, 2024 · 10 comments
Labels
Help-Wanted This is a good candidate work item from the community. Package-Update This package needs to be updated

Comments

@a-mnich
Copy link
Contributor

a-mnich commented Dec 24, 2024

What type of update are you requesting?

Something else not listed

Current Package Identifier

Microsoft.Dotnet.*

Package Version

multiple

Please describe the changes you would like to see

As announced in dotnet/core#9671 the hoster of the CDNs https://dotnetcli.azureedge.net/ and https://dotnetbuilds.azureedge.net/ will soon seize operations due to bankrupcy.
It is recommended that users switch to the new CDN domains as quickly as possible.
While a majority of the newest versions Microsoft.Dotnet.*packages has already been switched to the new CDN by @mthalman (e.g. in PR #200316) there are still a lot of packages with the old domains remaining.

  • A GitHub Search for https://dotnetcli.azureedge.net/ gets results in 620 files within this repository.
  • https://dotnetbuilds.azureedge.net/ isn't used within winget-pkgs

Action steps:
Open PRs to switch the installer domain from https://dotnetcli.azureedge.net/ to builds.dotnet.microsoft.com for all packages found in the GitHub search.

@a-mnich a-mnich added Help-Wanted This is a good candidate work item from the community. Package-Update This package needs to be updated labels Dec 24, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Triage This work item needs to be triaged by a member of the core team. label Dec 24, 2024
@flo7000
Copy link
Contributor

flo7000 commented Dec 25, 2024

hello,
i'd like to do that but i have a question:
i should do a PR for each package, right?

and starting at which point of the path do i count it as a whole package? at "SDK" in that example?
/manifests/m/Microsoft/DotNet/SDK/6/6.0.122/Microsoft.DotNet.SDK.6.installer.yaml

and use the whole link "https://builds.dotnet.microsoft.com" (and the rest of the installer path stays the same right?)

@Dragon1573
Copy link
Contributor

Dragon1573 commented Dec 25, 2024

I should do a PR for each package, right?

I think you may need to create pull requests for each versions of these packages.

"Sparse checkout" feature is enabled on my local clone. Sparse checkout the directory /manifests/m/Microsoft/DotNet with --no-cone mode and try following command. You'll have to create such that LARGE amount of pull requests ...

22:06:14 D:\...\winget-pkgs  [master ≡] 1ms pwsh> C:\Applications\Git\Git\usr\bin\find.exe ./manifests/m/Microsoft/DotNet -type f -name '*installer.yaml' | C:\Applications\Git\Git\usr\bin\wc.exe -l
585

Note

It's easy using IDEs or editors to perform a string replacement and submitting them via a single PR. But it's hard (wasting time accurately) creating 585 for each of them. 😞

@flo7000
Copy link
Contributor

flo7000 commented Dec 25, 2024

okay, i understand, thank you!

so i can be sure that all the paths exist at the new domain? not that i'd have to test that too :D

@Dragon1573
Copy link
Contributor

So I can be sure that all the paths exist at the new domain? not that I'd have to test that too :D

Neither can't I. I've just tested one URL and it's valid. There might need a script for automatically checking these URLs.

@stephengillie stephengillie removed the Needs-Triage This work item needs to be triaged by a member of the core team. label Dec 25, 2024
@flo7000
Copy link
Contributor

flo7000 commented Dec 26, 2024

Checklist for all DotNet "Subpackages"

  • SDK -> 189 files

    • SDK 3_1 -> 8 files
    • SDK 5 -> 2 files
    • SDK 6 -> 77 files
    • SDK 7 -> 50 files
    • SDK 8 -> 26 files
    • SDK 9 -> 1 file
    • SDK Preview -> 25 files
  • Runtime -> 93 files

    • Runtime 3_1 -> 8 files
    • Runtime 5 -> 1 file
    • Runtime 6 -> 30 files
    • Runtime 7 -> 20 files
    • Runtime 8 -> 10 files
    • Runtime 9 -> Nothing left
    • Runtime Preview -> 24 files
  • HostingBundle -> 92 files

    • HostingBundle 3_1 -> 8 files
    • HostingBundle 5 -> 1 file
    • HostingBundle 6 -> 29 files
    • HostingBundle 7 -> 20 files
    • HostingBundle 8 -> 10 files
    • HostingBundle 9 -> Nothing left
    • HostingBundle Preview -> 24 files
  • DesktopRuntime -> 93 files

    • DesktopRuntime 3_1 -> 8 files
    • DesktopRuntime 5 -> 1 file
    • DesktopRuntime 6 -> 30 files
    • DesktopRuntime 7 -> 20 files
    • DesktopRuntime 8 -> 10 files
    • DesktopRuntime 9 -> Nothing left
    • DesktopRuntime Preview -> 24 files
  • AspNetCore -> 92 files

    • AspNetCore 3_1 -> 8 files
    • AspNetCore 5 -> 1 file
    • AspNetCore 6 -> 29 files
    • AspNetCore 7 -> 20 files
    • AspNetCore 8 -> 10 files
    • AspNetCore 9 -> Nothing left
    • AspNetCore Preview -> 24 files

I've started with SDK 3_1 as a test (because I can be sure that nobody else is editing that manifest) but I'll do all 8/9/Preview Manifests first :)

@flo7000
Copy link
Contributor

flo7000 commented Dec 26, 2024

@a-mnich , is this due January 15 2025 ?

https://learn.microsoft.com/en-us/azure/cdn/edgio-retirement-faq

this FAQ says the following:

Edgio has informed us that their services will remain operational until at least January 15, 2025. However, we can't guarantee that Edgio won't unexpectedly cease operations before this date due to circumstances beyond our control.

@a-mnich
Copy link
Contributor Author

a-mnich commented Dec 26, 2024

@flo7000 I don't have any additional insight. Therefore I would stick to the information from the FAQ.

@flo7000
Copy link
Contributor

flo7000 commented Dec 26, 2024

@a-mnich what packages are you gonna update? not that we do something double

and how do you update them all with a bot? :O

@a-mnich
Copy link
Contributor Author

a-mnich commented Dec 26, 2024

@flo7000
I think it's easiest to go package wise, so I'm gonna do all Microsoft.DotNet.AspNetCore versions now. (I'm working bottom to top in your list :D)
If no problems arise in the validation pipeline, I will open subsequent PRs after the first batch is merged.

Since you worked on it first I don't want to “take anything away” from you and we always appreciate new contributors for maintaining the packages. :) I's quite easy for me to create bulk package upgrades like this as I've implemented an automation for such jobs.

I will open up the PRs for:

  • Microsoft.DotNet.AspNetCore
  • Microsoft.DotNet.DesktopRuntime
  • Microsoft.DotNet.HostingRuntime
  • Microsoft.DotNet.Runtime

Then there's Microsoft.DotNet.SDK left for you to do. Just leave a quick message in this thread if I should take over anything. :)

This was referenced Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help-Wanted This is a good candidate work item from the community. Package-Update This package needs to be updated
Projects
None yet
Development

No branches or pull requests

4 participants