diff options
author | Jake Hunsaker <jhunsake@redhat.com> | 2018-06-23 18:55:30 -0400 |
---|---|---|
committer | Bryn M. Reeves <bmr@redhat.com> | 2018-06-25 11:42:14 +0100 |
commit | 57d515a2fdcc0b5ed9536d4ab622dc291cfab3c7 (patch) | |
tree | 30b7ba24e6516e6e96d371c58dffd63088e0719c | |
parent | fb726230bfdda14fcf2f26eae748cf94df08493f (diff) | |
download | sos-57d515a2fdcc0b5ed9536d4ab622dc291cfab3c7.tar.gz |
[redhat] Fix RHELPolicy.dist_version()
The RHELPolicy dist_version() was always returning False due to an
invalid call to all_pkgs_by_name_regex(). Correct this call and use the
discovered redhat-release package to determine version.
Resolves: #216
Signed-off-by: Jake Hunsaker <jhunsake@redhat.com>
-rw-r--r-- | sos/policies/redhat.py | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/sos/policies/redhat.py b/sos/policies/redhat.py index f12cb847..cfbf7808 100644 --- a/sos/policies/redhat.py +++ b/sos/policies/redhat.py @@ -65,11 +65,11 @@ class RedHatPolicy(LinuxPolicy): self.valid_subclasses = [RedHatPlugin] - pkgs = self.package_manager.all_pkgs() + self.pkgs = self.package_manager.all_pkgs() files = self.package_manager.all_files() # If rpm query failed, exit - if not pkgs: + if not self.pkgs: print("Could not obtain installed package list", file=sys.stderr) sys.exit(1) @@ -79,7 +79,7 @@ class RedHatPolicy(LinuxPolicy): manager", file=sys.stderr) sys.exit(1) - self.usrmove = self.check_usrmove(pkgs) + self.usrmove = self.check_usrmove(self.pkgs) if self.usrmove: self.PATH = "/usr/sbin:/usr/bin:/root/bin" @@ -282,9 +282,8 @@ No changes will be made to system configuration. def dist_version(self): try: - pkg = self.pkg_by_name("redhat-release") or \ - self.all_pkgs_by_name_regex("redhat-release-.*")[-1] - pkgname = pkg["version"] + rr = self.package_manager.all_pkgs_by_name_regex("redhat-release*") + pkgname = self.pkgs[rr[0]]["version"] if pkgname[0] == "4": return 4 elif pkgname[0] in ["5Server", "5Client"]: |