Merge lp:~free.ekanayaka/charms/trusty/landscape-client/machine-id-charm-info into lp:charms/trusty/landscape-client
Proposed by
Free Ekanayaka
Status: | Superseded |
---|---|
Proposed branch: | lp:~free.ekanayaka/charms/trusty/landscape-client/machine-id-charm-info |
Merge into: | lp:charms/trusty/landscape-client |
Diff against target: |
98 lines (+25/-2) 4 files modified
hooks/common.py (+13/-0) hooks/hooks.py (+1/-0) hooks/install.py (+6/-2) hooks/test_hooks.py (+5/-0) |
To merge this branch: | bzr merge lp:~free.ekanayaka/charms/trusty/landscape-client/machine-id-charm-info |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Adam Collard | Approve | ||
Alberto Donato (community) | Approve | ||
Review via email: mp+232544@code.launchpad.net |
This proposal has been superseded by a proposal from 2014-09-02.
Description of the change
This branch adds the machine ID to the Juju info json file, so it can be grabbed by the client and sent to the server at registration. See also:
https:/
and
https:/
To post a comment you must log in.
Looks good, +1
#1:
+ return match[(len(pattern) - 1):].replace("-", "/")
Please add a comment for this line, it took me a bit to understand the reason for the replace.