summaryrefslogtreecommitdiff
path: root/wla-redhat-Fix-command-sequence-for-restarting-net-inter.patch
diff options
context:
space:
mode:
Diffstat (limited to 'wla-redhat-Fix-command-sequence-for-restarting-net-inter.patch')
-rw-r--r--wla-redhat-Fix-command-sequence-for-restarting-net-inter.patch57
1 files changed, 57 insertions, 0 deletions
diff --git a/wla-redhat-Fix-command-sequence-for-restarting-net-inter.patch b/wla-redhat-Fix-command-sequence-for-restarting-net-inter.patch
new file mode 100644
index 0000000..2fafa98
--- /dev/null
+++ b/wla-redhat-Fix-command-sequence-for-restarting-net-inter.patch
@@ -0,0 +1,57 @@
+From 6d452b28383c59a8193595daaca13fa8db9338a1 Mon Sep 17 00:00:00 2001
+From: Mohammed Gamal <mgamal@redhat.com>
+Date: Wed, 22 Jun 2022 13:36:07 +0200
+Subject: [PATCH] redhat: Fix command sequence for restarting net interface
+
+RH-Author: Mohamed Gamal Morsy <mmorsy@redhat.com>
+RH-MergeRequest: 2: redhat: Fix command sequence for restarting net interface
+RH-Commit: [1/1] e838ff196b8823f64e08247b3a4dac9d60e5ed72
+RH-Bugzilla: 2098233
+RH-Acked-by: Vitaly Kuznetsov <vkuznets@redhat.com>
+RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
+
+Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2098233
+
+Apparently the down and up commands need to be run in the same command, so
+connect them together with "&&" operator. Also re-implement restart_if ot handle
+warnings same wat as other distros
+
+Signed-off-by: Mohammed Gamal <mgamal@redhat.com>
+---
+ azurelinuxagent/common/osutil/redhat.py | 22 ++++++++++------------
+ 1 file changed, 10 insertions(+), 12 deletions(-)
+
+diff --git a/azurelinuxagent/common/osutil/redhat.py b/azurelinuxagent/common/osutil/redhat.py
+index a02647cd..5c397ae8 100644
+--- a/azurelinuxagent/common/osutil/redhat.py
++++ b/azurelinuxagent/common/osutil/redhat.py
+@@ -147,16 +147,14 @@ class RedhatOSUtil(Redhat6xOSUtil):
+ """
+ Restart an interface by bouncing the link.
+ """
+- retry_limit=retries+1
++ retry_limit = retries + 1
+ for attempt in range(1, retry_limit):
+- try:
+- shellutil.run_command(["ip", "link", "set", ifname, "down"])
+- shellutil.run_command(["ip", "link", "set", ifname, "up"])
+-
+- except shellutil.CommandError as cmd_err:
+- logger.warn("failed to restart {0}: return code {1}".format(ifname, cmd_err.returncode))
+- if attempt < retry_limit:
+- logger.info("retrying in {0} seconds".format(wait))
+- time.sleep(wait)
+- else:
+- logger.warn("exceeded restart retries")
++ return_code = shellutil.run("ip link set {0} down && ip link set {0} up".format(ifname), expected_errors=[1] if attempt < retries else [])
++ if return_code == 0:
++ return
++ logger.warn("failed to restart {0}: return code {1}".format(ifname, return_code))
++ if attempt < retry_limit:
++ logger.info("retrying in {0} seconds".format(wait))
++ time.sleep(wait)
++ else:
++ logger.warn("exceeded restart retries")
+--
+2.31.1
+