Bug #13092

802.3ad bonding setup doesn't work correctly

Added by Bryan Kearney over 1 year ago. Updated 3 days ago.

Status:Ready For Testing
Priority:Normal
Assigned To:Ondřej Pražák
Category:Templates
Target version:Team Marek Iteration 16
Difficulty: Bugzilla link:1288420
Found in release: Pull request:https://github.com/theforeman/foreman/pull/4583
Story points-
Velocity based estimate-

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1288420
Description of problem:
Creating a bonded interface with 802.3ad in Satellite 6.1.3 does not deploy correctly. We expect to see a bond0 interface with a number of bond0.101, bond0.102 etc interfaces.

We try to create a bond0 which incorporates two devices, enp12s0f1 and enp22s0. Bonding options are deployed correctly. We use the MAC of the first interface, enp12s0f1 in the Sat 6 interface. When the host comes up after re-provisioning it sometimes creates a ifcfg-enp12s0f1.101 file or sometimes a ifcfg-.101 file.

Version-Release number of selected component (if applicable):
Satellite 6.1.3

Actual results:
The host comes up after re-provisioning it sometimes creates a ifcfg-enp12s0f1.101 file or sometimes a ifcfg-.101 file.

Expected results:
The host to have the configured bond options

Additional info:
We are wanting to deploy an OSP 7 environment using these network bonds so this will shortly be a blocker in doing so.

Customer worked around the issue by doing:
1) Making up a MAC address for the bond interface, something entirely bogus and then re-using that same MAC for the VLAN interfaces rendered us a kickstart that looks better.

Making up a MAC adress got us the bond created but then, adding VLAN:s on top of the bond fails as the snippet is doing a grep for the made-up MAC in the output of 'ip -l'

Next workaround was to acutally fix-up the broken config files for the VLANs

2) The workaround we used was to following patch against kickstart_network_setup

Do note that I doubt the workaround is generic enough to be used a generic solution

diff --git a/Satellite6/OSP7/templates/snippets/kickstart_network_setup_RH1544500.erb b/Satellite6/OSP7/templates/snippets/kickstart_network_setup_RH1544500.erb
index 0bf4ccc..00d2ab5 100644
--- a/Satellite6/OSP7/templates/snippets/kickstart_network_setup_RH1544500.erb
+++ b/Satellite6/OSP7/templates/snippets/kickstart_network_setup_RH1544500.erb
@@ -114,6 +114,7 @@ real=`ip -o link | grep <%= interface.mac -%> | awk '{print $2;}' | sed s/:$//`
 <% if virtual -%>
   real=`echo <%= interface.identifier -%> | sed s/<%= interface.attached_to -%>/$real/`
 <% end -%>
+real="<%= interface.identifier -%>" 

 # ifcfg files are ignored by NM if their name contains colons so we convert colons to underscore
 <% if alias_type -%>

History

#1 Updated by Marek Hulán over 1 year ago

  • Category set to Templates

#2 Updated by Marek Hulán over 1 year ago

  • Description updated (diff)

#3 Updated by Tomáš Strachota 10 months ago

  • Target version set to Team Marek backlog

#4 Updated by Marek Hulán about 1 month ago

  • Target version changed from Team Marek backlog to Team Marek Iteration 15

#5 Updated by The Foreman Bot 10 days ago

  • Status changed from New to Ready For Testing
  • Assigned To set to Ondřej Pražák
  • Pull request https://github.com/theforeman/foreman/pull/4583 added

#6 Updated by Marek Hulán 7 days ago

  • Target version changed from Team Marek Iteration 15 to Team Marek Iteration 16

Also available in: Atom PDF