Merge ~philroche/maas-images:bugfix/esm-eol-keyerror into maas-images:master
Status: | Merged |
---|---|
Merge reported by: | Andres Rodriguez |
Merged at revision: | a45f3e3c8d3bde939091d8b3129afb97387b1372 |
Proposed branch: | ~philroche/maas-images:bugfix/esm-eol-keyerror |
Merge into: | maas-images:master |
Diff against target: |
31 lines (+8/-2) 1 file modified
meph2/ubuntu_info.py (+8/-2) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Andres Rodriguez (community) | Approve | ||
Robert C Jennings (community) | Approve | ||
Review via email: mp+366911@code.launchpad.net |
Commit message
ESM EOL code must support running on systems with older versions of python-distro-info
This was discovered when running this on Ubuntu Precise with python-distro-info 0.8.2ubuntu1
and distro-info-data 0.8ubuntu0.17
When running on a system like this the _rows attribute is used and as such the dict
key for ESM EOL is eol-esm and not eol_esm as it is on newer systems where we use a
dict representation of DistroRelease objects instead.
Description of the change
ESM EOL code must support running on systems with older versions of python-distro-info
This was discovered when running this on Ubuntu Precise with python-distro-info 0.8.2ubuntu1
and distro-info-data 0.8ubuntu0.17
When running on a system like this the _rows attribute is used and as such the dict
key for ESM EOL is eol-esm and not eol_esm as it is on newer systems where we use a
dict representation of DistroRelease objects instead.
Following on from MP https:/ /code.launchpad .net/~philroche /maas-images/ +git/maas- images/ +merge/ 366852
I am opening this MP to add another fix for issue found in https:/ /cloud- images- jenkins. canonical. com/job/ MAAS_v2_ MasterBuilder/ ARCH_TYPE= amd64/1442/ console
``` codename] ['eol_esm' ]:
10:51:54 if hack_all[
10:51:54 KeyError: 'eol_esm'
```
I could not replicate this locally but could when testing on a 12.04 system with ESM.