Georg Fritzsche [:gfritzsche] Assignee
	Description • 9 years ago

(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.

	Georg Fritzsche [:gfritzsche] Assignee
	Comment 1 • 9 years ago

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.

	Georg Fritzsche [:gfritzsche] Assignee
	Comment 2 • 9 years ago

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

	Katie Parlante
	Comment 3 • 9 years ago

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

	Pulsebot
	Comment 4 • 9 years ago

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

	Georg Fritzsche [:gfritzsche] Assignee
	Comment 5 • 9 years ago

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.

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 6 • 9 years ago

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

	Pulsebot
	Comment 7 • 9 years ago

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

	Georg Fritzsche [:gfritzsche] Assignee
	Comment 8 • 9 years ago

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.

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 9 • 9 years ago

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

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 10 • 9 years ago

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.

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 11 • 9 years ago

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+

	Georg Fritzsche [:gfritzsche] Assignee
	Comment 12 • 9 years ago

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

	Ada [:adalucinet]
	Comment 13 • 9 years ago

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.

	Georg Fritzsche [:gfritzsche] Assignee
	Comment 14 • 9 years ago

Yes, thank you.

	Ada [:adalucinet]
	Comment 15 • 9 years ago

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