1615358 - Cleanup group names in ActiveData (backfill corrections to existing data)

Status: RESOLVED WONTFIX

(Testing :: General, enhancement, P3)

Description

[Marco Castelluccio [:marco]]

We can definitely fix slashes (bug 1613939) from past tasks, can we do the same for group names that contained absolute paths?

Comment 1

[Kyle Lahnakoski [:ekyle]]

:marco May you add an example, and an artifact link with a problem?

Comment 2

[Marco Castelluccio [:marco]]

:gbrown fixed these problems in mozilla-central in bug 1613929, bug 1613937, and bug 1613939, so the group names going forward will be good.
I was wondering if we could somehow "backfill" the fixes (by making the ActiveData ETL rewrite the group names) so we have good group names for the past too.

Some bad examples were: Z:\task_1580434589\build\tests\marionette\tests\testing\marionette\harness\marionette_harness\tests\unit-tests.ini or /Users/cltbld/tasks/task_1580435791/build/tests/marionette/tests/testing/marionette/harness/marionette_harness/tests/unit-tests.ini or devtools\client\shared\components\test\browser\browser.ini (instead of devtools/client/shared/components/test/browser/browser.ini)

Comment 3

[Kyle Lahnakoski [:ekyle]]

:marco, Backfilling in general needs a some thought. I appreciate the importance of this.

https://github.com/klahnakoski/ActiveData-ETL/issues/72

Comment 4

[Marco Castelluccio [:marco]]

We could possibly also do this in mozci, might be easier.

Comment 5

[Kyle Lahnakoski [:ekyle]]

:gbrown So, it turns out the problem is in the harness: From

https://github.com/klahnakoski/ActiveData-ETL/issues/66#issuecomment-589832120

The {group: [test1, test2, ..., testN]} map maps to an array of strings, while the test names are the concatenated result:

"layout/reftests/list-item/numbering-1.html == layout/reftests/list-item/numbering-1-ref.html"

Comment 6

[Marco Castelluccio [:marco]]

I've filed bug 1617636 for that problem, as this bug is about applying workarounds to ActiveData to backfill the harness fixes.

Comment 7

[Marco Castelluccio [:marco]]

We switched to using hgmo, the Treeherder API and the Taskcluster API, so this is no longer needed.