Blame SOURCES/wla-redhat-Fix-command-sequence-for-restarting-net-inter.patch

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