Ohad Basan has posted comments on this change.
Change subject: packaging: change handling of sysctl.conf
......................................................................
Patch Set 4: (4 inline comments)
....................................................
File packaging/fedora/setup/engine-setup.py
Line 1623: if currentShmmax and (int(currentShmmax) >=
basedefs.CONST_SHMMAX):
Line 1624: logging.debug("current shared memory max in kernel is %s,
there is no need to update the kernel parameters", currentShmmax)
Line 1625: return
Line 1626:
Line 1627: # In some systems the sysctl file doesn't exist, so we may need
to create it:
Done
Line 1628: # Decide what is the file that we are going to modify according
to the
Line 1629: # existence or not of sysctl.conf, it exists in older
distributions but
Line 1630: # has been replaced by a sysctl.d directory in newer
distributions using
Line 1631: # systemd
Line 1630: # has been replaced by a sysctl.d directory in newer
distributions using
Line 1631: # systemd
Line 1632:
Line 1633: sysctlFile = basedefs.FILE_SYSCTL
Line 1634: if not os.path.exists(basedefs.sysctlFile):
Done
Line 1635: sysctlFile = basedefs.FILE_ENGINE_SYSCTL
Line 1636: # Create the chosen file if it doesn't exist.
Line 1637: open(basedefs.sysctlFile, "w").close()
Line 1638:
Line 1631: # systemd
Line 1632:
Line 1633: sysctlFile = basedefs.FILE_SYSCTL
Line 1634: if not os.path.exists(basedefs.sysctlFile):
Line 1635: sysctlFile = basedefs.FILE_ENGINE_SYSCTL
Done
Line 1636: # Create the chosen file if it doesn't exist.
Line 1637: open(basedefs.sysctlFile, "w").close()
Line 1638:
Line 1639: # If we got here, it means we need to update kernel.shmmax in
sysctl.conf
Line 1632:
Line 1633: sysctlFile = basedefs.FILE_SYSCTL
Line 1634: if not os.path.exists(basedefs.sysctlFile):
Line 1635: sysctlFile = basedefs.FILE_ENGINE_SYSCTL
Line 1636: # Create the chosen file if it doesn't exist.
three lines above
if not os.path.exists(basedefs.sysctlFile):
since if /etc/sysctl.conf doesn't exist than obviously
00-ovirt... doesn't exist in /etc/sysctl.d
Line 1637: open(basedefs.sysctlFile, "w").close()
Line 1638:
Line 1639: # If we got here, it means we need to update kernel.shmmax in
sysctl.conf
Line 1640: txtHandler = utils.TextConfigFileHandler(sysctlFile)
--
To view, visit http://gerrit.ovirt.org/8705
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ca4988beea1627b2da442a7fcf8fef3cf3db6a5
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ohad Basan <oba...@redhat.com>
Gerrit-Reviewer: Alex Lourie <alou...@redhat.com>
Gerrit-Reviewer: Itamar Heim <ih...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgold...@redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com>
Gerrit-Reviewer: Ohad Basan <oba...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
