Douglas Schilling Landgraf has posted comments on this change.
Change subject: engine_page: rewrite register
......................................................................
Patch Set 4:
(3 comments)
....................................................
File src/misc.py
Line 206: def get_ovirtnode_kernel_params(self):
Line 207:
Line 208: kparams = {}
Line 209: try:
Line 210: with open(self._OVIRTNODE_KERNEL_OPTS, 'r') as configfile:
to avoid if "OVIRT_BOOTPARAMS" in line ?
Line 211: for line in configfile:
Line 212: if "OVIRT_BOOTPARAMS" in line:
Line 213: for item in line.split("\"")[1].split(" "):
Line 214: if "=" in item:
Line 206: def get_ovirtnode_kernel_params(self):
Line 207:
Line 208: kparams = {}
Line 209: try:
Line 210: with open(self._OVIRTNODE_KERNEL_OPTS, 'r') as configfile:
this file doesn't have session.
Line 211: for line in configfile:
Line 212: if "OVIRT_BOOTPARAMS" in line:
Line 213: for item in line.split("\"")[1].split(" "):
Line 214: if "=" in item:
Line 218: kparams.update({item:''})
Line 219: except Exception as e:
Line 220: logging.debug(
Line 221: "Cannot get kernel params %s error %s",
Line 222: self._OVIRTNODE_KERNEL_OPTS, e)
Done
Line 223:
--
To view, visit http://gerrit.ovirt.org/17682
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I38f3b800c445f8dbb0fa0e89d128cea1e3407798
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
