Issue metadata
Sign in to add a comment
|
test results of different test steps are uploaded under the same step name |
||||||||||||||||||||
Issue descriptionThis happens mostly on ios-simulator. The problem here might be "test (with patch)" is uploaded as "test". See sample here in this query for data in the last 14 days. https://bigquery.cloud.google.com/results/findit-for-me:US.bquijob_b4a096e_165c5a1a705
,
Sep 11
Does ios use a separate recipe? It would be great if we could resolve the inconsistent naming here.
,
Sep 11
iOS try bots use this recipe: https://chromium.googlesource.com/chromium/tools/build/+/master/scripts/slave/recipes/ios/try.py Can you give an example build with incorrect uploaded test results? I don't know what to make of the query.
,
Sep 11
I see. "Upload test-results" steps don't encode platform information in their step name. Here's an example from the spreadsheet: https://ci.chromium.org/p/chromium/builders/luci.chromium.try/ios-simulator/90617 Compare steps #198 and #217: #198 name: "Upload test-results [component_unittests]" #217 name: "Upload test-results [components_unittests] (2)" Step names do differ, but the difference is outside the brackets. We should encode platform information. e.g. "Upload test-results [components_unittests (iPhone 6s iOS 12.0)]" instead of "Upload test-results [components_unittests] (2)".
,
Oct 22
Infra>Client>iOS is dead; all of its issues should be filed in I>C>C labeled with "iOS".
,
Oct 25
,
Oct 25
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build/+/fdd5011d0209a5a59eadeb77cc849aa2b22f3b4b commit fdd5011d0209a5a59eadeb77cc849aa2b22f3b4b Author: John Budorick <jbudorick@chromium.org> Date: Thu Oct 25 18:07:38 2018 ios: use step name rather than app name when uploading to test-results. Bug: 882637 Change-Id: I12f66990aa7b0667c3af7b3a1813313b5aa08fc0 Reviewed-on: https://chromium-review.googlesource.com/c/1299553 Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> [modify] https://crrev.com/fdd5011d0209a5a59eadeb77cc849aa2b22f3b4b/scripts/slave/README.recipes.md [modify] https://crrev.com/fdd5011d0209a5a59eadeb77cc849aa2b22f3b4b/scripts/slave/recipe_modules/ios/api.py [modify] https://crrev.com/fdd5011d0209a5a59eadeb77cc849aa2b22f3b4b/scripts/slave/recipe_modules/ios/examples/full.expected/basic.json
,
Nov 2
Will check on this once #9 has been live for > 2 weeks.
,
Nov 9
The NextAction date has arrived: 2018-11-09 |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by martiniss@chromium.org
, Sep 10Components: -Infra>Client>Chrome Infra>Client>iOS