Skip to content

Comments

[3006.x] fix #66686: don't fail for InvalidVersion, instead drop the version as-if it was never there#66687

Open
Legrems wants to merge 1 commit intosaltstack:3006.xfrom
ArcaniteSolutions:3006.x
Open

[3006.x] fix #66686: don't fail for InvalidVersion, instead drop the version as-if it was never there#66687
Legrems wants to merge 1 commit intosaltstack:3006.xfrom
ArcaniteSolutions:3006.x

Conversation

@Legrems
Copy link

@Legrems Legrems commented Jul 3, 2024

What does this PR do?

Add "support" for InvalidVersion: ignore the version instead of failing completly

What issues does this PR fix or reference?

Fixes #66686

Previous Behavior

Failure when listing all version of a package when there is an InvalidVersion

New Behavior

State pass, but ignore if there is an InvalidVersion, and add a INFO log about this

Merge requirements satisfied?

Commits signed with GPG?

Yes

@Legrems Legrems requested a review from a team as a code owner July 3, 2024 12:19
@salt-project-bot-prod-environment salt-project-bot-prod-environment bot changed the title fix #66686: don't fail for InvalidVersion, instead drop the version as-if it was never there [3006.x] fix #66686: don't fail for InvalidVersion, instead drop the version as-if it was never there Jul 3, 2024
@dwoz dwoz added the needs-testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases label Jul 3, 2024
@twangboy
Copy link
Contributor

This needs some tests and a changelog please

@twangboy
Copy link
Contributor

please rebase and resolve conflicts

@twangboy twangboy removed this from the Sulphur v3006.21 milestone Feb 13, 2026
@twangboy twangboy added this to the Sulphur v3006.22 milestone Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-changelog needs-testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Removed reliance on LegacyVersion from pkg_resources can break pip modules

3 participants