Closed Bug 1869679 Opened 1 year ago Closed 1 year ago

Perma [tier 2] /builds/worker/checkouts/gecko/toolkit/mozapps/defaultagent/tests/xpcshell/test_windows_mutex.js:12:3 | Use of unknown interface Ci.nsIWindowsMutexFactory (mozilla/valid-ci-uses)

Categories

(Toolkit :: Default Browser Agent, defect, P3)

defect

Tracking

()

RESOLVED FIXED
123 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox120 --- unaffected
firefox121 --- unaffected
firefox122 --- wontfix
firefox123 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: nrishel)

References

(Regression)

Details

(Keywords: intermittent-failure, regression, Whiteboard: [fidedi-wdba])

Attachments

(1 file)

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


[task 2023-12-13T00:45:47.930Z] /builds/worker/checkouts/gecko/js
[task 2023-12-13T00:45:47.936Z] 00:45:47.936 eslint (67) | ESLint command: /usr/local/bin/node /builds/worker/checkouts/gecko/node_modules/eslint/bin/eslint.js --ext [mjs,js,jsm,json,jsx,html,sjs,xhtml] --format json --no-error-on-unmatched-pattern --quiet --rule mozilla/valid-services-property: error --rule mozilla/valid-ci-uses: error --ignore-pattern third_party/webkit/PerformanceTests/Speedometer/resources/todomvc/architecture-examples/inferno/node_modules --ignore-pattern js/src/vtune/ittnotify_static.c --ignore-pattern third_party/webkit/PerformanceTests/Speedometer/resources/todomvc/functional-prog-examples/elm/node_modules --ignore-pattern browser/extensions/formautofill/content/third-party --ignore-pattern js/src/vtune/ittnotify.h --ignore-pattern modules/brotli --ignore-pattern modules/fdlibm --ignore-pattern js/src/zydis --ignore-pattern js/src/vtune/ittnotify_config.h --ignore-pattern intl/icu_capi --ignore-pattern third_party/webkit/PerformanceTests/Speedometer/resources/todomvc/architecture-examples/angularjs/node_modules --ignore-pattern third_party/webkit/PerformanceTests/Speedometer/resources/todomvc/architecture-examples/backbone/node_modules --ignore-pattern js/src/ctypes/libffi --ignore-pattern js/src/vtune/legacy --ignore-pattern intl/unicharutil/util/nsSpecialCasingData.cpp --ignore-pattern browser/components/newtab/content-src/asrouter/schemas/MessagingExperiment.schema.json --ignore-pattern js/src/vtune/ittnotify_static.h --ignore-pattern third_party/webkit/PerformanceTests/Speedometer/resources/todomvc/dependency-examples/flight/flight/node_modules --ignore-pattern intl/unicharutil/util/nsUnicodePropertyData.cpp --ignore-pattern modules/woff2 --ignore-pattern intl/components/src/UnicodeScriptCodes.h --ignore-pattern intl/icu --ignore-pattern js/src/vtune/jitprofiling.h --ignore-pattern browser/components/newtab/content-src/asrouter/schemas/BackgroundTaskMessagingExperiment.schema.json --ignore-pattern browser/extensions/formautofill/test/fixtures/third_party --ignore-pattern browser/locales/l10n-onchange-changesets.json --ignore-pattern third_party/webkit/PerformanceTests/Speedometer/resources/todomvc/vanilla-examples/es2015/node_modules --ignore-pattern third_party/webkit/PerformanceTests/Speedometer/resources/todomvc/architecture-examples/react/node_modules --ignore-pattern js/src/vtune/jitprofiling.c --ignore-pattern js/src/jit/arm64/vixl --ignore-pattern browser/components/newtab/vendor --ignore-pattern modules/freetype2 --ignore-pattern browser/locales/l10n-changesets.json --ignore-pattern modules/zlib --ignore-pattern browser/components/pocket/content/panels/css/normalize.scss --ignore-pattern js/src/vtune/disable_warnings.h --ignore-pattern js/src/dtoa.c --ignore-pattern browser/components/aboutwelcome/content/aboutwelcome.bundle.js --ignore-pattern modules/xz-embedded --ignore-pattern js/src/vtune/ittnotify_types.h --ignore-pattern js/src/octane --ignore-pattern js/src/editline --ignore-pattern layout/docs/css-gap-decorations --ignore-pattern third_party/webkit/PerformanceTests/Speedometer/resources/todomvc/architecture-examples/jquery/node_modules --ignore-pattern third_party/webkit/PerformanceTests/Speedometer/resources/todomvc/vanilla-examples/vanillajs/node_modules /builds/worker/checkouts/gecko/browser /builds/worker/checkouts/gecko/intl /builds/worker/checkouts/gecko/image /builds/worker/checkouts/gecko/gradle /builds/worker/checkouts/gecko/docs /builds/worker/checkouts/gecko/.prettierrc.js /builds/worker/checkouts/gecko/uriloader /builds/worker/checkouts/gecko/layout /builds/worker/checkouts/gecko/modules /builds/worker/checkouts/gecko/accessible /builds/worker/checkouts/gecko/servo /builds/worker/checkouts/gecko/.cargo /builds/worker/checkouts/gecko/hal /builds/worker/checkouts/gecko/js
[task 2023-12-13T00:46:23.171Z] 00:46:23.169 eslint (68) | Prettier command: /usr/local/bin/node /builds/worker/checkouts/gecko/node_modules/prettier/bin-prettier.js --list-different --no-error-on-unmatched-pattern --quiet --rule mozilla/valid-services-property: error --rule mozilla/valid-ci-uses: error /builds/worker/checkouts/gecko/startupcache /builds/worker/checkouts/gecko/widget /builds/worker/checkouts/gecko/xpcom /builds/worker/checkouts/gecko/build /builds/worker/checkouts/gecko/mozglue /builds/worker/checkouts/gecko/caps /builds/worker/checkouts/gecko/supply-chain /builds/worker/checkouts/gecko/.vscode /builds/worker/checkouts/gecko/media /builds/worker/checkouts/gecko/python /builds/worker/checkouts/gecko/testing /builds/worker/checkouts/gecko/security
[task 2023-12-13T00:46:42.069Z] 00:46:42.68 eslint (68) | Finished in 57.90 seconds
[task 2023-12-13T00:48:46.555Z] 00:48:46.554 eslint (70) | Prettier command: /usr/local/bin/node /builds/worker/checkouts/gecko/node_modules/prettier/bin-prettier.js --list-different --no-error-on-unmatched-pattern --quiet --rule mozilla/valid-services-property: error --rule mozilla/valid-ci-uses: error /builds/worker/checkouts/gecko/ipc /builds/worker/checkouts/gecko/package.json /builds/worker/checkouts/gecko/taskcluster /builds/worker/checkouts/gecko/xpfe /builds/worker/checkouts/gecko/.eslintrc.js /builds/worker/checkouts/gecko/netwerk /builds/worker/checkouts/gecko/extensions /builds/worker/checkouts/gecko/.github /builds/worker/checkouts/gecko/memory /builds/worker/checkouts/gecko/dom /builds/worker/checkouts/gecko/services /builds/worker/checkouts/gecko/view /builds/worker/checkouts/gecko/.eslintrc-test-paths.js
[task 2023-12-13T00:48:57.525Z] 00:48:57.524 eslint (67) | Prettier command: /usr/local/bin/node /builds/worker/checkouts/gecko/node_modules/prettier/bin-prettier.js --list-different --no-error-on-unmatched-pattern --quiet --rule mozilla/valid-services-property: error --rule mozilla/valid-ci-uses: error /builds/worker/checkouts/gecko/browser /builds/worker/checkouts/gecko/intl /builds/worker/checkouts/gecko/image /builds/worker/checkouts/gecko/gradle /builds/worker/checkouts/gecko/docs /builds/worker/checkouts/gecko/.prettierrc.js /builds/worker/checkouts/gecko/uriloader /builds/worker/checkouts/gecko/layout /builds/worker/checkouts/gecko/modules /builds/worker/checkouts/gecko/accessible /builds/worker/checkouts/gecko/servo /builds/worker/checkouts/gecko/.cargo /builds/worker/checkouts/gecko/hal /builds/worker/checkouts/gecko/js
[task 2023-12-13T00:49:36.670Z] 00:49:36.669 eslint (70) | Finished in 232.50 seconds
[task 2023-12-13T00:50:06.670Z] 00:50:06.670 eslint (67) | Finished in 262.51 seconds
[task 2023-12-13T00:50:46.844Z] 00:50:46.844 eslint (69) | Prettier command: /usr/local/bin/node /builds/worker/checkouts/gecko/node_modules/prettier/bin-prettier.js --list-different --no-error-on-unmatched-pattern --quiet --rule mozilla/valid-services-property: error --rule mozilla/valid-ci-uses: error /builds/worker/checkouts/gecko/config /builds/worker/checkouts/gecko/mfbt /builds/worker/checkouts/gecko/storage /builds/worker/checkouts/gecko/remote /builds/worker/checkouts/gecko/.stylelintrc.js /builds/worker/checkouts/gecko/gfx /builds/worker/checkouts/gecko/mobile /builds/worker/checkouts/gecko/editor /builds/worker/checkouts/gecko/docshell /builds/worker/checkouts/gecko/package-lock.json /builds/worker/checkouts/gecko/parser /builds/worker/checkouts/gecko/devtools /builds/worker/checkouts/gecko/tools /builds/worker/checkouts/gecko/toolkit /builds/worker/checkouts/gecko/.babel-eslint.rc.js /builds/worker/checkouts/gecko/chrome
[task 2023-12-13T00:52:05.282Z] 00:52:05.281 eslint (69) | Finished in 381.12 seconds
[task 2023-12-13T00:52:05.297Z] TEST-UNEXPECTED-ERROR | /builds/worker/checkouts/gecko/toolkit/mozapps/defaultagent/tests/xpcshell/test_windows_mutex.js:12:3 | Use of unknown interface Ci.nsIWindowsMutexFactory (mozilla/valid-ci-uses)
[task 2023-12-13T00:52:05.297Z] TEST-UNEXPECTED-ERROR | /builds/worker/checkouts/gecko/toolkit/mozapps/defaultagent/tests/xpcshell/test_windows_mutex.js:29:5 | Use of unknown interface Ci.nsIWindowsMutexFactory (mozilla/valid-ci-uses)
[task 2023-12-13T00:52:05.297Z] TEST-UNEXPECTED-ERROR | /builds/worker/checkouts/gecko/toolkit/mozapps/defaultagent/tests/xpcshell/test_windows_mutex.js:50:5 | Use of unknown interface Ci.nsIWindowsMutexFactory (mozilla/valid-ci-uses)
[taskcluster 2023-12-13 00:52:05.805Z] === Task Finished ===
[taskcluster 2023-12-13 00:52:06.136Z] Unsuccessful task run with exit code: 1 completed in 522.913 seconds

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

