Bug 36339

Summary: Bonding configuration generates wrong miimon configuration
Product: UCS Reporter: Janis Meybohm <meybohm>
Component: UMC - Basic settingsAssignee: Florian Best <best>
Status: CLOSED FIXED QA Contact: Philipp Hahn <hahn>
Severity: critical    
Priority: P5 CC: gohmann, grandjean, hahn, jmm, stephan.hendl
Version: UCS 3.2Flags: meybohm: Patch_Available+
Target Milestone: UCS 3.2-4-errata   
Hardware: Other   
OS: Linux   
What kind of report is it?: --- What type of bug is this?: ---
Who will be affected by this bug?: --- How will those affected feel about the bug?: ---
User Pain: Enterprise Customer affected?:
School Customer affected?: ISV affected?:
Waiting Support: Flags outvoted (downgraded) after PO Review:
Ticket number: Bug group (optional):
Max CVSS v3 score:
Bug Depends on:    
Bug Blocks: 36340    
Attachments: Fix UMC wizard and correct UCR variables (untested)

Description Janis Meybohm univentionstaff 2014-10-29 15:21:13 CET
Created attachment 6254 [details]
Fix UMC wizard and correct UCR variables (untested)

2014102821000257

The UMC module does not set MII link monitoring frequency correctly:

interfaces/bond0/options/2: miimon 100

should be:

interfaces/bond0/options/2: bond-miimon 100

Therefore, in bond-modes like active-backup, the active slave is _never_ switched so the network is down when the active slave goes down!


A fix should also correct all broken configurations in UCR.

Workaround:
1) Set "bond-miimon <Frequency in milliseconds>" manually via "Additional bonding options" in UMC wizard.
BUT: this would need to be manually removed when this bug is fixed because ifup complains about duplicate options

/etc/network/interfaces:39: duplicate option
ifup: couldn't read interfaces file "/etc/network/interfaces"


2) Correct UCR manually:
ucr search --brief --non-empty '^interfaces/.*/options/' | \
  sed -rne 's,(interfaces/.*/options/.*): (miimon .*)$,\1=bond-\2,p' | \
  xargs -d '\n' --no-run-if-empty ucr set

BUT: Re-running the UMC wizard for that interface will again break the configuration!
Comment 1 Florian Best univentionstaff 2014-11-21 16:55:59 CET
The patch has been adapted and applied.

# diff before_update after_update:
< interfaces/bond0/options/2: miimon 98
---
> interfaces/bond0/options/2: bond-miimon 98

# cat /sys/class/net/bond0/bonding/miimon 
98
Comment 2 Philipp Hahn univentionstaff 2014-11-28 07:41:25 CET
OK: r56048 r56047
OK: aptitude install '?source-package(univention-system-setup)?installed'
OK: cat /etc/network/interfaces
OK: cat /sys/class/net/bond0/bonding/miimon
OK: 2014-11-21-univention-system-setup.yaml
FIXED: announce_errata.pmh -V 2014-11-21-univention-system-setup.yaml
 wrap, "MII monitoring", German → r56267
Comment 3 Moritz Muehlenhoff univentionstaff 2014-12-03 14:49:22 CET
http://errata.univention.de/ucs/3.2/246.html