From: Sage Weil <sage@redhat.com>
Date: Wed, 11 Dec 2019 16:11:08 +0000 (+0000)
Subject: task/kernel: drop removal of old kernels
X-Git-Tag: 1.1.0~179^2~2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=3be03ed0376b7fe5c7ea35dae2c4144c20e4cdc9;p=teuthology.git

task/kernel: drop removal of old kernels

1- this isn't necessary anymore with fog
2- the package-cleanup --oldkernels option doesn't exist in el8

Signed-off-by: Sage Weil <sage@redhat.com>
---

diff --git a/teuthology/task/kernel.py b/teuthology/task/kernel.py
index 8cd596d474..6b01a3cf33 100644
--- a/teuthology/task/kernel.py
+++ b/teuthology/task/kernel.py
@@ -1105,15 +1105,6 @@ def get_sha1_from_pkg_name(path):
     return sha1
 
 
-def remove_old_kernels(ctx):
-    for remote in ctx.cluster.remotes.keys():
-        package_type = remote.os.package_type
-        if package_type == 'rpm':
-            log.info("Removing old kernels from %s", remote)
-            args = ['sudo', 'package-cleanup', '-y', '--oldkernels']
-            remote.run(args=args)
-
-
 def task(ctx, config):
     """
     Make sure the specified kernel is installed.
@@ -1218,8 +1209,6 @@ def task(ctx, config):
     need_version = {}  # utsrelease or sha1
     kdb = {}
 
-    remove_old_kernels(ctx)
-
     for role, role_config in config.items():
         # gather information about this remote
         (role_remote,) = ctx.cluster.only(role).remotes.keys()