diff options
| author |   Frank A. Zdarsky <fzdarsky@redhat.com> | 2017-01-05 12:55:55 +0100 |
|---|---|---|
| committer | Frank A. Zdarsky <fzdarsky@redhat.com> | 2017-01-05 12:55:55 +0100 |
| commit | 8c841946d441d6c90ba61ee0a64698b59c711c3a (patch) | |
| tree | 2c604939738d97f88e04f779aa372b9d4c4a0ceb /etc | |
| parent | d929214c7a904da57a406dd56ac06f001fdc7d7b (diff) | |
Add check that ovs_extra is passed as list
This patch adds a check that ensures the ovs_extra option, if present,
is passed in as list and raises an InvalidConfigException if not.
It addresses the issue that a user may mistakingly pass the value as
string, which would cause an error later when appending the failure
mode or when formatting the ovs_extra parameter.
Note: Also fixes a sample file in which ovs_extra was passed as string.
Change-Id: I9e8e47390b63d284de10d27b1db2c2cc54c86924
Closes-Bug: #1654196
Diffstat (limited to 'etc')
| -rw-r--r-- | etc/os-net-config/samples/ovs_patch_port.yaml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/etc/os-net-config/samples/ovs_patch_port.yaml b/etc/os-net-config/samples/ovs_patch_port.yaml index 91858be..bae8880 100644 --- a/etc/os-net-config/samples/ovs_patch_port.yaml +++ b/etc/os-net-config/samples/ovs_patch_port.yaml @@ -10,7 +10,7 @@ network_config: # force the MAC address of the bridge to this interface primary: true mtu: 1500 - ovs_extra: "br-set-external-id br-ctlplane bridge-id br-ctlplane" + ovs_extra: ["br-set-external-id br-ctlplane bridge-id br-ctlplane"] - type: ovs_patch_port name: br_pub-patch |