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

revert .net sdk version to 9.0.101 #15320

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from
Draft

Conversation

haruna99
Copy link

@haruna99 haruna99 commented Dec 10, 2024

Related Issue

dotnet/dnceng#4635

To double check:

PR Summary

This pull request reverts .net sdk to version 9 to resolve build issues

@ViktorHofer
Copy link
Member

We don't want to downgrade back to a 9.0.xxx SDK. We explicitly need the .NET 10 SDK for the .NET 10 Preview 1 release. cc @mmitche

Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

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

This is a pipeline specific issue and therefore shouldn't force Arcade to go back to a previous SDK. The dotnet-arcade-daily pipeline must be able to use prerelease SDKs or should manually pin back to an older SDK.

@mmitche
Copy link
Member

mmitche commented Dec 10, 2024

@haruna99 Viktor is right here. I think the correct thing to do may just be to do a restore (Restore.cmd) of the arcade repo instead of the 'Install global.json SDK' step. That will also install the SDK from the global.json.

@mmitche
Copy link
Member

mmitche commented Dec 12, 2024

@haruna99 Any success here?

@haruna99 haruna99 closed this Dec 12, 2024
@haruna99 haruna99 reopened this Dec 12, 2024
@haruna99
Copy link
Author

@mmitche I am still working on it. I attempted to restore but encountered errors during the secret-manager synchronization stage. This stage attempts to access .NET 10 online and is unable to access the version downloaded by the restore stage.

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.

3 participants