Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Order and filter tasks and jobs #34

Merged
merged 5 commits into from
Feb 21, 2025
Merged

Conversation

evamillan
Copy link
Collaborator

This PR adds the option to filter the list of tasks and jobs by status, both on the API and the UI. The task list is ordered by latest run by default, with currently running tasks first.

It also adds some small improvements to the UI:

  • Adds a spinner if the tasks or jobs are loading.
  • Shows the last 10 jobs run in order and indicates if there are more than those 10 runs.
  • The icon next to the last run info shows its status instead of the status of the task (which is usually 'enqueued').
  • Shows the number of failures of the task if there have been any.

- The task list can be filtered by status and ordered by the
'scheduled_at' and 'last_run' fields. The default order is by
latest run, with currently running tasks first.
- The jobs list can be filtered by 'status'. The default order
is set to '-scheduled_at'.

Signed-off-by: Eva Millán <[email protected]>
Adds tabs to filter the list of tasks by their status and
a selector to order them by last run or schedule date.

Signed-off-by: Eva Millán <[email protected]>
- Adds a spinner to show if the request to fetch the list
of tasks is loading.
- Adds a vue composable that intercepts requests and saves
their status to a 'isLoading' variable.

Signed-off-by: Eva Millán <[email protected]>
Adds tabs to the jobs list to filter them by their status.

Signed-off-by: Eva Millán <[email protected]>
- Show the last 10 jobs run in order and indicate if
  there are more than those 10 runs.
- Use an icon for the status of the latest run job instead
  of the task status.
- Show the number of failures if there have been any.
- Upgrade Vuetify's version to fix a bug with a separator
  not working properly.

Signed-off-by: Eva Millán <[email protected]>
Copy link
Collaborator

@jjmerchante jjmerchante left a comment

Choose a reason for hiding this comment

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

LGTM

@jjmerchante jjmerchante merged commit 868028d into chaoss:main Feb 21, 2025
5 checks passed
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.

2 participants