:nrishel, since you are the author of the regressor, bug 1866882, could you take a look?

For more information, please visit BugBot documentation.

Huh. I think the issue must be that this interface is Windows only, so the underlying XPCOM details from the Linux64 job https://firefox-ci-tc.services.mozilla.com/tasks/HxjeFpgVSTSw3eJzVegCog don't witness it. And the linting infrastructure doesn't recognize that the test file is also Windows only. I'm not sure what the best outcome is here -- perhaps we can just neuter the relevant lint? Yes, I see examples like this: https://searchfox.org/mozilla-central/rev/da48f565f70a57ac28862090828fbaa7fd8556f6/extensions/pref/autoconfig/src/prefcalls.js#133-135. That makes the most sense to me for now.

I'm pretty sure we saw exactly this error with the nsIDefaultAgent object. Leaving the NI so that I'll investigate after winding down current work.

Assignee: nobody → nrishel
Flags: needinfo?(nrishel)

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

Priority: P5 → P3
Status: NEW → ASSIGNED
See Also: → 1872415
Pushed by nrishel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/786d3c41e703 Mark `nsIWindowsMutexFactory` as a platform specific interface. r=mhughes
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch

The patch landed in nightly and beta is affected.
:nrishel, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox122 to wontfix.

For more information, please visit BugBot documentation.

Flags: needinfo?(nrishel)

Not user facing, not worth uplifting.

Flags: needinfo?(nrishel)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: