From 0390e377a534f3203336fff191d9a1cab7c19014 Mon Sep 17 00:00:00 2001
From: Travis Rhoden <trhoden@redhat.com>
Date: Fri, 31 Jul 2015 20:15:22 -0700
Subject: [PATCH] [RM-12439] suse: Use Zypper Class pkg manager

Refs: #12439

Signed-off-by: Travis Rhoden <trhoden@redhat.com>
---
 ceph_deploy/hosts/__init__.py       |  4 +---
 ceph_deploy/hosts/suse/__init__.py  |  6 ++++++
 ceph_deploy/hosts/suse/install.py   | 18 +++++++++---------
 ceph_deploy/hosts/suse/uninstall.py |  5 +----
 4 files changed, 17 insertions(+), 16 deletions(-)

diff --git a/ceph_deploy/hosts/__init__.py b/ceph_deploy/hosts/__init__.py
index 297f342..6a2a2aa 100644
--- a/ceph_deploy/hosts/__init__.py
+++ b/ceph_deploy/hosts/__init__.py
@@ -69,9 +69,7 @@ def get(hostname,
     module.conn = conn
     module.machine_type = machine_type
     module.init = module.choose_init()
-    if module.normalized_name in ['fedora', 'centos', 'redhat',
-                                  'ubuntu', 'debian']:
-        module.packager = module.get_packager(module)
+    module.packager = module.get_packager(module)
     return module
 
 
diff --git a/ceph_deploy/hosts/suse/__init__.py b/ceph_deploy/hosts/suse/__init__.py
index 65b1255..e2900e7 100644
--- a/ceph_deploy/hosts/suse/__init__.py
+++ b/ceph_deploy/hosts/suse/__init__.py
@@ -4,6 +4,8 @@ from install import install, mirror_install, repo_install  # noqa
 from uninstall import uninstall  # noqa
 import logging
 
+from ceph_deploy.util import pkg_managers
+
 # Allow to set some information about this distro
 #
 
@@ -24,3 +26,7 @@ def choose_init():
         '13.1' : 'systemd',             # openSUSE_13.1
         }
     return init_mapping.get(release, 'sysvinit')
+
+
+def get_packager(module):
+    return pkg_managers.Zypper(module)
diff --git a/ceph_deploy/hosts/suse/install.py b/ceph_deploy/hosts/suse/install.py
index 5a08295..10b682e 100644
--- a/ceph_deploy/hosts/suse/install.py
+++ b/ceph_deploy/hosts/suse/install.py
@@ -1,6 +1,6 @@
 import logging
 
-from ceph_deploy.util import templates, pkg_managers
+from ceph_deploy.util import templates
 from ceph_deploy.lib import remoto
 from ceph_deploy.hosts.common import map_components
 
@@ -15,9 +15,9 @@ def install(distro, version_kind, version, adjust_repos, **kw):
         kw.get('components', [])
     )
 
-    pkg_managers.zypper_refresh(distro.conn)
-    if len(packages):
-        pkg_managers.zypper(distro.conn, packages)
+    distro.packager.clean()
+    if packages:
+        distro.packager.install(packages)
 
 
 def mirror_install(distro, repo_url, gpg_url, adjust_repos, **kw):
@@ -45,10 +45,10 @@ def mirror_install(distro, repo_url, gpg_url, adjust_repos, **kw):
         distro.conn.remote_module.write_file(
             '/etc/zypp/repos.d/ceph.repo',
             ceph_repo_content)
-        pkg_managers.zypper_refresh(distro.conn)
+        distro.packager.clean()
 
-    if len(packages):
-        pkg_managers.zypper(distro.conn, packages)
+    if packages:
+        distro.packager.install(packages)
 
 
 def repo_install(distro, reponame, baseurl, gpgkey, **kw):
@@ -92,5 +92,5 @@ def repo_install(distro, reponame, baseurl, gpgkey, **kw):
     )
 
     # Some custom repos do not need to install ceph
-    if install_ceph and len(packages):
-        pkg_managers.zypper(distro.conn, packages)
+    if install_ceph and packages:
+        distro.packager.install(packages)
diff --git a/ceph_deploy/hosts/suse/uninstall.py b/ceph_deploy/hosts/suse/uninstall.py
index 5128c09..163d891 100644
--- a/ceph_deploy/hosts/suse/uninstall.py
+++ b/ceph_deploy/hosts/suse/uninstall.py
@@ -1,6 +1,3 @@
-from ceph_deploy.util import pkg_managers
-
-
 def uninstall(distro, purge=False):
     packages = [
         'ceph',
@@ -10,4 +7,4 @@ def uninstall(distro, purge=False):
         'librbd1',
         'ceph-radosgw',
         ]
-    pkg_managers.zypper_remove(distro.conn, packages)
+    distro.packager.remove(packages)
-- 
2.47.3