Add metadata to json results before uploading

Add metadata to json results before uploading

BUG=649762
Committed: https://chromium.googlesource.com/chromium/tools/build/+/86356c164014cf93140c7548236b2761fe6dc93c

[nednguyen, 2016-11-11 00:51:28]

nednguyen@google.com changed reviewers:
+ estaab@chromium.org

[nednguyen, 2016-11-11 00:53:15]

https://codereview.chromium.org/2494883003/diff/1/scripts/slave/recipe_module...
File
scripts/slave/recipe_modules/test_results/resources/upload_test_results_unittest.py
(left):

https://codereview.chromium.org/2494883003/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/test_results/resources/upload_test_results_unittest.py:155:
files = [(os.path.basename(input_json_file_path), input_json_file_path)]
I notice that this wasn't updated when you uploaded your previous CL. We will
need to move all these unittest to
scripts/slave/recipe_modules/test_results/resources/test/ or them to triggered
(https://cs.chromium.org/chromium/build/PRESUBMIT.py?rcl=0&l=119). I will do
that in a different CL.

[estaab, 2016-11-11 04:57:31]

lgtm

Looks great, thanks for the quick turnaround!

https://codereview.chromium.org/2494883003/diff/1/scripts/slave/recipe_module...
File
scripts/slave/recipe_modules/test_results/resources/upload_test_results_unittest.py
(left):

https://codereview.chromium.org/2494883003/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/test_results/resources/upload_test_results_unittest.py:155:
files = [(os.path.basename(input_json_file_path), input_json_file_path)]
On 2016/11/11 at 00:53:15, nednguyen wrote:
> I notice that this wasn't updated when you uploaded your previous CL. We will
need to move all these unittest to
scripts/slave/recipe_modules/test_results/resources/test/ or them to triggered
(https://cs.chromium.org/chromium/build/PRESUBMIT.py?rcl=0&l=119). I will do
that in a different CL.

Ah, thanks for fixing this.

[nednguyen, 2016-11-11 11:34:24]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2016-11-11 11:34:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-11 11:44:11]

Description was changed from

==========
Add metadata to json results before uploading

BUG=649762
==========

to

==========
Add metadata to json results before uploading

BUG=649762

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/86356c164014cf93140c...
==========

[commit-bot: I haz the power, 2016-11-11 11:44:12]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/86356c164014cf93140c...