X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=plugins%2Fomf_resctl.py;h=fb32834a7269b56d59e50b20626ef369d1ff54bd;hb=5fe3db92ad8c5aba4002a14542e915177430243b;hp=85824e189df7de5e94bff96e32e46ebea022db15;hpb=185a2e3bf48406e5184c2d9259715adc9e6fd975;p=nodemanager.git diff --git a/plugins/omf_resctl.py b/plugins/omf_resctl.py index 85824e1..fb32834 100644 --- a/plugins/omf_resctl.py +++ b/plugins/omf_resctl.py @@ -13,21 +13,26 @@ import subprocess import tools import logger -priority = 50 +# we need this to run after sliverauth +priority = 150 def start(): pass -### the new template for v6 +### the new template for OMF v6 # hard-wire this for now # once the variables are expanded, this is expected to go into config_ple_template="""--- -# Example: -# _slicename_ = nicta_ruby -# _hostname_ = planetlab1.research.nicta.com.au -# _xmpp_server_ = xmpp.planet-lab.eu - -:uid: _slicename_@_hostname_ +# we extract expires time here, even in a comment so that the +# trigger script gets called whenever this changes +# expires: _expires_ + +# these are not actual OMF parameters, they are only used by the trigger script +:hostname: _hostname_ +:slicename: _slicename_ + +# OMF configuration +:uid: _slicename_%_hostname_ :uri: xmpp://_slicename_-_hostname_-<%= "#{Process.pid}" %>:_slicename_-_hostname_-<%= "#{Process.pid}" %>@_xmpp_server_ :environment: production :debug: false @@ -41,7 +46,22 @@ config_ple_template="""--- # the path where the config is expected from within the sliver yaml_slice_path="/etc/omf_rc/config.yml" # the path for the script that we call when a change occurs -omf_rc_trigger_script="plc_trigger_omf_rc" +# given that we're now responsible for fetching this one, I have to +# decide on an actual path - not jsut a name to search for in PATH +omf_rc_trigger_script="/usr/bin/plc_trigger_omf_rc" + +# hopefully temporary: when trigger script is missing, fetch it at the url here +omf_rc_trigger_url="http://git.mytestbed.net/?p=omf.git;a=blob_plain;f=omf_rc/bin/plc_trigger_omf_rc;hb=HEAD" +def fetch_trigger_script_if_missing (slicename): + full_path="/vservers/%s/%s"%(slicename,omf_rc_trigger_script) + if not os.path.isfile (full_path): + retcod=subprocess.call (['curl','--silent','-o',full_path,omf_rc_trigger_url]) + if retcod!=0: + logger.log("Could not fetch %s"%omf_rc_trigger_url) + else: + subprocess.call(['chmod','+x',full_path]) + logger.log("omf_resctl: fetched %s"%(full_path)) + logger.log("omf_resctl: from %s"%(omf_rc_trigger_url)) def GetSlivers(data, conf = None, plc = None): logger.log("omf_resctl.GetSlivers") @@ -69,11 +89,13 @@ def GetSlivers(data, conf = None, plc = None): # skip non OMF-friendly slices if not is_omf_friendly (sliver): continue slicename=sliver['name'] + expires=str(sliver['expires']) yaml_template = config_ple_template yaml_contents = yaml_template\ .replace('_xmpp_server_',xmpp_server)\ .replace('_slicename_',slicename)\ - .replace('_hostname_',hostname) + .replace('_hostname_',hostname)\ + .replace('_expires_',expires) yaml_full_path="/vservers/%s/%s"%(slicename,yaml_slice_path) yaml_full_dir=os.path.dirname(yaml_full_path) if not os.path.isdir(yaml_full_dir): @@ -89,11 +111,11 @@ def GetSlivers(data, conf = None, plc = None): if config_changes or keys_changes: # instead of restarting the service we call a companion script try: + fetch_trigger_script_if_missing (slicename) # the trigger script actually needs to be run in the slice context of course - # xxx we might need to use - # slice_command=['bash','-l','-c',omf_rc_trigger_script] - slice_command = [omf_rc_trigger_script] + slice_command = [ "sudo", "-i", omf_rc_trigger_script ] to_run = tools.command_in_slice (slicename, slice_command) + logger.log("command_in_slice: %s"%to_run) sp=subprocess.Popen(to_run, stdout=subprocess.PIPE,stderr=subprocess.STDOUT) (out,err)=sp.communicate() retcod=sp.returncode