[ci] Enable source index #26557
[ci] Enable source index #26557
Conversation
fix variables Update azure-pipelines-internal.yml Update NuGet.config try pass this one Update NuGet.config enable source index Update build-test-pack.yml Update build-test-pack.yml try enable source index try this again no warn and bin log try fix binlog
rmarinho
requested review from
StephaneDelcroix,
mattleibow and
jonathanpeppers
rmarinho
added
area-infrastructure
| enableSourceIndex: false | ||
| enableSourceIndex: ${{ parameters.enableSourceIndex }} | ||
| sourceIndexParams: | ||
| sourceIndexBuildCommand: build.cmd -restore -build -ci -warnAsError 0 /bl:$(Build.Arcade.LogsPath)sourceIndexBuild.binlog /p:OfficialBuildId=$(_BuildOfficalId) /p:_SkipUpdateBuildNumber=true |
There was a problem hiding this comment.
This mixes - and /, do you want to pick one to be consistent?
| <!-- Added manually for .NET 8.0.12 --> | ||
| <add key="darc-pub-dotnet-runtime-c1ae9626" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/darc-pub-dotnet-runtime-c1ae9626/nuget/v3/index.json" /> |
There was a problem hiding this comment.
What needed this feed? The build.cmd below?
Does it use the system-installed .NET on the CI machine or something else?
There was a problem hiding this comment.
the build.cmd by default will use the existing dotnet if it exists but I don t really want that for now. So I bumped do t make it fix our build here:
https://dev.azure.com/dnceng/internal/_build/results?buildId=2599947&view=results
| "tools": { | ||
| "dotnet": "9.0.100" | ||
| "dotnet": "9.0.102-servicing.24577.25" |
There was a problem hiding this comment.
Is this why the feed is required above? Could you just use 9.0.101 that just came out?
There was a problem hiding this comment.
Yap, kinda of, right now the setup I have it needs to provision a new dotnet, if the global dotnet matches my script fails, I need to fix that. thats why I bumped this.
There was a problem hiding this comment.
If this is main, was the policy to only use stable/released bits? Like 9.0.101?
There was a problem hiding this comment.
This is only used by our internal setup on dnceng and builds with arcade. For a developer suing cake shouldn't t make a difference. the version for the sdk on the version props is still 9.0.100
Description of Change
Make our internal build on dnceng and arcade push to source.dot.net
After merging on our side we need to wait for the merge of this one dotnet/source-indexer#178
Issues Fixed
Fixes #6814