MIRRORS/poky
1
0
Fork 0
mirror of git://git.yoctoproject.org/poky.git synced 2025-07-19 12:59:02 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

bitbake: prserv/cooker: Handle PRService errors cleanly

Browse Source 
Current if the PR Service fails to start, bitbake carries on regardless or
hangs with no error message. This adds an exception and then handles it correctly
so the UIs correctly handle the error and exit cleanly.

[YOCTO #4010]

(Bitbake rev: 9c52c73fd2498e65be5f0da24dc2ae3803eb42eb)

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Richard Purdie 2013-05-02 15:46:11 +01:00
parent 2806646a26
commit 5e0124f00d
2 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
bitbake/lib/bb/cooker.py
View File

@ -1353,7 +1353,10 @@ class BBCooker:
# Empty the environment. The environment will be populated as # Empty the environment. The environment will be populated as
# necessary from the data store. # necessary from the data store.
#bb.utils.empty_environment() #bb.utils.empty_environment()
prserv.serv.auto_start(self.configuration.data) try:
prserv.serv.auto_start(self.configuration.data)
except prserv.serv.PRServiceConfigError:
bb.event.fire(CookerExit(), self.configuration.event_data)
return return
def post_serve(self): def post_serve(self):

9
bitbake/lib/prserv/serv.py
View File

@ -266,17 +266,20 @@ def is_local_special(host, port):
else: else:
return False return False
class PRServiceConfigError(Exception):
pass
def auto_start(d): def auto_start(d):
global singleton global singleton
if (not d.getVar('PRSERV_HOST', True)) or (not d.getVar('PRSERV_PORT', True)): if (not d.getVar('PRSERV_HOST', True)) or (not d.getVar('PRSERV_PORT', True)):
return True return
if is_local_special(d.getVar('PRSERV_HOST', True), int(d.getVar('PRSERV_PORT', True))) and not singleton: if is_local_special(d.getVar('PRSERV_HOST', True), int(d.getVar('PRSERV_PORT', True))) and not singleton:
import bb.utils import bb.utils
cachedir = (d.getVar("PERSISTENT_DIR", True) or d.getVar("CACHE", True)) cachedir = (d.getVar("PERSISTENT_DIR", True) or d.getVar("CACHE", True))
if not cachedir: if not cachedir:
logger.critical("Please set the 'PERSISTENT_DIR' or 'CACHE' variable") logger.critical("Please set the 'PERSISTENT_DIR' or 'CACHE' variable")
sys.exit(1) raise PRServiceConfigError
bb.utils.mkdirhier(cachedir) bb.utils.mkdirhier(cachedir)
dbfile = os.path.join(cachedir, "prserv.sqlite3") dbfile = os.path.join(cachedir, "prserv.sqlite3")
logfile = os.path.join(cachedir, "prserv.log") logfile = os.path.join(cachedir, "prserv.log")
@ -292,7 +295,7 @@ def auto_start(d):
return PRServerConnection(host,port).ping() return PRServerConnection(host,port).ping()
except Exception: except Exception:
logger.critical("PRservice %s:%d not available" % (host, port)) logger.critical("PRservice %s:%d not available" % (host, port))
return False raise PRServiceConfigError
def auto_shutdown(d=None): def auto_shutdown(d=None):
global singleton global singleton