[PATCH] Archive test traces to outer directory named by build number after lldb-test

Siva Chandra sivachandra at google.com
Mon May 4 15:00:26 PDT 2015


================
Comment at: zorg/buildbot/builders/LLDBBuilder.py:306
@@ -299,1 +305,3 @@
+                           env=env,
+                           workdir='%s' % llvm_builddir))
     return f
----------------
chying wrote:
> sivachandra wrote:
> > The summary note says that a separate task would zip and upload traces to google storage. Why cant it be done inline right here as part of this step?
> There were two concerns behind this, one is that certain builder might not have access to google storage, or would upload the traces to different locations. Another is that in case of network latency, doing this step inline would slow down the builder, zip&upload step normally only take 6 seconds though.
I don't understand the first of the concerns. For the second one, chromium builders all archive to google storage. To my knowledge, the archive step was never treated as a time bottle neck.

http://reviews.llvm.org/D9453

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list