Closed Bug 1652615 Opened 5 years ago Closed 5 years ago

Sentry and telemetry should share "is_telemetry_enabled" logic

Categories

(Firefox Build System :: Mach Core, task)

task

Tracking

(firefox80 fixed)

RESOLVED FIXED
mozilla80
Tracking Status
firefox80 --- fixed

People

(Reporter: mhentges, Assigned: mhentges)

References

Details

Attachments

(1 file)

The sentry setup logic currently parses .mozbuild/machrc separately while the existing telemetry checks the already-parsed context.settings.build object. These should be consistent.

Note that this moves settings-loading earlier in the bootstrap procedure,
from "_run()" to "run()".

Assignee: nobody → mhentges
Status: NEW → ASSIGNED
Pushed by mhentges@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d074709201fe Share is_telemetry_enabled() logic between Sentry and telemetry r=firefox-build-system-reviewers,rstewart
Regressions: 1653135
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: