Skip to content

fix: Use same sort keys for listings of trees on HardwareDetails page.#1908

Merged
gustavobtflores merged 1 commit into
kernelci:mainfrom
profusion:fix/treesort-hardware-details
May 18, 2026
Merged

fix: Use same sort keys for listings of trees on HardwareDetails page.#1908
gustavobtflores merged 1 commit into
kernelci:mainfrom
profusion:fix/treesort-hardware-details

Conversation

@alanpeixinho
Copy link
Copy Markdown
Contributor

There were instances of sorting the lists of trees in the hardware
page using a different set of keys (due to mistake on the performance
improvements of #1832).
This case might have not triggered, because the sorting provided by
the first 2 keys tree_name/branch, might suffice for basically most of
the comparisons.

@alanpeixinho alanpeixinho marked this pull request as ready for review May 18, 2026 18:00
  There were instances of sorting the lists of trees in the hardware
  page using a different set of keys (due to mistake on the performance
  improvements of kernelci#1832).
  This case might have not triggered, because the sorting provided by
  the first 2 keys tree_name/branch, might suffice for basically most of
  the comparisons.
  * All of them are now using the keys (in order of importance):
    1. tree_name
    2. branch,
    3. git_repository_url
    4. git_commit_hash

Signed-off-by: Alan Peixinho <alan.peixinho@profusion.mobi>
@alanpeixinho alanpeixinho force-pushed the fix/treesort-hardware-details branch from 0080ffc to d7c069b Compare May 18, 2026 18:04
Copy link
Copy Markdown
Contributor

@gustavobtflores gustavobtflores left a comment

Choose a reason for hiding this comment

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

Looks good

@gustavobtflores gustavobtflores added Backend Most or all of the changes for this issue will be in the backend code. bug Something isn't working labels May 18, 2026
@gustavobtflores gustavobtflores added this pull request to the merge queue May 18, 2026
Merged via the queue into kernelci:main with commit 9ac1dc6 May 18, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backend Most or all of the changes for this issue will be in the backend code. bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wrong sorting of trees on hardware details.

2 participants