-
Notifications
You must be signed in to change notification settings - Fork 439
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
CI fails when SQL Server 2022 CUs can't be dowloaded #1666
Comments
This is the line where the error comes from: https://github.com/tediousjs/setup-sqlserver/blob/v2.1.0/src/utils.ts#L199 There's not much logging going on here, so it's hard to know exactly why this is happening. It could be:
I'll add a PR to increase logging so we can see what is going on here. |
Something I had thought about when first authoring the cumulative updates feature was using the CI cache to hold onto these update URLs to prevent having to repeatedly hit the website for them, but the caching is more for caching files and not arbitrary data (https://github.com/actions/toolkit/tree/main/packages/cache). |
Yeah. It would be nice to cache all the installer files, but I guess that can easily bump into the 10GB actions cache limit. |
Do the URLs change regularly, or could the action ship with some defaults, which could then be used as a fallback if the check doesn't work? |
@arthurschreiber has this issue happened again recently? |
@dhensby Every now and then I see the warning
Warning: Unable to download cumulative updates
fromtediousjs/setup-sqlserver@v2
, and if that happens, builds on SQL Server 2022 fail because the TDS 8 feature is broken on an unpatched 2022 version.Can you take a look into why downloading the updates fails sporadically?
The text was updated successfully, but these errors were encountered: