Skip to content

Bug 2017277 - [RLB] Expose client_info.os_version - #3419

Merged
travis79 merged 1 commit into
mainfrom
push-pvqykwtwxyyr
Jun 9, 2026
Merged

travis79 merged 1 commit into
mainfrom
push-pvqykwtwxyyr

Conversation

@badboy

@badboy badboy commented Mar 12, 2026

Copy link
Copy Markdown
Member

Note this by itself doesn't even change the value! We still fall back to what whatsys gives us.

@badboy
badboy requested a review from a team as a code owner March 12, 2026 13:40
@badboy
badboy requested review from jeddai and removed request for a team March 12, 2026 13:40
@badboy
badboy force-pushed the push-pvqykwtwxyyr branch from 11319b4 to 88ccd33 Compare March 12, 2026 13:40
@badboy
badboy requested review from chutten and removed request for jeddai March 12, 2026 13:40

@chutten chutten left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chutten chutten left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I guess it doesn't yet. This is still behind RLB's closed doors.

@badboy

badboy commented Mar 12, 2026

Copy link
Copy Markdown
Member Author

Does it require a doc update to mozilla.github.io/glean/book/reference/general/initializing.html#gleaninitializeconfiguration ?

Yes, it does. At least the Rust parts are not correct anymore (ClientInfoMetrics has an additional field).
fwiw, I'm holding back on this for the moment while I figure out if anything in bigquery-etl does anything more with os_version.

@badboy
badboy force-pushed the push-pvqykwtwxyyr branch from 88ccd33 to c7e5d24 Compare March 13, 2026 10:43
@badboy
badboy force-pushed the push-pvqykwtwxyyr branch from c7e5d24 to b584167 Compare March 13, 2026 10:43
@badboy
badboy force-pushed the push-pvqykwtwxyyr branch from b584167 to 858e708 Compare March 16, 2026 12:06
@badboy
badboy force-pushed the push-pvqykwtwxyyr branch 2 times, most recently from 2866d3c to 64985f8 Compare April 15, 2026 15:26
@badboy
badboy force-pushed the push-pvqykwtwxyyr branch 2 times, most recently from 635acfa to 1319c6c Compare May 28, 2026 12:40
@badboy

badboy commented May 28, 2026

Copy link
Copy Markdown
Member Author

/run-ios

@badboy
badboy force-pushed the push-pvqykwtwxyyr branch from 1319c6c to 233e323 Compare June 9, 2026 13:54
@travis79
travis79 merged commit 882c333 into main Jun 9, 2026
29 of 30 checks passed
@travis79
travis79 deleted the push-pvqykwtwxyyr branch June 9, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants