Skip to content

Commit 50169a8

Browse files
committed
BuilderUtils.py: default to more logging output
For quite a while, the build-with-autotools Open MPI nightly build on Open MPI v5.0.0 has been failing. It turns out that upstream PMIx upgraded the version of Sphinx that it required, which ultimately caused the build failure because the Sphinx version installed on aws.open-mpi.org was too old. Upgrading the Sphinx on aws.open-mpi.org was easy enough. The real issue is that the failure notification email did not include enough logging output to reveal that the root cause of the build failure was the Sphinx version. As such, this commit increases the default number of log lines included in the failure message in the email so that this particular problem, at least, will be evident in future emails. Signed-off-by: Jeff Squyres <jeff@squyres.com>
1 parent 7cfa610 commit 50169a8

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

nightly-tarball/BuilderUtils.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
#
22
# Copyright (c) 2017 Amazon.com, Inc. or its affiliates. All Rights
33
# Reserved.
4+
# Copyright (c) 2024 Jeffrey M. Squyres. All rights reserved.
45
#
56
# Additional copyrights may follow
67
#
@@ -14,7 +15,7 @@
1415
def logged_call(args,
1516
wrapper_args=None,
1617
log_file=None,
17-
err_log_len=20,
18+
err_log_len=30,
1819
env=None):
1920
"""Wrapper around check_call to log output
2021

0 commit comments

Comments
 (0)