Give TPS a configurable option to point at a non-production Sync server

VERIFIED FIXED in Firefox 51

Status

Cloud Services
Firefox Sync: Backend
P2
normal
VERIFIED FIXED
a year ago
a year ago

People

(Reporter: kthiessen, Assigned: tcsc)

Tracking

unspecified
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

a year ago
This is necessary for bug 1219456 -- the intensive TPS job should run somewhere where it won't interfere with production services.

Thom and Mark, please feel free to email or comment with any questions/caveats, etc.

Thanks.

Updated

a year ago
Priority: -- → P2
Comment hidden (mozreview-request)
(Assignee)

Comment 2

a year ago
Ended up doing this sooner rather than later so that I could test my work on bug 1253051. Basically, just looks for stuff like `"preferences": { "foo.bar.baz": "asdf", ... }` and merges them into the preferences it would start Firefox with. This definitely allows syncing with stage if you specify all the right prefs (this is a bit unwieldy, admittedly).

Comment 3

a year ago
mozreview-review
Comment on attachment 8780189 [details]
Bug 1293426 - Allow TPS config to specify preference values so it can be pointed at non-production servers

https://reviewboard.mozilla.org/r/70936/#review68584

LGTM, thanks. Can you please update the bug with instructions for Karl to follow on how to specify the prefs that point this at stage (ie, the specific values he should add, and where he should specify them)
Attachment #8780189 - Flags: review?(markh) → review+
(Assignee)

Comment 4

a year ago
Karl: https://pastebin.mozilla.org/8891597 is my TPS config (sans user/pass). I believe only lines 17 to 28 are relevant but it seemed better to paste the whole thing. This is the file located at `<tps-venv-root>/config.json`.  This probably goes without saying, but you will need to register the account on stage for this to work.

Some of these values are probably superfluous (For example, I can't imagine TPS needs the signup URI to work), but they don't do any harm and it was easier for me to specify them all than to find the relevant subset.

Let me know if you have any issues.
Keywords: checkin-needed

Comment 5

a year ago
Pushed by kwierso@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/2c7b15b477ed
Allow TPS config to specify preference values so it can be pointed at non-production servers r=markh
Keywords: checkin-needed
Hey Karl,
  Please see comment 4 - can you please re-enable TPS on stage?
Flags: needinfo?(kthiessen)
(Reporter)

Comment 7

a year ago
I will set up a job for Stage sometime today PDT.
Flags: needinfo?(kthiessen)

Comment 8

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/2c7b15b477ed
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
(Reporter)

Comment 9

a year ago
This got shuffled back behind some high-priority options; I'll give it another stab this week.
(Reporter)

Comment 10

a year ago
This is verified; Thom is using it regularly.  And as soon as :rpapa and I figure out where to put it, we'll get it running on a schedule.  See bug 1219456 for details.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.