Change in vdsm[master]: New sysctl module for run-time changes.

danken at redhat.com danken at redhat.com
Tue Jan 6 13:01:58 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: New sysctl module for run-time changes.
......................................................................


New sysctl module for run-time changes.

Some sysctl tunables must be dynamically set to configure system in given
situations. Those sets won't persist after boot.

This patch is creating the sysctl module to allow such configuration and preparing
this module with the functions needed to handle rp_filter configuration. Since sysctl
tunables can only be set by root user, supervdsmServer is used as proxy. Further
use of this module can be based in rp_filter configuration.

Notice that to have mandatory changes sysctl.d/vdsm.conf still should be used.

Change-Id: I950c85a6f8d69494482ff63e569cb17ffc723624
Signed-off-by: Amador Pahim <apahim at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/36531
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/Makefile.am
A lib/vdsm/sysctl.py
M vdsm.spec.in
M vdsm/supervdsmServer
4 files changed, 48 insertions(+), 0 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Amador Pahim: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I950c85a6f8d69494482ff63e569cb17ffc723624
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Amador Pahim <apahim at redhat.com>
Gerrit-Reviewer: Amador Pahim <apahim at redhat.com>
Gerrit-Reviewer: Amir Shladovsky <a.shlado at gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list