-
Notifications
You must be signed in to change notification settings - Fork 376
Update SourceIndexPackageVersion #14172
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
|
No ideas, as I'm not familiar with the source index code at all. Can you remind me what is driving the need for a new binlog version, and therefore this chain of changes? |
|
MSBuild.StructuredLogViewer updates sometimes break source-index:
We've been seeing this in every Source-Index job in aspnetcore for the past few days, which motivated dotnet/source-indexer#120. For now we've disabled source-index in aspnetcore |
|
Do you know what caused the aspnetcore build to try to use newer binlog versions? |
I think msbuild got updated to generate the new binlog versions, so the build machines we run on must have gotten updated to use that new version. We haven't changed anything with regards to binlog generation |
|
@dotnet/dnceng who else might know about source-index? We've had to disable it in aspnetcore because of this error |
|
Ownership of this was transferred to @SteveMCarroll a bit ago. I'm not sure who he has to work on it right now, but off course, we're happy to assist whoever that is where we can. |
Correct. We need to keep the StructuredLogViewer nuget package reference up-to-date so that it can handle newer binlog versions. MSBuild from time to time revs the binlog version when they add new stuff to the format. |
|
This is likely required before updating Arcade to a .NET 9 SDK. @wtgodbe what's the status on this? Is there anything left? |
|
I don't know what needs to be done here, I tried doing a simple update but it failed - I don't own SourceIndex, I was hoping someone with more experience with the tool could handle the update. |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Loading 15.1 MSBuild...odd...okay |
|
Should we also update the .NET SDK to 8? arcade/eng/common/templates/job/source-index-stage1.yml Lines 42 to 48 in 1f6c5ac
|
|
When I was doing the update in source-indexer, I had to update some MSBuild deps to fix transitive dependency errors in the official build - maybe related? https://github.com/dotnet/source-indexer/pull/121/files |
|
Yeah, 17.5 of msbuild packages should require that version of msbuild. So we probably need to update the SDK. |
|
I added a rollforward switch to the two global tools for now but we ideally should just make them target net8.0 instead. |
|
@mmitche now I'm unsure if the |
|
Our stock 2022 pool has 17.8. Let me switch to that. |
Hmm I'm out of ideas unless trying to repro this locally. cc @rainersigwald in case you understand why the msbuild assembly isn't found / loaded. |
And why 15.1.0.0?...that such an old number |
that's our current version, frozen in amber at the point we went open-source :-/ |
|
|
I don't know what's going on here, though. I'll plan to take a look tomorrow morning. |
|
I root caused the failure and will have a PR shortly for dotnet/source-indexer |
|
@akoeplinger thank you so much! |

Dupe of dotnet/aspnetcore#51715. Related: dotnet/source-indexer#120