From: Justin Pettit Date: Tue, 8 Dec 2009 02:18:29 +0000 (-0800) Subject: xenserver: Modify init scripts to use new configuration database X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=fea28b07bb69201ebaa1724a95fc764483a72f7d;hp=5c8ef29c2efb73ab3bc8ac30f1499fb6499692cf;p=openvswitch xenserver: Modify init scripts to use new configuration database --- diff --git a/xenserver/etc_init.d_vswitch b/xenserver/etc_init.d_vswitch index 18dd7bc6..ea2db629 100755 --- a/xenserver/etc_init.d_vswitch +++ b/xenserver/etc_init.d_vswitch @@ -28,8 +28,25 @@ test -e /etc/sysconfig/vswitch && . /etc/sysconfig/vswitch : ${ENABLE_FAKE_PROC_NET:=y} : ${FORCE_COREFILES:=y} +# Config variables specific to ovsdb-server +: ${OVSDB_SERVER_LISTEN:=punix:/var/run/ovsdb-server} +: ${OVSDB_SERVER_CONNECT:=} +: ${OVSDB_SERVER_DB:=/etc/ovs-vswitchd.conf.db} +: ${OVSDB_SERVER_PIDFILE:=/var/run/ovsdb-server.pid} +: ${OVSDB_SERVER_RUN_DIR:=/var/xen/vswitch} +: ${OVSDB_SERVER_PRIORITY:=-10} +: ${OVSDB_SERVER_LOGFILE:=/var/log/ovsdb-server.log} +: ${OVSDB_SERVER_FILE_LOGLEVEL:=INFO} +: ${OVSDB_SERVER_SYSLOG_LOGLEVEL:=ERR} +: ${OVSDB_SERVER_MEMLEAK_LOGFILE:=} +: ${OVSDB_SERVER_STRACE_LOG:=} +: ${OVSDB_SERVER_STRACE_OPT:=} +: ${OVSDB_SERVER_VALGRIND_LOG:=} +: ${OVSDB_SERVER_VALGRIND_OPT:=} + # Config variables specific to ovs-vswitchd -: ${VSWITCHD_CONF:=/etc/ovs-vswitchd.conf} +: ${VSWITCHD_OVSDB_SERVER:=unix:/var/run/ovsdb-server} +: ${VSWITCHD_OVSDB_SCHEMA:=/usr/share/vswitch/vswitch-idl.ovsschema} : ${VSWITCHD_PIDFILE:=/var/run/ovs-vswitchd.pid} : ${VSWITCHD_RUN_DIR:=/var/xen/vswitch} : ${VSWITCHD_PRIORITY:=-10} @@ -56,6 +73,8 @@ test -e /etc/sysconfig/vswitch && . /etc/sysconfig/vswitch : ${BRCOMPATD_VALGRIND_OPT:=} # Full paths to executables & modules +ovsdb_server="/usr/sbin/ovsdb-server" +ovsdb_tool="/usr/bin/ovsdb-tool" vswitchd="/usr/sbin/ovs-vswitchd" brcompatd="/usr/sbin/ovs-brcompatd" dpctl="/usr/bin/ovs-dpctl" @@ -121,6 +140,56 @@ function reload_brcompatd { fi } +function start_ovsdb_server { + local syslog_opt="-vANY:SYSLOG:${OVSDB_SERVER_SYSLOG_LOGLEVEL}" + local logfile_file_opt="" + local logfile_level_opt="" + if [ ! -d "$OVSDB_SERVER_RUN_DIR" ]; then + mkdir -p "$OVSDB_SERVER_RUN_DIR" + fi + cd "$OVSDB_SERVER_RUN_DIR" + local listen_method="" + if [ -n "$OVSDB_SERVER_LISTEN" ]; then + listen_method="--listen=$OVSDB_SERVER_LISTEN" + fi + local connect_method="" + if [ -n "$OVSDB_SERVER_CONNECT" ]; then + connect_method="--connect=$OVSDB_SERVER_CONNECT" + fi + if [ -n "$OVSDB_SERVER_FILE_LOGLEVEL" ]; then + logfile_level_opt="-vANY:FILE:${OVSDB_SERVER_FILE_LOGLEVEL}" + logfile_file_opt="--log-file=$OVSDB_SERVER_LOGFILE" + fi + local leak_opt="" + if [ -n "$OVSDB_SERVER_MEMLEAK_LOGFILE" ]; then + leak_opt="--check-leaks=$OVSDB_SERVER_MEMLEAK_LOGFILE" + if [ -e "$OVSDB_SERVER_MEMLEAK_LOGFILE" ]; then + mv "$OVSDB_SERVER_MEMLEAK_LOGFILE" "$OVSDB_SERVER_MEMLEAK_LOGFILE.prev" + fi + fi + local strace_opt="" + local daemonize="y" + if [ -n "$OVSDB_SERVER_STRACE_LOG" ] && [ -n "$OVSDB_SERVER_VALGRIND_LOG" ]; then + printf "Can not start with both VALGRIND and STRACE\n" + exit 1 + fi + if [ -n "$OVSDB_SERVER_STRACE_LOG" ]; then + strace_opt="strace -o $OVSDB_SERVER_STRACE_LOG $OVSDB_SERVER_STRACE_OPT" + daemonize="n" + fi + if [ -n "$OVSDB_SERVER_VALGRIND_LOG" ]; then + valgrind_opt="valgrind --log-file=$OVSDB_SERVER_VALGRIND_LOG $OVSDB_SERVER_VALGRIND_OPT" + daemonize="n" + fi + if [ "$daemonize" != "y" ]; then + # Start in background and force a "success" message + action "Starting ovsdb_server ($strace_opt$valgrind_opt)" true + (nice -n "$OVSDB_SERVER_PRIORITY" $strace_opt $valgrind_opt "$ovsdb_server" "$OVSDB_SERVER_DB" --pidfile="$OVSDB_SERVER_PIDFILE" --detach --no-chdir -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt $connect_method $listen_method) & + else + action "Starting ovsdb-server" nice -n "$OVSDB_SERVER_PRIORITY" "$ovsdb_server" "$OVSDB_SERVER_DB" --pidfile="$OVSDB_SERVER_PIDFILE" --detach --no-chdir -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt $connect_method $listen_method + fi +} + function start_vswitchd { local syslog_opt="-vANY:SYSLOG:${VSWITCHD_SYSLOG_LOGLEVEL}" local logfile_file_opt="" @@ -161,9 +230,9 @@ function start_vswitchd { if [ "$daemonize" != "y" ]; then # Start in background and force a "success" message action "Starting ovs-vswitchd ($strace_opt$valgrind_opt)" true - (nice -n "$VSWITCHD_PRIORITY" $strace_opt $valgrind_opt "$vswitchd" --pidfile="$VSWITCHD_PIDFILE" --detach --no-chdir $fake_proc_net_opt -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt "$VSWITCHD_CONF") & + (nice -n "$VSWITCHD_PRIORITY" $strace_opt $valgrind_opt "$vswitchd" --pidfile="$VSWITCHD_PIDFILE" --detach --no-chdir $fake_proc_net_opt -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt "$VSWITCHD_OVSDB_SERVER") & else - action "Starting ovs-vswitchd" nice -n "$VSWITCHD_PRIORITY" "$vswitchd" --pidfile="$VSWITCHD_PIDFILE" --detach --no-chdir $fake_proc_net_opt -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt "$VSWITCHD_CONF" + action "Starting ovs-vswitchd" nice -n "$VSWITCHD_PRIORITY" "$vswitchd" --pidfile="$VSWITCHD_PIDFILE" --detach --no-chdir $fake_proc_net_opt -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt "$VSWITCHD_OVSDB_SERVER" fi } @@ -204,9 +273,17 @@ function start_brcompatd { if [ "$daemonize" != "y" ]; then # Start in background and force a "success" message action "Starting ovs-brcompatd ($strace_opt$valgrind_opt)" true - (nice -n "$VSWITCHD_PRIORITY" $strace_opt $valgrind_opt "$brcompatd"--no-chdir --appctl-command="$appctl_cmd" --pidfile=$BRCOMPATD_PIDFILE -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt "$VSWITCHD_CONF") & + (nice -n "$VSWITCHD_PRIORITY" $strace_opt $valgrind_opt "$brcompatd"--no-chdir --appctl-command="$appctl_cmd" --pidfile=$BRCOMPATD_PIDFILE -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt "$VSWITCHD_OVSDB_SERVER") & else - action "Starting ovs-brcompatd" nice -n "$BRCOMPATD_PRIORITY" $strace_opt $valgrind_opt "$brcompatd" --no-chdir --appctl-command="$appctl_cmd" --pidfile=$BRCOMPATD_PIDFILE --detach -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt "$VSWITCHD_CONF" + action "Starting ovs-brcompatd" nice -n "$BRCOMPATD_PRIORITY" $strace_opt $valgrind_opt "$brcompatd" --no-chdir --appctl-command="$appctl_cmd" --pidfile=$BRCOMPATD_PIDFILE --detach -vANY:CONSOLE:EMER $syslog_opt $logfile_level_opt $logfile_file_opt $leak_opt "$VSWITCHD_OVSDB_SERVER" + fi +} + +function stop_ovsdb_server { + if [ -f "$OVSDB_SERVER_PIDFILE" ]; then + local pid=$(cat "$OVSDB_SERVER_PIDFILE") + action "Killing ovsdb-server ($pid)" kill -TERM $pid + rm -f "$OVSDB_SERVER_PIDFILE" fi } @@ -267,29 +344,32 @@ function start { # Allow GRE traffic. iptables -I INPUT -p gre -j ACCEPT - if [ ! -e "$VSWITCHD_CONF" ]; then - warning "$VSWITCHD_CONF does not exist" - action "Creating empty $VSWITCHD_CONF" touch "$VSWITCHD_CONF" - elif [ ! -e /var/run/vswitch.booted ]; then - touch /var/run/vswitch.booted - /usr/bin/ovs-cfg-mod '-vANY:console:emer' -F "$VSWITCHD_CONF" \ - '--del-match=bridge.*' \ - '--del-match=port.*' \ - '--del-match=bonding.*' \ - '--del-match=iface.*' \ - '--del-match=vlan.*.trunks=*' \ - '--del-match=vlan.*.tag=*' + if [ ! -e "$OVSDB_SERVER_DB" ]; then + warning "$OVSDB_SERVER_DB does not exist" + action "Creating empty $OVSDB_SERVER_DB" $ovsdb_tool create "$OVSDB_SERVER_DB" "$VSWITCHD_OVSDB_SCHEMA" + action "Creating initial table in $OVSDB_SERVER_DB" $ovsdb_tool transact "$OVSDB_SERVER_DB" '[{"op": "insert", "table": "Open_vSwitch", "row": {}}]' + #elif [ ! -e /var/run/vswitch.booted ]; then + #touch /var/run/vswitch.booted + #/usr/bin/ovs-cfg-mod '-vANY:console:emer' -F "$VSWITCHD_CONF" \ + #'--del-match=bridge.*' \ + #'--del-match=port.*' \ + #'--del-match=bonding.*' \ + #'--del-match=iface.*' \ + #'--del-match=vlan.*.trunks=*' \ + #'--del-match=vlan.*.tag=*' fi + start_ovsdb_server start_vswitchd start_brcompatd - reload_vswitchd # ensures ovs-vswitchd has fully read config file. + reload_vswitchd # ensures ovs-vswitchd has fully read config. touch /var/lock/subsys/vswitch } function stop { stop_brcompatd stop_vswitchd + stop_ovsdb_server rm -f /var/lock/subsys/vswitch } diff --git a/xenserver/usr_share_vswitch_scripts_sysconfig.template b/xenserver/usr_share_vswitch_scripts_sysconfig.template index 32094773..f1d19785 100644 --- a/xenserver/usr_share_vswitch_scripts_sysconfig.template +++ b/xenserver/usr_share_vswitch_scripts_sysconfig.template @@ -25,8 +25,85 @@ # See the manpage for "core". # COREFILE_PATTERN="/var/log/%e-%t" -# VSWITCHD_CONF: File in which ovs-vswitchd stores its configuration. -# VSWITCHD_CONF=/etc/ovs-vswitchd.conf +# OVSDB_SERVER_LISTEN: Method to have ovsdb-server listen for a JSON-RPC +# connection. +# OVSDB_SERVER_LISTEN=punix:/var/run/ovsdb-server + +# OVSDB_SERVER_CONNECT: Method to have ovsdb-server initiate a JSON-RPC +# connection to a remote server. +# OVSDB_SERVER_CONNECT= + +# OVSDB_SERVER_DB: File for which ovsdb-server uses for storage. +# OVSDB_SERVER_DB=/etc/ovs-vswitchd.conf.db + +# OVSDB_SERVER_PIDFILE: File in which to store the pid of the running +# ovsdb-server. +# OVSDB_SERVER_PIDFILE=/var/run/ovsdb-server.pid + +# OVSDB_SERVER_RUN_DIR: Set the directory in which ovsdb-server should be +# run. This mainly affects where core files will be placed. +# OVSDB_SERVER_RUN_DIR=/var/xen/vswitch + +# OVSDB_SERVER_PRIORITY: "nice" priority at which to run ovsdb-server and +# related processes. +# OVSDB_SERVER_PRIORITY=-10 + +# OVSDB_SERVER_LOGFILE: File to send the FILE_LOGLEVEL log messages to. +# OVSDB_SERVER_LOGFILE=/var/log/ovsdb-server.log + +# OVSDB_SERVER_FILE_LOGLEVEL: Log level at which to log into the +# OVSDB_SERVER_LOG file. If this is null or not set the logfile will +# not be created and nothing will be sent to it. This is the +# default. The available options are: EMER, WARN, INFO and DBG. +# OVSDB_SERVER_FILE_LOGLEVEL="" + +# OVSDB_SERVER_SYSLOG_LOGLEVEL: Log level at which to log into syslog. If +# this is null or not set the default is to log to syslog +# emergency and warning level messages only. +# OVSDB_SERVER_SYSLOG_LOGLEVEL="WARN" + +# OVSDB_SERVER_MEMLEAK_LOGFILE: File for logging memory leak data. +# Enabling this option will slow ovsdb-server significantly. Do not +# enable it except to debug a suspected memory leak. Use the +# ovs-parse-leaks utility included with Open vSwitch to parse the +# log file. For best results, you also need debug symbols. +# OVSDB_SERVER_MEMLEAK_LOGFILE="" + +# OVSDB_SERVER_STRACE_LOG: File for logging strace output. +# If this is set to a nonempty string, then ovsdb-server will run +# under strace, whose output will be logged to the specified file. +# Enabling this option will slow ovsdb-server significantly. +# OVSDB_SERVER_STRACE_LOG and OVSDB_SERVER_VALGRIND_LOG are mutually +# exclusive. +# OVSDB_SERVER_STRACE_LOG="" + +# OVSDB_SERVER_STRACE_OPT: Options to pass to strace. +# This option's value is honored only when OVSDB_SERVER_STRACE_LOG is +# set to a nonempty string. +# OVSDB_SERVER_STRACE_OPT="" + +# OVSDB_SERVER_VALGRIND_LOG: File for logging valgrind output. +# If this is set to a nonempty string, then ovsdb-server will run +# under valgrind, whose output will be logged to the specified file. +# Enabling this option will slow ovsdb-server by 100X or more. +# valgrind is not installed by default on XenServer systems; you must +# install it by hand to usefully enable this option. +# OVSDB_SERVER_STRACE_LOG and OVSDB_SERVER_VALGRIND_LOG are mutually +# exclusive. +# OVSDB_SERVER_VALGRIND_LOG="" + +# OVSDB_SERVER_VALGRIND_OPT: Options to pass to valgrind. +# This option's value is honored only when OVSDB_SERVER_VALGRIND_LOG is +# set to a nonempty string. +# OVSDB_SERVER_VALGRIND_OPT="" + +# VSWITCHD_OVSDB_SERVER: Method to have ovs-vswitchd initiate a JSON-RPC +# connection to an ovsdb-server instance. +# VSWITCHD_OVSDB_SERVER=unix:/var/run/ovsdb-server + +# VSWITCHD_OVSDB_SCHEMA: Schema file to use for generating a new OVSDB +# ovs-vswitchd database. +# VSWITCHD_OVSDB_SCHEMA=/usr/share/vswitch/vswitch-idl.ovsschema # VSWITCHD_PIDFILE: File in which to store the pid of the running # ovs-vswitchd. @@ -43,13 +120,6 @@ # VSWITCHD_LOGFILE: File to send the FILE_LOGLEVEL log messages to. # VSWITCHD_LOGFILE=/var/log/ovs-vswitchd.log -# VSWITCHD_MEMLEAK_LOGFILE: File for logging memory leak data. -# Enabling this option will slow ovs-vswitchd significantly. Do not -# enable it except to debug a suspected memory leak. Use the -# ovs-parse-leaks utility included with Open vSwitch to parse the -# log file. For best results, you also need debug symbols. -# VSWITCHD_MEMLEAK_LOGFILE="" - # VSWITCHD_FILE_LOGLEVEL: Log level at which to log into the # VSWITCHD_LOG file. If this is null or not set the logfile will # not be created and nothing will be sent to it. This is the @@ -61,6 +131,13 @@ # emergency and warning level messages only. # VSWITCHD_SYSLOG_LOGLEVEL="WARN" +# VSWITCHD_MEMLEAK_LOGFILE: File for logging memory leak data. +# Enabling this option will slow ovs-vswitchd significantly. Do not +# enable it except to debug a suspected memory leak. Use the +# ovs-parse-leaks utility included with Open vSwitch to parse the +# log file. For best results, you also need debug symbols. +# VSWITCHD_MEMLEAK_LOGFILE="" + # VSWITCHD_STRACE_LOG: File for logging strace output. # If this is set to a nonempty string, then ovs-vswitchd will run # under strace, whose output will be logged to the specified file. diff --git a/xenserver/vswitch-xen.spec b/xenserver/vswitch-xen.spec index a0899edd..313f1b63 100644 --- a/xenserver/vswitch-xen.spec +++ b/xenserver/vswitch-xen.spec @@ -67,6 +67,8 @@ install -d -m 755 $RPM_BUILD_ROOT/etc/xapi.d/plugins install -m 755 xenserver/etc_xapi.d_plugins_vswitch-cfg-update \ $RPM_BUILD_ROOT/etc/xapi.d/plugins/vswitch-cfg-update install -d -m 755 $RPM_BUILD_ROOT/usr/share/vswitch/scripts +install -m 644 vswitchd/vswitch-idl.ovsschema \ + $RPM_BUILD_ROOT/usr/share/vswitch/vswitch-idl.ovsschema install -m 755 xenserver/opt_xensource_libexec_interface-reconfigure \ $RPM_BUILD_ROOT/usr/share/vswitch/scripts/interface-reconfigure install -m 755 xenserver/etc_xensource_scripts_vif \ @@ -186,8 +188,9 @@ net.ipv4.conf.all.arp_filter = 1 EOF fi -# Ensure ovs-vswitchd.conf exists -touch /etc/ovs-vswitchd.conf +# Create ovs-vswitchd config database +ovsdb-tool create /etc/ovs-vswitchd.conf.db \ + /usr/share/vswitch/vswitch-idl.ovsschema # Create default or update existing /etc/sysconfig/vswitch. SYSCONFIG=/etc/sysconfig/vswitch @@ -284,7 +287,7 @@ if [ "$1" = "0" ]; then # $1 = 1 for upgrade done # Remove all configuration files - rm -f /etc/ovs-vswitchd.conf + rm -f /etc/ovs-vswitchd.conf.db rm -f /etc/sysconfig/vswitch rm -f /etc/ovs-vswitchd.cacert rm -f /var/lib/openvswitch/dbcache @@ -323,6 +326,7 @@ fi # include them. /usr/share/vswitch/scripts/XSFeatureVSwitch.pyc /usr/share/vswitch/scripts/XSFeatureVSwitch.pyo +/usr/share/vswitch/vswitch-idl.ovsschema /usr/sbin/ovs-brcompatd /usr/sbin/ovs-vswitchd /usr/sbin/ovsdb-server