1689002 - Perma Tier 2 0:36.78 python/mozbuild/mozbuild/test/configure/lint.py::Lint::test_browser TEST-UNEXPECTED-FAIL | test_mobile_android TEST-UNEXPECTED-FAIL

Status: RESOLVED FIXED

(Toolkit :: Crash Reporting, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: btara [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=327892006&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/Yb6bhpaTSFO5Rt171V9nIg/runs/0/artifacts/public/logs/live_backing.log

---

...
[task 2021-01-26T22:22:39.059Z]  0:36.78 /builds/worker/checkouts/gecko/python/mozbuild/mozbuild/test/configure/lint.py
[task 2021-01-26T22:22:39.059Z]  0:36.78 ============================= test session starts ==============================
[task 2021-01-26T22:22:39.059Z]  0:36.78 platform linux -- Python 3.6.9, pytest-3.6.2, py-1.5.4, pluggy-0.6.0 -- /builds/worker/checkouts/gecko/obj-x86_64-pc-linux-gnu/_virtualenvs/python-test/bin/python
[task 2021-01-26T22:22:39.059Z]  0:36.78 rootdir: /builds/worker/checkouts/gecko, inifile: /builds/worker/checkouts/gecko/config/mozunit/mozunit/pytest.ini
[task 2021-01-26T22:22:39.059Z]  0:36.78 collecting ... collected 6 items
[task 2021-01-26T22:22:39.059Z]  0:36.78 
[task 2021-01-26T22:22:39.059Z]  0:36.78 python/mozbuild/mozbuild/test/configure/lint.py::Lint::test_browser TEST-UNEXPECTED-FAIL
[task 2021-01-26T22:22:39.059Z]  0:36.78 python/mozbuild/mozbuild/test/configure/lint.py::Lint::test_extensions PASSED
[task 2021-01-26T22:22:39.059Z]  0:36.78 python/mozbuild/mozbuild/test/configure/lint.py::Lint::test_js PASSED
[task 2021-01-26T22:22:39.059Z]  0:36.78 python/mozbuild/mozbuild/test/configure/lint.py::Lint::test_memory PASSED
[task 2021-01-26T22:22:39.059Z]  0:36.78 python/mozbuild/mozbuild/test/configure/lint.py::Lint::test_mobile_android TEST-UNEXPECTED-FAIL
[task 2021-01-26T22:22:39.059Z]  0:36.78 python/mozbuild/mozbuild/test/configure/lint.py::Lint::test_tools_update-programs PASSED
[task 2021-01-26T22:22:39.059Z]  0:36.78 
[task 2021-01-26T22:22:39.059Z]  0:36.78 =================================== FAILURES ===================================
[task 2021-01-26T22:22:39.059Z]  0:36.78 ______________________________ Lint.test_browser _______________________________
[task 2021-01-26T22:22:39.059Z]  0:36.78 
[task 2021-01-26T22:22:39.059Z]  0:36.78 self = <lint.Lint testMethod=test_browser>
[task 2021-01-26T22:22:39.059Z]  0:36.78 
[task 2021-01-26T22:22:39.059Z]  0:36.78     def test(self):
[task 2021-01-26T22:22:39.059Z]  0:36.78 >       return func(self, project)
[task 2021-01-26T22:22:39.059Z]  0:36.78 
[task 2021-01-26T22:22:39.059Z]  0:36.78 python/mozbuild/mozbuild/test/configure/lint.py:26:
[task 2021-01-26T22:22:39.059Z]  0:36.78 _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
[task 2021-01-26T22:22:39.060Z]  0:36.78 python/mozbuild/mozbuild/test/configure/lint.py:65: in lint
[task 2021-01-26T22:22:39.060Z]  0:36.78     sandbox.run(os.path.join(topsrcdir, "moz.configure"))
[task 2021-01-26T22:22:39.060Z]  0:36.79 python/mozbuild/mozbuild/configure/lint.py:47: in run
[task 2021-01-26T22:22:39.060Z]  0:36.79     self._check_dependencies(dep)
[task 2021-01-26T22:22:39.060Z]  0:36.79 python/mozbuild/mozbuild/configure/lint.py:156: in _check_dependencies
[task 2021-01-26T22:22:39.060Z]  0:36.79     self._raise_from(e, func)
[task 2021-01-26T22:22:39.060Z]  0:36.79 python/mozbuild/mozbuild/configure/lint.py:121: in _raise_from
[task 2021-01-26T22:22:39.060Z]  0:36.79     thrower(exception)
[task 2021-01-26T22:22:39.060Z]  0:36.79 _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
[task 2021-01-26T22:22:39.060Z]  0:36.79 
[task 2021-01-26T22:22:39.060Z]  0:36.79 e = ConfigureError('The dependency on `target` is unused',)
[task 2021-01-26T22:22:39.060Z]  0:36.79 
[task 2021-01-26T22:22:39.060Z]  0:36.79 >   @depends(target)
[task 2021-01-26T22:22:39.060Z]  0:36.79     def oxidized_breakpad(target):
[task 2021-01-26T22:22:39.060Z]  0:36.79 E   mozbuild.configure.ConfigureError: The dependency on `target` is unused
[task 2021-01-26T22:22:39.060Z]  0:36.79 
[task 2021-01-26T22:22:39.060Z]  0:36.79 toolkit/moz.configure:2489: ConfigureError
[task 2021-01-26T22:22:39.060Z]  0:36.79 ------------------------------ Captured log call -------------------------------
[task 2021-01-26T22:22:39.060Z]  0:36.79 __init__.py                432 INFO     Adding configure options from /builds/worker/checkouts/gecko/python/mozbuild/mozbuild/test/configure/data/empty_mozconfig
[task 2021-01-26T22:22:39.060Z]  0:36.79 __init__.py                432 INFO     checking for vcs source checkout...
[task 2021-01-26T22:22:39.060Z]  0:36.79 __init__.py                432 INFO     hg
[task 2021-01-26T22:22:39.060Z]  0:36.79 __init__.py                432 INFO     checking whether cross compiling...
[task 2021-01-26T22:22:39.060Z]  0:36.79 __init__.py                432 INFO     no
[task 2021-01-26T22:22:39.060Z]  0:36.79 ___________________________ Lint.test_mobile_android ___________________________
[task 2021-01-26T22:22:39.060Z]  0:36.79 
...```

Comment 1

[Bogdan Tara[:bogdan_tara | bogdant]]

Gabriele, could you, please, take a look?

Comment 2

[Gabriele Svelto [:gsvelto]]

Patch incoming

Comment 3

[Gabriele Svelto [:gsvelto]]

Attachment: Bug 1689002 - Fix linter issue when disabling the oxidized Linux minidump writer code r=KrisWright

Comment 4

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

Set release status flags based on info from the regressing bug 1688849

Comment 5

[Intermittent Failures Robot]

25 failures in 711 pushes (0.035 failures/push) were associated with this bug yesterday.

Repository breakdown:

• mozilla-central: 7
• autoland: 18

Platform and build breakdown:

• macosx1014-64: 8
  • opt: 8
• windows10-64: 8
  • opt: 8
• linux1804-64: 9
  • opt: 9

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1689002&startday=2021-01-27&endday=2021-01-27&tree=all

Comment 6

[Gabriele Svelto [:gsvelto]]

Bug 1688882 contains the fix for this too

Comment 7

[Julien Cristau [:jcristau]]

We should still get a fix for this on beta, presumably separately from bug 1688882?

Comment 8

[Gabriele Svelto [:gsvelto]]

Good point, I'll resurrect the patch here just for beta

Comment 9

[Gabriele Svelto [:gsvelto]]

Comment on attachment 9199468 [details]
Bug 1689002 - Fix linter issue when disabling the oxidized Linux minidump writer code r=KrisWright

Beta/Release Uplift Approval Request

• User impact if declined: None, but the linter complains
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This just keeps the linter happy on beta, a more comprehensive change has been landed on nightly
• String changes made/needed: none

Comment 10

[Pascal Chevrel:pascalc]

Comment on attachment 9199468 [details]
Bug 1689002 - Fix linter issue when disabling the oxidized Linux minidump writer code r=KrisWright

Approved for 86 beta 4, thanks.

Comment 11

[Julien Cristau [:jcristau]]

Bug 1688882 fixed this on trunk.

Comment 12

[Julien Cristau [:jcristau]]

https://hg.mozilla.org/releases/mozilla-beta/rev/9cc1589e5b1e

Comment 13

[Intermittent Failures Robot]

47 failures in 3891 pushes (0.012 failures/push) were associated with this bug in the last 7 days.

This is the #15 most frequent failure this week.

** This failure happened more than 30 times this week! Resolving this bug is a high priority. **

** Try to resolve this bug as soon as possible. If unresolved for 2 weeks, the affected test(s) may be disabled. **

Repository breakdown:

• autoland: 28
• mozilla-central: 10
• mozilla-beta: 3
• try: 6

Platform and build breakdown:

• windows10-64: 14
  • opt: 14
• macosx1014-64: 14
  • opt: 14
• linux1804-64: 19
  • opt: 19

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1689002&startday=2021-01-25&endday=2021-01-31&tree=all