xCAT/confluent
2
0
Fork 0
mirror of https://github.com/xcat2/confluent.git synced 2024-11-26 03:19:48 +00:00
Code Issues Projects Releases Wiki Activity

Remove more vestiges of the thread support

Browse Source 
Also, remove a required function from console interface.  Now a plugin
is only responsible for their thread behavior and never responsible
for the consoleserver mechanism requirement for helping user session
do polling IO.  This is additionally great because it means the
coordination is contained to each session object and many client connections
are not causing multiple calls to iterate the wait loop.
This commit is contained in:
Jarrod Johnson 2014-02-06 09:27:38 -05:00
parent fd27835424
commit 80d4518a49
2 changed files with 16 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
confluent/consoleserver.py
View File

@ -13,6 +13,7 @@ import confluent.util as util
import eventlet
import eventlet.green.threading as threading
import random
import time
_handled_consoles = {}
@ -160,25 +161,17 @@ class ConsoleSession(object):
at least one case where we don't have that luxury
"""
self.reaper.cancel()
currtime = util.monotonic_time()
deadline = currtime + 45
try:
while len(self.databuffer) == 0 and currtime < deadline:
timeo = deadline - currtime
# change to a threading event object
# got_data will trigger this function to move
if self._evt is None:
self._evt = threading.Event()
if len(self.databuffer) == 0:
self._evt.wait(timeout)
self.conshdl._console.wait_for_data(timeout=timeo)
currtime = util.monotonic_time()
except TypeError:
import traceback
traceback.print_exc()
return ""
retval = self.databuffer
self.databuffer = ""
self._evt.clear()
if self._evt is not None:
self._evt.clear()
self.reaper = eventlet.spawn_after(15, self.destroy)
return retval

41
plugins/hardwaremanagement/ipmi.py
View File

@ -13,18 +13,12 @@ import pyghmi.ipmi.command as ipmicommand
console.session.select = eventlet.green.select
console.session.threading = eventlet.green.threading
tmptimeout = None
_ipmithread = None
def _ipmi_evtloop():
global tmptimeout
while (1):
while True:
try:
if tmptimeout is not None:
console.session.Session.wait_for_rsp(timeout=tmptimeout)
tmptimeout = None
else:
console.session.Session.wait_for_rsp(timeout=600)
console.session.Session.wait_for_rsp(timeout=600)
except:
import traceback
traceback.print_exc()
@ -91,36 +85,16 @@ class IpmiConsole(conapi.Console):
self.datacallback(data)
def connect(self,callback):
global _ipmithread
self.datacallback = callback
self.solconnection=console.Console(bmc=self.bmc, port=self.port,
self.solconnection = console.Console(bmc=self.bmc, port=self.port,
userid=self.username,
password=self.password, kg=self.kg,
force=True,
iohandler=self.handle_data)
if _ipmithread is None:
_ipmithread = eventlet.spawn(_ipmi_evtloop)
def write(self, data):
self.solconnection.send_data(data)
def wait_for_data(self, timeout=600):
"""Wait for some network event.
This is currently not guaranteed to actually have data when
return. This is supposed to be something more appropriate
than sleep(0), but only marginally so.
"""
# reason for this is that we currently nicely pass through the callback
# straight to ipmi library. To implement this accurately, easiest path
# would be to add a layer through the callback. IMO there isn't enough
# value in assuring data coming back to bother with making the stack
# taller than it has to be
#TODO: a channel for the ipmithread to tug back instead of busy wait
#while tmptimeout is not None:
# eventlet.sleep(0)
console.session.Session.wait_for_rsp(timeout=timeout)
class IpmiIterator(object):
def __init__(self, operator, nodes, element, cfg, inputdata):
@ -223,7 +197,14 @@ class IpmiHandler(object):
def initthread():
global _ipmithread
if _ipmithread is None:
_ipmithread = eventlet.spawn(_ipmi_evtloop)
def create(nodes, element, configmanager, inputdata):
initthread()
if element == [ '_console', 'session' ]:
if len(nodes) > 1:
raise Exception("_console/session does not support multiple nodes")
@ -232,10 +213,12 @@ def create(nodes, element, configmanager, inputdata):
return IpmiIterator('update', nodes, element, configmanager, inputdata)
def update(nodes, element, configmanager, inputdata):
initthread()
return create(nodes, element, configmanager, inputdata)
def retrieve(nodes, element, configmanager, inputdata):
initthread()
return IpmiIterator('read', nodes, element, configmanager, inputdata)