Alex Lourie has uploaded a new change for review.

Change subject: packaging: Updated the nfs service name usage
......................................................................

packaging: Updated the nfs service name usage

This patch updates the usage of the nfs service
name for different distributions. It sets the default
service name to 'nfs' updating it to 'nfs-server' for
Fedora.

Change-Id: I17f4b564517c180e0fe14a0a4ed3d0e0b97a5e19
Signed-off-by: Alex Lourie <alou...@redhat.com>
---
M packaging/setup/plugins/ovirt-engine-setup/system/nfs.py
1 file changed, 10 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/54/15954/1

diff --git a/packaging/setup/plugins/ovirt-engine-setup/system/nfs.py 
b/packaging/setup/plugins/ovirt-engine-setup/system/nfs.py
index ac82b77..1d76de9 100644
--- a/packaging/setup/plugins/ovirt-engine-setup/system/nfs.py
+++ b/packaging/setup/plugins/ovirt-engine-setup/system/nfs.py
@@ -56,6 +56,7 @@
             None
         )
         self._enabled = True
+        self._servicename = 'nfs'
 
     @plugin.event(
         stage=plugin.Stages.STAGE_SETUP,
@@ -69,6 +70,9 @@
                 _('Unsupported distribution disabling nfs export')
             )
             self._enabled = False
+
+        if self._distribution == 'fedora':
+            self._servicename = 'nfs-server'
 
     @plugin.event(
         stage=plugin.Stages.STAGE_CUSTOMIZATION,
@@ -175,17 +179,14 @@
             )
 
         self.services.startup(
-            name='nfs-server',
+            name=self._servicename,
             state=True,
         )
-        self.services.state(
-            name='nfs-server',
-            state=False,
-        )
-        self.services.state(
-            name='nfs-server',
-            state=True,
-        )
+        for state in (False, True):
+            self.services.state(
+                name=self._servicename,
+                state=state,
+            )
 
 
 # vim: expandtab tabstop=4 shiftwidth=4


-- 
To view, visit http://gerrit.ovirt.org/15954
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I17f4b564517c180e0fe14a0a4ed3d0e0b97a5e19
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alex Lourie <alou...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to