+dgarrett@

You touched some code around here lately, I think. Any chance this was collateral damage?

I did change that. I forget why, probably not for any important reason.

I found the logic that was generating that, based on the upload URL:
  '/'.join(url.split('/')[-3:-1]

I'll see if I can come up with a better way to generate that data.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/9df16c30b34e6a226ab6824de16951174b3498c4

commit 9df16c30b34e6a226ab6824de16951174b3498c4
Author: Don Garrett <dgarrett@google.com>
Date: Tue Aug 09 22:47:51 2016

ReportStage: Adjust where artifacts are linked.

We want to generate index.html files and use them for public build
artifacts, so that public builds can be viewed from an unauthenticated
account.

However, we want to link directly to pantheon for private builds so that
developers can more easily navigate to related artifacts.

Rename _UploadArchiveIndex -> _LinkArtifacts, and move link generation
directly into it so that it can adjust the links as needed.

BUG= chromium:633330 ,  chromium:638254 
TEST=run_tests

Change-Id: If04d2c041af4a0888098952bccb3e1b910c1d83a
Reviewed-on: https://chromium-review.googlesource.com/367516
Commit-Ready: Don Garrett <dgarrett@chromium.org>
Tested-by: Don Garrett <dgarrett@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>

[modify] https://crrev.com/9df16c30b34e6a226ab6824de16951174b3498c4/cbuildbot/stages/report_stages_unittest.py
[modify] https://crrev.com/9df16c30b34e6a226ab6824de16951174b3498c4/cbuildbot/stages/report_stages.py

I believe this is now fixed.

https://uberchromegw.corp.google.com/i/chromeos/builders/beaglebone-paladin/builds/11021

bulk verified