witine/lookingglass
1
0
Fork 0
forked from mirrors/thatmattlove-hyperglass
Code Issues Pull requests Releases Activity

improve ui build timeout handling

Browse source
This commit is contained in:
checktheroads 2021-01-04 00:58:55 -07:00
parent 426b9adfad
commit 434e7bad6d
1 changed files with 17 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
hyperglass/util/__init__.py
View file

@ -97,13 +97,9 @@ def check_python() -> str:
return platform.python_version()
async def build_ui(app_path):
"""Execute `next build` & `next export` from UI directory.
def get_ui_build_timeout() -> int:
"""Read the UI build timeout from environment variables or set a default."""
Raises:
RuntimeError: Raised if exit code is not 0.
RuntimeError: Raised when any other error occurs.
"""
timeout = 90
if "HYPERGLASS_UI_BUILD_TIMEOUT" in os.environ:
@ -114,6 +110,18 @@ async def build_ui(app_path):
timeout = int(os.environ["POETRY_HYPERGLASS_UI_BUILD_TIMEOUT"])
log.info("Found UI build timeout environment variable: {}", timeout)
return timeout
async def build_ui(app_path):
"""Execute `next build` & `next export` from UI directory.
Raises:
RuntimeError: Raised if exit code is not 0.
RuntimeError: Raised when any other error occurs.
"""
timeout = get_ui_build_timeout()
ui_dir = Path(__file__).parent.parent / "ui"
build_dir = app_path / "static" / "ui"
@ -320,6 +328,8 @@ async def node_initial(dev_mode=False):
ui_path = Path(__file__).parent.parent / "ui"
timeout = get_ui_build_timeout()
all_messages = []
try:
proc = await asyncio.create_subprocess_shell(
@ -329,7 +339,7 @@ async def node_initial(dev_mode=False):
cwd=ui_path,
)
stdout, stderr = await asyncio.wait_for(proc.communicate(), timeout=60)
stdout, stderr = await asyncio.wait_for(proc.communicate(), timeout=timeout)
messages = stdout.decode("utf-8").strip()
errors = stderr.decode("utf-8").strip()