[node-patches] Change in ovirt-node[master]: Netconsole model tweaks

rbarry at redhat.com rbarry at redhat.com
Wed Jun 25 16:28:07 UTC 2014


Ryan Barry has uploaded a new change for review.

Change subject: Netconsole model tweaks
......................................................................

Netconsole model tweaks

Don't use persistence from the old codebase.
Set a default port if there isn't one.

Change-Id: Ifaf0faa9c62e50c5d57727d3284199807df6427e
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/config/defaults.py
1 file changed, 4 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/16/29216/1

diff --git a/src/ovirt/node/config/defaults.py b/src/ovirt/node/config/defaults.py
index 7fef333..3e147ed 100644
--- a/src/ovirt/node/config/defaults.py
+++ b/src/ovirt/node/config/defaults.py
@@ -1337,14 +1337,15 @@
                                        "service. Is the host resolvable?")
             else:
                 self._clear_config()
-            if ovirt_store_config("/etc/sysconfig/netconsole"):
-                self.logger.info("Netconsole Configuration Updated")
+            fs.Config().persist("/etc/sysconfig/netconsole"):
+            self.logger.info("Netconsole Configuration Updated")
 
         cfg = dict(self.retrieve())
         server, port = (cfg["server"], cfg["port"])
 
         class CreateNetconsoleConfig(utils.Transaction.Element):
-            if server and port:
+            if server:
+                port = port if port else 6666
                 title = "Setting netconsole server and port"
             else:
                 title = "Disabling netconsole"


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifaf0faa9c62e50c5d57727d3284199807df6427e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>



More information about the node-patches mailing list