-
Notifications
You must be signed in to change notification settings - Fork 668
DYN-9196:Downgrade to .NET 10.0 Preview 5 for stability #16566
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-9196
|
@dnenov Could you add more details about stability. I would also like to hear inputs from @BogdanZavu as well. If we must downgrade, we need to make couple more changes,
|
It is part of this thread - https://autodesk.slack.com/archives/C0531NMR189/p1759417135610879?thread_ts=1753895883.337709&cid=C0531NMR189. @Mikhinja is reporting severe speed issues potentially due to parts of Dynamo not being correctly ran under Preview 5 SDK environment. |
So, basically we need to lock against Preview 5 without roll forward? OK, let me do these changes just in case. This PR is a provision, in case we are forced to use it. |
|
We will need to install Preview 5 SDK on the machine: |
With Getting the exact version is easier in GitHub actions with setup-dotnet action. For internal agents, we need some changes. |
|
My take on this is to wait a little bit before doing a downgrade.
We need to confirm/further look into this. Imo as long as it still works even if it's slower we should be good for now. |
|
This one did not build, so re-triggered as https://master-5.jenkins.autodesk.com/job/Dynamo/job/DynamoSelfServe/job/pullRequestValidation/18513/ @BogdanZavu This is a backup plan in case the performance issue was real between our build and D4R. We need to be prepared that the ITF boston machine does not come with latest .NET10 SDKs |
This is expected to fail since we don't have the exact version of net10 SDK installed in the build image. |
So @avidit @QilongTang @dnenov I'm having some second thoughts on this. Probably our best bet to be ready for ITF Boston would be to stick to preview 5 across the board. |
My brief chat with @mjkkirschner suggests that some components (specifically the Flag CLI), which were built with Preview 7 in mind, are actually being executed under Preview 5 within D4R and Revit. This mismatch appears to be generating the large volume of empty logs - likely due to repeated runtime failures - which in turn causes the significant performance degradation. I’m not entirely sure how to validate or mitigate this while remaining on Preview 7, but I agree with @BogdanZavu that the most pragmatic approach for now is to revert to Preview 5 and align with the SDK version agreed for this release. |
|
@QilongTang @BogdanZavu @avidit I tried removing |
|
@dnenov @avidit PR check failing, restarted at https://master-5.jenkins.autodesk.com/job/Dynamo/job/DynamoSelfServe/job/pullRequestValidation/18535/ |
| dotnet-version: | | ||
| 8.0.x | ||
| 10.0.x | ||
| 10.0.100-preview.5.25277.114 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@avidit Do we need to touch this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, since we have "rollForward": "disable" in this PR, we will need the exact version.
|
Merging this to align with Revit preview build |

Purpose
Change SDK version in global.json to Preview 5
Declarations
Check these if you believe they are true
Release Notes
10.0.100-preview.5.25277.114Reviewers
@QilongTang
@zeusongit
@avidit
FYIs
@Mikhinja