Merge ~bjornt/maas:bug-1932136-disconnected-interfaces-enabled into maas:master
Proposed by
Björn Tillenius
Status: | Merged |
---|---|
Approved by: | Björn Tillenius |
Approved revision: | b08b26442b6241b70c25099535634a1663576a69 |
Merge reported by: | MAAS Lander |
Merged at revision: | not available |
Proposed branch: | ~bjornt/maas:bug-1932136-disconnected-interfaces-enabled |
Merge into: | maas:master |
Diff against target: |
102 lines (+15/-9) 2 files modified
src/metadataserver/builtin_scripts/network.py (+8/-8) src/metadataserver/builtin_scripts/tests/test_network.py (+7/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alberto Donato (community) | Approve | ||
MAAS Lander | unittests | Pending | |
Review via email: mp+404391@code.launchpad.net |
Commit message
LP #1932136: interface with a warning is not configured properly
Now all interfaces are considered enabled, since an admin should
explicitly disable an interface if he doesn't want it configured.
I also noted that we never set link_connected, so all interfaces were
considered connected. Now we set link_connected to False if there's no
link.
To post a comment you must log in.
nice, +1
one comment inline