Merge ~b-candler/cloud-init:bridge-parameters into cloud-init:master
Proposed by
Brian Candler
Status: | Needs review | ||||
---|---|---|---|---|---|
Proposed branch: | ~b-candler/cloud-init:bridge-parameters | ||||
Merge into: | cloud-init:master | ||||
Diff against target: |
13 lines (+1/-1) 1 file modified
cloudinit/net/network_state.py (+1/-1) |
||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
cloud-init Commiters | Pending | ||
Review via email: mp+385242@code.launchpad.net |
Commit message
Fixes KeyError for bridge with no "parameters:" setting
LP #1879673
Description of the change
Commit ded1ec81 introduced a regression whereby a bridge with no "parameters:" setting caused a KeyError exception.
To post a comment you must log in.
Note: commit ded1ec81 did include a test case, but didn't test the case where no "parameters:" were provided. /git.launchpad. net/cloud- init/commit/ ?id=ded1ec81
https:/
I don't have an environment set up to be able to run the test suite for cloud-init, but someone might want to add the missing test case.