Merge lp:~ltrager/maas/lp1687500 into lp:~maas-committers/maas/trunk
Proposed by
Lee Trager
Status: | Merged |
---|---|
Approved by: | Lee Trager |
Approved revision: | no longer in the source branch. |
Merged at revision: | 6034 |
Proposed branch: | lp:~ltrager/maas/lp1687500 |
Merge into: | lp:~maas-committers/maas/trunk |
Diff against target: |
83 lines (+19/-0) 2 files modified
src/maasserver/dhcp.py (+1/-0) src/maasserver/tests/test_dhcp.py (+18/-0) |
To merge this branch: | bzr merge lp:~ltrager/maas/lp1687500 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Newell Jensen (community) | Approve | ||
Review via email: mp+323480@code.launchpad.net |
Commit message
If a StaticIPAddress has no subnet ip_is_on_vlan now returns False to prevent an AttributeError
To post a comment you must log in.
LGTM logic wise. Whether or not this is the way forward as we discussed on IRC is TBD.