|
 |
33cfa9 |
From 953da73c7f21c85f68f9316d33c76643fee2a7dd Mon Sep 17 00:00:00 2001
|
|
|
33cfa9 |
From: Marius Vollmer <mvollmer@redhat.com>
|
|
|
33cfa9 |
Date: Wed, 2 Nov 2016 14:44:00 +0200
|
|
|
33cfa9 |
Subject: [3/7] networkmanager: Always set REORDER_HDR flag for new VLAN
|
|
|
33cfa9 |
devices
|
|
|
33cfa9 |
|
|
|
33cfa9 |
https://bugzilla.redhat.com/show_bug.cgi?id=1390605
|
|
|
33cfa9 |
|
|
|
33cfa9 |
Upstream commit:
|
|
|
33cfa9 |
8ceb7172b80efba8e2d5269be078f0e65559b091
|
|
|
33cfa9 |
networkmanager: Always set REORDER_HDR flag for new VLAN devices
|
|
|
33cfa9 |
|
|
|
33cfa9 |
Closes #5363
|
|
|
33cfa9 |
Signed-off-by: Stef Walter <stefw@redhat.com>
|
|
|
33cfa9 |
* Signing off inclusion in the rhel-7.3.1 branch
|
|
|
33cfa9 |
---
|
|
|
33cfa9 |
dist/networkmanager/network.js | Bin 132949 -> 132973 bytes
|
|
|
33cfa9 |
dist/networkmanager/network.js.map | Bin 1028731 -> 1029014 bytes
|
|
|
33cfa9 |
pkg/networkmanager/interfaces.js | 2 ++
|
|
|
33cfa9 |
test/verify/check-networking | 5 +++++
|
|
|
33cfa9 |
4 files changed, 7 insertions(+)
|
|
|
33cfa9 |
|
|
|
33cfa9 |
diff --git a/dist/networkmanager/network.js b/dist/networkmanager/network.js
|
|
|
33cfa9 |
index 2cdd210a34666121baf778b40b08a492e131861b..ac346740af42b474d67e169402a8147b6a46708f 100644
|
|
|
33cfa9 |
GIT binary patch
|
|
|
33cfa9 |
delta 42
|
|
|
33cfa9 |
wcmccG$MLq0V?(5ZxK5siQdv%7o|2AIT25kmF_11*(lKm~P}m-!!1(tW06tp|vj6}9
|
|
|
33cfa9 |
|
|
|
33cfa9 |
delta 18
|
|
|
33cfa9 |
ZcmaFc$8oifV?(4ubD_faLIuV@&j3${2v-0A
|
|
|
33cfa9 |
|
|
|
33cfa9 |
diff --git a/dist/networkmanager/network.js.map b/dist/networkmanager/network.js.map
|
|
|
33cfa9 |
index b037e826b92c6599f15c6c719cf933b98d53936b..dc53b3f715bd55e48472b2325ddf21fa1d556ee4 100644
|
|
|
33cfa9 |
GIT binary patch
|
|
|
33cfa9 |
delta 766
|
|
|
33cfa9 |
zcmezU#BSPiy9u!(X2wZoMuvu_7KVmqmd1w3sTRp58w+RTaB91iIP3U3X1Q(_$}MLR
|
|
|
33cfa9 |
zNq2H|b+mBOadmWb)G2jxba%}0-aMgj6Dy
|
|
|
33cfa9 |
zw07OBV$U)sXRsa@oj{<@{30ivK#$U5PhXv2N5|kiAQxz0po@;XXW8_AZAMXPGq{`=
|
|
|
33cfa9 |
zR0^md*h9zN+kE?eZANoRnc@
|
|
|
33cfa9 |
zA~ikR7gd3KU|xYAs#|p|oP#P0{2=PUo&dQsxC%pEda+Z8Hip7b{pk}XGKxqQ7NQve
|
|
|
33cfa9 |
zb!>Ri_I(o>cSy=uz!X#%qWLGXYWt+sjK04av$sE3#$-@5T|kVP#a>@uLETVYA+uPa
|
|
|
33cfa9 |
zBqLQJB{eOvG^a#IA*r-Pp*&R~FEuqqp(J0SIJHC}vqT}aq97+TIkP0EQXw&~vOKX;
|
|
|
33cfa9 |
zZ@Qp3v)FWf31%+Qw4B8B;us|z1vpSIRi7TPo>gi3gcmF<(;e8DIj8p@=HzX+mShHE
|
|
|
33cfa9 |
z79eH?Vm2UV2V#!x){>kbU9hI1>4hRZLDTKlF$r$JBEqwnX}en@r+(7(2ip9U`{3ds
|
|
|
33cfa9 |
rPB{szj@MBzoc?eox9s!_D>ymZfldcvE+FOxVjdvo-QItg@69^^8|3Zo
|
|
|
33cfa9 |
|
|
|
33cfa9 |
delta 595
|
|
|
33cfa9 |
zcmbRC-0t@iy9u!(<|Y<K28pJr$tFq0M&>DINhxWT8w+RTaGJT5IP3U3>bq
|
|
|
33cfa9 |
z&vbHhcP#MSJhyNYE2HV=fbv^Rj7FQatBf@nO((}MUddRznR7{6JClCV^g?Y$QMbZk
|
|
|
33cfa9 |
zPhXv2N5|m2A}1YJN5?=H9e2;7V$U)sXAsxXMJEtQ=6mS`dl|u%dO=kJ6$E?exO*FK
|
|
|
33cfa9 |
zKcmfPE-8~=0yQkB3}zTqiLd!~c~{1-#xjK!UJ&~nQO)o#$?~19=*p
|
|
|
33cfa9 |
zmM^M0_rR
|
|
|
33cfa9 |
z5$c8TyzQqaGVYL+F#)RpDyT3-^JQe&_W7$BeSb5iZ-2Xt$)IR@ff%#kbcNGwtkd5!
|
|
|
33cfa9 |
zGILB{b%c|5`gbE{zUc*$%xu$79^tfXmy==!Viq7~1!6WJW(Q)9?Q&9_OI)x<`}7`B
|
|
|
33cfa9 |
zo}lU8>zD+$zZd0M%(VSZBBy@R^m~VZh6~*1Vwrwn4JSwY$|IaW%mu{UK+FTgyxUhE
|
|
|
33cfa9 |
I;d}cI00B+B-T(jq
|
|
|
33cfa9 |
|
|
|
33cfa9 |
diff --git a/pkg/networkmanager/interfaces.js b/pkg/networkmanager/interfaces.js
|
|
|
33cfa9 |
index 9c2cf43..fa95d9c 100644
|
|
|
33cfa9 |
--- a/pkg/networkmanager/interfaces.js
|
|
|
33cfa9 |
+++ b/pkg/networkmanager/interfaces.js
|
|
|
33cfa9 |
@@ -743,6 +743,8 @@ function NetworkManagerModel() {
|
|
|
33cfa9 |
set("vlan", "parent", 's', settings.vlan.parent);
|
|
|
33cfa9 |
set("vlan", "id", 'u', settings.vlan.id);
|
|
|
33cfa9 |
set("vlan", "interface-name", 's', settings.vlan.interface_name);
|
|
|
33cfa9 |
+ // '1' is the default, but we need to set it explicitly anyway.
|
|
|
33cfa9 |
+ set("vlan", "flags", 'u', 1);
|
|
|
33cfa9 |
}
|
|
|
33cfa9 |
|
|
|
33cfa9 |
if (settings.ethernet) {
|
|
|
33cfa9 |
diff --git a/test/verify/check-networking b/test/verify/check-networking
|
|
|
33cfa9 |
index 1133eec..711cd0c 100755
|
|
|
33cfa9 |
--- a/test/verify/check-networking
|
|
|
33cfa9 |
+++ b/test/verify/check-networking
|
|
|
33cfa9 |
@@ -396,6 +396,11 @@ class TestNetworking(MachineCase):
|
|
|
33cfa9 |
b.click(".panel-heading .btn:contains('On')")
|
|
|
33cfa9 |
b.wait_not_in_text("tr:contains('Status')", "Inactive")
|
|
|
33cfa9 |
|
|
|
33cfa9 |
+ # Check that the actual kernel device has the REORDER_HDR flag
|
|
|
33cfa9 |
+ # set. NetworkManager stopped doing that for connections
|
|
|
33cfa9 |
+ # created via D-Bus at some point.
|
|
|
33cfa9 |
+ self.assertIn("REORDER_HDR", m.execute("ip -d link show tvlan | grep vlan"))
|
|
|
33cfa9 |
+
|
|
|
33cfa9 |
# Delete it
|
|
|
33cfa9 |
b.click("#network-interface button:contains('Delete')")
|
|
|
33cfa9 |
b.wait_visible("#networking")
|
|
|
33cfa9 |
--
|
|
|
33cfa9 |
2.5.5
|
|
|
33cfa9 |
|