Skip to content

[PR #2571/b8794efa backport][0.30] Add default descending semver ordering to collection version search - #2606

Closed
patchback[bot] wants to merge 1 commit into
0.30from
patchback/backports/0.30/b8794efa07af1786c592f62b87a79d790362a047/pr-2571
Closed

[PR #2571/b8794efa backport][0.30] Add default descending semver ordering to collection version search#2606
patchback[bot] wants to merge 1 commit into
0.30from
patchback/backports/0.30/b8794efa07af1786c592f62b87a79d790362a047/pr-2571

Conversation

@patchback

@patchback patchback Bot commented Jul 29, 2026

Copy link
Copy Markdown

This is a backport of PR #2571 as merged into main (b8794ef).

The cross-repo search endpoint (v3/plugin/ansible/search/collection-versions/)
returns versions in arbitrary order when no order_by parameter is passed.
This causes version dropdowns to display 4.7.10 before 4.7.5 (string sorting).

Adds a filter_queryset override that applies descending semver ordering
using the existing SemanticVersionOrderingFilter when no explicit ordering
is requested. Explicit order_by and full-text search ?q= ordering are
preserved.

Fixes: AAP-78435
Also addresses: AAP-56212

Tested on AAP 2.6-next using aap-dev. Verified:

  • Default (no order_by): returns correct descending semver
  • Explicit order_by=version: returns correct ascending semver
  • Explicit order_by=-version: returns correct descending semver
  • ?q= search: preserves relevance ordering

The cross-repo search endpoint returns versions in arbitrary order
when no order_by parameter is passed. Add default descending version
ordering in get_queryset so versions display newest-first. Explicit
order_by and full-text search ordering replace it.

Fixes: AAP-78435

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
(cherry picked from commit b8794ef)
@mdellweg
mdellweg force-pushed the patchback/backports/0.30/b8794efa07af1786c592f62b87a79d790362a047/pr-2571 branch from 0906201 to 9626666 Compare July 29, 2026 08:52
@mdellweg

Copy link
Copy Markdown
Member

Ohhh the change wasn't new here...

@mdellweg mdellweg closed this Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants