Merge lp://qastaging/~brendan-donegan/hwcert-tools/certification_blockers_exclude_future_certified into lp://qastaging/~hardware-certification/hwcert-tools/reporting-tools

Proposed by Brendan Donegan
Status: Merged
Merged at revision: 102
Proposed branch: lp://qastaging/~brendan-donegan/hwcert-tools/certification_blockers_exclude_future_certified
Merge into: lp://qastaging/~hardware-certification/hwcert-tools/reporting-tools
Diff against target: 22 lines (+7/-0)
1 file modified
certification_reports/certification_blockers.py (+7/-0)
To merge this branch: bzr merge lp://qastaging/~brendan-donegan/hwcert-tools/certification_blockers_exclude_future_certified
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
hardware-certification-devel Pending
Review via email: mp+165657@code.qastaging.launchpad.net

Description of the change

Exclude systems which have been certified successfully with a later release from the report.

This turned out to be super-easy. We already get the pass certificates to create a list of them to display on the report for each system, so we just insert a bit of logic to jump out of the main loop if the release of any of those certificates is found to be 'greater' than one we are blocked on. This takes advantage of the fact that '12.04 LTS' < '12.04.1 LTS' and so on in python :)

To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

haha :) Thanks!

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