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

[release/9.0.1xx] [FIX] tool-update: Compare tool versions correctly #45383

Merged
merged 3 commits into from
Jan 15, 2025

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Dec 9, 2024

Backport of #44517 to release/9.0.1xx

Customer Impact/Description

When updating a tool, versions are not compared correctly, making it impossible for users to update their tools to new minor versions. This means users are unable to update a tool from version 1.0.0-preview to version 1.0.0 for example. (see #43089)

The introduced behavior compares the versions correctly, meaning users should now be able to update their tools.

Risk

Low - the version comparison is used to avoid installing already existing versions. This is the expected behavior.

Regression?

No. This is introduced for 9.0.1xx

Testing

New automated tests were added for installing newer minor versions of tools. Did manual testing to make sure tool was updated correctly between minor versions.

/cc @edvilme

@marcpopMSFT
Copy link
Member

@edvilme can you prep the tactics template and set to servicing consider in Jan. We'll take for 9.0.103 for Feb release.

@edvilme edvilme force-pushed the backport/pr-44517-to-release/9.0.1xx branch from 50f3c47 to 8c07f4f Compare January 6, 2025 19:13
@marcpopMSFT
Copy link
Member

@edvilme can you check with @nagilson and send mail to tactics as we have no tactics meetings this week because of other conflicts.

@marcpopMSFT
Copy link
Member

Approved over email

@marcpopMSFT marcpopMSFT merged commit a5b3141 into release/9.0.1xx Jan 15, 2025
31 checks passed
@marcpopMSFT marcpopMSFT deleted the backport/pr-44517-to-release/9.0.1xx branch January 15, 2025 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants