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

Update dotnet releases index url #20779

Merged
merged 3 commits into from
Jan 13, 2025
Merged

Update dotnet releases index url #20779

merged 3 commits into from
Jan 13, 2025

Conversation

tarunramsinghani
Copy link
Collaborator

Cherry Picked from #20748

Task name: DotNetCoreInstaller、UseDotNet

Description: Update dotnet releases index url

Documentation changes required: N

Added unit tests: N

Attached related issue: N

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@Varorbc
Copy link
Contributor

Varorbc commented Jan 10, 2025

Could you help me out by copying the error log? I don't have the permission to view it myself.

@tarunramsinghani
Copy link
Collaborator Author

tarunramsinghani commented Jan 13, 2025

@Varorbc, you should be able to look at the logs as anonymous user. You can try opening the pipeline in InPrivate window.

For now the error I see related to generated files not matching the original, need to run build locally and update the generated files. Committed the change

Content doesn't match /mnt/vss/_work/1/s/Tasks/DotNetCoreInstallerV1/README.md to /mnt/vss/_work/1/s/_generated/DotNetCoreInstallerV1/README.md (overwrite=true).  Dest file doesn't match source.
System.Exception: Updates needed, please run node make.js --task DotNetCoreInstallerV1 
   at BuildConfigGen.Program.ThrowWithUserFriendlyErrorToRerunWithWriteUpdatesIfVeriferError(String task, Boolean skipContentCheck) in /mnt/vss/_work/1/s/BuildConfigGen/Program.cs:line 502
   at BuildConfigGen.Program.MainUpdateTask(TaskStateStruct taskVersionState, String task, IEnumerable`1 configs, Boolean writeUpdates, Int32 currentSprint, IDebugConfigGenerator debugConfigGen, Boolean includeLocalPackagesBuildConfig, Boolean hasGlobalVersion, String generatedFolder, String altGeneratedFolder) in /mnt/vss/_work/1/s/BuildConfigGen/Program.cs:line 710
   at BuildConfigGen.Program.MainInner(String task, String configs, Nullable`1 currentSprintNullable, Boolean writeUpdates, Boolean allTasks, Boolean getTaskVersionTable, String debugAgentDir, Boolean includeLocalPackagesBuildConfig) in /mnt/vss/_work/1/s/BuildConfigGen/Program.cs:line 293
   at BuildConfigGen.Program.Main(String task, String configs, Nullable`1 currentSprint, Boolean writeUpdates, Boolean allTasks, Boolean getTaskVersionTable, String debugAgentDir, Boolean includeLocalPackagesBuildConfig) in /mnt/vss/_work/1/s/BuildConfigGen/Program.cs:line 77

@Varorbc
Copy link
Contributor

Varorbc commented Jan 13, 2025

@tarunramsinghani Alright, everything's good to go. Let's merge it.

@tarunramsinghani tarunramsinghani enabled auto-merge (squash) January 13, 2025 09:51
@tarunramsinghani tarunramsinghani merged commit e2d68a3 into master Jan 13, 2025
11 checks passed
@tarunramsinghani tarunramsinghani deleted the users/tarun/release branch January 13, 2025 10:02
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