Closed Bug 1680230 Opened 4 years ago Closed 4 years ago

Update rkv to 0.16.0

Categories

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

task

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: janerik, Assigned: janerik)

References

Details

(Whiteboard: [telemetry:fog:m6])

Attachments

(4 files)

We need to upgrade rkv, then land that update in m-c in coordination with other users of rkv.

What's the process? Update glean, publish it to crates, then update rkv and glean together in m-c?

Yup, though we're landing another big Glean update right now, so that needs to happen first.

Depends on D98594

We should have made its own bug for the code you're landing, :vporof. Renaming this bug.
We can keep it here for now, but in the future we should probably track these separately.

Keywords: leave-open
Summary: Upgrade rkv in Glean to 0.16.0 → Update rkv to 0.16.0

It's a little bit the other way around – the tsan failure is very likely caused by updating to 0.16, which is an issue that I'm looking into fixing right now. I agree with renaming this bug.

Flags: needinfo?(vporof)

Unfortunately the lock-order-inversion failure seems to be in FOG code.
We might have just been unlucky to find it here.

Talked to Victor in private.
It looks like by upgrading we resurfaced a false-positive (bug 1606804) due the to the change in paths for only Glean/FOG (because we duplicate rkv versions).

A Glean upgrade is coming today or tomorrow also upgrading its rkv dependency, that should fix this without requiring yet another exception for TSAN (as done here: https://searchfox.org/mozilla-central/rev/c14733e74e540b71856cc1d963bc424b38aa4bef/mozglue/build/TsanOptions.cpp#101-102)

Taking over the bug to get that update in.

Thanks Jan-Erik!

Keywords: leave-open
Blocks: 1675286
Blocks: 1677455
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: