1623630 - consume distribution.ini from partners instead of defining it in-tree

Status: RESOLVED FIXED

(Release Engineering :: Release Automation: Other, task)

Description

[Mihai Tabara [:mtabara]⌚️GMT]

Currently we're using the distribution.ini to set the right distribution ID in Flatpak to be able to read Telemetry from our side. For Snap we consume the mozilla-partners one here while in Flatpaks we have another file defined in-tree.

This increases the chance to have them out-of-sync at some point so let's switch to doing the same in Flatpak world.

Comment 1

[Mihai Tabara [:mtabara]⌚️GMT]

Attachment: Bug 1623630 - flatpak distribution.ini to be consumed from partners. r=rail

Comment 2

[Mihai Tabara [:mtabara]⌚️GMT]

Comment on attachment 9134604 [details]
Bug 1623630 - flatpak distribution.ini to be consumed from partners. r=rail

Beta/Release Uplift Approval Request

• User impact if declined: None. It's an automation change to consume the distribution.ini file from the partners-repo directly, rather than duplicating the same file in-tree.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky):
• String changes made/needed:

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/517ca5e228dd
flatpak distribution.ini to be consumed from partners. r=rail

Comment 4

[Julien Cristau [:jcristau]]

Comment on attachment 9134604 [details]
Bug 1623630 - flatpak distribution.ini to be consumed from partners. r=rail

This is effectively a no-op right now, I don't think it's critical to uplift.

Comment 5

[Julien Cristau [:jcristau]]

Comment on attachment 9134604 [details]
Bug 1623630 - flatpak distribution.ini to be consumed from partners. r=rail

sigh

Comment 6

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/517ca5e228dd