Bug 996120 - The bonding params need to be verified and adjusted if needed to prevent serious network degredations
Summary: The bonding params need to be verified and adjusted if needed to prevent seri...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-node
Version: 3.2
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: ---
Assignee: Fabian Deutsch
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1001468
TreeView+ depends on / blocked
 
Reported: 2013-08-12 12:53 UTC by Fabian Deutsch
Modified: 2013-11-28 11:56 UTC (History)
7 users (show)

Fixed In Version: ovirt-node-3.0.3
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1001468 (view as bug list)
Environment:
Last Closed: 2013-11-28 11:56:49 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 18925 0 None None None Never

Description Fabian Deutsch 2013-08-12 12:53:42 UTC
Description of problem:
"""
	It is critical that either the miimon or arp_interval and
arp_ip_target parameters be specified, otherwise serious network
degradation will occur during link failures.  Very few devices do not
support at least miimon, so there is really no reason not to use it.
"""
source: https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/networking/bonding.txt#n153

To prevent this case we should check and adjust the arguments if needed during ai and the tui install.
Currently both ways to configure the device don't do any checks on the params and don't even require any params to be set.

Comment 1 Mike Burns 2013-08-15 14:27:28 UTC
IMO (and feel free to argue):

Auto-install:  if the user specifies something for the parameters, then don't change anything.  If they don't pass any parameters, set some sane default.  We generally expect that people know what they're doing with autoinstall

TUI:  when creating a new bond, add default options, but allow the user to change/remove.  If they're editing, don't suddenly add options that weren't there before.

Comment 2 Fabian Deutsch 2013-09-06 11:08:20 UTC
I agree on this position.


Note You need to log in before you can comment on or make changes to this bug.