Merge lp://staging/~stylesen/lava-scheduler/review-take-1 into lp://staging/lava-scheduler/multinode

Proposed by Senthil Kumaran S
Status: Merged
Approved by: Neil Williams
Approved revision: 287
Merged at revision: 285
Proposed branch: lp://staging/~stylesen/lava-scheduler/review-take-1
Merge into: lp://staging/lava-scheduler/multinode
Diff against target: 1036 lines (+293/-550)
11 files modified
lava_scheduler_app/api.py (+2/-2)
lava_scheduler_app/management/commands/schedulermonitor.py (+1/-1)
lava_scheduler_app/models.py (+8/-1)
lava_scheduler_app/templates/lava_scheduler_app/job_sidebar.html (+2/-2)
lava_scheduler_app/utils.py (+50/-49)
lava_scheduler_app/views.py (+2/-2)
lava_scheduler_daemon/board.py (+0/-355)
lava_scheduler_daemon/dbjobsource.py (+2/-76)
lava_scheduler_daemon/job.py (+204/-6)
lava_scheduler_daemon/service.py (+21/-55)
lava_scheduler_daemon/tests/test_board.py (+1/-1)
To merge this branch: bzr merge lp://staging/~stylesen/lava-scheduler/review-take-1
Reviewer Review Type Date Requested Status
Neil Williams Approve
Review via email: mp+182635@code.staging.launchpad.net

Description of the change

Addressed all review comments from Antonio and removed all legacy code which is no longer required.

To post a comment you must log in.
Revision history for this message
Neil Williams (codehelp) wrote :

Thanks Senthil - tested on multinode.v.l.o & approved.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: