Skip to content

Fix versionLT for wrapped CD plugin versions - #2415

Open
oleksii-tumanov wants to merge 1 commit into
jenkinsci:masterfrom
oleksii-tumanov:fix/versionlt-cd-suffix
Open

Fix versionLT for wrapped CD plugin versions#2415
oleksii-tumanov wants to merge 1 commit into
jenkinsci:masterfrom
oleksii-tumanov:fix/versionlt-cd-suffix

Conversation

@oleksii-tumanov

Copy link
Copy Markdown

Summary

  • Compare the numeric wrapped-component prefix before the Jenkins CD release suffix in versionLT.
  • Limit the special case to CD release suffixes matching <number>.v..., then fall back to the existing comparison.
  • Keep Linux and Windows startup plugin-copy behavior aligned.

Fixes #2361

Result

Representative final-patch behavior:

  • versionLT 9.10-211.v7d13903b_a_d89 9.10.1-216.va_9256d3b_844b_ succeeds.
  • versionLT 9.10.1-216.va_9256d3b_844b_ 9.10-211.v7d13903b_a_d89 fails.

Testing

  • env TEST_SUITES=./tests/functions.bats BATS_FLAGS='--filter versionLT' make test-debian_jdk21 (versionLT passed)
  • bash -n jenkins-support (passed)
  • Compare-VersionLessThan PowerShell matrix for the same issue examples (passed)

@oleksii-tumanov
oleksii-tumanov requested a review from a team as a code owner July 18, 2026 09:09
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.

tr '-' '.' conflates semver and build number in versionLT() of jenkins-support script

1 participant