Details

Reviewers

vlorentz
ardumont

Group Reviewers

Reviewers

Commits

rDSCHb3fd48e4a6d3: Add a 'full' flag argument to the pretty_print_task function
rDSCH1aadc10d2877: Add a 'task list' cli command to list tasks with search criterions
rDSCHe0f224e9d3a6: Add a SchedulerBackend.search_tasks method

Summary

Add a SchedulerBackend.search_tasks method

Add a 'full' flag argument to the pretty_print_task function

to display also the status and priority fields.

Add a 'task list' cli command to list tasks with search criterions

and not only pending tasks.

Depends on D932

Diff Detail

Repository

rDSCH Scheduling utilities

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

douardda created this revision.Jan 14 2019, 11:43 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptJan 14 2019, 11:43 AM

Harbormaster failed remote builds in B3452: Diff 2958!Jan 14 2019, 11:43 AM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DSCH/job/tox/82/
See console output for more information: https://jenkins.softwareheritage.org/job/DSCH/job/tox/82/console

douardda added a child revision: D935: Add a new 'task respawn' command.Jan 14 2019, 11:44 AM

vlorentz accepted this revision.Jan 14 2019, 12:19 PM

vlorentz added a subscriber: vlorentz.

vlorentz added inline comments.

swh/scheduler/cli.py
52 ↗	(On Diff #2958)	`full` could be documented.

This revision is now accepted and ready to land.Jan 14 2019, 12:19 PM

ardumont added a subscriber: ardumont.Jan 14 2019, 12:19 PM

ardumont added inline comments.

swh/scheduler/backend.py
350	Isn't there something missing here? where 1=1' + ' and ...

ardumont added inline comments.Jan 14 2019, 12:22 PM

swh/scheduler/backend.py
350	Nevermind, should have checked prior to submit ;) I checked and no: $ python3 >>> ' and '.join(['1 = 1', '2 = 2']) '1 = 1 and 2 = 2'

ardumont added inline comments.Jan 14 2019, 12:25 PM

swh/scheduler/cli.py
295 ↗	(On Diff #2958)	List tasks whose status ... Same for options below ;)

ardumont accepted this revision.Jan 14 2019, 12:27 PM

Add missing parts of added api endpoint server side

Harbormaster failed remote builds in B3470: Diff 2974!Jan 14 2019, 3:24 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DSCH/job/tox/87/
See console output for more information: https://jenkins.softwareheritage.org/job/DSCH/job/tox/87/console

rebased

Harbormaster failed remote builds in B3477: Diff 2981!Jan 15 2019, 11:34 AM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DSCH/job/tox/93/
See console output for more information: https://jenkins.softwareheritage.org/job/DSCH/job/tox/93/console

rebased and fix typos/wording according to comments in phab

Closed by commit rDSCHe0f224e9d3a6: Add a SchedulerBackend.search_tasks method (authored by douardda). · Explain WhyJan 15 2019, 3:05 PM

This revision was automatically updated to reflect the committed changes.

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DSCH/job/tox/99/
See console output for more information: https://jenkins.softwareheritage.org/job/DSCH/job/tox/99/console

Harbormaster failed remote builds in B3491: Diff 2992!Jan 15 2019, 3:08 PM

Add a new 'task list' cli command
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 2993

swh/scheduler/api/client.py

swh/scheduler/api/server.py

swh/scheduler/backend.py

Add a new 'task list' cli commandClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 2993

swh/scheduler/api/client.py

swh/scheduler/api/server.py

swh/scheduler/backend.py

Add a new 'task list' cli command
ClosedPublic
Actions

Revision Contents
Changeset List