diff options
| author |   Brent Eagles <beagles@redhat.com> | 2016-08-12 18:07:24 -0230 |
|---|---|---|
| committer | Brent Eagles <beagles@redhat.com> | 2016-08-25 21:57:28 -0230 |
| commit | 6298b7c35ed4a7411d8e8da1008efc0f585ffb75 (patch) | |
| tree | 8aa3a38c638fe5764f6605b7f5a0d16d0fa550fd /net-config-bond.yaml | |
| parent | c58ed3b63e0c3bf56f58553af31afe5018f31695 (diff) | |
Add constraint to prohibit balance-tcp from BondInterfaceOvsOptions
This patch adds an allowed_pattern contraint that uses a negative
lookahead assertion to only allow options strings that do not contain
the 'balance-tcp' option.
Change-Id: Icf8874e4e585f9a42d38091f8b38c3685f403cf1
Partial-Bug: #1612786
Diffstat (limited to 'net-config-bond.yaml')
| -rw-r--r-- | net-config-bond.yaml | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/net-config-bond.yaml b/net-config-bond.yaml index 0a162e77..ec881bdc 100644 --- a/net-config-bond.yaml +++ b/net-config-bond.yaml @@ -1,4 +1,4 @@ -heat_template_version: 2015-04-30 +heat_template_version: 2016-10-14 description: > Software Config to drive os-net-config with 2 bonded nics on a bridge. @@ -6,9 +6,15 @@ description: > parameters: BondInterfaceOvsOptions: default: '' - description: The ovs_options string for the bond interface. Set things like - lacp=active and/or bond_mode=balance-slb using this option. + description: | + The ovs_options string for the bond interface. Set things like + lacp=active and/or bond_mode=balance-slb using this option. type: string + constraints: + - allowed_pattern: "^((?!balance.tcp).)*$" + description: | + The balance-tcp bond mode is known to cause packet loss and + should not be used in BondInterfaceOvsOptions. ControlPlaneIp: default: '' description: IP address/subnet on the ctlplane network |