Closed Bug 1293426 Opened 5 years ago Closed 5 years ago

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

Categories

(Firefox :: Sync, defect, P2)

defect

Tracking

()

VERIFIED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: kthiessen, Assigned: tcsc)

References

Details

Attachments

(1 file)

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.
Priority: -- → P2
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 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+
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
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)
I will set up a job for Stage sometime today PDT.
Flags: needinfo?(kthiessen)
https://hg.mozilla.org/mozilla-central/rev/2c7b15b477ed
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
This got shuffled back behind some high-priority options; I'll give it another stab this week.
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
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.