Skip to content

Resort queue after adding new task#4731

Merged
happz merged 12 commits intomainfrom
queue-ordering
Mar 31, 2026
Merged

Resort queue after adding new task#4731
happz merged 12 commits intomainfrom
queue-ordering

Conversation

@happz
Copy link
Copy Markdown
Contributor

@happz happz commented Mar 23, 2026

Adding concept of ordering to queue tasks. Most of the time, tasks will inherit their order from phases they represent. This will allow reordering of queue after adding new tasks while the queue is already running - today, the queue is sorted at the beginning, because steps already add tasks in the order of phases they represent.

Related to #4726.

Pull Request Checklist

  • implement the feature
  • extend the test coverage

@happz happz added this to planning Mar 23, 2026
@happz happz added code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. ci | full test Pull request is ready for the full test execution labels Mar 23, 2026
@github-project-automation github-project-automation bot moved this to backlog in planning Mar 23, 2026
@happz happz moved this from backlog to implement in planning Mar 23, 2026
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

Gemini is experiencing higher than usual traffic and was unable to create the review. Please try again in a few hours by commenting /gemini review.

@happz happz marked this pull request as ready for review March 23, 2026 22:38
@happz happz moved this from implement to review in planning Mar 23, 2026
@LecrisUT LecrisUT self-assigned this Mar 24, 2026
@LecrisUT LecrisUT self-requested a review March 24, 2026 16:31
Comment thread tmt/steps/__init__.py Outdated
Comment thread tmt/queue.py
Comment thread tmt/queue.py Outdated
Comment thread tmt/queue.py Outdated
Comment thread tmt/queue.py Outdated
Comment thread tmt/queue.py Outdated
Comment thread tmt/queue.py Outdated
Comment thread tmt/queue.py Outdated
Comment thread tmt/queue.py
@thrix thrix added this to the 1.71 milestone Mar 27, 2026
Comment thread tmt/queue.py Outdated
@tcornell-bus tcornell-bus self-assigned this Mar 27, 2026
Comment thread tmt/steps/__init__.py
Comment thread tmt/queue.py
Comment thread tests/unit/test_queue.py Outdated
Comment thread tmt/queue.py Outdated
@happz
Copy link
Copy Markdown
Contributor Author

happz commented Mar 29, 2026

Tests are failing because of #4731 (comment) - once the dust settles on logging format, I will fix the tests.

Comment thread tmt/queue.py Outdated
Comment thread tests/unit/test_queue.py
@thrix thrix requested a review from tcornell-bus March 30, 2026 14:28
@happz happz moved this from review to merge in planning Mar 30, 2026
happz added 12 commits March 30, 2026 19:46
Adding concept of ordering to queue tasks. Most of the time, tasks will
inherit their order from phases they represent. This will allow
reordering of queue after adding new tasks while the queue is already
running - today, the queue is sorted at the beginning, because steps
already add tasks in the order of phases they represent.

Related to #4726.
@happz happz merged commit a18f805 into main Mar 31, 2026
38 of 40 checks passed
@happz happz deleted the queue-ordering branch March 31, 2026 08:20
@github-project-automation github-project-automation bot moved this from merge to done in planning Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci | full test Pull request is ready for the full test execution code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way.

Projects

Status: done

Development

Successfully merging this pull request may close these issues.

4 participants