1996952 - Bake the M4 experiment Merino OHTTP endpoints into Nightly

Status: RESOLVED FIXED

(Firefox :: Address Bar, task, P1)

Description

[Nan Jiang [:nanj]]

We have finalized various endpoints for the Fastly OHTTP Relay and our OHTTP Gateway:

• Relay endpoint:

  • Pref: "browser.urlbar.merino.ohttpRelayURL"
  • stage: https://ohttp-merino-stage.mozilla.fastly-edge.com
  • prod: https://ohttp-merino.mozilla.fastly-edge.com

• OHTTP config endpoint:

  • Pref: "browser.urlbar.merino.ohttpConfigURL"
  • prod: https://merino.services.mozilla.com/api/v1/suggest/ohttp-configs

• The Merino suggest endpoint remained unchanged

  • Pref: "browser.urlbar.merino.endpointURL"
  • prod: https://merino.services.mozilla.com/api/v1/suggest

The production endpoints can be baked into m-c as the default values for Merino's OHTTP prefs now.

Note:

• The OHTTP config prod endpoint will be available soon once we migrate Merino from v1 to v2.
• The Nimbus variables for those prefs can be kept to facilitate future rollouts & experimentation.

Comment 1

[Drew Willcoxon :adw]

Thanks Nan! To add more in case it's not obvious to readers of this bug -- and please correct me if I'm misunderstanding:

We'll want to make a client code change for this only after 146, in whichever release where we're ready to roll out OHTTP to all online users. Until then, we can use the relevant Nimbus variables (as Nan said).

Comment 2

[Drew Willcoxon :adw]

I'll move this to bug 1996166 so it's clearer that this is not a requirement for the initial client changes in 146.

Comment 3

[Drew Willcoxon :adw]

Attachment: Bug 1996952 - Bake the M4 experiment Merino OHTTP endpoints into Nightly.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/74cdb92a9ce4
https://hg.mozilla.org/integration/autoland/rev/5e313f997037
Bake the M4 experiment Merino OHTTP endpoints into Nightly. r=daisuke,nanj,Sasha

Comment 5

[Drew Willcoxon :adw]

We ended up landing the M4 ohttp endpoints right now, and for Nightly only, so I'll resummarize this bug. I probably should have attached the patch to a new bug and kept this one open, oh well.

From https://phabricator.services.mozilla.com/D281320#9778620:

Offline discussion with Nan: Infra isn't ready for Nightly users to hit the final ohttp endpoints quite yet, but to unblock this patch and D281321, we can use the M4 experiment endpoints temporarily. We want to make sure those endpoints don't ride the trains; they should remain Nightly only. So, this diff has some #ifdef NIGHTLY_BUILD guards.

I'll file a follow-up for updating the endpoints to the final URLs.

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/5e313f997037