[node-patches] Change in ovirt-node[master]: network: Fix setting NFSv4 domain

fabiand at fedoraproject.org fabiand at fedoraproject.org
Fri Dec 14 14:05:46 UTC 2012


Fabian Deutsch has uploaded a new change for review.

Change subject: network: Fix setting NFSv4 domain
......................................................................

network: Fix setting NFSv4 domain

Previously some functions weren't imported correctly this is now fixed.

Change-Id: I4684cc549662de36a693d6308ad80771d6b48bfb
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/ovirtnode/network.py
1 file changed, 6 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/63/10063/1

diff --git a/scripts/ovirtnode/network.py b/scripts/ovirtnode/network.py
index 6d2d5da..03fac99 100644
--- a/scripts/ovirtnode/network.py
+++ b/scripts/ovirtnode/network.py
@@ -576,20 +576,20 @@
 def set_nfsv4_domain(domain):
     idmap_conf = "/etc/idmapd.conf"
     current_domain = get_current_nfsv4_domain()
-    unmount_config(idmap_conf)
+    _functions.unmount_config(idmap_conf)
     if current_domain.startswith("#"):
         current_domain = "#Domain = %s" % current_domain.replace("# ","")
-        system("sed -i 's/%s/Domain = %s/g' %s" \
+        _functions.system("sed -i 's/%s/Domain = %s/g' %s" \
             % (current_domain, domain, idmap_conf))
     else:
-        system("sed -i 's/%s/%s/g' %s" \
+        _functions.system("sed -i 's/%s/%s/g' %s" \
             % (current_domain, domain, idmap_conf))
-    if ovirt_store_config(idmap_conf):
+    if _functions.ovirt_store_config(idmap_conf):
         logger.info("NFSv4 domain set as: " + domain)
     else:
         logger.warning("Setting nfsv4 domain failed")
-    system_closefds("service rpcidmapd restart")
-    system_closefds("nfsidmap -c &>/dev/null")
+    _functions.system_closefds("service rpcidmapd restart")
+    _functions.system_closefds("nfsidmap -c &>/dev/null")
 
 def convert_to_biosdevname():
     if not "BIOSDEVNAMES_CONVERSION" in OVIRT_VARS:


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4684cc549662de36a693d6308ad80771d6b48bfb
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>



More information about the node-patches mailing list