Merge lp://staging/~wido/cloud-init/cloudstack-dhcp into lp://staging/~cloud-init-dev/cloud-init/trunk

Proposed by Dan Watkins
Status: Merged
Merged at revision: 1213
Proposed branch: lp://staging/~wido/cloud-init/cloudstack-dhcp
Merge into: lp://staging/~cloud-init-dev/cloud-init/trunk
Diff against target: 13 lines (+2/-1)
1 file modified
cloudinit/sources/DataSourceCloudStack.py (+2/-1)
To merge this branch: bzr merge lp://staging/~wido/cloud-init/cloudstack-dhcp
Reviewer Review Type Date Requested Status
Wido den Hollander (community) Approve
cloud-init Commiters Pending
Review via email: mp+293379@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Wido den Hollander (wido) wrote :

I run this patch in production on our CloudStack environment which also has IPv6 with DHCPv6 enabled. Previous to this patch cloud-init would fail and with this patch it works.

review: Approve
Revision history for this message
Scott Moser (smoser) wrote :

i really hate \ at the end of the line.
would have preferred

if (file_name.startswith("dhclient.") and
        (file_name.endswith(".lease") or file_name.endswith(".leases"))):

oh well.

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.