Merge lp://staging/~smoser/maas-images/trunk.require-distro-info into lp://staging/maas-images

Proposed by Scott Moser
Status: Merged
Merged at revision: 372
Proposed branch: lp://staging/~smoser/maas-images/trunk.require-distro-info
Merge into: lp://staging/maas-images
Diff against target: 113 lines (+13/-66)
1 file modified
meph2/ubuntu_info.py (+13/-66)
To merge this branch: bzr merge lp://staging/~smoser/maas-images/trunk.require-distro-info
Reviewer Review Type Date Requested Status
Lee Trager (community) Approve
Review via email: mp+323748@code.staging.launchpad.net

Commit message

ubuntu_info: remove the path that did not require distro-info

At one point we had a functional path that did not require installation
of python-distro-info. At some point that path broke because we were
exporting data in REL2VER was not accounted for in the __RELEASE_DATA.
Specifically missing was 'supported'.

The primary path that was enabled by the hard coded data was testing
with a new distro info before the package was sru'd. To account for
that I've put a comment in the code showing how one can be added.

As a drive-by, this also adds the 'release_date' field to the data
exported.

To post a comment you must log in.
Revision history for this message
Lee Trager (ltrager) wrote :

LGTM!

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