Intermittent browser/modules/test/browser_CaptivePortalWatcher.js | Test timed out -

NEW
Assigned to

Status

()

defect
P5
normal
3 years ago
5 months ago

People

(Reporter: intermittent-bug-filer, Assigned: nhnt11, NeedInfo)

Tracking

({intermittent-failure, leave-open})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Blocks: 1279491
Flags: needinfo?(nhnt11)
See Also: → 1313862
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
FYI, this is basically permafailing on Win8 e10s (which only runs on Try and the Ash repo since we don't have the capacity to do so for mozilla-central & integration repos).
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
this is now our #4 intermittent.  I saw recent activity on this test via bug 1313862, it would be nice to communicate in this bug to indicate priority and expected fix times.
Assignee

Comment 8

3 years ago
I propose we disable this test on Windows for the moment, until more prioritized captive portal work has landed. (see https://public.etherpad-mozilla.org/p/captive-portal-roadmap)
Flags: needinfo?(nhnt11)
Comment on attachment 8808136 [details]
Bug 1313894 - Temporarily disable browser_CaptivePortalWatcher.js on Windows.

https://reviewboard.mozilla.org/r/91046/#review90770

::: browser/modules/test/browser.ini:10
(Diff revision 1)
>  [browser_BrowserUITelemetry_buckets.js]
>  [browser_BrowserUITelemetry_defaults.js]
>  [browser_BrowserUITelemetry_sidebar.js]
>  [browser_BrowserUITelemetry_syncedtabs.js]
>  [browser_CaptivePortalWatcher.js]
> +skip-if = os == "win" # Bug 1313894

OK, but the bug reference should point to the bug that will deal with re-enabling the test. Either mark this as leave-open or file a new bug and reference it here.
Attachment #8808136 - Flags: review?(past) → review+
Assignee

Comment 10

3 years ago
(In reply to Panos Astithas [:past] from comment #9)
> Comment on attachment 8808136 [details]
> Bug 1313894 - Temporarily disable browser_CaptivePortalWatcher.js on Windows.
> 
> https://reviewboard.mozilla.org/r/91046/#review90770
> 
> ::: browser/modules/test/browser.ini:10
> (Diff revision 1)
> >  [browser_BrowserUITelemetry_buckets.js]
> >  [browser_BrowserUITelemetry_defaults.js]
> >  [browser_BrowserUITelemetry_sidebar.js]
> >  [browser_BrowserUITelemetry_syncedtabs.js]
> >  [browser_CaptivePortalWatcher.js]
> > +skip-if = os == "win" # Bug 1313894
> 
> OK, but the bug reference should point to the bug that will deal with
> re-enabling the test. Either mark this as leave-open or file a new bug and
> reference it here.

Cool, I intended to leave this open for the eventual fix. Added the flag.
Keywords: leave-open
Assignee

Updated

3 years ago
Assignee: nobody → nhnt11

Comment 11

3 years ago
Pushed by nhnt11@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/6c1d4d2b9110
Temporarily disable browser_CaptivePortalWatcher.js on Windows. r=past
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
The leave-open keyword is there and there is no activity for 6 months.
:nhnt11, maybe it's time to close this bug?
Flags: needinfo?(nhnt11)
Comment hidden (Intermittent Failures Robot)

(In reply to Release mgmt bot [:sylvestre / :calixte] from comment #17)

The leave-open keyword is there and there is no activity for 6 months.
:nhnt11, maybe it's time to close this bug?

The test is still disabled on Windows, I think that keeps it open doesn't it?

Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.