1180673 - Update about:healthreport URL for the new unified content

Status: VERIFIED FIXED

(Toolkit :: Telemetry, defect)

Description

[Georg Fritzsche [:gfritzsche]]

(In reply to Benjamin Smedberg  [:bsmedberg] from comment #2)
> Suggestion for implementation: Keep the current version exactly as is, and
> point the client at a new URL e.g. https://fhr.cdn.mozilla.net/v4/%LOCALE%/

I'm waiting for feedback on this in bug 1150491.
We also need to make sure that either the new content is up under that URL or at least the old one duplicated.

Comment 1

[Georg Fritzsche [:gfritzsche]]

Attachment: Part 1 - Use a different URL pref for healthreport when Unified Telemetry is on and opt-out

Introducing a new pref allows us to keep manual testing and other use-cases sane. We can remove this and only use one again after we shipped unified Telemetry.

Comment 2

[Georg Fritzsche [:gfritzsche]]

Attachment: Part 2 - Change the URL for the unified healthreport content to point to the v4 version

This actually flips the pref. We cant land this until the current remote content is duplicated under that URL.

Comment 3

[Katie Parlante]

Simple copying of files to v4 directory in fhr-jelly: https://github.com/mozilla/fhr-jelly/pull/171

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/0557c248da59

Comment 5

[Georg Fritzsche [:gfritzsche]]

Comment on attachment 8640510 [details] [diff] [review]
Part 1 - Use a different URL pref for healthreport when Unified Telemetry is on and opt-out

Approval Request Comment
[Feature/regressing bug #]:
Telemetry Unification
[User impact if declined]:
This is a shipping requirement for Unified Telemetry. This adds a pref to override the URL we load the about:healthreport content from.
This patch doesn't change it yet, this happens in part 2 here which we will not land until the content is duplicated properly under the new URL.
It is part of a mostly fixup & diagnostic uplift batch for 41: http://bit.ly/1LYhA16
[Describe test coverage new/current, TreeHerder]:
We have automated test-coverage, have manually verified behavior, will do so again before landing on Aurora.
[Risks and why]:
Low-risk - this is a minimal change just adding an overriding URL pref. I'm not expecting any surprises here.
[String/UUID change made/needed]:
None.

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/0557c248da59

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/18299a261f36

Comment 8

[Georg Fritzsche [:gfritzsche]]

Comment on attachment 8640511 [details] [diff] [review]
Part 2 - Change the URL for the unified healthreport content to point to the v4 version

Approval Request Comment
[Feature/regressing bug #]:
Telemetry Unification
[User impact if declined]:
This is a shipping requirement for Unified Telemetry.
This part just changes the pref to point to the new v4 version for those users who have unified Telemetry on.
It is part of a mostly fixup & diagnostic uplift batch for 41: http://bit.ly/1LYhA16
[Describe test coverage new/current, TreeHerder]:
We have automated test-coverage, have manually verified behavior, will do so again before landing on Aurora.
[Risks and why]:
Low-risk - this just changes the pref to point to the new URL. I'm not expecting any surprises here and tested this manually before landing.
[String/UUID change made/needed]:
None.

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/18299a261f36

Comment 10

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8640510 [details] [diff] [review]
Part 1 - Use a different URL pref for healthreport when Unified Telemetry is on and opt-out

[Triage Comment]

We need these as UT may be opt-in for FF41. Let's uplift to m-b.

Comment 11

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8640511 [details] [diff] [review]
Part 2 - Change the URL for the unified healthreport content to point to the v4 version

[Triage Comment]
Beta+

Comment 12

[Georg Fritzsche [:gfritzsche]]

https://hg.mozilla.org/releases/mozilla-beta/rev/85aa9f19ebd7
https://hg.mozilla.org/releases/mozilla-beta/rev/7a23c634f65e

Comment 13

[Ada [:adalucinet]]

Verified on 41.0b8 that 'datareporting.healthreport.about.reportUrlUnified' pref has the proper https://fhr.cdn.mozilla.net/%LOCALE%/v4/ value in about:config and about:healthreport page is correctly displayed. Is it enough to call this fix verified or something else needs to be checked here? Thanks in advance.

Comment 14

[Georg Fritzsche [:gfritzsche]]

Yes, thank you.

Comment 15

[Ada [:adalucinet]]

Verified fixed with 42.0b1 (Build ID: 20150921151815), across platforms [1].

[1] Windows 7 64-bit, Mac OS X 10.11 Beta and Ubuntu 14.04 32-bit