1798871 - Remove android + fission OK/TIMEOUT annotations again

Status: REOPENED

(Testing :: web-platform-tests, defect)

Description

[Tom S [:evilpie]]

The last two or so WPT updates introduced a lot of new meta annotations for otherwise completely working tests. It would be nice to get rid of them again.

  expected:
    if (os == "android") and fission: [OK, TIMEOUT]

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:jgraham, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 2

[Tom Schuster (MoCo)]

This is really quite annoying when triaging still failing tests by looking at the meta directory. Could we try removing all those annotations and see on try how much fails?

Comment 3

[James Graham [:jgraham]]

Yeah, I don't think there's too much difficulty in removing them. The problem really is I don't understand where they came from in the first place, so it's possible the sync will just recreate them again.

Comment 4

[Tom S [:evilpie]]

Could we dry run sync and see if they would come back after deletion? I can't be the only one who is annoyed by this.

Comment 5

[Kagami Rosylight [:saschanaz] (they/them)]

I just remove them whenever I touch any test metadata, and batch removal will be great to prevent such hassle.

Comment 6

[Joshua Marshall]

Attachment: Bug 1798871 - Fix non-unified build compilation error r=jolin!

Comment 7

[Joshua Marshall]

Attachment: WIP: Bug 1798871 - Remove android + fission OK/TIMEOUT annotations

Depends on D181384

Comment 8

[Phabricator Automation]

Comment on attachment 9340028 [details]
Bug 1798871 - Fix non-unified build compilation error r=jolin!

Revision D181384 was moved to bug 1839174. Setting attachment 9340028 [details] to obsolete.

Comment 9

[Joshua Marshall]

I see failures when I try to remove some of these annotations: https://treeherder.mozilla.org/jobs?repo=try&revision=296fec4fda52037dbd4de573ec2a2923ae238cbf. Created https://bugzilla.mozilla.org/show_bug.cgi?id=1843877.

Comment 10

[Kagami Rosylight [:saschanaz] (they/them)]

That doesn't mean every marked test are affected, the annotation has been successfully removed here and there.

Comment 11

[Jihye Hong]

Attachment: Bug 1798871 - Remove android + fission OK/TIMEOUT annotations from content-visibility TCs

This patch removes android + fission OK/TIMEOUT annotation from working content-visibility TCs

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3577bdad9e31
Remove android + fission OK/TIMEOUT annotations from content-visibility TCs r=emilio

Comment 13

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/3577bdad9e31

Comment 14

[Jihye Hong]

Attachment: Bug 1798871 - Remove android + fission OK/TIMEOUT annotations from remaining content-visibility TCs

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9577d0b99781
Remove android + fission OK/TIMEOUT annotations from remaining content-visibility TCs r=emilio

Comment 16

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/9577d0b99781

Comment 17

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1798871 - Remove android + fission OK/TIMEOUT annotations from web-locks r=#dom-core,smaug

Comment 18

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1798871 - Remove android + fission OK/TIMEOUT annotations from web-locks r=#dom-core,smaug

Comment 19

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bdeabe432169
Remove android + fission OK/TIMEOUT annotations from web-locks r=smaug

Comment 20

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/bdeabe432169

Comment 21

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/bdeabe432169

Comment 22

[BugBot (nomail) [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.