Skip to content

Fix dotnet latest resolution#1598

Open
sliekens wants to merge 4 commits intodevcontainers:mainfrom
sliekens:dotnet-latest-releases-index
Open

Fix dotnet latest resolution#1598
sliekens wants to merge 4 commits intodevcontainers:mainfrom
sliekens:dotnet-latest-releases-index

Conversation

@sliekens
Copy link
Contributor

@sliekens sliekens commented Mar 9, 2026

Summary

Fix the dotnet feature's latest resolution so it is resolved from the .NET releases index instead of comparing STS and LTS versions lexicographically.

Resolves #1597.

Changes

  • resolve latest from release-metadata/releases-index.json
  • use the first active entry from the releases index
  • read latest-sdk for SDK installs and latest-runtime for runtime installs
  • keep explicit versions and lts flowing through the vendored dotnet-install.sh logic
  • add jq to the feature install dependencies
  • update tests to stop depending on helper-side channel resolution
  • remove the deprecated Aspire workload from docs and workload tests

Validation

  • devcontainer features test -p . -f dotnet
  • sourced the updated helper and verified live resolution returns 10.0.103 for SDK and 10.0.3 for runtime
  • shell syntax checks passed for the updated scripts

@sliekens sliekens marked this pull request as ready for review March 9, 2026 14:47
@sliekens sliekens requested a review from a team as a code owner March 9, 2026 14:47
Copilot AI review requested due to automatic review settings March 9, 2026 14:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the dotnet feature’s latest version resolution to use the official .NET releases index metadata instead of lexicographic STS/LTS comparisons.

Changes:

  • Resolve latest from release-metadata/releases-index.json by selecting the first active entry.
  • Install jq as a dotnet feature dependency and update helpers/tests accordingly.
  • Remove deprecated aspire workload usage from docs and workload tests.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
test/dotnet/scenarios.json Drops aspire from workload scenario coverage.
test/dotnet/install_dotnet_workloads.sh Removes assertion for deprecated aspire workload installation.
test/dotnet/install_dotnet_specific_release.sh Adjusts specific-release test expectation for .NET 10.0.
test/dotnet/install_dotnet_lts.sh Stops using helper channel-resolution; fetches LTS expected version directly.
test/dotnet/dotnet_helpers.sh Updates test helper to resolve latest via releases-index.json + jq.
src/dotnet/scripts/dotnet-helpers.sh Updates feature helper to resolve latest via releases-index.json + jq.
src/dotnet/install.sh Adds jq to apt dependencies for releases-index parsing.
src/dotnet/README.md Documents prerelease suffixes and removes aspire from workload example.
src/dotnet/NOTES.md Removes aspire from workload example.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@sliekens sliekens force-pushed the dotnet-latest-releases-index branch from 1307a2e to b628fe5 Compare March 9, 2026 15:12
@sliekens sliekens requested a review from Copilot March 9, 2026 15:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

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.

dotnet: "latest" is stuck on .NET 9

2 participants