1058759 - Change default URL of about:home snippets service

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Osmose [:osmose, :mkelly]]

Currently, the about:config setting browser.aboutHomeSnippets.updateUrl has the hardcoded domain 'https://snippets.mozilla.com' in it. We'd like to update this to point to 'https://snippets.cdn.mozilla.net' instead, so users are requesting snippets from our CDN.

Comment 1

[Abhishek Potnis [:avp]]

Attachment: Changed default URL of about:home snippets service to Mozilla's CDN

Comment 2

[Osmose [:osmose, :mkelly]]

Comment on attachment 8486295 [details] [diff] [review]
Changed default URL of about:home snippets service to Mozilla's CDN

Review of attachment 8486295 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me, r+ :D

Comment 3

[:Gavin Sharp [email: [e-mail]]]

Is this ready to be checked in?

Comment 4

[Osmose [:osmose, :mkelly]]

Yep, this is good to go!

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/fx-team/rev/347d06afb237

Comment 6

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

https://hg.mozilla.org/mozilla-central/rev/347d06afb237

Comment 7

[Chris More [:cmore]]

Gavin: Is this change a candidate for uplifting?

Comment 8

[:Gavin Sharp [email: [e-mail]]]

Sure. Where do you want it?

Comment 9

[Chris More [:cmore]]

(In reply to :Gavin Sharp [email: [e-mail]] from comment #8)
> Sure. Where do you want it?

I assume you mean when. :)

We would like to get these snippet performance issues wrapped up as we are holding from doing more advanced snippets given the performance caused by static assets. 

What's your timeline on getting it up on beta?

Comment 10

[:Gavin Sharp [email: [e-mail]]]

I meant "which channel do you want it uplifted to". I can uplift it this week.

Comment 11

[Osmose [:osmose, :mkelly]]

Please hold off on uplifting this off of nightly for at least a week, and after that, let's uplift to Aurora at least for another week. I don't want to just immediately uplift this to beta without at least a bit of time to see how the CDN affects load on the snippets service.

Comment 12

[Osmose [:osmose, :mkelly]]

I guess the better question is: Is "spend a minimum of one week on Nightly, then uplift to Aurora once we've had a chance to look at the stats over that week, and then repeat that process prior to uplifting to Beta" a reasonable/possible plan?

Comment 13

[:Gavin Sharp [email: [e-mail]]]

Sure, whatever works best for you. We have a daily update cycle on Nightly/Aurora (though it can take roughly a week to migrate most users to the latest version), and new beta builds go out twice weekly (with a similar update lag). We don't want to uplift to beta too late in the cycle, and we're pretty much at the midpoint now, so if you want to hit the 33 release there's not much time for testing on beta left.

Comment 14

[Chris More [:cmore]]

(In reply to :Gavin Sharp [email: [e-mail]] from comment #13)
> Sure, whatever works best for you. We have a daily update cycle on
> Nightly/Aurora (though it can take roughly a week to migrate most users to
> the latest version), and new beta builds go out twice weekly (with a similar
> update lag). We don't want to uplift to beta too late in the cycle, and
> we're pretty much at the midpoint now, so if you want to hit the 33 release
> there's not much time for testing on beta left.

Let's give it some time on nightly and jump over to Aurora to get some more data. It is probably likely we'll pass the window for beta and will have to wait until the next release.

Comment 15

[Osmose [:osmose, :mkelly]]

Cool, I'll update this bug when we're ready to uplift. Thanks!

Comment 16

[Robert Kaiser]

I don't think the QA team needs to spend time verifying this manually, as you look at data anyhow to tell if the pref change works and what it brings us.

Comment 17

[Osmose [:osmose, :mkelly]]

After some testing of the effects, we've found the CDN won't solve some of our immediate problems, but still has benefits in it's use. It's good to ride the trains normally, but unless something changes (or I'm wrong), we won't be asking for this to be uplifted. Thanks!