Load files from partner bucket instead of internal bucket.

Load files from partner bucket instead of internal bucket.
This unblocks fix for https://code.google.com/p/chromium/issues/detail?id=448092

BUG=

Committed: https://crrev.com/541ccc34e7fd730e26ad8f213f741fc8f7f472ad
Cr-Commit-Position: refs/heads/master@{#318495}

[prabhur1, 2015-02-19 02:15:21]

prabhur@chromium.org changed reviewers:
+ aiolos@chromium.org, prabhur@chromium.org

[prabhur1, 2015-02-19 02:15:21]

Not a final version.
Sending to confirm if this is the right fix. Also needs to be tested once files
are moved from internal to partner bucket.

[sullivan, 2015-02-19 14:46:48]

On 2015/02/19 02:15:21, prabhur1 wrote:
> Not a final version.
> Sending to confirm if this is the right fix. Also needs to be tested once
files
> are moved from internal to partner bucket.

Thanks for taking a look! This should work once the files are moved.

[sullivan, 2015-02-23 15:03:34]

On 2015/02/19 14:46:48, sullivan wrote:
> On 2015/02/19 02:15:21, prabhur1 wrote:
> > Not a final version.
> > Sending to confirm if this is the right fix. Also needs to be tested once
> files
> > are moved from internal to partner bucket.
> 
> Thanks for taking a look! This should work once the files are moved.

Ping on this. Who is moving the files?

[chromium-reviews, 2015-02-23 18:38:44]

From Kari's response in the other thread I assumed she would be
copying/moving the files.


Prabhu

On Mon, Feb 23, 2015 at 7:03 AM, <sullivan@chromium.org> wrote:

> On 2015/02/19 14:46:48, sullivan wrote:
>
>> On 2015/02/19 02:15:21, prabhur1 wrote:
>> > Not a final version.
>> > Sending to confirm if this is the right fix. Also needs to be tested
>> once
>> files
>> > are moved from internal to partner bucket.
>>
>
>  Thanks for taking a look! This should work once the files are moved.
>>
>
> Ping on this. Who is moving the files?
>
> https://codereview.chromium.org/935203002/
>

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

[aiolos (Not reviewing), 2015-02-27 17:59:25]

On 2015/02/23 18:38:44, chromium-reviews wrote:
> From Kari's response in the other thread I assumed she would be
> copying/moving the files.
> 
> 
> Prabhu
> 
> On Mon, Feb 23, 2015 at 7:03 AM, <mailto:sullivan@chromium.org> wrote:
> 
> > On 2015/02/19 14:46:48, sullivan wrote:
> >
> >> On 2015/02/19 02:15:21, prabhur1 wrote:
> >> > Not a final version.
> >> > Sending to confirm if this is the right fix. Also needs to be tested
> >> once
> >> files
> >> > are moved from internal to partner bucket.
> >>
> >
> >  Thanks for taking a look! This should work once the files are moved.
> >>
> >
> > Ping on this. Who is moving the files?
> >
> > https://codereview.chromium.org/935203002/
> >
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

lgtm

[sullivan, 2015-02-27 18:00:42]

sullivan@chromium.org changed reviewers:
+ sullivan@chromium.org

[sullivan, 2015-02-27 18:00:42]

lgtm

[aiolos (Not reviewing), 2015-02-27 18:29:42]

The CQ bit was checked by aiolos@chromium.org

[commit-bot: I haz the power, 2015-02-27 18:30:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/935203002/1

[prabhur1, 2015-02-27 18:43:55]

On 2015/02/27 18:30:22, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/935203002/1

Confirmed that local run fetches files from gs://chrome-partner-telemetry/

[vignatti (out of this project), 2015-02-27 19:17:46]

lgtm

Thanks for doing this!

[commit-bot: I haz the power, 2015-02-27 19:43:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-27 19:43:46]

Patchset 1 (id:??) landed as
https://crrev.com/541ccc34e7fd730e26ad8f213f741fc8f7f472ad
Cr-Commit-Position: refs/heads/master@{#318495}