add section for failed log uploads
diff --git a/buildkite/pipelines/bazelci.py b/buildkite/pipelines/bazelci.py
index 482ed9a..88858ea 100644
--- a/buildkite/pipelines/bazelci.py
+++ b/buildkite/pipelines/bazelci.py
@@ -140,14 +140,14 @@
def downstream_projects_root(platform):
- path = os.path.join(agent_directory(platform), "downstream-projects")
- if not os.path.exists(path):
- os.makedirs(path)
- return path
+ path = os.path.join(agent_directory(platform), "downstream-projects")
+ if not os.path.exists(path):
+ os.makedirs(path)
+ return path
def agent_directory(platform):
- return os.path.expandvars(platforms_info()[platform]["agent-directory"])
+ return os.path.expandvars(platforms_info()[platform]["agent-directory"])
def supported_platforms():
@@ -233,15 +233,17 @@
clone_path = os.path.join(root, project_name)
print("\n--- Fetching " + project_name + " sources")
if os.path.exists(clone_path):
- shutil.rmtree(clone_path)
+ shutil.rmtree(clone_path)
fail_if_nonzero(execute_command(["git", "clone", git_repository, clone_path]))
os.chdir(clone_path)
+
def cleanup(bazel_binary):
if os.path.exists("WORKSPACE"):
print("\n--- Cleanup")
fail_if_nonzero(execute_command([bazel_binary, "clean", "--expunge"]))
+
def execute_shell_commands(commands):
if not commands:
return
@@ -283,9 +285,12 @@
def upload_failed_test_logs(bep_file, tmpdir):
if not os.path.exists(bep_file):
return
- for logfile in failed_logs_from_bep(bep_file, tmpdir):
- fail_if_nonzero(execute_command(["buildkite-agent", "artifact", "upload",
- logfile]))
+ logfiles = failed_logs_from_bep(bep_file, tmpdir)
+ if logfiles:
+ print("\n--- Uploading failed test logs")
+ for logfile in logfiles:
+ fail_if_nonzero(execute_command(["buildkite-agent", "artifact", "upload",
+ logfile]))
def failed_logs_from_bep(bep_file, tmpdir):
