|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
From ff9c477f3238aeca9b8ca2e0805d419a449ea4d9 Mon Sep 17 00:00:00 2001
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
From: Thomas Haller <thaller@redhat.com>
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
Date: Tue, 18 Feb 2020 18:30:19 +0100
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
Subject: [PATCH] ifcfg: fix writer when changing OVS slave to clear previous
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
settings
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
We need to reset the OVS_PORT and OVS_PORT_UUID variables.
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
Otherwise, clearing the slave type doesn't work.
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
On master this is solved differently, by automatically clearing all
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
variables that are not explicitly set.
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
Reproducer:
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
nmcli con del t-eth1
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
nmcli con add type ethernet autoconnect no ifname eth1 master port0 con-name t-eth1 slave-type ovs-port
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
echo "
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
remove ovs-interface
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
remove connection.master
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
remove connection.slave-type
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
print
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
save
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
quit
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
" | nmcli c edit t-eth1
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
nmcli con show t-eth1 | grep 'ovs\|slave-type'
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
Fixes: 1440fe6a8804 ('ifcfg: don't forget master of ovs interfaces')
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
https://bugzilla.redhat.com/show_bug.cgi?id=1804167
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
(cherry picked from commit 0c8046574e782ca926d265348d10866f1653aef1)
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
---
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c | 2 ++
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
1 file changed, 2 insertions(+)
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
diff --git a/src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c b/src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
index f328cc8f0..90a1a2b8f 100644
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
--- a/src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
+++ b/src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
@@ -1923,6 +1923,8 @@ write_connection_setting (NMSettingConnection *s_con, shvarFile *ifcfg)
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
svSetValueStr (ifcfg, "BRIDGE", NULL);
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
svSetValueStr (ifcfg, "TEAM_MASTER_UUID", NULL);
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
svSetValueStr (ifcfg, "TEAM_MASTER", NULL);
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
+ svSetValueStr (ifcfg, "OVS_PORT_UUID", NULL);
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
+ svSetValueStr (ifcfg, "OVS_PORT", NULL);
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
master = nm_setting_connection_get_master (s_con);
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
if (master) {
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
--
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
2.24.1
|
|
![](https://seccdn.libravatar.org/avatar/016a5f3568d521d40c077bd1f06aa5b9eab60232808db00b9e3488bbb7477299?s=16&d=retro) |
245932 |
|