1808586 - Wait for Glean initialization in GleanDebugActivity

Status: RESOLVED FIXED

(Data Platform and Tools :: Glean: SDK, task, P1)

Description

[Zac McKenney [:zmckenney]]

Because of the current race condition for determining Glean.isInitialized() in GleanDebugActivity, fenix tests now fail. Adding a wait with timeout will remove the race condition and ensure that tests are able to use Glean after initialization.

This is in relation to findings from this Bugzilla ticket 1808231, which fails due to changes in Fenix application initialization changes made in fenix pr 28350.

Comment 1

[Jan-Erik Rediger [:janerik]]

This blocks performance tests and we need to prioritize it. We need to get on this early next week.

Comment 2

[Chris Peterson [:cpeterson]]

Tracking for Fenix 111 because Dave Hunt says this Glean issue is causing a high number of failures (~50%) in Fenix performance tests in CI.

Comment 3

[BMO Automation]

Attachment: [mozilla/glean] Bug 1808586 - Kotlin: GleanDebugActivity can run without Glean being initialized (#2336)

Comment 4

[Jan-Erik Rediger [:janerik]]

The PR is merged.
This requires a new Glean release. If bug 1805427 lands, I can get on that.
Until then I keep this bug open.

Comment 5

[Jan-Erik Rediger [:janerik]]

Attachment: Bug 1808586 - Upgrade to Glean v52.0.1 r?chutten!

Depends on D167245

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/076e5dd4ec05
Upgrade to Glean v52.0.1 r=chutten,supply-chain-reviewers

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/94f6ca4f4103
Upgrade to Glean v52.0.1 r=chutten,supply-chain-reviewers

Comment 8

[Atila Butkovits]

Relanded the push. The real culprit was a different push.

Comment 9

[Atila Butkovits]

Backed out for causing build bustages complaining about Cargo and Glean.

Backout link: https://hg.mozilla.org/integration/autoland/rev/62c467a2a4d0d154a5abdf806dde1f052dc412cc

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&selectedTaskRun=Kf3v87aYQfW7yqt-300jJQ.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=b1dad9107c06df86f3107e58c27d88a50b6c53c1

Failure log: https://treeherder.mozilla.org/logviewer?job_id=402989836&repo=autoland&lineNumber=359

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/36264c44c201
Upgrade to Glean v52.0.1 r=chutten,supply-chain-reviewers

Comment 11

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/36264c44c201