[GN]: Fix bug in roller script with try servers

[GN]: Fix bug in roller script with try servers

The API endpoint being used appears to be broken. There is a direct URL
to the try_server_results data, so use that.

BUG=none

Committed: https://crrev.com/e1bbf3218766ea727b229398aaae0df17fe3856b
Cr-Commit-Position: refs/heads/master@{#350906}

[Bons, 2015-09-25 20:18:50]

andybons@chromium.org changed reviewers:
+ dpranke@chromium.org

[Dirk Pranke, 2015-09-25 20:28:56]

Hm. The old api was working for me as of a couple days ago, though your fix is
probably better regardless.

What sort of error are you getting?

[Bons, 2015-09-25 20:30:26]

It's returning an empty array.

Issue: https://codereview.chromium.org/1366253004/
Busted tryjob data url: https://codereview.chromium.org/api/1359133003/1/
New data url:
https://codereview.chromium.org/api/1359133003/1/try_job_results

On Fri, Sep 25, 2015 at 4:28 PM <dpranke@chromium.org> wrote:

> Hm. The old api was working for me as of a couple days ago, though your fix
> is
> probably better regardless.
>
> What sort of error are you getting?
>
> https://codereview.chromium.org/1365393002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Dirk Pranke, 2015-09-25 20:32:50]

Hm. That looks like a regression. Mind filing a bug?

lgtm to switch to the updated endpoint, though.

On Fri, Sep 25, 2015 at 1:30 PM, Andrew Bonventre <andybons@chromium.org>
wrote:

> It's returning an empty array.
>
> Issue: https://codereview.chromium.org/1366253004/
> Busted tryjob data url: https://codereview.chromium.org/api/1359133003/1/
> New data url:
> https://codereview.chromium.org/api/1359133003/1/try_job_results
>
> On Fri, Sep 25, 2015 at 4:28 PM <dpranke@chromium.org> wrote:
>
>> Hm. The old api was working for me as of a couple days ago, though your
>> fix
>> is
>> probably better regardless.
>>
>> What sort of error are you getting?
>>
>> https://codereview.chromium.org/1365393002/
>>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Bons, 2015-09-25 20:33:52]

Sure. Also updated description to say 'Roll buildtools'

On Fri, Sep 25, 2015 at 4:32 PM Dirk Pranke <dpranke@chromium.org> wrote:

> Hm. That looks like a regression. Mind filing a bug?
>
> lgtm to switch to the updated endpoint, though.
>
> On Fri, Sep 25, 2015 at 1:30 PM, Andrew Bonventre <andybons@chromium.org>
> wrote:
>
>> It's returning an empty array.
>>
>> Issue: https://codereview.chromium.org/1366253004/
>> Busted tryjob data url: https://codereview.chromium.org/api/1359133003/1/
>> New data url:
>> https://codereview.chromium.org/api/1359133003/1/try_job_results
>>
>> On Fri, Sep 25, 2015 at 4:28 PM <dpranke@chromium.org> wrote:
>>
>>> Hm. The old api was working for me as of a couple days ago, though your
>>> fix
>>> is
>>> probably better regardless.
>>>
>>> What sort of error are you getting?
>>>
>>> https://codereview.chromium.org/1365393002/
>>>
>>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Bons, 2015-09-25 20:34:42]

The CQ bit was checked by andybons@chromium.org

[commit-bot: I haz the power, 2015-09-25 20:36:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1365393002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1365393002/20001

[commit-bot: I haz the power, 2015-09-25 20:36:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-25 20:36:19]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Bons, 2015-09-25 20:39:46]

On 2015/09/25 20:36:19, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

Sigh. Looks like it didn't pick up your lgtm, Dirk.

[Dirk Pranke, 2015-09-25 20:40:15]

On 2015/09/25 20:36:19, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

that's weird.

lgtm .

[Dirk Pranke, 2015-09-25 20:40:19]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2015-09-25 20:41:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1365393002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1365393002/20001

[commit-bot: I haz the power, 2015-09-25 20:53:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-25 20:53:35]

Patchset 2 (id:??) landed as
https://crrev.com/e1bbf3218766ea727b229398aaae0df17fe3856b
Cr-Commit-Position: refs/heads/master@{#350906}